devfile / api

Kube-native API for cloud development workspaces specification
Apache License 2.0
235 stars 58 forks source link

Include deployment dependencies and readiness checks for registry operator integration test suite #1523

Open michael-valdron opened 1 month ago

michael-valdron commented 1 month ago

/kind epic /kind user-story

Which area this user story is related to?

/area registry

User Story

As a contributor to the registry operator, I want to have the ability to just use make test-integration whenever I want to run integration tests, so that I don't have to spend extra time on environment setup steps.

Currently, the integration test suite for the registry operator does not have a sufficient setup to ensure the registry operator deployment is ready before executing the integration test cases, as a result this leads consistent timeouts when trying to run make test-integration alone. To workaround this, one must do the following steps to ensure the target registry operator deployment is ready before integration test cases do:

  • Run make install-cert and wait for pods in cert-manager namespace.
  • Run make install && make deploy and wait for pods in registry-operator-system namespace.
  • Run make test-integration.

Doing the above will allow the integration test suite to run properly. It would be better though to include the first two steps as part of the setup process of the integration test suite so that you would only need to run make test-integration to execute everything.

Acceptance Criteria