Skip to content

Update error handling in pipeline script#920

Merged
qroll merged 4 commits intomasterfrom
update-pipeline-script
Dec 29, 2025
Merged

Update error handling in pipeline script#920
qroll merged 4 commits intomasterfrom
update-pipeline-script

Conversation

@qroll
Copy link
Contributor

@qroll qroll commented Dec 23, 2025

Changes

  • job runs infinitely when error is encountered, updated the script to exit when the curl fails on 401
  • adjust timings
  • remove unintended sleep when job status is complete
  • [delete] branch

@qroll qroll requested a review from chongwyb December 23, 2025 14:12
@qroll qroll added the type: chore For technical improvements or refactoring. label Dec 23, 2025
@qroll qroll added this to the v3.1.0-canary.6 milestone Dec 23, 2025
@qroll qroll force-pushed the update-pipeline-script branch 14 times, most recently from 4a109f0 to b082556 Compare December 26, 2025 15:49
@qroll qroll force-pushed the update-pipeline-script branch from ea31713 to 8cd91f4 Compare December 27, 2025 01:13
Copy link
Contributor

@chongwyb chongwyb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved the validation of token if invalid

@qroll qroll merged commit 91c77c3 into master Dec 29, 2025
1 check passed
@qroll qroll deleted the update-pipeline-script branch December 29, 2025 05:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: chore For technical improvements or refactoring.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments