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

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

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

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

[GitHub] rafaelweingartner commented on a change in pull request #2734: Fix invalid consoleproxy url after upgrade

2018-07-06 Thread GitBox
rafaelweingartner commented on a change in pull request #2734: Fix invalid consoleproxy url after upgrade URL: https://github.com/apache/cloudstack/pull/2734#discussion_r200783410 ## File path: core/src/main/java/com/cloud/info/ConsoleProxyInfo.java ## @@ -36,33 +36,33 @@

[GitHub] rafaelweingartner commented on a change in pull request #2734: Fix invalid consoleproxy url after upgrade

2018-07-06 Thread GitBox
rafaelweingartner commented on a change in pull request #2734: Fix invalid consoleproxy url after upgrade URL: https://github.com/apache/cloudstack/pull/2734#discussion_r200783318 ## File path: core/src/main/java/com/cloud/info/ConsoleProxyInfo.java ## @@ -36,33 +36,33 @@

[GitHub] resmo opened a new pull request #2734: Fix invalid consoleproxy url after upgrade

2018-07-06 Thread GitBox
resmo opened a new pull request #2734: Fix invalid consoleproxy url after upgrade URL: https://github.com/apache/cloudstack/pull/2734 ## Description Ensures we have a valid domain for protocol http even a `*` setting is used. See #2733 ## Types of changes - Bug fix

[GitHub] resmo opened a new issue #2733: invalid consoleproxy domain after upgrade from 4.5 to 4.11.1

2018-07-06 Thread GitBox
resmo opened a new issue #2733: invalid consoleproxy domain after upgrade from 4.5 to 4.11.1 URL: https://github.com/apache/cloudstack/issues/2733 Avoid a pitfall after upgrade from 4.5 to 4.11 using invalid domain with `*` for console proxy url # ISSUE TYPE * Bug

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

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

[GitHub] blueorangutan commented on issue #2732: security_group: Use execute() function instead of non-existing bash()

2018-07-06 Thread GitBox
blueorangutan commented on issue #2732: security_group: Use execute() function instead of non-existing bash() URL: https://github.com/apache/cloudstack/pull/2732#issuecomment-403125411 Trillian test result (tid-2837) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] khos2ow commented on issue #2728: Consistence POM version for 4.11.x.y

2018-07-06 Thread GitBox
khos2ow commented on issue #2728: Consistence POM version for 4.11.x.y URL: https://github.com/apache/cloudstack/pull/2728#issuecomment-403054761 @DaanHoogland I agree, i believe 3 digit would have been enough considering number and frequency of times we release. But since it's already

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

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

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

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

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

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

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

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

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

[cloudstack] branch master updated (4987e3e -> fc1874c)

2018-07-06 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 4987e3e Merge branch 'origin/4.11' add 67860d9 maven: Updating pom.xml version numbers for release

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

2018-07-06 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 fc1874c1c9c52abe1eed45a47baab6170d4b8b40 Merge: 4987e3e 67860d9 Author: Rohit Yadav AuthorDate: Fri Jul 6 17:29:17

[cloudstack] branch 4.11 updated: maven: Updating pom.xml version numbers for release 4.11.2.0-SNAPSHOT (#2728)

2018-07-06 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 67860d9 maven: Updating pom.xml version numbers

[GitHub] rhtyd closed pull request #2728: Consistence POM version for 4.11.x.y

2018-07-06 Thread GitBox
rhtyd closed pull request #2728: Consistence POM version for 4.11.x.y URL: https://github.com/apache/cloudstack/pull/2728 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

[GitHub] rhtyd commented on issue #2728: Consistence POM version for 4.11.x.y

2018-07-06 Thread GitBox
rhtyd commented on issue #2728: Consistence POM version for 4.11.x.y URL: https://github.com/apache/cloudstack/pull/2728#issuecomment-403012634 Thanks for taking away -1 @DaanHoogland. For the note, the master is set to 4.12.0.0-SNAPSHOT in poms etc. I'll merge this as this should have

[GitHub] blueorangutan commented on issue #2732: security_group: Use execute() function instead of non-existing bash()

2018-07-06 Thread GitBox
blueorangutan commented on issue #2732: security_group: Use execute() function instead of non-existing bash() URL: https://github.com/apache/cloudstack/pull/2732#issuecomment-402989534 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

[GitHub] borisstoyanov commented on issue #2732: security_group: Use execute() function instead of non-existing bash()

2018-07-06 Thread GitBox
borisstoyanov commented on issue #2732: security_group: Use execute() function instead of non-existing bash() URL: https://github.com/apache/cloudstack/pull/2732#issuecomment-402989459 @blueorangutan test This is an

[GitHub] blueorangutan commented on issue #2732: security_group: Use execute() function instead of non-existing bash()

2018-07-06 Thread GitBox
blueorangutan commented on issue #2732: security_group: Use execute() function instead of non-existing bash() URL: https://github.com/apache/cloudstack/pull/2732#issuecomment-402981171 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2166

[GitHub] blueorangutan commented on issue #2732: security_group: Use execute() function instead of non-existing bash()

2018-07-06 Thread GitBox
blueorangutan commented on issue #2732: security_group: Use execute() function instead of non-existing bash() URL: https://github.com/apache/cloudstack/pull/2732#issuecomment-402974311 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2732: security_group: Use execute() function instead of non-existing bash()

2018-07-06 Thread GitBox
rhtyd commented on issue #2732: security_group: Use execute() function instead of non-existing bash() URL: https://github.com/apache/cloudstack/pull/2732#issuecomment-402974192 @blueorangutan package This is an

[GitHub] blueorangutan commented on issue #2732: security_group: Use execute() function instead of non-existing bash()

2018-07-06 Thread GitBox
blueorangutan commented on issue #2732: security_group: Use execute() function instead of non-existing bash() URL: https://github.com/apache/cloudstack/pull/2732#issuecomment-402964558 Packaging result: ✖centos6 ✔centos7 ✖debian. JID-2165

[GitHub] rhtyd commented on issue #2721: api: Introducing a new diagnostics API command for system VMs for CloudStack admins

2018-07-06 Thread GitBox
rhtyd commented on issue #2721: api: Introducing a new diagnostics API command for system VMs for CloudStack admins URL: https://github.com/apache/cloudstack/pull/2721#issuecomment-402961392 Please hold merging, while everything is LGTM -- @dhlaluku will add support for simulator with his

[GitHub] rhtyd edited a comment on issue #2728: Consistence POM version for 4.11.x.y

2018-07-06 Thread GitBox
rhtyd edited a comment on issue #2728: Consistence POM version for 4.11.x.y URL: https://github.com/apache/cloudstack/pull/2728#issuecomment-402955026 @DaanHoogland that breaks what we've been following. I disagree, we should keep the next version to 4.11.2.0, just like 4.11.1.0 was done

[GitHub] blueorangutan commented on issue #2732: security_group: Use execute() function instead of non-existing bash()

2018-07-06 Thread GitBox
blueorangutan commented on issue #2732: security_group: Use execute() function instead of non-existing bash() URL: https://github.com/apache/cloudstack/pull/2732#issuecomment-402957069 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] borisstoyanov commented on issue #2732: security_group: Use execute() function instead of non-existing bash()

2018-07-06 Thread GitBox
borisstoyanov commented on issue #2732: security_group: Use execute() function instead of non-existing bash() URL: https://github.com/apache/cloudstack/pull/2732#issuecomment-402956958 @blueorangutan package This is an

[GitHub] rhtyd edited a comment on issue #2728: Consistence POM version for 4.11.x.y

2018-07-06 Thread GitBox
rhtyd edited a comment on issue #2728: Consistence POM version for 4.11.x.y URL: https://github.com/apache/cloudstack/pull/2728#issuecomment-402955026 @DaanHoogland that breaks what we've been following. I disagree, we should keep the next version to 4.11.2.0, just like 4.11.1.0 was done

[GitHub] rhtyd closed pull request #2731: maven: update pom.xml version numbers to release 4.11.2.0-SNAPSHOT

2018-07-06 Thread GitBox
rhtyd closed pull request #2731: maven: update pom.xml version numbers to release 4.11.2.0-SNAPSHOT URL: https://github.com/apache/cloudstack/pull/2731 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] rhtyd commented on issue #2731: maven: update pom.xml version numbers to release 4.11.2.0-SNAPSHOT

2018-07-06 Thread GitBox
rhtyd commented on issue #2731: maven: update pom.xml version numbers to release 4.11.2.0-SNAPSHOT URL: https://github.com/apache/cloudstack/pull/2731#issuecomment-402955152 Dupes https://github.com/apache/cloudstack/pull/2728/files so closing

[GitHub] rhtyd commented on issue #2728: Consistence POM version for 4.11.x.y

2018-07-06 Thread GitBox
rhtyd commented on issue #2728: Consistence POM version for 4.11.x.y URL: https://github.com/apache/cloudstack/pull/2728#issuecomment-402955026 @DaanHoogland that breaks what we've been following. I disagree, we should keep the next version to 4.11.2.0, just like 4.11.1.0 was done.

[GitHub] rhtyd commented on issue #2731: maven: update pom.xml version numbers to release 4.11.2.0-SNAPSHOT

2018-07-06 Thread GitBox
rhtyd commented on issue #2731: maven: update pom.xml version numbers to release 4.11.2.0-SNAPSHOT URL: https://github.com/apache/cloudstack/pull/2731#issuecomment-402954715 Ping @PaulAngus @wido @DaanHoogland @GabrielBrascher @borisstoyanov

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

2018-07-06 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 4987e3e71cf179b845a81ac3ad904ac03049e16e Merge: 45b9862 c1ca45a Author: Rohit Yadav AuthorDate: Fri Jul 6 13:02:38

[cloudstack] branch master updated (45b9862 -> 4987e3e)

2018-07-06 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 45b9862 Merge branch '4.11' add c1ca45a maven: bump up vmware sdk jar to 6.7 (#2727) new 4987e3e

[GitHub] rhtyd closed pull request #2727: maven: bump up vmware sdk jar to 6.7

2018-07-06 Thread GitBox
rhtyd closed pull request #2727: maven: bump up vmware sdk jar to 6.7 URL: https://github.com/apache/cloudstack/pull/2727 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: maven: bump up vmware sdk jar to 6.7 (#2727)

2018-07-06 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 c1ca45a maven: bump up vmware sdk jar to 6.7

[cloudstack] branch master updated (85ad285 -> 45b9862)

2018-07-06 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 85ad285 some formatting and a typo add 2cb2dac Updating pom.xml version numbers for release 4.11.1.0

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

2018-07-06 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 45b986294db83a9aa4a2c88fa9e48e17a05d5b47 Merge: 85ad285 89c2721 Author: Rohit Yadav AuthorDate: Fri Jul 6 12:54:23

[GitHub] rhtyd closed issue #2725: Version Update has corrupted some files

2018-07-06 Thread GitBox
rhtyd closed issue #2725: Version Update has corrupted some files URL: https://github.com/apache/cloudstack/issues/2725 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] rhtyd commented on issue #2731: maven: update pom.xml version numbers to release 4.11.2.0-SNAPSHOT

2018-07-06 Thread GitBox
rhtyd commented on issue #2731: maven: update pom.xml version numbers to release 4.11.2.0-SNAPSHOT URL: https://github.com/apache/cloudstack/pull/2731#issuecomment-402949534 The only changes are in the version string, no need to any regression test given packaging and travis have passed.

[GitHub] rhtyd commented on issue #2732: security_group: Use execute() function instead of non-existing bash()

2018-07-06 Thread GitBox
rhtyd commented on issue #2732: security_group: Use execute() function instead of non-existing bash() URL: https://github.com/apache/cloudstack/pull/2732#issuecomment-402949393 @wido @GabrielBrascher can you show some test results wrt SG+KVM?