[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386497971 Trillian test result (tid-2592) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 6 Total time taken:

[GitHub] blueorangutan commented on issue #2473: CLOUDSTACK-10309 Add option on if to VM HA power-on a OOB-shut-off-VM

2018-05-03 Thread GitBox
blueorangutan commented on issue #2473: CLOUDSTACK-10309 Add option on if to VM HA power-on a OOB-shut-off-VM URL: https://github.com/apache/cloudstack/pull/2473#issuecomment-386201594 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

[GitHub] borisstoyanov commented on issue #2473: CLOUDSTACK-10309 Add option on if to VM HA power-on a OOB-shut-off-VM

2018-05-03 Thread GitBox
borisstoyanov commented on issue #2473: CLOUDSTACK-10309 Add option on if to VM HA power-on a OOB-shut-off-VM URL: https://github.com/apache/cloudstack/pull/2473#issuecomment-386201460 @blueorangutan test This is an

[GitHub] rhtyd commented on a change in pull request #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
rhtyd commented on a change in pull request #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#discussion_r185724447 ## File path: systemvm/debian/opt/cloud/bin/setup/cloud-early-config ## @@ -71,16 +71,16 @@

[GitHub] rhtyd commented on a change in pull request #2465: CLOUDSTACK-10232: SystemVMs and VR to run as HVM on XenServer

2018-05-03 Thread GitBox
rhtyd commented on a change in pull request #2465: CLOUDSTACK-10232: SystemVMs and VR to run as HVM on XenServer URL: https://github.com/apache/cloudstack/pull/2465#discussion_r185723416 ## File path: systemvm/debian/opt/cloud/bin/setup/cloud-early-config ## @@ -64,10

[GitHub] rhtyd commented on a change in pull request #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
rhtyd commented on a change in pull request #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#discussion_r185724447 ## File path: systemvm/debian/opt/cloud/bin/setup/cloud-early-config ## @@ -71,16 +71,16 @@

[GitHub] rhtyd commented on a change in pull request #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
rhtyd commented on a change in pull request #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#discussion_r185726059 ## File path: tools/appliance/systemvmtemplate/scripts/configure_systemvm_services.sh ## @@ -31,6

[GitHub] rhtyd commented on a change in pull request #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
rhtyd commented on a change in pull request #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#discussion_r185726831 ## File path: systemvm/debian/opt/cloud/bin/setup/patchsystemvm.sh ## @@ -37,6 +37,8 @@

[GitHub] rhtyd commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
rhtyd commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-385981744 @blueorangutan package This is an automated message from the Apache

[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-385990102 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1996 This

[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386075457 Trillian test result (tid-2577) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time taken:

[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386087674 Trillian test result (tid-2578) Environment: xcpng74 (x2), Advanced Networking with Mgmt server 7 Total time taken:

[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386225886 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
rhtyd commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386225794 @blueorangutan package This is an automated message from the Apache

[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386232445 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1997 This

[GitHub] rhtyd commented on issue #2619: Remove "self-injection" of AccountManagerImpl

2018-05-03 Thread GitBox
rhtyd commented on issue #2619: Remove "self-injection" of AccountManagerImpl URL: https://github.com/apache/cloudstack/pull/2619#issuecomment-386236009 @blueorangutan package This is an automated message from the Apache Git

[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386236019 @rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been

[GitHub] rhtyd commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
rhtyd commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386235840 @blueorangutan test matrix This is an automated message from the

[GitHub] blueorangutan commented on issue #2619: Remove "self-injection" of AccountManagerImpl

2018-05-03 Thread GitBox
blueorangutan commented on issue #2619: Remove "self-injection" of AccountManagerImpl URL: https://github.com/apache/cloudstack/pull/2619#issuecomment-386236260 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2382: CLOUDSTACK-4045 IP address acquired with associateIpAddress is marked as source NAT

2018-05-03 Thread GitBox
blueorangutan commented on issue #2382: CLOUDSTACK-4045 IP address acquired with associateIpAddress is marked as source NAT URL: https://github.com/apache/cloudstack/pull/2382#issuecomment-386237167 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make

[GitHub] rhtyd commented on issue #2382: CLOUDSTACK-4045 IP address acquired with associateIpAddress is marked as source NAT

2018-05-03 Thread GitBox
rhtyd commented on issue #2382: CLOUDSTACK-4045 IP address acquired with associateIpAddress is marked as source NAT URL: https://github.com/apache/cloudstack/pull/2382#issuecomment-386237029 @blueorangutan package This is an

[GitHub] GabrielBrascher commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address

2018-05-03 Thread GitBox
GabrielBrascher commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address URL: https://github.com/apache/cloudstack/pull/2595#discussion_r185796315 ## File path:

[GitHub] rafaelweingartner commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop)

2018-05-03 Thread GitBox
rafaelweingartner commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop) URL: https://github.com/apache/cloudstack/pull/2511#issuecomment-386326837 Ok, I see what you did there. Our goal was not to tackle concurrency problems with

[GitHub] GabrielBrascher commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address

2018-05-03 Thread GitBox
GabrielBrascher commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address URL: https://github.com/apache/cloudstack/pull/2595#discussion_r185795854 ## File path: engine/schema/src/main/java/com/cloud/dc/dao/VlanDaoImpl.java

[GitHub] GabrielBrascher commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address

2018-05-03 Thread GitBox
GabrielBrascher commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address URL: https://github.com/apache/cloudstack/pull/2595#discussion_r185795774 ## File path:

[GitHub] fmaximus opened a new pull request #2622: CLOUDSTACK-10376: UI: Create VPC Offering does not show ConfigDrive a…

2018-05-03 Thread GitBox
fmaximus opened a new pull request #2622: CLOUDSTACK-10376: UI: Create VPC Offering does not show ConfigDrive a… URL: https://github.com/apache/cloudstack/pull/2622 ## Description Added ConfigDrive as UserData provider in dropdown of Create VPC Offering ##

[GitHub] blueorangutan commented on issue #2600: CLOUDSTACK-10362: Change the "getXXX" method names to "isXXX".

2018-05-03 Thread GitBox
blueorangutan commented on issue #2600: CLOUDSTACK-10362: Change the "getXXX" method names to "isXXX". URL: https://github.com/apache/cloudstack/pull/2600#issuecomment-386310835 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386378709 Trillian test result (tid-2583) Environment: vmware-65 (x2), Advanced Networking with Mgmt server 7 Total time taken:

[GitHub] PaulAngus commented on issue #1037: Fix for CLOUDSTACK-8937 - XenServer migrations with storage failing i…

2018-05-03 Thread GitBox
PaulAngus commented on issue #1037: Fix for CLOUDSTACK-8937 - XenServer migrations with storage failing i… URL: https://github.com/apache/cloudstack/pull/1037#issuecomment-386339279 closed as #1336 and #1596 address this problem.

[GitHub] blueorangutan commented on issue #2613: Cleanup POMs

2018-05-03 Thread GitBox
blueorangutan commented on issue #2613: Cleanup POMs URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-386330163 @khos2ow a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-03 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-386329926 @blueorangutan package This is an automated message from the Apache Git Service. To respond to the

[GitHub] blueorangutan commented on issue #2613: Cleanup POMs

2018-05-03 Thread GitBox
blueorangutan commented on issue #2613: Cleanup POMs URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-386338852 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-2003 This is an automated message from the

[GitHub] DaanHoogland commented on issue #2600: CLOUDSTACK-10362: Change the "getXXX" method names to "isXXX".

2018-05-03 Thread GitBox
DaanHoogland commented on issue #2600: CLOUDSTACK-10362: Change the "getXXX" method names to "isXXX". URL: https://github.com/apache/cloudstack/pull/2600#issuecomment-386310596 @blueorangutan test This is an automated

[GitHub] GabrielBrascher commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address

2018-05-03 Thread GitBox
GabrielBrascher commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address URL: https://github.com/apache/cloudstack/pull/2595#discussion_r185800197 ## File path:

[GitHub] GabrielBrascher commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address

2018-05-03 Thread GitBox
GabrielBrascher commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address URL: https://github.com/apache/cloudstack/pull/2595#discussion_r185809174 ## File path:

[GitHub] rafaelweingartner commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address

2018-05-03 Thread GitBox
rafaelweingartner commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address URL: https://github.com/apache/cloudstack/pull/2595#discussion_r185811769 ## File path:

[GitHub] DaanHoogland commented on issue #2116: CLOUDSTACK-8959: Option to attach the config drive

2018-05-03 Thread GitBox
DaanHoogland commented on issue #2116: CLOUDSTACK-8959: Option to attach the config drive URL: https://github.com/apache/cloudstack/pull/2116#issuecomment-386309824 @rhtyd I don't see anything in this PR that is needed given other ConfigDrive functionality submitted in different PRs,

[GitHub] PaulAngus closed pull request #1037: Fix for CLOUDSTACK-8937 - XenServer migrations with storage failing i…

2018-05-03 Thread GitBox
PaulAngus closed pull request #1037: Fix for CLOUDSTACK-8937 - XenServer migrations with storage failing i… URL: https://github.com/apache/cloudstack/pull/1037 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

[GitHub] DaanHoogland commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address

2018-05-03 Thread GitBox
DaanHoogland commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address URL: https://github.com/apache/cloudstack/pull/2595#discussion_r185807052 ## File path:

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

2018-05-03 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-386308564 @blueorangutan test This is an

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

2018-05-03 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-386308596 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

[GitHub] rhtyd commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
rhtyd commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386427092 @blueorangutan test matrix This is an automated message from the

[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386427379 @rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been

[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386425921 Trillian test result (tid-2586) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time taken:

[GitHub] rafaelweingartner commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address

2018-05-03 Thread GitBox
rafaelweingartner commented on a change in pull request #2595: CLOUDSTACK-10199: Support requesting a specific IPv4 address URL: https://github.com/apache/cloudstack/pull/2595#discussion_r185860073 ## File path: engine/schema/src/main/java/com/cloud/dc/dao/VlanDaoImpl.java

[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386364298 Trillian test result (tid-2582) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken:

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

2018-05-03 Thread GitBox
blueorangutan commented on issue #2399: CLOUDSTACK-10223 delete snapshots when deleting domain URL: https://github.com/apache/cloudstack/pull/2399#issuecomment-386362267 Trillian test result (tid-2571) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time

[cloudstack] branch 4.11 updated: tentative fix for issue where kvm / ceph cannot create volumes

2018-05-03 Thread nathanejohnson
This is an automated email from the ASF dual-hosted git repository. nathanejohnson 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 c005a44 tentative fix for issue where

[GitHub] rafaelweingartner commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop)

2018-05-03 Thread GitBox
rafaelweingartner commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop) URL: https://github.com/apache/cloudstack/pull/2511#issuecomment-386446908 @rhtyd I have been thinking about this concurrency problem and it can be problematic for

[cloudstack] branch 4.11 updated (c005a44 -> 314caff)

2018-05-03 Thread nathanejohnson
This is an automated email from the ASF dual-hosted git repository. nathanejohnson pushed a change to branch 4.11 in repository https://gitbox.apache.org/repos/asf/cloudstack.git. omit c005a44 tentative fix for issue where kvm / ceph cannot create volumes This update removed existing

[GitHub] nathanejohnson opened a new pull request #2623: fixes #2611

2018-05-03 Thread GitBox
nathanejohnson opened a new pull request #2623: fixes #2611 URL: https://github.com/apache/cloudstack/pull/2623 fix issue where kvm / ceph cannot create volumes #2611 ## Description This hard codes the disk format to RAW in the case of RBD Fixes: #2611 ## How Has

[GitHub] nathanejohnson commented on issue #2611: Ceph RBD primary storage fails connection and renders node unusable

2018-05-03 Thread GitBox
nathanejohnson commented on issue #2611: Ceph RBD primary storage fails connection and renders node unusable URL: https://github.com/apache/cloudstack/issues/2611#issuecomment-386461876 @wido The issue seems to be that for any KVM host, the disk format is always hard coded to QCOW2 here:

[GitHub] nathanejohnson commented on issue #2611: Ceph RBD primary storage fails connection and renders node unusable

2018-05-03 Thread GitBox
nathanejohnson commented on issue #2611: Ceph RBD primary storage fails connection and renders node unusable URL: https://github.com/apache/cloudstack/issues/2611#issuecomment-386461876 @wido The issue seems to be that for any KVM host, the disk format is always hard coded to QCOW2 here:

[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386472523 Trillian test result (tid-2591) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 6 Total time taken:

[GitHub] nathanejohnson commented on issue #2611: Ceph RBD primary storage fails connection and renders node unusable

2018-05-03 Thread GitBox
nathanejohnson commented on issue #2611: Ceph RBD primary storage fails connection and renders node unusable URL: https://github.com/apache/cloudstack/issues/2611#issuecomment-386464903 On second thought, this might be a separate bug we were chasing. We were getting seeing this error in

[GitHub] blueorangutan commented on issue #2619: Remove "self-injection" of AccountManagerImpl

2018-05-03 Thread GitBox
blueorangutan commented on issue #2619: Remove "self-injection" of AccountManagerImpl URL: https://github.com/apache/cloudstack/pull/2619#issuecomment-386485058 Trillian test result (tid-2585) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken:

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

2018-05-03 Thread GitBox
GabrielBrascher 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_r185835963 ## File path:

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

2018-05-03 Thread GitBox
GabrielBrascher 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_r185980340 ## File path: server/src/com/cloud/network/vpc/VpcManagerImpl.java ## @@

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

2018-05-03 Thread GitBox
GabrielBrascher 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_r185953401 ## File path:

[GitHub] blueorangutan commented on issue #2600: CLOUDSTACK-10362: Change the "getXXX" method names to "isXXX".

2018-05-03 Thread GitBox
blueorangutan commented on issue #2600: CLOUDSTACK-10362: Change the "getXXX" method names to "isXXX". URL: https://github.com/apache/cloudstack/pull/2600#issuecomment-386494010 Trillian test result (tid-2589) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] rafaelweingartner commented on issue #2619: Remove "self-injection" of AccountManagerImpl

2018-05-03 Thread GitBox
rafaelweingartner commented on issue #2619: Remove "self-injection" of AccountManagerImpl URL: https://github.com/apache/cloudstack/pull/2619#issuecomment-386260661 @rhtyd you did not need to package again. I forced a new commit only to push Jenkins to build again. BTW, do you

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

2018-05-03 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-386258945 Merging this based on code reviews and test results.

[cloudstack] branch master updated: api: Fixes #2528 move ostypeid from id to uuid (#2608)

2018-05-03 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 d94a5be api: Fixes #2528 move ostypeid from

[GitHub] rhtyd commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop)

2018-05-03 Thread GitBox
rhtyd commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop) URL: https://github.com/apache/cloudstack/pull/2511#issuecomment-386265921 @rafaelweingartner instead of catering to individual change, a fix could be to simply provide the

[GitHub] blueorangutan commented on issue #2621: Backports for 4.11 branch

2018-05-03 Thread GitBox
blueorangutan commented on issue #2621: Backports for 4.11 branch URL: https://github.com/apache/cloudstack/pull/2621#issuecomment-386264158 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2621: Backports for 4.11 branch

2018-05-03 Thread GitBox
blueorangutan commented on issue #2621: Backports for 4.11 branch URL: https://github.com/apache/cloudstack/pull/2621#issuecomment-386270586 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2621: Backports for 4.11 branch

2018-05-03 Thread GitBox
blueorangutan commented on issue #2621: Backports for 4.11 branch URL: https://github.com/apache/cloudstack/pull/2621#issuecomment-386270257 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2001 This is an automated message

[GitHub] rhtyd commented on issue #2621: Backports for 4.11 branch

2018-05-03 Thread GitBox
rhtyd commented on issue #2621: Backports for 4.11 branch URL: https://github.com/apache/cloudstack/pull/2621#issuecomment-386270517 @blueorangutan package This is an automated message from the Apache Git Service. To respond

[GitHub] blueorangutan commented on issue #2621: Backports for 4.11 branch

2018-05-03 Thread GitBox
blueorangutan commented on issue #2621: Backports for 4.11 branch URL: https://github.com/apache/cloudstack/pull/2621#issuecomment-386278062 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] borisstoyanov commented on issue #2621: Backports for 4.11 branch

2018-05-03 Thread GitBox
borisstoyanov commented on issue #2621: Backports for 4.11 branch URL: https://github.com/apache/cloudstack/pull/2621#issuecomment-386278034 @blueorangutan test This is an automated message from the Apache Git Service. To

[GitHub] blueorangutan commented on issue #2621: Backports for 4.11 branch

2018-05-03 Thread GitBox
blueorangutan commented on issue #2621: Backports for 4.11 branch URL: https://github.com/apache/cloudstack/pull/2621#issuecomment-386277850 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2002 This is an automated message

[GitHub] rafaelweingartner commented on a change in pull request #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
rafaelweingartner commented on a change in pull request #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#discussion_r185754918 ## File path: systemvm/debian/opt/cloud/bin/setup/patchsystemvm.sh ## @@ -37,6 +37,8

[cloudstack] branch master updated: CLOUDSTACK-9781: record resource uuid in events instead of integer id (#1940)

2018-05-03 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 1ff171d CLOUDSTACK-9781: record resource

[GitHub] rhtyd closed pull request #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID.

2018-05-03 Thread GitBox
rhtyd closed pull request #1940: CLOUDSTACK-9781:ACS records ID in events tables instead of UUID. URL: https://github.com/apache/cloudstack/pull/1940 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

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

2018-05-03 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] blueorangutan commented on issue #2382: CLOUDSTACK-4045 IP address acquired with associateIpAddress is marked as source NAT

2018-05-03 Thread GitBox
blueorangutan commented on issue #2382: CLOUDSTACK-4045 IP address acquired with associateIpAddress is marked as source NAT URL: https://github.com/apache/cloudstack/pull/2382#issuecomment-386246094 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1999

[GitHub] rhtyd closed pull request #2608: API: move ostypeid from DB id to DB uuid

2018-05-03 Thread GitBox
rhtyd closed pull request #2608: API: move ostypeid from DB id to DB uuid URL: https://github.com/apache/cloudstack/pull/2608 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 closed issue #2528: listVirtualMachines response ostypeid is foreign DB ID, not UUID

2018-05-03 Thread GitBox
rhtyd closed issue #2528: listVirtualMachines response ostypeid is foreign DB ID, not UUID URL: https://github.com/apache/cloudstack/issues/2528 This is an automated message from the Apache Git Service. To respond to the

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

2018-05-03 Thread GitBox
rhtyd commented on issue #2608: API: move ostypeid from DB id to DB uuid URL: https://github.com/apache/cloudstack/pull/2608#issuecomment-386258672 Test LGTM, comparing last two runs. Merging this based on reviews and tests.

[GitHub] rhtyd commented on issue #2619: Remove "self-injection" of AccountManagerImpl

2018-05-03 Thread GitBox
rhtyd commented on issue #2619: Remove "self-injection" of AccountManagerImpl URL: https://github.com/apache/cloudstack/pull/2619#issuecomment-386269134 Okay @rafaelweingartner, I saw a force push I thought something changed. I'll kick tests now. @blueorangutan test

[GitHub] blueorangutan commented on issue #2619: Remove "self-injection" of AccountManagerImpl

2018-05-03 Thread GitBox
blueorangutan commented on issue #2619: Remove "self-injection" of AccountManagerImpl URL: https://github.com/apache/cloudstack/pull/2619#issuecomment-386269263 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] blueorangutan commented on issue #2619: Remove "self-injection" of AccountManagerImpl

2018-05-03 Thread GitBox
blueorangutan commented on issue #2619: Remove "self-injection" of AccountManagerImpl URL: https://github.com/apache/cloudstack/pull/2619#issuecomment-386245824 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1998 This is

[GitHub] rhtyd commented on issue #2619: Remove "self-injection" of AccountManagerImpl

2018-05-03 Thread GitBox
rhtyd commented on issue #2619: Remove "self-injection" of AccountManagerImpl URL: https://github.com/apache/cloudstack/pull/2619#issuecomment-386259715 @blueorangutan package This is an automated message from the

[GitHub] rafaelweingartner commented on issue #2619: Remove "self-injection" of AccountManagerImpl

2018-05-03 Thread GitBox
rafaelweingartner commented on issue #2619: Remove "self-injection" of AccountManagerImpl URL: https://github.com/apache/cloudstack/pull/2619#issuecomment-386260661 @rhtyd you did not need to package again. I forced a new commit only to push Jenkins to build again. BTW, do you

[GitHub] blueorangutan commented on issue #2621: Backports for 4.11 branch

2018-05-03 Thread GitBox
blueorangutan commented on issue #2621: Backports for 4.11 branch URL: https://github.com/apache/cloudstack/pull/2621#issuecomment-386264158 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2619: Remove "self-injection" of AccountManagerImpl

2018-05-03 Thread GitBox
blueorangutan commented on issue #2619: Remove "self-injection" of AccountManagerImpl URL: https://github.com/apache/cloudstack/pull/2619#issuecomment-386266325 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2000 This is

[GitHub] rafaelweingartner commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop)

2018-05-03 Thread GitBox
rafaelweingartner commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop) URL: https://github.com/apache/cloudstack/pull/2511#issuecomment-386267109 I did not understand what you mean. Are you saying we keep in javascript a list with

[GitHub] blueorangutan commented on issue #2619: Remove "self-injection" of AccountManagerImpl

2018-05-03 Thread GitBox
blueorangutan commented on issue #2619: Remove "self-injection" of AccountManagerImpl URL: https://github.com/apache/cloudstack/pull/2619#issuecomment-386259939 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] borisstoyanov commented on issue #2621: Backports for 4.11 branch

2018-05-03 Thread GitBox
borisstoyanov commented on issue #2621: Backports for 4.11 branch URL: https://github.com/apache/cloudstack/pull/2621#issuecomment-386264136 @blueorangutan package This is an automated message from the Apache Git Service. To

[GitHub] rafaelweingartner commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop)

2018-05-03 Thread GitBox
rafaelweingartner commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop) URL: https://github.com/apache/cloudstack/pull/2511#issuecomment-386267109 I did not understand what you mean. Are you saying we keep in javascript a list with

[GitHub] rhtyd opened a new pull request #2621: Backports for 4.11 branch

2018-05-03 Thread GitBox
rhtyd opened a new pull request #2621: Backports for 4.11 branch URL: https://github.com/apache/cloudstack/pull/2621 ## Description This PR has backported changes/PRs merged on master that we can consider for 4.11, limiting to fixes and no features. ## Types of changes

[GitHub] rhtyd commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop)

2018-05-03 Thread GitBox
rhtyd commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop) URL: https://github.com/apache/cloudstack/pull/2511#issuecomment-386270351 @rafaelweingartner I think you're trying to solve the ordering problem. Here's another problem to this:

[GitHub] blueorangutan commented on issue #2621: Backports for 4.11 branch

2018-05-03 Thread GitBox
blueorangutan commented on issue #2621: Backports for 4.11 branch URL: https://github.com/apache/cloudstack/pull/2621#issuecomment-386270257 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2001 This is an automated message

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

2018-05-03 Thread GitBox
rhtyd commented on issue #2508: CLOUDSTACK-9114: Reduce VR downtime during network restart URL: https://github.com/apache/cloudstack/pull/2508#issuecomment-386272855 Ping - @ustcweizhou @resmo @rafaelweingartner @wido @nitin-maharana @nvazquez @NuxRo review and testing is requested.

[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386497666 Trillian test result (tid-2593) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken:

[GitHub] blueorangutan commented on issue #2621: Backports for 4.11 branch

2018-05-03 Thread GitBox
blueorangutan commented on issue #2621: Backports for 4.11 branch URL: https://github.com/apache/cloudstack/pull/2621#issuecomment-386498228 Trillian test result (tid-2587) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 52791 seconds Marvin

[GitHub] blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes

2018-05-03 Thread GitBox
blueorangutan commented on issue #2616: 4.11.1.0: Stabilization and blocker/issues fixes URL: https://github.com/apache/cloudstack/pull/2616#issuecomment-386502609 Trillian test result (tid-2594) Environment: vmware-65 (x2), Advanced Networking with Mgmt server 7 Total time taken: