Skip to content

Conversation

@kannon92
Copy link

@kannon92 kannon92 commented Dec 8, 2025

Apply stale kubeflow/kubeflow#7674 to this v1 branch.

This was a clean pick of opendatahub-io/kubeflow#646.

It looks like this was merged in opendatahub but I didn't see this patch contributed to kubeflow notebooks.

This should provide integration with Kueue based on kubernetes-sigs/kueue#3878 (comment).

@github-project-automation github-project-automation bot moved this to Needs Triage in Kubeflow Notebooks Dec 8, 2025
@google-oss-prow google-oss-prow bot added the area/controller area - related to controller components label Dec 8, 2025
@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign kimwnasptd for approval. For more information see the Kubernetes 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

@google-oss-prow google-oss-prow bot added area/v1 area - version - kubeflow notebooks v1 size/XS labels Dec 8, 2025
@kannon92
Copy link
Author

kannon92 commented Dec 8, 2025

cc @andreyvelich

@kannon92 kannon92 changed the title enable kueue integration by ensuing that labels added to Notebooks are propogated to the underlying StatefulSet feat: enable kueue integration by ensuing that labels added to Notebooks are propogated to the underlying StatefulSet Dec 8, 2025
@kannon92 kannon92 force-pushed the propogate-notebook-labels-kueue-integration branch from 62f2472 to 5b77921 Compare December 8, 2025 18:04
@google-oss-prow google-oss-prow bot added size/M and removed size/XS labels Dec 8, 2025
@google-oss-prow google-oss-prow bot added size/S and removed size/M labels Dec 8, 2025
@kannon92 kannon92 force-pushed the propogate-notebook-labels-kueue-integration branch from e1d2a72 to c2f5760 Compare December 8, 2025 18:46
@andreyvelich
Copy link
Member

Thank you @kannon92, looking forward to integrate Notebooks and Kueue!

/assign @kubeflow/wg-notebooks-leads @andyatmiami @ederign

…e propogated to the underlying StatefulSet

Signed-off-by: Kevin Hannon <kehannon@redhat.com>
@kannon92 kannon92 force-pushed the propogate-notebook-labels-kueue-integration branch from c2f5760 to 4da8e98 Compare December 8, 2025 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/controller area - related to controller components area/v1 area - version - kubeflow notebooks v1 size/S

Projects

Status: Needs Triage

Development

Successfully merging this pull request may close these issues.

3 participants