[GitHub] nvazquez opened a new pull request #2983: KVM live storage migration intra cluster from NFS source and destination

nvazquez opened a new pull request #2983: KVM live storage migration intra cluster from NFS source and destination URL: https://github.com/apache/cloudstack/pull/2983 ## Description Live storage migration on KVM under these conditions: - From source and destination hosts within the

[GitHub] bdonnahue commented on issue #2583: CPU cores per socket setting not functional

bdonnahue commented on issue #2583: CPU cores per socket setting not functional URL: https://github.com/apache/cloudstack/issues/2583#issuecomment-434037326 Sounds good, is there a scheduled release date @rhtyd ? This is an

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

svenvogel commented on issue #2981: Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2981#issuecomment-434119305 Hi @nvazquez wow great.. i will test fastly and done back greets Sven

[GitHub] DaanHoogland commented on issue #2793: Destroyvm also removes volumes

DaanHoogland commented on issue #2793: Destroyvm also removes volumes URL: https://github.com/apache/cloudstack/pull/2793#issuecomment-433843707 @blueorangutan package This is an automated message from the Apache Git Service.

[GitHub] rhtyd closed pull request #2977: Defer was broken in VR because of json name change. Fix if statement.

rhtyd closed pull request #2977: Defer was broken in VR because of json name change. Fix if statement. URL: https://github.com/apache/cloudstack/pull/2977 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 opened a new pull request #2979: vr: defer was broken in VR because of json name change

rhtyd opened a new pull request #2979: vr: defer was broken in VR because of json name change URL: https://github.com/apache/cloudstack/pull/2979 After upgrade from CS 4.10 to CS 4.11, multiple VRs did not start through. It did not properly defer the finalize config in update_config.py.

[GitHub] wido commented on issue #2977: Defer was broken in VR because of json name change. Fix if statement.

wido commented on issue #2977: Defer was broken in VR because of json name change. Fix if statement. URL: https://github.com/apache/cloudstack/pull/2977#issuecomment-433847602 @rhtyd Can you tell me why this PR was closed? It's very valid for 4.11.2, the processing of the DHCP data is too

[GitHub] wido commented on issue #2978: Router aggregate timeout does not seem to be honored

wido commented on issue #2978: Router aggregate timeout does not seem to be honored URL: https://github.com/apache/cloudstack/issues/2978#issuecomment-433847846 Seee #2979 , I was too fast with typing :) This is an automated

[GitHub] rhtyd commented on issue #2977: Defer was broken in VR because of json name change. Fix if statement.

rhtyd commented on issue #2977: Defer was broken in VR because of json name change. Fix if statement. URL: https://github.com/apache/cloudstack/pull/2977#issuecomment-433847690 oops sorry @RPDiep I was trying to fix the change for you, accidentally overwrote to your master branch with

[GitHub] DaanHoogland commented on a change in pull request #2919: vmware: updateVmwareDc API for updating vmware datacenter details

DaanHoogland commented on a change in pull request #2919: vmware: updateVmwareDc API for updating vmware datacenter details URL: https://github.com/apache/cloudstack/pull/2919#discussion_r228869910 ## File path:

[GitHub] rhtyd closed pull request #2928: Migrating VM from ISO failures

rhtyd closed pull request #2928: Migrating VM from ISO failures URL: https://github.com/apache/cloudstack/pull/2928 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

[cloudstack] branch 4.11 updated: kvm: Fix migrating VM from ISO failures (#2928)

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 dffb430 kvm: Fix migrating VM from ISO failures

[cloudstack] branch 4.11 updated: vr: defer was broken in VR because of json name change

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 f7bc580 vr: defer was broken in VR because of

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

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_r228012074 ## File path: ui/scripts/domains.js ## @@ -69,11

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

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_r228012074 ## File path: ui/scripts/domains.js ## @@ -69,11

[GitHub] svenvogel commented on issue #2929: add public IP range to domain fails in the inital setup (UI Problem)

svenvogel commented on issue #2929: add public IP range to domain fails in the inital setup (UI Problem) URL: https://github.com/apache/cloudstack/issues/2929#issuecomment-433838767 @nvazquez Can you check this? This is an

[GitHub] DennisKonrad commented on issue #2930: HA for Management Server - roundrobin: Certificate ownership

DennisKonrad commented on issue #2930: HA for Management Server - roundrobin: Certificate ownership URL: https://github.com/apache/cloudstack/issues/2930#issuecomment-433840933 @rhtyd So, unfortunately we are not able to test this on a 4.11 setup. The setup is kvm. Anything more

[GitHub] rhtyd commented on issue #2979: vr: defer was broken in VR because of json name change

rhtyd commented on issue #2979: vr: defer was broken in VR because of json name change URL: https://github.com/apache/cloudstack/pull/2979#issuecomment-433848347 For reference, this was authored by @RPDiep and he will not lose his authorship on the commit. We can use my review/lgtm from

[GitHub] DaanHoogland commented on a change in pull request #2919: vmware: updateVmwareDc API for updating vmware datacenter details

DaanHoogland commented on a change in pull request #2919: vmware: updateVmwareDc API for updating vmware datacenter details URL: https://github.com/apache/cloudstack/pull/2919#discussion_r228855906 ## File path:

[GitHub] DaanHoogland commented on a change in pull request #2919: vmware: updateVmwareDc API for updating vmware datacenter details

DaanHoogland commented on a change in pull request #2919: vmware: updateVmwareDc API for updating vmware datacenter details URL: https://github.com/apache/cloudstack/pull/2919#discussion_r228855574 ## File path:

[GitHub] rhtyd commented on issue #2977: Defer was broken in VR because of json name change. Fix if statement.

rhtyd commented on issue #2977: Defer was broken in VR because of json name change. Fix if statement. URL: https://github.com/apache/cloudstack/pull/2977#issuecomment-433848037 @wido sorry, my bad. I was trying to rebase but made a mistake and github closed it as I force pushed on his

[GitHub] rafaelweingartner commented on a change in pull request #1448: CLOUDSTACK-9315: Removed unused Classes

rafaelweingartner commented on a change in pull request #1448: CLOUDSTACK-9315: Removed unused Classes URL: https://github.com/apache/cloudstack/pull/1448#discussion_r228880867 ## File path: engine/schema/src/main/resources/META-INF/db/schema-41120to41200.sql ## @@ -34,4

[cloudstack] branch master updated (8738ca7 -> 323d381)

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 8738ca7 kvm: use libvirtd as service name for all distributions (#2925) add 9cf57d2 network: on rolling

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

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-433879641 @rhtyd look good? This is

[GitHub] rhtyd commented on a change in pull request #1448: CLOUDSTACK-9315: Removed unused Classes

rhtyd commented on a change in pull request #1448: CLOUDSTACK-9315: Removed unused Classes URL: https://github.com/apache/cloudstack/pull/1448#discussion_r228812528 ## File path: engine/schema/src/main/resources/META-INF/db/schema-41120to41200.sql ## @@ -34,4 +34,7 @@

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

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-433802857 @svenvogel can you initiate the variable `distro` you're using (perhaps to None or negative?),

[GitHub] rhtyd commented on a change in pull request #1448: CLOUDSTACK-9315: Removed unused Classes

rhtyd commented on a change in pull request #1448: CLOUDSTACK-9315: Removed unused Classes URL: https://github.com/apache/cloudstack/pull/1448#discussion_r228812528 ## File path: engine/schema/src/main/resources/META-INF/db/schema-41120to41200.sql ## @@ -34,4 +34,7 @@

[GitHub] rhtyd commented on a change in pull request #2977: Defer was broken in VR because of json name change. Fix if statement.

rhtyd commented on a change in pull request #2977: Defer was broken in VR because of json name change. Fix if statement. URL: https://github.com/apache/cloudstack/pull/2977#discussion_r228817465 ## File path: systemvm/debian/opt/cloud/bin/update_config.py ## @@ -52,7

[GitHub] rhtyd commented on issue #2977: Defer was broken in VR because of json name change. Fix if statement.

rhtyd commented on issue #2977: Defer was broken in VR because of json name change. Fix if statement. URL: https://github.com/apache/cloudstack/pull/2977#issuecomment-433808773 Ping @RPDiep @wido @GabrielBrascher we would want to cut RC4 soon, it would be great to include this bugfix

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

borisstoyanov edited a comment on issue #2890: KVMHAMonitor thread blocks indefinitely while NFS not available URL: https://github.com/apache/cloudstack/issues/2890#issuecomment-433810696 hi @csquire @somejfn, thanks for this issue! I think it's correct that the host goes into

[GitHub] wido commented on issue #2977: Defer was broken in VR because of json name change. Fix if statement.

wido commented on issue #2977: Defer was broken in VR because of json name change. Fix if statement. URL: https://github.com/apache/cloudstack/pull/2977#issuecomment-433811156 @rhtyd At the moment @RPDiep is offline after a long night of debugging this issue. I will ask him to

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

borisstoyanov commented on issue #2890: KVMHAMonitor thread blocks indefinitely while NFS not available URL: https://github.com/apache/cloudstack/issues/2890#issuecomment-433810696 hi @csquire @somejfn, thanks for this issue! I think it's correct that the host goes into 'Down'

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

svenvogel edited a comment on issue #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896#issuecomment-433834200 hi @rhtyd yes i can do that... let me check, normally it should not have no value

[GitHub] rhtyd commented on a change in pull request #2919: vmware: updateVmwareDc API for updating vmware datacenter details

rhtyd commented on a change in pull request #2919: vmware: updateVmwareDc API for updating vmware datacenter details URL: https://github.com/apache/cloudstack/pull/2919#discussion_r228861062 ## File path:

[GitHub] DennisKonrad commented on issue #2930: HA for Management Server - roundrobin: Certificate ownership

DennisKonrad commented on issue #2930: HA for Management Server - roundrobin: Certificate ownership URL: https://github.com/apache/cloudstack/issues/2930#issuecomment-433863948 @rhtyd Ok, so are the certificates regenerated for the primary management server when adding a secondary

[cloudstack] branch 4.11 updated: vr: defer was broken in VR because of json name change (#2979)

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 f0491d5 vr: defer was broken in VR because of

[GitHub] rhtyd closed pull request #2979: vr: defer was broken in VR because of json name change

rhtyd closed pull request #2979: vr: defer was broken in VR because of json name change URL: https://github.com/apache/cloudstack/pull/2979 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

[GitHub] rafaelweingartner closed pull request #1448: CLOUDSTACK-9315: Removed unused Classes

rafaelweingartner closed pull request #1448: CLOUDSTACK-9315: Removed unused Classes URL: https://github.com/apache/cloudstack/pull/1448 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

[cloudstack] branch master updated: CLOUDSTACK-9315: Removed unused Classes (#1448)

This is an automated email from the ASF dual-hosted git repository. rafael 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 cdc6e6e CLOUDSTACK-9315: Removed unused

[GitHub] nvazquez commented on issue #2929: add public IP range to domain fails in the inital setup (UI Problem)

nvazquez commented on issue #2929: add public IP range to domain fails in the inital setup (UI Problem) URL: https://github.com/apache/cloudstack/issues/2929#issuecomment-433879195 @svenvogel @rhtyd sure, I'll be working on this one and will update you soon

[GitHub] rhtyd commented on a change in pull request #2919: vmware: updateVmwareDc API for updating vmware datacenter details

rhtyd commented on a change in pull request #2919: vmware: updateVmwareDc API for updating vmware datacenter details URL: https://github.com/apache/cloudstack/pull/2919#discussion_r228892302 ## File path:

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

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-433880394 Thanks @svenvogel I'll test this soon (our labs are currently under maintenance).

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

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-433834200 hi @rhtyd yes i can do that... let me check

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

DaanHoogland commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-433844508 @blueorangutan package This is an automated message from the

[GitHub] rhtyd commented on issue #2978: Router aggregate timeout does not seem to be honored

rhtyd commented on issue #2978: Router aggregate timeout does not seem to be honored URL: https://github.com/apache/cloudstack/issues/2978#issuecomment-433849364 The fix shared by @RPDiep LGTM @wido This is an automated

[GitHub] rhtyd commented on issue #2928: Migrating VM from ISO failures

rhtyd commented on issue #2928: Migrating VM from ISO failures URL: https://github.com/apache/cloudstack/pull/2928#issuecomment-433855394 Yes the second case is indeed around direct download feature, @nvazquez can you comment?

[GitHub] rhtyd closed issue #2978: Router aggregate timeout does not seem to be honored

rhtyd closed issue #2978: Router aggregate timeout does not seem to be honored URL: https://github.com/apache/cloudstack/issues/2978 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] rhtyd commented on issue #2978: Router aggregate timeout does not seem to be honored

rhtyd commented on issue #2978: Router aggregate timeout does not seem to be honored URL: https://github.com/apache/cloudstack/issues/2978#issuecomment-433864165 https://github.com/apache/cloudstack/pull/2979 has been merged, closing this issue. /cc @wido

[GitHub] rhtyd edited a comment on issue #2978: Router aggregate timeout does not seem to be honored

rhtyd edited a comment on issue #2978: Router aggregate timeout does not seem to be honored URL: https://github.com/apache/cloudstack/issues/2978#issuecomment-433864165 https://github.com/apache/cloudstack/pull/2979 has been merged, closing this issue. /cc @wido please re-open if

[GitHub] rhtyd commented on issue #2930: HA for Management Server - roundrobin: Certificate ownership

rhtyd commented on issue #2930: HA for Management Server - roundrobin: Certificate ownership URL: https://github.com/apache/cloudstack/issues/2930#issuecomment-433872021 @DennisKonrad I've already replied to you how cert generation works, tl;dr - each mgmt server generates its own cert on

[GitHub] rhtyd commented on issue #2979: vr: defer was broken in VR because of json name change

rhtyd commented on issue #2979: vr: defer was broken in VR because of json name change URL: https://github.com/apache/cloudstack/pull/2979#issuecomment-433848826 Our lab is temporary under maintenance, I'll tested this locally in a [monkeybox](https://github.com/rhtyd/monkeybox)+KVM based

[GitHub] rhtyd commented on issue #2930: HA for Management Server - roundrobin: Certificate ownership

rhtyd commented on issue #2930: HA for Management Server - roundrobin: Certificate ownership URL: https://github.com/apache/cloudstack/issues/2930#issuecomment-433854973 @DennisKonrad thanks. When a new management server starts, it first upgrades the DB and then various

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

rhtyd commented on issue #2893: ApiServer: signature v3 to accept more formats URL: https://github.com/apache/cloudstack/pull/2893#issuecomment-433855135 @DaanHoogland sorry lab's down, BO queue will be stuck for a while.

[GitHub] wido opened a new issue #2978: Router aggregate timeout does not seem to be honored

wido opened a new issue #2978: Router aggregate timeout does not seem to be honored URL: https://github.com/apache/cloudstack/issues/2978 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ Virtual Router ~~~ # CLOUDSTACK VERSION ~~~ 4.11.1 ~~~

[GitHub] RPDiep opened a new pull request #2977: Defer was broken in VR because of json name change. Fix if statement.

RPDiep opened a new pull request #2977: Defer was broken in VR because of json name change. Fix if statement. URL: https://github.com/apache/cloudstack/pull/2977 ## Description After upgrade from CS 4.10 to CS 4.11, multiple VRs did not start through. It did not properly defer

[GitHub] rhtyd commented on issue #2928: Migrating VM from ISO failures

rhtyd commented on issue #2928: Migrating VM from ISO failures URL: https://github.com/apache/cloudstack/pull/2928#issuecomment-433801221 Test LGTM, can you review - @rafaelweingartner @wido @GabrielBrascher @nvazquez thanks.

[GitHub] wido commented on issue #2928: Migrating VM from ISO failures

wido commented on issue #2928: Migrating VM from ISO failures URL: https://github.com/apache/cloudstack/pull/2928#issuecomment-433813641 If I read the code correctly and what I'm missing in the description is that this only happens with VMs which were deployed using Direct Downloaded

[GitHub] rhtyd closed pull request #2925: kvm: use libvirtd as service name for all distributions

rhtyd closed pull request #2925: kvm: use libvirtd as service name for all distributions URL: https://github.com/apache/cloudstack/pull/2925 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

[GitHub] rhtyd closed issue #2909: debian does not have libvirt-bin it has libvirt

rhtyd closed issue #2909: debian does not have libvirt-bin it has libvirt URL: https://github.com/apache/cloudstack/issues/2909 This is an automated message from the Apache Git Service. To respond to the message, please log

[cloudstack] branch master updated: kvm: use libvirtd as service name for all distributions (#2925)

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 8738ca7 kvm: use libvirtd as service name

[GitHub] rhtyd commented on issue #2977: Defer was broken in VR because of json name change. Fix if statement.

rhtyd commented on issue #2977: Defer was broken in VR because of json name change. Fix if statement. URL: https://github.com/apache/cloudstack/pull/2977#issuecomment-433800860 Thanks @RPDiep can you change the branch to 4.11 and rebase your branch against 4.11?

[GitHub] rhtyd commented on issue #2583: CPU cores per socket setting not functional

rhtyd commented on issue #2583: CPU cores per socket setting not functional URL: https://github.com/apache/cloudstack/issues/2583#issuecomment-433802974 Let's revisit this after 4.11.2.0 gets released. /cc @DagSonsteboSB @PaulAngus

[GitHub] rhtyd commented on issue #2930: HA for Management Server - roundrobin: Certificate ownership

rhtyd commented on issue #2930: HA for Management Server - roundrobin: Certificate ownership URL: https://github.com/apache/cloudstack/issues/2930#issuecomment-434166435 Given this is a setup /env issue, if the advise process fixes your issue and you're unable to reproduce the errors

[GitHub] rhtyd closed issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs

rhtyd closed issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs URL: https://github.com/apache/cloudstack/issues/2880 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] rhtyd commented on issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs

rhtyd commented on issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs URL: https://github.com/apache/cloudstack/issues/2880#issuecomment-434166602 I'll close this issue based on confirmation from @resmo, the poweroff PR has been merged as well. Please reopen if you can

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

rhtyd commented on issue #2980: [4.11] Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2980#issuecomment-434166014 @blueorangutan package This is an automated message from the

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

blueorangutan commented on issue #2980: [4.11] Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2980#issuecomment-434166136 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

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

rhtyd commented on issue #2980: [4.11] Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2980#issuecomment-434170646 @blueorangutan test This is an automated message from the

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

blueorangutan commented on issue #2980: [4.11] Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2980#issuecomment-434170694 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

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

blueorangutan commented on issue #2980: [4.11] Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2980#issuecomment-434170547 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2391

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

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

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

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

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

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

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

blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-434175624 @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

blueorangutan commented on issue #2376: [4.11] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2376#issuecomment-434180214 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2392 This is an automated

[GitHub] DaanHoogland commented on a change in pull request #2919: vmware: updateVmwareDc API for updating vmware datacenter details

DaanHoogland commented on a change in pull request #2919: vmware: updateVmwareDc API for updating vmware datacenter details URL: https://github.com/apache/cloudstack/pull/2919#discussion_r228898339 ## File path:

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

somejfn commented on issue #2890: KVMHAMonitor thread blocks indefinitely while NFS not available URL: https://github.com/apache/cloudstack/issues/2890#issuecomment-433911908 With NFS not available and since those are hard mounts, even a "virsh destroy" would not work. Libvirtd will

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

borisstoyanov commented on issue #2890: KVMHAMonitor thread blocks indefinitely while NFS not available URL: https://github.com/apache/cloudstack/issues/2890#issuecomment-433906239 I think the leanest way to fence the resource would be, prior to setting the host down to iterate all it's

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

somejfn commented on issue #2890: KVMHAMonitor thread blocks indefinitely while NFS not available URL: https://github.com/apache/cloudstack/issues/2890#issuecomment-433896656 Correct. Only VM-HA enabled would get restarted and create a duplicate when the host goes down. Still, I

[GitHub] blueorangutan commented on issue #2793: Destroyvm also removes volumes

blueorangutan commented on issue #2793: Destroyvm also removes volumes URL: https://github.com/apache/cloudstack/pull/2793#issuecomment-433908870 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2979: vr: defer was broken in VR because of json name change

rhtyd commented on issue #2979: vr: defer was broken in VR because of json name change URL: https://github.com/apache/cloudstack/pull/2979#issuecomment-433965407 Alright, no need of a new PR. In some cases the path may be absolute, but I think we've moved to relative paths now in most

[GitHub] rhtyd commented on issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs

rhtyd commented on issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs URL: https://github.com/apache/cloudstack/issues/2880#issuecomment-433966732 Thanks for sharing @resmo, as @PaulAngus replied the power off issue during rolling restart has been fixed now as well.

[GitHub] andrijapanic commented on issue #2820: template downloading fails

andrijapanic commented on issue #2820: template downloading fails URL: https://github.com/apache/cloudstack/issues/2820#issuecomment-433974616 Issue still present in latest 4.11.2 RC3 for RAW - stream closed message. This is

[GitHub] RPDiep commented on issue #2979: vr: defer was broken in VR because of json name change

RPDiep commented on issue #2979: vr: defer was broken in VR because of json name change URL: https://github.com/apache/cloudstack/pull/2979#issuecomment-433924280 No problem on the new PR. I just read that @rhtyd preferred that the line would use `startswith`? I think that's indeed

[GitHub] PaulAngus commented on issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs

PaulAngus commented on issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs URL: https://github.com/apache/cloudstack/issues/2880#issuecomment-433937889 @resmo i believe Rohit corrected that here: https://github.com/apache/cloudstack/pull/2926

[GitHub] rhtyd closed pull request #2898: docker: add compose file for database

rhtyd closed pull request #2898: docker: add compose file for database URL: https://github.com/apache/cloudstack/pull/2898 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

[cloudstack] branch master updated: docker: add compose file for database (#2898)

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 3101612 docker: add compose file for

[GitHub] resmo edited a comment on issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs

resmo edited a comment on issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs URL: https://github.com/apache/cloudstack/issues/2880#issuecomment-433921820 @rhtyd Did set the `force top on destroy` to `true` and was not able to reproduce anymore, however, still seeing

[GitHub] wido commented on issue #2979: vr: defer was broken in VR because of json name change

wido commented on issue #2979: vr: defer was broken in VR because of json name change URL: https://github.com/apache/cloudstack/pull/2979#issuecomment-433932184 I doubt that *startswith* is the best method. We are now just providing a filename, but if somebody suddenly decides to change

[GitHub] nvazquez opened a new pull request #2980: Iprangedomainuifix

nvazquez opened a new pull request #2980: Iprangedomainuifix URL: https://github.com/apache/cloudstack/pull/2980 ## Description Fix initial reservation on public IP range addition. Also, if the range is dedicated for system VMs do not allow dedicating it to any account Fixes:

[GitHub] resmo commented on issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs

resmo commented on issue #2880: [4.11.2]: rolling upgrade of VR results in duplicate VRs URL: https://github.com/apache/cloudstack/issues/2880#issuecomment-433921820 * Did set the `force top on destroy` to `true` and was not able to reproduce anymore, however, still seeing "initiate guest

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

nvazquez commented on issue #2981: Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2981#issuecomment-433950581 Hi @svenvogel can you please test it? This is an automated message

[GitHub] nvazquez opened a new pull request #2981: Fix set initial reservation on public IP ranges

nvazquez opened a new pull request #2981: Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2981 ## Description Fix initial reservation on public IP range addition. Also, if the range is dedicated for system VMs do not allow dedicating it to

[GitHub] Overv opened a new issue #2982: Why are passwords encrypted with the SSH public key?

Overv opened a new issue #2982: Why are passwords encrypted with the SSH public key? URL: https://github.com/apache/cloudstack/issues/2982 This function is used to encrypt and store the VM password, but why is this done with the SSH public key?