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

Support pulling plugins by digest #3803

Merged
merged 1 commit into from May 18, 2021

Conversation

2uasimojo
Copy link
Contributor

@2uasimojo 2uasimojo commented May 14, 2021

Previously WithPlugins only supported passing image URIs "by tag" -- e.g. gcr.io/my-repo/my-image:v0.1.2. With this commit, we add support for pulling "by digest" -- e.g. gcr.io/my-repo/my-image@sha256:a75f9e8c3ced3943515f249597be389f8233e1258d289b11184796edceaa7dab

Signed-off-by: Eric Fried efried@redhat.com

Thank you for contributing to Velero!

Please add a summary of your change

Previously WithPlugins only supported passing image URIs "by tag" -- e.g. gcr.io/my-repo/my-image:v0.1.2. With this commit, we add support for pulling "by digest" -- e.g. gcr.io/my-repo/my-image@sha256:a75f9e8c3ced3943515f249597be389f8233e1258d289b11184796edceaa7dab

Does your change fix a particular issue?

Using velero under OpenShift with ImageContentSourcePolicy it is necessary to reference images by digest rather than by tag.

Fixes #(issue)

Please indicate you've done the following:

  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Created a changelog file or added /kind changelog-not-required.
  • Updated the corresponding documentation in site/content/docs/main.
    (n/a, I think)

Previously `WithPlugins` only supported passing image URIs "by tag" --
e.g. `gcr.io/my-repo/my-image:v0.1.2`. With this commit, we add support
for pulling "by digest" -- e.g.
`gcr.io/my-repo/my-image@sha256:a75f9e8c3ced3943515f249597be389f8233e1258d289b11184796edceaa7dab`

Signed-off-by: Eric Fried <efried@redhat.com>
@2uasimojo
Copy link
Contributor Author

/retest

Copy link
Contributor

@zubron zubron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@jenting jenting merged commit 7566962 into vmware-tanzu:main May 18, 2021
@2uasimojo 2uasimojo deleted the plugins-by-digest branch May 18, 2021 14:56
ywk253100 pushed a commit to ywk253100/velero that referenced this pull request Jun 29, 2021
Previously `WithPlugins` only supported passing image URIs "by tag" --
e.g. `gcr.io/my-repo/my-image:v0.1.2`. With this commit, we add support
for pulling "by digest" -- e.g.
`gcr.io/my-repo/my-image@sha256:a75f9e8c3ced3943515f249597be389f8233e1258d289b11184796edceaa7dab`

Signed-off-by: Eric Fried <efried@redhat.com>
gyaozhou pushed a commit to gyaozhou/velero-read that referenced this pull request May 14, 2022
Previously `WithPlugins` only supported passing image URIs "by tag" --
e.g. `gcr.io/my-repo/my-image:v0.1.2`. With this commit, we add support
for pulling "by digest" -- e.g.
`gcr.io/my-repo/my-image@sha256:a75f9e8c3ced3943515f249597be389f8233e1258d289b11184796edceaa7dab`

Signed-off-by: Eric Fried <efried@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants