Skip to content

Commit

Permalink
chore(CI): Add missing --use-consignor flag on smp call (vectordo…
Browse files Browse the repository at this point in the history
…tdev#17966)

The `smp` status wait command was missing the `--use-consignor` flag.
This caused it to wait for our EC2-hosted Analysis Worker to complete.
Subsequent steps read the lambda hosted result (`consignor`), which may
still be processing. Adding this flag corrects this synchronization bug.
  • Loading branch information
GeorgeHahn authored Jul 13, 2023
1 parent 98e47c1 commit 7cae000
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions .github/workflows/regression.yml
Original file line number Diff line number Diff line change
Expand Up @@ -530,6 +530,7 @@ jobs:
${{ runner.temp }}/bin/smp --team-id ${{ secrets.SINGLE_MACHINE_PERFORMANCE_TEAM_ID }} job status \
--use-curta \
--use-consignor \
--wait \
--wait-delay-seconds 60 \
--wait-timeout-minutes 90 \
Expand Down

0 comments on commit 7cae000

Please sign in to comment.