Merge lp:~matiasb/launchpad-buildd/set-build-request-id-prefix into lp:launchpad-buildd

Proposed by Matias Bordese
Status: Merged
Merged at revision: 378
Proposed branch: lp:~matiasb/launchpad-buildd/set-build-request-id-prefix
Merge into: lp:launchpad-buildd
Diff against target: 43 lines (+5/-4)
2 files modified
lpbuildd/target/build_snap.py (+2/-1)
lpbuildd/target/tests/test_build_snap.py (+3/-3)
To merge this branch: bzr merge lp:~matiasb/launchpad-buildd/set-build-request-id-prefix
Reviewer Review Type Date Requested Status
Colin Watson (community) Approve
Review via email: mp+367119@code.launchpad.net

Commit message

Updated build-request-id tag to set lp prefix.

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lpbuildd/target/build_snap.py'
--- lpbuildd/target/build_snap.py 2019-04-26 13:14:13 +0000
+++ lpbuildd/target/build_snap.py 2019-05-08 14:54:02 +0000
@@ -202,7 +202,8 @@
202 def image_info(self):202 def image_info(self):
203 data = {}203 data = {}
204 if self.args.build_request_id is not None:204 if self.args.build_request_id is not None:
205 data["build-request-id"] = self.args.build_request_id205 data["build-request-id"] = 'lp-{}'.format(
206 self.args.build_request_id)
206 if self.args.build_request_timestamp is not None:207 if self.args.build_request_timestamp is not None:
207 data["build-request-timestamp"] = self.args.build_request_timestamp208 data["build-request-timestamp"] = self.args.build_request_timestamp
208 if self.args.build_url is not None:209 if self.args.build_url is not None:
209210
=== modified file 'lpbuildd/target/tests/test_build_snap.py'
--- lpbuildd/target/tests/test_build_snap.py 2019-04-26 13:14:13 +0000
+++ lpbuildd/target/tests/test_build_snap.py 2019-05-08 14:54:02 +0000
@@ -460,7 +460,7 @@
460 RanBuildCommand(460 RanBuildCommand(
461 ["snapcraft"], cwd="/build/test-snap",461 ["snapcraft"], cwd="/build/test-snap",
462 SNAPCRAFT_BUILD_INFO="1",462 SNAPCRAFT_BUILD_INFO="1",
463 SNAPCRAFT_IMAGE_INFO='{"build-request-id": "13"}',463 SNAPCRAFT_IMAGE_INFO='{"build-request-id": "lp-13"}',
464 SNAPCRAFT_BUILD_ENVIRONMENT="host"),464 SNAPCRAFT_BUILD_ENVIRONMENT="host"),
465 ]))465 ]))
466466
@@ -506,14 +506,14 @@
506 SNAPCRAFT_LOCAL_SOURCES="1", SNAPCRAFT_SETUP_CORE="1",506 SNAPCRAFT_LOCAL_SOURCES="1", SNAPCRAFT_SETUP_CORE="1",
507 SNAPCRAFT_BUILD_INFO="1",507 SNAPCRAFT_BUILD_INFO="1",
508 SNAPCRAFT_IMAGE_INFO=(508 SNAPCRAFT_IMAGE_INFO=(
509 '{"build-request-id": "13",'509 '{"build-request-id": "lp-13",'
510 ' "build_url": "https://launchpad.example/build"}'),510 ' "build_url": "https://launchpad.example/build"}'),
511 SNAPCRAFT_BUILD_ENVIRONMENT="host")),511 SNAPCRAFT_BUILD_ENVIRONMENT="host")),
512 AnyMatch(RanBuildCommand(512 AnyMatch(RanBuildCommand(
513 ["snapcraft"], cwd="/build/test-snap",513 ["snapcraft"], cwd="/build/test-snap",
514 SNAPCRAFT_BUILD_INFO="1",514 SNAPCRAFT_BUILD_INFO="1",
515 SNAPCRAFT_IMAGE_INFO=(515 SNAPCRAFT_IMAGE_INFO=(
516 '{"build-request-id": "13",'516 '{"build-request-id": "lp-13",'
517 ' "build_url": "https://launchpad.example/build"}'),517 ' "build_url": "https://launchpad.example/build"}'),
518 SNAPCRAFT_BUILD_ENVIRONMENT="host")),518 SNAPCRAFT_BUILD_ENVIRONMENT="host")),
519 ))519 ))

Subscribers

People subscribed via source and target branches