defenseunicorns / uds-cli

Apache License 2.0
24 stars 11 forks source link

Ensure smoke tests cover both pre-existing cluster and no pre-existing cluster test cases #940

Closed UncleGedd closed 12 hours ago

UncleGedd commented 4 days ago

Context

When we attempted to release the UDS state feature, our release pipeline caught a bug where state was being created in a cluster that existed before deploying the slim-dev bundle.

This bug should have been caught sooner by the nightly pipelines but was not. Noting that the bug was caught because our release pipeline starts up a cluster using the k3d action which deploys an empty k3d cluster.

Describe what should be investigated or refactored

Smoke tests in our release pipeline vs nightly-uds-core are slightly different.

There are 2 scenarios that should be tested in both pipelines:

  1. Cluster already exists when we deploy slim-dev
  2. Cluster does not already exist when we deploy slim-dev

Update the smoke tests to cover the above scenarios in both nightly and release pipelines (can potentially create a resuable workflow to avoid duplicates configs)