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 from bb368b8 to c031ed7 Compare September 24, 2025 12:01
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 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.

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