SRU pollinate 4.25
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
pollinate (Ubuntu) |
Fix Released
|
Medium
|
Unassigned | ||
Trusty |
Fix Released
|
Medium
|
Scott Moser | ||
Xenial |
Fix Released
|
Medium
|
Scott Moser | ||
Zesty |
Fix Released
|
Medium
|
Unassigned | ||
Artful |
Fix Released
|
Medium
|
Unassigned |
Bug Description
=== Begin SRU Template ===
[Impact]
The key change is that the user agent string as of Zesty and Artful now
contains the contents of /proc/uptime. In aggregate, this gives us
important performance data to detect improvements and regressions in
Ubuntu boot times.
[Test Case]
Launch a privileged container with:
1. lxc launch ubuntu-daily:xenial -c security.
2. lxc exec x1-priv -- apt update && apt -y install pollen
Run a local pollen server and have the pollinate (client) connect with that via:
3. lxc exec x1-priv -- pollinate -r --insecure -s https:/
Then check in /var/log/syslog for the client user-agent string. On systems without the updated pollinate client, the USER_AGENT value does not include uptime:
root@x1-priv:~# grep pollen /var/log/syslog | grep -c uptime
0
root@x1-priv:~#
After upgrading to the newer client, uptime is now present in the pollen entries:
root@x1-priv:~# grep pollen /var/log/syslog | grep -c uptime
2
root@x1-priv:~#
The two changes are:
a.) the defaults file /etc/default/
execution. That was unlikely to cause issue, but the cleanup removed
unnecessary duplication.
b.) uptime is read from /proc/cpuinfo and added to the USER_AGENT
variable that is fed to curl.
[Regression Potential]
The script runs with 'set -e', and reads from /proc/uptime with:
read up idle < /proc/uptime
That means that the 'pollinate' script will exit failure if either:
a.) /proc/uptime did not exist (possibly proc not mounted)
b.) a read error occurred.
These scenarios seem unlikely, especially in Ubuntu environments.
The upstream code should check to at least check the readability of
/proc/uptime first. I've filed bug 1708461 to address that issue
and submitted a merge proposal for a fix.
[Other Info]
The reading of up/idle from uptime was originally added under bug 1638552.
=== End SRU Template ===
Changed in pollinate (Ubuntu Zesty): | |
status: | New → Fix Released |
importance: | Undecided → Medium |
Changed in pollinate (Ubuntu Artful): | |
status: | New → Fix Released |
importance: | Undecided → Medium |
Changed in pollinate (Ubuntu Xenial): | |
status: | New → Confirmed |
Changed in pollinate (Ubuntu Trusty): | |
status: | New → Confirmed |
importance: | Undecided → Medium |
Changed in pollinate (Ubuntu Xenial): | |
importance: | Undecided → Medium |
description: | updated |
summary: |
- SRU pollinate > 4.25 + SRU pollinate 4.25 |
description: | updated |
Changed in pollinate (Ubuntu Trusty): | |
status: | Confirmed → In Progress |
Changed in pollinate (Ubuntu Xenial): | |
status: | Confirmed → In Progress |
Changed in pollinate (Ubuntu Trusty): | |
assignee: | nobody → Scott Moser (smoser) |
Changed in pollinate (Ubuntu Xenial): | |
assignee: | nobody → Scott Moser (smoser) |
description: | updated |
tags: |
added: verification-done verification-done-trusty verification-done-xenial removed: verification-needed verification-needed-trusty verification-needed-xenial |
pollinate at 4.25-0ubuntu1~ 16.04.1 and 4.25-0ubuntu1~ 14.04.1 is in xenial and trusty queue: /launchpad. net/ubuntu/ xenial/ +queue? queue_state= 1&queue_ text=pollinate /launchpad. net/ubuntu/ trusty/ +queue? queue_state= 1&queue_ text=pollinate
https:/
https:/