Skip to content
This repository has been archived by the owner on Feb 1, 2023. It is now read-only.

Keep latest tags for branches #62

Merged
merged 1 commit into from
Sep 23, 2020
Merged

Keep latest tags for branches #62

merged 1 commit into from
Sep 23, 2020

Conversation

skovhus
Copy link
Contributor

@skovhus skovhus commented Sep 18, 2020

Resolves #61

We recently introduced a change to not not tag images on non-main branches with "latest". This avoids cache breaking as all branches will not all push the "latest" image – but it does break local development where the latest tag is used in Docker compose files.

I have a workaround if this indeed breaks the cache, when two branch builds tags the same repository with the latest tag and thereby breaks the cache.

Else we break local development
@madsnedergaard
Copy link
Member

The code make sense, but could you give me a small example of the impact of this? :)

@skovhus
Copy link
Contributor Author

skovhus commented Sep 23, 2020

The code make sense, but could you give me a small example of the impact of this? :)

The impact is that you locally can use the latest tag in the docker-compose files again. Most of our docker compose files requires this.

There should not be any other impact, unless the cache is invalidated by two branches pushing the latest tag.

Let me know if I can explain this better

Copy link
Member

@madsnedergaard madsnedergaard left a comment

Choose a reason for hiding this comment

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

Thanks for the explanation, it was mostly the cache invalidation part I'm unsure of - but let's just try it :)

@skovhus skovhus merged commit da0a7f5 into master Sep 23, 2020
@skovhus skovhus deleted the ks/tag-latest-on-branches branch September 23, 2020 09:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Keep latest tags for branches
2 participants