Skip to content

Ritika/headlessservice#50

Merged
unmarshall merged 2 commits into
ai-dynamo:mainfrom
ritikasrivastava:ritika/headlessservice
Apr 10, 2025
Merged

Ritika/headlessservice#50
unmarshall merged 2 commits into
ai-dynamo:mainfrom
ritikasrivastava:ritika/headlessservice

Conversation

@ritikasrivastava

Copy link
Copy Markdown
Contributor

No description provided.

@ritikasrivastava ritikasrivastava marked this pull request as ready for review March 21, 2025 05:08
@unmarshall

Copy link
Copy Markdown
Collaborator

Please rebase this PR. Then i can look at it as that will also allow me to test it.

@ritikasrivastava

Copy link
Copy Markdown
Contributor Author

Please rebase this PR. Then i can look at it as that will also allow me to test it.

Rebased and updated

Comment thread operator/internal/component/pgs/service/service.go Outdated
Comment thread operator/internal/component/pgs/service/service.go Outdated
Comment thread operator/internal/component/pgs/service/service.go Outdated
Comment thread operator/internal/component/pgs/service/service.go Outdated
@ritikasrivastava ritikasrivastava force-pushed the ritika/headlessservice branch 2 times, most recently from 46513aa to 58efe8a Compare April 10, 2025 00:21
Remove bool CreateHeadlessService variable
If Service Spec is specified, headless service will be created

Refactor the Pod Clique and Pod naming convention
PC name : <PGS.Name>-<PGS.ReplicaID>-<PC.Name>
Pod name : <PC.Name>-<PC.ReplicaID>

Signed-off-by: Ritika Srivastava <risrivastava@nvidia.com>
Signed-off-by: Ritika Srivastava <risrivastava@nvidia.com>
@unmarshall unmarshall merged commit 193c28b into ai-dynamo:main Apr 10, 2025
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