[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-08-29 Thread rhtyd
Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1593#discussion_r76731031 --- Diff: usage/src/com/cloud/usage/UsageSanityChecker.java --- @@ -43,9 +43,9 @@ protected static final int DEFAULT_AGGREGATION_RANGE = 1440;

[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-08-29 Thread rhtyd
Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1593#discussion_r76730973 --- Diff: usage/conf/log4j-cloud_usage.xml.in --- @@ -48,7 +48,7 @@ under the License. - + ---

[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-08-29 Thread rhtyd
Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1593#discussion_r76730939 --- Diff: debian/cloudstack-usage.postinst --- @@ -43,6 +43,10 @@ case "$1" in rm -rf /etc/cloudstack/usage/key ln -s

[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-08-22 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1593#discussion_r75795653 --- Diff: usage/src/com/cloud/usage/UsageSanityChecker.java --- @@ -72,7 +72,9 @@ protected boolean checkItemCountByPstmt(CheckCase checkCase) throws

[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-08-16 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1593#discussion_r74966728 --- Diff: usage/src/com/cloud/usage/UsageSanityChecker.java --- @@ -72,7 +72,9 @@ protected boolean checkItemCountByPstmt(CheckCase checkCase) throws

[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-08-16 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1593#discussion_r74957023 --- Diff: usage/src/com/cloud/usage/UsageSanityChecker.java --- @@ -72,7 +72,9 @@ protected boolean checkItemCountByPstmt(CheckCase checkCase)

[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-08-15 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1593#discussion_r74807418 --- Diff: usage/src/com/cloud/usage/UsageSanityChecker.java --- @@ -72,7 +72,9 @@ protected boolean checkItemCountByPstmt(CheckCase checkCase) throws

[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-08-11 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1593#discussion_r74468365 --- Diff: usage/src/com/cloud/usage/UsageSanityChecker.java --- @@ -72,7 +72,9 @@ protected boolean checkItemCountByPstmt(CheckCase checkCase)

[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-06-21 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1593#discussion_r67938119 --- Diff: usage/src/com/cloud/usage/UsageSanityChecker.java --- @@ -72,7 +72,9 @@ protected boolean checkItemCountByPstmt(CheckCase checkCase) throws

[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-06-19 Thread serg38
Github user serg38 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1593#discussion_r67618240 --- Diff: usage/conf/log4j-cloud_usage.xml.in --- @@ -48,7 +48,7 @@ under the License. - + ---

[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-06-17 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1593#discussion_r67565090 --- Diff: usage/conf/log4j-cloud_usage.xml.in --- @@ -48,7 +48,7 @@ under the License. - +

[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-06-17 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1593#discussion_r67563826 --- Diff: usage/src/com/cloud/usage/UsageSanityChecker.java --- @@ -72,7 +72,9 @@ protected boolean checkItemCountByPstmt(CheckCase checkCase) throws

[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-06-15 Thread nvazquez
GitHub user nvazquez opened a pull request: https://github.com/apache/cloudstack/pull/1593 CLOUDSTACK-9417: Usage module refactoring ### Introduction Usage sanity check file was not been updated on sanity check. It is proposed: * New usage folder