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

Keep original tag and only append sha256 digest #309

Open
nuriel77 opened this issue Dec 21, 2022 · 7 comments
Open

Keep original tag and only append sha256 digest #309

nuriel77 opened this issue Dec 21, 2022 · 7 comments
Labels
carvel accepted This issue should be considered for future work and that the triage process has been completed enhancement This issue is a feature request

Comments

@nuriel77
Copy link

Describe the problem/challenge you have
[A description of the current challenge that you are experiencing.]
Sorry if I have missed this while checking the documentation, but is it possible to maintain the original tag and simply append the digest? The effect would be the same, while keeping the image's original version tag readable.

Describe the solution you'd like
[A clear and concise description of what you want to happen. If applicable a visual representation of the UX (ex: new CLI argument name, the behavior expected).]

For example:
nginx:1.14.2

Would become:

index.docker.io/library/nginx:1.14.2@sha256:f7988fb6c02e0ce69257d9bd9cf37ae20a60f1df7563c3a2a6abe24160306b8d

Anything else you would like to add:
[Additional information that will assist in solving the issue.]


Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.

@nuriel77 nuriel77 added carvel triage This issue has not yet been reviewed for validity enhancement This issue is a feature request labels Dec 21, 2022
@carvel-bot carvel-bot moved this to To Triage in Carvel Dec 21, 2022
@joaopapereira
Copy link
Member

Hey @nuriel77, thanks for opening up this issue.
To the best of my knowledge, there is no way to keep the tag after we resolve the SHA.

Do you mind explaining to us a little more about your use case for this?

@nuriel77
Copy link
Author

@joaopapereira Hi!

Yes, so basically when you keep the tag it is easier to see which version of the image is being used rather than just having the SHA alone.

index.docker.io/library/nginx:1.14.2@sha256:f7988fb6c02e0ce69257d9bd9cf37ae20a60f1df7563c3a2a6abe24160306b8d

This is useful when checking pods and their deployed images to infer the version.

@joostvdg
Copy link

What @nuriel77 said.

In addition, there are the following arguments:

  • versioning of the application happens via setting Git Tags. I want to correlate the Tag in Git with the Docker Tag
  • automated cleanup processes of Tags with specific formats, e.g., x.y.z-rc*

@joaopapereira
Copy link
Member

Hey
Sorry for not replying earlier.

Is this something that yll would want to happen by default or only done when a configuration is done?

If we decide to do this by default this might cause issues to people that are currently using kbld because the output YAML would be different between versions(To be fair I do not think this is a huge issue but would like to state it out loud to see what yll think).
At the same time if we decide that this should be configurable we would have to add maybe a new key in the overrides section of the config something like

apiVersion: kbld.k14s.io/v1alpha1
kind: Config
overrides:
- image: nginx:1.2.3
  keepTag: true

What do yll think?

@nuriel77
Copy link
Author

Valid concern.
I think that adding the keepTag makes sense when introducing this new change. Maybe somewhere down the road it can become the default.

@joostvdg
Copy link

Agreed, looks good enough for me.

@joaopapereira
Copy link
Member

Cool, going to accept this feature.
Just to set expectations here, the maintainer team is a little busy with other work which might cause the development of this tool to take some time. Nevertheless, we are more than happy to review contributions or even help with the implementation of the feature.

@joaopapereira joaopapereira added carvel accepted This issue should be considered for future work and that the triage process has been completed and removed carvel triage This issue has not yet been reviewed for validity labels Jan 27, 2023
@github-project-automation github-project-automation bot moved this to To Triage in Carvel Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
carvel accepted This issue should be considered for future work and that the triage process has been completed enhancement This issue is a feature request
Projects
Status: To Triage
Development

No branches or pull requests

3 participants