From 4c5030e4784e26466dfa286325ae9c49d98cb1ba Mon Sep 17 00:00:00 2001 From: Ian Wienand Date: Wed, 26 Apr 2023 10:54:46 +1000 Subject: [PATCH] openafs-client: get logs better I'm not sure if something changed in dkms, but this log file is helpful on centos 9-stream and the other check doesn't match anything. Also update the README.rst slightly to be more in line with reality. Change-Id: Ic8cab980ef43490eb1b3ca0b7a0d0c2329bb94ce --- roles-test/post.yaml | 4 ++++ roles/openafs-client/README.rst | 6 +++--- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/roles-test/post.yaml b/roles-test/post.yaml index 584cc1a425..14bfe0fb6f 100644 --- a/roles-test/post.yaml +++ b/roles-test/post.yaml @@ -13,7 +13,11 @@ shell: | DKMS_MAKE_LOGS=/var/lib/dkms-make-logs mkdir ${DKMS_MAKE_LOGS} + # NOTE(ianw) 2023-04-27 This follwoing find may be redundant + # now; it does not seem to match anything on centos 9-stream + # at least. find /var/lib/dkms/ -type d -wholename "*/log" -exec cp -r --parents \{\} ${DKMS_MAKE_LOGS} \; + cp /var/lib/dkms/openafs/*/build/make.log ${DKMS_MAKE_LOGS} chown zuul:zuul ${DKMS_MAKE_LOGS} become: yes - include_role: diff --git a/roles/openafs-client/README.rst b/roles/openafs-client/README.rst index 5934709ef5..6894a21bce 100644 --- a/roles/openafs-client/README.rst +++ b/roles/openafs-client/README.rst @@ -2,8 +2,8 @@ An ansible role to configure an OpenAFS client .. note:: This role uses system packages where available, but for platforms or architectures where they are not available will - require external builds. Defaults will pick external - packages from OpenStack Infra builds, but you should + utilise external packages. Defaults will pick packages + built from the OpenDev infra project, but you should evaluate if this is suitable for your environment. This role configures the host to be an `OpenAFS @@ -13,7 +13,7 @@ role has limited platform support. Currently supported are * Debian family with system packages available * Ubuntu LTS family with external 1.8 series packages -* CentOS 7 and 8 with external packages +* CentOS 7, 8-stream and 9-stream with external packages **Role Variables**