Skip to content

Switch GCR -> Artifact-Registry#93

Merged
dimityrmirchev merged 2 commits intogardener:mainfrom
AleksandarSavchev:switch-gcr-artifact-reg
Jan 11, 2024
Merged

Switch GCR -> Artifact-Registry#93
dimityrmirchev merged 2 commits intogardener:mainfrom
AleksandarSavchev:switch-gcr-artifact-reg

Conversation

@AleksandarSavchev
Copy link
Copy Markdown
Member

What this PR does / why we need it:
GCR has been deprecated [0] in favour of Artifact-Registry.

Thus, change push-targets for OCI-Images:

  • europe-docker.pkg.dev/gardener-project/snapshots for snapshots
  • europe-docker.pkg.dev/gardener-project/releases for releases
  • europe-docker.pkg.dev/gardener-project/public for combined view of snapshots + releases

[0]
https://cloud.google.com/artifact-registry/docs/transition/transition-from-gcr

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

Change OCI Image Registry from GCR (`eu.gcr.io/gardener-project`) to Artifact-Registry (`europe-docker.pkg.dev/gardener-project/releases`). Users should update their references.

@AleksandarSavchev AleksandarSavchev requested a review from a team as a code owner January 5, 2024 15:24
@gardener-robot gardener-robot added needs/review size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jan 5, 2024
@gardener-robot
Copy link
Copy Markdown
Contributor

@dimityrmirchev You have pull request review open invite, please check

Comment thread .ci/pipeline_definitions Outdated
publish:
dockerimages:
diki:
image: europe-docker.pkg.dev/gardener-project/snapshots/gardener/diki
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Shouldn't these be pushed to releases? The same is relevant for the diki-ops image

Suggested change
image: europe-docker.pkg.dev/gardener-project/snapshots/gardener/diki
image: europe-docker.pkg.dev/gardener-project/releases/gardener/diki

Copy link
Copy Markdown
Member

@dimityrmirchev dimityrmirchev left a comment

Choose a reason for hiding this comment

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

/lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants