Skip to content

Conversation

@enoodle
Copy link
Collaborator

@enoodle enoodle commented Aug 30, 2025

To simplify creating objects from those CRDs by the helm chart, we will add them to the chart which means that they will be created when the chart is first installed.
The crd-upgrader will still run to upgrade the CRDs .

@enoodle enoodle force-pushed the erez/move-crds-into-chart branch from 4e61d79 to 6133adf Compare August 30, 2025 23:55
@github-actions
Copy link

Merging this branch will decrease overall coverage

Impacted Packages Coverage Δ 🤖
github.com/NVIDIA/KAI-scheduler/pkg/binder/controllers/integration_tests 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests 70.51% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/podgrouper 47.37% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers 83.72% (-2.33%) 👎

Coverage by file

Changed unit test files

  • github.com/NVIDIA/KAI-scheduler/pkg/binder/controllers/integration_tests/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/env-tests/env_tests_suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/podgrouper/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/suite_test.go

SiorMeir
SiorMeir previously approved these changes Aug 31, 2025
@github-actions
Copy link

Merging this branch will increase overall coverage

Impacted Packages Coverage Δ 🤖
github.com/NVIDIA/KAI-scheduler/pkg/binder/controllers/integration_tests 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests 71.79% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/podgrouper 47.37% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers 86.05% (+2.33%) 👍

Coverage by file

Changed unit test files

  • github.com/NVIDIA/KAI-scheduler/pkg/binder/controllers/integration_tests/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/env-tests/env_tests_suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/podgrouper/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/suite_test.go

@enoodle enoodle force-pushed the erez/move-crds-into-chart branch from abfa66e to b190db1 Compare September 1, 2025 07:47
@enoodle enoodle force-pushed the erez/move-crds-into-chart branch from b190db1 to 4271746 Compare September 1, 2025 14:02
@github-actions
Copy link

github-actions bot commented Sep 1, 2025

Merging this branch changes the coverage (1 decrease, 1 increase)

Impacted Packages Coverage Δ 🤖
github.com/NVIDIA/KAI-scheduler/pkg/binder/controllers/integration_tests 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests 70.51% (-1.28%) 👎
github.com/NVIDIA/KAI-scheduler/pkg/podgrouper 47.37% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers 86.05% (+2.33%) 👍

Coverage by file

Changed unit test files

  • github.com/NVIDIA/KAI-scheduler/pkg/binder/controllers/integration_tests/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/env-tests/env_tests_suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/podgrouper/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/suite_test.go

@enoodle enoodle force-pushed the erez/move-crds-into-chart branch from 4271746 to 3eb0f4a Compare September 2, 2025 06:59
@github-actions
Copy link

github-actions bot commented Sep 2, 2025

Merging this branch will increase overall coverage

Impacted Packages Coverage Δ 🤖
github.com/NVIDIA/KAI-scheduler/pkg/binder/controllers/integration_tests 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests 71.79% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/podgrouper 47.37% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers 86.05% (+2.33%) 👍

Coverage by file

Changed unit test files

  • github.com/NVIDIA/KAI-scheduler/pkg/binder/controllers/integration_tests/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/env-tests/env_tests_suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/podgrouper/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/suite_test.go

romanbaron
romanbaron previously approved these changes Sep 2, 2025
@github-actions
Copy link

github-actions bot commented Sep 2, 2025

Merging this branch will not change overall coverage

Impacted Packages Coverage Δ 🤖
github.com/NVIDIA/KAI-scheduler/pkg/binder/controllers/integration_tests 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests 71.79% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/podgrouper 47.37% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers 86.05% (ø)

Coverage by file

Changed unit test files

  • github.com/NVIDIA/KAI-scheduler/pkg/binder/controllers/integration_tests/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/env-tests/env_tests_suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/podgrouper/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/suite_test.go

@enoodle enoodle merged commit 1ddba0f into main Sep 2, 2025
4 checks passed
@enoodle enoodle deleted the erez/move-crds-into-chart branch September 2, 2025 09:22
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.

4 participants