~raharper/curtin:fix/vmtest-image-sync-after-maas-url-rename

Last commit made on 2021-01-14
Get this branch:
git clone -b fix/vmtest-image-sync-after-maas-url-rename https://git.launchpad.net/~raharper/curtin
Only Ryan Harper can upload to this branch. If you are Ryan Harper please log in for upload directions.

Branch merges

Branch information

Name:
fix/vmtest-image-sync-after-maas-url-rename
Repository:
lp:~raharper/curtin

Recent commits

48e8c3c... by Ryan Harper

vmtest: fix image-sync after maas URL stream rename

Image syncing for vmtest has been failing since MAAS renamed their
daily image stream to stable. In an effort to keep the 'daily' URL
valid the website redirects this to 'stable' path. However the content
was also modified and includes references to files and URLs which have
the 'stable' name included. This broke vmtest image sync which expected
the 'daily' string to be present in filenames and content.

The observable result was that curtin would attempt to find content it
had just *synced* to /srv/images but when searching, it would load up
the cached content files which only referenced the 'daily' stream data
and would indicate that the target file was missing.

This branch moves vmtest to using the 'stable' maas image stream
directly. This value is controllable via an environment variable,
MAAS_IMAGE_STREAM, if set will be used to compose the IMAGE_SRC_URL
and the STREAM_BASE variable which is used to find cached content in
/srv/images/.vmtest_data/ directory.

LP: #1908543

3bb8cc0... by Ryan Harper

install_grub: Fix bootloader-id for RHEL systems, must be redhat

In 7310b4fe61465, the port from shell dropped a change where the boot-id
value for Redhat family OSes is the value 'redhat' not the os-release ID
value (which is rhel, on RHEL systems).

Add this change back to install_grub and fix unittests that fail with
'rhel' instead of 'redhat' in os-release values. Hardcode 'redhat'
paths in expected values.

LP: #1906543

a259100... by Dan Watkins

vmtests: remove LP: #1888726 skip_by_date decorators

The bug has been fixed, these tests are now passing.

6b34b6a... by Michael Hudson-Doyle

storage_config: only produce type: dasd actions for ECKD dasds

this also attempts to improve the dasd documentation a bit

3999004... by Michael Hudson-Doyle

storage_config: handle some FBA dasd oddities

The storage_config handling of dasds only accounted for ECKD
dasds. FBA dasds need different handling -- in many ways they are
more like a regular disk (that can only be formatted with a DOS
partition table, but this code doesn't need to handle that), but
the kernel creates a magic fake partition for a FBA dasd with no
DOS partition table and we want to pretend that doesn't exist.

e099e32... by Nishanth Aravamudan <email address hidden>

apt_config: stop using the deprecated apt-key command

This applies the same changes as apt-setup has for
dumping the keys specified in the configuration to
the target filesystem.

LP: #1892494

698c5cd... by Michael Hudson-Doyle

allow adding a vtoc partition without a device id

this is needed for the "dasd passed to kvm via virtio" use case

4f2bdc5... by Michael Hudson-Doyle

simplify dasdview parsing code

Now that the only thing we need dasdview output for is to determine
the format type of the dasd (ldl vs cdl vs unformatted) we can make
a few things simpler.

f811df4... by Michael Hudson-Doyle

fix construction of DasdPartitionTable from fdasd output

DasdPartition had an extra argument in its constructor that should not
have been there. Add some tests.

d41a567... by Lee Trager

Don't install grub if it is already found on CentOS/RHEL

LP: #1895067