Skip to content

Make TopologyConstraint.TopologyName optional #582

@enoodle

Description

@enoodle

What you would like to be added?

The TopologyName field shouldn't be always required and can be inferred from the PCS / PCSG definitions instead.

https://github.com/ai-dynamo/grove/blob/main/operator/api/core/v1alpha1/podcliqueset.go#L278

This will require some updates in the admission validation of the PCS and the reconciler.

Why is this needed?

Simplifying the API and making it more flexible.

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No fields configured for Task.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions