Adds workaround for requires_ansible

Collections on galaxy.ansible.com may not have the expected `requires_ansible` data. closes pulp#806
bmbouter · Jan 27, 2022 · 98117a8 · 98117a8
1 parent 750e8b0
commit 98117a8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGES/806.bugfix b/CHANGES/806.bugfix
@@ -0,0 +1,2 @@
+Adds workaround to handle collections that do not have a ``requires_ansible`` in the
+``meta/runtime.yml`` data. This can happen in collections from ``galaxy.ansible.com``.
diff --git a/pulp_ansible/app/tasks/collections.py b/pulp_ansible/app/tasks/collections.py
@@ -1046,7 +1046,9 @@ def _post_save(self, batch):
                     )
                     if runtime_metadata:
                         runtime_yaml = yaml.safe_load(runtime_metadata)
-                        collection_version.requires_ansible = runtime_yaml.get("requires_ansible")
+                        if runtime_yaml:
+                            collection_version.requires_ansible = runtime_yaml.get(
+                                "requires_ansible")
                     manifest_data = json.load(
                         get_file_obj_from_tarball(tar, "MANIFEST.json", artifact.file.name)
                     )
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		Adds workaround to handle collections that do not have a ``requires_ansible`` in the
		``meta/runtime.yml`` data. This can happen in collections from ``galaxy.ansible.com``.