airship/drydock
Code Issues Proposed changes

Merge "(fix) Register error for missing disk"

Browse Source
This commit is contained in:
Zuul 2018-10-05 21:12:39 +00:00 committed by Gerrit Code Review
commit 3b4fc16c19
1 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
python/drydock_provisioner/drivers/node/maasdriver/actions/node.py
View File

@ -1814,11 +1814,7 @@ class ApplyNodeStorage(BaseMaasAction):
# for the partition table and once the table is written, there is not
# enough space for the 'full size' partition. So reserve the below
# when calculating 'rest of device' sizing w/ the '>' operator
#
# This size is based on documentation that for backwards compatability
# the first partition should start on LBA 63 and we'll assume 4096 byte
# blocks, thus 63 (add one for safety) x 4096 = 258048
PART_TABLE_RESERVATION = 258048
PART_TABLE_RESERVATION = 1024 * 1024 * 4 # 4MB reservation for partition size
def start(self):
try:
@ -1945,9 +1941,12 @@ class ApplyNodeStorage(BaseMaasAction):
n.get_logicalname(d.name)
})
if maas_dev is None:
self.logger.warning(
"Dev %s (%s) not found on node %s" %
(d.name, n.get_logicalname(d.name), n.name))
msg = "Dev %s (%s) not found on node %s" % (
d.name, n.get_logicalname(d.name), n.name)
self.logger.warning(msg)
self.task.add_status_msg(
msg=msg, error=True, ctx=n.name, ctx_type='node')
self.task.failure(focus=n.get_id())
continue
if d.volume_group is not None: