[GitHub] cloudstack pull request #1944: CLOUDSTACK-9783: Improve metrics view perform...
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1944 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1944: CLOUDSTACK-9783: Improve metrics view perform...
GitHub user rhtyd reopened a pull request: https://github.com/apache/cloudstack/pull/1944 CLOUDSTACK-9783: Improve metrics view performance This improves the metrics view feature by improving the rendering performance of metrics view tables, by re-implementing the logic at the backend and data served via APIs. In large environments, the older implementation would make several API calls that increases both network and database load. List of APIs introduced for improving the performance that re-implement the frontend logic at backend: listClustersMetrics listHostsMetrics listInfrastructure listStoragePoolsMetrics listVMsMetrics listVolumesMetrics listZonesMetrics Pinging for review - @abhinandanprateek @DaanHoogland @borisstoyanov @karuturi @rashmidixit Marvin test results: === TestName: test_list_clusters_metrics | Status : SUCCESS === === TestName: test_list_hosts_metrics | Status : SUCCESS === === TestName: test_list_infrastructure_metrics | Status : SUCCESS === === TestName: test_list_pstorage_metrics | Status : SUCCESS === === TestName: test_list_vms_metrics | Status : SUCCESS === === TestName: test_list_volumes_metrics | Status : SUCCESS === === TestName: test_list_zones_metrics | Status : SUCCESS === You can merge this pull request into a Git repository by running: $ git pull https://github.com/shapeblue/cloudstack 4.9-metrics-enhancement Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1944.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1944 commit 402253504e9520104caf9fbc1317042f2fd89474 Author: Rohit Yadav Date: 2017-02-16T08:15:15Z CLOUDSTACK-9783: Improve metrics view performance This improves the metrics view feature by improving the rendering performance of metrics view tables, by reimplementing the logic at the backend and data served via APIs. In large environments, the older implementation would make several API calls that increases both network and database load. List of APIs introduced for improving the performance: listClustersMetrics listHostsMetrics listInfrastructure listStoragePoolsMetrics listVMsMetrics listVolumesMetrics listZonesMetrics Signed-off-by: Rohit Yadav --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1944: CLOUDSTACK-9783: Improve metrics view perform...
Github user rhtyd closed the pull request at: https://github.com/apache/cloudstack/pull/1944 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1944: CLOUDSTACK-9783: Improve metrics view perform...
GitHub user rhtyd reopened a pull request: https://github.com/apache/cloudstack/pull/1944 CLOUDSTACK-9783: Improve metrics view performance This improves the metrics view feature by improving the rendering performance of metrics view tables, by re-implementing the logic at the backend and data served via APIs. In large environments, the older implementation would make several API calls that increases both network and database load. List of APIs introduced for improving the performance that re-implement the frontend logic at backend: listClustersMetrics listHostsMetrics listInfrastructure listStoragePoolsMetrics listVMsMetrics listVolumesMetrics listZonesMetrics Pinging for review - @abhinandanprateek @DaanHoogland @borisstoyanov @karuturi @rashmidixit Marvin test results: === TestName: test_list_clusters_metrics | Status : SUCCESS === === TestName: test_list_hosts_metrics | Status : SUCCESS === === TestName: test_list_infrastructure_metrics | Status : SUCCESS === === TestName: test_list_pstorage_metrics | Status : SUCCESS === === TestName: test_list_vms_metrics | Status : SUCCESS === === TestName: test_list_volumes_metrics | Status : SUCCESS === === TestName: test_list_zones_metrics | Status : SUCCESS === You can merge this pull request into a Git repository by running: $ git pull https://github.com/shapeblue/cloudstack 4.9-metrics-enhancement Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1944.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1944 commit 3881d249d07f16bde77c20450c5f39425a135e42 Author: Rohit Yadav Date: 2017-02-16T08:15:15Z CLOUDSTACK-9783: Improve metrics view performance This improves the metrics view feature by improving the rendering performance of metrics view tables, by reimplementing the logic at the backend and data served via APIs. In large environments, the older implementation would make several API calls that increases both network and database load. List of APIs introduced for improving the performance: listClustersMetrics listHostsMetrics listInfrastructure listStoragePoolsMetrics listVMsMetrics listVolumesMetrics listZonesMetrics Signed-off-by: Rohit Yadav --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1944: CLOUDSTACK-9783: Improve metrics view perform...
Github user rhtyd closed the pull request at: https://github.com/apache/cloudstack/pull/1944 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1944: CLOUDSTACK-9783: Improve metrics view perform...
Github user marcaurele commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1944#discussion_r103143033
--- Diff: plugins/metrics/pom.xml ---
@@ -0,0 +1,55 @@
+
+http://maven.apache.org/POM/4.0.0";
+ xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
+ xsi:schemaLocation="http://maven.apache.org/POM/4.0.0
+ http://maven.apache.org/xsd/maven-4.0.0.xsd";>
+ 4.0.0
+ cloud-plugin-metrics
+ Apache CloudStack Plugin - Metrics
+
+org.apache.cloudstack
+cloudstack-plugins
+4.9.3.0-SNAPSHOT
+../pom.xml
+
+
+
+ org.apache.cloudstack
+ cloud-api
+ ${project.version}
+
+
+ org.apache.cloudstack
+ cloud-utils
+ ${project.version}
+
+
+
+
--- End diff --
Would be nice to follow maven standards on new modules. Since you cannot
inherit from `cloud-maven-standard` you have to put all those lines:
```
${basedir}/src/main/java
${basedir}/src/main/scripts
${basedir}/src/test/java
${basedir}/target/classes
${basedir}/target/test-classes
${basedir}/src/main/resources
${basedir}/src/test/resources
...
```
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack pull request #1944: CLOUDSTACK-9783: Improve metrics view perform...
GitHub user rhtyd opened a pull request: https://github.com/apache/cloudstack/pull/1944 CLOUDSTACK-9783: Improve metrics view performance This improves the metrics view feature by improving the rendering performance of metrics view tables, by re-implementing the logic at the backend and data served via APIs. In large environments, the older implementation would make several API calls that increases both network and database load. List of APIs introduced for improving the performance that re-implement the frontend logic at backend: listClustersMetrics listHostsMetrics listInfrastructure listStoragePoolsMetrics listVMsMetrics listVolumesMetrics listZonesMetrics Pinging for review - @abhinandanprateek @DaanHoogland @borisstoyanov @karuturi @rashmidixit You can merge this pull request into a Git repository by running: $ git pull https://github.com/shapeblue/cloudstack 4.9-metrics-enhancement Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1944.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1944 commit 3aba6ad6c095c066ab0334c8a48565741b2913fe Author: Rohit Yadav Date: 2017-02-16T08:15:15Z CLOUDSTACK-9783: Improve metrics view performance This improves the metrics view feature by improving the rendering performance of metrics view tables, by reimplementing the logic at the backend and data served via APIs. In large environments, the older implementation would make several API calls that increases both network and database load. List of APIs introduced for improving the performance: listClustersMetrics listHostsMetrics listInfrastructure listStoragePoolsMetrics listVMsMetrics listVolumesMetrics listZonesMetrics Signed-off-by: Rohit Yadav --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
