[GitHub] blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-406160737 @dhlaluku a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] dhlaluku commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
dhlaluku commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-406160632 @blueorangutan test This is an automated message from

[GitHub] blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-406095441 Trillian test result (tid-2859) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total

[GitHub] GabrielBrascher edited a comment on issue #2732: security_group: Use execute() function instead of non-existing bash()

2018-07-18 Thread GitBox
GabrielBrascher edited a comment on issue #2732: security_group: Use execute() function instead of non-existing bash() URL: https://github.com/apache/cloudstack/pull/2732#issuecomment-406037059 The following log messages are from `/var/log/cloudstack/agent/security_group.log`. On

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

2018-07-18 Thread GitBox
GabrielBrascher commented on issue #2732: security_group: Use execute() function instead of non-existing bash() URL: https://github.com/apache/cloudstack/pull/2732#issuecomment-406037059 On 4.11.1: ``` 2018-07-10 21:27:00,514 - Executing command: network_rules_vmSecondaryIp

[GitHub] blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-406023730 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2185

[GitHub] blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-406013636 @dhlaluku a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] dhlaluku commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
dhlaluku commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-406013408 @blueorangutan package This is an automated message from

[GitHub] rafaelweingartner commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement

2018-07-18 Thread GitBox
rafaelweingartner commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement URL: https://github.com/apache/cloudstack/pull/2636#discussion_r203463594 ## File path:

[GitHub] blueorangutan commented on issue #2749: [master]Smoketest Health Check

2018-07-18 Thread GitBox
blueorangutan commented on issue #2749: [master]Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2749#issuecomment-406005245 Trillian test result (tid-2858) Environment: vmware-65 (x2), Advanced Networking with Mgmt server 7 Total time taken: 28061 seconds

[GitHub] blueorangutan commented on issue #2749: [master]Smoketest Health Check

2018-07-18 Thread GitBox
blueorangutan commented on issue #2749: [master]Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2749#issuecomment-406003653 Trillian test result (tid-2857) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 27582 seconds

[GitHub] DaanHoogland commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement

2018-07-18 Thread GitBox
DaanHoogland commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement URL: https://github.com/apache/cloudstack/pull/2636#discussion_r203450480 ## File path:

[GitHub] blueorangutan commented on issue #2749: [master]Smoketest Health Check

2018-07-18 Thread GitBox
blueorangutan commented on issue #2749: [master]Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2749#issuecomment-405986584 Trillian test result (tid-2856) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time taken: 24254 seconds

[GitHub] dhlaluku commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
dhlaluku commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-405986425 @rafaelweingartner checked the attributes, it seems like they can be set to private. I will modify them and do some tests

[GitHub] rafaelweingartner commented on issue #2734: Fix invalid consoleproxy url after upgrade

2018-07-18 Thread GitBox
rafaelweingartner commented on issue #2734: Fix invalid consoleproxy url after upgrade URL: https://github.com/apache/cloudstack/pull/2734#issuecomment-405975344 I think the forward merging is working (normally). Somehow git is managing the new folder structure that was applied between

[GitHub] resmo commented on issue #2734: Fix invalid consoleproxy url after upgrade

2018-07-18 Thread GitBox
resmo commented on issue #2734: Fix invalid consoleproxy url after upgrade URL: https://github.com/apache/cloudstack/pull/2734#issuecomment-405974841 will work on the unit test but +14 days and I assume forward merging will be tricky (file renames), suggesting to merge into 4.11 and add

[cloudstack-docs-install] 01/01: Merge pull request #40 from PaulAngus/4.11

2018-07-18 Thread paul_a
This is an automated email from the ASF dual-hosted git repository. paul_a pushed a commit to branch 4.11 in repository https://gitbox.apache.org/repos/asf/cloudstack-docs-install.git commit 94c543f3bd1437b4ea1b20b84653f63949039d55 Merge: fa75631 a15446a Author: Paul Angus AuthorDate: Wed Jul

[cloudstack-docs-install] branch 4.11 updated (fa75631 -> 94c543f)

2018-07-18 Thread paul_a
This is an automated email from the ASF dual-hosted git repository. paul_a pushed a change to branch 4.11 in repository https://gitbox.apache.org/repos/asf/cloudstack-docs-install.git. from fa75631 CLOUDSTACK-10333: update docs to enable libvirtd tls port (#36) add 619e95d Updates

[GitHub] rafaelweingartner commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement

2018-07-18 Thread GitBox
rafaelweingartner commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement URL: https://github.com/apache/cloudstack/pull/2636#discussion_r203420800 ## File path:

[GitHub] rafaelweingartner removed a comment on issue #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement

2018-07-18 Thread GitBox
rafaelweingartner removed a comment on issue #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement URL: https://github.com/apache/cloudstack/pull/2636#issuecomment-405968549 This new method that I implemented is the consolidation of the logic of tag

[GitHub] rafaelweingartner commented on issue #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement

2018-07-18 Thread GitBox
rafaelweingartner commented on issue #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement URL: https://github.com/apache/cloudstack/pull/2636#issuecomment-405968549 This new method that I implemented is the consolidation of the logic of tag matching in

[GitHub] DaanHoogland commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement

2018-07-18 Thread GitBox
DaanHoogland commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement URL: https://github.com/apache/cloudstack/pull/2636#discussion_r203406752 ## File path:

[GitHub] DaanHoogland commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement

2018-07-18 Thread GitBox
DaanHoogland commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement URL: https://github.com/apache/cloudstack/pull/2636#discussion_r203406752 ## File path:

[GitHub] rafaelweingartner commented on a change in pull request #2743: CLOUDSTACK-10380: Fix startvm giving another pw after pw reset

2018-07-18 Thread GitBox
rafaelweingartner commented on a change in pull request #2743: CLOUDSTACK-10380: Fix startvm giving another pw after pw reset URL: https://github.com/apache/cloudstack/pull/2743#discussion_r203399511 ## File path: server/src/com/cloud/network/element/VirtualRouterElement.java

[GitHub] blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-405950842 @dhlaluku a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rafaelweingartner commented on a change in pull request #2743: CLOUDSTACK-10380: Fix startvm giving another pw after pw reset

2018-07-18 Thread GitBox
rafaelweingartner commented on a change in pull request #2743: CLOUDSTACK-10380: Fix startvm giving another pw after pw reset URL: https://github.com/apache/cloudstack/pull/2743#discussion_r203400062 ## File path: server/src/com/cloud/vm/UserVmManagerImpl.java ## @@

[GitHub] dhlaluku commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
dhlaluku commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-405950652 @blueorangutan test This is an automated message from

[GitHub] blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-405948400 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2184

[GitHub] blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-405936718 @dhlaluku a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] dhlaluku commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
dhlaluku commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-405936375 @blueorangutan package This is an automated message from

[GitHub] blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-405935137 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-2183

[GitHub] dhlaluku commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
dhlaluku commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-405932506 @DaanHoogland @rafaelweingartner @resmo @marcaurele I am new to this community and I would like to ask for your

[GitHub] blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
blueorangutan commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-405925400 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] borisstoyanov commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
borisstoyanov commented on issue #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750#issuecomment-405925149 @blueorangutan package This is an automated message

[GitHub] dhlaluku opened a new pull request #2750: Refactor userVmDetailsDao field and remove unusued fields

2018-07-18 Thread GitBox
dhlaluku opened a new pull request #2750: Refactor userVmDetailsDao field and remove unusued fields URL: https://github.com/apache/cloudstack/pull/2750 ## Description This PR refactors a duplicated field vmDetailsDao and _uservmDetailsDao into userVmDetailsDao. vmDetailsDao,

[GitHub] borisstoyanov commented on issue #2743: CLOUDSTACK-10380: Fix startvm giving another pw after pw reset

2018-07-18 Thread GitBox
borisstoyanov commented on issue #2743: CLOUDSTACK-10380: Fix startvm giving another pw after pw reset URL: https://github.com/apache/cloudstack/pull/2743#issuecomment-405915192 there's seems to be an issue with the vm password I'm able to ssh using default password, looks like the

[GitHub] borisstoyanov commented on a change in pull request #2743: CLOUDSTACK-10380: Fix startvm giving another pw after pw reset

2018-07-18 Thread GitBox
borisstoyanov commented on a change in pull request #2743: CLOUDSTACK-10380: Fix startvm giving another pw after pw reset URL: https://github.com/apache/cloudstack/pull/2743#discussion_r203357320 ## File path: tools/marvin/setup.py ## @@ -54,10 +54,12 @@

[cloudstack] branch 4.11 updated: check volumes for state when retrieving pool for configDrive creation (#2709)

2018-07-18 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn 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 38d0274 check volumes for state when retrieving

[GitHub] rafaelweingartner commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement

2018-07-18 Thread GitBox
rafaelweingartner commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement URL: https://github.com/apache/cloudstack/pull/2636#discussion_r203339499 ## File path:

[GitHub] DaanHoogland closed pull request #2709: check volumes for state when retrieving pool for configDrive creation

2018-07-18 Thread GitBox
DaanHoogland closed pull request #2709: check volumes for state when retrieving pool for configDrive creation URL: https://github.com/apache/cloudstack/pull/2709 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] DaanHoogland commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement

2018-07-18 Thread GitBox
DaanHoogland commented on a change in pull request #2636: Fix limitation on tag matching in 'migrateVolume' with disk offering replacement URL: https://github.com/apache/cloudstack/pull/2636#discussion_r202960678 ## File path:

[GitHub] svenvogel commented on issue #2204: [CLOUDSTACK-10025] Adding Support for NoVNC Console for KVM and XENSERVER

2018-07-18 Thread GitBox
svenvogel commented on issue #2204: [CLOUDSTACK-10025] Adding Support for NoVNC Console for KVM and XENSERVER URL: https://github.com/apache/cloudstack/pull/2204#issuecomment-405875150 ping @syed some news on theis pull request?

[GitHub] blueorangutan commented on issue #2749: [master]Smoketest Health Check

2018-07-18 Thread GitBox
blueorangutan commented on issue #2749: [master]Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2749#issuecomment-405859303 @borisstoyanov a Trillian-Jenkins matrix job (centos6 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run

[GitHub] borisstoyanov commented on issue #2749: [master]Smoketest Health Check

2018-07-18 Thread GitBox
borisstoyanov commented on issue #2749: [master]Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2749#issuecomment-405859076 @blueorangutan test matrix This is an automated message from the Apache Git

[GitHub] blueorangutan commented on issue #2749: [master]Smoketest Health Check

2018-07-18 Thread GitBox
blueorangutan commented on issue #2749: [master]Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2749#issuecomment-405852569 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2182 This is an automated

[GitHub] blueorangutan commented on issue #2749: [master]Smoketest Health Check

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

[GitHub] borisstoyanov opened a new pull request #2749: [master]Smoketest Health Check

2018-07-18 Thread GitBox
borisstoyanov opened a new pull request #2749: [master]Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2749 --DO NOT MERGE-- Just a dummy PR for checking the state of smoketests This is an

[GitHub] borisstoyanov commented on issue #2749: [master]Smoketest Health Check

2018-07-18 Thread GitBox
borisstoyanov commented on issue #2749: [master]Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2749#issuecomment-405843685 @blueorangutan package This is an automated message from the Apache Git