Commit Graph

282 Commits

Author SHA1 Message Date
Jack Hodgkiss
f6636e34b5
Merge branch 'master' into use-ansible-facts 2023-09-28 18:11:13 +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
Jeff Geerling
914ca965e0
Merge pull request #396 from dale-c-anderson/382-fix-apt-errors-on-pop-os
Fix apt errors on Pop!_OS
2023-09-28 02:12:12 +00:00
Jeff Geerling
3ceffb4cee
Merge pull request #420 from eropple/eropple-patch-1
Fix a typo in README.md
2023-09-28 02:11:17 +00:00
Jeff Geerling
1fad07589a
Merge pull request #369 from kaysond/new_defaults
Change defaults in the next major release
2023-09-28 02:09:48 +00:00
Aram Akhavan
fcbd3d7098
Merge branch 'master' into new_defaults 2023-09-27 14:41:20 -07:00
Jack Hodgkiss
18f51bd2e3
feat: use ansible_facts rather than variables 2023-09-07 12:15:06 +01:00
Ed Ropple
b41084ad3f
Fix a typo in README.md 2023-08-12 14:47:52 -04:00
Dale Anderson
a23731506b
Restore snippet lost in previous conflict 2023-08-10 09:12:33 -07:00
Dale Anderson
0b419a0a12
Merge branch 'master' into 382-fix-apt-errors-on-pop-os 2023-08-10 09:10:36 -07:00
Jeff Geerling
8bb277bd7f
Merge pull request #418 from nevart/apt-key-fix-check-mode
APT key: don't force and provide checksum, to fix check mode
2023-08-10 00:18:41 -05:00
Jeff Geerling
7fa232d1c1
Merge pull request #419 from nevart/debian-bookworm
Add Debian 12 bookworm support
2023-08-10 00:17:22 -05:00
Jeff Geerling
e6996ccca8
Merge pull request #410 from etkecc/debian-signed-by
debian: add signed-by param to the apt source list
2023-08-10 00:16:13 -05:00
nevart
81f23a11dc Add Debian 12 bookworm support 2023-08-06 13:01:27 +00:00
Jeff Geerling
78e50d46e2 Stale bot is now Stale GitHub Action. 2023-07-10 17:47:39 -05: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
Jeff Geerling
635061e0a4 Remove official support for RHEL. Rocky/Alma/Stream support is best-effort. 2023-06-23 10:56:08 -05:00
Jeff Geerling
a29e9d51e9 Fix Molecule CI workflow since docker plugin has moved. 2023-04-20 17:27:13 -05:00
Aine
32307386d4
debian: add signed-by param to the apt source list, fixes #409 2023-04-02 14:33:48 +03:00
Aram Akhavan
c3a127134d change default compose install to plugin, apt repository filename to docker.list 2023-02-21 22:43:08 -08:00
Aram Akhavan
d64f9afe19 add filename option to apt repository add 2023-02-21 22:35:04 -08:00
Jeff Geerling
67e50e9af0
Merge pull request #394 from aurelien-bareille/master
Add flag to deactivate repository configuration
2023-02-02 21:28:43 -06:00
Dale Anderson
2638529927
Fix linter warning 2023-01-23 17:50:49 -08:00
Dale Anderson
75b8acc626
Linux Mint is also Ubuntu under the hood
Closes #334
2023-01-23 17:41:27 -08:00
Dale Anderson
a6bab2c9be
Make naming more consistent 2022-12-30 15:46:38 -08:00
Dale Anderson
48688d0faf
Handle the exception more cleanly 2022-12-30 15:17:35 -08:00
Dale Anderson
43b5ad4fef
Override defaults for Pop!_OS 2022-12-30 14:56:00 -08:00
aurelien-bareille
61d571a941 Add documentation 2022-12-26 16:53:42 +01: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
b4a2ff5ee7 Remove CentOS 7 from tests. 2022-12-06 19:24:50 -06:00
Jeff Geerling
32dfe4c00e Fix Molecule CI workflow for Ubuntu 22.04 GitHub Actions. 2022-12-06 18:23:40 -06:00
Jeff Geerling
03d5bd6d06 Remove deprecated warn arg from command and shell module invocations. 2022-11-26 22:51:49 -06:00
Jeff Geerling
9685a2a30d
Merge pull request #389 from abalage/bugfix/fix-overriding-os-variables
Make override work by adjusting variable name to match defaults/main.yml
2022-11-10 16:01:59 -06:00
Balázs Németh
17ddd4ef2e Make override work by adjusting variable to match defaults/main.yml 2022-11-10 22:11:22 +01:00
Jeff Geerling
9154e3bf75
Merge pull request #384 from berthin/master
Fix condition for empty docker compose svn
2022-11-05 14:21:48 -05:00
berthin
3800fd2d42
Fix condition for empty docker compose svn 2022-11-05 11:20:04 +01:00
Jeff Geerling
96eaae30e5
Merge pull request #380 from adamantike/fix/set-docker_package-for-archlinux
Set default docker_package for Archlinux distribution
2022-10-10 15:49:02 -05:00
Michael Manganiello
8e09a22180 Set default docker_package for Archlinux distribution
Instead of the current role's default value for `docker_package`
(`docker-ce`), the package name for the Archlinux OS distribution is
`docker` [1].

Considerations:
* Filename is `Archlinux`, as that is the possible value exposed by
  Ansible, in `ansible_distribution` [2] and `ansible_os_family` [3].
* Platform name in meta's `galaxy_info` has been changed to `ArchLinux`,
  as that's the supported platform name in Ansible Galaxy [4].

[1] https://archlinux.org/packages/community/x86_64/docker/
[2] https://docs.ansible.com/ansible/2.6/user_guide/playbooks_conditionals.html#ansible-distribution
[3] https://docs.ansible.com/ansible/2.6/user_guide/playbooks_conditionals.html#ansible-os-family
[4] https://galaxy.ansible.com/api/v1/platforms/?name__icontains=arch
2022-10-09 23:01:52 -03:00
Jeff Geerling
a15de78905
Merge pull request #376 from geerlingguy/290-arm
Fixes #290: Allow role to work on ARM64 out of the box.
2022-09-26 17:15:32 -05:00
Jeff Geerling
53c487c31a Fixes #290: Allow role to work on ARM64 out of the box. 2022-09-26 17:09:27 -05: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
Jeff Geerling
9f8ea35051
Merge pull request #370 from fardke/reset-ssh-connection-after-docker-install
reset ssh connection to allow user changes to affect current login user
2022-09-05 21:30:55 -05:00
Eric Miller
f463bb2c1a Download apt key directly
Fixes issue #372
2022-09-05 17:55:21 -05:00
Kevin Fardel
0884ae21b8 reset ssh connection to allow user changes to affect 'current login user'
Why:
Without this modification, we can't use docker with docker_users in same
playbook than the one which install docker.

How:
Check that docker_users are set and not in docker group before include
docker-users.yml. In docker-users.yml we call reset_connection from
ansible.builtin.meta collection after docker-users are added to docker
group.

Manual success tests:
* Try to install docker with only one user in docker-users and not in
  docker group => docker-users.yml include
* Try to install docker with only one user in docker-users but the user
  is in docker group => docker-users.yml not include
* Try to install docker with 2 users in docker-users, one user in docker
  group and the second not => docker-users.yml include
* Try to install docker with 2 users in docker-users, both are in docker
  group => docker-users.yml not include
* Try to install docker with 2 users in docker-users, both are not in
  docker group => docker-users.yml include
2022-08-24 17:07:45 +02:00
Jeff Geerling
53c5490523 PR #320: Use main.yml instead of default.yml for vars defaults. 2022-08-19 14:13:01 -05:00
Jeff Geerling
d3fcc22713
Merge pull request #320 from adamus1red/adamus1red/os-vars
Add OS specific variable overrides.
2022-08-19 14:09:16 -05:00
Jeff Geerling
93eef333bc
Update tasks/main.yml 2022-08-19 14:09:04 -05:00