Update variables to support Kali Linux#473
Update variables to support Kali Linux#473straysheep-dev wants to merge 2 commits intogeerlingguy:masterfrom
Conversation
|
Hey @straysheep-dev @geerlingguy 👋 This looks great ! I hope this will be merge soon 🙏 Cheers |
|
When will this PR be merged? @clem9669 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 |
146d6b1 to
22f3a4c
Compare
|
Updated this to work with the new I also noticed the refactor may have accidentally removed support for POP!_OS and Linux Mint by pointing directly to |
22f3a4c to
08314fb
Compare
|
This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution! Please read this blog post to see the reasons why I mark issues as stale. |
08314fb to
ee32658
Compare
@geerlingguy I checked back in on the documentation, but it still seems all the Debian and Ubuntu derivatives require this work around for now, similar to how Fedora, CentOS, and RHEL are handled in defaults/main.yml. There may be distro-specific packages that make more sense but this seems like the way to go just to continue support for Pop, Kali, and more. If there's anything I can adjust to get this merged please let me know. |
|
Please see the code review above — it seems like to support Kali, there's a hardcoded |
|
Thanks for this! This is one of the trickiest parts when building with Kali. I'll try to summarize what I've been running into just to share my notes in case anyone else has a better idea: The docs show either installing the older
All of this is to avoid trying to create a static list mapping Kali versions to Debian versions, since this will never work reliably. Right now Kali is based on "Forky" (they always use Debian Testing as the base). I imagine only the latest stable release (currently, that's "Trixie") is ever going to be supported by Docker. That means even if Kali referenced its Debian codename, that still won't work. Luckily, Debian's stable Release file is in a predictable path. This is the only reliable and dynamic solution I can work out after reviewing everything above, and sharing these notes and ideas with Claude. This isn't ideal but seems acceptable since this role already relies on network connections to function, and this seems like the option that would cover most cases on a rolling distro like Kali. |
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:
...and replacing the few instances of
ansible_distribution_releasewith that new variable.There may be a better way to do this since
bookwormwill 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.