pulumi / pulumi-kubernetes-operator

A Kubernetes Operator that automates the deployment of Pulumi Stacks
Apache License 2.0
221 stars 55 forks source link

Fix broken tests #449

Closed rquitales closed 1 year ago

rquitales commented 1 year ago

This PR addresses 3 outstanding issues with the current broken tests:

  1. Pulumi Yaml Program test is flakey as specs within it interact with ConfigMaps of the same name. We can run this sequentially, but I've fixed this by dynamically generating the require Pulumi Program CRs and randomising ConfigMap names.
  2. The stale state test is flakey and needs to be run sequentially by decorating it with Gingko's Ordered decorator.
  3. Usage of pulumi-kubernetes is pinned to v3.26.0 of the provider. v3.27.0 was recently released and caused our previously passing tests to fail (see run history for this commit). v3.27.0 switched to using foreground cascading deletion, which is causing our stacks to fail deletion on finalization. I'm pinning to v3.26.0 for now to unblock CI and releases, until we can determine a better solution going forwards.