opendatahub-io / model-registry-operator

Apache License 2.0
3 stars 17 forks source link

fix: start condition check for deployment with the assumption that it is not available, fixes RHOAIENG-9184 #111

Closed dhirajsb closed 2 months ago

dhirajsb commented 2 months ago

Description

Start by assuming that DeploymentAvailable condition is missing/not available, i.e. only set MR status to available when deployment is actually available. This can happen when the Deployment is initialized in K8s but without any of it's conditions set. Fixes RHOAIENG-9184

How Has This Been Tested?

Tested locally by waiting for availability right away by using the command:

kubectl apply -k config/samples/postgres/; kubectl wait --for=condition=Available=true modelregistries/modelregistry-sample --timeout=5m

Merge criteria:

dhirajsb commented 2 months ago

Ah, need to fix the integration test, since deployment status will never be true in setup-env test env.