-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
chore: disable publication from ci.jenkins.io #351
Conversation
@Library('pipeline-library@pull/784/head') _ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Library('pipeline-library@pull/784/head') _ |
To be removed in a future PR when jenkins-infra/pipeline-library#784 will be merged.
EDIT:
Cherry-picked #353 commit to get a passing job. |
buildDockerAndPublishImage('account-app', [ | ||
rebuildImageOnPeriodicJob: false, | ||
automaticSemanticVersioning: true, | ||
targetplatforms: 'linux/amd64,linux/arm64' | ||
]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should merge these 2 pipelines into a single one to ensure the same set of parameters is applied to both CI and CD process (and avoid CD issues which could have been caught by CI).
=> WDYT about the method you used (can't remember where though): disablePublication: !infra.isInfra()
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the next step.
First I needed to be sure that my pipeline change worked.
This PR disables tagging and publication of docker image and github release from ci.jenkins.io
The merge of this PR will also serve as a test for jenkins-infra/pipeline-library#784
Ref: