[ https://issues.apache.org/jira/browse/CLOUDSTACK-9417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15423130#comment-15423130 ]
ASF GitHub Bot commented on CLOUDSTACK-9417: -------------------------------------------- Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1593 @serg38 I only see code review LGTMs. We need at least one test LGTM before this PR. In this case, those tests must be run against real hardware and hypervisors. Also, we need an LGTM from all reviewers that raised an issue. I haven't had a chance tore-review as I am on leave this week. (All PRs require at least one code review and at least test LGTM to be merged). > Usage module refactoring > ------------------------ > > Key: CLOUDSTACK-9417 > URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9417 > Project: CloudStack > Issue Type: Bug > Security Level: Public(Anyone can view this level - this is the > default.) > Components: Usage > Affects Versions: 4.9.0 > Reporter: Nicolas Vazquez > Assignee: Nicolas Vazquez > Fix For: 4.9.1 > > > h3. Introduction > Usage sanity check file was not been updated on sanity check. > It is proposed: > * New usage folder {{/var/cache/cloudstack/usage}}, creation on > cloudstack-usage package built. > * New sanity check file location in new folder {{/var/cache/cloudstack/usage}} > * Timestamp included in {{usage.log}} file > * Include {{updateMaxId()}} on sanity check as it wasn't being updated -- This message was sent by Atlassian JIRA (v6.3.4#6332)