lp:~raharper/curtin/trunk.fix-vmtest-install-ok-check

Created by Ryan Harper and last modified
Get this branch:
bzr branch lp:~raharper/curtin/trunk.fix-vmtest-install-ok-check
Only Ryan Harper can upload to this branch. If you are Ryan Harper please log in for upload directions.

Branch merges

Related bugs

Related blueprints

Branch information

Owner:
Ryan Harper
Project:
curtin
Status:
Merged

Recent revisions

307. By Ryan Harper

from trunk

306. By Ryan Harper

vmtest: move install log checking to a function

Move install log checking logic to separate function.

305. By Ryan Harper

vmtest: check for curtin install success else search for errors

First look for curtin successful install message.
If we don't find one, search log for currently known errors
Lastly, report general install failure if we don't find success.

304. By Ryan Harper

from trunk

303. By Ryan Harper

vmtests: Add additional error checks on initial install

- Check for Unexpected error
- Check for missing package request

302. By Ryan Harper

Update regex to detect curtin install error

The install output of curtin is no longer prefixed with [cloud-init]
so we're missing some cases where install has failed and then continue
to wait in xkvm. Update the regexp to match output of failure.

301. By Scott Moser

change arguments to a class method to be 'cls' rather than 'self'.

passing 'self' to a class method is just confusing.
If you're looking at code out of context, or not paying attention
to the decorator, you very well expect for 'self.method()' to work.

This just changes all that from 'self' to 'cls' so its more obvious.

300. By Scott Moser

fix stacktrace and output on bad argument to mkfs

Running 'mkfs asdf' would stack trace.
Then, after fixing that fix the error message which was poorly quoted.

299. By Ryan Harper

silence logger error on __del__ call

PYTHON3 unsets globals on __del__ calls which breaks are call
to logger. Call logger prior to entering __del__ on TempDir class.

298. By Scott Moser

vmtest: install from url rather than 'cp:///'

Installing 'cp:///' as we were booting was always bad.
the / included logs or other information from the booted system.
some paths we could take:
 a.) install from a url
 b.) attach another disk, mount it, and cp:///path/to/clean/mount
 c.) boot with overlayroot and install cp:///media/mount-ro/

This patch implements 'a'. It has a nice side affect
that logs get progress from wget printed to them.

Branch metadata

Branch format:
Branch format 7
Repository format:
Bazaar repository format 2a (needs bzr 1.16 or later)
Stacked on:
lp:~curtin-dev/curtin/trunk
This branch contains Public information 
Everyone can see this information.

Subscribers