Merge lp:~nick-schutt/lava-dispatcher/nicks-highbank-support into lp:lava-dispatcher

[Nicholas Schutt]

Add dd to clear the start of the hard disk before deploying an image.

dd if=/dev/zero of=%s bs=4M count=4

[Antonio Terceiro (terceiro)]

Looks good to me

 review approve

review: Approve

[Fathi Boudra (fboudra)]

What's the issue observed? Since we deploy the image later using dd.

[Nicholas Schutt (nick-schutt)]

The basic idea is to corrupt the hdd so that if the deploy fails, then the
previously-written image won't boot.

On 24 April 2013 08:07, Fathi Boudra <email address hidden> wrote:

> What's the issue observed? Since we deploy the image later using dd.
> --
>
> https://code.launchpad.net/~nick-schutt/lava-dispatcher/nicks-highbank-support/+merge/160356
> You are the owner of
> lp:~nick-schutt/lava-dispatcher/nicks-highbank-support.
>

=== modified file 'lava_dispatcher/device/highbank.py'
--- lava_dispatcher/device/highbank.py	2013-04-22 13:32:10 +0000
+++ lava_dispatcher/device/highbank.py	2013-04-23 12:49:26 +0000
@@ -91,6 +91,9 @@
     def _deploy_image(self, image_file, device):
         with self._as_master() as runner:
 
+            # erase the first part of the disk to make sure the new deploy works
+            runner.run("dd if=/dev/zero of=%s bs=4M count=4" % device, timeout=1800)
+
             # compress the image to reduce the transfer size
             if not image_file.endswith('.bz2') and not image_file.endswith('gz'):
                 os.system('bzip2 -9v ' + image_file)
@@ -157,7 +160,6 @@
             else:
                 logging.warning("unknown partition type for resize: %s" % parttype)
 
-
     @contextlib.contextmanager
     def file_system(self, partition, directory):
         logging.info('attempting to access master filesystem %r:%s' %