[GitHub] [cloudstack] blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559614190 Trillian test result (tid-563) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 3

[GitHub] [cloudstack] blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559608050 Trillian test result (tid-564) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 2

[GitHub] [cloudstack] PaulAngus commented on issue #3371: Fix virtual template size for managed storage for KVM / refactor cloud-install-sys-tmplt

2019-11-28 Thread GitBox
PaulAngus commented on issue #3371: Fix virtual template size for managed storage for KVM / refactor cloud-install-sys-tmplt URL: https://github.com/apache/cloudstack/pull/3371#issuecomment-559593896 @svenvogel - My apologies, I've been swamped and this topic got buried... What happens w

[GitHub] [cloudstack] andrijapanicsb edited a comment on issue #3371: Fix virtual template size for managed storage for KVM / refactor cloud-install-sys-tmplt

2019-11-28 Thread GitBox
andrijapanicsb edited a comment on issue #3371: Fix virtual template size for managed storage for KVM / refactor cloud-install-sys-tmplt URL: https://github.com/apache/cloudstack/pull/3371#issuecomment-559590637 @svenvogel , if others agree @weizhouapache @rhtyd - see this PR from Rohit o

[GitHub] [cloudstack] andrijapanicsb commented on issue #3371: Fix virtual template size for managed storage for KVM / refactor cloud-install-sys-tmplt

2019-11-28 Thread GitBox
andrijapanicsb commented on issue #3371: Fix virtual template size for managed storage for KVM / refactor cloud-install-sys-tmplt URL: https://github.com/apache/cloudstack/pull/3371#issuecomment-559590637 @svenvogel , if others agree @weizhouapache @rhtyd - see this PR from Rohit on how p

[GitHub] [cloudstack] svenvogel commented on issue #3371: Fix virtual template size for managed storage for KVM / refactor cloud-install-sys-tmplt

2019-11-28 Thread GitBox
svenvogel commented on issue #3371: Fix virtual template size for managed storage for KVM / refactor cloud-install-sys-tmplt URL: https://github.com/apache/cloudstack/pull/3371#issuecomment-559587932 @ustcweizhou what do you think about the flag or remove it completely? ---

[GitHub] [cloudstack] 85matthew commented on issue #3651: Fix simulator docker db deploy issue (apache#3397)

2019-11-28 Thread GitBox
85matthew commented on issue #3651: Fix simulator docker db deploy issue (apache#3397) URL: https://github.com/apache/cloudstack/pull/3651#issuecomment-559575757 I ran into this issue with the docker container today & this does fix it. Are you able to get this merged for others? @rhtyd

[GitHub] [cloudstack] DaanHoogland opened a new pull request #3725: convert protocal names to be found as labels

2019-11-28 Thread GitBox
DaanHoogland opened a new pull request #3725: convert protocal names to be found as labels URL: https://github.com/apache/cloudstack/pull/3725 ## Description replace significant chars from string to find protocol labels Fixes: #3252 ## Types of c

[GitHub] [cloudstack] blueorangutan commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure

2019-11-28 Thread GitBox
blueorangutan commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure URL: https://github.com/apache/cloudstack/pull/3425#issuecomment-559556914 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-406 --

[cloudstack] branch master updated (e93f6bf -> b8676ae)

2019-11-28 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from e93f6bf utils: use iproute to get default network interface (#3704) add 94474af consoleproxy: Enable console

[GitHub] [cloudstack] anuragaw commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure

2019-11-28 Thread GitBox
anuragaw commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure URL: https://github.com/apache/cloudstack/pull/3425#issuecomment-559551887 @blueorangutan package This is an automated

[GitHub] [cloudstack] blueorangutan commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure

2019-11-28 Thread GitBox
blueorangutan commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure URL: https://github.com/apache/cloudstack/pull/3425#issuecomment-559551929 @anuragaw a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --

[GitHub] [cloudstack] wqmeng commented on issue #3509: 4.12 add host fail , using tls connection

2019-11-28 Thread GitBox
wqmeng commented on issue #3509: 4.12 add host fail ,using tls connection URL: https://github.com/apache/cloudstack/issues/3509#issuecomment-559527268 And in the server side, shows ``` Nov 28 23:01:49 localhost.localdomain java[17220]: WARN [c.c.a.AlertManagerImpl] (AgentConnect

[GitHub] [cloudstack] wqmeng commented on issue #3509: 4.12 add host fail , using tls connection

2019-11-28 Thread GitBox
wqmeng commented on issue #3509: 4.12 add host fail ,using tls connection URL: https://github.com/apache/cloudstack/issues/3509#issuecomment-559525718 I have also tried the cloudstack 4.14.0 in the github, and still get this error. ``` Nov 28 06:56:42 localhost.localdomain java[2614

[GitHub] [cloudstack] DaanHoogland commented on issue #3252: Load balancer protocols bug

2019-11-28 Thread GitBox
DaanHoogland commented on issue #3252: Load balancer protocols bug URL: https://github.com/apache/cloudstack/issues/3252#issuecomment-559521807 right i am; 'tcp.proxy' and 'tcp-proxy' are used interchangably. will create a PR to try and unify ---

[GitHub] [cloudstack] GabrielBrascher edited a comment on issue #3659: Fix typo: the past tense of shutdown is shutdown, not shutdowned

2019-11-28 Thread GitBox
GabrielBrascher edited a comment on issue #3659: Fix typo: the past tense of shutdown is shutdown, not shutdowned URL: https://github.com/apache/cloudstack/pull/3659#issuecomment-559489730 @DaanHoogland Makes sense adding it to the upgrade guide. ---

[GitHub] [cloudstack] DaanHoogland commented on issue #3252: Load balancer protocols bug

2019-11-28 Thread GitBox
DaanHoogland commented on issue #3252: Load balancer protocols bug URL: https://github.com/apache/cloudstack/issues/3252#issuecomment-559514251 Did a quick fix replacing spaces but only half fixed it: https://user-images.githubusercontent.com/2486961/69813960-90114f00-11f3-11ea-93b1-fa37

[GitHub] [cloudstack] weizhouapache commented on issue #3179: Redundant VRouter guest network on wrong interface

2019-11-28 Thread GitBox
weizhouapache commented on issue #3179: Redundant VRouter guest network on wrong interface URL: https://github.com/apache/cloudstack/issues/3179#issuecomment-559507839 @DennisKonrad do you use the network offering created by following http://docs.cloudstack.apache.org/en/latest/plugins

[GitHub] [cloudstack] borisstoyanov commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
borisstoyanov commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559501421 @blueorangutan test This is an automated message from the Ap

[GitHub] [cloudstack] blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559501652 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -

[GitHub] [cloudstack] blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559500090 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-405 This is

[GitHub] [cloudstack] slavkap opened a new pull request #3724: Storage-based Snapshots for KVM VMs

2019-11-28 Thread GitBox
slavkap opened a new pull request #3724: Storage-based Snapshots for KVM VMs URL: https://github.com/apache/cloudstack/pull/3724 ## Description Cloudstack with KVM as a hypervisor provides live VM snapshots only with volumes which image format is QCOW. This is a limitation for st

[GitHub] [cloudstack] blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559494257 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. ---

[GitHub] [cloudstack] blueorangutan removed a comment on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
blueorangutan removed a comment on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559493919 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-404

[GitHub] [cloudstack] borisstoyanov removed a comment on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
borisstoyanov removed a comment on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559493614 @blueorangutan package This is an automated message

[GitHub] [cloudstack] borisstoyanov commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
borisstoyanov commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559494048 @blueorangutan package This is an automated message from the

[GitHub] [cloudstack] blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559493919 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-404 This is

[GitHub] [cloudstack] blueorangutan commented on issue #3723: a conditional to prevent creation of a field

2019-11-28 Thread GitBox
blueorangutan commented on issue #3723: a conditional to prevent creation of a field URL: https://github.com/apache/cloudstack/pull/3723#issuecomment-559493790 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-403 This is an a

[GitHub] [cloudstack] borisstoyanov commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
borisstoyanov commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559493614 @blueorangutan package This is an automated message from the

[GitHub] [cloudstack] DaanHoogland commented on issue #3713: break session only on illegal origin

2019-11-28 Thread GitBox
DaanHoogland commented on issue #3713: break session only on illegal origin URL: https://github.com/apache/cloudstack/pull/3713#issuecomment-559492368 we need to decide on 4.11 and maybe backport This is an automated message f

[GitHub] [cloudstack] DaanHoogland closed pull request #3713: break session only on illegal origin

2019-11-28 Thread GitBox
DaanHoogland closed pull request #3713: break session only on illegal origin URL: https://github.com/apache/cloudstack/pull/3713 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [cloudstack] blueorangutan commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure

2019-11-28 Thread GitBox
blueorangutan commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure URL: https://github.com/apache/cloudstack/pull/3425#issuecomment-559491848 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-402 --

[GitHub] [cloudstack] GabrielBrascher commented on issue #3659: Fix typo: the past tense of shutdown is shutdown, not shutdowned

2019-11-28 Thread GitBox
GabrielBrascher commented on issue #3659: Fix typo: the past tense of shutdown is shutdown, not shutdowned URL: https://github.com/apache/cloudstack/pull/3659#issuecomment-559489730 @DaanHoogland Do you mean adding this change on a release notes? ---

[GitHub] [cloudstack] blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559483783 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. ---

[GitHub] [cloudstack] borisstoyanov commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
borisstoyanov commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559483669 @blueorangutan package This is an automated message from the

[GitHub] [cloudstack] blueorangutan commented on issue #3723: a conditional to prevent creation of a field

2019-11-28 Thread GitBox
blueorangutan commented on issue #3723: a conditional to prevent creation of a field URL: https://github.com/apache/cloudstack/pull/3723#issuecomment-559483476 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. -

[GitHub] [cloudstack] DaanHoogland commented on issue #3723: a conditional to prevent creation of a field

2019-11-28 Thread GitBox
DaanHoogland commented on issue #3723: a conditional to prevent creation of a field URL: https://github.com/apache/cloudstack/pull/3723#issuecomment-559483284 @blueorangutan package This is an automated message from the Apach

[GitHub] [cloudstack] blueorangutan commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure

2019-11-28 Thread GitBox
blueorangutan commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure URL: https://github.com/apache/cloudstack/pull/3425#issuecomment-559481652 @anuragaw a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --

[GitHub] [cloudstack] anuragaw commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure

2019-11-28 Thread GitBox
anuragaw commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure URL: https://github.com/apache/cloudstack/pull/3425#issuecomment-559481564 @blueorangutan package This is an automated

[GitHub] [cloudstack] borisstoyanov commented on a change in pull request #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
borisstoyanov commented on a change in pull request #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#discussion_r351748609 ## File path: test/integration/smoke/test_deploy_vm_extra_config_data.py ## @@ -0,0 +1,529

[GitHub] [cloudstack] blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
blueorangutan commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559464790 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -

[GitHub] [cloudstack] borisstoyanov commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms

2019-11-28 Thread GitBox
borisstoyanov commented on issue #3510: [WIP DO NOT MERGE] Allow additional config to vms URL: https://github.com/apache/cloudstack/pull/3510#issuecomment-559464592 @blueorangutan test This is an automated message from the Ap

[GitHub] [cloudstack] anuragaw commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure

2019-11-28 Thread GitBox
anuragaw commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure URL: https://github.com/apache/cloudstack/pull/3425#issuecomment-559464244 Failing tests don't seem related. This is a

[GitHub] [cloudstack] DaanHoogland commented on issue #3723: a conditional to prevent creation of a field

2019-11-28 Thread GitBox
DaanHoogland commented on issue #3723: a conditional to prevent creation of a field URL: https://github.com/apache/cloudstack/pull/3723#issuecomment-559456169 Not sure why i thought this would work, maybe a copy error in my dev env. -

[GitHub] [cloudstack] andrijapanicsb commented on issue #3721: network: cleanup dhcp/dns entries while remove a nic from vm

2019-11-28 Thread GitBox
andrijapanicsb commented on issue #3721: network: cleanup dhcp/dns entries while remove a nic from vm URL: https://github.com/apache/cloudstack/pull/3721#issuecomment-559452962 I'll be testing this one, keep away from the MERGE button... :) -

[GitHub] [cloudstack] blueorangutan commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure

2019-11-28 Thread GitBox
blueorangutan commented on issue #3425: [WIP DO NOT MERGE] Better tracking host maintanence success and failure URL: https://github.com/apache/cloudstack/pull/3425#issuecomment-559451473 Trillian test result (tid-561) Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7

[GitHub] [cloudstack] andrijapanicsb commented on issue #3716: creating template from snapshot does not work from snapshot list

2019-11-28 Thread GitBox
andrijapanicsb commented on issue #3716: creating template from snapshot does not work from snapshot list URL: https://github.com/apache/cloudstack/issues/3716#issuecomment-559451176 No @DaanHoogland, I was lazy... This is a

[cloudstack] branch 4.13 updated: consoleproxy: Enable console for vms in Stopping/Migrating state (#3640)

2019-11-28 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch 4.13 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.13 by this push: new 94474af consoleproxy: Enable console for vms in S

[GitHub] [cloudstack] DaanHoogland merged pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state

2019-11-28 Thread GitBox
DaanHoogland merged pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state URL: https://github.com/apache/cloudstack/pull/3640 This is an automated message from the Apache Git Service. To respon

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state

2019-11-28 Thread GitBox
DaanHoogland commented on a change in pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state URL: https://github.com/apache/cloudstack/pull/3640#discussion_r351716903 ## File path: server/src/main/java/com/cloud/consoleproxy/ConsoleProxyManagerImpl.jav

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state

2019-11-28 Thread GitBox
DaanHoogland commented on a change in pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state URL: https://github.com/apache/cloudstack/pull/3640#discussion_r351716984 ## File path: server/src/main/java/com/cloud/server/ManagementServerImpl.java ###

[cloudstack-primate] branch master updated: fix lint issue

2019-11-28 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-primate.git The following commit(s) were added to refs/heads/master by this push: new 4c60435 fix lint issue 4c60435 is de

[GitHub] [cloudstack] weizhouapache commented on a change in pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state

2019-11-28 Thread GitBox
weizhouapache commented on a change in pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state URL: https://github.com/apache/cloudstack/pull/3640#discussion_r351713089 ## File path: server/src/main/java/com/cloud/consoleproxy/ConsoleProxyManagerImpl.ja

[GitHub] [cloudstack] weizhouapache commented on a change in pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state

2019-11-28 Thread GitBox
weizhouapache commented on a change in pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state URL: https://github.com/apache/cloudstack/pull/3640#discussion_r351712553 ## File path: server/src/main/java/com/cloud/server/ManagementServerImpl.java ##

[GitHub] [cloudstack] DaanHoogland commented on issue #3669: server: Fix resource count of primary storage/volume because of Expunged volumes

2019-11-28 Thread GitBox
DaanHoogland commented on issue #3669: server: Fix resource count of primary storage/volume because of Expunged volumes URL: https://github.com/apache/cloudstack/pull/3669#issuecomment-559438576 @weizhouapache I am looking at the travis run; once again a maximum log length error. I restart

[GitHub] [cloudstack] blueorangutan commented on issue #3723: a conditional to prevent creation of a field

2019-11-28 Thread GitBox
blueorangutan commented on issue #3723: a conditional to prevent creation of a field URL: https://github.com/apache/cloudstack/pull/3723#issuecomment-559438355 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-401 This is an a

[GitHub] [cloudstack] DaanHoogland commented on issue #3640: consoleproxy: Enable console for vms in Stopping/Migrating state

2019-11-28 Thread GitBox
DaanHoogland commented on issue #3640: consoleproxy: Enable console for vms in Stopping/Migrating state URL: https://github.com/apache/cloudstack/pull/3640#issuecomment-559435975 travis job 5 ran out of log space. this has happened more often recently. no issue concerning this PR afaiac -

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state

2019-11-28 Thread GitBox
DaanHoogland commented on a change in pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state URL: https://github.com/apache/cloudstack/pull/3640#discussion_r351696566 ## File path: server/src/main/java/com/cloud/server/ManagementServerImpl.java ###

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state

2019-11-28 Thread GitBox
DaanHoogland commented on a change in pull request #3640: consoleproxy: Enable console for vms in Stopping/Migrating state URL: https://github.com/apache/cloudstack/pull/3640#discussion_r351694631 ## File path: server/src/main/java/com/cloud/consoleproxy/ConsoleProxyManagerImpl.jav

[GitHub] [cloudstack] andrijapanicsb commented on issue #3723: a conditional to prevent creation of a field

2019-11-28 Thread GitBox
andrijapanicsb commented on issue #3723: a conditional to prevent creation of a field URL: https://github.com/apache/cloudstack/pull/3723#issuecomment-559431247 @blueorangutan package This is an automated message from the Apa

[GitHub] [cloudstack] blueorangutan commented on issue #3723: a conditional to prevent creation of a field

2019-11-28 Thread GitBox
blueorangutan commented on issue #3723: a conditional to prevent creation of a field URL: https://github.com/apache/cloudstack/pull/3723#issuecomment-559431304 @andrijapanicsb a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. ---

[GitHub] [cloudstack] DaanHoogland opened a new pull request #3723: a conditional to prevent creation of a field

2019-11-28 Thread GitBox
DaanHoogland opened a new pull request #3723: a conditional to prevent creation of a field URL: https://github.com/apache/cloudstack/pull/3723 the files is based on volumes and will npe (if that's a thing in javascript) when applied in the snapshot dialog ## Description

[GitHub] [cloudstack] weizhouapache commented on issue #3669: server: Fix resource count of primary storage/volume because of Expunged volumes

2019-11-28 Thread GitBox
weizhouapache commented on issue #3669: server: Fix resource count of primary storage/volume because of Expunged volumes URL: https://github.com/apache/cloudstack/pull/3669#issuecomment-559427537 The PR has two approvals , and BO tests passed. is it ready for merge ? ---

[GitHub] [cloudstack] andrijapanicsb commented on issue #3712: [WORK IN PROGRES - do NOT merge] Replace builtin CentOS 5.x with 8.0 templates

2019-11-28 Thread GitBox
andrijapanicsb commented on issue #3712: [WORK IN PROGRES - do NOT merge] Replace builtin CentOS 5.x with 8.0 templates URL: https://github.com/apache/cloudstack/pull/3712#issuecomment-559425695 That makes sense @rhtyd - just needs some time and effort to be changed - but, again, can be do

[GitHub] [cloudstack] rhtyd edited a comment on issue #3712: [WORK IN PROGRES - do NOT merge] Replace builtin CentOS 5.x with 8.0 templates

2019-11-28 Thread GitBox
rhtyd edited a comment on issue #3712: [WORK IN PROGRES - do NOT merge] Replace builtin CentOS 5.x with 8.0 templates URL: https://github.com/apache/cloudstack/pull/3712#issuecomment-559424865 @andrijapanicsb use something lightweight like alpinelinux or even stripped down debian? (we can

[GitHub] [cloudstack] DaanHoogland commented on issue #3716: creating template from snapshot does not work from snapshot list

2019-11-28 Thread GitBox
DaanHoogland commented on issue #3716: creating template from snapshot does not work from snapshot list URL: https://github.com/apache/cloudstack/issues/3716#issuecomment-559424760 actually @andrijapanicsb half an hour extra search gave me a location to add a conditional, testing is proba

[GitHub] [cloudstack] rhtyd commented on issue #3712: [WORK IN PROGRES - do NOT merge] Replace builtin CentOS 5.x with 8.0 templates

2019-11-28 Thread GitBox
rhtyd commented on issue #3712: [WORK IN PROGRES - do NOT merge] Replace builtin CentOS 5.x with 8.0 templates URL: https://github.com/apache/cloudstack/pull/3712#issuecomment-559424865 @andrijapanicsb use something lightweight like alpinelinux?

[GitHub] [cloudstack] andrijapanicsb edited a comment on issue #3712: [WORK IN PROGRES - do NOT merge] Replace builtin CentOS 5.x with 8.0 templates

2019-11-28 Thread GitBox
andrijapanicsb edited a comment on issue #3712: [WORK IN PROGRES - do NOT merge] Replace builtin CentOS 5.x with 8.0 templates URL: https://github.com/apache/cloudstack/pull/3712#issuecomment-559416200 This one needs to be parked for some time - MANY Marvin tests are failing since we are r

[GitHub] [cloudstack] andrijapanicsb commented on issue #3712: [WORK IN PROGRES - do NOT merge] Replace builtin CentOS 5.x with 8.0 templates

2019-11-28 Thread GitBox
andrijapanicsb commented on issue #3712: [WORK IN PROGRES - do NOT merge] Replace builtin CentOS 5.x with 8.0 templates URL: https://github.com/apache/cloudstack/pull/3712#issuecomment-559416200 This one needs to be parked for some time - MANY Marvin tests are failing since we are relying

[cloudstack-primate] branch master updated: roles: boilerplate stuff

2019-11-28 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-primate.git The following commit(s) were added to refs/heads/master by this push: new 85b6b24 roles: boilerplate stuff 85b

[GitHub] [cloudstack] andrijapanicsb commented on issue #3716: creating template from snapshot does not work from snapshot list

2019-11-28 Thread GitBox
andrijapanicsb commented on issue #3716: creating template from snapshot does not work from snapshot list URL: https://github.com/apache/cloudstack/issues/3716#issuecomment-559413087 sounds good @DaanHoogland I've also seen (probably NOT related, but to mention...) that when you r

[GitHub] [cloudstack] DaanHoogland commented on issue #3716: creating template from snapshot does not work from snapshot list

2019-11-28 Thread GitBox
DaanHoogland commented on issue #3716: creating template from snapshot does not work from snapshot list URL: https://github.com/apache/cloudstack/issues/3716#issuecomment-559410283 during the latest update of jquery the createTemplate functions for volumes and snapshots were unified into

[GitHub] [cloudstack] andrijapanicsb commented on issue #3712: [WORK IN PROGRES] Replace builtin CentOS 5.x with 8.0 templates

2019-11-28 Thread GitBox
andrijapanicsb commented on issue #3712: [WORK IN PROGRES] Replace builtin CentOS 5.x with 8.0 templates URL: https://github.com/apache/cloudstack/pull/3712#issuecomment-559409713 Tested with latest hypervisors we support atm - and all works fine: - VMware 6.7 U3 - XenServer 7.6 -

[GitHub] [cloudstack] weizhouapache commented on issue #3179: Redundant VRouter guest network on wrong interface

2019-11-28 Thread GitBox
weizhouapache commented on issue #3179: Redundant VRouter guest network on wrong interface URL: https://github.com/apache/cloudstack/issues/3179#issuecomment-559398532 @DennisKonrad vpc looks good now. However I found an issue with isolated networks (rvr or non-rvr both) -