-
Notifications
You must be signed in to change notification settings - Fork 3
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
Removing the auto upload of support bundle code #378
Removing the auto upload of support bundle code #378
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why comment these changes out? Why not just remove the code entirely?
Can you update the tests too? We actually have tests that test upload to s3 and they are failing now :-) https://github.com/platform9/pf9ctl/actions/runs/10034316801/job/27728661832?pr=378 |
@vedantjoshi84 , can you merge this, update/ release pf9ctl and use the new pf9ctl in SMCP 5.10.1 (patch for Sambanova? |
@vedantjoshi84 #377 will solve the 2 open security issues flagged by dependabot. |
ISSUE(S):
PMK-6512
SUMMARY
The s3 bucket for the support bundle upload is disabled because of #incident351, so removing the auto upload of the generated support bundle to s3.
ISSUE TYPE
IMPACTED FEATURES/COMPONENTS:
pmk, pf9ctl
RELATED ISSUE(S):
DEPENDS ON:
TESTING DONE
Automated
Manual
Reviewers