← Back to team overview

launchpad-reviewers team mailing list archive

[Merge] ~jugmac00/launchpad:export-requested-processors-for-rockbuildrequests into launchpad:master

 

Jürgen Gmach has proposed merging ~jugmac00/launchpad:export-requested-processors-for-rockbuildrequests into launchpad:master.

Commit message:
Expose IRockRecipeBuildRequest.architectures via API

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~jugmac00/launchpad/+git/launchpad/+merge/475097
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~jugmac00/launchpad:export-requested-processors-for-rockbuildrequests into launchpad:master.
diff --git a/lib/lp/rocks/interfaces/rockrecipe.py b/lib/lp/rocks/interfaces/rockrecipe.py
index 203146d..786b4a0 100644
--- a/lib/lp/rocks/interfaces/rockrecipe.py
+++ b/lib/lp/rocks/interfaces/rockrecipe.py
@@ -289,11 +289,15 @@ class IRockRecipeBuildRequest(Interface):
         readonly=True,
     )
 
-    architectures = Set(
-        title=_("If set, this request is limited to these architecture tags"),
-        value_type=TextLine(),
-        required=False,
-        readonly=True,
+    architectures = exported(
+        Set(
+            title=_(
+                "If set, this request is limited to these architecture tags"
+            ),
+            value_type=TextLine(),
+            required=False,
+            readonly=True,
+        )
     )
 
 
diff --git a/lib/lp/rocks/tests/test_rockrecipe.py b/lib/lp/rocks/tests/test_rockrecipe.py
index 51b88f8..fcd1426 100644
--- a/lib/lp/rocks/tests/test_rockrecipe.py
+++ b/lib/lp/rocks/tests/test_rockrecipe.py
@@ -1932,6 +1932,12 @@ class TestRockRecipeWebservice(TestCaseWithFactory):
                     "builds_collection_link": Equals(
                         build_request_url + "/builds"
                     ),
+                    "architectures": Equals(
+                        [
+                            "amd640",
+                            "risc500",
+                        ]
+                    ),
                 }
             ),
         )