Skip to content

Commit

Permalink
[ci] Fix Jenkins quoting
Browse files Browse the repository at this point in the history
Merging #13368 caused CI to pass but run more than it needed to due to
some failures in determination. This fixes the interpolation to use `"`
which should correctly pass through the variables
  • Loading branch information
driazati committed Nov 14, 2022
1 parent 5a767d0 commit ec364ab
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions Jenkinsfile

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions ci/jenkins/Prepare.groovy.j2
Original file line number Diff line number Diff line change
Expand Up @@ -205,14 +205,14 @@ def prepare() {

is_docs_only_build = sh (
returnStatus: true,
script: './${jenkins_scripts_root}/git_change_docs.sh',
script: "./${jenkins_scripts_root}/git_change_docs.sh",
label: 'Check for docs only changes',
)
skip_ci = should_skip_ci(env.CHANGE_ID)
skip_slow_tests = should_skip_slow_tests(env.CHANGE_ID)
rebuild_docker_images = sh (
returnStatus: true,
script: './${jenkins_scripts_root}/git_change_docker.sh',
script: "./${jenkins_scripts_root}/git_change_docker.sh",
label: 'Check for any docker changes',
)

Expand Down

0 comments on commit ec364ab

Please sign in to comment.