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

fix: Name and tag of race image in docker repository should be different than production one #2179

Merged
merged 3 commits into from
Dec 11, 2024

Conversation

Matovidlo
Copy link
Contributor

@Matovidlo Matovidlo commented Dec 10, 2024

Changes:

  • Name and tag of race image in docker repository should be different than production one.

It does not override production image.
@Matovidlo Matovidlo marked this pull request as ready for review December 10, 2024 15:53
@Matovidlo Matovidlo changed the title fix: Name in ECR repository and tag should be different fix: Name and tag of race image in docker repository should be different than production one Dec 10, 2024
@Matovidlo Matovidlo requested a review from a team December 10, 2024 16:38
Copy link
Contributor

@hosekpeter hosekpeter left a comment

Choose a reason for hiding this comment

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

LGTM

@kacurez
Copy link
Member

kacurez commented Dec 11, 2024

@Matovidlo
Copy link
Contributor Author

@kacurez neviem ci to pomoze ci nie, niekde sa zvysil memory consumption a neviem teda ci je to imagom (skor nie) alebo kodom.

Copy link
Member

@kacurez kacurez left a comment

Choose a reason for hiding this comment

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

nie som si isty ci to pomozme ale mozme skusit, je vidiet ze sa nasadil image s tym race
ale pod prod tagom https://keboolaglobal.slack.com/archives/C055K9CRFDX/p1733913074422449?thread_ts=1733834074.491109&cid=C055K9CRFDX

@Matovidlo Matovidlo requested review from kacurez and wokoman December 11, 2024 11:02
Does not influence the `registry/repository` at all and overrides
existing image
@Matovidlo Matovidlo force-pushed the fix-race-image-name-ecr-docker-repository branch from fd0aca5 to cbb010d Compare December 11, 2024 11:10
@Matovidlo Matovidlo requested a review from wokoman December 11, 2024 11:11
Copy link
Contributor

@wokoman wokoman left a comment

Choose a reason for hiding this comment

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

Fabulozni, dik!

@Matovidlo Matovidlo merged commit 4a4b571 into main Dec 11, 2024
12 checks passed
@Matovidlo Matovidlo deleted the fix-race-image-name-ecr-docker-repository branch December 11, 2024 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants