-
Notifications
You must be signed in to change notification settings - Fork 301
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.
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
document manageJobsNamespaceSelector #3748
Conversation
✅ Deploy Preview for kubernetes-sigs-kueue ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I created a grouping for this topic on the web pages because I think it would make sense for the documentation for #2936 to be added as a a third file in site/content/en/docs/tasks/manage/enforce_job_management/. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just nits
site/content/en/docs/tasks/manage/enforce_job_management/manage_jobs_without_queue_name.md
Outdated
Show resolved
Hide resolved
site/content/en/docs/tasks/manage/enforce_job_management/manage_jobs_without_queue_name.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Michał Woźniak <mimowo@users.noreply.github.com>
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dgrove-oss, 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 |
LGTM label has been added. Git tree hash: c7dec7cc9b08775eb304a415ce534c832ffcf25f
|
* chore: use variadic function to set levels in TopologyWrapper (kubernetes-sigs#3744) * Use QueueNameForObject in StatefulSet Webhook. (kubernetes-sigs#3752) * TAS: Support rank-based ordering for StatefulSet. (kubernetes-sigs#3751) * TAS: Support rank-based ordering for StatefulSet. * Review remarks. * Sort labels and annotations. (kubernetes-sigs#3753) * document manageJobsNamespaceSelector (kubernetes-sigs#3748) * document manageJobsNamespaceSelector * Apply suggestions from code review Co-authored-by: Michał Woźniak <mimowo@users.noreply.github.com> * use full name for queue-name label --------- Co-authored-by: Michał Woźniak <mimowo@users.noreply.github.com> * performance optimization: only copy back when changes are made (kubernetes-sigs#3766) * managedJobsNamespaceSelector for Deployment, StatefulSets, and Pods (kubernetes-sigs#3765) * managedJobsNamespaceSelector for deployment and statefulset * normalize manageJobsWithoutQueueName logic in pod webhook * refactor to use common helper function * unit tests for WorkloadShouldBeSuspended * linter fixes * add test for job owned by kueue managed parent * bug fix in deployment/statefulset webhooks * Bump github.com/mikefarah/yq/v4 in /hack/internal/tools (kubernetes-sigs#3768) Bumps [github.com/mikefarah/yq/v4](https://github.com/mikefarah/yq) from 4.44.5 to 4.44.6. - [Release notes](https://github.com/mikefarah/yq/releases) - [Changelog](https://github.com/mikefarah/yq/blob/master/release_notes.txt) - [Commits](mikefarah/yq@v4.44.5...v4.44.6) --- updated-dependencies: - dependency-name: github.com/mikefarah/yq/v4 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Remove Denkensk from Kueue reviewers (kubernetes-sigs#3771) * Replace time.Now() with clock.Now() in pkg/core (kubernetes-sigs#3773) * add clock field to ClusterQueueReconciler and propagate clock into it * replace time.Now with Now() in ClusterQueueReconciler * fix lint errors * Implement default LocalQueue (kubernetes-sigs#3610) * Make sure test does not flake. (kubernetes-sigs#3734) * Refresh Kueue roadmap on the main page for 2025 (kubernetes-sigs#3763) * Refresh Kueue roadmap on the main page * Graduate the API to v1 * Review remarks * use node 22 instead of node 16 (kubernetes-sigs#3777) * README.md: Fix some minor typos (kubernetes-sigs#3781) Fix some minor typos in README.md. Signed-off-by: David Weinehall <david.weinehall@intel.com> * Add documentation for ProvReqRetry mechanism (kubernetes-sigs#3774) * Add doc for ProvReqRetry mechanism * Fix formula * Delete feature gate note as it's deprecated * Configure dependabot for kueue-viz. (kubernetes-sigs#3780) * Bump node from 22 to 23 in /cmd/experimental/kueue-viz/frontend (kubernetes-sigs#3784) Bumps node from 22 to 23. --- updated-dependencies: - dependency-name: node dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump github.com/onsi/gomega from 1.36.0 to 1.36.1 (kubernetes-sigs#3786) Bumps [github.com/onsi/gomega](https://github.com/onsi/gomega) from 1.36.0 to 1.36.1. - [Release notes](https://github.com/onsi/gomega/releases) - [Changelog](https://github.com/onsi/gomega/blob/master/CHANGELOG.md) - [Commits](onsi/gomega@v1.36.0...v1.36.1) --- updated-dependencies: - dependency-name: github.com/onsi/gomega dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump react-toastify in /cmd/experimental/kueue-viz/frontend (kubernetes-sigs#3787) Bumps [react-toastify](https://github.com/fkhadra/react-toastify) from 9.1.3 to 10.0.6. - [Release notes](https://github.com/fkhadra/react-toastify/releases) - [Commits](fkhadra/react-toastify@v9.1.3...v10.0.6) --- updated-dependencies: - dependency-name: react-toastify dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump react-chartjs-2 in /cmd/experimental/kueue-viz/frontend (kubernetes-sigs#3788) Bumps [react-chartjs-2](https://github.com/reactchartjs/react-chartjs-2) from 4.3.1 to 5.0.0. - [Release notes](https://github.com/reactchartjs/react-chartjs-2/releases) - [Changelog](https://github.com/reactchartjs/react-chartjs-2/blob/master/CHANGELOG.md) - [Commits](reactchartjs/react-chartjs-2@v4.3.1...v5.0.0) --- updated-dependencies: - dependency-name: react-chartjs-2 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump github.com/gohugoio/hugo in /hack/internal/tools (kubernetes-sigs#3789) Bumps [github.com/gohugoio/hugo](https://github.com/gohugoio/hugo) from 0.139.3 to 0.139.4. - [Release notes](https://github.com/gohugoio/hugo/releases) - [Changelog](https://github.com/gohugoio/hugo/blob/master/hugoreleaser.toml) - [Commits](gohugoio/hugo@v0.139.3...v0.139.4) --- updated-dependencies: - dependency-name: github.com/gohugoio/hugo dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump axios from 0.27.2 to 1.7.9 in /cmd/experimental/kueue-viz/frontend (kubernetes-sigs#3790) Bumps [axios](https://github.com/axios/axios) from 0.27.2 to 1.7.9. - [Release notes](https://github.com/axios/axios/releases) - [Changelog](https://github.com/axios/axios/blob/v1.x/CHANGELOG.md) - [Commits](axios/axios@v0.27.2...v1.7.9) --- updated-dependencies: - dependency-name: axios dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump react from 18.3.1 to 19.0.0 in /cmd/experimental/kueue-viz/frontend (kubernetes-sigs#3791) Bumps [react](https://github.com/facebook/react/tree/HEAD/packages/react) from 18.3.1 to 19.0.0. - [Release notes](https://github.com/facebook/react/releases) - [Changelog](https://github.com/facebook/react/blob/main/CHANGELOG.md) - [Commits](https://github.com/facebook/react/commits/v19.0.0/packages/react) --- updated-dependencies: - dependency-name: react dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump babel/plugin-proposal-private-property-in-object (kubernetes-sigs#3793) Bumps the all group in /cmd/experimental/kueue-viz/frontend with 1 update: [babel/plugin-proposal-private-property-in-object](https://github.com/babel/babel/tree/HEAD/packages/babel-plugin-proposal-private-property-in-object). Updates `babel/plugin-proposal-private-property-in-object` from 7.18.6 to 7.21.11 - [Release notes](https://github.com/babel/babel/releases) - [Commits](https://github.com/babel/babel/commits/HEAD/packages/babel-plugin-proposal-private-property-in-object) --- updated-dependencies: - dependency-name: "babel/plugin-proposal-private-property-in-object" dependency-type: direct:development update-type: version-update:semver-minor dependency-group: all ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [website] Add Innovatrics adopter (kubernetes-sigs#3800) * Migrate to React 19 (kubernetes-sigs#3799) * Update resource group example (kubernetes-sigs#3806) Signed-off-by: Troy Chiu <y.troychiu@gmail.com> * Do not default the queue-name for jobs with kueue managed owners (kubernetes-sigs#3795) * chore: add default topologies for tests (kubernetes-sigs#3769) * Bump actions/setup-go from 5.1.0 to 5.2.0 in the all group (kubernetes-sigs#3808) Bumps the all group with 1 update: [actions/setup-go](https://github.com/actions/setup-go). Updates `actions/setup-go` from 5.1.0 to 5.2.0 - [Release notes](https://github.com/actions/setup-go/releases) - [Commits](actions/setup-go@41dfa10...3041bf5) --- updated-dependencies: - dependency-name: actions/setup-go dependency-type: direct:production update-type: version-update:semver-minor dependency-group: all ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * add configuration for setting feature gates (kubernetes-sigs#3805) * generate should generate kueuectl and apiref rather than verify (kubernetes-sigs#3810) * update kuberbac proxy defaults (kubernetes-sigs#3811) * fix default suspension of Deployment/StatefulSet with Kueue-managed owners (kubernetes-sigs#3803) * fix deployment and statefulset setup so ownership works * enable deployment/statefulset to be recognized as child jobs * fix typo in rbacs * adjust podwebhook warning for forced suspension * redesign special pod/deployment/statefulset ownership logic * revert trivial change * linter fix * nit: grammar * Deactivate ClusterQueue if there is no Topology (kubernetes-sigs#3770) * Deactivate ClusterQueue if there is no Topology * Review remarks * Use lock when accessing IsIntegrationEnabled (kubernetes-sigs#3814) * Add kueue.x-k8s.io/pod-suspending-parent to the documentation page. (kubernetes-sigs#3816) * Use annotation rather than label for serving workloads (kubernetes-sigs#3815) * Documentation for LocalQueueDefaulting (kubernetes-sigs#3783) Update site/content/en/docs/reference/labels-and-annotations.md Co-authored-by: Michał Woźniak <mimowo@users.noreply.github.com> * avoid hardcoding framework names in pod wehook (kubernetes-sigs#3817) * remove hardcoding in pod webhook using GVK to look up framework * add unit tests * typo: match function name in comment * update Pod integration docs to prefer managedJobsNamespaceSelector (kubernetes-sigs#3828) * add section to metrics with (alpha) LQ metrics (kubernetes-sigs#3826) Signed-off-by: Kevin <kpostlet@redhat.com> * Added Log output for LocalQueue and ClusterQueue (kubernetes-sigs#3605) * Added Log output for LocalQueue and ClusterQueue Signed-off-by: 7h3-3mp7y-m4n <emailtorash@gmail.com> * Added changes Signed-off-by: 7h3-3mp7y-m4n <emailtorash@gmail.com> * added println Signed-off-by: 7h3-3mp7y-m4n <emailtorash@gmail.com> * updated the log output using default error Signed-off-by: 7h3-3mp7y-m4n <emailtorash@gmail.com> * Added test case Signed-off-by: 7h3-3mp7y-m4n <emailtorash@gmail.com> * Added changes Signed-off-by: 7h3-3mp7y-m4n <emailtorash@gmail.com> * Adding return statements for ci-lint Signed-off-by: 7h3-3mp7y-m4n <emailtorash@gmail.com> * removed comments Signed-off-by: 7h3-3mp7y-m4n <emailtorash@gmail.com> --------- Signed-off-by: 7h3-3mp7y-m4n <emailtorash@gmail.com> * update golang crypto to 0.31.0 (kubernetes-sigs#3825) * drop pod taints toleration from kueue repo (kubernetes-sigs#3834) * Check if unexpected errors did not occur in queue unit testings (kubernetes-sigs#3838) Signed-off-by: Yuki Iwai <yuki.iwai.tz@gmail.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: David Weinehall <david.weinehall@intel.com> Signed-off-by: Troy Chiu <y.troychiu@gmail.com> Signed-off-by: Kevin <kpostlet@redhat.com> Signed-off-by: 7h3-3mp7y-m4n <emailtorash@gmail.com> Signed-off-by: Yuki Iwai <yuki.iwai.tz@gmail.com> Co-authored-by: Mykyta Derhunov <ndergunov@gmail.com> Co-authored-by: Mykhailo Bobrovskyi <mikhail.bobrovsky@gmail.com> Co-authored-by: David Grove <dgrove-oss@users.noreply.github.com> Co-authored-by: Michał Woźniak <mimowo@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Patryk Bundyra <pbundyra@google.com> Co-authored-by: Tushar Mohapatra <137442734+TusharMohapatra07@users.noreply.github.com> Co-authored-by: Yaroslava Serdiuk <yaroslava@google.com> Co-authored-by: Michał Szadkowski <michal_szadkowski@epam.com> Co-authored-by: David Weinehall <david.weinehall@intel.com> Co-authored-by: mmolisch <141235459+mmolisch@users.noreply.github.com> Co-authored-by: Akram Ben Aissi <akram.benaissi@gmail.com> Co-authored-by: Troy Chiu <114708546+troychiu@users.noreply.github.com> Co-authored-by: Kevin Postlethwait <kpostlet@redhat.com> Co-authored-by: 7h3-3mp7y-m4n <115151332+7h3-3mp7y-m4n@users.noreply.github.com> Co-authored-by: Yuki Iwai <yuki.iwai.tz@gmail.com>
What type of PR is this?
/kind documentation
What this PR does / why we need it:
Document how to use
manageJobsNamespaceSelector
in conjunction withmanageJobsWithoutQueueName
Which issue(s) this PR fixes:
Part of #3589.
Special notes for your reviewer:
Does this PR introduce a user-facing change?