145 Commits

Author SHA1 Message Date
Benedikt Böhm
016fe1a279 Remove broken nightly repo 2026-01-15 14:09:10 +01:00
Jeff Geerling
94e6c5c86c Remove files added erroneously. 2025-11-27 22:20:23 -06:00
Jeff Geerling
24e05c4491 Attempt to fix ansible_facts deprecation warnings. 2025-11-27 22:15:34 -06:00
Tomasz Zwornicki
35fb3c519a Remove apt-transport-https from dependencies list 2025-10-21 03:51:57 +02:00
Jeff Geerling
2a4249b666 PR #522 follow-up - fix block formatting. 2025-10-15 13:11:34 -05:00
Christian Decker
b4b5c7732e Allow the start command of the Docker service to be overwritten. 2025-10-15 17:41:54 +02:00
Houssem Ben Ali
ff6196a292 Add more fixes 2025-10-01 18:18:30 +02:00
Houssem Ben Ali
0089992a9c Fix more checks 2025-10-01 18:09:51 +02:00
Houssem Ben Ali
4f9c2e4794 Fix more checks 2025-10-01 18:06:34 +02:00
Houssem Ben Ali
2f5715734b Avoid docker-runc removal 2025-10-01 17:53:33 +02:00
Houssem Ben Ali
e1515e97b9 Fix lint issues 2025-10-01 10:20:33 +02:00
Houssem Ben Ali
f04ad931db Add Suse Support 2025-09-30 16:50:32 +02:00
luudvig
0b09b07b3a fix uri in repository file for debian based distributions 2025-09-20 20:55:09 +02:00
Motalleb Fallahnezhad
423a8dba56 chore: remove unused variables 2025-09-14 16:57:11 +03:30
Norman Ziegner
c0ddaa9d64 Consistently access facts via the ansible_facts.* namespace
Signed-off-by: Norman Ziegner <n.ziegner@hzdr.de>
2025-08-20 10:42:25 +02:00
Sebastian
6343bc9148 Only one task for deleting the old repo is needed. Remove dependencies that are no longer needed for this role. 2025-08-18 20:56:23 +02:00
Sebastian
894e30654f Keyring removal is already handled by deb822 module. 2025-08-18 20:45:45 +02:00
Sebastian
8818e2b3e3 Make sure that handlers run before package install 2025-07-21 14:54:41 +02:00
Sebastian
a44d1bee2a Refactor Debian setup to use deb822_repository
- Replaces deprecated apt_repository and manual GPG key management with ansible.builtin.deb822_repository, introduced in apt 1.1.
- Updates handlers to use FQCNs.
2025-07-21 14:36:38 +02:00
Jeff Geerling
24dee13f70 Revert "Ensure the Docker daemon options file (/etc/docker/daemon.json) is deleted when no longer needed" 2025-04-05 07:41:55 -07:00
Jeff Geerling
c8df987a56 Merge pull request #498 from spantaleev/patch-1
Ensure the Docker daemon options file (`/etc/docker/daemon.json`) is deleted when no longer needed
2025-04-05 07:41:35 -07:00
boite
047f7a163c fix: lowercase distro name when removing apt source
This makes removal succeed when docker_apt_ansible_distribution is
"Ubuntu".
2025-03-18 23:57:55 +00:00
Slavi Pantaleev
5a2fd928ad Ensure the Docker daemon options file (/etc/docker/daemon.json) is deleted when no longer needed
If people set options in `docker_daemon_options` (e.g. `{'ipv6': false}`), the role will create the `/etc/docker/daemon.json` file and populate it with the options.

Later on, if people stop setting *all of these* options, the file used to remain in place (containing the old options) and cause trouble. This is unexpected.

If the Docker installation is managed by this role and the `/etc/docker/daemon.json` file is managed by it, it should also take care to:

- either delete the file when it's no longer necessary
- or populate it with empty options if that is what `docker_daemon_options` contains

Deleting the file instead of putting `{}` in it seems like the cleaner approach.

There's a chance that people would like to manage options in `/etc/docker/daemon.json` by themselves (without Ansible) and this new behavior when `docker_daemon_options` is empty, but this runs against having a managed Docker installation via Ansible.

This patch only deletes the `/etc/docker/daemon.json` file.

We could possibly delete the `/etc/docker` directory too, but it's more tricky to delete it only when it's empty. In some cases, the directory may contain other files and invoking the `file` module with `state: absent` will delete everything recursively, which is undesirable.
2025-03-09 16:20:51 +02:00
Ali Mehraji
78dec194bd update: docker_obsolete_packages separated for Debian and RedHat OS-Family 2025-02-17 01:21:15 +03:30
dbrennand
d490bcea4c fix: issue with #470 fix
This patch ensures that the correct filename is specified based on the distribution. Observed this behaviour on a Debian system, the old file was not removed due to it being hardcoded to ubuntu instead.
2025-02-06 21:09:29 +00:00
dbrennand
110cb89663 fix: https://github.com/geerlingguy/ansible-role-docker/issues/470 2024-11-23 17:13:20 +00:00
Dan Rough
f666736e79 Remove trusted.gpg.d artifacts. Fixes geerlingguy/ansible-role-docker#460 2024-07-24 11:03:00 +01:00
Első András
5b3eea8f97 Allow change obsolete packages 2024-06-27 15:18:55 +02:00
James Myatt
9c4e21caec gnupg no longer prerequisite
See 811f32afe5
2024-06-09 23:15:02 +01:00
Jeff Geerling
1f1f5cd69c Merge branch 'master' into kawade/fix-434 2024-05-27 13:02:27 -05:00
James Myatt
fe69df76fa Update list of obsolete packages 2024-05-27 11:17:28 +01:00
Omkar Kawade
d8f92e1874 434 remove permissions update, add reverse compatibility 2023-12-18 12:56:14 -08:00
Omkar Kawade
08ae86e0b5 434 update apt key destination 2023-12-18 11:06:21 -08:00
Omkar Kawade
6cb849c416 435 ansible-lint 2023-11-16 21:20:56 -08:00
Omkar Kawade
ae29f9f9c0 434 changed_when false for adding docker apt key shell cmd 2023-11-16 21:14:58 -08:00
Omkar Kawade
12ad263ef6 435 Update apt key ansible task 2023-11-16 20:23:52 -08:00
Omkar Kawade
1dca07177f 434 Add GPG keys to keyrings instead of trusted.gpg.d 2023-11-16 18:59:51 -08:00
Yethal
a76a05024a Uninstall docker.io on apt distros 2023-11-10 16:20:54 +01:00
Jeff Geerling
9d51ec8802 Merge pull request #424 from rpetti/fix-runc-conflict-on-rhel
remove runc, as it conflicts with containerd
2023-09-28 16:53:38 +00:00
Rob Petti
95d13bf2b8 remove runc, as it conflicts with containerd 2023-09-27 23:16:32 -06:00
Aram Akhavan
fcbd3d7098 Merge branch 'master' into new_defaults 2023-09-27 14:41:20 -07:00
nevart
7c86fe1739 APT key: don't force and provide checksum, to fix check mode
As described in https://github.com/ansible/ansible/issues/65687, get_url
only partially supports check_mode: "the changed status will reflect
comparison to an empty source file".

Before this change, executing this code, with the key already being in
place on the target system, would report "OK", while check_mode would
report "changed".

Due to this change, both now either report "OK" or "changed", depending
on the state of the target system.
2023-06-26 07:34:10 +00:00
Aram Akhavan
d64f9afe19 add filename option to apt repository add 2023-02-21 22:35:04 -08:00
aurelien-bareille
0ad2ddd2c6 Include repository setup dependancies in flag scope 2022-12-14 18:25:29 +01:00
aurelien-bareille
8b623b2266 Add flag to deactivate repository configuration 2022-12-14 16:31:46 +01:00
Jeff Geerling
03d5bd6d06 Remove deprecated warn arg from command and shell module invocations. 2022-11-26 22:51:49 -06:00
berthin
3800fd2d42 Fix condition for empty docker compose svn 2022-11-05 11:20:04 +01:00
Jeff Geerling
6f636dc579 Merge pull request #373 from sosheskaz/fix-gpg
Download apt key directly
2022-09-08 17:34:14 -05:00
Jeff Geerling
928fe893e9 PR #370: Style cleanup. 2022-09-05 21:32:10 -05:00
Eric Miller
f463bb2c1a Download apt key directly
Fixes issue #372
2022-09-05 17:55:21 -05:00