pulumi-staging[bot]
commented
1 year ago :tropical_drink: The Update for moolumi/k8s-ci-cluster/cb32be60cb4ba67f106ddcc6f9c178c032167ce4-938 was successful.
Resource Changes
Name Type Operation
+ multicloud pulumi-kubernetes:ci:GkeCluster create
+ password random:index/randomPassword:RandomPassword create
+ ephemeral-ci-cluster gcp:container/cluster:Cluster create
+ primary-node-pool gcp:container/nodePool:NodePool create
+ gke pulumi:providers:kubernetes create
+ k8s-ci-cluster-cb32be60cb4ba67f106ddcc6f9c178c032167ce4-938 pulumi:pulumi:Stack create
scottslowe
cnunciato
Fixes #1347. Fixes https://github.com/pulumi/examples/issues/1391.
This change also adds a handful of test-related fixes to get this PR's tests to pass and to clear the way for others going forward. (PR tests had been passing, but apparently began failing again last week.) The gist is that because many of our integration tests seem to produce diffs unexpectedly, we've historically sprinkled lots of exceptions throughout the code to ignore these diffs in order to make the tests pass. I've been propagating this pattern for a little while myself, but I'd rather not do that anymore. Instead, considering this repo is not (or is no longer) concerned with low-level provider behavior like this, and since we don't seem to have done anything to address these issues up to now (other than just adding more exceptions), I've chosen to apply these exceptions to all tests, across the board.