[GitHub] DaanHoogland commented on issue #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds

2018-02-19 Thread GitBox
DaanHoogland commented on issue #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds URL: https://github.com/apache/cloudstack/pull/2244#issuecomment-366626731 @rhtyd can you see if your objections are addressed? @mrunalinikankariya can you look at the output for test_hosth

[GitHub] blueorangutan commented on issue #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds

2018-02-19 Thread GitBox
blueorangutan commented on issue #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds URL: https://github.com/apache/cloudstack/pull/2244#issuecomment-366626935 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -

[GitHub] DaanHoogland commented on issue #2425: [CLOUDSTACK-10240] ACS cannot migrate a local volume to shared storage

2018-02-19 Thread GitBox
DaanHoogland commented on issue #2425: [CLOUDSTACK-10240] ACS cannot migrate a local volume to shared storage URL: https://github.com/apache/cloudstack/pull/2425#issuecomment-366627617 @blueorangutan package This is an automa

[GitHub] blueorangutan commented on issue #2425: [CLOUDSTACK-10240] ACS cannot migrate a local volume to shared storage

2018-02-19 Thread GitBox
blueorangutan commented on issue #2425: [CLOUDSTACK-10240] ACS cannot migrate a local volume to shared storage URL: https://github.com/apache/cloudstack/pull/2425#issuecomment-366627862 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] DaanHoogland commented on issue #2459: CLOUDSTACK-10297: Isolation network defined subnet mask failure

2018-02-19 Thread GitBox
DaanHoogland commented on issue #2459: CLOUDSTACK-10297: Isolation network defined subnet mask failure URL: https://github.com/apache/cloudstack/pull/2459#issuecomment-366628055 this is not to go in, it is not based on the right version and takes in loads of unwanted stuff in its slur. --

[GitHub] DaanHoogland closed pull request #2459: CLOUDSTACK-10297: Isolation network defined subnet mask failure

2018-02-19 Thread GitBox
DaanHoogland closed pull request #2459: CLOUDSTACK-10297: Isolation network defined subnet mask failure URL: https://github.com/apache/cloudstack/pull/2459 This is an automated message from the Apache Git Service. To respond

[GitHub] blueorangutan commented on issue #2425: [CLOUDSTACK-10240] ACS cannot migrate a local volume to shared storage

2018-02-19 Thread GitBox
blueorangutan commented on issue #2425: [CLOUDSTACK-10240] ACS cannot migrate a local volume to shared storage URL: https://github.com/apache/cloudstack/pull/2425#issuecomment-366636864 Packaging result: ?centos6 ?centos7 ?debian. JID-1726 --

[GitHub] rafaelweingartner commented on a change in pull request #2456: [CLOUDSTACK-10293] Single view network ACL rules listing

2018-02-19 Thread GitBox
rafaelweingartner commented on a change in pull request #2456: [CLOUDSTACK-10293] Single view network ACL rules listing URL: https://github.com/apache/cloudstack/pull/2456#discussion_r169043024 ## File path: ui/scripts/vpc.js ## @@ -1333,35 +1321,65 @@

[GitHub] rafaelweingartner commented on a change in pull request #2456: [CLOUDSTACK-10293] Single view network ACL rules listing

2018-02-19 Thread GitBox
rafaelweingartner commented on a change in pull request #2456: [CLOUDSTACK-10293] Single view network ACL rules listing URL: https://github.com/apache/cloudstack/pull/2456#discussion_r169043241 ## File path: ui/scripts/vpc.js ## @@ -1333,35 +1321,65 @@

[GitHub] rafaelweingartner commented on a change in pull request #2456: [CLOUDSTACK-10293] Single view network ACL rules listing

2018-02-19 Thread GitBox
rafaelweingartner commented on a change in pull request #2456: [CLOUDSTACK-10293] Single view network ACL rules listing URL: https://github.com/apache/cloudstack/pull/2456#discussion_r169045024 ## File path: ui/scripts/vpc.js ## @@ -436,6 +401,38 @@ isOpti

[GitHub] rafaelweingartner commented on issue #2456: [CLOUDSTACK-10293] Single view network ACL rules listing

2018-02-19 Thread GitBox
rafaelweingartner commented on issue #2456: [CLOUDSTACK-10293] Single view network ACL rules listing URL: https://github.com/apache/cloudstack/pull/2456#issuecomment-366658774 Thanks for your review @nvazquez ! I have made the changes you suggested. -

[GitHub] nvazquez commented on a change in pull request #2456: [CLOUDSTACK-10293] Single view network ACL rules listing

2018-02-19 Thread GitBox
nvazquez commented on a change in pull request #2456: [CLOUDSTACK-10293] Single view network ACL rules listing URL: https://github.com/apache/cloudstack/pull/2456#discussion_r169060895 ## File path: ui/scripts/vpc.js ## @@ -1333,35 +1321,65 @@

[GitHub] nvazquez commented on issue #2456: [CLOUDSTACK-10293] Single view network ACL rules listing

2018-02-19 Thread GitBox
nvazquez commented on issue #2456: [CLOUDSTACK-10293] Single view network ACL rules listing URL: https://github.com/apache/cloudstack/pull/2456#issuecomment-366678807 Thanks @rafaelweingartner! Now code LGTM This is an automa

[GitHub] borisstoyanov commented on issue #2457: CLOUDSTACK-10295 Marvin: add support for password-enabled templates

2018-02-19 Thread GitBox
borisstoyanov commented on issue #2457: CLOUDSTACK-10295 Marvin: add support for password-enabled templates URL: https://github.com/apache/cloudstack/pull/2457#issuecomment-366680629 @dmytro-shevchenko ping This is an automat

[GitHub] sgoeminn opened a new pull request #2460: CLOUDSTACK-10298: fix for recreation of an earlier deleted Nuage managed network

2018-02-19 Thread GitBox
sgoeminn opened a new pull request #2460: CLOUDSTACK-10298: fix for recreation of an earlier deleted Nuage managed network URL: https://github.com/apache/cloudstack/pull/2460 CLOUDSTACK-10298: Recreation of an earlier deleted Nuage managed isolated or vpc tier network fails Added ne

[GitHub] blueorangutan commented on issue #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds

2018-02-19 Thread GitBox
blueorangutan commented on issue #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds URL: https://github.com/apache/cloudstack/pull/2244#issuecomment-366733669 Trillian test result (tid-2272) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total tim

[GitHub] khos2ow commented on issue #2433: CLOUDSTACK-10268: Fix and enhance package script

2018-02-19 Thread GitBox
khos2ow commented on issue #2433: CLOUDSTACK-10268: Fix and enhance package script URL: https://github.com/apache/cloudstack/pull/2433#issuecomment-366770781 @rafaelweingartner @DaanHoogland @rhtyd I updated the scripts and description of the PR again. Now when `--use-timestamp` is spec

[GitHub] rafaelweingartner opened a new pull request #2461: [CLOUDSTACK-10301] Allow updating the network ACL list name and Description

2018-02-19 Thread GitBox
rafaelweingartner opened a new pull request #2461: [CLOUDSTACK-10301] Allow updating the network ACL list name and Description URL: https://github.com/apache/cloudstack/pull/2461 Currently, it is not possible to update the Network ACL name or Description of an ACL after it has been created

[GitHub] rafaelweingartner commented on issue #2402: [CLOUDSTACK-10128] Simplify the search for host to execute command

2018-02-19 Thread GitBox
rafaelweingartner commented on issue #2402: [CLOUDSTACK-10128] Simplify the search for host to execute command URL: https://github.com/apache/cloudstack/pull/2402#issuecomment-366796206 reviewers (@DaanHoogland, @mike-tutkowski, @rhtyd ) what is your positions here? It seems that the tests

[GitHub] mike-tutkowski commented on issue #2402: [CLOUDSTACK-10128] Simplify the search for host to execute command

2018-02-19 Thread GitBox
mike-tutkowski commented on issue #2402: [CLOUDSTACK-10128] Simplify the search for host to execute command URL: https://github.com/apache/cloudstack/pull/2402#issuecomment-366797995 Yes, per my prior comments, the code LGTM and the two tests that previously failed now pass. -

[GitHub] rafaelweingartner commented on issue #2402: [CLOUDSTACK-10128] Simplify the search for host to execute command

2018-02-19 Thread GitBox
rafaelweingartner commented on issue #2402: [CLOUDSTACK-10128] Simplify the search for host to execute command URL: https://github.com/apache/cloudstack/pull/2402#issuecomment-366798899 @mike-tutkowski thanks! I am getting used to look directly to the "reviweres" section on Github and c

[GitHub] mike-tutkowski commented on issue #2402: [CLOUDSTACK-10128] Simplify the search for host to execute command

2018-02-19 Thread GitBox
mike-tutkowski commented on issue #2402: [CLOUDSTACK-10128] Simplify the search for host to execute command URL: https://github.com/apache/cloudstack/pull/2402#issuecomment-366800222 Sounds good - we should all get used to marking that field. On Feb 19, 2018, at 12:46 PM, Rafael Wein

[GitHub] rafaelweingartner commented on issue #2402: [CLOUDSTACK-10128] Simplify the search for host to execute command

2018-02-19 Thread GitBox
rafaelweingartner commented on issue #2402: [CLOUDSTACK-10128] Simplify the search for host to execute command URL: https://github.com/apache/cloudstack/pull/2402#issuecomment-366800251 Since everything is ok, I will merge this one. Let's see if I still know how to do it :) ---

[cloudstack] branch 4.11 updated (97441a8 -> 264036b)

2018-02-19 Thread rafael
This is an automated email from the ASF dual-hosted git repository. rafael pushed a change to branch 4.11 in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 97441a8 CLOUDSTACK-10282: ipv6 firewall rules operation should be done with ip6tables (#2450) add 1c8bbdd [

[GitHub] rafaelweingartner closed pull request #2402: [CLOUDSTACK-10128] Simplify the search for host to execute command

2018-02-19 Thread GitBox
rafaelweingartner closed pull request #2402: [CLOUDSTACK-10128] Simplify the search for host to execute command URL: https://github.com/apache/cloudstack/pull/2402 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[cloudstack] 01/01: Merge pull request #2402 from rafaelweingartner/fixCLOUDSTACK-10128

2018-02-19 Thread rafael
This is an automated email from the ASF dual-hosted git repository. rafael pushed a commit to branch 4.11 in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 264036bd2c28dec03b07c9822d99092b826c1a87 Merge: 97441a8 1c8bbdd Author: Rafael Weingärtner AuthorDate: Mon Feb 19 17:5

[cloudstack] branch master updated (6233a77 -> 1dc7cf1)

2018-02-19 Thread rafael
This is an automated email from the ASF dual-hosted git repository. rafael pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 6233a77 CLOUDSTACK-10294: PEP-8 fixes and enhancements to security_group.py (#2432) add 876c464 [CLOUDSTA

[GitHub] rafaelweingartner closed pull request #2461: [CLOUDSTACK-10301] Allow updating the network ACL list name and Description

2018-02-19 Thread GitBox
rafaelweingartner closed pull request #2461: [CLOUDSTACK-10301] Allow updating the network ACL list name and Description URL: https://github.com/apache/cloudstack/pull/2461 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[cloudstack] branch master updated (1dc7cf1 -> 6233a77)

2018-02-19 Thread rafael
This is an automated email from the ASF dual-hosted git repository. rafael pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. omit 1dc7cf1 Cleanups and formatting to make checkstyle happy omit 876c464 [CLOUDSTACK-10301] Allow updating the

[GitHub] rafaelweingartner opened a new pull request #2462: [CLOUDSTACK-10301] Allow updating the network ACL list name and Description

2018-02-19 Thread GitBox
rafaelweingartner opened a new pull request #2462: [CLOUDSTACK-10301] Allow updating the network ACL list name and Description URL: https://github.com/apache/cloudstack/pull/2462 Currently, it is not possible to update the Network ACL name or Description of an ACL after it has been created

[cloudstack] 01/01: Forward merge #2402 merged on '4.11' branch

2018-02-19 Thread rafael
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 commit e532b574ddb186a117da638fb6059356fe7c266c Merge: 6233a77 264036b Author: Rafael Weingärtner AuthorDate: Mon Feb 19 18

[cloudstack] branch master updated (6233a77 -> e532b57)

2018-02-19 Thread rafael
This is an automated email from the ASF dual-hosted git repository. rafael pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 6233a77 CLOUDSTACK-10294: PEP-8 fixes and enhancements to security_group.py (#2432) add 1c8bbdd [CLOUDSTA

[GitHub] mrunalinikankariya commented on issue #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds

2018-02-19 Thread GitBox
mrunalinikankariya commented on issue #2244: CLOUDSTACK-10054:Volume download times out in 3600 seconds URL: https://github.com/apache/cloudstack/pull/2244#issuecomment-366866181 @DaanHoogland test_hostha_enable_ha_when_host_in_maintenance test is faling with error code 530 and error '