/etc/default/qemu-kvm inadvertently dropped from packaging in qemu 2.8

Bug #1692530 reported by Christian Ehrhardt 
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
qemu (Ubuntu)
Fix Released
High
Christian Ehrhardt 
Zesty
Fix Released
High
Christian Ehrhardt 

Bug Description

[Impact]

 * Users installing fresh on Zesty or later will lack the
   /etc/default/qemu-kvm file to control the convenience features of the
   qemu-kvm init script. They have to "guess" what to set in there if they
   want to use it.

[Test Case]

 * installing qemu-system-common should place the default config file
   /etc/default/qemu-kvm

[Regression Potential]

 * Overall I'd consider the regression potential of the fix low because of
   the very narrow window where it has a "direct" effect other than
   providing the base file to be modified by the user if needed (which is
   what it should do). The reasons that this window is so narrow is as
   such:
   - Users who upgraded from anything before qemu 2.8 as in zesty still
     have the old (equal content) conffile around - they are not affected
     by the issue nor by the fix.
   - Users who installed fresh on Zesty or later miss the conffile, but
     these days the qemu-kvm script is opt-in as the defaults are good. So
     adding the missing conffile for those won't change it either
   - Those who really needed the features of the qemu-kvm init script
     these days enabled the service AND wrote some sort of config. Those
     will get a normal conffile prompt on upgrade and be fine.
   - So only those that enabled the service, but did not place a config
     file will see a change at all. And then since most in that config
     file is defaulting to do nothing the only changing thing even for
     them is that KSM is enabled if you are on Bare Metal.

 * While the change seems rather safe I still pushed it through migration
   and upgrade tests and it didn't raise a flag there.

[Other Info]

 * n/a

---

While clearing out the sysv and upstart bits this got inadvertently dropped as well.

The reason it wasn't realized for so long is that:
a) it is a conffile which was not autoremoved on upgrade, so any upgrade continued to work correctly with the old file
b) it is an optional file not needed to be modified for most common use cases, the scripts are hardened against the file being missing so they work

Yet if someone "wants" to use one of the features in those files AND is on a new zesty/artful install the file will be missing.

Fortunately we can add that without too much extra conffile care.
If on a system the file is around already it will be considered as a usual conffile on upgrade (and content didn't change) and if a system never had the file it will deploy its default content.

Changed in qemu (Ubuntu):
status: New → In Progress
assignee: nobody → ChristianEhrhardt (paelzer)
importance: Undecided → High
Changed in qemu (Ubuntu Zesty):
status: New → Triaged
importance: Undecided → High
assignee: nobody → ChristianEhrhardt (paelzer)
description: updated
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Given the change that was to be assumed, but all tests look good so far.
Pushing to Artful now and prepping for Zesty then.

description: updated
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qemu - 1:2.8+dfsg-3ubuntu4

---------------
qemu (1:2.8+dfsg-3ubuntu4) artful; urgency=medium

  * debian/rules: fix installation of /etc/default/qemu-kvm (LP: #1692530)
    This was inadvertently dropped on 2.8 merge.

 -- Christian Ehrhardt <email address hidden> Mon, 22 May 2017 15:45:58 +0200

Changed in qemu (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Artful is released and working, pushed the matching fix for Zesty into the SRU unapproved queue and waiting for review now.

Changed in qemu (Ubuntu Zesty):
status: Triaged → Fix Committed
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Hmm, no update why it might be blocked on SRU in unapproved for so long ...
Asking in #ubuntu-release

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Ah this was a week were I was stupid, I had another case like this - grml my fault :-/.

I set to Fix committed as it was ready for review but according to the SRU Process [1] it should have been "In Progress" - maybe that made it being skipped - fixed the status and looking forward to a SRU check now.

[1]: https://wiki.ubuntu.com/StableReleaseUpdates#Procedure

Changed in qemu (Ubuntu Zesty):
status: Fix Committed → In Progress
Revision history for this message
Andy Whitcroft (apw) wrote : Please test proposed package

Hello ChristianEhrhardt, or anyone else affected,

Accepted qemu into zesty-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/qemu/1:2.8+dfsg-3ubuntu2.3 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 qemu (Ubuntu Zesty):
status: In Progress → Fix Committed
tags: added: verification-needed
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Verified in zesty-proposed - working fine now on upgrades and on new installs.
Saw now related conffile conflict, package ownership and all that is fine.

Setting v-d

tags: added: verification-done
removed: verification-needed
tags: added: verification-done-zesty
removed: verification-done
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Found that there are test issues - should not be releated, retriggered them now and will look deeper if they are not transient.

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

As I expected all but one test were transient and fixed on retry.
The remaining issue seems to be perma-fails in systemd dep8 tests on zesty.
Armhf [1] seems to never work. And s390x [2] doesn't seem to be much better.
I tried to get s390x good by re-triggering, but it becomes a waste of test ressources and since the armhf fail will stay it will not show up all-good in pending SRUs [3] anyway.

I'd ask you to consider migrating this SRU still as the test issues are unrelated to the change.

[1]: http://autopkgtest.ubuntu.com/packages/systemd/zesty/armhf
[2]: http://autopkgtest.ubuntu.com/packages/systemd/zesty/s390x
[3]: people.canonical.com/~ubuntu-archive/pending-sru

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qemu - 1:2.8+dfsg-3ubuntu2.3

---------------
qemu (1:2.8+dfsg-3ubuntu2.3) zesty; urgency=medium

  * debian/rules: fix installation of /etc/default/qemu-kvm (LP: #1692530)
    This was inadvertently dropped on 2.8 merge.

 -- Christian Ehrhardt <email address hidden> Mon, 22 May 2017 15:45:58 +0200

Changed in qemu (Ubuntu Zesty):
status: Fix Committed → Fix Released
Revision history for this message
Łukasz Zemczak (sil2100) wrote : Update Released

The verification of the Stable Release Update for qemu 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.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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