nova-lxd needs to read 'product_name' in environment, not 'platform'
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
cloud-init |
Fix Released
|
High
|
Scott Moser | ||
cloud-init (Ubuntu) |
Fix Released
|
Medium
|
Unassigned | ||
Xenial |
Fix Released
|
Medium
|
Unassigned | ||
Yakkety |
Fix Released
|
Medium
|
Unassigned | ||
Zesty |
Fix Released
|
Medium
|
Unassigned | ||
Artful |
Fix Released
|
Medium
|
Unassigned |
Bug Description
=== Begin SRU Template ===
[Impact]
Warning message for nova-lxd images in openstack clouds due to no valid datasource found.
[Test Case]
# It downloads a cloud image of a given release, and then creates a -proposed
image with cloud-init upgraded.
wget https:/
chmod 755 get-proposed-image
source novarc
for release in xenial yakkety zesty do;
ref=$release-
lxc-proposed-
lxc image export $ref .;
imagefile=`ls -tr *gz | tail -n 1`
mkdir $ref;
cd $ref;
sudo tar -zxvpf ../$imagefile; # preserve permissions
cd rootfs;
sudo tar zcpvf $ref.tar.gz *;
#upload raw image to your cloud
openstack image create --disk-format raw --container-format bare --file $ref.tar.gz testing/
openstack server create --image testing/$ref.tar.gz --flavor=m1.tiny lxd-$release --key-name <your-key>;
nova floating-ip-create;
nova foating-
ssh ubuntu@<address> 'sudo DEBUG_LEVEL=2 DI_LOG=stderr /usr/lib/
done
[Regression Potential]
Low as this only addresses the warning by correctly identifying the OpenStack cloud datasource.
[Other Info]
=== End SRU Template ===
It seems that signals were crossed in bug 1661797.
cloud-init implementation reads the environment variable 'platform' from pid1, and nova-lxd implementation exported 'product_name'.
Thus, ssh to nova-lxd provided container, user sees a warning.
Also
$ sudo DEBUG_LEVEL=2 DI_LOG=stderr /usr/lib/
PID_1_PLATFORM=
While:
$ sudo cat /proc/1/environ | tr '\0' '\n'
product_
container=lxc
Related bugs:
* bug 1661797: identify lxd-nova platform to enable Openstack datasource
Related branches
- Server Team CI bot: Approve (continuous-integration)
- cloud-init Commiters: Pending requested
-
Diff: 65 lines (+9/-9)1 file modifiedtools/ds-identify (+9/-9)
description: | updated |
Changed in cloud-init: | |
status: | Confirmed → Fix Committed |
Changed in cloud-init (Ubuntu): | |
status: | New → Confirmed |
importance: | Undecided → Medium |
Changed in cloud-init (Ubuntu Xenial): | |
status: | New → Confirmed |
Changed in cloud-init (Ubuntu Yakkety): | |
status: | New → Confirmed |
Changed in cloud-init (Ubuntu Zesty): | |
status: | New → Confirmed |
Changed in cloud-init (Ubuntu Xenial): | |
importance: | Undecided → Medium |
Changed in cloud-init (Ubuntu Yakkety): | |
importance: | Undecided → Medium |
Changed in cloud-init (Ubuntu Zesty): | |
importance: | Undecided → Medium |
description: | updated |
description: | updated |
This bug was fixed in the package cloud-init - 0.7.9-113- g513e99e0- 0ubuntu1
--------------- 113-g513e99e0- 0ubuntu1) artful; urgency=medium
cloud-init (0.7.9-
* New upstream snapshot.
- nova-lxd: read product_name from environment, not platform.
(LP: #1685810)
- Fix yum repo config where keys contain array values
[Dylan Perry] (LP: #1592150)
- template: Update debian backports template [Joshua Powers] (LP: #1627293)
- rsyslog: replace ~ with stop [Joshua Powers] (LP: #1367899)
- Doc: add additional RTD examples [Joshua Powers] (LP: #1459604)
- Fix growpart for some cases when booted with root=PARTUUID.
(LP: #1684869)
- pylint: update output style to parseable [Joshua Powers]
- pylint: fix all logging warnings [Joshua Powers]
- CloudStack: Add NetworkManager to list of supported DHCP lease dirs.
[Syed Mushtaq Ahmed]
- net: kernel lies about vlans not stealing mac addresses, when they do
[Dimitri John Ledkov] (LP: #1682871)
- ds-identify: Check correct path for "latest" config drive
[Daniel Watkins] (LP: #1673637)
- doc: Fix example for resolve.conf configuration.
[Jon Grimm] (LP: #1531582)
- Fix examples that reference upstream chef repository.
[Jon Grimm] (LP: #1678145)
- doc: correct grammar and improve clarity in merging documentation.
[David Tagatac]
- doc: Add missing doc link to snap-config module. [Ryan Harper]
- snap: allows for creating cloud-init snap [Joshua Powers]
- DigitalOcean: assign IPv4ll address to lowest indexed interface.
[Ben Howard]
- DigitalOcean: configure all NICs presented in meta-data. [Ben Howard]
- Remove (and/or fix) URL shortener references [Jon Grimm] (LP: #1669727)
- HACKING.rst: more info on filling out contributors agreement.
[Scott Moser]
- util: teach write_file about copy_mode option
[Lars Kellogg-Stedman] (LP: #1644064)
- DigitalOcean: bind resolvers to loopback interface. [Ben Howard]
- tests: fix AltCloud tests to not rely on blkid (LP: #1636531)
-- Scott Moser <email address hidden> Tue, 25 Apr 2017 16:34:08 -0400