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

v1.2.2 (latest) is not uploaded to docker hub #5416

Closed
thle40 opened this issue Oct 9, 2023 · 3 comments · Fixed by #5460
Closed

v1.2.2 (latest) is not uploaded to docker hub #5416

thle40 opened this issue Oct 9, 2023 · 3 comments · Fixed by #5460

Comments

@thle40
Copy link

thle40 commented Oct 9, 2023

Version of Cadence server, and client(which language)
This is very important to root cause bugs.

  • Server version: v.1.2.2

Describe the bug
All versions which are pre-release or used to be pre-release are not uploaded to dockerhub

To Reproduce
Is the issue reproducible?

  • yes

Steps to reproduce the behavior:
The docker images of latest version (v1.2.2) is not uploaded to docker hub

Expected behavior
The docker image of all versions are uploaded to docker hub
image

@sonhuynhpham
Copy link

It looks like Cadence server v1.2.2 was pre-released initially before it's officially released. The workflow that builds and publishes the server image is triggered once when the release is published:

types: [ published ]

However, all of its steps are conditionally to not run for prereleases:

if: "!github.event.release.prerelease"

After the release change its state from prerelease to release, no workflow is triggered. Hence, no image is published for v1.2.2 as the result.

I believe the solution is to add the released condition to the workflow (doc). Note that, this change will only take effect for the release drafted after this change. So, I think the Cadence team will have to manually build and publish the Docker image for this version.

@Nepoxx
Copy link

Nepoxx commented Oct 20, 2023

This seems to have been the case since 1.0.0 as it is the latest tagged released on dockerhub: https://hub.docker.com/r/ubercadence/server/tags?page=1&name=v1

@3vilhamster
Copy link
Contributor

Thank you for reporting!
We will try to fix it on the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants