Skip to content

Publish docker image on push to main#26

Merged
lukpueh merged 2 commits into
eclipse-csi:mainfrom
lukpueh:docker-cd
Mar 9, 2026
Merged

Publish docker image on push to main#26
lukpueh merged 2 commits into
eclipse-csi:mainfrom
lukpueh:docker-cd

Conversation

@lukpueh

@lukpueh lukpueh commented Mar 6, 2026

Copy link
Copy Markdown
Contributor

These images are annotated as "latest". If desired, we can also annotate with the short sha by adding the following line in the tags section of the "Extract metadata" step:

type=sha,format=short,enable={{is_default_branch}}

This would improve traceability, but has the downside of quickly filling up the list in:
https://github.com/eclipse-csi/pia/pkgs/container/pia/versions

Since traceability is not so important for images built off of main, I left it out for now.

The PR also includes two unrelated fixes (see commit message for details).

lukpueh added 2 commits March 6, 2026 12:56
These images are annotated as "latest". If desired, we can also
annotated with the short sha by adding the following line in the
tags section of the Extract metadata step:

```
type=sha,format=short,enable={{is_default_branch}}
```

This would improve traceability, but has the downside of quickly filling
up the list in:
https://github.com/eclipse-csi/pia/pkgs/container/pia/versions

Since traceability is not so important for images built off of main, I
let it out for now.

Signed-off-by: Lukas Puehringer <lukas.puehringer@eclipse-foundation.org>
* Remove redundant release event trigger
  ->  We already listen on tag push
* Remove unused/unneeded annotations in "Extract metadata" step.
  ->  We don't care for image annotations of type ref (branch name, pr
      number, etc.), and we don't even run on PRs

Signed-off-by: Lukas Puehringer <lukas.puehringer@eclipse-foundation.org>
@lukpueh
lukpueh merged commit 54671df into eclipse-csi:main Mar 9, 2026
3 checks passed
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.

2 participants