What problem are you facing?
The deployment function needs to match all InferenceEnvironments and all ModelPlacements as required resources. Crossplane's required resources API doesn't support matching all resources of a kind without a label selector (empty match_labels hits a protobuf bug), so we work around this by requiring presence labels:
The IE label is the worse of the two — it's a mandatory label with no semantic meaning that the platform team has to remember on every InferenceEnvironment.
How could Modelplane help solve your problem?
Once crossplane/crossplane#7241 is backported and released, drop both presence labels and use empty match_labels to match all resources of a kind.
What problem are you facing?
The deployment function needs to match all InferenceEnvironments and all ModelPlacements as required resources. Crossplane's required resources API doesn't support matching all resources of a kind without a label selector (empty
match_labelshits a protobuf bug), so we work around this by requiring presence labels:modelplane.ai/environment: "true"— the platform team has to add this manually or the scheduler won't find the environment.modelplane.ai/placement: "true"— set by the deployment function when it creates placements, so this one is invisible to users.The IE label is the worse of the two — it's a mandatory label with no semantic meaning that the platform team has to remember on every InferenceEnvironment.
How could Modelplane help solve your problem?
Once crossplane/crossplane#7241 is backported and released, drop both presence labels and use empty
match_labelsto match all resources of a kind.