Skip to content

workflows: -new-gke.tfvars -> -gke.tfvars - #960

Merged
fajpunk merged 1 commit into
mainfrom
u/danfuchs/no-longer-new
Oct 8, 2026
Merged

fajpunk merged 1 commit into
mainfrom
u/danfuchs/no-longer-new

Conversation

@fajpunk

@fajpunk fajpunk commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Now that all of the clusters have been rebuilt and the old clusters destroyed, the existing clusters are no longer "new"

@fajpunk
fajpunk requested a review from dspeck1 October 5, 2026 15:06
@fajpunk
fajpunk force-pushed the u/danfuchs/no-longer-new branch from b605b3d to b4a4ccc Compare October 5, 2026 15:09

@dspeck1 dspeck1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. The RSP Int plan had changes to user lab pool to add more zones. I would consider breaking Prod into a separate change to give flexibility on the apply. The state prefixes still have the new path. That is fine. Just noting in case you wanted to remove the old state prefixes and move the new into those paths.

@fajpunk

fajpunk commented Oct 5, 2026

Copy link
Copy Markdown
Contributor Author

Yeah, I decided not to mess with the state prefixes. There's another PR that enshrines the node pool AZ changes, I'm fine reverting them when I merge this if that other PR isn't merged by then. I'm planning on merging this during the patch window, so I think we're good keeping it as one change for now.

@fajpunk
fajpunk force-pushed the u/danfuchs/no-longer-new branch 2 times, most recently from 785f197 to 07b80e3 Compare October 8, 2026 22:16
Now that all of the clusters have been rebuilt and the old clusters
destroyed, the existing clusters are no longer "new"
@fajpunk
fajpunk force-pushed the u/danfuchs/no-longer-new branch from 07b80e3 to 91bc561 Compare October 8, 2026 22:18
@fajpunk
fajpunk merged commit c819cc4 into main Oct 8, 2026
6 checks passed
@fajpunk
fajpunk deleted the u/danfuchs/no-longer-new branch October 8, 2026 22:21
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.

2 participants