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

Add signing metadata during push to API #193

Closed
aweiteka opened this issue Sep 7, 2016 · 6 comments
Closed

Add signing metadata during push to API #193

aweiteka opened this issue Sep 7, 2016 · 6 comments

Comments

@aweiteka
Copy link
Contributor

aweiteka commented Sep 7, 2016

The OpenShift API ImageSignature object provides storage for useful metadata. Skopeo should write this data if possible.

  • imageIdentity: <image_reference>
  • created:
  • issuedBy:
  • issuedTo: # what's the difference?

To clarify to the end user that this information is not verified, populate the SignatureCondition object with the following fields:

  • type: "Complete"
  • status: "Unknown"
  • lastProbeTime:
  • reason: "Initial signing request"
  • message: <???>
@aweiteka
Copy link
Contributor Author

aweiteka commented Sep 7, 2016

cc @stefwalter

@mtrmac
Copy link
Contributor

mtrmac commented Sep 7, 2016

AFAIK OpenShift explicitly prohibits updating these fields, per https://github.com/openshift/origin/pull/9181/files#diff-338f8de5c2e444ebed64ed7ac61109e4R111 and https://github.com/openshift/origin/pull/8371/files#diff-338f8de5c2e444ebed64ed7ac61109e4R101 . So at the moment there is nothing for skopeo to do. (The original design was that OpenShift would populate these fields from the received blob. We can of course change/renegotiate that. Just clarifying that this won’t happen this week.)

(

  • issuedBy is ~nonsense (it says which CA signed the key)
  • issuedTo = the key owner

)

@aweiteka
Copy link
Contributor Author

aweiteka commented Sep 7, 2016

The original design was that OpenShift would populate these fields from the received blob

That's disappointing. The data isn't in the blob! So we cannot POST any arbitrarily data to the ImageSignature object?

@mtrmac
Copy link
Contributor

mtrmac commented Sep 7, 2016

imageIdentity is in the blob directly; for issuedTo the blob gives us the fingerprint but not the real name. issuedBy doesn’t have clear semantics with GPG, and is not in the blob. The verification status data of course isn’t in the blob.

@mtrmac
Copy link
Contributor

mtrmac commented May 10, 2017

@aweiteka Given openshift/origin#13585 , is (some variant of) this still necessary?

@aweiteka
Copy link
Contributor Author

Given openshift/origin#13585 , is (some variant of) this still necessary?

No. Closing.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants