Fix duplicated YAML keys in c9k-failure-report.yml#11744
Conversation
Workflow file accidentally contained two stacked definitions, causing 'Invalid workflow file' errors (duplicate name/on/permissions/jobs keys). Replace with a single, valid workflow definition.
|
This PR requires exactly 1 of the following labels: pr:standard, pr:important. Label descriptions:
@sylvainsf, please add the appropriate label to this PR before merging. |
Radius functional test overviewClick here to see the test run details
Test Status⌛ Building Radius and pushing container images for functional tests... |
There was a problem hiding this comment.
Pull request overview
Fixes a YAML parse error in the C9K failure report workflow by removing an accidentally duplicated, concatenated workflow definition.
Changes:
- Remove the duplicated top-level workflow block that caused duplicate
name/on/permissions/jobskeys. - Keep a single valid workflow definition with SHA-pinned
sylvainsf/causinator9000usage. - Preserve the intended schedule/inputs (48h window, dry-run, etc.) while restoring workflow validity.
| @@ -33,31 +33,3 @@ jobs: | |||
| auto-close-flaky: "true" | |||
| assign-copilot: "false" | |||
| github-token: ${{ secrets.GITHUB_TOKEN }} | |||
There was a problem hiding this comment.
Use ${{ github.token }} instead of ${{ secrets.GITHUB_TOKEN }} for the default workflow token. This matches the convention used across other workflows (e.g., .github/workflows/close-stale-prs.yml:25) and avoids treating the default token like a user-defined secret.
| github-token: ${{ secrets.GITHUB_TOKEN }} | |
| github-token: ${{ github.token }} |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #11744 +/- ##
=======================================
Coverage 51.42% 51.42%
=======================================
Files 699 699
Lines 55723 55723
=======================================
+ Hits 28654 28657 +3
+ Misses 24901 24899 -2
+ Partials 2168 2167 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
What
Fixes
.github/workflows/c9k-failure-report.yml, which currently fails to parse:Why
The merged file accidentally contained two stacked workflow definitions concatenated together, producing duplicate top-level keys (
name,on,permissions,jobs) and a non-SHA-pinneduses:line in the second copy.How
Replaced the file with a single, valid workflow definition:
name,on,permissions: {}, andjobs:blocksylvainsf/causinator9000@5be8cf7dfbb78394eb7f6f2c97e2a9a5ab95deef # v1.9.0No behavioral changes from the originally intended workflow, just removes the duplicated, invalid second copy.