lp:~smoser/curtin/trunk.lp1520400

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:~curtin-dev/curtin/trunk at revision 334

Ryan Harper (community): Approve on 2016-01-06

Server Team CI bot: Approve (continuous-integration) on 2016-01-05

Diff: 21 lines (+3/-1)

1 file modified

curtin/url_helper.py (+3/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1520400: curtin error on arm64 wily deployment for xgene-2 Soc

Undecided

Fix Released

Link a bug report

Related blueprints

315. By Scott Moser on 2015-12-01

initial fix for reading skew_data

I picked these from cloud-init revno 1147 that made similar changes.
One thing I don't yet have here is the retries that went in there.
Essentially you have to retry to put the skew in place.

We probably should get that here too.

There is actually one other set of changes that i'd like to get.
Currently we set the oauth skew, and then use it, but if something
(systemd-timed) updated the clock during these things happening
then our skew could be come wrong.

The fix I want to put in place in both cloud-init and curtin
for that is to store the offset between the uptime() and the server
reported time in seconds since epoch. Then we can just read the current
uptime and add the stored delta, then convert that to a date and use it.
It wont matter then if something changed the system clock in between.

314. By Scott Moser on 2015-11-20

debian/control: do not list e2fsprogs

e2fsprogs is essential and thus we do not need an explicit
depends per lintian-info depends-on-essential-package-without-using-version.

313. By Scott Moser on 2015-11-19

Rename vmtest test class names to match module.

312. By Scott Moser on 2015-11-19

Modify how vmtests checks if a curtin install was successful.

1. we missed finding failed installs on precise/trusty due to non-systemd
   output. systemd prepends non-pid-1 process output to console with the
   calling process, so we had 'cloud-init[.*]', however on pre-systemd
   systems, that's not present. Fix this by dropping the cloud-init
   portion of the regular expression

2. Add in additional error messages we've seen during debugging of test
   cases

3. Modify the check to first see if we can find curtin's successful
   install message; if that's not found, then look for common errors.

311. By Scott Moser on 2015-11-19

use 'smtar' for extracting tarballs.

smtar allows us to not specify a compression type, but rather
have it be magically determined. This means that in the future
an .tar.xz or .tar.bz2 or .tar can be used as input.

310. By Scott Moser on 2015-11-19

fix curtin.deps.install main

running 'python -m curtin.deps.install'
shows the mismatch between what was in main in curtin/deps/install.py
and in curtin.deps.install_deps.

There isnt any code that currently uses this function, but its likely
we will use in the future, and is a nice way to get dependencies installed.

309. By Scott Moser on 2015-11-18

fix pack_install

pack_install's command was ending up with a list in it.
That would get passed through to subp and fail.

308. By Scott Moser on 2015-11-18

fix errors reported by tox, rename a test

white space issues and a test named
  TrustTestRaid5Bcache
rather than
  TrustyTestRaid5Bcache

307. By Scott Moser on 2015-11-18

tools/launch, tools/xkvm: specify format on disk images

newer versions of qemu complain about disks that do not have
a format= parameter on them. Worse, that can cause serious
issues for an installer as qemu prints messages like:

WARNING: Image format was not specified for <disk> and probing guessed raw.
    Automatically detecting the format is dangerous for raw images, write
    operations on block 0 will be restricted.
    Specify the 'raw' format explicitly to remove the restrictions.

clearly if we can't write to block 0, then we may have issues formatting
disks and such. Wily and later give these errors.

306. By Scott Moser on 2015-11-18

Improve dependency checking and installation

This makes several changes all toward the goal of fixing
curtin running on precise and better handing dependency installation.

 * change launcher to stop calling 'install-deps' itself, but rather just
   invoke the command it needs with the best python available
   the curtin main now handles installing its own dependencies if
   the user provides --install-deps.

 * pack_install now calls 'curtin pack-install --install-deps'

 * curtin/util and portions of curtin/commands/main.py are expected
   to be runnable with any python with only standard library modules.

Lastly, in order to verify all this is functional we add a Precise test.
That required some work to the test suite to handle powering the
system off because cloud-init in 12.04 does not have power_state module.