Skip to content

Update variables to support Kali Linux #473

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

straysheep-dev
Copy link

After seeing how Pop!_OS and Linux Mint were supported, it looks like the same can be done for Kali using a separate variable for the release string here:

docker_apt_ansible_distribution_release: "{{ 'bookworm' if ansible_distribution in ['Kali'] else ansible_distribution_release }}"

...and replacing the few instances of ansible_distribution_release with that new variable.

There may be a better way to do this since bookworm will need updated with each Debian major release. However it's infrequent enough that it may be acceptable for now, similar to the GPG key checksum being a static value.

@clem9669
Copy link

clem9669 commented Feb 6, 2025

Hey @straysheep-dev @geerlingguy 👋

This looks great ! I hope this will be merge soon 🙏

Cheers

@mr-pmillz
Copy link

mr-pmillz commented Mar 1, 2025

When will this PR be merged? @clem9669
I'm getting failing builds for installing docker on Kali via this role due to the wrong url getting set for this task: https://github.com/geerlingguy/ansible-role-docker/blob/master/tasks/setup-Debian.yml#L40

this causes it to fall back to attempting to download docker gpg key from a non-existent url "https://download.docker.com/linux/kali/gpg" and piping it to the now deprecated apt-key command.
starting at Debian 11 and Ubuntu 20.10, apt-key is now deprecated and it will no longer be available after Debian 11 and Ubuntu 22.04
https://community.learnlinux.tv/t/how-to-apt-key-is-deprecated-heres-how-to-fix-it/489

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