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

Automated Docker builds don't appear to be triggering #474

Closed
jcormier opened this issue Aug 3, 2021 · 33 comments · Fixed by #491
Closed

Automated Docker builds don't appear to be triggering #474

jcormier opened this issue Aug 3, 2021 · 33 comments · Fixed by #491
Assignees
Labels

Comments

@jcormier
Copy link
Collaborator

jcormier commented Aug 3, 2021

@sameersbn Could you check to see if something needs attention?

Tag 4.1.4 and 4.2.2 don't appear to have triggered a build.

@sameersbn
Copy link
Owner

sameersbn commented Aug 4, 2021

Hey @jcormier . Turns out docker has disabled automated builds for free accounts.
I'll work on updating the CI to build and push the image over the weekend.

@frank-dspeed
Copy link
Collaborator

@sameersbn migration to github tasks is the solution. and push it from there to docker and github registry.

@frank-dspeed
Copy link
Collaborator

@sameersbn we need 2 values configured as secrets so that i can enable pushing to the docker registry when ever something hits master so everything should be tested before it hits master

The login-action options required for Docker Hub are:

username and password: This is your Docker Hub username and password. We recommend storing your Docker Hub username and password as secrets so they aren't exposed in your workflow file. For more information, see "Creating and using encrypted secrets." https://docs.github.com/en/actions/automating-your-workflow-with-github-actions/creating-and-using-encrypted-secrets

it would be best if you would create them with that names

         username: ${{ secrets.DOCKER_USERNAME }}
          password: ${{ secrets.DOCKER_PASSWORD }}

@jcormier
Copy link
Collaborator Author

jcormier commented Aug 4, 2021

i can enable pushing to the docker registry when ever something hits master

Can the github workflow trigger on pushed tags like how the old docker builds worked? This allows building tagged docker images and support updates to older redmine versions.

It also triggered on pushes to master which got labeled as :latest

@frank-dspeed
Copy link
Collaborator

frank-dspeed commented Aug 4, 2021

@jcormier at present i did configure it to trigger build push on github release creation that is maybe the most easy flow and matches the old one?
so yes it is tag based.

@jcormier
Copy link
Collaborator Author

jcormier commented Aug 4, 2021

Add okay that should work fine. Thanks.

@jcormier
Copy link
Collaborator Author

@sameersbn Looks like you build and pushed 4.1.4 and 4.2.2 today? Are these automated? Any info on how you've set it up if so?

@sameersbn
Copy link
Owner

Hey. Since I was unable to find the time to work on the automation I manually built and pushed the images.

@josefglatz
Copy link
Contributor

josefglatz commented Oct 7, 2021

@sameersbn Could you please also push 4.2.2-1 to docker?

@josefglatz
Copy link
Contributor

@sameersbn Could you please also push 4.2.3 to docker?

@jcormier
Copy link
Collaborator Author

Also 4.1.5

@sameersbn
Copy link
Owner

sameersbn commented Oct 13, 2021

Hey folks, sorry about this. Pushed the following tags

  • 4.1.5
  • 4.2.2-1
  • 4.2.3

@jcormier
Copy link
Collaborator Author

jcormier commented Feb 9, 2022

#488 Request to push 4.2.3-1

@sameersbn
Copy link
Owner

working in the releases...

@sameersbn
Copy link
Owner

✅ sameersbn/redmine:4.2.3-1
✅ sameersbn/redmine:4.2.3-2

@sameersbn
Copy link
Owner

#491 implements gh actions for building docker images when commits occur on the master branch and when new tags are created. Additionally the tags are published to docker.io, quay.io and ghcr.io (which will show up under packages link of the repo which is pretty neat).

Sorry about the delay in getting this done.

@jcormier
Copy link
Collaborator Author

This is great, thanks

@jcormier
Copy link
Collaborator Author

jcormier commented Mar 8, 2022

#494

@jcormier
Copy link
Collaborator Author

Tried delete pushing the 4.1.6 tag and repushing. Build didn't trigger. Then created a new tag 4.1.6-1 and pushed tag but build didn't trigger.

git push --delete origin 4.1.6
git push origin 4.1.6

@jcormier
Copy link
Collaborator Author

Tried switching to triggering on releases but the release for 4.1.6-1 didn't trigger a github action...

@jcormier
Copy link
Collaborator Author

@sameersbn Could you manually trigger build for 4.1.6-1 and 4.2.4

@frank-dspeed
Copy link
Collaborator

frank-dspeed commented Mar 24, 2022

@jcormier can you define the final behavior that you want and i simply integrate it simply only tell me the commands that should lead to a release and i integrate it.

on release? any way i will simply move this repo over to my organisation @dockerimages/docker-redmine and will configure automated builds from there

we got no need to publish it under the sameersbn namespace. on dockerhub

@frank-dspeed
Copy link
Collaborator

@jcormier 4.1.6-1 is not semver compatible! semver allows only chars after the - not digits!4.1.6-pre0 would be valid but there must be a char after -

@sameersbn
Copy link
Owner

sameersbn commented Mar 24, 2022 via email

@sameersbn
Copy link
Owner

@frank-dspeed is right. the tags need to be semver compatible. so I propose using -rX for the revisions, fe. 4.1.6-r1

@sameersbn
Copy link
Owner

there were a couple of issues:

  1. as @frank-dspeed mentioned the tags need to be semver compatible.
  2. the ci configuration did not exist in the 4.1-stable branch. as a result tags created from that branch were not being picked up by the CI

I have fixed the above issues and created the tag 4.1.6-r2 on the 4.1-stable branch. for 4.2.4 release, I deleted the existing tag and recreated it from the tip of the master

@sameersbn
Copy link
Owner

i can see the tags are now built and also available on the github releases page https://github.com/sameersbn/docker-redmine/pkgs/container/redmine.

Again.. sorry about the delay.

@frank-dspeed
Copy link
Collaborator

@jcormier invited you as admin jcormier has been added as a collaborator with admin permissions on the repository. https://github.com/dockerimages/docker-redmine

i will configure now automated publishing and will integrate the rest of my stashed changes directly we will produce the best redmine image ever and will automate the maintainance. we will publish it https://hub.docker.com/u/dockerimages/redmine

@jcormier
Copy link
Collaborator Author

jcormier commented Mar 24, 2022

@jcormier invited you as admin jcormier has been added as a collaborator with admin permissions on the repository. https://github.com/dockerimages/docker-redmine

@sameersbn is a busy guy so I see the value in having a shared repo where more of us have admin rights to keep things working. Though getting people to move over will be a bit bumpy.

i will configure now automated publishing and will integrate the rest of my stashed changes directly we will produce the best redmine image ever and will automate the maintainance. we will publish it https://hub.docker.com/u/dockerimages/redmine

Any large structural changes I'd like a chance to look over before it makes it into master. I don't have a large amount of time to redesign the project or anything.

@sameersbn
Copy link
Owner

sameersbn commented Mar 25, 2022 via email

@frank-dspeed
Copy link
Collaborator

@sameersbn you can have my "dockerimages" org if you like i had never the time and contributors to maintain all that.

@frank-dspeed
Copy link
Collaborator

frank-dspeed commented Mar 31, 2022

Closed by #498 and in Favor of

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

Successfully merging a pull request may close this issue.

4 participants