Skip to content

Introduced PodCliqueScalingGroup and refactored API modules#78

Merged
renormalize merged 4 commits into
ai-dynamo:mainfrom
unmarshall:operator
May 13, 2025
Merged

Introduced PodCliqueScalingGroup and refactored API modules#78
renormalize merged 4 commits into
ai-dynamo:mainfrom
unmarshall:operator

Conversation

@unmarshall

Copy link
Copy Markdown
Collaborator

This PR introduces the following changes:

  • Introduces PodCliqueScalingGroup custom resource.
  • Adds PodCliqueScalingGroupConfig to PodGangSet.Spec.TemplateSpec.PodCliqueScalingGroupConfigs
  • Moves Operator API to its own go module.
  • Adjusts the Makefile targets.
  • Adapts the Operator code to use the new Operator API module.
  • Changed the structure of scheduler-api go module.
  • Regenerated code.

…or scheduler-api and operator api

Signed-off-by: madhav bhargava <madhav.bhargava@sap.com>
Signed-off-by: madhav bhargava <madhav.bhargava@sap.com>
Signed-off-by: madhav bhargava <madhav.bhargava@sap.com>

@renormalize renormalize 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.

The build is broken. Will push changes to fix that and address these comments.

Comment thread operator/internal/controller/podgangset/register.go
Comment thread operator/go.mod Outdated
Comment thread operator/internal/controller/podgangset/register.go Outdated
Signed-off-by: madhav bhargava <madhav.bhargava@sap.com>
@renormalize renormalize merged commit 07b850c into ai-dynamo:main May 13, 2025
1 check passed
@unmarshall unmarshall deleted the operator branch May 13, 2025 10:27
renormalize added a commit to renormalize/grove that referenced this pull request May 14, 2025
…mo#78.

* Fix broken targets due to reorganization of the repository.

* Add missing license headers.

* Format code.

Signed-off-by: Saketh Kalaga <51327242+renormalize@users.noreply.github.com>
renormalize added a commit to renormalize/grove that referenced this pull request May 14, 2025
…mo#78.

* Fix broken targets due to reorganization of the repository.

* Add missing license headers.

* Format code.

Signed-off-by: Saketh Kalaga <51327242+renormalize@users.noreply.github.com>
renormalize added a commit to renormalize/grove that referenced this pull request May 15, 2025
Signed-off-by: Saketh Kalaga <51327242+renormalize@users.noreply.github.com>
renormalize added a commit that referenced this pull request May 19, 2025
… of `grove-kube-scheduler`. (#80)

* Introduce make targets and charts for the development, and deployment of `grove-kube-scheduler`.

* Introduce the `replace-default-scheduler` target which replaces the
  vanilla kube-scheduler with `grove-kube-scheduler` in the local kind
  cluster.

* Introduce the `deploy-second-scheduler` target to deploy the
  `grove-kube-scheduler` as a second scheduler in the local kind cluster.

* Introduce the `Dockerfile` for the `grove-kube-scheduler`.

* Introduce charts to enable running `grove-kube-scheduler` as a second scheduler.

* Introduce `skaffold.yaml`.

* Enhance more targets `build-%`, `docker-build-%` and corresponding
  shell scripts.

* Add documentation on how to test the kube-scheduler plugin with the
  `replace-default-scheduler` target.

* Fix broken targets due to reorganization of the repository in #78.

* Add missing license headers.

* Format code.

Signed-off-by: Saketh Kalaga <51327242+renormalize@users.noreply.github.com>
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.

2 participants