-
-
Notifications
You must be signed in to change notification settings - Fork 10
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
speed up the Docker image library to create/push tags at the same time for both GH and Docker (instead of running additional build) #3751
Comments
Proposal : Problem: how to handle manual tag and still not build on automatic tag from the pipeline library ? |
By "handle manual tag" I assume you mean "When manually creating a tag in the GitHub repository"? In any cases, we'll have to disable the "automatic build trigger" rule in infra.ci.jenkins: this rule usually says "when you discover a tag, then trigger a build automatically on the git reference pointed by this tag unless it is older than 3 days". It is configured as a custom "build strategy" defined in JobDSL template here: https://github.com/jenkins-infra/helm-charts/blob/436ec843b49a0dea1088bb00388a9469f94a7b40/charts/jenkins-jobs/templates/_helpers.tpl#L168-L188. => We will have to disable this to allow your change to work at first, to stop build being triggered each time a tag is pushed. Next step: we'll have to decide if we also keep discovering tags in infra.ci. I believe we should not, but in that case we have to find a solution to NOT remove the builds of the |
…s in infra.ci.jenkins.io as per jenkins-infra/helpdesk#3751 to test the PR on the shared pipeline library
…s in infra.ci.jenkins.io (#4515) as per jenkins-infra/helpdesk#3751 to test the PR on the shared pipeline library
Service(s)
infra.ci.jenkins.io
Summary
following jenkins-infra/pipeline-library#730
we can now build and publish the image in docker from main and the tag in one pass to save some power and avoid one pass in main and then on the github tag.
Reproduction steps
No response
The text was updated successfully, but these errors were encountered: