Skip to content

Misc fixes and partial implementation of Pod component#90

Merged
unmarshall merged 2 commits into
ai-dynamo:mainfrom
unmarshall:reconcilers
Jun 17, 2025
Merged

Misc fixes and partial implementation of Pod component#90
unmarshall merged 2 commits into
ai-dynamo:mainfrom
unmarshall:reconcilers

Conversation

@unmarshall

@unmarshall unmarshall commented Jun 17, 2025

Copy link
Copy Markdown
Collaborator
  • Partially implements Pod component. With this commit pods without scheduling gates can be created.
  • Enables Pod component in the PodClique reconciler.
  • Fixed HPA component which now correctly sets the target resource ref.
  • Fixed service component, which now creates a headless service.
  • Introduced some convenient functions.
  • Changed the pgs-replica-index label key as it was not as per allowed conventions.
  • Fixes Makefile targets
  • Fixed formatting and linting issues
  • Adds missing license header files

* Partially implements Pod component. With this commit pods without
  scheduling gates can be created.
* Enables Pod component in the PodClique reconciler.
* Fixed HPA component which now correctly sets the target resource ref.
* Fixed service component, which now creates a headless service.
* Introduced some convenient functions.
* Changed the pgs-replica-index label key as it was not as per allowed
  conventions.

Signed-off-by: madhav bhargava <madhav.bhargava@sap.com>
@unmarshall unmarshall requested review from renormalize and sanjaychatterjee and removed request for dmitsh and sanjaychatterjee June 17, 2025 08:39
* Added missing license headers to new files
* Fixed linting issues
* Fixed formatting issues

Signed-off-by: madhav bhargava <madhav.bhargava@sap.com>
Comment thread operator/internal/component/pclq/pod/pod.go

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

thanks for bringing in these changes!

@unmarshall unmarshall merged commit 8f4ac6f into ai-dynamo:main Jun 17, 2025
1 check passed
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