Skip to content

Commit

Permalink
Bug fix for trigger pipeline Buildkite pipeline (#15700)
Browse files Browse the repository at this point in the history
This commit fixes a few bugs introduced in #15668 related to paths for
the calling script. We also stop limiting the execution only from the
main branch (to facilitate e.g. tests from PRs) and, finally, remove
the async clause, which is not needed, since by default BK steps are
run in parallel.
  • Loading branch information
dliappis authored Dec 19, 2023
1 parent c2eaecc commit 42497ed
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
6 changes: 2 additions & 4 deletions .buildkite/scripts/common/trigger-pipeline-generate-steps.sh
Original file line number Diff line number Diff line change
Expand Up @@ -47,16 +47,14 @@ for BRANCH in $BRANCHES; do
cat >>pipeline_steps.yaml <<EOF
- trigger: $PIPELINE_TO_TRIGGER
label: ":testexecute: Triggering ${PIPELINE_TO_TRIGGER} / ${BRANCH}"
branches: "main"
async: true
build:
branch: "$BRANCH"
message: ":testexecute: Triggering ${PIPELINE_TO_TRIGGER} / ${BRANCH}"
message: ":testexecute: Scheduled build for ${BRANCH}"
EOF
done

echo "--- Printing generated steps"
yq . pipeline_steps.yaml

echo "--- Uploading steps to buildkite"
cat pipeline_steps.yml | buildkite-agent pipeline upload
cat pipeline_steps.yaml | buildkite-agent pipeline upload
2 changes: 1 addition & 1 deletion .buildkite/trigger_pipeline.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@

steps:
- label: ":pipeline: Generate trigger steps for $PIPELINE_TO_TRIGGER"
command: ".buildkite/scripts/trigger-pipeline-generate-steps.sh"
command: ".buildkite/scripts/common/trigger-pipeline-generate-steps.sh"

0 comments on commit 42497ed

Please sign in to comment.