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

Align runtime-watcher image names #1859

Closed
5 tasks done
c-pius opened this issue Sep 11, 2024 · 2 comments · Fixed by #1868, #1884 or kyma-project/runtime-watcher#390
Closed
5 tasks done

Align runtime-watcher image names #1859

c-pius opened this issue Sep 11, 2024 · 2 comments · Fixed by #1868, #1884 or kyma-project/runtime-watcher#390
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@c-pius
Copy link
Contributor

c-pius commented Sep 11, 2024

Description

We use different image names for runtime-watcher in Dev and Prod registries:

const (
watcherRegProd = "europe-docker.pkg.dev/kyma-project/prod/runtime-watcher-skr"
watcherRegDev = "europe-docker.pkg.dev/kyma-project/dev/runtime-watcher"
)

Recently, when adapting build pipelines, we confused things up already. We should align the name and use the same for both registries.

Reasons

Consistency and less prone to mistakes

Acceptance Criteria

  • agreed on a common name => runtime-watcher (NO -skr suffix)
  • updated pipelines to build the image using the common name
  • updated code to use the image using the common name
  • verify that it works e2e without -skr suffix
  • EDIT CS: consider to update management plane charts to provide the registry value explicitly (as of now it would take the default)

Feature Testing

No response

Testing approach

No response

Attachments

No response

@c-pius c-pius added the kind/feature Categorizes issue or PR as related to a new feature. label Sep 11, 2024
@c-pius
Copy link
Contributor Author

c-pius commented Sep 11, 2024

Prio to be discussed. We may need to have it fixed before the next watcher release.

@c-pius
Copy link
Contributor Author

c-pius commented Sep 13, 2024

updated pipelines to build the image using the common name

watcher is now uniformely built with runtime-watcher name: https://github.com/kyma-project/runtime-watcher/blob/main/.github/workflows/build-image.yml#L41

updated code to use the image using the common name

implementation at KLM side is: #1868

See the comments wrt. to the issues with e2e tests

verify that it works e2e without -skr suffix

runtime watcher also needs to be updated to pass the new patch. PR for this is here: kyma-project/runtime-watcher#385

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