Merge ~adam-collard/maas:architecture-wildcard-constraint into maas:master

[Adam Collard]

wildcard architectures returns a list, use .extend() not .append()

[Alexsander de Souza (alexsander-souza)]

+1

review: Approve

[Anton Troyanov (troyanov)]

review: Approve

diff --git a/src/maasserver/node_constraint_filter_forms.py b/src/maasserver/node_constraint_filter_forms.py
index 53e5ccd..12dce7f 100644
--- a/src/maasserver/node_constraint_filter_forms.py
+++ b/src/maasserver/node_constraint_filter_forms.py
@@ -1025,7 +1025,7 @@ class FilterNodeForm(forms.Form):
             elif value in architecture_wildcards:
                 # Try to expand 'arch' to all available 'arch/subarch'
                 # matches.
-                arches.append(*architecture_wildcards[value])
+                arches.extend(architecture_wildcards[value])
             else:
                 set_form_error(
                     self,
diff --git a/src/maasserver/tests/test_node_constraint_filter_forms.py b/src/maasserver/tests/test_node_constraint_filter_forms.py
index 0f5ac2c..1222323 100644
--- a/src/maasserver/tests/test_node_constraint_filter_forms.py
+++ b/src/maasserver/tests/test_node_constraint_filter_forms.py
@@ -1444,6 +1444,11 @@ class TestFilterNodeForm(MAASServerTestCase, FilterConstraintsMixin):
             [node_big], {"arch": arch, "tags": ["big"]}
         )
 
+    def test_wildcard_arch_constraint(self):
+        patch_usable_architectures(self, ["foo/bar", "foo/baz"])
+        form = FilterNodeForm(data={"arch": "foo"})
+        self.assertTrue(form.is_valid())
+
     def test_invalid_combined_constraints(self):
         form = FilterNodeForm(data={"tags": ["unknown"], "arch": "invalid"})
         self.assertEqual(