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

[backport v1.3] [FEATURE] support bundle timeout enhancement #5839

Closed
harvesterhci-io-github-bot opened this issue May 21, 2024 · 1 comment
Assignees
Labels
area/support-bundle-kit kind/enhancement Issues that improve or augment existing functionality not-require/test-plan Skip to create a e2e automation test issue priority/2 Nice to fix in this release
Milestone

Comments

@harvesterhci-io-github-bot

backport the issue #1646

@harvesterhci-io-github-bot harvesterhci-io-github-bot added area/support-bundle-kit kind/enhancement Issues that improve or augment existing functionality not-require/test-plan Skip to create a e2e automation test issue priority/2 Nice to fix in this release labels May 21, 2024
@harvesterhci-io-github-bot harvesterhci-io-github-bot added this to the v1.3.2 milestone May 21, 2024
@harvesterhci-io-github-bot
Copy link
Author

harvesterhci-io-github-bot commented May 30, 2024

Pre Ready-For-Testing Checklist

  • If labeled: require/HEP Has the Harvester Enhancement Proposal PR submitted?
    The HEP PR is at:

  • Where is the reproduce steps/test steps documented?
    The reproduce steps/test steps are at:

Prepare:

  1. Use new harvester with new support bundle kit image
  2. Define settings > support-bundle-node-collection-timeout (unit is minute)
  3. Create support bundle
  4. Check env fields of support bundle manager deployment.apps
    image

Two different cases:
Case A: define large and enough minutes, it should get nodes folder successfully.
Case B: define a small minute, like "1". This one depends how long your environment takes. Just define a very small minute to make timeout happen. Although It won't get nodes folder, but It will get the support bundle file zip.

  • Is there a workaround for the issue? If so, where is it documented?
    The workaround is at:

  • Have the backend code been merged (harvester, harvester-installer, etc) (including backport-needed/*)?
    The PR is at:

  • If NOT labeled: not-require/test-plan Has the e2e test plan been merged? Have QAs agreed on the automation test case? If only test case skeleton w/o implementation, have you created an implementation issue?

    • The automation skeleton PR is at:
    • The automation test case PR is at:
  • If the fix introduces the code for backward compatibility Has a separate issue been filed with the label release/obsolete-compatibility?
    The compatibility issue is filed at:

@Yu-Jack Yu-Jack closed this as not planned Won't fix, can't repro, duplicate, stale May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/support-bundle-kit kind/enhancement Issues that improve or augment existing functionality not-require/test-plan Skip to create a e2e automation test issue priority/2 Nice to fix in this release
Projects
None yet
Development

No branches or pull requests

2 participants