[Group.of.nepali.translators] [Bug 1709029] Re: Tests failing on xenial due to missing fakeroot command

2017-08-28 Thread Łukasz Zemczak 
** Changed in: ubuntu-image
   Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of नेपाली
भाषा समायोजकहरुको समूह, which is subscribed to Xenial.
Matching subscriptions: Ubuntu 16.04 Bugs
https://bugs.launchpad.net/bugs/1709029

Title:
  Tests failing on xenial due to missing fakeroot command

Status in Ubuntu Image:
  Fix Released
Status in ubuntu-image package in Ubuntu:
  Fix Released
Status in ubuntu-image source package in Xenial:
  Fix Released

Bug description:
  [Impact]

  Currently the xenial ubuntu-image package fails autopkgtests due to
  the missing fakeroot dependency. Xenial has an older mkfs.ext4 that
  does not support a parameter that all the other series are using.
  Because of that, fakeroot is not really used on this release as by
  default the first call of mkfs.ext4 which is assuming the existence of
  the '-d' option is supposed to fail and then u-i is moving on to using
  a fallback method of copying rootfs contents into the filesystem. To
  the first call of the mkfs.ext4 call we have recently added fakeroot
  (due to a security vulnerability discovered).

  The failure of calling mkfs.ext4 with an unrecognized parameter and
  calling fakeroot mkfs.ext4 when fakeroot is not installed both
  generate different errors (the first one causes run() to exit with
  error and the second one rises a FileNotFoundError exception). We can
  easily support both cases, but then we are encountering code coverage
  issues - as in the best case we want to have the same code for all
  supported series. It's not easy to mock this case in unit tests as in
  theory we will never be able to reach this FileNotFoundError branch on
  non-xenial systems. It basically requires a lot of unnecessary work
  that's not bringing any real merits.

  [Fix]

  Just dropping this part of the delta and adding the fakeroot
  dependency fixes all problems with little effort.

  [Test Case]

  Make sure that the xenial autopkgtests pass.

  [Regression Potential]

  None - really. We're only adding the fakeroot dependency that is
  installed on all the other series for the same ubuntu-image version
  already. Installing this dependency does not change anything in the
  logic of ubuntu-image - the only time it is used is for the first
  mkfs.ext4 command which always fails (due to being too old to support
  the required options).

  [Original description]

  There is a delta in xenial. We need to support missing fakeroot.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-image/+bug/1709029/+subscriptions

___
Mailing list: https://launchpad.net/~group.of.nepali.translators
Post to : group.of.nepali.translators@lists.launchpad.net
Unsubscribe : https://launchpad.net/~group.of.nepali.translators
More help   : https://help.launchpad.net/ListHelp

[Group.of.nepali.translators] [Bug 1709029] Re: Tests failing on xenial due to missing fakeroot command

2017-08-17 Thread Launchpad Bug Tracker 
This bug was fixed in the package ubuntu-image - 1.1+16.04ubuntu4

---
ubuntu-image (1.1+16.04ubuntu4) xenial; urgency=medium

  * Drop part of the xenial delta, adding the fakeroot dependency as per master.
Properly handling the lack of the dependency and still getting 100% code
coverage on all series requires too much unnecessary work.  (LP: #1709029)

ubuntu-image (1.1+16.04ubuntu3) xenial; urgency=medium

  * Only run the snap.sh test on github PRs as it doesn't make sense to run it
as part of regular deb package migration.  (LP: #1704979)

ubuntu-image (1.1+16.04ubuntu1) xenial; urgency=medium

  [ Łukasz 'sil2100' Zemczak ]
  * SRU tracking number LP: #1702933
  * Fix snapcraft.yaml for classic snappage.  (LP:1673576)
  * Add a new step in the builder that generates .manifest files listing all
snaps installed in the resulting image.  (LP:1680574)
  * Further fixes to snapcraft.yaml, making sure that we only have one
ubuntu-image version installed in the snap.  (LP:1692901)
  * Move files around during snap build to have all needed libraries and python
files in common accessible paths, install missing dependencies into the
snap.  (LP:1694982, LP:1694993)

  [ Michael Vogt ]
  * Use fakeroot when running mkfs.ext4.  (LP:1702628)

  [ Steve Langasek ]
  * Add fakeroot to the snapcraft.yaml and as a test dep.  (LP:1702628)

 -- Łukasz 'sil2100' Zemczak   Tue, 08 Aug
2017 10:44:33 +0200

** Changed in: ubuntu-image (Ubuntu Xenial)
   Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of नेपाली
भाषा समायोजकहरुको समूह, which is subscribed to Xenial.
Matching subscriptions: Ubuntu 16.04 Bugs
https://bugs.launchpad.net/bugs/1709029

Title:
  Tests failing on xenial due to missing fakeroot command

Status in Ubuntu Image:
  In Progress
Status in ubuntu-image package in Ubuntu:
  Fix Released
Status in ubuntu-image source package in Xenial:
  Fix Released

Bug description:
  [Impact]

  Currently the xenial ubuntu-image package fails autopkgtests due to
  the missing fakeroot dependency. Xenial has an older mkfs.ext4 that
  does not support a parameter that all the other series are using.
  Because of that, fakeroot is not really used on this release as by
  default the first call of mkfs.ext4 which is assuming the existence of
  the '-d' option is supposed to fail and then u-i is moving on to using
  a fallback method of copying rootfs contents into the filesystem. To
  the first call of the mkfs.ext4 call we have recently added fakeroot
  (due to a security vulnerability discovered).

  The failure of calling mkfs.ext4 with an unrecognized parameter and
  calling fakeroot mkfs.ext4 when fakeroot is not installed both
  generate different errors (the first one causes run() to exit with
  error and the second one rises a FileNotFoundError exception). We can
  easily support both cases, but then we are encountering code coverage
  issues - as in the best case we want to have the same code for all
  supported series. It's not easy to mock this case in unit tests as in
  theory we will never be able to reach this FileNotFoundError branch on
  non-xenial systems. It basically requires a lot of unnecessary work
  that's not bringing any real merits.

  [Fix]

  Just dropping this part of the delta and adding the fakeroot
  dependency fixes all problems with little effort.

  [Test Case]

  Make sure that the xenial autopkgtests pass.

  [Regression Potential]

  None - really. We're only adding the fakeroot dependency that is
  installed on all the other series for the same ubuntu-image version
  already. Installing this dependency does not change anything in the
  logic of ubuntu-image - the only time it is used is for the first
  mkfs.ext4 command which always fails (due to being too old to support
  the required options).

  [Original description]

  There is a delta in xenial. We need to support missing fakeroot.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-image/+bug/1709029/+subscriptions

___
Mailing list: https://launchpad.net/~group.of.nepali.translators
Post to : group.of.nepali.translators@lists.launchpad.net
Unsubscribe : https://launchpad.net/~group.of.nepali.translators
More help   : https://help.launchpad.net/ListHelp