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

[Gavin Panella]

Use CompressedAmpList in UpdateLeases.

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

[Graham Binns (gmb)]

review: Approve

[Andres Rodriguez (andreserl)]

Is it possible to backport this to 1.7 as OIL is being affected? Thanks
On Oct 30, 2014 7:34 AM, "Gavin Panella" <email address hidden>
wrote:

> Gavin Panella has proposed merging
> lp:~allenap/maas/compress-leases--bug-1387515 into lp:maas with
> lp:~allenap/maas/compressed-amp-list--bug-1387515 as a prerequisite.
>
> Commit message:
> Use CompressedAmpList in UpdateLeases.
>
> Previously this call would fail with more than ~1450 leases. It will now
> work with up to ~3500 leases.
>
> Requested reviews:
> MAAS Maintainers (maas-maintainers)
> Related bugs:
> Bug #1387515 in MAAS: "Failed to upload leases"
> https://bugs.launchpad.net/maas/+bug/1387515
>
> For more details, see:
>
> https://code.launchpad.net/~allenap/maas/compress-leases--bug-1387515/+merge/240103
> --
>
> https://code.launchpad.net/~allenap/maas/compress-leases--bug-1387515/+merge/240103
> You are subscribed to branch lp:maas.
>
> === modified file 'src/provisioningserver/rpc/region.py'
> --- src/provisioningserver/rpc/region.py 2014-10-09 21:43:07 +0000
> +++ src/provisioningserver/rpc/region.py 2014-10-30 11:32:47 +0000
> @@ -43,6 +43,7 @@
> from provisioningserver.rpc.arguments import (
> Bytes,
> Choice,
> + CompressedAmpList,
> ParsedURL,
> StructureAsJSON,
> )
> @@ -163,7 +164,7 @@
> arguments = [
> # The cluster UUID.
> (b"uuid", amp.Unicode()),
> - (b"mappings", amp.AmpList(
> + (b"mappings", CompressedAmpList(
> [(b"ip", amp.Unicode()),
> (b"mac", amp.Unicode())]))
> ]
>
>
>

=== modified file 'src/provisioningserver/rpc/region.py'
--- src/provisioningserver/rpc/region.py	2014-10-09 21:43:07 +0000
+++ src/provisioningserver/rpc/region.py	2014-10-30 11:36:22 +0000
@@ -43,6 +43,7 @@
 from provisioningserver.rpc.arguments import (
     Bytes,
     Choice,
+    CompressedAmpList,
     ParsedURL,
     StructureAsJSON,
     )
@@ -163,7 +164,7 @@
     arguments = [
         # The cluster UUID.
         (b"uuid", amp.Unicode()),
-        (b"mappings", amp.AmpList(
+        (b"mappings", CompressedAmpList(
             [(b"ip", amp.Unicode()),
              (b"mac", amp.Unicode())]))
     ]