chore(ci): add Dependabot grouping, PR auto-labeler, and PR size warning#89
Merged
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Comment on lines
+19
to
+23
| - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6 | ||
| with: | ||
| fetch-depth: 0 | ||
| ref: ${{ github.event.pull_request.head.sha }} | ||
| - name: Calculate diff size and apply label |
- Group Dependabot Go module updates: k8s.io/* together, remaining minor/patch together. Group all GitHub Actions updates into a single PR. Reduces weekly Dependabot PR noise from ~13 individual PRs to ~3. - Add actions/labeler workflow to auto-label PRs by changed paths (area/api, area/controller, area/helm, area/ci, area/docs, area/e2e, area/cli). - Add PR size labeler (xs/s/m/l/xl) with a warning comment on XL PRs (500+ lines) suggesting a split. Closes #65 Closes #62 Closes #73 Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
Merge the two E2E jobs (Chainsaw + Go) into a single job that: 1. Starts cluster setup immediately (no dependency on lint/unit) 2. Polls the GitHub API to wait for lint+unit to pass 3. Runs both Chainsaw and Go E2E concurrently on the shared cluster Before: lint (3m) -> E2E setup (4m) -> tests (5.5m) = ~12.5m critical path After: max(lint 3m, E2E setup 4m) -> tests (5.5m) = ~9.5m critical path Saves ~3 minutes per CI run, eliminates duplicate cluster provisioning (one cluster instead of two), and halves the runner cost for E2E. Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
SebTardif
force-pushed
the
chore/ci-quick-wins
branch
from
4993529 to
d38a660
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Batch of CI/repo improvements:
E2E pipeline optimization (new)
Dependabot grouping (#65)
k8s.io/*andsigs.k8s.io/*grouped together; remaining minor/patch updates grouped separatelyPR auto-labeling (#62)
actions/labelerv5 workflow auto-labels PRs by changed pathsarea/api,area/controller,area/helm,area/ci,area/docs,area/e2e,area/cliPR size warning (#73)
size/xs(<10),size/s(10-49),size/m(50-249),size/l(250-499),size/xl(500+)Also closed (settings changes, no code):
Closes #65
Closes #62
Closes #73