Skip to content

kueue: Add Cohort views - #1004

Open
Utkarshpandey0001 wants to merge 3 commits into
headlamp-k8s:mainfrom
Utkarshpandey0001:kueue-cohort-views
Open

kueue: Add Cohort views#1004
Utkarshpandey0001 wants to merge 3 commits into
headlamp-k8s:mainfrom
Utkarshpandey0001:kueue-cohort-views

Conversation

@Utkarshpandey0001

@Utkarshpandey0001 Utkarshpandey0001 commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a typed Cohort resource model.
  • Add Cohort list and detail pages.
  • Add parent and child Cohort navigation.
  • Add Cohort-to-ClusterQueue relationship navigation.
  • Add ClusterQueue-to-Cohort navigation.
  • Add sidebar entries and routes.
  • Add test manifests, tests, and README documentation.

Scope

This implementation is read-only.

Branch status

This PR has been unstacked and rebased onto the latest upstream main. It no longer includes the Workload PR commits.

Validation

  • npm run format
  • npm run lint
  • npm run tsc
  • npm run build
  • CI=true npm run test

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds read-only Cohort support and relationship navigation to the Kueue plugin.

Changes:

  • Adds typed Cohort resources, formatting, relationships, tests, and manifests.
  • Adds Cohort list/detail pages, routes, and sidebar navigation.
  • Links Cohorts with parent/child Cohorts, ClusterQueues, and ResourceFlavors.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
kueue/test-files/deploy/cohort-default.yaml Adds sample Cohorts.
kueue/src/utils/kueueRoutes.ts Defines Cohort routes.
kueue/src/utils/kueueApi.test.ts Tests Cohort route paths.
kueue/src/resources/cohortRelations.ts Derives Cohort relationships.
kueue/src/resources/cohortRelations.test.ts Tests relationship helpers.
kueue/src/resources/cohortFormatters.ts Adds Cohort display formatters.
kueue/src/resources/cohortFormatters.test.ts Tests Cohort formatting.
kueue/src/resources/cohort.ts Defines the typed Cohort model.
kueue/src/index.tsx Registers Cohort navigation and pages.
kueue/src/components/cohorts/List.tsx Implements the Cohort list.
kueue/src/components/cohorts/Detail.tsx Implements Cohort details and relationships.
kueue/src/components/clusterqueues/List.tsx Links ClusterQueues to Cohorts.
kueue/src/components/clusterqueues/Detail.tsx Adds Cohort detail navigation.
kueue/README.md Documents Cohort support and setup.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread kueue/src/components/cohorts/List.tsx Outdated
Comment thread kueue/src/components/cohorts/List.tsx Outdated
Comment thread kueue/src/components/cohorts/Detail.tsx Outdated
Comment thread kueue/src/components/cohorts/List.tsx Outdated
Comment thread kueue/src/components/clusterqueues/Detail.tsx Outdated
Signed-off-by: Utkarshpandey0001 <rajutkarshpandey2003@gmail.com>
Signed-off-by: Utkarshpandey0001 <rajutkarshpandey2003@gmail.com>
@Utkarshpandey0001

Copy link
Copy Markdown
Contributor Author

@kannon92 Hi , i have removed duplicating function from list.tsx and made it to reuse the existing. And also resolved the open comment. Could u pls review it once

Comment thread kueue/src/components/cohorts/Detail.tsx Outdated
Signed-off-by: Utkarshpandey0001 <rajutkarshpandey2003@gmail.com>
@Utkarshpandey0001

Copy link
Copy Markdown
Contributor Author

@kannon92 i have removed almost all duplicated functions. Could u pls check ?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants