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

Removing the auto upload of support bundle code #378

Merged
merged 5 commits into from
Jul 25, 2024

Conversation

vedantjoshi84
Copy link
Contributor

@vedantjoshi84 vedantjoshi84 commented Jul 19, 2024

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

  • Bug fix (non-breaking change which fixes an issue)
  • This change requires a documentation update

IMPACTED FEATURES/COMPONENTS:

pmk, pf9ctl

RELATED ISSUE(S):

DEPENDS ON:

TESTING DONE

Automated

Manual

Reviewers

Copy link
Member

@mithilarun mithilarun left a 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?

@cruizen
Copy link
Contributor

cruizen commented Jul 22, 2024

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

mithilarun
mithilarun previously approved these changes Jul 22, 2024
@vedantjoshi84 vedantjoshi84 changed the title Commenting out the auto upload of support bundle Removing the auto upload of support bundle code Jul 22, 2024
@cruizen
Copy link
Contributor

cruizen commented Jul 24, 2024

@vedantjoshi84 , can you merge this, update/ release pf9ctl and use the new pf9ctl in SMCP 5.10.1 (patch for Sambanova?

@cruizen
Copy link
Contributor

cruizen commented Jul 24, 2024

@vedantjoshi84 #377 will solve the 2 open security issues flagged by dependabot.

@vedantjoshi84 vedantjoshi84 merged commit 82c0f0c into master Jul 25, 2024
4 checks passed
@vedantjoshi84 vedantjoshi84 deleted the private/vjoshi/master/disable-auto-upload branch July 25, 2024 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants