lp:~smoser/curtin/trunk.lp1520400
- Get this branch:
- bzr branch lp:~smoser/curtin/trunk.lp1520400
Branch merges
- Ryan Harper (community): Approve
- Server Team CI bot: Approve (continuous-integration)
-
Diff: 21 lines (+3/-1)1 file modifiedcurtin/url_helper.py (+3/-1)
Related bugs
Bug #1520400: curtin error on arm64 wily deployment for xgene-2 Soc | Undecided | Fix Released |
|
Related blueprints
Branch information
Recent revisions
- 315. By Scott Moser
-
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
-
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. - 312. By Scott Moser
-
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 expression2. Add in additional error messages we've seen during debugging of test
cases3. 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
-
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
-
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
-
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
-
fix errors reported by tox, rename a test
white space issues and a test named
TrustTestRaid5Bcache
rather than
TrustyTestRaid5Bcache - 307. By Scott Moser
-
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
-
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.
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