Skip to content

Conversation

@natasha41575
Copy link
Contributor

@natasha41575 natasha41575 commented Nov 12, 2025

Here is a proposal for how we can reorganize the SIG Node CI testgrid to make tests easier to find. I generally find it hard to navigate the current testgrid; it is not obvious (at least to me) where or if a particular test is running in the testgrid today.

This came out of me picking up kubernetes/kubernetes#134862; while starting to look at what we had previously defined as our test coverage guidance (which is still not implemented), I thought the current proposal can be improved. This is intended to be an incremental improvement, and can be iterated upon in the future. I also want to be careful not to remove any existing test coverage.

Also another, note, there are some highly specialized test tabs like sig-node-problem-detector, sig-node-security-profiles-operator, sig-node-ec2, sig-node-gpu etc, I don't intend to touch those.

All feedback and suggestions are welcome. I am not an expert on test-infra or all node features so please feel free to bring up issues or mistakes :)

/sig node

/assign @SergeyKanzhelev
/cc @mariafromano-25 @haircommander @kannon92

/hold
just to gate merging until we are ready

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 12, 2025
@k8s-ci-robot k8s-ci-robot added the sig/node Categorizes an issue or PR as relevant to SIG Node. label Nov 12, 2025
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Nov 12, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: natasha41575
Once this PR has been reviewed and has the lgtm label, please assign haircommander for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@k8s-ci-robot k8s-ci-robot added area/config Issues or PRs related to code in /config area/jobs sig/testing Categorizes an issue or PR as relevant to SIG Testing. labels Nov 12, 2025
@natasha41575 natasha41575 changed the title proposal: new SIG Node CI testgrid organization guide Update SIG Node CI test coverage guide Nov 12, 2025
<td>os
</td>
<td>cos, ubuntu, fedora
<td>COS, ubuntu
Copy link
Contributor

Choose a reason for hiding this comment

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

curious why fedora was dropped? they're the only cri-o tests AFAIU

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah I can put them back for cri-o, thanks for the callout!

<td>cri
</td>
<td>containerd, CRI-O
<td>containerd
Copy link
Contributor

Choose a reason for hiding this comment

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

I know we sometimes struggle to keep them green but I'd love cri-o tests to be kept here if possible

@haircommander
Copy link
Contributor

/triage accepted
/priority important-longterm

@k8s-ci-robot k8s-ci-robot added triage/accepted Indicates an issue or PR is ready to be actively worked on. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. labels Dec 17, 2025
@haircommander haircommander moved this from Triage to PRs Waiting on Author in SIG Node CI/Test Board Dec 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/config Issues or PRs related to code in /config area/jobs cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. sig/node Categorizes an issue or PR as relevant to SIG Node. sig/testing Categorizes an issue or PR as relevant to SIG Testing. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. triage/accepted Indicates an issue or PR is ready to be actively worked on.

Projects

Status: PRs Waiting on Author

Development

Successfully merging this pull request may close these issues.

4 participants