Fix update-schedule workflow: rename branch, add proper error handling#54932
Fix update-schedule workflow: rename branch, add proper error handling#54932Caesarsage wants to merge 1 commit intokubernetes:mainfrom
Conversation
The workflow has been silently failing because the stale remote branch "update-schedule" could not be force-pushed to, preventing PR creation. Changes: - Rename PR branch from "update-schedule" to "auto/update-schedule" - Delete stale remote branch before creating PR to prevent push failures - Fix retry loop to properly fail the job after 3 attempts - Remove unnecessary fetch-depth: 0
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
This really needs an issue filed. Also: before making any update, we ought to make sure that we know why the existing workflow is changing. |
|
I expect that using a different branch name might fail due to rules on branch protection. |
Okay, I didn't think about that. Another way would be to delete the stale update-schedule branch. The workflow should then work as-is on the next run. We could then add a step to the action to always delete the stale PR branch before creating a new one, to prevent this from happening again. |
The workflow has been silently failing because the stale remote branch "update-schedule" could not be force-pushed to, preventing PR creation.
Changes:
I tried this on my own branch to confirm work flow works which created this : Caesarsage#2
my debug/test changes:
Caesarsage@201909b
closes #54940