]> git.apps.os.sepia.ceph.com Git - ceph-cm-ansible.git/log
ceph-cm-ansible.git
2 months agoMerge pull request #772 from dmick/checkcerts-telemetry-and-david main
David Galloway [Mon, 21 Apr 2025 22:06:26 +0000 (18:06 -0400)]
Merge pull request #772 from dmick/checkcerts-telemetry-and-david

checkcerts.py: add telemetry.ceph.com to hosts to check

3 months agoMerge pull request #779 from kshtsk/wip-testnode-time-sync
David Galloway [Tue, 8 Apr 2025 13:45:25 +0000 (09:45 -0400)]
Merge pull request #779 from kshtsk/wip-testnode-time-sync

Don't even try hwclock time sync on  testnode if node is kvm

3 months agotestnode: install pip for dnf as well 779/head
Kyr Shatskyy [Sat, 5 Apr 2025 17:31:20 +0000 (19:31 +0200)]
testnode: install pip for dnf as well

On CentOS ansible ansible_pkg_mgr is 'dnf' instead of 'yum',
but the python3-pip still needed.

Signed-off-by: Kyr Shatskyy <kyrylo.shatskyy@clyso.com>
3 months agoroles/common: use timedatectl to set timezone for apt too
Kyr Shatskyy [Thu, 3 Apr 2025 17:11:17 +0000 (19:11 +0200)]
roles/common: use timedatectl to set timezone for apt too

Overwriting of /etc/timezone with new time zone and calling:

  dpkg-reconfigure --frontend noninteractive tzdata

does not work. Just use timedatectl same as for all modern systems.

Signed-off-by: Kyr Shatskyy <kyrylo.shatskyy@clyso.com>
3 months agoroles/common: do not set hwclock in kvm node
Kyr Shatskyy [Thu, 3 Apr 2025 15:02:34 +0000 (17:02 +0200)]
roles/common: do not set hwclock in kvm node

The hwclock is not accessible for kvm nodes and trying to set it
creates an error:

    hwclock: Cannot access the Hardware Clock via any known method.
    hwclock: Use the --verbose option to see the details of our search for an access method.

In that case try and check if current clock is using 'kvm-clock'
and just skip this step.

Signed-off-by: Kyr Shatskyy <kyrylo.shatskyy@clyso.com>
3 months agoMerge pull request #775 from ceph/ipaddrdhcp
David Galloway [Wed, 2 Apr 2025 20:01:37 +0000 (16:01 -0400)]
Merge pull request #775 from ceph/ipaddrdhcp

tools: Exclude `option host-name` from grep hostname in set-next-server

3 months agoMerge pull request #777 from ceph/ubuntu24
Dan Mick [Wed, 26 Mar 2025 21:18:29 +0000 (14:18 -0700)]
Merge pull request #777 from ceph/ubuntu24

Ubuntu Noble vars

3 months agotestnode: Force install of pip packages globally on Ubuntu >=24 777/head
David Galloway [Fri, 21 Mar 2025 21:55:29 +0000 (17:55 -0400)]
testnode: Force install of pip packages globally on Ubuntu >=24

Fixes:
```
error: externally-managed-environment

This environment is externally managed

To install Python packages system-wide, try apt install
python3-xyz, where xyz is the package you are trying to
install.

If you wish to install a non-Debian-packaged Python package,
create a virtual environment using python3 -m venv path/to/venv.
Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
sure you have python3-full installed.

If you wish to install a non-Debian packaged Python application,
it may be easiest to use pipx install xyz, which will manage a
virtual environment for you. Make sure you have pipx installed.

See /usr/share/doc/python3.12/README.venv for more information.

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.
```

Signed-off-by: David Galloway <david.galloway@ibm.com>
3 months agoMerge pull request #776 from ceph/apt-cruft
David Galloway [Fri, 21 Mar 2025 22:08:34 +0000 (18:08 -0400)]
Merge pull request #776 from ceph/apt-cruft

testnode: Remove needless tasks

3 months agotestnode: Remove needless tasks 776/head
David Galloway [Fri, 21 Mar 2025 21:22:36 +0000 (17:22 -0400)]
testnode: Remove needless tasks

The linked tracker above the custom repo removal task is from long before we had golden OS images (pre-FOG).

Signed-off-by: David Galloway <david.galloway@ibm.com>
3 months agotestnode: Ubuntu Noble vars
David Galloway [Fri, 21 Mar 2025 21:21:52 +0000 (17:21 -0400)]
testnode: Ubuntu Noble vars

Signed-off-by: David Galloway <david.galloway@ibm.com>
3 months agocommon: Install util-linux-extra to provide hwclock on Ubuntu 24
David Galloway [Fri, 21 Mar 2025 21:19:00 +0000 (17:19 -0400)]
common: Install util-linux-extra to provide hwclock on Ubuntu 24

Signed-off-by: David Galloway <david.galloway@ibm.com>
3 months agotools: Exclude `option host-name` from grep hostname in set-next-server 775/head
David Galloway [Thu, 20 Mar 2025 20:34:09 +0000 (16:34 -0400)]
tools: Exclude `option host-name` from grep hostname in set-next-server

Otherwise, the `linenum=` line gets two line numbers.

Signed-off-by: David Galloway <david.galloway@ibm.com>
3 months agoMerge pull request #774 from ceph/wip-openvpn-logrotate
David Galloway [Wed, 19 Mar 2025 21:23:15 +0000 (17:23 -0400)]
Merge pull request #774 from ceph/wip-openvpn-logrotate

OpenVPN logrotate configuration to keep the logs for a year.

3 months agoModifying the structure of the script to create log rotation configuration for openvp... 774/head
Fernando [Wed, 19 Mar 2025 00:07:00 +0000 (18:07 -0600)]
Modifying the structure of the script to create log rotation configuration for openvpn logs.
Adding trap command to do a cleanup in case of failure.
Changing the name in the module that creates log rotate configuration script.

3 months agoModyfing openVPN logrotate configuration to keep the logs for a year.
Fernando [Thu, 30 Jan 2025 17:09:20 +0000 (11:09 -0600)]
Modyfing openVPN logrotate configuration to keep the logs for a year.

The daily rotation was configured to keep daily logs for a week.
A new script to manage weekly rotation and keep those logs for a year was generated to be put into /etc/cron.weekly

Fixes: https://ibm.monday.com/boards/5591222586/pulses/8354436354
Signed-off-by: Fernando <fernando.alcocer.ochoa@ibm.com>
4 months agoMerge pull request #773 from ceph/skip-grafana-agent
adam kraitman [Wed, 12 Mar 2025 15:11:43 +0000 (17:11 +0200)]
Merge pull request #773 from ceph/skip-grafana-agent

testnode: Run grafana_agent role optionally

4 months agotestnode: Run grafana_agent role optionally 773/head
David Galloway [Mon, 3 Mar 2025 20:16:01 +0000 (15:16 -0500)]
testnode: Run grafana_agent role optionally

Signed-off-by: David Galloway <david.galloway@ibm.com>
4 months agocheckcerts.py: add telemetry.ceph.com to hosts to check 772/head
Dan Mick [Thu, 27 Feb 2025 18:22:15 +0000 (10:22 -0800)]
checkcerts.py: add telemetry.ceph.com to hosts to check

also, remove Andrew Schoen and add David Galloway to email

Signed-off-by: Dan Mick <dan.mick@redhat.com>
4 months agoMerge pull request #768 from ceph/wip-useradd-selinux
David Galloway [Thu, 27 Feb 2025 14:26:50 +0000 (09:26 -0500)]
Merge pull request #768 from ceph/wip-useradd-selinux

Creating a selinux module to resolve selinux conflicts with grafana agent

4 months agoMerge pull request #767 from dmick/fix-checkcerts.py-email
David Galloway [Thu, 27 Feb 2025 00:01:00 +0000 (19:01 -0500)]
Merge pull request #767 from dmick/fix-checkcerts.py-email

checkcerts.py: actually fix "send email"

4 months agoCreate a module to resolve selinux conflict with grafana-agent 768/head
Fernando [Wed, 12 Feb 2025 18:49:36 +0000 (12:49 -0600)]
Create a module to resolve selinux conflict with grafana-agent

Fixes: https://ibm.monday.com/boards/5591222586/pulses/8269716052
Signed-off-by: Fernando <fernando.alcocer.ochoa@ibm.com>
4 months agocheckcerts.py: actually fix "send email" 767/head
Dan Mick [Tue, 18 Feb 2025 23:21:00 +0000 (15:21 -0800)]
checkcerts.py: actually fix "send email"

argparse can't do a nargs="*" optional arg *and* check for its
presence; add a separate arg -E to send the email, and keep -e as
an optional list of addressees.

Also add the full path and host where checkcerts.py is running.

Signed-off-by: Dan Mick <dan.mick@redhat.com>
5 months agoMerge pull request #766 from ceph/wip-rocky-9
David Galloway [Thu, 13 Feb 2025 16:25:02 +0000 (11:25 -0500)]
Merge pull request #766 from ceph/wip-rocky-9

Adding Rocky 9.5 to sepia cobbler

5 months agoAdding Rocky 9.5 to sepia cobbler 766/head
Adam Kraitman [Thu, 13 Feb 2025 16:21:40 +0000 (18:21 +0200)]
Adding Rocky 9.5 to sepia cobbler

Signed-off-by: Adam Kraitman <akraitma@li-8b09b2cc-35b7-11b2-a85c-cd1dbade58f9.ibm.com>
5 months agoMerge pull request #732 from ceph/rm-puddle
David Galloway [Mon, 10 Feb 2025 20:56:50 +0000 (15:56 -0500)]
Merge pull request #732 from ceph/rm-puddle

remove puddle playbook and role

5 months agoMerge pull request #763 from ceph/wip-grafana-keyrings
adam kraitman [Thu, 23 Jan 2025 16:22:36 +0000 (18:22 +0200)]
Merge pull request #763 from ceph/wip-grafana-keyrings

Adding a validation to ensure /etc/apt/keyrings path exists

5 months agoAdding a validation to ensure /etc/apt/keyrings path exists 763/head
Fernando [Wed, 22 Jan 2025 18:35:29 +0000 (12:35 -0600)]
Adding a validation to ensure /etc/apt/keyrings path exists

Fixes: https://ibm.monday.com/boards/5591222586/pulses/8269716052
Signed-off-by: Fernando <fernando.alcocer.ochoa@ibm.com>
5 months agoMerge pull request #734 from ceph/wip-grafana-agent
David Galloway [Fri, 17 Jan 2025 16:46:58 +0000 (11:46 -0500)]
Merge pull request #734 from ceph/wip-grafana-agent

Ansible Role for deploying grafana-agent

6 months agoMerge pull request #762 from dmick/wip-add-sepia-client-tools
Dan Mick [Wed, 18 Dec 2024 22:35:58 +0000 (14:35 -0800)]
Merge pull request #762 from dmick/wip-add-sepia-client-tools

Add openvpn client tools (the sepia-vpn-client.tar.gz stuff) for tracking/improvement

6 months agoopenvpn/sepia/new-client: save a tarball of secret and secret.hash 762/head
Dan Mick [Wed, 18 Dec 2024 21:48:54 +0000 (13:48 -0800)]
openvpn/sepia/new-client: save a tarball of secret and secret.hash

Also, explain a little bit more about what new-client has done.
hopefully this helps users understand/keep track of their secrets,
and hopefully this streamlines diagnosing when things go wrong

Signed-off-by: Dan Mick <dan.mick@redhat.com>
6 months agoInitial version of tarball as it is on download.ceph.com today
Dan Mick [Wed, 18 Dec 2024 21:44:51 +0000 (13:44 -0800)]
Initial version of tarball as it is on download.ceph.com today

Signed-off-by: Dan Mick <dan.mick@redhat.com>
7 months agoAnsible Role for deploying grafana-agent 734/head
Adam Kraitman [Mon, 24 Apr 2023 18:22:06 +0000 (21:22 +0300)]
Ansible Role for deploying grafana-agent

Signed-off-by: Adam Kraitman <akraitma@redhat.com>
9 months agoMerge pull request #761 from ceph/wip-move-openmpi-common
adam kraitman [Tue, 1 Oct 2024 13:00:20 +0000 (16:00 +0300)]
Merge pull request #761 from ceph/wip-move-openmpi-common

testnode: move removing openmpi-common to apt_systems.yml

9 months agotestnode: move removing openmpi-common to apt_systems.yml 761/head
Ilya Dryomov [Tue, 1 Oct 2024 10:57:09 +0000 (12:57 +0200)]
testnode: move removing openmpi-common to apt_systems.yml

Otherwise, because it's placed in ubuntu.yml, it resets
packages_to_remove list defined in apt_systems.yml to just
openmpi-common and commit 701d3594d220 ("testnode: remove tgt")
doesn't take effect.

While at it, fix a typo in the comment -- it's mpich.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
9 months agoMerge pull request #760 from ceph/wip-remove-tgt
adam kraitman [Tue, 1 Oct 2024 05:53:14 +0000 (08:53 +0300)]
Merge pull request #760 from ceph/wip-remove-tgt

testnode: remove tgt

9 months agotestnode: remove tgt 760/head
Ilya Dryomov [Mon, 30 Sep 2024 17:32:39 +0000 (19:32 +0200)]
testnode: remove tgt

As a follow up for commit 67a92953a5a2 ("testnode: don't install tgt"),
explicitly remove tgt to cover the case of it already being there.  This
also documents that we actively don't want tgt to be there.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
9 months agoMerge pull request #759 from ceph/wip-drop-tgt
adam kraitman [Mon, 30 Sep 2024 11:15:16 +0000 (14:15 +0300)]
Merge pull request #759 from ceph/wip-drop-tgt

testnode: don't install tgt

9 months agotestnode: don't install tgt 759/head
Ilya Dryomov [Thu, 26 Sep 2024 14:49:14 +0000 (16:49 +0200)]
testnode: don't install tgt

We haven't done anything with tgt for over 10 years and tgt can
interfere with setting up ceph-iscsi targets, resulting in "Could not
create NetworkPortal in configFS" errors.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
10 months agoMerge pull request #758 from dmick/wip-rhel-93-dbus
adam kraitman [Sun, 8 Sep 2024 08:10:01 +0000 (11:10 +0300)]
Merge pull request #758 from dmick/wip-rhel-93-dbus

roles/cobbler/templates/snippets/cephlab_packages_rhel: add dbus pkgs

10 months agoroles/cobbler/templates/snippets/cephlab_packages_rhel: add dbus pkgs 758/head
Dan Mick [Sat, 7 Sep 2024 01:39:30 +0000 (18:39 -0700)]
roles/cobbler/templates/snippets/cephlab_packages_rhel: add dbus pkgs

FOG capture requires dbus-uuidgen (in dbus-tools) and the presence of
/var/lib/dbus (from dbus-daemon) to do the bug-avoidance dance for
Satellite.  It's not clear that bug-avoidance dance is still necessary,
but this is the minimal-investigation minimal-touch change to make the
capture process work.

Signed-off-by: Dan Mick <dmick@redhat.com>
10 months agoMerge pull request #757 from dmick/wip-mount-to-posix-mount
Zack Cerza [Wed, 28 Aug 2024 16:44:27 +0000 (10:44 -0600)]
Merge pull request #757 from dmick/wip-mount-to-posix-mount

prep-fog-capture.yml: mount -> ansible.posix.mount

11 months agoprep-fog-capture.yml: mount -> ansible.posix.mount 757/head
Dan Mick [Fri, 16 Aug 2024 07:41:58 +0000 (00:41 -0700)]
prep-fog-capture.yml: mount -> ansible.posix.mount

Signed-off-by: Dan Mick <dmick@redhat.com>
13 months agoMerge pull request #756 from VallariAg/wip-remove-get_md5
adam kraitman [Mon, 20 May 2024 15:32:39 +0000 (18:32 +0300)]
Merge pull request #756 from VallariAg/wip-remove-get_md5

Fix "Unsupported parameters for (stat) module: get_md5"

13 months agoFix "Unsupported parameters for (stat) module: get_md5" 756/head
Vallari Agrawal [Mon, 20 May 2024 03:34:39 +0000 (09:04 +0530)]
Fix "Unsupported parameters for (stat) module: get_md5"

Remove get_md5 because it was removed in ansible 2.9:
https://github.com/ansible-community/ansible-build-data/blob/0dee49ac8a7674153606ddc6432d4029eb20172d/9/CHANGELOG-v9.rst#L5195

fixes: https://tracker.ceph.com/issues/66106

Signed-off-by: Vallari Agrawal <val.agl002@gmail.com>
14 months agoMerge pull request #755 from dmick/wip-centos9-repos
adam kraitman [Fri, 19 Apr 2024 08:25:48 +0000 (11:25 +0300)]
Merge pull request #755 from dmick/wip-centos9-repos

Remove unnecessary duplicate repo definitions for centos9

14 months agoRemove unnecessary duplicate repo definitions for centos9 755/head
Dan Mick [Fri, 19 Apr 2024 02:15:46 +0000 (19:15 -0700)]
Remove unnecessary duplicate repo definitions for centos9

CentOS 8 had different .repo files for each major section (BaseOS,
AppStream, etc.).  CentOS 9 has apparently moved to a single file,
centos.repo.  This change 1) removes the management of separate repo files
for BaseOS and AppStream, since those repos are included in centos.repo,
and 2) stops using the perhaps-questionable single baseurl in favor of
the default metalink/mirrors setup

There are errors occurring for teuthology tests on centos9 that may
be related to this, with the errors of the form "<pkg> from <repo>
does not belong to a distupgrade repository".  As near as I can tell,
a "distupgrade repository" is one used only for upgrade, and I can't
find information on how exactly it's indicated, so I don't know if this
change will resolve the error or not.

Signed-off-by: Dan Mick <dmick@redhat.com>
15 months agoMerge pull request #754 from dmick/wip-checkcerts
Zack Cerza [Mon, 18 Mar 2024 20:07:37 +0000 (14:07 -0600)]
Merge pull request #754 from dmick/wip-checkcerts

checkcerts: fix no reporting on cert error; change From address on emails

16 months agocheckcerts: use ceph-infra-admins@redhat.com as a "From" address 754/head
Dan Mick [Wed, 13 Mar 2024 19:40:08 +0000 (12:40 -0700)]
checkcerts: use ceph-infra-admins@redhat.com as a "From" address

...so that bounces stand a chance of being seen

Signed-off-by: Dan Mick <dmick@redhat.com>
16 months agocheckcerts.py: certificate errors were not noted
Dan Mick [Wed, 13 Mar 2024 19:33:50 +0000 (12:33 -0700)]
checkcerts.py: certificate errors were not noted

When a certificate is already expired, its expiry was not noted
(loop exited early).  This stills doesn't explain the lack of early
warning, but at least it'll fix the "no email on actual errors" issue.

Signed-off-by: Dan Mick <dmick@redhat.com>
17 months agoMerge pull request #752 from ceph/wip-adding-rhel9.3
Dan Mick [Fri, 2 Feb 2024 20:03:22 +0000 (12:03 -0800)]
Merge pull request #752 from ceph/wip-adding-rhel9.3

Adding rhel 9.3 to cobbler

17 months agoAdding rhel 9.3 to cobbler 752/head
Adam Kraitman [Thu, 1 Feb 2024 16:42:51 +0000 (18:42 +0200)]
Adding rhel 9.3 to cobbler

Fixes: https://tracker.ceph.com/issues/64148
Signed-off-by: Adam Kraitman <akraitma@redhat.com>
18 months agoMerge pull request #751 from dmick/wip-cobbler-profile
Zack Cerza [Wed, 20 Dec 2023 21:53:59 +0000 (14:53 -0700)]
Merge pull request #751 from dmick/wip-cobbler-profile

cobbler_profile: fix two bugs discovered while importing inktank-rescue

19 months agocobbler_profile: fix two bugs discovered while importing inktank-rescue 751/head
Dan Mick [Wed, 13 Dec 2023 22:33:38 +0000 (14:33 -0800)]
cobbler_profile: fix two bugs discovered while importing inktank-rescue

1) ansible's get_url moves from sha256 to checksum
2) fix 'registered' name (import -> imported) in import_distro_image

Signed-off-by: Dan Mick <dmick@redhat.com>
19 months agoMerge pull request #750 from dmick/wip-releasever
Zack Cerza [Tue, 12 Dec 2023 22:10:51 +0000 (15:10 -0700)]
Merge pull request #750 from dmick/wip-releasever

19 months agoMerge pull request #749 from ceph/rhel-codeready
Dan Mick [Tue, 12 Dec 2023 21:42:56 +0000 (13:42 -0800)]
Merge pull request #749 from ceph/rhel-codeready

roles/common: Enable codeready repos on RHEL

19 months agorhel-entitlements.yml: Avoid releasever hack on all but el7 750/head
Dan Mick [Tue, 12 Dec 2023 01:36:54 +0000 (17:36 -0800)]
rhel-entitlements.yml: Avoid releasever hack on all but el7

Signed-off-by: Dan Mick <dmick@redhat.com>
19 months agoroles/common: Enable codeready repos on RHEL 749/head
Zack Cerza [Wed, 29 Nov 2023 21:08:19 +0000 (14:08 -0700)]
roles/common: Enable codeready repos on RHEL

Signed-off-by: Zack Cerza <zack@redhat.com>
20 months agoMerge pull request #745 from ceph/wip-add-esxi70
Dan Mick [Mon, 23 Oct 2023 20:50:18 +0000 (13:50 -0700)]
Merge pull request #745 from ceph/wip-add-esxi70

Adding VMware ESXi-7.0 to cobbler

22 months agoMerge pull request #748 from dmick/wip-centos-9-lab-extras
Dan Mick [Sat, 26 Aug 2023 01:23:01 +0000 (18:23 -0700)]
Merge pull request #748 from dmick/wip-centos-9-lab-extras

roles/testnode/vars/centos_9.yml: add lab-extras and iozone

22 months agoroles/testnode/vars/centos_9.yml: add lab-extras and iozone 748/head
Dan Mick [Fri, 25 Aug 2023 04:29:02 +0000 (21:29 -0700)]
roles/testnode/vars/centos_9.yml: add lab-extras and iozone

iozone is present in the newly-created lab-extras repo, for
the same reasons it's in other CentOS configs

Signed-off-by: Dan Mick <dmick@redhat.com>
23 months agoMerge pull request #747 from lxbsz/wip-62227-1
Dan Mick [Wed, 2 Aug 2023 04:23:19 +0000 (21:23 -0700)]
Merge pull request #747 from lxbsz/wip-62227-1

centos_8_stream: install dbench package from epel

23 months agocentos_8_stream: install dbench package from epel 747/head
Xiubo Li [Wed, 2 Aug 2023 04:17:11 +0000 (12:17 +0800)]
centos_8_stream: install dbench package from epel

Fixes: https://tracker.ceph.com/issues/62227
Signed-off-by: Xiubo Li <xiubli@redhat.com>
23 months agoMerge pull request #746 from lxbsz/wip-62227
Dan Mick [Wed, 2 Aug 2023 03:54:07 +0000 (20:54 -0700)]
Merge pull request #746 from lxbsz/wip-62227

centos8, centos9: install dbench package from epel instead of lab-extras

23 months agocentos_8: install dbench package from epel 746/head
Xiubo Li [Wed, 2 Aug 2023 02:14:10 +0000 (10:14 +0800)]
centos_8: install dbench package from epel

Fixes: https://tracker.ceph.com/issues/62227
Signed-off-by: Xiubo Li <xiubli@redhat.com>
23 months agocentos_9: install dbench package
Xiubo Li [Wed, 2 Aug 2023 01:12:53 +0000 (09:12 +0800)]
centos_9: install dbench package

Fixes: https://tracker.ceph.com/issues/62227
Signed-off-by: Xiubo Li <xiubli@redhat.com>
23 months agoAdding VMware ESXi-7.0 to cobbler 745/head
Adam Kraitman [Sun, 30 Jul 2023 13:54:31 +0000 (16:54 +0300)]
Adding VMware ESXi-7.0 to cobbler

Fixes: https://tracker.ceph.com/issues/61779
Signed-off-by: Adam Kraitman <akraitma@redhat.com>
2 years agoMerge pull request #740 from dmick/wip-machine-id
Zack Cerza [Tue, 27 Jun 2023 21:56:36 +0000 (15:56 -0600)]
Merge pull request #740 from dmick/wip-machine-id

2 years agocommon/tasks/rhel-entitlements.yml: regenerate machine-id 740/head
Dan Mick [Tue, 16 May 2023 01:38:17 +0000 (18:38 -0700)]
common/tasks/rhel-entitlements.yml: regenerate machine-id

We can't leave /etc/machine-id blank; it breaks things, one of
which is the kernel install, which runs a postinstall script to update
/boot/loader/entries, which does nothing (silently) if there's
nothing in /etc/machine-id.  Since it can come from the dbus id,
and does by default, and there's a command to generate the dbus
id, generate both, dbus first.  This fixes the kernel postinstall.
I don't know if there should be any correlation between
machine-id and the subscription-manager/katello IDs.

Signed-off-by: Dan Mick <dmick@redhat.com>
2 years agoMerge pull request #743 from dmick/wip-checkcerts
Zack Cerza [Mon, 26 Jun 2023 23:06:12 +0000 (17:06 -0600)]
Merge pull request #743 from dmick/wip-checkcerts

2 years agoMerge pull request #744 from ceph/wip-61648
Dan Mick [Wed, 21 Jun 2023 22:29:26 +0000 (15:29 -0700)]
Merge pull request #744 from ceph/wip-61648

testnode: replace 'powertools' with 'crb' for centos 9+

2 years agotestnode: replace 'powertools' with 'crb' for centos 9+ 744/head
Casey Bodley [Wed, 21 Jun 2023 16:51:08 +0000 (12:51 -0400)]
testnode: replace 'powertools' with 'crb' for centos 9+

Signed-off-by: Casey Bodley <cbodley@redhat.com>
2 years agotools/checkcerts.py: pretty up the output a bit for reading ease 743/head
Dan Mick [Wed, 21 Jun 2023 04:08:41 +0000 (21:08 -0700)]
tools/checkcerts.py: pretty up the output a bit for reading ease

Signed-off-by: Dan Mick <dmick@redhat.com>
2 years agoMerge pull request #741 from ceph/logs-no-sniff
Dan Mick [Wed, 21 Jun 2023 03:46:40 +0000 (20:46 -0700)]
Merge pull request #741 from ceph/logs-no-sniff

teuthology: Tell browsers not to sniff test logs

2 years agoMerge pull request #742 from dmick/main
Zack Cerza [Fri, 2 Jun 2023 18:58:00 +0000 (12:58 -0600)]
Merge pull request #742 from dmick/main

2 years agotestnode: Make sure PowerTools repo is enabled on CentOS 742/head
Dan Mick [Fri, 2 Jun 2023 09:12:59 +0000 (02:12 -0700)]
testnode: Make sure PowerTools repo is enabled on CentOS

https://github.com/ceph/ceph-cm-ansible/pull/731 removed the
custom-made repo files that added mirrorlists; however, it also
removed the side-effect of enabling the Power Tools repo (which
is not enabled by default).  This adds a call to dnf config-manager
to enable the repo, whatever its repo file name, on CentOS
testnodes.

Fixes: https://tracker.ceph.com/issues/59678
Signed-off-by: Dan Mick <dmick@redhat.com>
2 years agoteuthology: Tell browsers not to sniff test logs 741/head
Zack Cerza [Wed, 31 May 2023 22:44:01 +0000 (16:44 -0600)]
teuthology: Tell browsers not to sniff test logs

Signed-off-by: Zack Cerza <zack@redhat.com>
2 years agoMerge pull request #739 from dmick/main
Zack Cerza [Thu, 11 May 2023 21:48:15 +0000 (15:48 -0600)]
Merge pull request #739 from dmick/main

tools/checkcerts.py: add paddles.front to list of checked hosts

2 years agotools/checkcerts.py: add paddles.front to list of checked hosts 739/head
Dan Mick [Thu, 11 May 2023 21:40:05 +0000 (14:40 -0700)]
tools/checkcerts.py: add paddles.front to list of checked hosts

Signed-off-by: Dan Mick <dmick@redhat.com>
2 years agoMerge pull request #733 from ceph/tests
adam kraitman [Tue, 9 May 2023 10:32:38 +0000 (13:32 +0300)]
Merge pull request #733 from ceph/tests

github: add syntax and lint checks

2 years agoMerge pull request #738 from dmick/wip-cobbler-python-scl
Zack Cerza [Thu, 4 May 2023 18:43:24 +0000 (12:43 -0600)]
Merge pull request #738 from dmick/wip-cobbler-python-scl

cephlab_ansible.sh: use scl rh-python38 on CentOS 7

2 years agoMerge pull request #737 from dmick/wip-centos-8.5
Zack Cerza [Thu, 4 May 2023 18:42:57 +0000 (12:42 -0600)]
Merge pull request #737 from dmick/wip-centos-8.5

cobbler: Add CentOS 8.5, fix up obsolete get_url parameter

2 years agocephlab_ansible.sh: use scl rh-python38 on CentOS 7 738/head
Dan Mick [Thu, 4 May 2023 07:58:02 +0000 (00:58 -0700)]
cephlab_ansible.sh: use scl rh-python38 on CentOS 7

cephlab_ansible.sh runs at the very end of the installation process
during a cobbler install for fog image capture, on first reboot of the
freshly-cobblered system.

Cobbler runs on a CentOS 7 installation today, but its python is too
old to support modern ansible.  The SCL for python 3.8 is installed
on cobbler.  Add code here to, if installed, enable the SCL (by setting
some paths in the trigger script that is executed on the cobbler server
after the installed host reboots; a curl fetch is placed at the end of
/etc/rc.local, and this script runs to finish up all the configuration
of the host for teuthology use.

Signed-off-by: Dan Mick <dmick@redhat.com>
2 years agoAdd CentOS-8.5 (to build centos8.stream on top of newer release) 737/head
Dan Mick [Wed, 3 May 2023 23:28:08 +0000 (16:28 -0700)]
Add CentOS-8.5 (to build centos8.stream on top of newer release)

centos8.stream is an odd build because of bugs; we don't use CentOS 8.5
directly anymore, but it's now the base of the stream image.

Signed-off-by: Dan Mick <dmick@redhat.com>
2 years agocobbler_profile download_iso.yml: use checksum parameter
Dan Mick [Wed, 3 May 2023 23:26:30 +0000 (16:26 -0700)]
cobbler_profile download_iso.yml: use checksum parameter

sha256sum was deprecated, and disappeared by ansible-core 2.10.
checksum: "sha256:<sum>" is the same.

Signed-off-by: Dan Mick <dmick@redhat.com>
2 years agoMerge pull request #731 from dmick/wip-remove-centos-mirrorlists
Ken Dreyer [Tue, 25 Apr 2023 16:04:20 +0000 (12:04 -0400)]
Merge pull request #731 from dmick/wip-remove-centos-mirrorlists

Remove local mirrorlist cache for Centos8 testhosts

2 years agoRemove stream.yml from testnode role; leave repo files be 731/head
Dan Mick [Mon, 24 Apr 2023 22:59:52 +0000 (15:59 -0700)]
Remove stream.yml from testnode role; leave repo files be

Signed-off-by: Dan Mick <dmick@redhat.com>
2 years agopublic_facing: skip no-tabs linter rule on single task 733/head
Ken Dreyer [Fri, 21 Apr 2023 14:57:15 +0000 (10:57 -0400)]
public_facing: skip no-tabs linter rule on single task

Instead of skipping ansible-lint's no-tabs rule globally, apply it on
this single task that uses a tab (\t) character.

Longer-term, we could replace this tab with a space because /etc/hosts
can use either whitespace character. I'm taking a cautious approach
today for simplicity.

2 years agouse spaces around variables for ansible-lint
Ken Dreyer [Fri, 21 Apr 2023 14:48:14 +0000 (10:48 -0400)]
use spaces around variables for ansible-lint

2 years agofirmware: fix task imports for ansible-lint
Ken Dreyer [Fri, 21 Apr 2023 14:49:34 +0000 (10:49 -0400)]
firmware: fix task imports for ansible-lint

ansible-lint cannot load tasks with these longer parent paths.

load-failure: [Errno 2] No such file or directory (filenotfounderror)

2 years agorook: remove duplicate YAML key "wait"
Ken Dreyer [Thu, 20 Apr 2023 21:45:06 +0000 (17:45 -0400)]
rook: remove duplicate YAML key "wait"

caught with ansible-lint

2 years agogithub: add syntax and lint checks
Ken Dreyer [Thu, 20 Apr 2023 21:04:55 +0000 (17:04 -0400)]
github: add syntax and lint checks

Run Ansible's syntax checks and ansible-lint on every PR and Git push.

Ignore some common ansible-lint errors so we can get to green quickly.
We can re-evaluate this list later and perhaps fix some of them down the
road.

2 years agoMerge pull request #725 from dmick/main
Dan Mick [Fri, 21 Apr 2023 10:00:52 +0000 (03:00 -0700)]
Merge pull request #725 from dmick/main

tools/checkcerts.py: apparently SSLError is also possible, catch

2 years agoremove puddle playbook and role 732/head
Ken Dreyer [Thu, 20 Apr 2023 21:13:56 +0000 (17:13 -0400)]
remove puddle playbook and role

This (internal) host is retired.

2 years agoRemove mirrorlists for CentOS 8
Dan Mick [Thu, 20 Apr 2023 20:50:12 +0000 (13:50 -0700)]
Remove mirrorlists for CentOS 8

They were failing similarly to EPEL mirrorlists (old broken mirror
machines, out-of-date lists), so let's try going back to out-of-the-box
repo configurations.  Perhaps several years later they'll work better.

Signed-off-by: Dan Mick <dmick@redhat.com>
2 years agoRemove "switch back from mirrorlist" code for CentOS
Dan Mick [Thu, 20 Apr 2023 20:26:12 +0000 (13:26 -0700)]
Remove "switch back from mirrorlist" code for CentOS

The plan is to use mirrorlist exclusively (as we've done for
EPEL) because the upstream infra is changing more rapidly than
our fixed list of mirrors, and hopefully it's more stable than
it was in the past when we were driven to this coping mechanism
of caching mirror lists.

Signed-off-by: Dan Mick <dmick@redhat.com>
2 years agoMerge pull request #730 from ceph/rm-beta
adam kraitman [Thu, 20 Apr 2023 13:15:16 +0000 (16:15 +0300)]
Merge pull request #730 from ceph/rm-beta

common: remove RHEL beta handling

2 years agocommon: remove RHEL beta handling 730/head
Ken Dreyer [Mon, 17 Apr 2023 20:56:40 +0000 (16:56 -0400)]
common: remove RHEL beta handling

Now that mainly care about containers and we test against CentOS Stream,
there's almost no value in testing Ceph with RHEL Beta content.

The rhel-entitlements.yml playbook is very complicated, and dropping
this feature will make it easier to simply and refactor further.

2 years agoMerge pull request #729 from ceph/epel-major-version
adam kraitman [Mon, 17 Apr 2023 20:07:12 +0000 (23:07 +0300)]
Merge pull request #729 from ceph/epel-major-version

common: use ansible_distribution_major_version in epel repos

2 years agocommon: use ansible_distribution_major_version in epel repos 729/head
Ken Dreyer [Mon, 17 Apr 2023 19:28:22 +0000 (15:28 -0400)]
common: use ansible_distribution_major_version in epel repos

RHEL systems use roles/common/tasks/rhel-entitlements.yml, and this sets
Yum's $releasever to a specific RHEL minor release (eg. 8.4 or 8.6). As
a result. Fedora's MirrorManager does not return any EPEL repositories
for these minor RHEL versions.

We set a static $releasever in rhel-entitlements.yml so that we pin to
old RHEL RPM content in our old RHEL nodes. We probably need to re-think
this strategy since our CentOS Stream nodes do not (cannot) do this, and
Red Hat does not really support pinning to old versions without an EUS
subscription.

Rather than untangling all that and removing our $releasever
manipulation altogether, this commit simply hard-codes
ansible_distribution_major_version ("8", "9", etc) into the EPEL .repo
files, ignoring $releasever for EPEL.

A longer-term fix would be to stop mangling $releasever on RHEL.