* [PATCH] update_etc_hosts: include the "service" hosts in the update
@ 2025-08-27 14:16 Jeff Layton
2025-08-27 14:22 ` Chuck Lever
0 siblings, 1 reply; 2+ messages in thread
From: Jeff Layton @ 2025-08-27 14:16 UTC (permalink / raw)
To: kdevops; +Cc: Daniel Gomez, Chuck Lever, Jeff Layton
Services on the network need to have their hostnames added on the
clients, and adding the client hostnames on the servers should be
harmless.
Fixes: 1cf0800c9ffc ("gen_hosts: templates: include localhost in the all group")
Signed-off-by: Jeff Layton <jlayton@kernel.org>
---
This seems to fix the issue for me. Any objections?
---
playbooks/update_etc_hosts.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/playbooks/update_etc_hosts.yml b/playbooks/update_etc_hosts.yml
index 6fb8f7870b4bba137aaf7c1eb50d858b17550244..a8e9bc26929079c6ce46d26e9a4df60f178bfb23 100644
--- a/playbooks/update_etc_hosts.yml
+++ b/playbooks/update_etc_hosts.yml
@@ -1,6 +1,6 @@
---
- name: Update target /etc/hosts with all targets and disable cloud-init
- hosts: baseline:dev
+ hosts: baseline:dev:service
gather_facts: false
roles:
- role: update_etc_hosts
---
base-commit: e18ea52dee6d5f6760e7b177ac5cbded65dc61e1
change-id: 20250827-ueh-fix-d7bdb7333fdf
Best regards,
--
Jeff Layton <jlayton@kernel.org>
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] update_etc_hosts: include the "service" hosts in the update
2025-08-27 14:16 [PATCH] update_etc_hosts: include the "service" hosts in the update Jeff Layton
@ 2025-08-27 14:22 ` Chuck Lever
0 siblings, 0 replies; 2+ messages in thread
From: Chuck Lever @ 2025-08-27 14:22 UTC (permalink / raw)
To: Jeff Layton, kdevops; +Cc: Daniel Gomez
On 8/27/25 10:16 AM, Jeff Layton wrote:
> Services on the network need to have their hostnames added on the
> clients, and adding the client hostnames on the servers should be
> harmless.
>
> Fixes: 1cf0800c9ffc ("gen_hosts: templates: include localhost in the all group")
> Signed-off-by: Jeff Layton <jlayton@kernel.org>
> ---
> This seems to fix the issue for me. Any objections?
> ---
> playbooks/update_etc_hosts.yml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/playbooks/update_etc_hosts.yml b/playbooks/update_etc_hosts.yml
> index 6fb8f7870b4bba137aaf7c1eb50d858b17550244..a8e9bc26929079c6ce46d26e9a4df60f178bfb23 100644
> --- a/playbooks/update_etc_hosts.yml
> +++ b/playbooks/update_etc_hosts.yml
> @@ -1,6 +1,6 @@
> ---
> - name: Update target /etc/hosts with all targets and disable cloud-init
> - hosts: baseline:dev
> + hosts: baseline:dev:service
> gather_facts: false
> roles:
> - role: update_etc_hosts
>
> ---
> base-commit: e18ea52dee6d5f6760e7b177ac5cbded65dc61e1
> change-id: 20250827-ueh-fix-d7bdb7333fdf
>
> Best regards,
Reviewed-by: Chuck Lever <chuck.lever@oracle.com>
--
Chuck Lever
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-08-27 14:22 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-27 14:16 [PATCH] update_etc_hosts: include the "service" hosts in the update Jeff Layton
2025-08-27 14:22 ` Chuck Lever
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox