Failed to upload leases
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
MAAS |
Fix Released
|
Critical
|
Gavin Panella | ||
1.7 |
Fix Released
|
Critical
|
Gavin Panella |
Bug Description
==> maas.log <== [45/530]
Oct 30 05:18:16 maas-trusty-
==> pserv.log <==
2014-10-30 05:18:16+0000 [-] Unhandled Error
Traceback (most recent call last):
File "/usr/lib/
File "/usr/lib/
File "/usr/lib/
File "/usr/lib/
--- <exception caught here> ---
File "/usr/lib/
result = result.
File "/usr/lib/
return g.throw(self.type, self.value, self.tb)
File "/usr/lib/
yield self._start_
File "/usr/lib/
result = g.send(result)
File "/usr/lib/
File "/usr/lib/
return func(*args, **kwargs)
File "/usr/lib/
return self._conn.
File "/usr/lib/
return co._doCommand(self)
File "/usr/lib/
File "/usr/lib/
File "/usr/lib/
File "/usr/lib/
File "/usr/lib/
raise TooLong(False, True, v, k)
==> maas.log <==
Oct 30 05:18:16 maas-trusty-
Related branches
- Graham Binns (community): Approve
-
Diff: 83 lines (+53/-0)2 files modifiedsrc/provisioningserver/rpc/arguments.py (+17/-0)
src/provisioningserver/rpc/tests/test_arguments.py (+36/-0)
- Graham Binns (community): Approve
-
Diff: 20 lines (+2/-1)1 file modifiedsrc/provisioningserver/rpc/region.py (+2/-1)
- Christian Reis (community): Approve
-
Diff: 105 lines (+57/-0)3 files modifiedsrc/maasserver/models/tests/test_node.py (+4/-0)
src/provisioningserver/rpc/arguments.py (+17/-0)
src/provisioningserver/rpc/tests/test_arguments.py (+36/-0)
- Christian Reis (community): Approve
-
Diff: 20 lines (+2/-1)1 file modifiedsrc/provisioningserver/rpc/region.py (+2/-1)
- Andres Rodriguez (community): Approve
-
Diff: 36 lines (+20/-2)1 file modifieddebian/changelog (+20/-2)
tags: | added: oil |
tags: | added: dhcp provisioning |
Changed in maas: | |
assignee: | nobody → Gavin Panella (allenap) |
status: | Triaged → In Progress |
AMP's maximum message size is now not big enough to cope with all the leases, which is slowing down the pserv and causing this stacktrace.
We have a couple of options:
1. Get rid of lease parsing. Nothing needs it.
2. Switch to the things previously discussed, namely a daemon that uses inotify to pick up changes as they happen
The problem is that the current parser tries to upload the current status every time, which is probably unnecessary, it only needs to be done once on startup.