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

Image syncer amd64Only flag doesn’t prevent syncing image index #11712

Open
Sawthis opened this issue Aug 28, 2024 · 2 comments
Open

Image syncer amd64Only flag doesn’t prevent syncing image index #11712

Sawthis opened this issue Aug 28, 2024 · 2 comments
Labels
area/ci Issues or PRs related to CI related topics kind/bug Categorizes issue or PR as related to a bug.

Comments

@Sawthis
Copy link
Contributor

Sawthis commented Aug 28, 2024

Description

An amd64Only flag doesn’t prevent syncing image index and most probably single images for other architecture.
Example, node image identified by digest is in fact an image index, as such was synced with our registry.

- source: "node@sha256:a3816e038e05ea70d2640c845c285f49e416bdae2481a7ff94fde96647a10607"
tag: "22.3.0"
amd64Only: true

https://console.cloud.google.com/artifacts/docker/kyma-project/europe/prod/external%2Flib[…]b=manifest?project=kyma-project&supportedpurview=project
In reality, this flag enables allowing syncing an image manifest if it’s defined as source. There is no checks about architecture.

I believe this flag suppose to implement:

Goal is to extend the current image cloner to support also Mac M1 architectures (please make it configurable which archs it has to support).

From #6388

Possible solution:

  • We can change a flag name to indicate what it does in reality.
  • Change the tool to really sync only amd64.
  • Delete the flag as it doesn't do what it should.

Reasons

Acceptance Criteria

Attachments

@Sawthis Sawthis added area/ci Issues or PRs related to CI related topics kind/bug Categorizes issue or PR as related to a bug. labels Aug 28, 2024
@Sawthis
Copy link
Contributor Author

Sawthis commented Sep 12, 2024

@dekiel I think we should change the flag name to something like: allowSyncImageManifest.

@Sawthis
Copy link
Contributor Author

Sawthis commented Sep 18, 2024

Decision from Kyma PO meeting: Flag can be deleted / changed.

CC: @dekiel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci Issues or PRs related to CI related topics kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

2 participants
@Sawthis and others