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

Disable SRP action from Kubeapps #7064

Merged
merged 2 commits into from
Nov 10, 2023
Merged

Conversation

ppbaena
Copy link
Collaborator

@ppbaena ppbaena commented Nov 10, 2023

Description of the change

Due announcement that starting on Nov. 13 (Monday), calls to the 2.5 APIs will return a 400 error message. This means that the SRPCLI commands to submit provenance 2.5 will begin to fail, which in turn may fail your build.
The implementation of calls to v3 directly from Bitnami makes unnecessary to double-check in Kubeapps development pipelines.

Benefits

SRP process is centralized in the official assets for Kubeapps and not necessary in the development ones.

Possible drawbacks

N/A

Copy link

netlify bot commented Nov 10, 2023

Deploy Preview for kubeapps-dev failed.

Built without sensitive environment variables

Name Link
🔨 Latest commit 9efb59f
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/654e0b4851ed3e0008188006

Comment on lines -604 to -606
SRP_CLI_VERSION: ${{env.SRP_CLI_VERSION}}
SRP_CLIENT_ID: ${{secrets.SRP_CLIENT_ID}}
SRP_CLIENT_SECRET: ${{secrets.SRP_CLIENT_SECRET}}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should delete these vars (SRP_CLI_VERSION, SRP_CLIENT_ID and SRP_CLIENT_SECRET) from other places as well?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Copy link
Contributor

@antgamdia antgamdia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!!

@ppbaena ppbaena merged commit 959c70a into vmware-tanzu:main Nov 10, 2023
35 of 39 checks passed
@ppbaena ppbaena deleted the srp-action-removal branch November 10, 2023 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants