[GitHub] blueorangutan commented on issue #2067: CLOUDSTACK-9852: This patch implements the storage lb feature

2017-12-25 Thread GitBox
blueorangutan commented on issue #2067: CLOUDSTACK-9852: This patch implements the storage lb feature URL: https://github.com/apache/cloudstack/pull/2067#issuecomment-353932586 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2067: CLOUDSTACK-9852: This patch implements the storage lb feature

2017-12-25 Thread GitBox
rhtyd commented on issue #2067: CLOUDSTACK-9852: This patch implements the storage lb feature URL: https://github.com/apache/cloudstack/pull/2067#issuecomment-353932553 I've fixed the conflicts @anshul-gangwar but your engagement is requested for getting the PR accepted. Additional

[GitHub] blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-12-25 Thread GitBox
blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-353931496 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

[GitHub] blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-12-25 Thread GitBox
blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-353931499 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-12-25 Thread GitBox
rhtyd commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-353931480 @blueorangutan test centos7 kvm-centos7 component/test_ss_volume_usage.py

[GitHub] rhtyd commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-12-25 Thread GitBox
rhtyd commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-353931415 @blueorangutan test This is an automated message from the Apache Git Service. To respond to

[GitHub] marcaurele commented on a change in pull request #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume.

2017-12-25 Thread GitBox
marcaurele commented on a change in pull request #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume. URL: https://github.com/apache/cloudstack/pull/2373#discussion_r158675822 ## File path:

[GitHub] blueorangutan commented on issue #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume.

2017-12-25 Thread GitBox
blueorangutan commented on issue #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume. URL: https://github.com/apache/cloudstack/pull/2373#issuecomment-353931193 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I

[GitHub] rhtyd commented on issue #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume.

2017-12-25 Thread GitBox
rhtyd commented on issue #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume. URL: https://github.com/apache/cloudstack/pull/2373#issuecomment-353931174 @blueorangutan package This

[GitHub] blueorangutan commented on issue #2368: CLOUDSTACK-10126: Separate Subnet for SSVM and CPVM

2017-12-25 Thread GitBox
blueorangutan commented on issue #2368: CLOUDSTACK-10126: Separate Subnet for SSVM and CPVM URL: https://github.com/apache/cloudstack/pull/2368#issuecomment-353931096 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage metrics

2017-12-25 Thread GitBox
blueorangutan commented on issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage metrics URL: https://github.com/apache/cloudstack/pull/2035#issuecomment-353931094 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2368: CLOUDSTACK-10126: Separate Subnet for SSVM and CPVM

2017-12-25 Thread GitBox
rhtyd commented on issue #2368: CLOUDSTACK-10126: Separate Subnet for SSVM and CPVM URL: https://github.com/apache/cloudstack/pull/2368#issuecomment-353931041 @blueorangutan package This is an automated message from the

[GitHub] rhtyd commented on issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage metrics

2017-12-25 Thread GitBox
rhtyd commented on issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage metrics URL: https://github.com/apache/cloudstack/pull/2035#issuecomment-353931059 @blueorangutan package This is an automated message from

[GitHub] blueorangutan commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

2017-12-25 Thread GitBox
blueorangutan commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls URL: https://github.com/apache/cloudstack/pull/2046#issuecomment-353930988 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

2017-12-25 Thread GitBox
rhtyd commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls URL: https://github.com/apache/cloudstack/pull/2046#issuecomment-353930933 @blueorangutan package This is an automated

[GitHub] blueorangutan commented on issue #2045: Fix snmptrap alert bug

2017-12-25 Thread GitBox
blueorangutan commented on issue #2045: Fix snmptrap alert bug URL: https://github.com/apache/cloudstack/pull/2045#issuecomment-353930772 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2045: Fix snmptrap alert bug

2017-12-25 Thread GitBox
rhtyd commented on issue #2045: Fix snmptrap alert bug URL: https://github.com/apache/cloudstack/pull/2045#issuecomment-353930662 @blueorangutan package This is an automated message from the Apache Git Service. To respond to

[GitHub] rhtyd commented on issue #2362: CLOUDSTACK-10188 - Resource Accounting for primary storage is Broken when Domains are in use

2017-12-25 Thread GitBox
rhtyd commented on issue #2362: CLOUDSTACK-10188 - Resource Accounting for primary storage is Broken when Domains are in use URL: https://github.com/apache/cloudstack/pull/2362#issuecomment-353930633 @bwsw can you fix the conflicts, thanks.

[GitHub] blueorangutan commented on issue #1773: CLOUDSTACK-9607: Preventing template deletion when template is in use.

2017-12-25 Thread GitBox
blueorangutan commented on issue #1773: CLOUDSTACK-9607: Preventing template deletion when template is in use. URL: https://github.com/apache/cloudstack/pull/1773#issuecomment-353930668 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #1773: CLOUDSTACK-9607: Preventing template deletion when template is in use.

2017-12-25 Thread GitBox
rhtyd commented on issue #1773: CLOUDSTACK-9607: Preventing template deletion when template is in use. URL: https://github.com/apache/cloudstack/pull/1773#issuecomment-353930588 @blueorangutan package This is an automated

[GitHub] blueorangutan commented on issue #2149: CLOUDSTACK-9932 snapshot is getting deleted while volume is in creating state

2017-12-25 Thread GitBox
blueorangutan commented on issue #2149: CLOUDSTACK-9932 snapshot is getting deleted while volume is in creating state URL: https://github.com/apache/cloudstack/pull/2149#issuecomment-353930477 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2149: CLOUDSTACK-9932 snapshot is getting deleted while volume is in creating state

2017-12-25 Thread GitBox
rhtyd commented on issue #2149: CLOUDSTACK-9932 snapshot is getting deleted while volume is in creating state URL: https://github.com/apache/cloudstack/pull/2149#issuecomment-353930378 @blueorangutan package This is an

[GitHub] rhtyd commented on issue #2352: CLOUDSTACK-10175: prevent VPC list leakage

2017-12-25 Thread GitBox
rhtyd commented on issue #2352: CLOUDSTACK-10175: prevent VPC list leakage URL: https://github.com/apache/cloudstack/pull/2352#issuecomment-353930350 @khos2ow any update? @blueorangutan package This is an automated message

[GitHub] rhtyd commented on issue #2340: CLOUDSTACK-10106: GPU/vGPU Support on VMware

2017-12-25 Thread GitBox
rhtyd commented on issue #2340: CLOUDSTACK-10106: GPU/vGPU Support on VMware URL: https://github.com/apache/cloudstack/pull/2340#issuecomment-353930286 @nitin-maharana I've left some remarks and I feel the code may be further refactored, thanks.

[GitHub] blueorangutan commented on issue #2352: CLOUDSTACK-10175: prevent VPC list leakage

2017-12-25 Thread GitBox
blueorangutan commented on issue #2352: CLOUDSTACK-10175: prevent VPC list leakage URL: https://github.com/apache/cloudstack/pull/2352#issuecomment-353930373 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-12-25 Thread GitBox
blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-353930213 Packaging result: ?centos6 ?centos7 ?debian. JID-1489

[GitHub] blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-12-25 Thread GitBox
blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-353930212 Packaging result: ?centos6 ?centos7 ?debian. JID-1490 This is an automated message

[GitHub] rhtyd commented on a change in pull request #2340: CLOUDSTACK-10106: GPU/vGPU Support on VMware

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2340: CLOUDSTACK-10106: GPU/vGPU Support on VMware URL: https://github.com/apache/cloudstack/pull/2340#discussion_r158675087 ## File path: vmware-base/src/com/cloud/hypervisor/vmware/mo/HostMO.java ## @@ -1184,4 +1206,261 @@

[GitHub] rhtyd commented on a change in pull request #2340: CLOUDSTACK-10106: GPU/vGPU Support on VMware

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2340: CLOUDSTACK-10106: GPU/vGPU Support on VMware URL: https://github.com/apache/cloudstack/pull/2340#discussion_r158675008 ## File path: vmware-base/src/com/cloud/hypervisor/vmware/mo/HostMO.java ## @@ -1184,4 +1206,261 @@

[GitHub] rhtyd commented on a change in pull request #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds URL: https://github.com/apache/cloudstack/pull/2244#discussion_r158674795 ## File path:

[GitHub] rhtyd commented on issue #2203: [CLOUDSTACK-10029] All private, public, and guest interfaceses are marked as untagged ca?

2017-12-25 Thread GitBox
rhtyd commented on issue #2203: [CLOUDSTACK-10029] All private, public, and guest interfaceses are marked as untagged ca? URL: https://github.com/apache/cloudstack/pull/2203#issuecomment-353929855 @konstantintrushin can you check if this is necessary given #2304 got merged.

[GitHub] rhtyd commented on a change in pull request #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds URL: https://github.com/apache/cloudstack/pull/2244#discussion_r158674795 ## File path:

[GitHub] rhtyd commented on a change in pull request #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds URL: https://github.com/apache/cloudstack/pull/2244#discussion_r158674726 ## File path:

[GitHub] blueorangutan commented on issue #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume.

2017-12-25 Thread GitBox
blueorangutan commented on issue #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume. URL: https://github.com/apache/cloudstack/pull/2373#issuecomment-353929623 Packaging result: ?centos6 ?centos7 ?debian. JID-1488

[GitHub] blueorangutan commented on issue #2315: A comprehensive solution for CLOUDSTACK-9025 and CLOUDSTACK-10128.

2017-12-25 Thread GitBox
blueorangutan commented on issue #2315: A comprehensive solution for CLOUDSTACK-9025 and CLOUDSTACK-10128. URL: https://github.com/apache/cloudstack/pull/2315#issuecomment-353929528 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2347: CLOUDSTACK-10166: Get accountId and domainId from VPC when tagging a NetworkACL as a user

2017-12-25 Thread GitBox
blueorangutan commented on issue #2347: CLOUDSTACK-10166: Get accountId and domainId from VPC when tagging a NetworkACL as a user URL: https://github.com/apache/cloudstack/pull/2347#issuecomment-353929522 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I

[GitHub] rhtyd commented on issue #2347: CLOUDSTACK-10166: Get accountId and domainId from VPC when tagging a NetworkACL as a user

2017-12-25 Thread GitBox
rhtyd commented on issue #2347: CLOUDSTACK-10166: Get accountId and domainId from VPC when tagging a NetworkACL as a user URL: https://github.com/apache/cloudstack/pull/2347#issuecomment-353929456 @blueorangutan package This

[GitHub] rhtyd commented on issue #2315: A comprehensive solution for CLOUDSTACK-9025 and CLOUDSTACK-10128.

2017-12-25 Thread GitBox
rhtyd commented on issue #2315: A comprehensive solution for CLOUDSTACK-9025 and CLOUDSTACK-10128. URL: https://github.com/apache/cloudstack/pull/2315#issuecomment-353929419 @blueorangutan package This is an automated

[GitHub] blueorangutan commented on issue #1934: [CLOUDSTACK-9772] Template: perform a HEAD request to check file size from a URL

2017-12-25 Thread GitBox
blueorangutan commented on issue #1934: [CLOUDSTACK-9772] Template: perform a HEAD request to check file size from a URL URL: https://github.com/apache/cloudstack/pull/1934#issuecomment-353929307 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

[GitHub] rhtyd commented on issue #2298: CLOUDSTACK-9620: Enhancements for managed storage

2017-12-25 Thread GitBox
rhtyd commented on issue #2298: CLOUDSTACK-9620: Enhancements for managed storage URL: https://github.com/apache/cloudstack/pull/2298#issuecomment-353929041 @mike-tutkowski I've left some comments, my only major concern is the placement of the `SIOC API plug-in for VMware SIOC` otherwise

[GitHub] rhtyd commented on issue #1934: [CLOUDSTACK-9772] Template: perform a HEAD request to check file size from a URL

2017-12-25 Thread GitBox
rhtyd commented on issue #1934: [CLOUDSTACK-9772] Template: perform a HEAD request to check file size from a URL URL: https://github.com/apache/cloudstack/pull/1934#issuecomment-353929226 @blueorangutan test This is an

[GitHub] rhtyd commented on issue #2298: CLOUDSTACK-9620: Enhancements for managed storage

2017-12-25 Thread GitBox
rhtyd commented on issue #2298: CLOUDSTACK-9620: Enhancements for managed storage URL: https://github.com/apache/cloudstack/pull/2298#issuecomment-353929041 @mike-tutkowski I've left some comments, my only major concern is the placement of the `SIOC API plug-in for VMware SIOC` otherwise

[GitHub] rhtyd commented on a change in pull request #2298: CLOUDSTACK-9620: Enhancements for managed storage

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2298: CLOUDSTACK-9620: Enhancements for managed storage URL: https://github.com/apache/cloudstack/pull/2298#discussion_r158673781 ## File path: plugins/api/solidfire-sioc/pom.xml ## @@ -0,0 +1,47 @@ +

[GitHub] rhtyd commented on a change in pull request #2298: CLOUDSTACK-9620: Enhancements for managed storage

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2298: CLOUDSTACK-9620: Enhancements for managed storage URL: https://github.com/apache/cloudstack/pull/2298#discussion_r158673931 ## File path:

[GitHub] rhtyd commented on a change in pull request #2298: CLOUDSTACK-9620: Enhancements for managed storage

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2298: CLOUDSTACK-9620: Enhancements for managed storage URL: https://github.com/apache/cloudstack/pull/2298#discussion_r158673893 ## File path:

[GitHub] rhtyd commented on a change in pull request #2298: CLOUDSTACK-9620: Enhancements for managed storage

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2298: CLOUDSTACK-9620: Enhancements for managed storage URL: https://github.com/apache/cloudstack/pull/2298#discussion_r158673781 ## File path: plugins/api/solidfire-sioc/pom.xml ## @@ -0,0 +1,47 @@ +

[GitHub] rhtyd commented on issue #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID.

2017-12-25 Thread GitBox
rhtyd commented on issue #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID. URL: https://github.com/apache/cloudstack/pull/1940#issuecomment-353927976 Tests LGTM. @DaanHoogland @syed are you still -1/nLGTM on it consider test results and code review? Additional

[GitHub] blueorangutan commented on issue #2146: CLOUDSTACK-4757: Support OVA files with multiple disks for templates

2017-12-25 Thread GitBox
blueorangutan commented on issue #2146: CLOUDSTACK-4757: Support OVA files with multiple disks for templates URL: https://github.com/apache/cloudstack/pull/2146#issuecomment-353927772 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2102: CLOUDSTACK-9889 Dedication of guest vlan range to a domain

2017-12-25 Thread GitBox
rhtyd commented on issue #2102: CLOUDSTACK-9889 Dedication of guest vlan range to a domain URL: https://github.com/apache/cloudstack/pull/2102#issuecomment-353927763 @yvsubhash please check/fix travis build/test failures

[GitHub] rhtyd commented on issue #2146: CLOUDSTACK-4757: Support OVA files with multiple disks for templates

2017-12-25 Thread GitBox
rhtyd commented on issue #2146: CLOUDSTACK-4757: Support OVA files with multiple disks for templates URL: https://github.com/apache/cloudstack/pull/2146#issuecomment-353927716 @blueorangutan package This is an automated

[GitHub] rhtyd commented on issue #2344: CLOUDSTACK-10163: Component tests sanity

2017-12-25 Thread GitBox
rhtyd commented on issue #2344: CLOUDSTACK-10163: Component tests sanity URL: https://github.com/apache/cloudstack/pull/2344#issuecomment-353927645 Pinging for additional review - @nvazquez @DaanHoogland @wido @marcaurele @rafaelweingartner and others, thanks.

[GitHub] blueorangutan commented on issue #2374: CLOUDSTACK-10024: Network Migration: Tackle review comments

2017-12-25 Thread GitBox
blueorangutan commented on issue #2374: CLOUDSTACK-10024: Network Migration: Tackle review comments URL: https://github.com/apache/cloudstack/pull/2374#issuecomment-353926876 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2374: CLOUDSTACK-10024: Network Migration: Tackle review comments

2017-12-25 Thread GitBox
rhtyd commented on issue #2374: CLOUDSTACK-10024: Network Migration: Tackle review comments URL: https://github.com/apache/cloudstack/pull/2374#issuecomment-353926773 Additional review requested - @borisstoyanov @nvazquez @DaanHoogland @wido @rafaelweingartner @marcaurele and others,

[GitHub] rhtyd commented on issue #2350: Cloudstack 10170 - fixes resource tags security bugs and adds account tags support

2017-12-25 Thread GitBox
rhtyd commented on issue #2350: Cloudstack 10170 - fixes resource tags security bugs and adds account tags support URL: https://github.com/apache/cloudstack/pull/2350#issuecomment-353926643 @bwsw can you have a look at build/Travis failures?

[GitHub] blueorangutan commented on issue #2360: Adding nuage VSD managed network support to CloudStack.

2017-12-25 Thread GitBox
blueorangutan commented on issue #2360: Adding nuage VSD managed network support to CloudStack. URL: https://github.com/apache/cloudstack/pull/2360#issuecomment-353926577 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2360: Adding nuage VSD managed network support to CloudStack.

2017-12-25 Thread GitBox
rhtyd commented on issue #2360: Adding nuage VSD managed network support to CloudStack. URL: https://github.com/apache/cloudstack/pull/2360#issuecomment-353926538 @blueorangutan package This is an automated message from the

[GitHub] blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-12-25 Thread GitBox
blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-353926466 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-12-25 Thread GitBox
rhtyd commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-353926402 @blueorangutan package This is an automated message from the Apache Git Service. To respond

[GitHub] blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters

2017-12-25 Thread GitBox
blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-353773850 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters

2017-12-25 Thread GitBox
blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-353796536 Trillian test result (tid-1903) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] rhtyd commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters

2017-12-25 Thread GitBox
rhtyd commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-353774923 @blueorangutan test This is an automated

[GitHub] blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-12-25 Thread GitBox
blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-353926041 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume.

2017-12-25 Thread GitBox
rhtyd commented on issue #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume. URL: https://github.com/apache/cloudstack/pull/2373#issuecomment-353926005 @blueorangutan package This

[GitHub] blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters

2017-12-25 Thread GitBox
blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-353774854 Packaging result: ?centos6 ?centos7 ?debian. JID-1480

[GitHub] rhtyd commented on issue #1554: CLOUDSTACK-9602: API: improve resource limits comprehension

2017-12-25 Thread GitBox
rhtyd commented on issue #1554: CLOUDSTACK-9602: API: improve resource limits comprehension URL: https://github.com/apache/cloudstack/pull/1554#issuecomment-353926181 Tests LGTM, can we have a more code review/lgtm to merge this -- @borisstoyanov @wido @nvazquez @rafaelweingartner

[GitHub] blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters

2017-12-25 Thread GitBox
blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-353774954 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] blueorangutan commented on issue #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume.

2017-12-25 Thread GitBox
blueorangutan commented on issue #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume. URL: https://github.com/apache/cloudstack/pull/2373#issuecomment-353926037 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I

[GitHub] rhtyd commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-12-25 Thread GitBox
rhtyd commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-353925980 @blueorangutan package This is an

[GitHub] rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua?

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua? URL: https://github.com/apache/cloudstack/pull/2292#discussion_r158672220 ## File path:

[GitHub] rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua?

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua? URL: https://github.com/apache/cloudstack/pull/2292#discussion_r158671979 ## File path:

[GitHub] rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua?

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua? URL: https://github.com/apache/cloudstack/pull/2292#discussion_r158672111 ## File path: engine/orchestration/src/com/cloud/agent/manager/AgentManagerImpl.java

[GitHub] rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua?

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua? URL: https://github.com/apache/cloudstack/pull/2292#discussion_r158672147 ## File path:

[GitHub] rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua?

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua? URL: https://github.com/apache/cloudstack/pull/2292#discussion_r158671990 ## File path: engine/schema/src/com/cloud/host/dao/HostDaoImpl.java ## @@

[GitHub] rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua?

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua? URL: https://github.com/apache/cloudstack/pull/2292#discussion_r158672100 ## File path: engine/orchestration/src/com/cloud/agent/manager/AgentManagerImpl.java

[GitHub] rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua?

2017-12-25 Thread GitBox
rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua? URL: https://github.com/apache/cloudstack/pull/2292#discussion_r158672304 ## File path: engine/schema/src/com/cloud/host/dao/HostDaoImpl.java ## @@

[cloudstack] branch master updated: CLOUDSTACK-9456: Upgrade spring and misc dependencies (#2375)

2017-12-25 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new e0d5364 CLOUDSTACK-9456: Upgrade spring and

[GitHub] rhtyd commented on issue #2375: CLOUDSTACK-9456: Upgrade spring, jetty and misc dependencies

2017-12-25 Thread GitBox
rhtyd commented on issue #2375: CLOUDSTACK-9456: Upgrade spring, jetty and misc dependencies URL: https://github.com/apache/cloudstack/pull/2375#issuecomment-353924879 Merging this based on two lgtms and test results with 0 smoketest failures. I'll look into intermittent failures and

[GitHub] rhtyd closed pull request #2375: CLOUDSTACK-9456: Upgrade spring, jetty and misc dependencies

2017-12-25 Thread GitBox
rhtyd closed pull request #2375: CLOUDSTACK-9456: Upgrade spring, jetty and misc dependencies URL: https://github.com/apache/cloudstack/pull/2375 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] bwsw commented on issue #2350: Cloudstack 10170 - fixes resource tags security bugs and adds account tags support

2017-12-25 Thread GitBox
bwsw commented on issue #2350: Cloudstack 10170 - fixes resource tags security bugs and adds account tags support URL: https://github.com/apache/cloudstack/pull/2350#issuecomment-353922687 @borisstoyanov conflicts are resolved.

[GitHub] mrunalinikankariya commented on issue #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua?

2017-12-25 Thread GitBox
mrunalinikankariya commented on issue #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua? URL: https://github.com/apache/cloudstack/pull/2292#issuecomment-353919345 @rhtyd , can you please review the changes

[GitHub] blueorangutan commented on issue #1554: CLOUDSTACK-9602: API: improve resource limits comprehension

2017-12-25 Thread GitBox
blueorangutan commented on issue #1554: CLOUDSTACK-9602: API: improve resource limits comprehension URL: https://github.com/apache/cloudstack/pull/1554#issuecomment-353894268 Trillian test result (tid-1913) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total

[GitHub] blueorangutan commented on issue #2375: CLOUDSTACK-9456: Upgrade spring, jetty and misc dependencies

2017-12-25 Thread GitBox
blueorangutan commented on issue #2375: CLOUDSTACK-9456: Upgrade spring, jetty and misc dependencies URL: https://github.com/apache/cloudstack/pull/2375#issuecomment-353887694 Trillian test result (tid-1915) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total

[GitHub] blueorangutan commented on issue #1554: CLOUDSTACK-9602: API: improve resource limits comprehension

2017-12-25 Thread GitBox
blueorangutan commented on issue #1554: CLOUDSTACK-9602: API: improve resource limits comprehension URL: https://github.com/apache/cloudstack/pull/1554#issuecomment-353865787 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters

2017-12-25 Thread GitBox
blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-353865794 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] blueorangutan commented on issue #2375: CLOUDSTACK-9456: Upgrade spring, jetty and misc dependencies

2017-12-25 Thread GitBox
blueorangutan commented on issue #2375: CLOUDSTACK-9456: Upgrade spring, jetty and misc dependencies URL: https://github.com/apache/cloudstack/pull/2375#issuecomment-353865791 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #1554: CLOUDSTACK-9602: API: improve resource limits comprehension

2017-12-25 Thread GitBox
rhtyd commented on issue #1554: CLOUDSTACK-9602: API: improve resource limits comprehension URL: https://github.com/apache/cloudstack/pull/1554#issuecomment-353865756 @blueorangutan test This is an automated message from the

[GitHub] rhtyd commented on issue #2375: CLOUDSTACK-9456: Upgrade spring, jetty and misc dependencies

2017-12-25 Thread GitBox
rhtyd commented on issue #2375: CLOUDSTACK-9456: Upgrade spring, jetty and misc dependencies URL: https://github.com/apache/cloudstack/pull/2375#issuecomment-353865751 @blueorangutan test This is an automated message from

[GitHub] rhtyd commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters

2017-12-25 Thread GitBox
rhtyd commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-353865746 @blueorangutan test This is an automated

[GitHub] blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-12-25 Thread GitBox
blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-353860041 Packaging result: ?centos6 ?centos7 ?debian. JID-1487 This is an automated message

[GitHub] blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters

2017-12-25 Thread GitBox
blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-353859891 Packaging result: ?centos6 ?centos7 ?debian. JID-1486

[GitHub] blueorangutan commented on issue #2376: [4.11/master] Smoketest Health Check

2017-12-25 Thread GitBox
blueorangutan commented on issue #2376: [4.11/master] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-353859435 Packaging result: ?centos6 ?centos7 ?debian. JID-1485 This is an

[GitHub] rafaelweingartner commented on issue #1740: CLOUDSTACK-9572 Snapshot on primary storage not cleaned up after Stor?

2017-12-25 Thread GitBox
rafaelweingartner commented on issue #1740: CLOUDSTACK-9572 Snapshot on primary storage not cleaned up after Stor? URL: https://github.com/apache/cloudstack/pull/1740#issuecomment-353858935 @yvsubhash you are right... My problem is not related to this one here. The snapshots in XenServer

[GitHub] rafaelweingartner commented on issue #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume.

2017-12-25 Thread GitBox
rafaelweingartner commented on issue #2373: CLOUDSTACK-10202:createSnapshotPolicy API create multiple entries in DB for same volume. URL: https://github.com/apache/cloudstack/pull/2373#issuecomment-353858705 I have experienced the same problem some time ago. I did not manage to reproduce

[GitHub] blueorangutan commented on issue #1554: CLOUDSTACK-9602: API: improve resource limits comprehension

2017-12-25 Thread GitBox
blueorangutan commented on issue #1554: CLOUDSTACK-9602: API: improve resource limits comprehension URL: https://github.com/apache/cloudstack/pull/1554#issuecomment-353858033 Packaging result: ?centos6 ?centos7 ?debian. JID-1484

[GitHub] blueorangutan commented on issue #2375: CLOUDSTACK-9456: Upgrade spring, jetty and misc dependencies

2017-12-25 Thread GitBox
blueorangutan commented on issue #2375: CLOUDSTACK-9456: Upgrade spring, jetty and misc dependencies URL: https://github.com/apache/cloudstack/pull/2375#issuecomment-353857764 Packaging result: ?centos6 ?centos7 ?debian. JID-1482

[GitHub] blueorangutan commented on issue #1934: [CLOUDSTACK-9772] Template: perform a HEAD request to check file size from a URL

2017-12-25 Thread GitBox
blueorangutan commented on issue #1934: [CLOUDSTACK-9772] Template: perform a HEAD request to check file size from a URL URL: https://github.com/apache/cloudstack/pull/1934#issuecomment-353857763 Packaging result: ?centos6 ?centos7 ?debian. JID-1483

[GitHub] bwsw commented on issue #2350: Cloudstack 10170 - fixes resource tags security bugs and adds account tags support

2017-12-25 Thread GitBox
bwsw commented on issue #2350: Cloudstack 10170 - fixes resource tags security bugs and adds account tags support URL: https://github.com/apache/cloudstack/pull/2350#issuecomment-353856183 @borisstoyanov could you please direct me what is the most supposable test to look at. I tried

[GitHub] blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-12-25 Thread GitBox
blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-353855395 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-12-25 Thread GitBox
rhtyd commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-353855314 Thanks @borisstoyanov let me re-run tests as master has changed. @blueorangutan package

[GitHub] blueorangutan commented on issue #2376: [4.11/master] Smoketest Health Check

2017-12-25 Thread GitBox
blueorangutan commented on issue #2376: [4.11/master] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-353791499 Trillian test result (tid-1899) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 28753 seconds

  1   2   >