Bug 772755 - aeolus conductor uptime is not reset when user "restarts" instance via webui
Summary: aeolus conductor uptime is not reset when user "restarts" instance via webui
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-conductor
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
Assignee: Imre Farkas
QA Contact: wes hayutin
URL: http://server/conductor/deployments/1...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-09 21:34 UTC by wes hayutin
Modified: 2012-05-15 21:33 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-05-15 21:33:35 UTC


Attachments (Terms of Use)
total_usage_time (215.28 KB, image/png)
2012-01-31 14:48 UTC, Shveta
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2012:0583 0 normal SHIPPED_LIVE new packages: aeolus-conductor 2012-05-15 22:31:59 UTC

Description wes hayutin 2012-01-09 21:34:52 UTC
Description of problem:

1. start an instance via webui, in the case I used vsphere
2. notice the instance's uptime
3. restart the instance

Expected results:
The uptime of the instance is reset

Actual results:
The uptime continues to add to the original uptime


[root@qeblade31 yum.repos.d]# rpm -qa | grep aeolus
aeolus-conductor-daemons-0.9.0-1.el6.noarch
rubygem-aeolus-cli-0.4.0-1.el6.noarch
rubygem-aeolus-image-0.4.0-1.el6.noarch
aeolus-all-0.9.0-1.el6.noarch
aeolus-configure-2.6.0-1.el6.noarch
aeolus-conductor-0.9.0-1.el6.noarch
aeolus-conductor-doc-0.9.0-1.el6.noarch
[root@qeblade31 yum.repos.d]# rpm -qa | grepd deltacloud
-bash: grepd: command not found
[root@qeblade31 yum.repos.d]# rpm -qa | grep deltacloud
deltacloud-core-ec2-0.5.0-0.rc1.el6.noarch
deltacloud-core-vsphere-0.5.0-0.rc1.el6.noarch
deltacloud-core-0.5.0-0.rc1.el6.noarch
rubygem-deltacloud-client-0.4.0-3.el6.noarch
deltacloud-core-rhevm-0.5.0-0.rc1.el6.noarch

Comment 1 Scott Seago 2012-01-10 15:26:03 UTC
So the real question here is whether 'uptime' is intended to mean 'how long since last boot', or 'what is the accumulated uptime of this instance'?

I was under the impression that we were really measuring the latter -- the reason being that back when we were thinking about stateful instances a couple years ago, that's how we were starting to define the model. I can see both as useful metrics, though -- but I don't know if we are explicitly modeling the 'how long since last boot' metric.

Until we added reboot (and only had stateless/non-restartable instances), the issue wasn't important. It's marginally more so now, and once we add restartable stateful instances (i.e. I can stop it now, and I can start the same instance again tomorrow) it will be significantly more important.

Comment 2 Jason Guiditta 2012-01-10 16:20:29 UTC
yeah, this sounds like a measurement of 'total instance usage time' rather than 'uptime'.  Uptime is total continuous time running, imo

Comment 3 Angus Thomas 2012-01-10 16:27:24 UTC
For 1.0, we should continue to report the metric that we currently show, but should label it as "total usage time", in order to remove the ambiguity.

Comment 4 wes hayutin 2012-01-12 16:35:20 UTC
adding to ce-sprint

Comment 5 wes hayutin 2012-01-12 16:41:42 UTC
removing ce-sprint-next tracker

Comment 6 Imre Farkas 2012-01-17 15:54:20 UTC
Patch has been posted: https://fedorahosted.org/pipermail/aeolus-devel/2012-January/008145.html

Comment 7 Imre Farkas 2012-01-25 08:09:47 UTC
This issue has been fixed. Please verify the commit ffc52615aefbeca3dc3f945fe1c800ad5aa32459

Comment 8 Steve Linabery 2012-01-26 16:47:38 UTC
ffc5261  in aeolus-conductor-0.8.0-12

Comment 10 Shveta 2012-01-31 14:48:00 UTC
Created attachment 558631 [details]
total_usage_time

Comment 11 Shveta 2012-01-31 14:49:10 UTC
Uptime replaced by "total usage time"

Verified in
rpm -qa|grep aeolus
aeolus-conductor-doc-0.8.0-16.el6.noarch
aeolus-configure-2.5.0-11.el6.noarch
aeolus-conductor-daemons-0.8.0-16.el6.noarch
rubygem-aeolus-image-0.3.0-6.el6.noarch
aeolus-all-0.8.0-16.el6.noarch
aeolus-conductor-0.8.0-16.el6.noarch
rubygem-aeolus-cli-0.3.0-7.el6.noarch

Comment 12 errata-xmlrpc 2012-05-15 21:33:35 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHEA-2012-0583.html


Note You need to log in before you can comment on or make changes to this bug.