-
Notifications
You must be signed in to change notification settings - Fork 642
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
Using CAS for content-trust #3382
Conversation
@pvizeli ok so Notice that message at the top. That's what's breaking it because the response isn't JSON with that there. It happens the first time you run Think we can just ship the image with this key? I assume based on this that it's stable. That would eliminate this from the response. As for the other errors I'm not entirely sure what to make of them. Neither seems like they could be related to this change. The one on the Build Supervisor workflow actually sounds like a bug with the action itself based on the output. Found this issue but I don't see a force push in the history of this PR so not sure how that would've happened. |
Yeah, we should ship the image with preinstalled key on dockerfile + install the key:
Then it should be solved on our workflows as well |
Yeah, push the file into is even better over the rootfs, nice catch |
Proposed change
Type of change
Additional information
Checklist
black --fast supervisor tests
)If API endpoints of add-on configuration are added/changed: