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 8F3932BE65D for ; Fri, 25 Jul 2025 08:36:04 +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=1753432564; cv=none; b=YD4/BINJbP/BlM/1MfQBOneD0obyuwzJqeiFj1KAJlVnZKdvTZAgW6TuXB496RUSyNnCoXOfBC09FyXocszRvl916PjYEbYWEcWn2kxQFqnw/nrPbbJF2ins864BBUW2XgcKggKFmvrfHMFftShum3LJEEk/3gHk35zpbheduD4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1753432564; c=relaxed/simple; bh=/Xvr/VdGUXB38ELZ2FTu+jBAL9gAwCG9xxi0b00iVLo=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=UkAwy9aMWO0a3/0YnBgZQmHwJg4uQXogpiNnopBk79FnW2PGUm3SYs7on2MeyPn1nz5pcR43gMHw9KqJOISS5zL4dCf0Htj63aZGFtgflhExCoZSWhg0mo9IwMPWnmld+TTb49GE4B6HNpd0WmY3crDRvmjyBFILeA7nohjRPIA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=KZohlV5H; 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="KZohlV5H" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 16365C4CEE7; Fri, 25 Jul 2025 08:36:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1753432564; bh=/Xvr/VdGUXB38ELZ2FTu+jBAL9gAwCG9xxi0b00iVLo=; h=Date:Reply-To:Subject:To:Cc:References:From:In-Reply-To:From; b=KZohlV5HfOq+rH3HnKQqyJUkBTC7tDq1CcfASDXbKIkseP+5P950v/zDMRkctro/Z +NPphgduSqPdzN3mrLPT14nmYFvSMb0hhjxH1rBuiXhcsfagi/ryjDNY7UEGBAtkeL k8SZuXp9tt1OaFgN1JHy22RvFsqttBCkChIPY3izk/jwz/C/wF//W+OtONjgVh92dO EoX0DOv6yAfBqjKPhBseAj/7O8HFMtgf3cNZ6uefxvGRQtAwmf8VYDO0Nl/AfDc7Xu Z0aBgw/2SiuEpAreycszU1unjVJkXOLqX3NLkfmGiySGPr/+CviWCylH1rZRCPBGS/ 3MeXpmhdPjv4Q== Message-ID: <53c1819b-e1f7-4138-b7e2-34cc855874eb@kernel.org> Date: Fri, 25 Jul 2025 10:36:01 +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: Daniel Gomez Subject: Re: [PATCH v5 07/12] gen_hosts: add service group to inventory templates To: Chuck Lever , Luis Chamberlain Cc: kdevops@lists.linux.dev, Daniel Gomez , Jeff Layton References: <20250724-ansible_cfg_inventory-v5-0-9bf1db24d9fa@samsung.com> <20250724-ansible_cfg_inventory-v5-7-9bf1db24d9fa@samsung.com> <86f19a3e-2a3e-4b47-b61a-aac67df6dfa9@oracle.com> Content-Language: en-US From: Daniel Gomez Organization: kernel.org In-Reply-To: <86f19a3e-2a3e-4b47-b61a-aac67df6dfa9@oracle.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 24/07/2025 22.02, Chuck Lever wrote: > On 7/24/25 3:36 PM, Daniel Gomez wrote: >> From: Daniel Gomez >> >> Introduce a new 'service' group in all inventory templates that includes >> service hosts (iscsi, nfsd, smbd, kdc). This provides a convenient way >> to target all service hosts collectively for operations like guestfs >> bringup and SSH configuration. >> >> The service group includes: >> - iscsi hosts (when kdevops_enable_iscsi is enabled) >> - nfsd hosts (when kdevops_nfsd_enable is enabled) >> - smbd hosts (when kdevops_smbd_enable is enabled) >> - kdc hosts (when kdevops_krb5_enable is enabled) >> >> This addresses issues where operations limited to 'baseline:dev' would >> miss required service hosts, causing workflow failures. >> >> Generated-by: Claude AI >> >> Signed-off-by: Daniel Gomez >> --- >> playbooks/roles/gen_hosts/templates/fstests.j2 | 15 +++++++++++++++ >> playbooks/roles/gen_hosts/templates/gitr.j2 | 9 +++++++++ >> playbooks/roles/gen_hosts/templates/hosts.j2 | 9 +++++++++ >> playbooks/roles/gen_hosts/templates/nfstest.j2 | 9 +++++++++ >> playbooks/roles/gen_hosts/templates/pynfs.j2 | 7 +++++++ >> 5 files changed, 49 insertions(+) >> >> diff --git a/playbooks/roles/gen_hosts/templates/fstests.j2 b/playbooks/roles/gen_hosts/templates/fstests.j2 >> index abcdafd..a489c12 100644 >> --- a/playbooks/roles/gen_hosts/templates/fstests.j2 >> +++ b/playbooks/roles/gen_hosts/templates/fstests.j2 >> @@ -69,3 +69,18 @@ ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> [krb5:vars] >> ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> {% endif %} >> +[service] >> +{% if kdevops_enable_iscsi %} >> +{{ kdevops_hosts_prefix }}-iscsi >> +{% endif %} >> +{% if kdevops_nfsd_enable %} >> +{{ kdevops_hosts_prefix }}-nfsd >> +{% endif %} >> +{% if kdevops_smbd_enable %} >> +{{ kdevops_hosts_prefix }}-smbd >> +{% endif %} >> +{% if kdevops_krb5_enable %} >> +{{ kdevops_hosts_prefix }}-kdc >> +{% endif %} >> +[service:vars] >> +ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> diff --git a/playbooks/roles/gen_hosts/templates/gitr.j2 b/playbooks/roles/gen_hosts/templates/gitr.j2 >> index b8f393d..41aee2b 100644 >> --- a/playbooks/roles/gen_hosts/templates/gitr.j2 >> +++ b/playbooks/roles/gen_hosts/templates/gitr.j2 >> @@ -37,3 +37,12 @@ ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> [nfsd:vars] >> ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> {% endif %} >> +[service] >> +{% if kdevops_enable_iscsi %} >> +{{ kdevops_hosts_prefix }}-iscsi >> +{% endif %} >> +{% if kdevops_nfsd_enable %} >> +{{ kdevops_hosts_prefix }}-nfsd >> +{% endif %} >> +[service:vars] >> +ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> diff --git a/playbooks/roles/gen_hosts/templates/hosts.j2 b/playbooks/roles/gen_hosts/templates/hosts.j2 >> index 5032f59..639f5a4 100644 >> --- a/playbooks/roles/gen_hosts/templates/hosts.j2 >> +++ b/playbooks/roles/gen_hosts/templates/hosts.j2 >> @@ -44,4 +44,13 @@ ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> [nfsd:vars] >> ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> {% endif %} >> +[service] >> +{% if kdevops_enable_iscsi %} >> +{{ kdevops_hosts_prefix }}-iscsi >> +{% endif %} >> +{% if kdevops_nfsd_enable %} >> +{{ kdevops_hosts_prefix }}-nfsd >> +{% endif %} >> +[service:vars] >> +ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> {% endif %} >> diff --git a/playbooks/roles/gen_hosts/templates/nfstest.j2 b/playbooks/roles/gen_hosts/templates/nfstest.j2 >> index 6a0d6cd..fb47471 100644 >> --- a/playbooks/roles/gen_hosts/templates/nfstest.j2 >> +++ b/playbooks/roles/gen_hosts/templates/nfstest.j2 >> @@ -37,3 +37,12 @@ ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> [nfsd:vars] >> ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> {% endif %} >> +[service] >> +{% if kdevops_enable_iscsi %} >> +{{ kdevops_hosts_prefix }}-iscsi >> +{% endif %} >> +{% if kdevops_nfsd_enable %} >> +{{ kdevops_hosts_prefix }}-nfsd >> +{% endif %} >> +[service:vars] >> +ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> diff --git a/playbooks/roles/gen_hosts/templates/pynfs.j2 b/playbooks/roles/gen_hosts/templates/pynfs.j2 >> index c018c54..cac6fc6 100644 >> --- a/playbooks/roles/gen_hosts/templates/pynfs.j2 >> +++ b/playbooks/roles/gen_hosts/templates/pynfs.j2 >> @@ -22,3 +22,10 @@ ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> {{ kdevops_hosts_prefix }}-nfsd >> [nfsd:vars] >> ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> +[service] >> +{% if kdevops_enable_iscsi %} >> +{{ kdevops_hosts_prefix }}-iscsi >> +{% endif %} >> +{{ kdevops_hosts_prefix }}-nfsd >> +[service:vars] >> +ansible_python_interpreter = "{{ kdevops_python_interpreter }}" >> > > The issue with the pynfs workflow is that the "nfsd" host is also where > the tests run. I think what you're doing is adding that host into both > the test-runners group and the new "service" group, which is probably > OK. That is correct. > > Would it be better if the pynfs workflow created a separate test runner > and nfsd host? Or even two test runners, one for NFSv4.0 and one for > NFSv4.1 ? That way the pynfs workflow would operate like the others. You mean like in fstests, right? FSTESTS_NFS_SECTION_V42 FSTESTS_NFS_SECTION_V41 FSTESTS_NFS_SECTION_V40 I agree pynfs is not consistent with the rest of the workflows. But I'm not familiar with pynfs to agree if your suggestion make sense. Can you clarify if your suggestion means having 2 kdevops profiles (i.e. 2 separate guests/vms) with NFSv4.0 and NFS4.1 with both the option to run CONFIG_PYNFS_PNFS_BLOCK or not. How is it done now? Does the test switch between NFS versions or uses 2 different NFS mount points to run the pynfs suite tests on them? Would nfsd guest support both new vms? > > Just a random thought, not an objection. Would you agree to do this separately?