From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5FAB08F6F for ; Sat, 17 May 2025 20:34:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1747514097; cv=none; b=tdoGNBGJFNulEQzzSf6ORtmAIWvdCsBLFuGkaqfGJGB1nl5z+4RuURo+b8mHDLBQsEAFJunpkY/8yF4UWu1R5eMHpjliLSScLR6dx2jBX382B7nzf/GlLC08MU+JhTRc2/GItW27P2YKEWY0EDPBLou8Cgo0r1JMJMeWyY8dlcI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1747514097; c=relaxed/simple; bh=j2daMJUYlKbFtg+nMUjsfJAdwOTWyvZ3f5Kwx5zUMoQ=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=JA+BkgGpPBc/WFp1THvEk+dKVdeA+zd6QLBFah5Py1T79kl6lGnZsWs1AKY8VOikg0L51pZ3jrYTQNM961qEPnO0JWsauzzGqQiI/ZLVEXNXrpEnbY31tCnIK0PI948z3Fxw+5SQRG1fFpt0KGgWaquBWScJhgcVUoHXQfCu2vU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=usYEvV2F; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="usYEvV2F" Received: by smtp.kernel.org (Postfix) with ESMTPSA id E8BC5C4CEE3; Sat, 17 May 2025 20:34:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1747514096; bh=j2daMJUYlKbFtg+nMUjsfJAdwOTWyvZ3f5Kwx5zUMoQ=; h=Date:Reply-To:Subject:To:Cc:References:From:In-Reply-To:From; b=usYEvV2FyQuOZw7Zo3OU3ypH2qL3pxDJg04uj37/zhTeGjZF/NFOpjC61Mf2z3Fzz hVkrSyJmA17rmXYjfgbHsvXo64m0VyV0MSEbO3FxhPKoSoDUxdSN3FQ1v218x8/vQ5 TLL9MtVrsaT7Ld96ojnh3w085zcI5SJZxPJPQcJ9lChdfBhfFrvxKv5CMRS59TYKCA bYtVrBhZBy4IockiPTxT/YDDFQpRIHPxcwTqUDMhMK6EBYvKkTVAmb+D5JFdKuCcZM mYN7KQBgkt8QE6U+w0og4tPzVUGxez7A4rWymm0FkjS2kQVQMjem3g0g6+LTJUmdqA gz2smvgrG1H0g== Message-ID: <3a2f00d5-49cb-4ecc-97b4-5ecff937f688@kernel.org> Date: Sat, 17 May 2025 22:34:53 +0200 Precedence: bulk X-Mailing-List: kdevops@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Reply-To: da.gomez@kernel.org Subject: Re: [PATCH v3 01/10] playbooks: fix playbook name for all hosts plays To: Chuck Lever , Luis Chamberlain Cc: kdevops@lists.linux.dev, Daniel Gomez References: <20250505-ansible_cfg_inventory-v3-0-a153d93c35c4@samsung.com> <20250505-ansible_cfg_inventory-v3-1-a153d93c35c4@samsung.com> <6dedbabd-79b5-4350-8a11-287cc33de568@oracle.com> <7cd31cfc-751e-41c2-a9e5-8962d842edd9@kernel.org> <36173f19-3bca-4089-9974-b2f4e7e61af0@oracle.com> Content-Language: en-US From: Daniel Gomez Organization: kernel.org In-Reply-To: <36173f19-3bca-4089-9974-b2f4e7e61af0@oracle.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 17/05/2025 21.43, Chuck Lever wrote: > On 5/16/25 8:51 AM, Daniel Gomez wrote: >> >> >> On 06/05/2025 22.39, Chuck Lever wrote: >>> On 5/5/25 5:24 PM, Daniel Gomez wrote: >>>> From: Daniel Gomez >>>> >>>> Fix name rule by naming all the playbooks that run in all hosts. >>>> >>>> https://ansible.readthedocs.io/projects/lint/rules/name/ >>> >>> Nit: name[casing] states these names must begin with an upper case >>> letter. Ditto for 02/10. >>> >>> Would be slightly nicer, perhaps, if these names were documentary >>> sentences, as they are in the roles themselves. >> >> I'm getting a bit of “buddy” assistance on some playbooks I'm less/not >> familiar with. Here's the first pass covering about half of them. > > Some suggestions below. Thanks! > > >> --- a/playbooks/blktests.yml >> +++ b/playbooks/blktests.yml >> -- name: blktests >> +- name: Linux kernel block layer tests (blktests) > > "Configure and run the block layer testing workflow" > > >> --- a/playbooks/bootlinux-local.yml >> +++ b/playbooks/bootlinux-local.yml >> -- name: bootlinux-local >> +- name: Bootlinux (localhost only) > > I have a patch or two that folds this role into bootlinux. > Should I send it now, or wait until your clean-up is merged? I still need to test the crash CI script and update the playbook description. That said, I'd go ahead and send the bootlinux cleanup patches now as they will also be helpful for the bisect workflow. > > >> --- a/playbooks/bringup_guestfs.yml >> +++ b/playbooks/bringup_guestfs.yml >> -- name: bringup_guestfs >> +- name: Bringup Guests (libguestfs) > > "Bring up libvirt target nodes" > > >> --- a/playbooks/build_qemu.yml >> +++ b/playbooks/build_qemu.yml >> -- name: build_qemu >> +- name: Build QEMU >> --- a/playbooks/common.yml >> +++ b/playbooks/common.yml >> -- name: common >> +- name: Common (shared role) > > "Common tasks" > > >> --- a/playbooks/create_data_partition.yml >> +++ b/playbooks/create_data_partition.yml >> -- name: create_data_partition >> +- name: Create Directory Data Path (e.g. /data partition) >> --- a/playbooks/devconfig.yml >> +++ b/playbooks/devconfig.yml >> -- name: devconfig >> +- name: Configure developer environment on target systems >> --- a/playbooks/fstests.yml >> +++ b/playbooks/fstests.yml >> -- name: fstests >> +- name: "Filesystem testing suite setup - (x)fstests (control node only)" >> -- hosts: baseline:dev >> +- name: Filesystem testing suite setup - (x)fstests >> +  hosts: baseline:dev >> --- a/playbooks/pynfs.yml >> +++ b/playbooks/pynfs.yml >> -- name: pynfs >> +- name: Configure and run pynfs testing workflow >> --- a/playbooks/reboot-limit.yml >> +++ b/playbooks/reboot-limit.yml >> -- name: reboot-limit >> +- name: reboot-limit stability test workflow > > "Configure and run the reboot-limit workflow" > > >> --- a/playbooks/rxe.yml >> +++ b/playbooks/rxe.yml >> -- name: rxe >> +- name: Set up software-emulated RoCE RDMA over Ethernet (rxe) via udev >> --- a/playbooks/selftests.yml >> +++ b/playbooks/selftests.yml >> -- name: selftests >> +- name: Set up and run Linux kernel selftests (kselftests) >> --- a/playbooks/siw.yml >> +++ b/playbooks/siw.yml >> -- name: siw >> +- name: Set up software-emulated iWARP RDMA over TCP/IP (siw) via udev >> --- a/playbooks/smbd.yml >> +++ b/playbooks/smbd.yml >> -- name: smbd >> +- name: Set up Samba server with shared volume and system integration >> --- a/playbooks/sysbench.yml >> +++ b/playbooks/sysbench.yml >> -- name: sysbench >> +- name: Sysbench workflow > > "Configure and run the sysbench workflow" > > >> --- a/playbooks/terraform.yml >> +++ b/playbooks/terraform.yml >> -- name: terraform >> +- name: Manage infrastructure lifecycle and SSH access with Terraform >> --- a/playbooks/update_etc_hosts.yml >> +++ b/playbooks/update_etc_hosts.yml >> -- name: update_etc_hosts >> +- name: Update target /etc/hosts with all targets and disable cloud-init >> --- a/playbooks/update_ssh_config_guestfs.yml >> +++ b/playbooks/update_ssh_config_guestfs.yml >> -- name: update_ssh_config_guestfs >> +- name: Update OpenSSH client configuration file (libguestfs) >> >> > >