Skip to content

WIP: v1beta2: migrate logic and tests #7369

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

WIP: v1beta2: migrate logic and tests #7369

wants to merge 1 commit into from
+822 −964

Conversation

@mimowo
Copy link
Contributor

@mimowo mimowo commented Oct 23, 2025

What type of PR is this?

/kind api-change

What this PR does / why we need it:

Which issue(s) this PR fixes:

Part of #7113

Special notes for your reviewer:

No release note because the introduction of v1beta2 was already mentioned.

Does this PR introduce a user-facing change?

NONE

@k8s-ci-robot k8s-ci-robot added release-note-none Denotes a PR that doesn't merit a release note. kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Oct 23, 2025
@netlify
Copy link

netlify bot commented Oct 23, 2025 •
edited
Loading

✅ Deploy Preview for kubernetes-sigs-kueue canceled.

Name Link
🔨 Latest commit dd79fc8
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-sigs-kueue/deploys/68fba62962dcd400089b468d

@k8s-ci-robot k8s-ci-robot requested a review from kannon92 October 23, 2025 17:48
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mimowo

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

The pull request process is described here

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 requested a review from PBundyra October 23, 2025 17:48
@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Oct 23, 2025
@mimowo mimowo force-pushed the v1beta2-migrate-logic branch 4 times, most recently from 5cd4460 to 3a80984 Compare October 24, 2025 07:09
@mimowo mimowo changed the title v1beta2: migrate logic and tests to use v1beta2 WIP: v1beta2: migrate logic and tests Oct 24, 2025
@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 Oct 24, 2025
@mimowo mimowo force-pushed the v1beta2-migrate-logic branch 4 times, most recently from a73ed58 to 47f6338 Compare October 24, 2025 08:45
@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 24, 2025
@mimowo mimowo force-pushed the v1beta2-migrate-logic branch from 47f6338 to e7ecf33 Compare October 24, 2025 08:56
@k8s-ci-robot k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 24, 2025
@mimowo mimowo force-pushed the v1beta2-migrate-logic branch from e7ecf33 to 2a9be5d Compare October 24, 2025 08:56
@mimowo mimowo mentioned this pull request Oct 24, 2025
Merged
@mimowo mimowo force-pushed the v1beta2-migrate-logic branch 3 times, most recently from 8356e95 to 0e2c950 Compare October 24, 2025 13:44
@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 24, 2025
@mimowo mimowo force-pushed the v1beta2-migrate-logic branch from 0e2c950 to 916621d Compare October 24, 2025 13:52
@k8s-ci-robot k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 24, 2025
@mimowo
Copy link
Contributor Author

mimowo commented Oct 24, 2025 •
edited
Loading

/test pull-kueue-test-e2e-tas-main
this looks like a flake, but let me try
https://prow.k8s.io/view/gs/kubernetes-ci-logs/pr-logs/pull/kubernetes-sigs_kueue/7369/pull-kueue-test-e2e-tas-main/1981720490477096960

@mimowo mimowo force-pushed the v1beta2-migrate-logic branch 4 times, most recently from 70c4a55 to 0523af3 Compare October 24, 2025 15:10
@mimowo
v1beta2-convert-logic-and-tests
dd79fc8 
Signed-off-by: Michał Woźniak <[email protected]>
@mimowo mimowo force-pushed the v1beta2-migrate-logic branch from 0523af3 to dd79fc8 Compare October 24, 2025 16:15
@k8s-ci-robot
Copy link
Contributor

@mimowo: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-kueue-test-e2e-tas-main dd79fc8 link true /test pull-kueue-test-e2e-tas-main
pull-kueue-test-integration-baseline-main dd79fc8 link true /test pull-kueue-test-integration-baseline-main

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

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-sigs/prow repository. I understand the commands that are listed here.

@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 24, 2025
@k8s-ci-robot
Copy link
Contributor

PR needs rebase.

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-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kannon92 kannon92 Awaiting requested review from kannon92

@PBundyra PBundyra Awaiting requested review from PBundyra

Assignees

No one assigned

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/work-in-progress Indicates that a PR should not merge because it is a work in progress. kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. release-note-none Denotes a PR that doesn't merit a release note. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mimowo @k8s-ci-robot