MAAS

Merge lp:~allenap/maas/compress-leases--bug-1387515--1.7 into lp:maas/1.7

  1. compress-leases--bug-1387515--1.7
  2. Merge into 1.7
Proposed by Gavin Panella
Status: Merged
Approved by: Gavin Panella
Approved revision: no longer in the source branch.
Merged at revision: 3291
Proposed branch: lp:~allenap/maas/compress-leases--bug-1387515--1.7
Merge into: lp:maas/1.7
Prerequisite: lp:~allenap/maas/compressed-amp-list--bug-1387515--1.7
Diff against target: 20 lines (+2/-1)
1 file modified
src/provisioningserver/rpc/region.py (+2/-1)
To merge this branch: bzr merge lp:~allenap/maas/compress-leases--bug-1387515--1.7
Reviewer Review Type Date Requested Status
Christian Reis (community) Approve
Review via email: mp+240113@code.launchpad.net

Commit message

Backport of r3318 from lp:maas: Use CompressedAmpList in UpdateLeases.

Previously this call would fail with more than ~1450 leases. It will now work with up to ~3500 leases.

To post a comment you must log in.
Revision history for this message
Christian Reis (kiko) :
review: Approve
Revision history for this message
Christian Reis (kiko) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/provisioningserver/rpc/region.py'
2--- src/provisioningserver/rpc/region.py 2014-10-09 21:43:07 +0000
3+++ src/provisioningserver/rpc/region.py 2014-10-30 12:51:51 +0000
4@@ -43,6 +43,7 @@
5 from provisioningserver.rpc.arguments import (
6 Bytes,
7 Choice,
8+ CompressedAmpList,
9 ParsedURL,
10 StructureAsJSON,
11 )
12@@ -163,7 +164,7 @@
13 arguments = [
14 # The cluster UUID.
15 (b"uuid", amp.Unicode()),
16- (b"mappings", amp.AmpList(
17+ (b"mappings", CompressedAmpList(
18 [(b"ip", amp.Unicode()),
19 (b"mac", amp.Unicode())]))
20 ]

Subscribers

People subscribed via source and target branches

to all changes: