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

key revisions are now calculated from existing public AND secret key files #399

Merged
merged 1 commit into from
Apr 20, 2016

Conversation

bookshelfdave
Copy link
Contributor

This bug forced an artifact signer to have both the secret .sig.key as well as the .pub key in the key cache.

@bookshelfdave bookshelfdave changed the title key revisions are now calculated from existing public AND secret key files key revisions are now calculated from existing public OR secret key files Apr 18, 2016
@bookshelfdave bookshelfdave changed the title key revisions are now calculated from existing public OR secret key files key revisions are now calculated from existing public AND secret key files Apr 18, 2016
@chef-delivery
Copy link
Contributor

This PR has passed 'Verify' and is ready for review and approval!
Use: '@delivery approve' when code review is complete.

@fnichol
Copy link
Collaborator

fnichol commented Apr 20, 2016

Nice, will be even more convenient for Plan builds!

gif-keyboard-10598023118860026018

@fnichol
Copy link
Collaborator

fnichol commented Apr 20, 2016

@delivery approve

@chef-delivery chef-delivery merged commit 4055fdc into master Apr 20, 2016
@chef-delivery
Copy link
Contributor

Change: 39bf3cbf-f365-4724-ab0f-782c95394e70 approved by: @fnichol

@chef-delivery chef-delivery deleted the dp_key_revision_bug branch April 20, 2016 18:16
raskchanky pushed a commit that referenced this pull request Apr 16, 2019
Merged change 39bf3cbf-f365-4724-ab0f-782c95394e70

From review branch dp_key_revision_bug into master

Signed-off-by: fnichol <[email protected]>
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

Successfully merging this pull request may close these issues.

3 participants