sru curtin 2019-06-05 - 19.1-7-g37a7a0f4-0ubuntu1

Bug #1831772 reported by Ryan Harper on 2019-06-05
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
curtin
Undecided
Unassigned
curtin (Ubuntu)
Undecided
Unassigned
Xenial
Undecided
Unassigned
Bionic
Undecided
Unassigned
Cosmic
Undecided
Unassigned
Disco
Undecided
Unassigned

Bug Description

== Begin SRU Template ==
[Impact]
This release sports both bug-fixes and new features and we would like to
make sure all of our supported customers have access to these improvements.
The notable ones are:

 - block-discover: add cli/API for exporting existing storage to config
 - curthoooks: disable daemons while reconfiguring mdadm
 - mdadm: fix install to existing raid
 - block: support multipath devices in lookup and partition_kname
 - grub: add grub config to control os_prober,terminal settings in target
 - curthooks: chzdev_import must encode data if provided

See the changelog entry below for a full list of changes and bugs.

[Test Case]
The following development and SRU process was followed:
https://wiki.ubuntu.com/CurtinUpdates

Curtin now contains an extensive integration test suite that is ran using
the SRU package for each releases. These suite has documentation here:
https://curtin.readthedocs.io/en/latest/topics/integration-testing.html

In order to avoid regression to existing MAAS product, the MAAS team will
run their continuous integration test against the curtin that is in
-proposed. A successful run will be required before the proposed curtin
can be let into -updates.

The curtin team will be in charge of attaching the artifacts and console
output of the appropriate run to the bug. Curtin team members will not
mark ‘verification-done’ until this has happened.

[Regression Potential]
In order to mitigate the regression potential, the results of the
aforementioned integration tests are attached to this bug.

DONE: attach curtin-proposed test artifacts from vmtest for every SRU release
DONE: attach maas-proposed test artifacts from vmtest for every SRU release
DONE: attach curtin manual verification of install not including probert

[Discussion]

== End SRU Template ==

== changelog ==

curtin (19.1-7-g37a7a0f4-0ubuntu1~19.04.1) disco; urgency=medium

  * New upstream snapshot.
    - vmtests: determine block name via dname when verifying volume groups
    - vmtest: add Centos66/Centos70 FromBionic release and re-add tests
    - block-discover: add cli/API for exporting existing storage to config
    - vmtest: refactor test_network code for Eoan
    - curthoooks: disable daemons while reconfiguring mdadm
      [Michael Hudson-Doyle] (LP: #1829325.)
    - mdadm: fix install to existing raid [Michael Hudson-Doyle] (LP: #1830157)
    - apt_proxy env var set but empty => do not setup any proxy
      [Paride Legovini]
    - release 19.1
    - vmtest: add missing skip_by_date on Eoan test_network_mtu
    - vmtest: remove skip_by_date on Disco Allindata test
    - block: support multipath devices in lookup and partition_kname
      (LP: #1813228)
    - grub: add grub config to control os_prober,terminal settings in target
    - vmtest: add eoan tests [Paride Legovini]
    - vmtest: add dependency on python3-jsonschema [Paride Legovini]
    - Pylint compatibility with Python 3.7 [Paride Legovini] (LP: #1828229)
    - vmtest: disable disco multipath test
    - vmtests: bump network mtu tests out a bit
    - Fix up yaml.load warnings
    - vmtest: disable trusty, it's dead jim
    - vmtest: reenable UEFI 4k tests, which got dropped when fixing
      subclassing.
    - vmtests: Add test_kernel_img_conf stub for PsuedoVMBaseClass
    - curthooks: chzdev_import must encode data if provided (LP: #1825007)
    - vmtest: test for kernel-img conf on ubuntu only

 -- Ryan Harper <email address hidden> Wed, 05 Jun 2019 13:46:57 -0500

Ryan Harper (raharper) wrote :

Not an upstream bug.

Changed in curtin:
status: New → Invalid
Ryan Harper (raharper) on 2019-06-05
description: updated
Chris Halse Rogers (raof) wrote :

So, this appears to add a dependency on the "probert" package to the "curtin" binary package.

probert is in universe.

This is fine for Xenial, because while the curtin source package is in main there, the curtin *binary* package is in universe. But for Bionic and newer both curtin source and curtin binary are in main, so the probert package would need to be promoted to main.

Other than that this looks fine.

Changed in curtin (Ubuntu Bionic):
status: New → Incomplete
Changed in curtin (Ubuntu Cosmic):
status: New → Incomplete
Changed in curtin (Ubuntu Disco):
status: New → Incomplete
Ryan Harper (raharper) wrote :

Thanks, we'll be uploading a replacement source package which drops the dependency on probert in bionic, cosmic and disco. Thanks for the review

Hello Ryan, or anyone else affected,

Accepted curtin into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/curtin/19.1-7-g37a7a0f4-0ubuntu1~18.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in curtin (Ubuntu Bionic):
status: Incomplete → Fix Committed
tags: added: verification-needed verification-needed-bionic
Changed in curtin (Ubuntu Cosmic):
status: Incomplete → Fix Committed
tags: added: verification-needed-cosmic
Łukasz Zemczak (sil2100) wrote :

Hello Ryan, or anyone else affected,

Accepted curtin into cosmic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/curtin/19.1-7-g37a7a0f4-0ubuntu1~18.10.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-cosmic to verification-done-cosmic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-cosmic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in curtin (Ubuntu Xenial):
status: New → Fix Committed
tags: added: verification-needed-xenial
Łukasz Zemczak (sil2100) wrote :

Hello Ryan, or anyone else affected,

Accepted curtin into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/curtin/19.1-7-g37a7a0f4-0ubuntu1~16.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-xenial to verification-done-xenial. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-xenial. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Łukasz Zemczak (sil2100) wrote :

Hello Ryan, or anyone else affected,

Accepted curtin into disco-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/curtin/19.1-7-g37a7a0f4-0ubuntu1~19.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-disco to verification-done-disco. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-disco. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in curtin (Ubuntu Disco):
status: Incomplete → Fix Committed
tags: added: verification-needed-disco
John George (jog) wrote :

Passing run details from Solutions QA using python3-curtin_19.1-7-g37a7a0f4-0ubuntu1~18.04.1 with maas_2.5.3-7533-g65952b418-0ubuntu1~18.04.1 during a hyper-converged Openstack cloud deployment are available at:
https://solutions.qa.canonical.com/#/qa/testRun/4a79a1f5-a0d9-4d77-982d-ae1b78462f97

Ryan Harper (raharper) wrote :

Bionic curtin-proposed test artifacts.

Ryan Harper (raharper) wrote :

Bionic vmtest console log

Ryan Harper (raharper) wrote :

Bionic vmtest artifacts from repeating tests that timed out.

Ryan Harper (raharper) wrote :

Bionic vmtest artifacts from repeating tests that timed out.

Ryan Harper (raharper) wrote :

Cosmic vmtest artifacts

Ryan Harper (raharper) wrote :

Cosmic vmtest console log

Ryan Harper (raharper) wrote :

Cosmic vmtest artifacts from rerunning tests that timed out.

Ryan Harper (raharper) wrote :

Cosmic vmtest console log from rerunning tests that timed out.

Ryan Harper (raharper) wrote :

Disco vmtest artifacts

Ryan Harper (raharper) wrote :

Disco vmtest console log

Ryan Harper (raharper) wrote :

Disco vmtest artifacts from rerunning tests that timed out.

Ryan Harper (raharper) wrote :

Disco vmtest console-log from rerunning tests that timed out.

Ryan Harper (raharper) wrote :

Xenial vmtest artifacts

Ryan Harper (raharper) wrote :

Xenial vmtest console log

Ryan Harper (raharper) wrote :

Xenial vmtest artifacts from rerunning tests that timed out.

Ryan Harper (raharper) wrote :

Xenial vmtest console log from rerunning tests that timed out.

Ryan Harper (raharper) wrote :

Bionic manual verification that installing curtin from -proposed does not pull in probert package.

Ryan Harper (raharper) wrote :

Cosmic manual verification that installing curtin from -proposed does not pull in probert package.

Ryan Harper (raharper) wrote :

Disco manual verification that installing curtin from -proposed does not pull in probert package.

Ryan Harper (raharper) wrote :

Xenial manual verification that installing curtin from -proposed does not pull in probert package.

description: updated
tags: added: verification-done verification-done-bionic verification-done-cosmic verification-done-disco verification-done-xenial
removed: verification-needed verification-needed-bionic verification-needed-cosmic verification-needed-disco verification-needed-xenial
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package curtin - 19.1-7-g37a7a0f4-0ubuntu1~19.04.1

---------------
curtin (19.1-7-g37a7a0f4-0ubuntu1~19.04.1) disco; urgency=medium

  * d/control: drop probert dependency, not yet in main (LP: #1831757)
  * New upstream snapshot. (LP: #1831772)
    - vmtests: determine block name via dname when verifying volume groups
    - vmtest: add Centos66/Centos70 FromBionic release and re-add tests
    - block-discover: add cli/API for exporting existing storage to config
    - vmtest: refactor test_network code for Eoan
    - curthoooks: disable daemons while reconfiguring mdadm
      [Michael Hudson-Doyle]
    - mdadm: fix install to existing raid [Michael Hudson-Doyle]
    - apt_proxy env var set but empty => do not setup any proxy
      [Paride Legovini]
    - release 19.1
    - vmtest: add missing skip_by_date on Eoan test_network_mtu
    - vmtest: remove skip_by_date on Disco Allindata test
    - block: support multipath devices in lookup and partition_kname
    - grub: add grub config to control os_prober,terminal settings in target
    - vmtest: add eoan tests [Paride Legovini]
    - vmtest: add dependency on python3-jsonschema [Paride Legovini]
    - Pylint compatibility with Python 3.7 [Paride Legovini]
    - vmtest: disable disco multipath test
    - vmtests: bump network mtu tests out a bit
    - Fix up yaml.load warnings
    - vmtest: disable trusty, it's dead jim
    - vmtest: reenable UEFI 4k tests, which got dropped when fixing
      subclassing.
    - vmtests: Add test_kernel_img_conf stub for PsuedoVMBaseClass
    - curthooks: chzdev_import must encode data if provided
    - vmtest: test for kernel-img conf on ubuntu only

 -- Ryan Harper <email address hidden> Wed, 05 Jun 2019 13:46:57 -0500

Changed in curtin (Ubuntu Disco):
status: Fix Committed → Fix Released

The verification of the Stable Release Update for curtin has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package curtin - 19.1-7-g37a7a0f4-0ubuntu1~18.10.1

---------------
curtin (19.1-7-g37a7a0f4-0ubuntu1~18.10.1) cosmic; urgency=medium

  * d/control: drop probert dependency, not yet in main (LP: #1831757)
  * New upstream snapshot. (LP: #1831772)
    - vmtests: determine block name via dname when verifying volume groups
    - vmtest: add Centos66/Centos70 FromBionic release and re-add tests
    - block-discover: add cli/API for exporting existing storage to config
    - vmtest: refactor test_network code for Eoan
    - curthoooks: disable daemons while reconfiguring mdadm
      [Michael Hudson-Doyle]
    - mdadm: fix install to existing raid [Michael Hudson-Doyle]
    - apt_proxy env var set but empty => do not setup any proxy
      [Paride Legovini]
    - release 19.1
    - vmtest: add missing skip_by_date on Eoan test_network_mtu
    - vmtest: remove skip_by_date on Disco Allindata test
    - block: support multipath devices in lookup and partition_kname
    - grub: add grub config to control os_prober,terminal settings in target
    - vmtest: add eoan tests [Paride Legovini]
    - vmtest: add dependency on python3-jsonschema [Paride Legovini]
    - Pylint compatibility with Python 3.7 [Paride Legovini]
    - vmtest: disable disco multipath test
    - vmtests: bump network mtu tests out a bit
    - Fix up yaml.load warnings
    - vmtest: disable trusty, it's dead jim
    - vmtest: reenable UEFI 4k tests, which got dropped when fixing
      subclassing.
    - vmtests: Add test_kernel_img_conf stub for PsuedoVMBaseClass
    - curthooks: chzdev_import must encode data if provided
    - vmtest: test for kernel-img conf on ubuntu only
    - Setup kernel-img.conf like base-installer/live-installer did.
      [Dimitri John Ledkov]
    - zfs: pass pool and fs properties to zpool_create
      [Jean-Baptiste Lallement]
    - block_meta: pass lvm_partition size through human2bytes
      [Michael Hudson-Doyle]
    - clear-holders: refactor bcache shutdown and add longer timeout
    - schema-validate-storage: remove unused imports [Paride Legovini]
    - add subcommand schema for storage-config validation
    - clear-holders: wipe raid members at data offsets when not assembled
    - Add support for s390 DASD devices
    - vmtest: skip DiscoTestAllindata while initrd lacks crypto modules
    - block_meta: allow passing a dm_crypt key by keyfile
      [Michael Hudson-Doyle]
    - ChrootableTarget: add /run to mounts for redhat targets
    - vmtest: consolidate vm mem config, defaults via release, env override

 -- Ryan Harper <email address hidden> Wed, 05 Jun 2019 13:32:52 -0500

Changed in curtin (Ubuntu Cosmic):
status: Fix Committed → Fix Released
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package curtin - 19.1-7-g37a7a0f4-0ubuntu1~18.04.1

---------------
curtin (19.1-7-g37a7a0f4-0ubuntu1~18.04.1) bionic; urgency=medium

  * d/control: drop probert dependency, not yet in main (LP: #1831757)
  * New upstream snapshot. (LP: #1831772)
    - vmtests: determine block name via dname when verifying volume groups
    - vmtest: add Centos66/Centos70 FromBionic release and re-add tests
    - block-discover: add cli/API for exporting existing storage to config
    - vmtest: refactor test_network code for Eoan
    - curthoooks: disable daemons while reconfiguring mdadm
      [Michael Hudson-Doyle]
    - mdadm: fix install to existing raid [Michael Hudson-Doyle]
    - apt_proxy env var set but empty => do not setup any proxy
      [Paride Legovini]
    - release 19.1
    - vmtest: add missing skip_by_date on Eoan test_network_mtu
    - vmtest: remove skip_by_date on Disco Allindata test
    - block: support multipath devices in lookup and partition_kname
    - grub: add grub config to control os_prober,terminal settings in target
    - vmtest: add eoan tests [Paride Legovini]
    - vmtest: add dependency on python3-jsonschema [Paride Legovini]
    - Pylint compatibility with Python 3.7 [Paride Legovini]
    - vmtest: disable disco multipath test
    - vmtests: bump network mtu tests out a bit
    - Fix up yaml.load warnings
    - vmtest: disable trusty, it's dead jim
    - vmtest: reenable UEFI 4k tests, which got dropped when fixing
      subclassing.
    - vmtests: Add test_kernel_img_conf stub for PsuedoVMBaseClass
    - curthooks: chzdev_import must encode data if provided
    - vmtest: test for kernel-img conf on ubuntu only
    - Setup kernel-img.conf like base-installer/live-installer did.
      [Dimitri John Ledkov]
    - zfs: pass pool and fs properties to zpool_create
      [Jean-Baptiste Lallement]
    - block_meta: pass lvm_partition size through human2bytes
      [Michael Hudson-Doyle]
    - clear-holders: refactor bcache shutdown and add longer timeout
    - schema-validate-storage: remove unused imports [Paride Legovini]
    - add subcommand schema for storage-config validation
    - clear-holders: wipe raid members at data offsets when not assembled
    - Add support for s390 DASD devices
    - vmtest: skip DiscoTestAllindata while initrd lacks crypto modules
    - block_meta: allow passing a dm_crypt key by keyfile
      [Michael Hudson-Doyle]
    - ChrootableTarget: add /run to mounts for redhat targets
    - vmtest: consolidate vm mem config, defaults via release, env override

 -- Ryan Harper <email address hidden> Wed, 05 Jun 2019 12:19:06 -0500

Changed in curtin (Ubuntu Bionic):
status: Fix Committed → Fix Released
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package curtin - 19.1-7-g37a7a0f4-0ubuntu1~16.04.1

---------------
curtin (19.1-7-g37a7a0f4-0ubuntu1~16.04.1) xenial; urgency=medium

  * d/control: drop probert dependency for Xenial (LP: #1831757)
  * New upstream snapshot. (LP: #1831772)
    - vmtests: determine block name via dname when verifying volume groups
    - vmtest: add Centos66/Centos70 FromBionic release and re-add tests
    - block-discover: add cli/API for exporting existing storage to config
    - vmtest: refactor test_network code for Eoan
    - curthoooks: disable daemons while reconfiguring mdadm
      [Michael Hudson-Doyle]
    - mdadm: fix install to existing raid [Michael Hudson-Doyle]
    - apt_proxy env var set but empty => do not setup any proxy
      [Paride Legovini]
    - release 19.1
    - vmtest: add missing skip_by_date on Eoan test_network_mtu
    - vmtest: remove skip_by_date on Disco Allindata test
    - block: support multipath devices in lookup and partition_kname
    - grub: add grub config to control os_prober,terminal settings in target
    - vmtest: add eoan tests [Paride Legovini]
    - vmtest: add dependency on python3-jsonschema [Paride Legovini]
    - Pylint compatibility with Python 3.7 [Paride Legovini]
    - vmtest: disable disco multipath test
    - vmtests: bump network mtu tests out a bit
    - Fix up yaml.load warnings
    - vmtest: disable trusty, it's dead jim
    - vmtest: reenable UEFI 4k tests, which got dropped when fixing
      subclassing.
    - vmtests: Add test_kernel_img_conf stub for PsuedoVMBaseClass
    - curthooks: chzdev_import must encode data if provided
    - vmtest: test for kernel-img conf on ubuntu only
    - Setup kernel-img.conf like base-installer/live-installer did.
      [Dimitri John Ledkov]
    - zfs: pass pool and fs properties to zpool_create
      [Jean-Baptiste Lallement]
    - block_meta: pass lvm_partition size through human2bytes
      [Michael Hudson-Doyle]
    - clear-holders: refactor bcache shutdown and add longer timeout
    - schema-validate-storage: remove unused imports [Paride Legovini]
    - add subcommand schema for storage-config validation
    - clear-holders: wipe raid members at data offsets when not assembled
    - Add support for s390 DASD devices
    - vmtest: skip DiscoTestAllindata while initrd lacks crypto modules
    - block_meta: allow passing a dm_crypt key by keyfile
      [Michael Hudson-Doyle]
    - ChrootableTarget: add /run to mounts for redhat targets
    - vmtest: consolidate vm mem config, defaults via release, env override

 -- Ryan Harper <email address hidden> Wed, 05 Jun 2019 11:59:57 -0500

Changed in curtin (Ubuntu Xenial):
status: Fix Committed → Fix Released
Changed in curtin (Ubuntu):
status: New → Invalid
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers