Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix:the chart value tuneSysctlConfig does not work #3988

Merged

Conversation

cyclinder
Copy link
Collaborator

Thanks for contributing!

What type of PR is this?

  • release/bug

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #3987

Special notes for your reviewer:

@cyclinder cyclinder added release/bug cherrypick-release-v0.8 Cherry-pick the PR to branch release-v0.8. labels Sep 2, 2024
@weizhoublue weizhoublue added the cherrypick-main Cherry-pick the PR to branch main. label Sep 2, 2024
@cyclinder cyclinder added the cherrypick-release-v1.0 Cherry-pick the PR to branch release-v1.0. label Sep 3, 2024
@cyclinder cyclinder force-pushed the fix_tuneSysctlConfig_chart branch 2 times, most recently from 4daf186 to 417d135 Compare September 3, 2024 03:04
@weizhoublue weizhoublue changed the title Fix set tuneSysctlConfig did not works in chart Fix:the chart value tuneSysctlConfig does not works Sep 3, 2024
@weizhoublue weizhoublue changed the title Fix:the chart value tuneSysctlConfig does not works Fix:the chart value tuneSysctlConfig does not work Sep 3, 2024
Signed-off-by: cyclinder <qifeng.guo@daocloud.io>
@cyclinder
Copy link
Collaborator Author

• [FAILED] [317.151 seconds]
Chaos Testing of GC [reclaim]
/home/runner/work/spiderpool/spiderpool/test/e2e/reclaim/chaos_test.go:27
  Chaos Testing of GC
  /home/runner/work/spiderpool/spiderpool/test/e2e/reclaim/chaos_test.go:195
    [It] Multiple resource types compete for a single IPPool. In scenarios of creation, scaling up/down, and deletion, GC all can correctly handle IP addresses. [G00012]
    /home/runner/work/spiderpool/spiderpool/test/e2e/reclaim/chaos_test.go:269

  Timeline >>
  > Enter [BeforeEach] Chaos Testing of GC - /home/runner/work/spiderpool/spiderpool/test/e2e/reclaim/chaos_test.go:209 @ 09/03/24 03:48:39.23
  SPIDERPOOL_GC_DEFAULT_INTERVAL_DURATION: 30 
  succeed to create namespace gc-chaos-ns-4618-958381613 
  succeed to create v4 IPPool v4pool-1232c9d26bfb679 
  < Exit [BeforeEach] Chaos Testing of GC - /home/runner/work/spiderpool/spiderpool/test/e2e/reclaim/chaos_test.go:209 @ 09/03/24 03:48:40.266 (1.036s)
  > Enter [It] Multiple resource types compete for a single IPPool. In scenarios of creation, scaling up/down, and deletion, GC all can correctly handle IP addresses. - /home/

@cyclinder
Copy link
Collaborator Author

@cyclinder cyclinder merged commit 4ca5709 into spidernet-io:release-v0.9 Sep 3, 2024
46 of 47 checks passed
@cyclinder cyclinder deleted the fix_tuneSysctlConfig_chart branch September 3, 2024 05:49
github-actions bot pushed a commit that referenced this pull request Sep 3, 2024
Fix:the chart value tuneSysctlConfig does not work

Signed-off-by: robot <tao.yang@daocloud.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherrypick-main Cherry-pick the PR to branch main. cherrypick-release-v0.8 Cherry-pick the PR to branch release-v0.8. cherrypick-release-v1.0 Cherry-pick the PR to branch release-v1.0. release/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants