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

Missing 0.15.x releases on docker hub #1432

Closed
FlorianLudwig opened this issue May 30, 2024 · 7 comments · Fixed by #1568
Closed

Missing 0.15.x releases on docker hub #1432

FlorianLudwig opened this issue May 30, 2024 · 7 comments · Fixed by #1568

Comments

@FlorianLudwig
Copy link

On docker hub no tags for 0.15 releases can be found:

https://hub.docker.com/r/lycheeverse/lychee/tags?page=&page_size=&ordering=&name=0.15

@thomas-zahner
Copy link
Member

Thanks for reporting. Hmm, for some reason the release pipeline did not publish them..
We re-ran them manually so 0.15.0 and 0.15.1 are now available

@mre mre closed this as completed Jun 11, 2024
@eread
Copy link
Contributor

eread commented Nov 10, 2024

@thomas-zahner @mre It looks like this persists with the last couple of versions also. Only SHA tags, no version tags.

@mre mre reopened this Nov 12, 2024
@mre
Copy link
Member

mre commented Nov 12, 2024

It feels like we broke it when switching to release-plz. I tried to re-run the pipeline, but it didn't create a new version tag. Not entirely sure how to fix it. Maybe someone has an idea.

@eread
Copy link
Contributor

eread commented Nov 14, 2024

It feels like we broke it when switching to release-plz. I tried to re-run the pipeline, but it didn't create a new version tag. Not entirely sure how to fix it. Maybe someone has an idea.

@mre I noticed at the bottom of Push Image (debian) https://github.com/lycheeverse/lychee/actions/runs/11708052907/job/32608866699 this warning:

WARNING: No output specified with docker-container driver. Build result will only remain in the build cache. To push result image into registry use --push or to load image into docker use --load

Looking at the project docs at: https://github.com/docker/build-push-action, it looks like you might need to set with.push = true to have the push actually happen.

Because that isn't set anywhere in this project, I don't understand how any pushes are happening.

I wonder if anything has changed at: https://github.com/docker/build-push-action/releases? 🤔

@mre
Copy link
Member

mre commented Nov 15, 2024

Very interesting. I think it pushed in the past at least -- otherwise there would be no images. But yeah, we should set that. If you like, you could add it and we'll give it a shot?

@eread
Copy link
Contributor

eread commented Nov 15, 2024

Very interesting. I think it pushed in the past at least -- otherwise there would be no images. But yeah, we should set that. If you like, you could add it and we'll give it a shot?

@mre I think I figured it out!

Looking more closely (not sure why I missed this before): https://github.com/lycheeverse/lychee/blame/master/.github/workflows/docker.yml, there's:

push: ${{ github.event_name != 'pull_request' && github.actor != 'dependabot[bot]' }}

That's been in place for a while: 6887543, so at some point that's stopped evaluating to true when releases are cut.

I think that's happened because you've started raising pull requests for releases, like: #1549. I think all the "missing" releases are ones released using pull requests.

So it seems we need to figure out how have that evaluate on some pull requests (release PRs), but not all.

@eread
Copy link
Contributor

eread commented Nov 18, 2024

@mre I've raised: #1568 to try and figure this out.

@mre mre closed this as completed in #1568 Dec 5, 2024
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 a pull request may close this issue.

4 participants