Ubuntu
curtin package

package does not include dpkg version

Bug #1666986 reported by Scott Moser on 2017-02-22

This bug affects 1 person

	Status	Importance	Assigned to
curtin (Ubuntu)	Fix Released	Medium	Unassigned
Xenial	Fix Released	Medium	Unassigned
Yakkety	Fix Released	Medium	Unassigned
Zesty	Fix Released	Medium	Unassigned

Bug Description

==== Begin SRU Template ====
[Impact]
A feature was added to curtin upstream to allow the package build
process to add its version to the curtin version reported.

That version is exposed via:
$ curtin version
0.1.0~bzr460-0ubuntu1

And then also
$ python3 -c 'from curtin import version; print(version.version_string())'
0.1.0~bzr460-0ubuntu1

The above are shown in zesty. However, the SRU'd versions did not
get the packaging change to write that information, so they show only

0.1.0

The fix is to make the change made in zesty's debian/rules.

[Test Case]

$ apt-get install curtin
$ curtin version

Expect to see the packaged version reported, with bzr460 in it.

[Regression Potential]
Low chance of regression, as 'curtin version' and the python library
function version_string was only recently added for this feature, so
nothing would depend on the broken behavior.

==== End SRU Template ====

in trunk revno 448, we added patching of a file in curtin via debian/rules so that the packaged version would report its version correctly.

ProblemType: Bug
DistroRelease: Ubuntu 16.10
Package: curtin 0.1.0~bzr460-0ubuntu1~16.10.1
ProcVersionSignature: Ubuntu 4.9.0-15.16-generic 4.9.5
Uname: Linux 4.9.0-15-generic x86_64
ApportVersion: 2.20.3-0ubuntu8.2
Architecture: amd64
Date: Wed Feb 22 16:20:42 2017
PackageArchitecture: all
ProcEnviron:
TERM=xterm-256color
PATH=(custom, no user)
LANG=C.UTF-8
SourcePackage: curtin
UpgradeStatus: No upgrade log present (probably fresh install)

See original description

Tags:

Related branches

lp:~smoser/curtin/xenial.lp1666986

Merged into lp:~curtin-dev/curtin/xenial at revision 62

Ryan Harper (community): Approve on 2017-02-22

lp:~smoser/curtin/yakkety.lp1666986

Merged into lp:~curtin-dev/curtin/yakkety at revision 66

Ryan Harper (community): Approve on 2017-02-22

Revision history for this message

Scott Moser (smoser) wrote on 2017-02-22:

NonfreeKernelModules.txt Edit (1.8 KiB, text/plain; charset="utf-8")
Dependencies.txt Edit (4.2 KiB, text/plain; charset="utf-8")
JournalErrors.txt Edit (12.7 KiB, text/plain; charset="utf-8")

Scott Moser (smoser) on 2017-02-22

Changed in curtin (Ubuntu):
importance:	Undecided → Medium
status:	New → Confirmed
Changed in curtin (Ubuntu Xenial):
status:	New → Confirmed
Changed in curtin (Ubuntu Yakkety):
status:	New → Confirmed
Changed in curtin (Ubuntu Zesty):
status:	Confirmed → Fix Released
Changed in curtin (Ubuntu Yakkety):
importance:	Undecided → Medium
Changed in curtin (Ubuntu Xenial):
importance:	Undecided → Medium

Scott Moser (smoser) on 2017-02-22

description:

updated

Revision history for this message

Brian Murray (brian-murray) wrote on 2017-03-02: Please test proposed package

Hello Scott, or anyone else affected,

Accepted curtin into yakkety-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/curtin/0.1.0~bzr470-0ubuntu1~16.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 to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

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

Changed in curtin (Ubuntu Yakkety):
status:	Confirmed → Fix Committed
tags:	added: verification-needed

Revision history for this message

Brian Murray (brian-murray) wrote on 2017-03-02:

Hello Scott, or anyone else affected,

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

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

Changed in curtin (Ubuntu Xenial):
status:	Confirmed → Fix Committed

Revision history for this message

Scott Moser (smoser) wrote on 2017-03-09:

Verification done on xenial with:
$ rel=xenial
$ name=xenial-1
$ lxc launch ubuntu-daily:$rel $name
$ lxc exec $name -- sh -c 'while ! [ -e /run/cloud-init/result.json ]; do echo .; sleep 1; done' </dev/null
$ lxc exec $name -- sh -e <<"EOF"
rel=$(lsb_release -sc)
line=$(awk '$1 == "deb" && $2 ~ /ubuntu.com/ {
printf("%s %s %s-proposed main universe\n", $1, $2, rel); exit(0) };
' "rel=$rel" /etc/apt/sources.list)
echo "$line" | sudo tee /etc/apt/sources.list.d/proposed.list
apt-get update -qq
apt-get install -qy curtin </dev/null
EOF
...
$ lxc exec $name -- dpkg-query --show curtin </dev/null
curtin 0.1.0~bzr470-0ubuntu1~16.04.1

Revision history for this message

Scott Moser (smoser) wrote on 2017-03-09:

$ lxc exec $name -- curtin version </dev/null
0.1.0~bzr470-0ubuntu1~16.04.1

Revision history for this message

Scott Moser (smoser) wrote on 2017-03-09:

$ rel=yakkety
$ name=yakkety-1
$ lxc launch ubuntu-daily:$rel $name
$ lxc exec $name -- sh -c 'while ! [ -e /run/cloud-init/result.json ]; do echo .; sleep 1; done' </dev/null
$ lxc exec $name -- sh -e <<"EOF"
rel=$(lsb_release -sc)
line=$(awk '$1 == "deb" && $2 ~ /ubuntu.com/ {
printf("%s %s %s-proposed main universe\n", $1, $2, rel); exit(0) };
' "rel=$rel" /etc/apt/sources.list)
echo "$line" | sudo tee /etc/apt/sources.list.d/proposed.list
apt-get update -qq
apt-get install -qy curtin </dev/null
EOF
...
$ lxc exec $name -- dpkg-query --show curtin </dev/null
curtin 0.1.0~bzr470-0ubuntu1~16.10.1
$ lxc exec $name -- curtin version </dev/null
0.1.0~bzr470-0ubuntu1~16.10.1

tags:

added: verification-done-xenial verification-done-yakkety
removed: verification-needed

Revision history for this message

Scott Moser (smoser) wrote on 2017-03-09:

Additional verification can be seen in curtin vmtest output,
$ grep -r "Installation started" archive/ | head -n 1
archive/output/ZestyTestNetworkIPV6Vlan/logs/install-serial.log:[ 10.529395] cloud-init[1503]: curtin: Installation started. (0.1.0~bzr470-0ubuntu1~16.10.1)

Revision history for this message

Launchpad Janitor (janitor) wrote on 2017-03-09:

This bug was fixed in the package curtin - 0.1.0~bzr470-0ubuntu1~16.04.1

---------------
curtin (0.1.0~bzr470-0ubuntu1~16.04.1) xenial-proposed; urgency=medium

  * debian/rules: record debian package version in installed package.
    (LP: #1666986)
  * New upstream snapshot.
    - vmtest: allow time-based skipping of tests, apply to ZestyMdadmBcache
      to temporarily avoid bug 1667078.
    - Add iSCSI disk support. (LP: #1645515)
    - vmtest: handle change in btrfs-progs when finding fs UUID. (LP: #1665701)
    - pycodestyle: fix bare except in vmtests
    - unittests: add subp coverage for retries, and rcs
    - reporting: Add better descriptions and granularity of events
    - tox: do not bother with pep8 from trusty in trusty-check
    - tests: add two scripts used for running vmtest with installed package.
    - Make add-apt-repository command retry
    - Add tests for network bridging configuration

-- Scott Moser <email address hidden> Wed, 01 Mar 2017 11:13:58 -0500

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

Revision history for this message

Brian Murray (brian-murray) wrote on 2017-03-09: Update 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.

Revision history for this message

Launchpad Janitor (janitor) wrote on 2017-03-09:

#10

This bug was fixed in the package curtin - 0.1.0~bzr470-0ubuntu1~16.10.1

---------------
curtin (0.1.0~bzr470-0ubuntu1~16.10.1) yakkety-proposed; urgency=medium

-- Scott Moser <email address hidden> Wed, 01 Mar 2017 11:17:12 -0500

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

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Bug attachments

Add attachment

Remote bug watches

Bug watches keep track of this bug in other bug trackers.

Ubuntucurtin package

package does not include dpkg version

Bug Description

Related branches

Other bug subscribers

Bug attachments

Remote bug watches

Ubuntu
curtin package