-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Use new secret store API #15723
Use new secret store API #15723
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ashley-cui The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
5713ac8
to
c97f155
Compare
Refactored secrets API in common for stability purposes. Move podman to said API. [NO NEW TESTS NEEDED] Signed-off-by: Ashley Cui <[email protected]>
c97f155
to
89c893e
Compare
Ready to look at! This should unblock vendoring, as it vendors common@main and deals with the breaking change in the secrets API. |
Event tests aren't happy. @Luap99 probably you need to cherry-pick this commit here or the other way around. |
I cherry-picked it into #15717. |
Thanks! Let's close this PR there then. |
Refactored secrets API in common for stability purposes. Move podman to
said API.
Does this PR introduce a user-facing change?