Re: [PATCH v2 33/37] devconfig: include the "service" group in the playbooks hosts list

[Daniel Gomez <da.gomez@kernel.org>]

On 02/09/2025 15.54, Chuck Lever wrote:
> From: Chuck Lever <chuck.lever@oracle.com>
> 
> Since commit 1cf0800c9ffc ("gen_hosts: templates: include localhost
> in the all group"), the devconfig playbook is no longer updating the
> nfsd service host.
> 
> 1. Actually all the service hosts need to be updated by devconfig,
>    so ensure devconfig runs against the service group.
> 
> 2. Since devconfig can run against most workflows, add blank
>    [service] groups to any inventory template that doesn't already
>    have it.
> 
> Fixes: 1cf0800c9ffc ("gen_hosts: templates: include localhost in the all group")
> Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
> ---
>  playbooks/devconfig.yml                          | 2 +-
>  playbooks/roles/gen_hosts/templates/blktests.j2  | 3 +++
>  playbooks/roles/gen_hosts/templates/cxl.j2       | 3 +++
>  playbooks/roles/gen_hosts/templates/ltp.j2       | 4 ++++
>  playbooks/roles/gen_hosts/templates/mmtests.j2   | 3 +++
>  playbooks/roles/gen_hosts/templates/selftests.j2 | 3 +++
>  playbooks/roles/gen_hosts/templates/sysbench.j2  | 3 +++
>  scripts/devconfig.Makefile                       | 9 ++++-----
>  scripts/firstconfig.Makefile                     | 2 +-
>  scripts/provision.Makefile                       | 2 +-
>  10 files changed, 26 insertions(+), 8 deletions(-)
> 
> diff --git a/playbooks/devconfig.yml b/playbooks/devconfig.yml
> index cebaa87a7228..f0ca7d767efe 100644
> --- a/playbooks/devconfig.yml
> +++ b/playbooks/devconfig.yml
> @@ -1,6 +1,6 @@
>  ---
>  - name: Configure developer environment on target systems
> -  hosts: baseline:dev
> +  hosts: baseline:dev:service
>    gather_facts: false
>    roles:
>      - role: devconfig
> diff --git a/playbooks/roles/gen_hosts/templates/blktests.j2 b/playbooks/roles/gen_hosts/templates/blktests.j2
> index 5bf50acbddac..6aa434c3383c 100644
> --- a/playbooks/roles/gen_hosts/templates/blktests.j2
> +++ b/playbooks/roles/gen_hosts/templates/blktests.j2
> @@ -22,3 +22,6 @@ ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
>  {% endif %}
>  [dev:vars]
>  ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
> +[service]
> +[service:vars]
> +ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
> diff --git a/playbooks/roles/gen_hosts/templates/cxl.j2 b/playbooks/roles/gen_hosts/templates/cxl.j2
> index 64bec4561b6c..cab7da246b11 100644
> --- a/playbooks/roles/gen_hosts/templates/cxl.j2
> +++ b/playbooks/roles/gen_hosts/templates/cxl.j2
> @@ -16,3 +16,6 @@ ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
>  {% endif %}
>  [dev:vars]
>  ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
> +[service]
> +[service:vars]
> +ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
> diff --git a/playbooks/roles/gen_hosts/templates/ltp.j2 b/playbooks/roles/gen_hosts/templates/ltp.j2
> index 828fd6362718..6946b3f3593b 100644
> --- a/playbooks/roles/gen_hosts/templates/ltp.j2
> +++ b/playbooks/roles/gen_hosts/templates/ltp.j2
> @@ -24,3 +24,7 @@ ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
>  {% endif %}
>  [dev:vars]
>  ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
> +
> +[service]
> +[service:vars]
> +ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
> diff --git a/playbooks/roles/gen_hosts/templates/mmtests.j2 b/playbooks/roles/gen_hosts/templates/mmtests.j2
> index 1252fe876e6c..8df4a3c31213 100644
> --- a/playbooks/roles/gen_hosts/templates/mmtests.j2
> +++ b/playbooks/roles/gen_hosts/templates/mmtests.j2
> @@ -31,3 +31,6 @@ ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
>  {% endfor %}
>  [mmtests:vars]
>  ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
> +[service]
> +[service:vars]
> +ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
> diff --git a/playbooks/roles/gen_hosts/templates/selftests.j2 b/playbooks/roles/gen_hosts/templates/selftests.j2
> index e505bbf194b4..af0d6b341048 100644
> --- a/playbooks/roles/gen_hosts/templates/selftests.j2
> +++ b/playbooks/roles/gen_hosts/templates/selftests.j2
> @@ -22,3 +22,6 @@ ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
>  {% endif %}
>  [dev:vars]
>  ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
> +[service]
> +[service:vars]
> +ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
> diff --git a/playbooks/roles/gen_hosts/templates/sysbench.j2 b/playbooks/roles/gen_hosts/templates/sysbench.j2
> index 107eebc5fbce..e30773c1b84a 100644
> --- a/playbooks/roles/gen_hosts/templates/sysbench.j2
> +++ b/playbooks/roles/gen_hosts/templates/sysbench.j2
> @@ -22,3 +22,6 @@ ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
>  {% endif %}
>  [dev:vars]
>  ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
> +[service]
> +[service:vars]
> +ansible_python_interpreter =  "{{ kdevops_python_interpreter }}"
> diff --git a/scripts/devconfig.Makefile b/scripts/devconfig.Makefile
> index 146c70f78897..fedf1f2ca21f 100644
> --- a/scripts/devconfig.Makefile
> +++ b/scripts/devconfig.Makefile
> @@ -57,11 +57,10 @@ extend-extra-args-devconfig:
>  PHONY += devconfig
>  devconfig: $(KDEVOPS_NODES)
>  	$(Q)ansible-playbook $(ANSIBLE_VERBOSE) \
> -		--limit 'baseline:dev:nfsd' \
> +		--limit 'baseline:dev:service' \

You don't need --limit at all (I missed this cleanup!). The playbook already has
hosts: baseline:dev:service.

>  		$(KDEVOPS_PLAYBOOKS_DIR)/devconfig.yml \
>  		--extra-vars="$(BOOTLINUX_ARGS)" \
> -		--extra-vars '{ kdevops_cli_install: True }' \
> -		$(LIMIT_HOSTS)
> +		--extra-vars '{ kdevops_cli_install: True }'

Any reason to remove LIMIT_HOSTS support from here? Maybe not useful for these
targets?

>  
>  devconfig-generic-help-menu:
>  	@echo "devconfig          - Ensures generic system setup and is up to date"
> @@ -72,9 +71,9 @@ ifeq (y,$(CONFIG_SYSCTL_TUNING))
>  PHONY += sysctl-tunings
>  sysctl-tunings: $(KDEVOPS_NODES)
>  	$(Q)ansible-playbook $(ANSIBLE_VERBOSE) \
> -		--limit 'baseline:dev:nfsd' \
> +		--limit 'baseline:dev:service' \

Same here. Not needed.

>  		$(KDEVOPS_PLAYBOOKS_DIR)/devconfig.yml \
> -		--extra-vars="$(BOOTLINUX_ARGS)" $(LIMIT_HOSTS) --tags vars,sysctl
> +		--extra-vars="$(BOOTLINUX_ARGS)" --tags vars,sysctl
>  
>  devconfig-help-menu:
>  	@echo "Target node configuration options"
> diff --git a/scripts/firstconfig.Makefile b/scripts/firstconfig.Makefile
> index 9f8a63a8ff84..5651a07d0b66 100644
> --- a/scripts/firstconfig.Makefile
> +++ b/scripts/firstconfig.Makefile
> @@ -2,7 +2,7 @@
>  
>  firstconfig:
>  	$(Q)ansible-playbook $(ANSIBLE_VERBOSE) \
> -		--limit 'baseline:dev' \
> +		--limit 'baseline:dev:service' \

Same here...

>  		--extra-vars '{ kdevops_cli_install: True }' \
>  		--tags vars_simple,firstconfig \
>  		$(KDEVOPS_PLAYBOOKS_DIR)/devconfig.yml
> diff --git a/scripts/provision.Makefile b/scripts/provision.Makefile
> index f04264f6b9ea..62a5692904c9 100644
> --- a/scripts/provision.Makefile
> +++ b/scripts/provision.Makefile
> @@ -70,7 +70,7 @@ KDEVOPS_MRPROPER += $(KDEVOPS_PROVISIONED_DEVCONFIG)
>  $(KDEVOPS_PROVISIONED_DEVCONFIG):
>  	$(Q)if [[ "$(CONFIG_KDEVOPS_ANSIBLE_PROVISION_PLAYBOOK)" != "" ]]; then \
>  		ansible-playbook $(ANSIBLE_VERBOSE) \
> -			--limit 'baseline:dev' \
> +			--limit 'baseline:dev:service' \

And here (playbook is devconfig.yml)

>  			$(KDEVOPS_PLAYBOOKS_DIR)/$(KDEVOPS_ANSIBLE_PROVISION_PLAYBOOK) ;\
>  	fi
>  	$(Q)touch $(KDEVOPS_PROVISIONED_DEVCONFIG)