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

AWS blog guide ain't updated accordingly to current situation in a repository #133

Open
vponoikoait opened this issue Jan 22, 2024 · 4 comments
Assignees

Comments

@vponoikoait
Copy link
Contributor

source: #124
AWS blog guide ain't updated accordingly to current situation in a repository, as processing changed

              - key: images
                value: "{{ request.object.spec.[ephemeralContainers, initContainers, containers][].image }}"

Doesn't work anymore, which may result in confusion further for whoever tries to implement it and provide bad user experience
More details: https://aws.amazon.com/blogs/containers/announcing-container-image-signing-with-aws-signer-and-amazon-eks/
From my side, it was assumed initially will work, but created a lot of confusion & time in order to implement working PoC

@vishal-chdhry
Copy link
Contributor

Thanks @vponoikoait for opening these issues, I will fix them as soon as I can

@vishal-chdhry vishal-chdhry self-assigned this Jan 23, 2024
@vishal-chdhry
Copy link
Contributor

@vponoikoait, That blog was published when this service was in early stages and we added a lot of features after that. Unfortunately, some of those required us to break backwards compatibility with 0.x alpha versions.
You can follow this blog: https://nirmata.com/2023/11/20/verifying-images-and-attestations-using-aws-signer-notation-and-kyverno/

We will see if we can get that AWS blog updated

@vponoikoait
Copy link
Contributor Author

@vishal-chdhry would you be kind to include there that current auth requires for kyverno to stay in kyverno namespace and have specific SA name? So nobody would potentially go confused.
Referencing: nirmata/kyverno-notation-verifier#27

@vponoikoait
Copy link
Contributor Author

It would be also a valid mentioning regarding that it's available since Kyverno version 1.10+, when service calls started to be a thing

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

No branches or pull requests

2 participants