k3s-oci-cluster icon indicating copy to clipboard operation
k3s-oci-cluster copied to clipboard

Alternative operating system support?

Open Djelibeybi opened this issue 1 year ago • 6 comments

First, thanks for creating such a useful Terraform module. It works great.

Are you interested in supporting a choice of operating system for the instances? I've created a downstream fork that allows you to chose either Ubuntu (which remains the default) or Oracle Linux as well as allows users to change the shape and resource allocation of the instances.

I'm happy to submit the feature set as a PR if you want. If not, I'll probably remove the Ubuntu stuff and just maintain an Oracle Linux based fork downstream.

Djelibeybi avatar Aug 26 '22 22:08 Djelibeybi

Hi @Djelibeybi yes sure, you can submit a new PR

garutilorenzo avatar Aug 30 '22 09:08 garutilorenzo

I've done some refactoring and added the option of deploying the OCI Cloud Controller Manager during the install. I'll post the PR shortly so you can review and decide if it's still worth merging.

Djelibeybi avatar Aug 30 '22 09:08 Djelibeybi

Hi @Djelibeybi I have created a new PR #20 for Oracle Linux OS support. I haven't used a variable for the OS determination but I check if /etc/lsb-release exist. If true the OS is ubuntu, else is Oracle. The check can be improved.

I've tested both Oracle Linux 8 and 9

garutilorenzo avatar Sep 14 '22 12:09 garutilorenzo

That's awesome. Will take it for a spin myself later today.

Djelibeybi avatar Sep 14 '22 18:09 Djelibeybi

@Djelibeybi merged PR #20. I get most of the new features from your oraclelinux-oci-ccm branch and thank you for your revison on the PR. In the next days I will open a new PR for the OCI CCM.

garutilorenzo avatar Sep 16 '22 14:09 garutilorenzo

That's great. I didn't manage to take it for that spin last week, but will do so on Monday or Tuesday.

You can go ahead and close this issue if you want, or keep it open for your own tracking purposes.

Djelibeybi avatar Sep 17 '22 01:09 Djelibeybi

I didn't realise I'd left this issue open. My bad!

Djelibeybi avatar Apr 21 '23 21:04 Djelibeybi