[GitHub] mike-tutkowski commented on issue #2503: Support multiple volume access groups per compute cluster

2018-04-30 Thread GitBox
mike-tutkowski commented on issue #2503: Support multiple volume access groups per compute cluster URL: https://github.com/apache/cloudstack/pull/2503#issuecomment-385595671 Sounds good. I was under the impression, though, that we always wanted to squash commits going into a PR, but I

[GitHub] mike-tutkowski commented on issue #2501: Only perform certain actions with managed storage if the VM is s…

2018-04-30 Thread GitBox
mike-tutkowski commented on issue #2501: Only perform certain actions with managed storage if the VM is s… URL: https://github.com/apache/cloudstack/pull/2501#issuecomment-385595549 I’m not sure I understand the recommendation. Could you specify in more detail? Thanks! On Apr 30,

[GitHub] DaanHoogland commented on a change in pull request #2503: Support multiple volume access groups per compute cluster

2018-04-30 Thread GitBox
DaanHoogland commented on a change in pull request #2503: Support multiple volume access groups per compute cluster URL: https://github.com/apache/cloudstack/pull/2503#discussion_r185163509 ## File path:

[GitHub] nitin-maharana commented on a change in pull request #2501: Only perform certain actions with managed storage if the VM is s…

2018-04-30 Thread GitBox
nitin-maharana commented on a change in pull request #2501: Only perform certain actions with managed storage if the VM is s… URL: https://github.com/apache/cloudstack/pull/2501#discussion_r185162940 ## File path:

[GitHub] mike-tutkowski commented on issue #2501: Only perform certain actions with managed storage if the VM is s…

2018-04-30 Thread GitBox
mike-tutkowski commented on issue #2501: Only perform certain actions with managed storage if the VM is s… URL: https://github.com/apache/cloudstack/pull/2501#issuecomment-385585571 I doubt this Jenkins error has anything to do with this PR:

[GitHub] mike-tutkowski commented on issue #2503: Support multiple volume access groups per compute cluster

2018-04-30 Thread GitBox
mike-tutkowski commented on issue #2503: Support multiple volume access groups per compute cluster URL: https://github.com/apache/cloudstack/pull/2503#issuecomment-385585396 I doubt this Jenkins error has anything to do with this PR:

[GitHub] mike-tutkowski commented on issue #2502: [CLOUDSTACK-10352] XenServer: Support online migration of a virtual disk from non-managed to managed storage

2018-04-30 Thread GitBox
mike-tutkowski commented on issue #2502: [CLOUDSTACK-10352] XenServer: Support online migration of a virtual disk from non-managed to managed storage URL: https://github.com/apache/cloudstack/pull/2502#issuecomment-385585217 I doubt this Jenkins error has anything to do this with PR:

[GitHub] blueorangutan commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart

2018-04-30 Thread GitBox
blueorangutan commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart URL: https://github.com/apache/cloudstack/pull/2508#issuecomment-385577308 Trillian test result (tid-2553) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total

[GitHub] blueorangutan commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource

2018-04-30 Thread GitBox
blueorangutan commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource URL: https://github.com/apache/cloudstack/pull/1499#issuecomment-385574373 Trillian test result (tid-2558) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total

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

2018-04-30 Thread GitBox
blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-385570724 Trillian test result (tid-2552) Environment: vmware-65 (x2), Advanced Networking with Mgmt server 7 Total time taken: 57211 seconds

[GitHub] rafaelweingartner commented on a change in pull request #2503: Support multiple volume access groups per compute cluster

2018-04-30 Thread GitBox
rafaelweingartner commented on a change in pull request #2503: Support multiple volume access groups per compute cluster URL: https://github.com/apache/cloudstack/pull/2503#discussion_r185146809 ## File path:

[GitHub] rafaelweingartner commented on a change in pull request #2503: Support multiple volume access groups per compute cluster

2018-04-30 Thread GitBox
rafaelweingartner commented on a change in pull request #2503: Support multiple volume access groups per compute cluster URL: https://github.com/apache/cloudstack/pull/2503#discussion_r185146809 ## File path:

[GitHub] mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster

2018-04-30 Thread GitBox
mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster URL: https://github.com/apache/cloudstack/pull/2503#discussion_r185142537 ## File path:

[GitHub] rafaelweingartner commented on a change in pull request #2503: Support multiple volume access groups per compute cluster

2018-04-30 Thread GitBox
rafaelweingartner commented on a change in pull request #2503: Support multiple volume access groups per compute cluster URL: https://github.com/apache/cloudstack/pull/2503#discussion_r185140869 ## File path:

[GitHub] mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster

2018-04-30 Thread GitBox
mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster URL: https://github.com/apache/cloudstack/pull/2503#discussion_r185131384 ## File path:

[GitHub] mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster

2018-04-30 Thread GitBox
mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster URL: https://github.com/apache/cloudstack/pull/2503#discussion_r185129323 ## File path:

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

2018-04-30 Thread GitBox
blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-385547575 Trillian test result (tid-2550) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 6 Total time taken: 48598 seconds

[GitHub] mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster

2018-04-30 Thread GitBox
mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster URL: https://github.com/apache/cloudstack/pull/2503#discussion_r185127187 ## File path:

[GitHub] mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster

2018-04-30 Thread GitBox
mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster URL: https://github.com/apache/cloudstack/pull/2503#discussion_r185126158 ## File path:

[GitHub] mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster

2018-04-30 Thread GitBox
mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster URL: https://github.com/apache/cloudstack/pull/2503#discussion_r185125933 ## File path:

[GitHub] mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster

2018-04-30 Thread GitBox
mike-tutkowski commented on a change in pull request #2503: Support multiple volume access groups per compute cluster URL: https://github.com/apache/cloudstack/pull/2503#discussion_r185125254 ## File path:

[GitHub] mike-tutkowski commented on issue #2502: [CLOUDSTACK-10352] XenServer: Support online migration of a virtual disk from non-managed to managed storage

2018-04-30 Thread GitBox
mike-tutkowski commented on issue #2502: [CLOUDSTACK-10352] XenServer: Support online migration of a virtual disk from non-managed to managed storage URL: https://github.com/apache/cloudstack/pull/2502#issuecomment-385543312 Whoops, I meant @borisstoyanov had approved it. :)

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

2018-04-30 Thread GitBox
blueorangutan commented on issue #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID. URL: https://github.com/apache/cloudstack/pull/1940#issuecomment-385510218 Trillian test result (tid-2554) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] resmo commented on issue #2525: details not in listVirtualMachines response

2018-04-30 Thread GitBox
resmo commented on issue #2525: details not in listVirtualMachines response URL: https://github.com/apache/cloudstack/issues/2525#issuecomment-385455181 I used root This is an automated message from the Apache Git Service.

[GitHub] khos2ow commented on a change in pull request #2613: Cleanup POMs

2018-04-30 Thread GitBox
khos2ow commented on a change in pull request #2613: Cleanup POMs URL: https://github.com/apache/cloudstack/pull/2613#discussion_r185026193 ## File path: client/pom.xml ## @@ -1,1190 +1,1219 @@ - -http://maven.apache.org/POM/4.0.0; -

[GitHub] blueorangutan commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource

2018-04-30 Thread GitBox
blueorangutan commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource URL: https://github.com/apache/cloudstack/pull/1499#issuecomment-385435962 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] DaanHoogland commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource

2018-04-30 Thread GitBox
DaanHoogland commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource URL: https://github.com/apache/cloudstack/pull/1499#issuecomment-385435833 @blueorangutan test This is an automated message

[GitHub] rafaelweingartner commented on a change in pull request #2613: Cleanup POMs

2018-04-30 Thread GitBox
rafaelweingartner commented on a change in pull request #2613: Cleanup POMs URL: https://github.com/apache/cloudstack/pull/2613#discussion_r185019115 ## File path: client/pom.xml ## @@ -1,1190 +1,1219 @@ - -http://maven.apache.org/POM/4.0.0; -

[GitHub] khos2ow commented on a change in pull request #2613: Cleanup POMs

2018-04-30 Thread GitBox
khos2ow commented on a change in pull request #2613: Cleanup POMs URL: https://github.com/apache/cloudstack/pull/2613#discussion_r185014401 ## File path: client/pom.xml ## @@ -1,1190 +1,1219 @@ - -http://maven.apache.org/POM/4.0.0; -

[GitHub] rafaelweingartner commented on a change in pull request #2613: Cleanup POMs

2018-04-30 Thread GitBox
rafaelweingartner commented on a change in pull request #2613: Cleanup POMs URL: https://github.com/apache/cloudstack/pull/2613#discussion_r185009348 ## File path: client/pom.xml ## @@ -1,1190 +1,1219 @@ - -http://maven.apache.org/POM/4.0.0; -

[GitHub] khos2ow commented on a change in pull request #2613: Cleanup POMs

2018-04-30 Thread GitBox
khos2ow commented on a change in pull request #2613: Cleanup POMs URL: https://github.com/apache/cloudstack/pull/2613#discussion_r185007887 ## File path: client/pom.xml ## @@ -1,1190 +1,1219 @@ - -http://maven.apache.org/POM/4.0.0; -

[GitHub] blueorangutan commented on issue #2607: Allow changing disk offering of VMs' root volume during volume migration

2018-04-30 Thread GitBox
blueorangutan commented on issue #2607: Allow changing disk offering of VMs' root volume during volume migration URL: https://github.com/apache/cloudstack/pull/2607#issuecomment-385416949 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

[GitHub] DaanHoogland commented on issue #2607: Allow changing disk offering of VMs' root volume during volume migration

2018-04-30 Thread GitBox
DaanHoogland commented on issue #2607: Allow changing disk offering of VMs' root volume during volume migration URL: https://github.com/apache/cloudstack/pull/2607#issuecomment-385416701 @blueorangutan test This is an

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

2018-04-30 Thread GitBox
blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-385415566 Trillian test result (tid-2551) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 20363 seconds

[GitHub] blueorangutan commented on issue #2399: CLOUDSTACK-10223 delete snapshots when deleting domain

2018-04-30 Thread GitBox
blueorangutan commented on issue #2399: CLOUDSTACK-10223 delete snapshots when deleting domain URL: https://github.com/apache/cloudstack/pull/2399#issuecomment-385414697 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] DaanHoogland commented on issue #2399: CLOUDSTACK-10223 delete snapshots when deleting domain

2018-04-30 Thread GitBox
DaanHoogland commented on issue #2399: CLOUDSTACK-10223 delete snapshots when deleting domain URL: https://github.com/apache/cloudstack/pull/2399#issuecomment-385414441 @blueorangutan test This is an automated message from

[GitHub] rhtyd commented on issue #2530: KVM hosts fail to connect if there is more than one IP address on the host

2018-04-30 Thread GitBox
rhtyd commented on issue #2530: KVM hosts fail to connect if there is more than one IP address on the host URL: https://github.com/apache/cloudstack/issues/2530#issuecomment-385381597 I've implemented and tested a fix by which during CSR generation, we consider all the IPs available on

[GitHub] rhtyd opened a new pull request #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-04-30 Thread GitBox
rhtyd opened a new pull request #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616 ## Description This includes several (smoke) test, stabilization fixes and blocker fixes towards 4.11.1.0 milestone. Please see individually

[GitHub] blueorangutan commented on issue #2615: WIP: Supporting ConfigDrive user data on L2 networks

2018-04-30 Thread GitBox
blueorangutan commented on issue #2615: WIP: Supporting ConfigDrive user data on L2 networks URL: https://github.com/apache/cloudstack/pull/2615#issuecomment-385376892 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1983

[GitHub] nitin-maharana commented on a change in pull request #2612: [migrateVolume API method] Filter disk offerings based on target storage pool selected

2018-04-30 Thread GitBox
nitin-maharana commented on a change in pull request #2612: [migrateVolume API method] Filter disk offerings based on target storage pool selected URL: https://github.com/apache/cloudstack/pull/2612#discussion_r184963277 ## File path: ui/scripts/sharedFunctions.js ## @@

[GitHub] blueorangutan commented on issue #2615: WIP: Supporting ConfigDrive user data on L2 networks

2018-04-30 Thread GitBox
blueorangutan commented on issue #2615: WIP: Supporting ConfigDrive user data on L2 networks URL: https://github.com/apache/cloudstack/pull/2615#issuecomment-385371594 @PaulAngus a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] PaulAngus commented on issue #2615: WIP: Supporting ConfigDrive user data on L2 networks

2018-04-30 Thread GitBox
PaulAngus commented on issue #2615: WIP: Supporting ConfigDrive user data on L2 networks URL: https://github.com/apache/cloudstack/pull/2615#issuecomment-385371462 @blueorangutan package This is an automated message from the

[GitHub] rafaelweingartner commented on a change in pull request #2612: [migrateVolume API method] Filter disk offerings based on target storage pool selected

2018-04-30 Thread GitBox
rafaelweingartner commented on a change in pull request #2612: [migrateVolume API method] Filter disk offerings based on target storage pool selected URL: https://github.com/apache/cloudstack/pull/2612#discussion_r184960135 ## File path: ui/scripts/sharedFunctions.js ##

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

2018-04-30 Thread GitBox
blueorangutan commented on issue #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID. URL: https://github.com/apache/cloudstack/pull/1940#issuecomment-385368281 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

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

2018-04-30 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-385368045 @blueorangutan test This is an automated message

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

2018-04-30 Thread GitBox
blueorangutan commented on issue #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID. URL: https://github.com/apache/cloudstack/pull/1940#issuecomment-385367618 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1982

[GitHub] nvazquez opened a new pull request #2615: WIP: Supporting ConfigDrive user data on L2 networks

2018-04-30 Thread GitBox
nvazquez opened a new pull request #2615: WIP: Supporting ConfigDrive user data on L2 networks URL: https://github.com/apache/cloudstack/pull/2615 ## Description Work in progress PR, to provide the ability to create L2 network providing ConfigDrive service. ## Types of changes

[GitHub] nitin-maharana commented on a change in pull request #2612: [migrateVolume API method] Filter disk offerings based on target storage pool selected

2018-04-30 Thread GitBox
nitin-maharana commented on a change in pull request #2612: [migrateVolume API method] Filter disk offerings based on target storage pool selected URL: https://github.com/apache/cloudstack/pull/2612#discussion_r184958003 ## File path: ui/scripts/sharedFunctions.js ## @@

[GitHub] nitin-maharana commented on a change in pull request #2612: [migrateVolume API method] Filter disk offerings based on target storage pool selected

2018-04-30 Thread GitBox
nitin-maharana commented on a change in pull request #2612: [migrateVolume API method] Filter disk offerings based on target storage pool selected URL: https://github.com/apache/cloudstack/pull/2612#discussion_r184956662 ## File path: ui/scripts/sharedFunctions.js ## @@

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

2018-04-30 Thread GitBox
blueorangutan commented on issue #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID. URL: https://github.com/apache/cloudstack/pull/1940#issuecomment-385362623 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

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

2018-04-30 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-385362461 @blueorangutan package This is an automated message

[GitHub] rafaelweingartner commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource

2018-04-30 Thread GitBox
rafaelweingartner commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource URL: https://github.com/apache/cloudstack/pull/1499#issuecomment-385360689 I just built the centos7 packages here (using this PR). I am using the packaging scripts found in

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

2018-04-30 Thread GitBox
rafaelweingartner commented on issue #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID. URL: https://github.com/apache/cloudstack/pull/1940#issuecomment-385360526 In my opinion we can proceed with the merge. @syed inquiries were already addressed.

[GitHub] blueorangutan commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart

2018-04-30 Thread GitBox
blueorangutan commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart URL: https://github.com/apache/cloudstack/pull/2508#issuecomment-385358428 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart

2018-04-30 Thread GitBox
rhtyd commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart URL: https://github.com/apache/cloudstack/pull/2508#issuecomment-385358311 @blueorangutan test This is an automated message from the

[GitHub] rhtyd commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart

2018-04-30 Thread GitBox
rhtyd commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart URL: https://github.com/apache/cloudstack/pull/2508#issuecomment-385358259 @ustcweizhou @resmo @DaanHoogland @rafaelweingartner @wido @nitin-maharana @nvazquez @NuxRo this is ready for review,

[GitHub] DaanHoogland opened a new pull request #2614: WIP perform config drive creation on primary storage (CLOUDSTACK-10290)

2018-04-30 Thread GitBox
DaanHoogland opened a new pull request #2614: WIP perform config drive creation on primary storage (CLOUDSTACK-10290) URL: https://github.com/apache/cloudstack/pull/2614 ## Description The present configdrive generation happens on secondary storage, which is also used in public facing

[GitHub] rafaelweingartner commented on a change in pull request #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart

2018-04-30 Thread GitBox
rafaelweingartner commented on a change in pull request #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart URL: https://github.com/apache/cloudstack/pull/2508#discussion_r184948864 ## File path: server/src/com/cloud/network/vpc/VpcManagerImpl.java ## @@

[GitHub] rafaelweingartner commented on a change in pull request #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart

2018-04-30 Thread GitBox
rafaelweingartner commented on a change in pull request #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart URL: https://github.com/apache/cloudstack/pull/2508#discussion_r184948864 ## File path: server/src/com/cloud/network/vpc/VpcManagerImpl.java ## @@

[GitHub] DaanHoogland closed pull request #2083: Iptables speedup

2018-04-30 Thread GitBox
DaanHoogland closed pull request #2083: Iptables speedup URL: https://github.com/apache/cloudstack/pull/2083 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: As this is a foreign pull request (from

[GitHub] blueorangutan commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart

2018-04-30 Thread GitBox
blueorangutan commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart URL: https://github.com/apache/cloudstack/pull/2508#issuecomment-385353120 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1981

[GitHub] blueorangutan commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart

2018-04-30 Thread GitBox
blueorangutan commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart URL: https://github.com/apache/cloudstack/pull/2508#issuecomment-385347107 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart

2018-04-30 Thread GitBox
rhtyd commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart URL: https://github.com/apache/cloudstack/pull/2508#issuecomment-385347017 @blueorangutan package This is an automated message from

[GitHub] rhtyd commented on a change in pull request #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart

2018-04-30 Thread GitBox
rhtyd commented on a change in pull request #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart URL: https://github.com/apache/cloudstack/pull/2508#discussion_r184940056 ## File path: systemvm/debian/opt/cloud/bin/configure.py ## @@ -957,6 +957,7 @@ def

[GitHub] rhtyd commented on a change in pull request #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart

2018-04-30 Thread GitBox
rhtyd commented on a change in pull request #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart URL: https://github.com/apache/cloudstack/pull/2508#discussion_r184939883 ## File path: server/src/com/cloud/network/vpc/VpcManagerImpl.java ## @@ -1508,19

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

2018-04-30 Thread GitBox
blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-385337262 @rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2376: [4.11] Smoketest Health Check

2018-04-30 Thread GitBox
rhtyd commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-385337014 @blueorangutan test matrix This is an automated message from the Apache Git Service. To

[GitHub] blueorangutan commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource

2018-04-30 Thread GitBox
blueorangutan commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource URL: https://github.com/apache/cloudstack/pull/1499#issuecomment-385332068 Packaging result: ✔centos6 ✖centos7 ✔debian. JID-1980

[GitHub] marcaurele commented on a change in pull request #2613: Cleanup POMs

2018-04-30 Thread GitBox
marcaurele commented on a change in pull request #2613: Cleanup POMs URL: https://github.com/apache/cloudstack/pull/2613#discussion_r184925274 ## File path: client/pom.xml ## @@ -1,1190 +1,1219 @@ - -http://maven.apache.org/POM/4.0.0; -

[GitHub] marcaurele commented on a change in pull request #2613: Cleanup POMs

2018-04-30 Thread GitBox
marcaurele commented on a change in pull request #2613: Cleanup POMs URL: https://github.com/apache/cloudstack/pull/2613#discussion_r184925812 ## File path: services/console-proxy/rdpconsole/pom.xml ## @@ -0,0 +1,53 @@ + +http://maven.apache.org/POM/4.0.0;

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

2018-04-30 Thread GitBox
blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-385330028 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1979 This is an automated

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

2018-04-30 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-385328743 It's also a good idea to start a conversation on dev/user ML about this change. This PR has been blocked for months

[GitHub] blueorangutan commented on issue #2608: API: move ostypeid from DB id to DB uuid

2018-04-30 Thread GitBox
blueorangutan commented on issue #2608: API: move ostypeid from DB id to DB uuid URL: https://github.com/apache/cloudstack/pull/2608#issuecomment-385328432 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] DaanHoogland commented on issue #2608: API: move ostypeid from DB id to DB uuid

2018-04-30 Thread GitBox
DaanHoogland commented on issue #2608: API: move ostypeid from DB id to DB uuid URL: https://github.com/apache/cloudstack/pull/2608#issuecomment-385328221 @blueorangutan test This is an automated message from the Apache Git

[GitHub] rhtyd closed pull request #2598: CLOUDSTACK-10360: Change the method name.

2018-04-30 Thread GitBox
rhtyd closed pull request #2598: CLOUDSTACK-10360: Change the method name. URL: https://github.com/apache/cloudstack/pull/2598 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: As this is a foreign

[GitHub] rhtyd commented on issue #2598: CLOUDSTACK-10360: Change the method name.

2018-04-30 Thread GitBox
rhtyd commented on issue #2598: CLOUDSTACK-10360: Change the method name. URL: https://github.com/apache/cloudstack/pull/2598#issuecomment-385327981 We can merge this since it's a cosmetic fix - name change refactoring, no effective code change. I'll merge this based on code reviews and

[cloudstack] branch master updated: CLOUDSTACK-10360: Change the method name. (#2598)

2018-04-30 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 93509a4 CLOUDSTACK-10360: Change the method

[GitHub] DaanHoogland commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource

2018-04-30 Thread GitBox
DaanHoogland commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource URL: https://github.com/apache/cloudstack/pull/1499#issuecomment-385327748 just found that the success reports for centos ackaging where false :( @blueorangutan package

[GitHub] blueorangutan commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource

2018-04-30 Thread GitBox
blueorangutan commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource URL: https://github.com/apache/cloudstack/pull/1499#issuecomment-385327810 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2091: CLOUDSTACK-8609: [VMware] VM is not accessible after migration across clusters

2018-04-30 Thread GitBox
rhtyd commented on issue #2091: CLOUDSTACK-8609: [VMware] VM is not accessible after migration across clusters URL: https://github.com/apache/cloudstack/pull/2091#issuecomment-385327709 @resmo I'll inform that PR author, ultimately it will depend on how responsive the PR author is and if

[GitHub] blueorangutan commented on issue #2607: Allow changing disk offering of VMs' root volume during volume migration

2018-04-30 Thread GitBox
blueorangutan commented on issue #2607: Allow changing disk offering of VMs' root volume during volume migration URL: https://github.com/apache/cloudstack/pull/2607#issuecomment-385327524 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

[GitHub] DaanHoogland commented on issue #2607: Allow changing disk offering of VMs' root volume during volume migration

2018-04-30 Thread GitBox
DaanHoogland commented on issue #2607: Allow changing disk offering of VMs' root volume during volume migration URL: https://github.com/apache/cloudstack/pull/2607#issuecomment-385327362 not enough public networks available for the blue primate :( failed @blueorangutan test

[GitHub] marcaurele commented on issue #2609: Using the simulator to run tests

2018-04-30 Thread GitBox
marcaurele commented on issue #2609: Using the simulator to run tests URL: https://github.com/apache/cloudstack/issues/2609#issuecomment-385326035 Thanks @rhtyd , I'll update the documentation/pom with the correct command too

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

2018-04-30 Thread GitBox
blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-385325545 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2376: [4.11] Smoketest Health Check

2018-04-30 Thread GitBox
rhtyd commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-385325504 @blueorangutan package This is an automated message from the Apache Git Service. To

[GitHub] blueorangutan commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource

2018-04-30 Thread GitBox
blueorangutan commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource URL: https://github.com/apache/cloudstack/pull/1499#issuecomment-385325298 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] DaanHoogland commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource

2018-04-30 Thread GitBox
DaanHoogland commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource URL: https://github.com/apache/cloudstack/pull/1499#issuecomment-385325139 @blueorangutan test This is an automated message

[GitHub] DaanHoogland commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource

2018-04-30 Thread GitBox
DaanHoogland commented on issue #1499: Remove unused "umount" method in NfsSecondaryStorageResource URL: https://github.com/apache/cloudstack/pull/1499#issuecomment-385324667 @rafaelweingartner no This is an automated

[GitHub] rhtyd commented on issue #2581: Latest Docker container is broken

2018-04-30 Thread GitBox
rhtyd commented on issue #2581: Latest Docker container is broken URL: https://github.com/apache/cloudstack/issues/2581#issuecomment-385324561 @pdion891 @swill any ideas who maintains the docker image? Can you help? This is

[GitHub] DaanHoogland commented on issue #2605: xenserver: Add support for XS 7.3, 7.4 and XCP-ng 7.4

2018-04-30 Thread GitBox
DaanHoogland commented on issue #2605: xenserver: Add support for XS 7.3, 7.4 and XCP-ng 7.4 URL: https://github.com/apache/cloudstack/pull/2605#issuecomment-385324361 @rhtyd thank for the failure analysis. no issues there

[GitHub] rhtyd commented on issue #2575: HTML shown to end-user in UI

2018-04-30 Thread GitBox
rhtyd commented on issue #2575: HTML shown to end-user in UI URL: https://github.com/apache/cloudstack/issues/2575#issuecomment-385324022 @jorgesumle Supported browsers for current UI are Chrome/Chromium and Firefox. We usually don't test against other browsers. However, I was able to

[GitHub] rhtyd commented on issue #2575: HTML shown to end-user in UI

2018-04-30 Thread GitBox
rhtyd commented on issue #2575: HTML shown to end-user in UI URL: https://github.com/apache/cloudstack/issues/2575#issuecomment-385324022 @jorgesumle Supported browsers for current UI are Chrome/Chromium and Firefox. We usually don't test against other browsers. However, I was able to see

[GitHub] rhtyd commented on issue #2525: details not in listVirtualMachines response

2018-04-30 Thread GitBox
rhtyd commented on issue #2525: details not in listVirtualMachines response URL: https://github.com/apache/cloudstack/issues/2525#issuecomment-385323907 @resmo did you try that as a normal user or root/domain admin? This is

[GitHub] rhtyd commented on issue #2565: Web UI is not responsive

2018-04-30 Thread GitBox
rhtyd commented on issue #2565: Web UI is not responsive URL: https://github.com/apache/cloudstack/issues/2565#issuecomment-385323685 The UI assumes 1920x1080 (modern screens). The entire UI include CSS and widgets would need to be re-written in order to support responsiveness, which

[GitHub] rhtyd commented on issue #2565: Web UI is not responsive

2018-04-30 Thread GitBox
rhtyd commented on issue #2565: Web UI is not responsive URL: https://github.com/apache/cloudstack/issues/2565#issuecomment-385323685 The UI assumes 1920x1080 (modern screens). The entire UI include CSS and widgets would need to be re-written in order to support responsiveness, which

[GitHub] rhtyd closed issue #2565: Web UI is not responsive

2018-04-30 Thread GitBox
rhtyd closed issue #2565: Web UI is not responsive URL: https://github.com/apache/cloudstack/issues/2565 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the

[GitHub] rhtyd commented on issue #2609: Using the simulator to run tests

2018-04-30 Thread GitBox
rhtyd commented on issue #2609: Using the simulator to run tests URL: https://github.com/apache/cloudstack/issues/2609#issuecomment-385323336 @marcaurele see Travis related scripts, Travis uses simulator to run test (the wiki/docs may be old and needs to be updated):

[GitHub] asfgit closed issue #2523: XCP-ng 7.4 support

2018-04-30 Thread GitBox
asfgit closed issue #2523: XCP-ng 7.4 support URL: https://github.com/apache/cloudstack/issues/2523 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL

[cloudstack] branch master updated (586edec -> 4277b92)

2018-04-30 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 586edec CLOUDSTACK-10253: JSON response for SuccessResponse as boolean instead of string (#2428) add

[cloudstack] branch 4.11 updated: xenserver: Add support for XS 7.3, 7.4 and XCP-ng 7.4 (#2605)

2018-04-30 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch 4.11 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.11 by this push: new 4645512 xenserver: Add support for XS 7.3, 7.4

  1   2   >