[GitHub] mike-tutkowski commented on issue #2983: KVM live storage migration intra cluster from NFS source and destination

2018-10-30 Thread GitBox
mike-tutkowski commented on issue #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#issuecomment-434525934 @nvazquez Hi Nicolas - Just an FYI that I plan to look at this tomorrow.

[GitHub] svenvogel commented on issue #2986: set affinity for user (ui problem)

2018-10-30 Thread GitBox
svenvogel commented on issue #2986: set affinity for user (ui problem) URL: https://github.com/apache/cloudstack/issues/2986#issuecomment-434479774 yes. we approved your pull request #2987 . it works fine. This is an

[GitHub] nvazquez commented on issue #2983: KVM live storage migration intra cluster from NFS source and destination

2018-10-30 Thread GitBox
nvazquez commented on issue #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#issuecomment-434469627 Feature specification link added to the PR description

[GitHub] somejfn opened a new issue #2989: Unable to create network post upgrade to 4.11

2018-10-30 Thread GitBox
somejfn opened a new issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ API ~~~ # CLOUDSTACK VERSION ~~~ 4.11.2rc3 ~~~

[GitHub] blueorangutan commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-10-30 Thread GitBox
blueorangutan commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-434450931 Trillian test result (tid-3135) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken:

[GitHub] rafaelweingartner commented on issue #2986: set affinity for user (ui problem)

2018-10-30 Thread GitBox
rafaelweingartner commented on issue #2986: set affinity for user (ui problem) URL: https://github.com/apache/cloudstack/issues/2986#issuecomment-434449217 can you go to https://github.com/apache/cloudstack/pull/2987 and approve the PR?

[GitHub] svenvogel commented on issue #2986: set affinity for user (ui problem)

2018-10-30 Thread GitBox
svenvogel commented on issue #2986: set affinity for user (ui problem) URL: https://github.com/apache/cloudstack/issues/2986#issuecomment-434448240 @rafaelweingartner we tested it. it works! great work from you! LGTM! This

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

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

[GitHub] RPDiep commented on issue #2988: Security Group: add secondary ips to the correct ipset based on ip family

2018-10-30 Thread GitBox
RPDiep commented on issue #2988: Security Group: add secondary ips to the correct ipset based on ip family URL: https://github.com/apache/cloudstack/pull/2988#issuecomment-434383596 @GabrielBrascher Can you run tests?

[GitHub] RPDiep opened a new pull request #2988: Security Group: add secondary ips to the correct ipset based on ip family

2018-10-30 Thread GitBox
RPDiep opened a new pull request #2988: Security Group: add secondary ips to the correct ipset based on ip family URL: https://github.com/apache/cloudstack/pull/2988 ## Description Currently secondary ipv6 addresses are added to the ipv4 ipset in `security_group.py`. This

[GitHub] rafaelweingartner commented on issue #2986: set affinity for user (ui problem)

2018-10-30 Thread GitBox
rafaelweingartner commented on issue #2986: set affinity for user (ui problem) URL: https://github.com/apache/cloudstack/issues/2986#issuecomment-434380357 @svenvogel can you take a look at #2987? This is an automated message

[GitHub] rafaelweingartner opened a new pull request #2987: Fix set affinity for user overlay problem (Fixes #2986)

2018-10-30 Thread GitBox
rafaelweingartner opened a new pull request #2987: Fix set affinity for user overlay problem (Fixes #2986) URL: https://github.com/apache/cloudstack/pull/2987 ## Description This PR fixes #2986 ## Types of changes - [ ] Breaking change (fix

[GitHub] rafaelweingartner commented on a change in pull request #2905: Fix delete domain button in the UI to update only after deleteDomain job finishes

2018-10-30 Thread GitBox
rafaelweingartner commented on a change in pull request #2905: Fix delete domain button in the UI to update only after deleteDomain job finishes URL: https://github.com/apache/cloudstack/pull/2905#discussion_r229386718 ## File path: ui/scripts/domains.js ## @@ -69,11

[GitHub] svenvogel commented on issue #2986: set affinity for user (ui problem)

2018-10-30 Thread GitBox
svenvogel commented on issue #2986: set affinity for user (ui problem) URL: https://github.com/apache/cloudstack/issues/2986#issuecomment-434375533 @nvazquez another ui problem, maybe you can have a look :) This is an

[GitHub] svenvogel opened a new issue #2986: set affinity for user (ui problem)

2018-10-30 Thread GitBox
svenvogel opened a new issue #2986: set affinity for user (ui problem) URL: https://github.com/apache/cloudstack/issues/2986 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ UI ~~~ # CLOUDSTACK VERSION ~~~ 4.11 -> 4.12

[GitHub] wido commented on issue #2913: Deactivate ehcache

2018-10-30 Thread GitBox
wido commented on issue #2913: Deactivate ehcache URL: https://github.com/apache/cloudstack/pull/2913#issuecomment-434361353 @marcaurele We just disable it in the XML, we do not use any custom ones. This is an automated

[GitHub] marcaurele commented on issue #2913: Deactivate ehcache

2018-10-30 Thread GitBox
marcaurele commented on issue #2913: Deactivate ehcache URL: https://github.com/apache/cloudstack/pull/2913#issuecomment-434356358 @wido do you use some custom configuration, or do you simply keep the one provided by default? My next move will be to remove all the code related to ehcache

[GitHub] svenvogel commented on a change in pull request #2980: [4.11] Fix set initial reservation on public IP ranges

2018-10-30 Thread GitBox
svenvogel commented on a change in pull request #2980: [4.11] Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2980#discussion_r229360520 ## File path: server/src/com/cloud/configuration/ConfigurationManagerImpl.java ## @@

[GitHub] svenvogel commented on a change in pull request #2980: [4.11] Fix set initial reservation on public IP ranges

2018-10-30 Thread GitBox
svenvogel commented on a change in pull request #2980: [4.11] Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2980#discussion_r229360520 ## File path: server/src/com/cloud/configuration/ConfigurationManagerImpl.java ## @@

[GitHub] blueorangutan commented on issue #2984: kvm: reset KVM host on heartbeat failure

2018-10-30 Thread GitBox
blueorangutan commented on issue #2984: kvm: reset KVM host on heartbeat failure URL: https://github.com/apache/cloudstack/pull/2984#issuecomment-434336816 Trillian test result (tid-3133) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 17447

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

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

[GitHub] svenvogel commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-10-30 Thread GitBox
svenvogel commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-434304345 LGTM! This is an

[GitHub] nvazquez commented on issue #2983: KVM live storage migration intra cluster from NFS source and destination

2018-10-30 Thread GitBox
nvazquez commented on issue #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#issuecomment-434303631 Thanks @wido @DaanHoogland for reviewing, I'll address your comments soon. Along with @borisstoyanov we'll

[GitHub] nvazquez commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination

2018-10-30 Thread GitBox
nvazquez commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#discussion_r229306734 ## File path:

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

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

[GitHub] wido commented on a change in pull request #2985: kvm/ceph: When create a new RBD image use modern flags/features for t…

2018-10-30 Thread GitBox
wido commented on a change in pull request #2985: kvm/ceph: When create a new RBD image use modern flags/features for t… URL: https://github.com/apache/cloudstack/pull/2985#discussion_r229305470 ## File path:

[GitHub] rhtyd commented on issue #2985: kvm/ceph: When create a new RBD image use modern flags/features for t…

2018-10-30 Thread GitBox
rhtyd commented on issue #2985: kvm/ceph: When create a new RBD image use modern flags/features for t… URL: https://github.com/apache/cloudstack/pull/2985#issuecomment-434301211 @blueorangutan package This is an automated

[GitHub] nvazquez commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination

2018-10-30 Thread GitBox
nvazquez commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#discussion_r229305185 ## File path:

[GitHub] rhtyd commented on a change in pull request #2985: kvm/ceph: When create a new RBD image use modern flags/features for t…

2018-10-30 Thread GitBox
rhtyd commented on a change in pull request #2985: kvm/ceph: When create a new RBD image use modern flags/features for t… URL: https://github.com/apache/cloudstack/pull/2985#discussion_r229304982 ## File path:

[GitHub] wido commented on issue #2985: kvm/ceph: When create a new RBD image use modern flags/features for t…

2018-10-30 Thread GitBox
wido commented on issue #2985: kvm/ceph: When create a new RBD image use modern flags/features for t… URL: https://github.com/apache/cloudstack/pull/2985#issuecomment-434299170 This also partially fixes #2650 This is an

[GitHub] wido opened a new pull request #2985: kvm/ceph: When create a new RBD image use modern flags/features for t…

2018-10-30 Thread GitBox
wido opened a new pull request #2985: kvm/ceph: When create a new RBD image use modern flags/features for t… URL: https://github.com/apache/cloudstack/pull/2985 …he image These additional RBD features allow for faster lookups of how much space a RBD image is using, but with the

[GitHub] wido commented on issue #2984: kvm: reset KVM host on heartbeat failure

2018-10-30 Thread GitBox
wido commented on issue #2984: kvm: reset KVM host on heartbeat failure URL: https://github.com/apache/cloudstack/pull/2984#issuecomment-434297763 I understand @somejfn , but 22 minutes is a very long time. In this case you can't blame CloudStack for the fact that it would reboot,

[GitHub] blueorangutan commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-10-30 Thread GitBox
blueorangutan commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-434296319 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-10-30 Thread GitBox
rhtyd commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-434294580 @blueorangutan test This is an automated message from the Apache Git

[GitHub] rhtyd commented on issue #2984: kvm: reset KVM host on heartbeat failure

2018-10-30 Thread GitBox
rhtyd commented on issue #2984: kvm: reset KVM host on heartbeat failure URL: https://github.com/apache/cloudstack/pull/2984#issuecomment-434294238 @somejfn most people would choose VM downtime over potential disk corruption

[GitHub] blueorangutan commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-10-30 Thread GitBox
blueorangutan commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-434291399 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2396 This is

[GitHub] somejfn commented on issue #2984: kvm: reset KVM host on heartbeat failure

2018-10-30 Thread GitBox
somejfn commented on issue #2984: kvm: reset KVM host on heartbeat failure URL: https://github.com/apache/cloudstack/pull/2984#issuecomment-434282987 @wido Allow me to disagree. If a transient NFS issue happens and result in mass reset of KVM hosts after 5 minutes, this creates way more

[GitHub] somejfn commented on issue #2890: KVMHAMonitor thread blocks indefinitely while NFS not available

2018-10-30 Thread GitBox
somejfn commented on issue #2890: KVMHAMonitor thread blocks indefinitely while NFS not available URL: https://github.com/apache/cloudstack/issues/2890#issuecomment-434282030 If you just shutdown the NFS server, the NFS client will get immediate response (TCP reset) so this is not the

[GitHub] blueorangutan commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-10-30 Thread GitBox
blueorangutan commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-434281036 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] somejfn commented on issue #2722: CLOUDSTACK-10310 Fix KVM reboot on storage issue

2018-10-30 Thread GitBox
somejfn commented on issue #2722: CLOUDSTACK-10310 Fix KVM reboot on storage issue URL: https://github.com/apache/cloudstack/pull/2722#issuecomment-434280872 On problem is while NFS is unavailable, you wont be able to destroy the VM libvirt will just hang. So if you attempt to

[GitHub] rhtyd commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-10-30 Thread GitBox
rhtyd commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-434280814 @blueorangutan package This is an automated message from the Apache

[GitHub] rhtyd commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-10-30 Thread GitBox
rhtyd commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-434280623 @blueorangutan test This is

[GitHub] somejfn commented on issue #2722: CLOUDSTACK-10310 Fix KVM reboot on storage issue

2018-10-30 Thread GitBox
somejfn commented on issue #2722: CLOUDSTACK-10310 Fix KVM reboot on storage issue URL: https://github.com/apache/cloudstack/pull/2722#issuecomment-434280034 On precision about #2 With primary storage on NFS hard mounts VMs don't go read-only (tested with OL5/6/7) and will resume

[GitHub] blueorangutan commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-10-30 Thread GitBox
blueorangutan commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-434279450 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-2395 This is

[GitHub] blueorangutan commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-10-30 Thread GitBox
blueorangutan commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-434279403 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2394

[GitHub] blueorangutan commented on issue #2980: [4.11] Fix set initial reservation on public IP ranges

2018-10-30 Thread GitBox
blueorangutan commented on issue #2980: [4.11] Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2980#issuecomment-434270479 Trillian test result (tid-3129) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time

[GitHub] blueorangutan commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-10-30 Thread GitBox
blueorangutan commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-434269409 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2893: ApiServer: signature v3 to accept more formats

2018-10-30 Thread GitBox
rhtyd commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-434269162 @blueorangutan package This is an automated message from the Apache

[GitHub] rhtyd commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-10-30 Thread GitBox
rhtyd commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-434269031 @blueorangutan package This

[GitHub] blueorangutan commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-10-30 Thread GitBox
blueorangutan commented on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-434269134 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make

[cloudstack] 01/01: Updating pom.xml version numbers for release 4.11.2.0

2018-10-30 Thread paul_a
This is an automated email from the ASF dual-hosted git repository. paul_a pushed a commit to branch 4.11.2.0-RC20181030T1040 in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 840ad40017612e169665fa799a6d31a23ecad347 Author: Paul Angus AuthorDate: Tue Oct 30 10:40:05 2018

[cloudstack] branch 4.11.2.0-RC20181030T1040 created (now 840ad40)

2018-10-30 Thread paul_a
This is an automated email from the ASF dual-hosted git repository. paul_a pushed a change to branch 4.11.2.0-RC20181030T1040 in repository https://gitbox.apache.org/repos/asf/cloudstack.git. at 840ad40 Updating pom.xml version numbers for release 4.11.2.0 This branch includes the

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

2018-10-30 Thread GitBox
rhtyd commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-434264440 @blueorangutan test centos7 xenserver-71 This is an automated message from the Apache

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

2018-10-30 Thread GitBox
blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-434264523 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + xenserver-71) has been kicked to run smoke tests

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

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

svn commit: r30522 - in /dev/cloudstack/4.11.2.0: apache-cloudstack-4.11.2.0-src.tar.bz2 apache-cloudstack-4.11.2.0-src.tar.bz2.asc apache-cloudstack-4.11.2.0-src.tar.bz2.md5 apache-cloudstack-4.11.2.

2018-10-30 Thread paul_a
Author: paul_a Date: Tue Oct 30 10:40:54 2018 New Revision: 30522 Log: Committing release candidate artifacts for 4.11.2.0 to dist/dev/cloudstack in preparation for release vote Added: dev/cloudstack/4.11.2.0/apache-cloudstack-4.11.2.0-src.tar.bz2 (with props)

[cloudstack] branch master updated (3101612 -> 7479e28)

2018-10-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 3101612 docker: add compose file for database (#2898) add c6e53f6 kvm: reset KVM host on heartbeat failure

[cloudstack] 01/01: Merge remote-tracking branch 'origin/4.11'

2018-10-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 commit 7479e2877fad4f9ad86147bb7edbee9f58116661 Merge: 3101612 c6e53f6 Author: Rohit Yadav AuthorDate: Tue Oct 30 15:15:35

[GitHub] blueorangutan commented on issue #2984: kvm: reset KVM host on heartbeat failure

2018-10-30 Thread GitBox
blueorangutan commented on issue #2984: kvm: reset KVM host on heartbeat failure URL: https://github.com/apache/cloudstack/pull/2984#issuecomment-434234323 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd closed pull request #2984: kvm: reset KVM host on heartbeat failure

2018-10-30 Thread GitBox
rhtyd closed pull request #2984: kvm: reset KVM host on heartbeat failure URL: https://github.com/apache/cloudstack/pull/2984 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

[cloudstack] branch 4.11 updated: kvm: reset KVM host on heartbeat failure (#2984)

2018-10-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 c6e53f6 kvm: reset KVM host on heartbeat

[GitHub] rhtyd commented on issue #2984: kvm: reset KVM host on heartbeat failure

2018-10-30 Thread GitBox
rhtyd commented on issue #2984: kvm: reset KVM host on heartbeat failure URL: https://github.com/apache/cloudstack/pull/2984#issuecomment-434234201 Testing may not be needed as the changes are not covered by smoketests. Manually tested. Travis is green, however I'll kick test as an

[GitHub] PaulAngus commented on issue #2722: CLOUDSTACK-10310 Fix KVM reboot on storage issue

2018-10-30 Thread GitBox
PaulAngus commented on issue #2722: CLOUDSTACK-10310 Fix KVM reboot on storage issue URL: https://github.com/apache/cloudstack/pull/2722#issuecomment-434230839 IMHO i'd say if a VM on that storage is marked as ha-enabled, it should be powered-off and restarted somewhere else, and if it

[GitHub] PaulAngus commented on issue #2722: CLOUDSTACK-10310 Fix KVM reboot on storage issue

2018-10-30 Thread GitBox
PaulAngus commented on issue #2722: CLOUDSTACK-10310 Fix KVM reboot on storage issue URL: https://github.com/apache/cloudstack/pull/2722#issuecomment-434229093 I'll try to summarise the scenario so that we're all trying to fix the same thing... 1. A host cannot write to one of it's

[GitHub] wido commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination

2018-10-30 Thread GitBox
wido commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#discussion_r229227969 ## File path:

[GitHub] wido commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination

2018-10-30 Thread GitBox
wido commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#discussion_r229227013 ## File path:

[GitHub] wido commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination

2018-10-30 Thread GitBox
wido commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#discussion_r229227154 ## File path: engine/schema/src/main/resources/META-INF/db/schema-41120to41200.sql

[GitHub] wido commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination

2018-10-30 Thread GitBox
wido commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#discussion_r229228131 ## File path:

[GitHub] blueorangutan commented on issue #2984: kvm: reset KVM host on heartbeat failure

2018-10-30 Thread GitBox
blueorangutan commented on issue #2984: kvm: reset KVM host on heartbeat failure URL: https://github.com/apache/cloudstack/pull/2984#issuecomment-434224827 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2393 This is an

[GitHub] rhtyd commented on issue #2890: KVMHAMonitor thread blocks indefinitely while NFS not available

2018-10-30 Thread GitBox
rhtyd commented on issue #2890: KVMHAMonitor thread blocks indefinitely while NFS not available URL: https://github.com/apache/cloudstack/issues/2890#issuecomment-434217528 Based on the triaging exercise, I've moved this to 4.11.3.0 as further discussion is pending. I've taken the least

[GitHub] wido commented on issue #2984: kvm: reset KVM host on heartbeat failure

2018-10-30 Thread GitBox
wido commented on issue #2984: kvm: reset KVM host on heartbeat failure URL: https://github.com/apache/cloudstack/pull/2984#issuecomment-434217189 LGTM #2722 should have never been merged. If the host has a problem, it has to kill itself to prevent these weird issues and disk

[GitHub] blueorangutan commented on issue #2984: kvm: reset KVM host on heartbeat failure

2018-10-30 Thread GitBox
blueorangutan commented on issue #2984: kvm: reset KVM host on heartbeat failure URL: https://github.com/apache/cloudstack/pull/2984#issuecomment-434215976 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2984: kvm: reset KVM host on heartbeat failure

2018-10-30 Thread GitBox
rhtyd commented on issue #2984: kvm: reset KVM host on heartbeat failure URL: https://github.com/apache/cloudstack/pull/2984#issuecomment-434215759 @blueorangutan package This is an automated message from the Apache Git

[GitHub] rhtyd opened a new pull request #2984: kvm: reset KVM host on heartbeat failure

2018-10-30 Thread GitBox
rhtyd opened a new pull request #2984: kvm: reset KVM host on heartbeat failure URL: https://github.com/apache/cloudstack/pull/2984 On actual testing, I could see that kvmheartbeat.sh script fails on NFS server failure and stops the agent only. Any HA VMs could be launched in

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

2018-10-30 Thread GitBox
rhtyd commented on issue #2722: CLOUDSTACK-10310 Fix KVM reboot on storage issue URL: https://github.com/apache/cloudstack/pull/2722#issuecomment-434213795 I should n't have taken word on this, I tested and could not reproduce the behaviour @csquire reported. I think we should revert the

[GitHub] rhtyd edited a comment on issue #2890: KVMHAMonitor thread blocks indefinitely while NFS not available

2018-10-30 Thread GitBox
rhtyd edited a comment on issue #2890: KVMHAMonitor thread blocks indefinitely while NFS not available URL: https://github.com/apache/cloudstack/issues/2890#issuecomment-434207830 @csquire I could not reproduce the issue. I guess it depends on how you're simulating NFS/shared-storage

[GitHub] rhtyd commented on issue #2890: KVMHAMonitor thread blocks indefinitely while NFS not available

2018-10-30 Thread GitBox
rhtyd commented on issue #2890: KVMHAMonitor thread blocks indefinitely while NFS not available URL: https://github.com/apache/cloudstack/issues/2890#issuecomment-434207830 @csquire I could not reproduce the issue. I guess it depends on how you're simulating NFS/shared-storage failure and

[GitHub] DaanHoogland commented on issue #2982: Why are passwords encrypted with the SSH public key?

2018-10-30 Thread GitBox
DaanHoogland commented on issue #2982: Why are passwords encrypted with the SSH public key? URL: https://github.com/apache/cloudstack/issues/2982#issuecomment-43413 Good question @Overv , obviously someone considered it a piece of sensitive information, but I have no idea why. I

[GitHub] rhtyd commented on issue #2684: Host metrics show wrong total memory and memory allocation

2018-10-30 Thread GitBox
rhtyd commented on issue #2684: Host metrics show wrong total memory and memory allocation URL: https://github.com/apache/cloudstack/issues/2684#issuecomment-434195005 Can you comment if you can reproduce this with 4.11 @GabrielBrascher @Overv ?

[GitHub] DaanHoogland commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination

2018-10-30 Thread GitBox
DaanHoogland commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#discussion_r229193941 ## File path:

[GitHub] DaanHoogland commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination

2018-10-30 Thread GitBox
DaanHoogland commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#discussion_r229193235 ## File path:

[GitHub] DaanHoogland commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination

2018-10-30 Thread GitBox
DaanHoogland commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#discussion_r229192372 ## File path:

[GitHub] DaanHoogland commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination

2018-10-30 Thread GitBox
DaanHoogland commented on a change in pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983#discussion_r229192109 ## File path:

[GitHub] rhtyd commented on issue #2981: Fix set initial reservation on public IP ranges

2018-10-30 Thread GitBox
rhtyd commented on issue #2981: Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2981#issuecomment-434189252 If there is already a 4.11 based PR, this can be closed @nvazquez

[GitHub] DaanHoogland commented on issue #2981: Fix set initial reservation on public IP ranges

2018-10-30 Thread GitBox
DaanHoogland commented on issue #2981: Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2981#issuecomment-434189009 #2980 fixes this for 4.11. we should not merge this as it will be merged forward from there.

[GitHub] DaanHoogland commented on a change in pull request #2980: [4.11] Fix set initial reservation on public IP ranges

2018-10-30 Thread GitBox
DaanHoogland commented on a change in pull request #2980: [4.11] Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2980#discussion_r229189353 ## File path: server/src/com/cloud/configuration/ConfigurationManagerImpl.java ## @@