Skip to content

Conversation

@nikolaszimmermann
Copy link
Member

OpenTofu / Helm / packer / kubectl are now included.

@nikolaszimmermann nikolaszimmermann force-pushed the nzimmermann/add_cluster_admin_tools branch from ded3a07 to bb368b8 Compare September 16, 2025 07:12
@TingPing
Copy link
Member

TingPing commented Sep 17, 2025

I feel these are a bit out of scope of the sdk, very few will use them. This increases the size by over 600MB (~8%). It also looks like it will be fairly hard to maintain over time during major OS updates.

Maybe we could have a different image for these, aimed at those administering the servers?

@nikolaszimmermann
Copy link
Member Author

@TingPing we had the wci-admin image before in the webkit-container-infrastructure repo separated but lack the tooling to use wkdev-create with a different base image - we agreed that it would be convenient to use our standard tools such as wkdev-create/enter for the admin tasks.

What we could do instead is to add a script that you can call to install the necessary tools if you want to - WDYT?

@TingPing
Copy link
Member

What we could do instead is to add a script that you can call to install the necessary tools if you want to - WDYT?

Fine by me.

@nikolaszimmermann nikolaszimmermann force-pushed the nzimmermann/add_cluster_admin_tools branch 2 times, most recently from c031ed7 to f889779 Compare September 24, 2025 12:02
@nikolaszimmermann
Copy link
Member Author

Fixed by providing a wkdev-setup-cluster-administration-tools script instead that folks that care about it can run themselves.

Provide a new script that installs all tools needed to administrate the
kubernetes cluster, described in webkit-container-infrastructure repo.

OpenTofu / Helm / Packer / kubectl / hcloud-cli will be installed by
this script.
@nikolaszimmermann nikolaszimmermann force-pushed the nzimmermann/add_cluster_admin_tools branch from f889779 to cf3ea65 Compare September 30, 2025 06:43
@nikolaszimmermann nikolaszimmermann enabled auto-merge (rebase) September 30, 2025 12:07
@nikolaszimmermann nikolaszimmermann merged commit 1d6c9c5 into main Oct 1, 2025
4 checks passed
@nikolaszimmermann nikolaszimmermann deleted the nzimmermann/add_cluster_admin_tools branch October 1, 2025 14:31
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.

3 participants