Skip to content

Commit

Permalink
Merge pull request #4288 from locriandev/ocp4-scan-konflux
Browse files Browse the repository at this point in the history
ocp4-scan: ignore lock on dry run mode
  • Loading branch information
openshift-merge-bot[bot] authored Nov 15, 2024
2 parents 9bbc226 + 329384a commit 73b881a
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 18 deletions.
12 changes: 6 additions & 6 deletions jobs/build/ocp4-scan-konflux/Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,12 @@ timeout(activity: true, time: 60, unit: 'MINUTES') {
defaultValue: "",
trim: true,
),
booleanParam(
name: 'DRY_RUN',
description: 'Run scan without triggering subsequent jobs',
defaultValue: false,
),
commonlib.mockParam(),
booleanParam(
name: 'DRY_RUN',
description: 'Run scan without triggering subsequent jobs',
defaultValue: false,
),
commonlib.mockParam(),
]
],
disableResume()
Expand Down
12 changes: 0 additions & 12 deletions jobs/build/ocp4_scan/Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -37,11 +37,6 @@ timeout(activity: true, time: 60, unit: 'MINUTES') {
commonlib.artToolsParam(),
commonlib.ocpVersionParam('VERSION', '4'),
commonlib.suppressEmailParam(),
booleanParam(
name: 'IGNORE_LOCKS',
description: 'Do not wait for other builds in this version to complete (allowed only if DRY_RUN is true)',
defaultValue: false
),
string(
name: 'MAIL_LIST_SUCCESS',
description: 'Success Mailing List',
Expand Down Expand Up @@ -79,10 +74,6 @@ timeout(activity: true, time: 60, unit: 'MINUTES') {

if (params.DRY_RUN) {
currentBuild.displayName += "[DRY_RUN]"
} else {
if (params.IGNORE_LOCKS) {
error("IGNORE_LOCKS can be used only with dry run mode")
}
}

buildlib.cleanWorkdir(artcd_working, true)
Expand All @@ -104,9 +95,6 @@ timeout(activity: true, time: 60, unit: 'MINUTES') {
"ocp4-scan",
"--version=${params.VERSION}"
]
if (params.IGNORE_LOCKS) {
cmd << "--ignore-locks"
}

// Run pipeline
buildlib.withAppCiAsArtPublish() {
Expand Down

0 comments on commit 73b881a

Please sign in to comment.