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

Use DEB822 repo for all Debian platforms except for Buster #84

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from

Conversation

jsf9k
Copy link
Member

@jsf9k jsf9k commented Dec 27, 2024

🗣 Description

This pull request modifies the code to use a DEB822 repo for all Debian platforms except for Debian Buster. This style of package repo is not supported by Debian Buster.

Note

This PR is based on #83 and should be merged after that one.

💭 Motivation and context

This simplifies the Ansible code and treats Buster as the special case, not Kali.

🧪 Testing

All automated tests pass.

✅ Pre-approval checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • All relevant type-of-change labels have been added.
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.
  • All new and existing tests pass.

Kali recently updated to a version of apt that no longer provides the
deprecated apt-key binary, so we cannot use ansible.builtin.apt_key to
get the official Docker repo GPG key.  We may as well upgrade to a
DEB822 repo while we're at it.
The latter is more specific and better connotes what the variable
contains.
This will allow us to treat Kali as less of a special case.
This will allow us to stop treating Kali as a special case.
@jsf9k jsf9k added the improvement This issue or pull request will add or improve functionality, maintainability, or ease of use label Dec 27, 2024
@jsf9k jsf9k self-assigned this Dec 27, 2024
@jsf9k jsf9k force-pushed the improvement/use-deb822-repo-for-everything-but-debian-buster branch from 238326e to b2b46ee Compare December 27, 2024 15:09
This task should only run if the ansible.builtin.deb822_repository
task actually changes anything.
@jsf9k jsf9k force-pushed the improvement/use-deb822-repo-for-everything-but-debian-buster branch from cdd927e to 78d2f03 Compare December 27, 2024 17:16
@jsf9k jsf9k marked this pull request as ready for review December 27, 2024 19:03
@jsf9k jsf9k requested a review from a team December 27, 2024 19:03
@jsf9k
Copy link
Member Author

jsf9k commented Dec 27, 2024

The Fedora 40 and 41 tests are failing because of fedora-cloud/docker-brew-fedora#117. See also this RedHat Bugzilla issue.

vars/Kali.yml Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement This issue or pull request will add or improve functionality, maintainability, or ease of use
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

3 participants