It would be nice if cloud-init provides full version in logs
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
cloud-init |
Fix Released
|
Medium
|
Unassigned | ||
cloud-init (Ubuntu) |
Fix Released
|
Medium
|
Unassigned | ||
Xenial |
Fix Released
|
Medium
|
Unassigned | ||
Artful |
Won't Fix
|
Medium
|
Unassigned | ||
Bionic |
Fix Released
|
Medium
|
Unassigned | ||
Cosmic |
Fix Released
|
Medium
|
Unassigned |
Bug Description
=== Begin SRU Template ===
[Impact]
Cloud-init logs should now contain full packaged version of cloud-init on xenial, artful and bionic.
[Test Case]
# We should see specific version and patch information
for series in xenial artful bionic;
do
echo '=== BEGIN ' $series ' ==='
ref=
lxc delete test-$series --force;
lxc-
lxc init $ref test-$series;
lxc start test-$series;
packaged_
lxc exec test-$series -- grep $packaged_version /var/log/
lxc exec test-$series -- cloud-init --version;
done
# Also, cloud-init --version should show the packaged version
# it should contain a -0ubuntu portion.
$ cloud-init --version
/usr/bin/cloud-init 18.3-9-
[Regression Potential]
This really should be low chance of regression. The chance would be
if something is running 'cloud-init --version' and parsing the output,
or parsing the output of /var/log/
Such specific parsing of a log seems brittle anyway. Parsing output
of --version that expected to not have a -0ubuntuX in it would need to
be updated.
[Other Info]
Upstream commit at
https:/
=== End SRU Template ===
[Test Case]
# We should see specific version and patch information
$ packaged_
$ grep $packaged_version /var/log/
...
2018-07-10 19:33:16,406 - util.py[DEBUG]: Cloud-init v. 18.3-9-
# Also, cloud-init --version should show the packaged version
# it should contain a -0ubuntu portion.
$ cloud-init --version
/usr/bin/cloud-init 18.3-9-
[Regression Potential]
This really should be low chance of regression. The chance would be
if something is running 'cloud-init --version' and parsing the output,
or parsing the output of /var/log/
Such specific parsing of a log seems brittle anyway. Parsing output
of --version that expected to not have a -0ubuntuX in it would need to
be updated.
[Other Info]
Upstream commit at
https:/
=== End SRU Template ===
=== Original Description ===
Cloud-init rsyslog has the major version of cloud-init:
May 11 17:40:51 maas-enlisting-node cloud-init[550]: Cloud-init v. 18.2 running 'init-local' at Fri, 11 May 2018 17:40:47 +0000. Up 15.63 seconds.
However, it would be nice if it places the whole version, so that we can now exactly what version of cloud-init its running, e.g:
May 11 17:40:51 maas-enlisting-node cloud-init[550]: Cloud-init v. 18.2 (27-g6ef92c98-
Related branches
- Chad Smith: Approve
- Server Team CI bot: Approve (continuous-integration)
-
Diff: 32 lines (+9/-0)2 files modifieddebian/changelog (+7/-0)
debian/rules (+2/-0)
- Chad Smith: Approve
- Server Team CI bot: Approve (continuous-integration)
-
Diff: 32 lines (+9/-0)2 files modifieddebian/changelog (+7/-0)
debian/rules (+2/-0)
- Chad Smith: Approve
- Server Team CI bot: Approve (continuous-integration)
-
Diff: 32 lines (+9/-0)2 files modifieddebian/changelog (+7/-0)
debian/rules (+2/-0)
- Chad Smith: Approve
- Server Team CI bot: Approve (continuous-integration)
-
Diff: 32 lines (+9/-0)2 files modifieddebian/changelog (+7/-0)
debian/rules (+2/-0)
- Server Team CI bot: Approve (continuous-integration)
- Chad Smith: Approve
- Ryan Harper: Approve
-
Diff: 75 lines (+23/-0)3 files modifiedcloudinit/tests/test_version.py (+17/-0)
cloudinit/version.py (+4/-0)
packages/debian/rules.in (+2/-0)
summary: |
- It would be nice if cloud-init spits out the full version + It would be nice if cloud-init provides full version in logs |
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 Artful): | |
status: | New → Confirmed |
Changed in cloud-init (Ubuntu Bionic): | |
status: | New → Confirmed |
Changed in cloud-init (Ubuntu Xenial): | |
importance: | Undecided → Medium |
Changed in cloud-init (Ubuntu Artful): | |
importance: | Undecided → Medium |
Changed in cloud-init (Ubuntu Bionic): | |
importance: | Undecided → Medium |
description: | updated |
We can just add the same mechanism for packaging to patch in the version that we have in curtin. /git.launchpad. net/~ssh- import- id/ssh- import- id/commit/ ?id=39226c638de 6b0210b78866e50 b08627084b4868
I just added the same basic thing to ssh-import-id
https:/