openstack/ironic-python-agent
Code Issues Proposed changes

Clear GPT and MBR headers with dd to avoid sgdisk CRC errors

Browse Source 
This change adds a dd before the existing sgdisk -Z command to
workaround CRC verification errors.

Change-Id: Ia1ac4e1c0faf14ad4bb11c2a1c796c93ca8cb5e3
Closes-Bug: #1737556
Story: 1737556
Task: 11496
This commit is contained in:
Giulio Fidente 2018-08-08 12:16:19 +02:00
parent 667589bb00
commit bc21b5b140
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
ironic_python_agent/shell/write_image.sh
View File

@ -36,7 +36,13 @@ DEVICE="$2"
# In production this will be replaced with secure erasing the drives # In production this will be replaced with secure erasing the drives
# For now we need to ensure there aren't any old (GPT) partitions on the drive # For now we need to ensure there aren't any old (GPT) partitions on the drive
log "Erasing existing GPT and MBR data structures from ${DEVICE}" log "Erasing existing GPT and MBR data structures from ${DEVICE}"
sgdisk -Z $DEVICE || sgdisk -o $DEVICE
# NOTE(gfidente): GPT uses 33*512 sectors, this is an attempt to avoid bug:
# https://bugs.launchpad.net/ironic-python-agent/+bug/1737556
DEVICE_SECTORS_COUNT=`blockdev --getsz $DEVICE`
dd bs=512 if=/dev/zero of=$DEVICE count=33
dd bs=512 if=/dev/zero of=$DEVICE count=33 seek=$((${DEVICE_SECTORS_COUNT} - 33))
sgdisk -Z $DEVICE
log "Imaging $IMAGEFILE to $DEVICE" log "Imaging $IMAGEFILE to $DEVICE"