MAAS

Merge ~adam-collard/maas:release-status-with-snap-revisions into maas:master

  1. Git
  2. lp:~adam-collard/maas
  3. release-status-with-snap-revisions
  4. Merge into master
Proposed by Adam Collard
Status: Merged
Approved by: Adam Collard
Approved revision: 188061022c92b8d3e8a30fd2a71800d41306a7d0
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~adam-collard/maas:release-status-with-snap-revisions
Merge into: maas:master
Diff against target: 23 lines (+10/-2)
1 file modified
utilities/release-status (+10/-2)
Reviewer Review Type Date Requested Status
Björn Tillenius Approve
MAAS Lander unittests Pending
Review via email: mp+409009@code.launchpad.net

Commit message

[release-status] Add revision information to SnapsUploaded

Description of the change

sample output for 3.0.0

Snaps have been built and uploaded to the store.: ✓
  Latest revision for amd64: 16464
  Latest revision for arm64: 16463
  Latest revision for ppc64el: 16462
  Latest revision for s390x: 16461

To post a comment you must log in.
Revision history for this message
Björn Tillenius (bjornt) wrote :

Looks good, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/utilities/release-status b/utilities/release-status
2index a8c2e77..f93f9d3 100755
3--- a/utilities/release-status
4+++ b/utilities/release-status
5@@ -525,8 +525,16 @@ class SnapsUploaded(ReleaseStep):
6 if missing_archs:
7 return False, f"Missing builds for: {', '.join(missing_archs)}"
8
9- # XXX: print out which revisions were found
10- return True, None
11+ revision_info = []
12+ for arch, revisions in revision_map.items():
13+ latest_revision = max(
14+ revision["revision"] for revision in revisions
15+ )
16+ revision_info.append(
17+ f"Latest revision for {arch}: {latest_revision}"
18+ )
19+
20+ return True, "\n".join(revision_info)
21
22
23 class SnapsInChannel(SnapsUploaded):

Subscribers

People subscribed via source and target branches