lxd missing dependency on iproute2

Bug #1770395 reported by Julian Andres Klode
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
cloud-images
Invalid
Undecided
Unassigned
lxd (Ubuntu)
Fix Released
Undecided
Unassigned
Bionic
Fix Released
Undecided
Unassigned

Bug Description

lxd uses ip(8) to setup networking, but does not depend on it, causing errors:

Error: Failed to create the new macvlan interface: Failed to run: ip link add dev veth583d5df1 link eth0 type macvlan mode bridge:

Yes, iproute2 is part of the minimal task and should be installed, but it's not essential, so a dependency makes sense.

tags: added: id-5aea01d25f0497dbf084cd37
Revision history for this message
Robert C Jennings (rcj) wrote :

minimal cloud images should consider inclusion of iproute2 even if lxd adds iproute2 as an explicit dependency

Revision history for this message
Dan Watkins (oddbloke) wrote :

I don't think we need to make a change in the minimal cloud images; this (and similar bugs) should be fixed in the packages that have the bug.

(Leaving this open for cloud-images so we can ensure that we handle rolling out the fix once it's in the lxd packaging.)

Revision history for this message
Julian Andres Klode (juliank) wrote :

Having cloud images with only legacy network configuration tools like ifconfig, and not the current ip tool seems weird, though.

Revision history for this message
Dan Watkins (oddbloke) wrote :

If we consider iproute2 to be essential to a functioning Ubuntu then, you know... ;)

summary: - Missing dependency on iproute2
+ lxd missing dependency on iproute2
Changed in lxd (Ubuntu):
status: New → Fix Committed
Revision history for this message
Steve Langasek (vorlon) wrote :

it's an unfortunate side effect of the way the minimal images are constructed that net-tools is still present in the minimal image. Ideally we would not have net-tools in these images either; if we fix livecd-rootfs to remove it, would that be happier? ;)

Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (7.0 KiB)

This bug was fixed in the package lxd - 3.0.1-0ubuntu1

---------------
lxd (3.0.1-0ubuntu1) cosmic; urgency=medium

  * New usptream bugfix release:
    - lxc: Fix mistakenly hidden commands
    - i18n: Update translation templates
    - lxd/migration: Pre-validate profiles
    - client: Improve remote operation errors
    - Fix some typos and wording.
    - Wording fix.
    - lxc/image: Fix crash due to bad arg parsing
    - lxd: add missing limits.h include
    - lxd/init: Fix --auto with network config
    - lxc: Consistent naming of clustering terms
    - i18n: Update translation templates
    - lxc/file: Fix pushing files to remote
    - lxd/init: Don’t setup a remote storage pool by default
    - Fix lxd init failing to join cluster interactively with existing zfs pool
    - lxc/query: Fix -d and -X
    - lxc/help: Make help respect --all too
    - Fix typo in help of “lxc network”
    - Properly filter node-level storage configs by pool ID
    - i18n: Update translation templates
    - lxd/init: Consistency
    - Make new gofmt happy
    - lxc/file: Allow using -r to follow symlinks
    - Replace juju/idmclient with CanonicalLtd/candidclient
    - lxc/config: Fix adding trust cert on snap
    - lxc/alias: Fix example in help message
    - i18n: Update translation templates
    - client: Introduce LXD_SOCKET
    - Makefile: Add a manifest
    - containers: fix snapshot deletion
    - lxc/init: Add missing --no-profiles
    - i18n: Update translations
    - lxc/file: Fix pull target logic
    - doc: Fix example in userns-idmap
    - devices: fail if Nvidia device minor is missing
    - Add db.ContainersNodeList
    - storage: createContainerMountpoint() fix perms
    - ceph: s/0755/0711/g
    - lvm: s/0755/0711/g
    - storage utils: s/0755/0711/g
    - zfs: s/0755/0711/g
    - patches: add “storage_api_path_permissions”
    - sys/fs: s/MkdirAll/Mkdir/g
    - btrfs: fix permissions
    - Pass a logger to raft-http
    - Add new cluster.Promote function
    - Add new cluster.Rebalance function
    - Notify the cluster leader after a node removal, so it can rebalance
    - Add integration test
    - doc: Tweak backup.md
    - lxd/init: Require root for interactive cluster join
    - Disable flaky unit tests for now
    - Log the error that made Daemon.Init() fail
    - client: Expose http URL in ConnectionInfo
    - lxc/query: Add support for non-JSON endpoints
    - Handle empty query strings
    - Support reading queries from standard in
    - Support passing multiple queries
    - Rename database files
    - Support querying both local and global database
    - Update integration tests
    - Normalize name of images_aliases table
    - Add query.Dump helper to dump schema and data
    - Add support for dump command in lxd sql
    - lxd/containers: Fix lxc.net 1 check
    - doc/backup.md: update snap path
    - Add lxc cluster enable command
    - Fix command description formatting
    - Update .pot files
    - Use an isolated LXD instance in integration tests
    - Start a container in the integration test
    - Address style comments
    - add LXD_UNPRIVILEGED_ONLY to disallow privileged containers.
    - lxd: tweak LXD_UNPRIVILEGED_O...

Read more...

Changed in lxd (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Łukasz Zemczak (sil2100) wrote : Please test proposed package

Hello Julian, or anyone else affected,

Accepted lxd into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/lxd/3.0.1-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!

Changed in lxd (Ubuntu Bionic):
status: New → Fix Committed
tags: added: verification-needed verification-needed-bionic
Revision history for this message
Julian Andres Klode (juliank) wrote :

I looked at https://launchpad.net/ubuntu/bionic/amd64/lxd/3.0.1-0ubuntu1~18.04.1 and the depends is there now, so verified.

tags: added: verification-done verification-done-bionic
removed: verification-needed verification-needed-bionic
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (7.0 KiB)

This bug was fixed in the package lxd - 3.0.1-0ubuntu1~18.04.1

---------------
lxd (3.0.1-0ubuntu1~18.04.1) bionic; urgency=medium

  * New usptream bugfix release (LP: #1775290):
    - lxc: Fix mistakenly hidden commands
    - i18n: Update translation templates
    - lxd/migration: Pre-validate profiles
    - client: Improve remote operation errors
    - Fix some typos and wording.
    - Wording fix.
    - lxc/image: Fix crash due to bad arg parsing
    - lxd: add missing limits.h include
    - lxd/init: Fix --auto with network config
    - lxc: Consistent naming of clustering terms
    - i18n: Update translation templates
    - lxc/file: Fix pushing files to remote
    - lxd/init: Don’t setup a remote storage pool by default
    - Fix lxd init failing to join cluster interactively with existing zfs pool
    - lxc/query: Fix -d and -X
    - lxc/help: Make help respect --all too
    - Fix typo in help of “lxc network”
    - Properly filter node-level storage configs by pool ID
    - i18n: Update translation templates
    - lxd/init: Consistency
    - Make new gofmt happy
    - lxc/file: Allow using -r to follow symlinks
    - Replace juju/idmclient with CanonicalLtd/candidclient
    - lxc/config: Fix adding trust cert on snap
    - lxc/alias: Fix example in help message
    - i18n: Update translation templates
    - client: Introduce LXD_SOCKET
    - Makefile: Add a manifest
    - containers: fix snapshot deletion
    - lxc/init: Add missing --no-profiles
    - i18n: Update translations
    - lxc/file: Fix pull target logic
    - doc: Fix example in userns-idmap
    - devices: fail if Nvidia device minor is missing
    - Add db.ContainersNodeList
    - storage: createContainerMountpoint() fix perms
    - ceph: s/0755/0711/g
    - lvm: s/0755/0711/g
    - storage utils: s/0755/0711/g
    - zfs: s/0755/0711/g
    - patches: add “storage_api_path_permissions”
    - sys/fs: s/MkdirAll/Mkdir/g
    - btrfs: fix permissions
    - Pass a logger to raft-http
    - Add new cluster.Promote function
    - Add new cluster.Rebalance function
    - Notify the cluster leader after a node removal, so it can rebalance
    - Add integration test
    - doc: Tweak backup.md
    - lxd/init: Require root for interactive cluster join
    - Disable flaky unit tests for now
    - Log the error that made Daemon.Init() fail
    - client: Expose http URL in ConnectionInfo
    - lxc/query: Add support for non-JSON endpoints
    - Handle empty query strings
    - Support reading queries from standard in
    - Support passing multiple queries
    - Rename database files
    - Support querying both local and global database
    - Update integration tests
    - Normalize name of images_aliases table
    - Add query.Dump helper to dump schema and data
    - Add support for dump command in lxd sql
    - lxd/containers: Fix lxc.net 1 check
    - doc/backup.md: update snap path
    - Add lxc cluster enable command
    - Fix command description formatting
    - Update .pot files
    - Use an isolated LXD instance in integration tests
    - Start a container in the integration test
    - Address style comments
    - add LXD_UNPRIVILEGED_ONLY to disallow privileged containers.
    ...

Read more...

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

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

Scott Moser (smoser)
Changed in cloud-images:
status: New → Invalid
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.