[GitHub] wido commented on issue #2472: CLOUDSTACK-10310 Fix KVM reboot on storage issue

2018-05-21 Thread GitBox
wido commented on issue #2472: CLOUDSTACK-10310 Fix KVM reboot on storage issue URL: https://github.com/apache/cloudstack/pull/2472#issuecomment-390883172 @rhtyd Both cases are not ideal. Because if you stop the Agent due to a single Primary Storage failure you leave VMs running there which

[GitHub] rhtyd commented on issue #2639: [CLOUDSTACK-10276] listVolumes not working when storage UUID is not a UUID

2018-05-21 Thread GitBox
rhtyd commented on issue #2639: [CLOUDSTACK-10276] listVolumes not working when storage UUID is not a UUID URL: https://github.com/apache/cloudstack/pull/2639#issuecomment-390881251 I'm unable to find time to do the investigation on this. @DaanHoogland can you advise, and if it's all okay

[GitHub] rhtyd commented on issue #2558: API uploadSslCert does not work

2018-05-21 Thread GitBox
rhtyd commented on issue #2558: API uploadSslCert does not work URL: https://github.com/apache/cloudstack/issues/2558#issuecomment-390879871 @GabrielBrascher can you at least help test this against a normal certificate of size less than 8k? Usually certificates are 2k-4k in size. If there

[GitHub] DaanHoogland commented on issue #2558: API uploadSslCert does not work

2018-05-21 Thread GitBox
DaanHoogland commented on issue #2558: API uploadSslCert does not work URL: https://github.com/apache/cloudstack/issues/2558#issuecomment-390877241 :) -(a weekend) The question is not how much time you have but how much we need this in, @GabrielBrascher

[GitHub] DaanHoogland commented on issue #2472: CLOUDSTACK-10310 Fix KVM reboot on storage issue

2018-05-21 Thread GitBox
DaanHoogland commented on issue #2472: CLOUDSTACK-10310 Fix KVM reboot on storage issue URL: https://github.com/apache/cloudstack/pull/2472#issuecomment-390876820 echo b > /proc/sysrq-trigger is not acceptable to more people, I don't see how killing (and maybe restarting) the agent can be

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

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

[GitHub] blueorangutan commented on issue #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid

2018-05-21 Thread GitBox
blueorangutan commented on issue #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid URL: https://github.com/apache/cloudstack/pull/2653#issuecomment-390794056 Trillian test result (tid-2693) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt ser

[GitHub] mike-tutkowski commented on issue #2655: Handle Ceph.

2018-05-21 Thread GitBox
mike-tutkowski commented on issue #2655: Handle Ceph. URL: https://github.com/apache/cloudstack/pull/2655#issuecomment-390762543 Wow, that was fast. :) I’ve seen in the recent past where we cache a bunch of the PRs for forward merging and then do so from time to time in a batch fashion. Lo

[GitHub] mike-tutkowski commented on issue #2655: Handle Ceph.

2018-05-21 Thread GitBox
mike-tutkowski commented on issue #2655: Handle Ceph. URL: https://github.com/apache/cloudstack/pull/2655#issuecomment-390754751 @rhtyd I'm not sure on the process here. Will you be merging #2655 and #2656 to master or should I do that? Thanks! -

[GitHub] rhtyd commented on issue #2655: Handle Ceph.

2018-05-21 Thread GitBox
rhtyd commented on issue #2655: Handle Ceph. URL: https://github.com/apache/cloudstack/pull/2655#issuecomment-390754969 Mike you can check, they've been forward merged to master. This is an automated message from the Apache G

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

2018-05-21 Thread GitBox
blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-390706643 Trillian test result (tid-2695) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time taken: 21166 seconds Mar

[GitHub] rhtyd commented on issue #2639: [CLOUDSTACK-10276] listVolumes not working when storage UUID is not a UUID

2018-05-21 Thread GitBox
rhtyd commented on issue #2639: [CLOUDSTACK-10276] listVolumes not working when storage UUID is not a UUID URL: https://github.com/apache/cloudstack/pull/2639#issuecomment-390691842 @rafaelweingartner yes you're right, I tested it and it works due to a heuristic I had implemented (and forg

[GitHub] rhtyd commented on issue #2275: CLOUDSTACK-9975: Allow customizing system VM templates for SSVM and Console Proxy

2018-05-21 Thread GitBox
rhtyd commented on issue #2275: CLOUDSTACK-9975: Allow customizing system VM templates for SSVM and Console Proxy URL: https://github.com/apache/cloudstack/pull/2275#issuecomment-390690993 @GabrielBrascher perhaps you can explore refactoring of the systemvmtemplate upgrade code we've as pa

[GitHub] GabrielBrascher commented on issue #2275: CLOUDSTACK-9975: Allow customizing system VM templates for SSVM and Console Proxy

2018-05-21 Thread GitBox
GabrielBrascher commented on issue #2275: CLOUDSTACK-9975: Allow customizing system VM templates for SSVM and Console Proxy URL: https://github.com/apache/cloudstack/pull/2275#issuecomment-390689815 Thanks, @rhtyd. I used that code in CloudStack 4.9.3.0 when it was requested to edit system

[GitHub] GabrielBrascher commented on issue #2558: API uploadSslCert does not work

2018-05-21 Thread GitBox
GabrielBrascher commented on issue #2558: API uploadSslCert does not work URL: https://github.com/apache/cloudstack/issues/2558#issuecomment-390673108 @rhtyd @PaulAngus sorry for the delay but I am still working on this one. Do you think we should push this to 4.11.2? How much time do I ha

[GitHub] rafaelweingartner commented on issue #2639: [CLOUDSTACK-10276] listVolumes not working when storage UUID is not a UUID

2018-05-21 Thread GitBox
rafaelweingartner commented on issue #2639: [CLOUDSTACK-10276] listVolumes not working when storage UUID is not a UUID URL: https://github.com/apache/cloudstack/pull/2639#issuecomment-390626277 @rhtyd how does it break cloudmonkey? Cloudmonkey continues to see the same thing. There is n

[cloudstack] branch master updated (7c6777b -> 9c1eabf)

2018-05-21 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 7c6777b Merge branch '4.11': allow config drives on primary storage for KVM (#2651) add e471a46 managed-st

[GitHub] rhtyd closed issue #2540: problem adding new shared network NIC to VM "A NIC with this MAC address exits for network:"

2018-05-21 Thread GitBox
rhtyd closed issue #2540: problem adding new shared network NIC to VM "A NIC with this MAC address exits for network:" URL: https://github.com/apache/cloudstack/issues/2540 This is an automated message from the Apache Git Se

[cloudstack] branch 4.11 updated: addNicToVirtualMachine: Fixes #2540 handle invalid MAC address arg (#2653)

2018-05-21 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 02ece53 addNicToVirtualMachine: Fixes #2540 hand

[GitHub] rhtyd closed pull request #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid

2018-05-21 Thread GitBox
rhtyd closed pull request #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid URL: https://github.com/apache/cloudstack/pull/2653 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 #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid

2018-05-21 Thread GitBox
rhtyd commented on issue #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid URL: https://github.com/apache/cloudstack/pull/2653#issuecomment-390621442 To keep up with RC schedule, I'll merge this based on code reviews and travis result. The smoketest is don

[GitHub] rhtyd closed pull request #2656: Only perform certain actions here with managed storage if the VM is s…

2018-05-21 Thread GitBox
rhtyd closed pull request #2656: Only perform certain actions here with managed storage if the VM is s… URL: https://github.com/apache/cloudstack/pull/2656 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 prove

[cloudstack] branch 4.11 updated: managed-storage: handle VM start in a new cluster on different host (#2656)

2018-05-21 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 e471a46 managed-storage: handle VM start in a ne

[GitHub] rhtyd commented on issue #2656: Only perform certain actions here with managed storage if the VM is s…

2018-05-21 Thread GitBox
rhtyd commented on issue #2656: Only perform certain actions here with managed storage if the VM is s… URL: https://github.com/apache/cloudstack/pull/2656#issuecomment-390621121 Merging this based on regression tests and code review.

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

2018-05-21 Thread GitBox
rhtyd commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-390611974 Manually kicked tests via BO backend. This is an automated message from the Apache Git Se

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

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

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

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

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

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

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

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

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

2018-05-21 Thread GitBox
blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-390605048 @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-05-21 Thread GitBox
rhtyd commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-387981139 @blueorangutan package This is an automated message from the Apache Git Service. To respo

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

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

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

2018-05-21 Thread GitBox
blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-388297440 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-2045 This is an automated messag

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

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

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

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

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

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

[GitHub] melnik13 commented on issue #2657: KVM hosts are being reset by a script

2018-05-21 Thread GitBox
melnik13 commented on issue #2657: KVM hosts are being reset by a script URL: https://github.com/apache/cloudstack/issues/2657#issuecomment-390604935 Thank you! I didn't notice that 4.11.x have it fixed, in 4.9.2 it still can reset a host. In any case, the script and `echo b > /proc/

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

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

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

2018-05-21 Thread GitBox
blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-388287906 @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-05-21 Thread GitBox
rhtyd commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-388287831 @blueorangutan package This is an automated message from the Apache Git Service. To respo

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

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

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

2018-05-21 Thread GitBox
blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-388305624 Packaging result: ✖centos6 ✔centos7 ✖debian. JID-2046 This is an automated messag

[GitHub] melnik13 opened a new pull request #2658: Fix issue #2657

2018-05-21 Thread GitBox
melnik13 opened a new pull request #2658: Fix issue #2657 URL: https://github.com/apache/cloudstack/pull/2658 ## Description I'd propose to kill the qemu-kvm processes of the VMs whose volumes are located on an NFS storage instead of rebooting the whole host. ## Ty

[GitHub] serverchief commented on issue #2657: KVM hosts are being reset by a script

2018-05-21 Thread GitBox
serverchief commented on issue #2657: KVM hosts are being reset by a script URL: https://github.com/apache/cloudstack/issues/2657#issuecomment-390603337 As of 4.11 CloudStack has an enhancement for KVM HA. Instead of relying on shell script and specific primary store - we will check the wri

svn commit: r27013 - /release/cloudstack/KEYS

2018-05-21 Thread paul_a
Author: paul_a Date: Mon May 21 09:26:39 2018 New Revision: 27013 Log: KEYS: add key of Paul Angus (pau...@apache.org) Modified: release/cloudstack/KEYS Modified: release/cloudstack/KEYS == --- release/cloudstack

[cloudstack] branch master updated (2b7d6cf -> 7c6777b)

2018-05-21 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 2b7d6cf Merge branch '4.11': Add option on if to VM HA power-on a OOB-shut-off-VM (#2473) add acc5fdc CLOU

[GitHub] melnik13 opened a new issue #2657: KVM hosts are being reset by a script

2018-05-21 Thread GitBox
melnik13 opened a new issue #2657: KVM hosts are being reset by a script URL: https://github.com/apache/cloudstack/issues/2657 # ISSUE TYPE * Bug Report # COMPONENT NAME This issue is related to KVM hosts. As far as I can see, XEN hosts are affected too

[cloudstack] branch 4.11 updated: CLOUDSTACK-10290: allow config drives on primary storage for KVM (#2651)

2018-05-21 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 acc5fdc CLOUDSTACK-10290: allow config drives on

[GitHub] rhtyd closed pull request #2651: CLOUDSTACK-10290: Introduce option to have config drives on primary storage

2018-05-21 Thread GitBox
rhtyd closed pull request #2651: CLOUDSTACK-10290: Introduce option to have config drives on primary storage URL: https://github.com/apache/cloudstack/pull/2651 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 #2651: CLOUDSTACK-10290: Introduce option to have config drives on primary storage

2018-05-21 Thread GitBox
rhtyd commented on issue #2651: CLOUDSTACK-10290: Introduce option to have config drives on primary storage URL: https://github.com/apache/cloudstack/pull/2651#issuecomment-390594929 Merging based on code reviews, test results incl. manual tests. ---

[GitHub] blueorangutan commented on issue #2649: Catch error in Debian packaging script and fail the build

2018-05-21 Thread GitBox
blueorangutan commented on issue #2649: Catch error in Debian packaging script and fail the build URL: https://github.com/apache/cloudstack/pull/2649#issuecomment-390590880 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-2071 ---

[GitHub] PaulAngus commented on issue #2558: API uploadSslCert does not work

2018-05-21 Thread GitBox
PaulAngus commented on issue #2558: API uploadSslCert does not work URL: https://github.com/apache/cloudstack/issues/2558#issuecomment-390590387 @GabrielBrascher - we're looking to cut an RC soonish, do you have an eta for a fix please?

[GitHub] blueorangutan commented on issue #2649: Catch error in Debian packaging script and fail the build

2018-05-21 Thread GitBox
blueorangutan commented on issue #2649: Catch error in Debian packaging script and fail the build URL: https://github.com/apache/cloudstack/pull/2649#issuecomment-390585214 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. ---

[GitHub] rhtyd commented on issue #2649: Catch error in Debian packaging script and fail the build

2018-05-21 Thread GitBox
rhtyd commented on issue #2649: Catch error in Debian packaging script and fail the build URL: https://github.com/apache/cloudstack/pull/2649#issuecomment-390585138 @blueorangutan package This is an automated message from the

[cloudstack] branch master updated (93e3745 -> 2b7d6cf)

2018-05-21 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 93e3745 Merge branch '4.11' add f23278a CLOUDSTACK-10309: Add option on if to VM HA power-on a OOB-shut-of

[GitHub] blueorangutan commented on issue #2649: Catch error in Debian packaging script and fail the build

2018-05-21 Thread GitBox
blueorangutan commented on issue #2649: Catch error in Debian packaging script and fail the build URL: https://github.com/apache/cloudstack/pull/2649#issuecomment-390583805 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-2070 ---

[cloudstack] 01/01: Merge branch '4.11'

2018-05-21 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 commit 93e374599a23d23aed34e0793b0ed36cdb330fa2 Merge: 0afba54 7e6fddb Author: Rohit Yadav AuthorDate: Mon May 21 13:07:27 2

[cloudstack] branch master updated (0afba54 -> 93e3745)

2018-05-21 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 0afba54 Re-add code that was missed during merge forward of PR#2629 add 9aa1743 registerIso: Fixes #2654 re

[GitHub] rhtyd closed issue #2654: Register ISO image in all zones

2018-05-21 Thread GitBox
rhtyd closed issue #2654: Register ISO image in all zones URL: https://github.com/apache/cloudstack/issues/2654 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use

[GitHub] blueorangutan commented on issue #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid

2018-05-21 Thread GitBox
blueorangutan commented on issue #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid URL: https://github.com/apache/cloudstack/pull/2653#issuecomment-390581970 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid

2018-05-21 Thread GitBox
rhtyd commented on issue #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid URL: https://github.com/apache/cloudstack/pull/2653#issuecomment-390581933 @blueorangutan test This is an automated

[GitHub] blueorangutan commented on issue #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid

2018-05-21 Thread GitBox
blueorangutan commented on issue #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid URL: https://github.com/apache/cloudstack/pull/2653#issuecomment-390579951 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2069

[GitHub] rhtyd commented on issue #2275: CLOUDSTACK-9975: Allow customizing system VM templates for SSVM and Console Proxy

2018-05-21 Thread GitBox
rhtyd commented on issue #2275: CLOUDSTACK-9975: Allow customizing system VM templates for SSVM and Console Proxy URL: https://github.com/apache/cloudstack/pull/2275#issuecomment-390579766 I verified that changing systemvmtemplate by following the changes that the upgrade path does, indeed

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

2018-05-21 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-390579519 Ping @houthuis This is an automat

[GitHub] rhtyd commented on issue #2472: CLOUDSTACK-10310 Fix KVM reboot on storage issue

2018-05-21 Thread GitBox
rhtyd commented on issue #2472: CLOUDSTACK-10310 Fix KVM reboot on storage issue URL: https://github.com/apache/cloudstack/pull/2472#issuecomment-390579296 @DaanHoogland @rafaelweingartner @wido @bwsw - do you think stopping the agent can cause a bigger regression than the historic behaviou

[GitHub] rhtyd closed pull request #2473: CLOUDSTACK-10309 Add option on if to VM HA power-on a OOB-shut-off-VM

2018-05-21 Thread GitBox
rhtyd closed pull request #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 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 proven

[cloudstack] branch 4.11 updated: CLOUDSTACK-10309: Add option on if to VM HA power-on a OOB-shut-off-VM (#2473)

2018-05-21 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 f23278a CLOUDSTACK-10309: Add option on if to VM

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

2018-05-21 Thread GitBox
rhtyd 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-390578738 Test and code LGTM. This is an automated message

[GitHub] blueorangutan commented on issue #2649: Catch error in Debian packaging script and fail the build

2018-05-21 Thread GitBox
blueorangutan commented on issue #2649: Catch error in Debian packaging script and fail the build URL: https://github.com/apache/cloudstack/pull/2649#issuecomment-390578230 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. ---

[GitHub] rhtyd commented on issue #2649: Catch error in Debian packaging script and fail the build

2018-05-21 Thread GitBox
rhtyd commented on issue #2649: Catch error in Debian packaging script and fail the build URL: https://github.com/apache/cloudstack/pull/2649#issuecomment-390578060 @blueorangutan package This is an automated message from the

[GitHub] rhtyd commented on issue #2639: [CLOUDSTACK-10276] listVolumes not working when storage UUID is not a UUID

2018-05-21 Thread GitBox
rhtyd commented on issue #2639: [CLOUDSTACK-10276] listVolumes not working when storage UUID is not a UUID URL: https://github.com/apache/cloudstack/pull/2639#issuecomment-390575404 I've my reservations on the fix, instead perhaps it may be better and could be simpler to handle use of exte

[cloudstack] branch 4.11 updated: managed-storage: Handle Ceph (#2655)

2018-05-21 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 7e6fddb managed-storage: Handle Ceph (#2655) 7e6

[GitHub] rhtyd closed pull request #2655: Handle Ceph.

2018-05-21 Thread GitBox
rhtyd closed pull request #2655: Handle Ceph. URL: https://github.com/apache/cloudstack/pull/2655 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 a fork), th

[GitHub] blueorangutan commented on issue #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid

2018-05-21 Thread GitBox
blueorangutan commented on issue #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid URL: https://github.com/apache/cloudstack/pull/2653#issuecomment-390574179 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid

2018-05-21 Thread GitBox
rhtyd commented on issue #2653: Generate MAC address if the MAC in command addNicToVirtualMachine is invalid URL: https://github.com/apache/cloudstack/pull/2653#issuecomment-390574099 @blueorangutan package This is an automat

[GitHub] rhtyd closed pull request #2630: Host Affinity plugin

2018-05-21 Thread GitBox
rhtyd closed pull request #2630: Host Affinity plugin URL: https://github.com/apache/cloudstack/pull/2630 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 a f

[GitHub] GabrielBrascher commented on a change in pull request #2655: Handle Ceph.

2018-05-21 Thread GitBox
GabrielBrascher commented on a change in pull request #2655: Handle Ceph. URL: https://github.com/apache/cloudstack/pull/2655#discussion_r189511508 ## File path: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtMigrateCommandWrapper.java ## @@ -

[cloudstack] branch 4.11 updated: Host Affinity plugin (#2630)

2018-05-21 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 06f7e49 Host Affinity plugin (#2630) 06f7e49 is