From f9e38131a7a580c978ef4ab2342e6264694092e5 Mon Sep 17 00:00:00 2001 From: Ian Wienand Date: Wed, 27 Jul 2022 17:06:59 +1000 Subject: [PATCH] emit-job-header: noqa on error ignore This marks this particular ignore_errors as safe. Generally I think the rule is good, in that tasks get marked as failed when using ignore_errors which is confusing and hard to debug. But in this case it's because of missing hostnames (I8d67d8aa284b6ce9ae012608e8f1b12784ce836b) and not something we can catch easily. Change-Id: I9b5d6a85254f689e97bca91951632b4dea60cfde --- roles/emit-job-header/tasks/main.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/roles/emit-job-header/tasks/main.yaml b/roles/emit-job-header/tasks/main.yaml index 969feed79..3d429f503 100644 --- a/roles/emit-job-header/tasks/main.yaml +++ b/roles/emit-job-header/tasks/main.yaml @@ -39,4 +39,4 @@ loop: "{{ query('inventory_hostnames', 'all,!localhost') }}" loop_control: loop_var: zj_item - ignore_errors: yes + ignore_errors: yes # noqa ignore-errors