Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support for Debian 12 aka bookworm #10212

Closed
fenio opened this issue Jun 11, 2023 · 7 comments · Fixed by #10221
Closed

support for Debian 12 aka bookworm #10212

fenio opened this issue Jun 11, 2023 · 7 comments · Fixed by #10221
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@fenio
Copy link

fenio commented Jun 11, 2023

What would you like to be added:

Support for new Debian 12 version released yesterday.

Why is this needed:

Cause install fails due to missing packages.

Seems that in Debian 12 aka bookworm some packages are missing / changed their names.

I was able to provision k8s cluster with kubespray but I had to change two lines in roles/kubernetes/preinstall/vars/debian.yml

python-apt -> python3-apt
aufs-tools -> mergerfs

To be honest I'm not really sure if mergerfs/python3-apt were used underneath to actually provision cluster but I blindly changed it and provisioning passed.

So please treat this issue mostly as a first feedback after trying to run kubespray over Debian 12 and feature request to support it as this is currently not supported setup.

@fenio fenio added the kind/feature Categorizes issue or PR as related to a new feature. label Jun 11, 2023
@yankay
Copy link
Member

yankay commented Jun 13, 2023

HI @fenio

Would you please provide a PR to support Debian 12 :-)
Thanks

@sophware
Copy link

Hi @yankay

Wouldn't the PR not be based on what @fenio did? I'm going to do the same as them as a hack, but a new file is needed. There's debian.yml and debian-11.yml. It stands to reason a new debian-12.yml is needed and that one of us has to know how to make that new file get called. I'll take a look but hope someone nudges me in the right direction.

@sophware
Copy link

I created a roles/kubernetes/preinstall/vars/debian-12.yml with the following:


---
required_pkgs:
  - python3-apt
  - gnupg
  - apt-transport-https
  - software-properties-common
  - conntrack
  - iptables
  - apparmor
  - libseccomp2
  - mergerfs

That is a copy of debian-11.yml with mergerfs added. I'll do a run-through and, unless someone can amend what that list should actually be, create a PR.

@fenio
Copy link
Author

fenio commented Jun 14, 2023

Don't count on me @yankay. I managed to get it working but IMHO it needs tests before something usable will be ready to be merged. And I simply have no time to do that work so my bugreport was just a sign that there is a problem.

@yankay
Copy link
Member

yankay commented Jun 15, 2023

Don't count on me @yankay. I managed to get it working but IMHO it needs tests before something usable will be ready to be merged. And I simply have no time to do that work so my bugreport was just a sign that there is a problem.

Thanks @fenio for the bugreport :-)

@yankay
Copy link
Member

yankay commented Jun 15, 2023

Hi @yankay

Wouldn't the PR not be based on what @fenio did? I'm going to do the same as them as a hack, but a new file is needed. There's debian.yml and debian-11.yml. It stands to reason a new debian-12.yml is needed and that one of us has to know how to make that new file get called. I'll take a look but hope someone nudges me in the right direction.

HI @sophware

To enable the feature of Debian 12 support , there need some steps :

  1. Try to run kubespray on the Debian12 environment
  2. Find the issue and fix it
  3. Add CI support for the Debian 12
  4. Change the readme file.

@0ekk
Copy link
Member

0ekk commented Jun 15, 2023

Hi @yankay
Wouldn't the PR not be based on what @fenio did? I'm going to do the same as them as a hack, but a new file is needed. There's debian.yml and debian-11.yml. It stands to reason a new debian-12.yml is needed and that one of us has to know how to make that new file get called. I'll take a look but hope someone nudges me in the right direction.

HI @sophware

To enable the feature of Debian 12 support , there need some steps :

  1. Try to run kubespray on the Debian12 environment
  2. Find the issue and fix it
  3. Add CI support for the Debian 12
  4. Change the readme file.

I'll try to do this job

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants