cloud-init-dev team mailing list archive

[Antonio Romito <antonio.romito@xxxxxxxxxx>]

The proposal to merge ~aromito/cloud-init:cloud-init-BZ_1673170 into cloud-init:master has been updated.

Description changed to:

Applying the following patch the script make a double check on the keys present into the array before the if statement.

diff --git a/cloudinit/cmd/main.py b/cloudinit/cmd/main.py
index 933c019..a8e645a 100644
--- a/cloudinit/cmd/main.py
+++ b/cloudinit/cmd/main.py
@@ -673,8 +673,9 @@ def status_wrapper(name, args, data_d=None, link_d=None):
         # write the 'finished' file
         errors = []
         for m in modes:
-            if v1[m]['errors']:
-                errors.extend(v1[m].get('errors', []))
+            if m in v1:
+                if v1[m]['errors']:
+                    errors.extend(v1[m].get('errors', []))
 
         atomic_helper.write_json(
             result_path, {'v1': {'datasource': v1['datasource'],

For more details, see:
https://code.launchpad.net/~aromito/cloud-init/+git/cloud-init/+merge/362884
-- 
Your team cloud-init commiters is requested to review the proposed merge of ~aromito/cloud-init:cloud-init-BZ_1673170 into cloud-init:master.