Skip to content

GHA label checker#9050

Merged
tekton-robot merged 1 commit intotektoncd:mainfrom
afrittoli:kind_label
Oct 2, 2025
Merged

GHA label checker#9050
tekton-robot merged 1 commit intotektoncd:mainfrom
afrittoli:kind_label

Conversation

@afrittoli
Copy link
Member

@afrittoli afrittoli commented Sep 30, 2025 •
edited
Loading

Changes

Add a GHA based label checker

/kind misc

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

  • Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
  • Has Tests included if any functionality added or changed
  • pre-commit Passed
  • Follows the commit message standard
  • Meets the Tekton contributor standards (including functionality, content, code)
  • Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
  • Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
  • Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes

NONE

@tekton-robot tekton-robot added the release-note-none Denotes a PR that doesnt merit a release note. label Sep 30, 2025
@tekton-robot tekton-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 30, 2025
@afrittoli afrittoli added area/api Indicates an issue or PR that deals with the API. api-review Categorizes an issue or PR as actively needing an API review. and removed area/api Indicates an issue or PR that deals with the API. api-review Categorizes an issue or PR as actively needing an API review. labels Sep 30, 2025
@afrittoli
Copy link
Member Author

afrittoli commented Sep 30, 2025

/kind misc

@tekton-robot tekton-robot added the kind/misc Categorizes issue or PR as a miscellaneuous one. label Sep 30, 2025
@afrittoli
Copy link
Member Author

afrittoli commented Sep 30, 2025

/kind bug

@tekton-robot tekton-robot added the kind/bug Categorizes issue or PR as related to a bug. label Sep 30, 2025
@afrittoli afrittoli removed the kind/bug Categorizes issue or PR as related to a bug. label Sep 30, 2025
@afrittoli
Copy link
Member Author

afrittoli commented Sep 30, 2025

/retest

1 similar comment
@afrittoli
Copy link
Member Author

afrittoli commented Sep 30, 2025

/retest

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 30, 2025
@afrittoli
Copy link
Member Author

afrittoli commented Sep 30, 2025

/retest

@afrittoli afrittoli added the area/test-infra Issues or PRs related to the testing infrastructure label Sep 30, 2025
@afrittoli afrittoli mentioned this pull request Sep 30, 2025
Open
@afrittoli
Copy link
Member Author

afrittoli commented Oct 1, 2025

/retest

@tekton-robot tekton-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Oct 1, 2025
@afrittoli afrittoli mentioned this pull request Oct 1, 2025
Open
AlanGreene
AlanGreene reviewed Oct 1, 2025
View reviewed changes
@afrittoli
GHA label checker
0afa11c 
Free up space on the workers before running CI
as disk full is causing CI to fail and blocking PRs.

Signed-off-by: Andrea Frittoli <[email protected]>
@afrittoli
Copy link
Member Author

afrittoli commented Oct 1, 2025

@AlanGreene CI is green 🎉 ok to merge?

AlanGreene
AlanGreene approved these changes Oct 1, 2025
View reviewed changes
Copy link
Member

@AlanGreene AlanGreene left a comment

Choose a reason for hiding this comment

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

Don't think this will count, but looks good 👍
/lgtm

@tekton-robot
Copy link
Collaborator

tekton-robot commented Oct 1, 2025

@AlanGreene: changing LGTM is restricted to collaborators

Details

In response to this:

Don't think this will count, but looks good 👍
/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

vdemeester
vdemeester approved these changes Oct 2, 2025
View reviewed changes
Copy link
Member

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

/lgtm

My comment can be taken into account later or not, it really is a nit.

.github/workflows/e2e-matrix.yml
ARTIFACTS: ${{ github.workspace }}/artifacts

steps:
- name: Free disk space
Copy link
Member

@vdemeester vdemeester Oct 2, 2025

Choose a reason for hiding this comment

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

If I remember correctly, there is an action that does this, but seems good as well.

      - name: Maximize build space
        uses: AdityaGarg8/remove-unwanted-software@90e01b21170618765a73370fcc3abbd1684a7793 # v5
        with:
          remove-android: 'true'
          remove-dotnet: 'true'
          remove-haskell: 'true'
          remove-codeql: 'true'
          remove-docker-images: 'false'
          remove-large-packages: 'true'
          remove-cached-tools: 'true'
          remove-swapfile: 'true'

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 2, 2025
@tekton-robot
Copy link
Collaborator

tekton-robot commented Oct 2, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: AlanGreene, vdemeester

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot merged commit 81db63a into tektoncd:main Oct 2, 2025
49 of 50 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in Tekton Community Roadmap Oct 2, 2025
@waveywaves waveywaves mentioned this pull request Dec 17, 2025
Closed
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vdemeester vdemeester vdemeester approved these changes

@dibyom dibyom Awaiting requested review from dibyom

@twoGiants twoGiants Awaiting requested review from twoGiants

+1 more reviewer

@AlanGreene AlanGreene AlanGreene approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@vdemeester vdemeester

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/test-infra Issues or PRs related to the testing infrastructure kind/misc Categorizes issue or PR as a miscellaneuous one. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesnt merit a release note. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

Tekton Community Roadmap
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@afrittoli @tekton-robot @vdemeester @AlanGreene