Skip to content

ci: remove Tests workflow#255

Open
vitorfloriano wants to merge 1 commit into
kubernetes-sigs:mainfrom
vitorfloriano:ci-remove-test-gha-workflow
Open

ci: remove Tests workflow#255
vitorfloriano wants to merge 1 commit into
kubernetes-sigs:mainfrom
vitorfloriano:ci-remove-test-gha-workflow

Conversation

@vitorfloriano
Copy link
Copy Markdown
Contributor

This workflow has been migrated to Prow as the pull-node-readiness-controller-test presubmit job.

Follow-up to kubernetes/test-infra#37035

This workflow has been migrated to pull-node-readiness-controller-test.
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 14, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented May 14, 2026

Deploy Preview for node-readiness-controller canceled.

Name Link
🔨 Latest commit ad84403
🔍 Latest deploy log https://app.netlify.com/projects/node-readiness-controller/deploys/6a05ee40bd3bb600089779cf

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels May 14, 2026
@Priyankasaggu11929
Copy link
Copy Markdown
Member

LGTM (and thank you for creating the follow up PR).

keeping a hold until PR kubernetes/test-infra#37035 is merged first, and we have atleast one green test showing up in prow/testgrid.

/hold

@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 May 15, 2026
@vitorfloriano vitorfloriano marked this pull request as ready for review May 26, 2026 11:15
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 26, 2026
@k8s-ci-robot k8s-ci-robot requested a review from dchen1107 May 26, 2026 11:15
@vitorfloriano
Copy link
Copy Markdown
Contributor Author

/cc @ajaysundark

@k8s-ci-robot k8s-ci-robot requested a review from ajaysundark May 26, 2026 11:16
@ajaysundark
Copy link
Copy Markdown
Contributor

@vitorfloriano Thanks for handling this!

Could we also document our prow workflows, how to write new tests and relevant maintenance instructions in our project? Maybe also link at the contributing guide? Not necessarily at this PR, you could create an issue and followup as you find time.

@ajaysundark
Copy link
Copy Markdown
Contributor

/approve

+1 to @Priyankasaggu11929 's suggestion, we could wait for removing this until we get a green signal from our new prow setup.

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ajaysundark, vitorfloriano

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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. 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. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants