-
Notifications
You must be signed in to change notification settings - Fork 200
add artifact attestation #505
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
base: main
Are you sure you want to change the base?
Conversation
4b0013b to
a225727
Compare
Signed-off-by: drfaust92 <[email protected]>
a225727 to
4888461
Compare
Signed-off-by: Ilia Lazebnik <[email protected]>
|
Seems like we have an issue with unittest -> helm-unittest/helm-unittest#777. Let's follow up for some days and see if it gets solved. |
|
GMartinez-Sisti can we pin it to helm 3 meanwhile? |
Since the unittest repo is fairly active, I'd rather wait another week or so. This is a nice extra but AFAIK no one is blocked on this. |
|
@GMartinez-Sisti seems that many did this https://github.com/FlowFuse/helm/pull/747/files. wdyt? |
Decreasing the security of the CI tests to provide artifact attestation feels a bit contradictory 😅 Atlantis is an application with the highest level of permissions given its nature. I think it's safer to wait. We should be able to merge this soon! Thank you for all the help 🙏 |
closes #409
tested locally from fork: