[#5310] workload controller delete refactor #7585
Draft
+433
−69
Conversation
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
…rnetes-sigs#7321) Bumps [github.com/cert-manager/cert-manager](https://github.com/cert-manager/cert-manager) from 1.19.0 to 1.19.1. - [Release notes](https://github.com/cert-manager/cert-manager/releases) - [Changelog](https://github.com/cert-manager/cert-manager/blob/master/RELEASE.md) - [Commits](cert-manager/cert-manager@v1.19.0...v1.19.1) --- updated-dependencies: - dependency-name: github.com/cert-manager/cert-manager dependency-version: 1.19.1 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Remove dispatcherName from workload webhook * Update tests post webhook dispatcherName removal * Allow to set ClusterName with ElasticJob even though it is not one of the nominatedClusterNames.
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…7239) Bumps the all group in /test/e2e/kueueviz with 1 update: [cypress](https://github.com/cypress-io/cypress). Updates `cypress` from 15.3.0 to 15.4.0 - [Release notes](https://github.com/cypress-io/cypress/releases) - [Changelog](https://github.com/cypress-io/cypress/blob/develop/CHANGELOG.md) - [Commits](cypress-io/cypress@v15.3.0...v15.4.0) --- updated-dependencies: - dependency-name: cypress dependency-version: 15.4.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: all ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…igs#7323) Bumps node from 24-alpine to 25-alpine. --- updated-dependencies: - dependency-name: node dependency-version: 25-alpine dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…es-sigs#7324) Bumps node from 24-slim to 25-slim. --- updated-dependencies: - dependency-name: node dependency-version: 25-slim dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* E2e test for hotswap with slices * Address comments * Fix after merge * Timeout -> LongTimeout * Address comments
…feature gate is turned on (kubernetes-sigs#7346) * MultiKueueManagedByJob is failing when jobs manage a pod with pod integration * add cache to options
* Fix workload re-evaluation when clusters added to MultiKueueConfig Add handler to watch MultiKueueConfig changes and re-queue affected workloads for re-evaluation. Previously, existing workloads would not see newly added clusters until they were manually re-queued. Changes: - Add mkConfig event handler to watch MultiKueueConfig updates - Add workload indexer by admission check for efficient queries - Update integration test to validate re-evaluation behavior - Add unit tests for handler logic and indexer functionality The handler uses indexed queries to find workloads affected by config changes. Fixes workload re-evaluation bug where existing pending workloads wouldn't dispatch to newly available worker clusters. * Changes requested
…kload (kubernetes-sigs#7318) * Enable conversion webhooks for v1beta2 * Refactor the test to prepare for workloads * Add conversion for workload
Signed-off-by: Yuki Iwai <[email protected]>
Signed-off-by: Yuki Iwai <[email protected]>
…sigs#7337) * Deprecate LocalQueueFlavorStatus for v1beta2 * Generate manifests and helm and improve deprecation comments * Fix Linter * Fix Linter * Fix a typo * Fix typo after main branch merge * remove comments from generated files
Signed-off-by: Harshal Patil <[email protected]>
* Copy v1beta1 to v1beta2. * v1beta2: graduate Config API * Add conversion logic. * Keep one test with v1beta1 version.
…ics (kubernetes-sigs#7338) * use float instead of int in cluster_queue_weighted_share metric and add cohort label * don't use two fields for weighted share * adjust metric test util to the changes * make ExpectClusterQueueWeightedShareMetric accept float64 as value * adjust integration test * report NaN instead of max_int when weight is 0 * remove unused imports in e2e tests * use float instead of int in cohort_weighted_share metric * fix format and naming cleanup
…rnetes-sigs#7399) Bumps e2e-test-images/agnhost from 2.57 to 2.59. --- updated-dependencies: - dependency-name: e2e-test-images/agnhost dependency-version: '2.59' dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…rnetes-sigs#7488) fix violations but do not convert values to pointers if suggested Assisted with Claude <[email protected]>
Signed-off-by: Harshal Patil <[email protected]>
…kubernetes-sigs#7490) The deprecated .spec.admissionChecks field has been removed from the v1beta2 ClusterQueue API as part of the v1beta2 stabilization effort. All usage has been migrated to .spec.admissionChecksStrategy. Changes: - Removed AdmissionChecks field from v1beta2 ClusterQueueSpec - Added bidirectional conversion logic between v1beta1 and v1beta2 - Updated controllers to use AdmissionChecksStrategy exclusively - Removed webhook validation for the deprecated field - Updated integration tests to use the new field - Regenerated CRDs, client-go, and documentation ```release-note Remove deprecated .spec.admissionChecks field from v1beta2 ClusterQueue API. Use .spec.admissionChecksStrategy instead. ```
Signed-off-by: Yuki Iwai <[email protected]>
…rnetes-sigs#7567) * fix: ClusterQueue conversion for FlavorFungibility and AdmissionChecks Run autoConvert before modifying fields to ensure structs are populated. Fixes Preempt/Borrow->MayStopSearch and AdmissionChecks conversions. Signed-off-by: Sohan Kunkerkar <[email protected]> * test: Add conversion tests for LocalQueue and ClusterQueue Signed-off-by: Sohan Kunkerkar <[email protected]> --------- Signed-off-by: Sohan Kunkerkar <[email protected]>
…etes-sigs#7568) Bumps [github.com/containerd/containerd](https://github.com/containerd/containerd) from 1.7.28 to 1.7.29. - [Release notes](https://github.com/containerd/containerd/releases) - [Changelog](https://github.com/containerd/containerd/blob/main/RELEASES.md) - [Commits](containerd/containerd@v1.7.28...v1.7.29) --- updated-dependencies: - dependency-name: github.com/containerd/containerd dependency-version: 1.7.29 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…7564) * Update e2e tests to use utiltesting alias * Update remaining utiltesting alias in intergation tests
…sigs#5361) * Enable MultiKueue + TAS + ProvisioningRequest integration * Avoid leaking topology assignment details to manager cluster * Remove runtime node labeling for TAS in MultiKueue e2e tests * Refactoring
…ueue (kubernetes-sigs#7533) * Docs for kf trainer v2 Signed-off-by: narayanasabari <[email protected]> * updated on trainjobs Signed-off-by: narayanasabari <[email protected]> * Updated docs on andreyvelich feedback * update render for overview * Restructure docs: Move TrainJob to dedicated section Signed-off-by: narayanasabari <[email protected]> * update docker image for trainjob Signed-off-by: narayanasabari <[email protected]> * File Structure and batch user updated for trainjobs * Update docker image for ClusterTrainingRuntime Signed-off-by: narayanasabari <[email protected]> * Added docs for multi-cluster trainjobs Signed-off-by: narayanasabari <[email protected]> --------- Signed-off-by: narayanasabari <[email protected]>
…rnetes-sigs#7540) * v1beta2: change the API for Workload's spec.priorityClassSource * Use defaultObjectMeta.
Contributor
|
Keywords which can automatically close issues and at(@) or hashtag(#) mentions are not allowed in commit messages. The list of commits with invalid commit messages:
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
added
size/XL
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
24 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.
What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?