Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't create stacks in the moolumi org #1506

Merged
merged 2 commits into from
Oct 2, 2023
Merged

Conversation

cnunciato
Copy link
Contributor

@cnunciato cnunciato commented Sep 30, 2023

About a month ago, the pulumi user was apparently removed from the moolumi org in staging, which caused the test-infra-setup test (which creates stacks in that org) to begin failing with 404 errors. This change fixes that by removing the explicit moolumi org setting and allowing these stacks to be created in the pulumi user's regular account, as the rest of the tests in this repo do.

Fixes #1503.

@cnunciato cnunciato requested a review from a team September 30, 2023 03:21
@pulumi-staging
Copy link

🍹 The Update for pulumi/k8s-ci-cluster/7ad4e04bcb26e71f4b0a6f22d26cd6f41602e6a7-1529 was successful.

Resource Changes

    Name                                                          Type                                        Operation
+   primary-node-pool                                             gcp:container/nodePool:NodePool             create
+   gke                                                           pulumi:providers:kubernetes                 create
+   k8s-ci-cluster-7ad4e04bcb26e71f4b0a6f22d26cd6f41602e6a7-1529  pulumi:pulumi:Stack                         create
+   multicloud                                                    pulumi-kubernetes:ci:GkeCluster             create
+   password                                                      random:index/randomPassword:RandomPassword  create
+   ephemeral-ci-cluster                                          gcp:container/cluster:Cluster               create

@pulumi-staging
Copy link

🍹 The Destroy for pulumi/k8s-ci-cluster/7ad4e04bcb26e71f4b0a6f22d26cd6f41602e6a7-1529 was successful.

Resource Changes

    Name                  Type                                        Operation
-   gke                   pulumi:providers:kubernetes                 delete
-   primary-node-pool     gcp:container/nodePool:NodePool             delete
-   ephemeral-ci-cluster  gcp:container/cluster:Cluster               delete
-   password              random:index/randomPassword:RandomPassword  delete

Copy link
Contributor

@scottslowe scottslowe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor change to fix a missing quote, otherwise looks good

.github/workflows/smoke-test-provider-command.yml Outdated Show resolved Hide resolved
@cnunciato cnunciato merged commit 68ec39b into master Oct 2, 2023
1 check passed
@cnunciato cnunciato deleted the cnunciato/demoolumify branch October 2, 2023 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Workflow failure: Run Examples Tests From PR
2 participants