Skip to content

Commit ff20c05

Browse files
[CI][Docs] Minor fixes in cluster-management.md
There were a couple misses from when I last touched the docs that made them invalid in some subtle ways (mainly some slightly incorrect TF target names). This patch fixes that to make them easier to copy and paste.
1 parent c17ec7d commit ff20c05

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

premerge/cluster-management.md

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -134,15 +134,16 @@ ours.
134134
### Uninstalling the Helm Charts
135135

136136
For the example commands below we will be modifying the cluster in
137-
`us-central1-a`. You can replace `module.premerge_cluster_us_central` with
138-
`module.premerge_cluster_us_west` to switch which cluster you are working on.
137+
`us-central1-a`. You can replace `module.premerge_cluster_us_central_resources`
138+
with `module.premerge_cluster_us_west_resources` to switch which cluster you
139+
are working on.
139140

140141
To begin, start by uninstalling the helm charts by using resource targetting
141142
on a kubernetes destroy command:
142143

143144
```bash
144-
terraform destroy -target module.premerge_cluster_us_central.helm_release.github_actions_runner_set_linux
145-
terraform destroy -target module.premerge_cluster_us_central.helm_release.github_actions_runner_set_windows
145+
terraform destroy -target module.premerge_cluster_us_central_resources.helm_release.github_actions_runner_set_linux
146+
terraform destroy -target module.premerge_cluster_us_central_resources.helm_release.github_actions_runner_set_windows
146147
```
147148

148149
These should complete, but if they do not, we are still able to get things
@@ -153,8 +154,8 @@ manually delete them with `kubectl delete`. Follow up the previous terraform
153154
commands by deleting the kubernetes namespaces all the resources live in:
154155

155156
```bash
156-
terraform destroy -target module.premerge_cluster_us_central.kubernetes_namespace.llvm_premerge_linux_runners
157-
terraform destroy -target module.premerge_cluster_us_central.kubernetes_namespace.llvm_premerge_windows_runners
157+
terraform destroy -target module.premerge_cluster_us_central_resources.kubernetes_namespace.llvm_premerge_linux_runners
158+
terraform destroy -target module.premerge_cluster_us_central_resources.kubernetes_namespace.llvm_premerge_windows_runners
158159
```
159160

160161
If things go smoothly, these should complete quickly. If they do not complete,
@@ -198,12 +199,12 @@ version upgrades however.
198199

199200
Start by destroying the helm chart:
200201
```bash
201-
terraform destroy -target module.premerge_cluster_us_central.helm_release.github_actions_runner_controller
202+
terraform destroy -target module.premerge_cluster_us_central_resources.helm_release.github_actions_runner_controller
202203
```
203204

204205
Then delete the namespace to ensure there are no dangling resources
205206
```bash
206-
terraform destroy -target module.premerge_cluster_us_central.kubernetes_namespace.llvm_premerge_controller
207+
terraform destroy -target module.premerge_cluster_us_central_resources.kubernetes_namespace.llvm_premerge_controller
207208
```
208209

209210
### Bumping the Version Number

0 commit comments

Comments
 (0)