[GitHub] PaulAngus commented on issue #3010: Fix DirectNetworkGuru canHandle checks for lowercase isolation methods

2018-11-07 Thread GitBox
PaulAngus commented on issue #3010: Fix DirectNetworkGuru canHandle checks for lowercase isolation methods URL: https://github.com/apache/cloudstack/pull/3010#issuecomment-436579973 LGTM Tested with cloud.physical_network_isolation_methods.isolation_method set to 'VLAN', 'vlan' and 'Vla

[GitHub] rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436582691 I think you should change the view to include this even when it is deleted, instead of changing the joining table/vi

[cloudstack] branch 4.11 updated: Fix DirectNetworkGuru canHandle checks for lowercase isolation methods (#3010)

2018-11-07 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 The following commit(s) were added to refs/heads/4.11 by this push: new af0c1e4 Fix DirectNetworkGuru canHandle checks

[GitHub] rafaelweingartner closed pull request #3010: Fix DirectNetworkGuru canHandle checks for lowercase isolation methods

2018-11-07 Thread GitBox
rafaelweingartner closed pull request #3010: Fix DirectNetworkGuru canHandle checks for lowercase isolation methods URL: https://github.com/apache/cloudstack/pull/3010 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the s

[cloudstack] 01/01: Forward merge Fix DirectNetworkGuru canHandle for lowercase isolation methods

2018-11-07 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 077e692dbb4abef95ab62f77c0e62e5aa19b32e5 Merge: e175e93 af0c1e4 Author: Rafael Weingärtner AuthorDate: Wed Nov 7 09:

[cloudstack] branch master updated (e175e93 -> 077e692)

2018-11-07 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 e175e93 Fix set affinity for user overlay problem (Fixes #2986) (#2987) add af0c1e4 Fix DirectNetworkGuru

[GitHub] rafaelweingartner closed issue #2989: Unable to create network post upgrade to 4.11

2018-11-07 Thread GitBox
rafaelweingartner closed issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989 This is an automated message from the Apache Git Service. To respond to the message, p

[cloudstack] branch master updated: make sure no time skew can interfere with comparison (#3003)

2018-11-07 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 The following commit(s) were added to refs/heads/master by this push: new 604d2d1 make sure no time skew can interfer

[GitHub] rafaelweingartner closed pull request #3003: make sure no time skew can interfere with comparison

2018-11-07 Thread GitBox
rafaelweingartner closed pull request #3003: make sure no time skew can interfere with comparison URL: https://github.com/apache/cloudstack/pull/3003 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:

[GitHub] rafaelweingartner commented on a change in pull request #3000: CLOUDSTACK-2998: Added two more device name patterns to valid bridge slaves (lo* and dummy*)

2018-11-07 Thread GitBox
rafaelweingartner commented on a change in pull request #3000: CLOUDSTACK-2998: Added two more device name patterns to valid bridge slaves (lo* and dummy*) URL: https://github.com/apache/cloudstack/pull/3000#discussion_r231482349 ## File path: plugins/hypervisors/kvm/src/main/java/

[GitHub] nvazquez commented on issue #3004: [Placeholder] Add new data motion strategy for KVM non managed storage live migrations

2018-11-07 Thread GitBox
nvazquez commented on issue #3004: [Placeholder] Add new data motion strategy for KVM non managed storage live migrations URL: https://github.com/apache/cloudstack/pull/3004#issuecomment-436604927 Thanks @rafaelweingartner, the aim of this PR was just introducing the new strategy class as

[GitHub] rafaelweingartner commented on a change in pull request #3000: CLOUDSTACK-2998: Added two more device name patterns to valid bridge slaves (lo* and dummy*)

2018-11-07 Thread GitBox
rafaelweingartner commented on a change in pull request #3000: CLOUDSTACK-2998: Added two more device name patterns to valid bridge slaves (lo* and dummy*) URL: https://github.com/apache/cloudstack/pull/3000#discussion_r231482349 ## File path: plugins/hypervisors/kvm/src/main/java/

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

2018-11-07 Thread GitBox
nvazquez commented on issue #2980: [4.11] Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2980#issuecomment-436606816 @DaanHoogland @borisstoyanov @rafaelweingartner are we ok with this fix?

[GitHub] nvazquez closed pull request #2839: Enable DPDK support on KVM

2018-11-07 Thread GitBox
nvazquez closed pull request #2839: Enable DPDK support on KVM URL: https://github.com/apache/cloudstack/pull/2839 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 request

[cloudstack] branch master updated: Enable DPDK support on KVM (#2839)

2018-11-07 Thread nvazquez
This is an automated email from the ASF dual-hosted git repository. nvazquez 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 4de4eab Enable DPDK support on KVM (#2839

[GitHub] div8cn commented on issue #3008: Runing cloudstack failed in docker

2018-11-07 Thread GitBox
div8cn commented on issue #3008: Runing cloudstack failed in docker URL: https://github.com/apache/cloudstack/issues/3008#issuecomment-436611056 Cloudstack for docker, in 4.10 and previous versions, can be properly activated. If this can not be considered as Bug, should we write a docker

[GitHub] rafaelweingartner commented on issue #3008: Runing cloudstack failed in docker

2018-11-07 Thread GitBox
rafaelweingartner commented on issue #3008: Runing cloudstack failed in docker URL: https://github.com/apache/cloudstack/issues/3008#issuecomment-436611303 @div8cn that would be something very cool to have This is an automated

[cloudstack] branch 4.11 updated: [4.11] Fix set initial reservation on public IP ranges (#2980)

2018-11-07 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 The following commit(s) were added to refs/heads/4.11 by this push: new 7d8eb37 [4.11] Fix set initial reservation on p

[GitHub] rafaelweingartner closed pull request #2980: [4.11] Fix set initial reservation on public IP ranges

2018-11-07 Thread GitBox
rafaelweingartner closed pull request #2980: [4.11] Fix set initial reservation on public IP ranges URL: https://github.com/apache/cloudstack/pull/2980 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 provenanc

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

2018-11-07 Thread GitBox
rafaelweingartner closed issue #2929: add public IP range to domain fails in the inital setup (UI Problem) URL: https://github.com/apache/cloudstack/issues/2929 This is an automated message from the Apache Git Service. To re

[cloudstack] branch master updated (4de4eab -> f183268)

2018-11-07 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 4de4eab Enable DPDK support on KVM (#2839) add 7d8eb37 [4.11] Fix set initial reservation on public IP ran

[cloudstack-documentation] branch master updated (74aa4d7 -> 5c39daf)

2018-11-07 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-documentation.git. from 74aa4d7 Merge pull request #8 from nvazquez/dpdkimprovments add 5bc29f5 Change systemvm temp

[cloudstack-documentation] 01/01: Merge pull request #16 from GabrielBrascher/updateTemplateFrom4.6To4.11

2018-11-07 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-documentation.git commit 5c39daf2d42e4c37f786ec221bb54c5101a5 Merge: 74aa4d7 5bc29f5 Author: Rafael Weingärtner AuthorDate:

[cloudstack-documentation] branch master updated: Documentation for KVM live storage migration (#14)

2018-11-07 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-documentation.git The following commit(s) were added to refs/heads/master by this push: new 13d692c Documentation for KVM

[cloudstack-documentation] branch master updated: Fix cloud-set-guest-password download link (#18)

2018-11-07 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-documentation.git The following commit(s) were added to refs/heads/master by this push: new 2b96f26 Fix cloud-set-guest-

[cloudstack-documentation] branch master updated: CLOUDSTACK-10399: Added section regarding MySQL server-id (#17)

2018-11-07 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-documentation.git The following commit(s) were added to refs/heads/master by this push: new a96d7d6 CLOUDSTACK-10399: Add

[cloudstack-documentation] branch 4.11 updated (2b7f8e9 -> cfc18df)

2018-11-07 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-documentation.git. from 2b7f8e9 Merge pull request #9 from andrijapanic/patch-1 add cfc18df bridge section was missing

[cloudstack-documentation] 01/01: Forwardmerge #12 -- bridge section was missing

2018-11-07 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-documentation.git commit 02696614e5b2a763de7f60ef8863b035104299c8 Merge: a96d7d6 cfc18df Author: Rafael Weingärtner AuthorDate:

[cloudstack-documentation] branch master updated (a96d7d6 -> 0269661)

2018-11-07 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-documentation.git. from a96d7d6 CLOUDSTACK-10399: Added section regarding MySQL server-id (#17) add ddc2485 Formati

[cloudstack-documentation] branch 4.11 updated (cfc18df -> b44f390)

2018-11-07 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-documentation.git. from cfc18df bridge section was missing (#12) add b44f390 "yes" for KVM network throtling support (

[cloudstack-documentation] 01/01: Forwardmerge #11 -- "yes" for KVM network throttling

2018-11-07 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-documentation.git commit 72c1183bb28e72e06fcbfa89a01d44c7cd642a2b Merge: 0269661 b44f390 Author: Rafael Weingärtner AuthorDate:

[cloudstack-documentation] branch master updated (0269661 -> 72c1183)

2018-11-07 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-documentation.git. from 0269661 Forwardmerge #12 -- bridge section was missing add b44f390 "yes" for KVM network thr

[GitHub] rafaelweingartner commented on issue #3004: [Placeholder] Add new data motion strategy for KVM non managed storage live migrations

2018-11-07 Thread GitBox
rafaelweingartner commented on issue #3004: [Placeholder] Add new data motion strategy for KVM non managed storage live migrations URL: https://github.com/apache/cloudstack/pull/3004#issuecomment-436621395 I see your point here, but I do not think that this will help much. As I have seen y

[GitHub] somejfn commented on issue #3010: Fix DirectNetworkGuru canHandle checks for lowercase isolation methods

2018-11-07 Thread GitBox
somejfn commented on issue #3010: Fix DirectNetworkGuru canHandle checks for lowercase isolation methods URL: https://github.com/apache/cloudstack/pull/3010#issuecomment-436622527 Fix is confirmed. I can create a new network now based on the shared network offering --

[GitHub] somejfn commented on issue #2989: Unable to create network post upgrade to 4.11

2018-11-07 Thread GitBox
somejfn commented on issue #2989: Unable to create network post upgrade to 4.11 URL: https://github.com/apache/cloudstack/issues/2989#issuecomment-436624335 Fix is confirmed. I can create a new network now based on the shared network offering. thanks for the great support @borisstoyanov

[GitHub] rafaelweingartner commented on issue #2990: Security Group: add secondary ips to the correct ipset based on ip family (4.11)

2018-11-07 Thread GitBox
rafaelweingartner commented on issue #2990: Security Group: add secondary ips to the correct ipset based on ip family (4.11) URL: https://github.com/apache/cloudstack/pull/2990#issuecomment-436625991 @blueorangutan package Th

[GitHub] bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436626311 @rafaelweingartner potentially it could lead to many bugs. I think the change could some other places to fix. Now it's a small f

[GitHub] rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436626755 Can you explain further why (your concerns)?I mean, if logically the result is the same when finding the physical in

[GitHub] rafaelweingartner commented on a change in pull request #3007: Add missing ConfigDrive entries on existing zones after upgrade

2018-11-07 Thread GitBox
rafaelweingartner commented on a change in pull request #3007: Add missing ConfigDrive entries on existing zones after upgrade URL: https://github.com/apache/cloudstack/pull/3007#discussion_r231508606 ## File path: server/src/com/cloud/network/NetworkServiceImpl.java ## @@

[GitHub] rafaelweingartner commented on a change in pull request #3007: Add missing ConfigDrive entries on existing zones after upgrade

2018-11-07 Thread GitBox
rafaelweingartner commented on a change in pull request #3007: Add missing ConfigDrive entries on existing zones after upgrade URL: https://github.com/apache/cloudstack/pull/3007#discussion_r231508775 ## File path: server/src/com/cloud/network/NetworkServiceImpl.java ## @@

[GitHub] bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436630514 @rafaelweingartner it's not thst issue. It's for resource accounting. ---

[GitHub] rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436631226 Ops, I was with the other issue in the cache... Sorry ;) Let me load this one out of the HDD again... --

[GitHub] rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436633277 Ok, I looked in the code, and the table is used in some other places, specially the `ServiceOfferingJoinVO`. Therefo

[GitHub] bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436633877 @rafaelweingartner may we add to 4.11.3 as well? This is an auto

[GitHub] rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436634502 Sure, but then the process is a little bit different. You need to target 4.11 branch. This will require you to cherr

[GitHub] rafaelweingartner commented on a change in pull request #3005: Discover tags field on superclass of API responses

2018-11-07 Thread GitBox
rafaelweingartner commented on a change in pull request #3005: Discover tags field on superclass of API responses URL: https://github.com/apache/cloudstack/pull/3005#discussion_r231516977 ## File path: plugins/api/discovery/src/org/apache/cloudstack/discovery/ApiDiscoveryServiceImp

[GitHub] nvazquez commented on issue #3004: [Placeholder] Add new data motion strategy for KVM non managed storage live migrations

2018-11-07 Thread GitBox
nvazquez commented on issue #3004: [Placeholder] Add new data motion strategy for KVM non managed storage live migrations URL: https://github.com/apache/cloudstack/pull/3004#issuecomment-436637357 Thanks for the input @rafaelweingartner, that was what we were looking for with this PR. Sure

[GitHub] bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436638209 Ok. then I leave this one too? Or close this one and open for 4.11? -

[GitHub] bwsw edited a comment on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
bwsw edited a comment on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436638209 @rafaelweingartner Ok. then I leave this one too? Or close this one and open for 4.11? --

[GitHub] rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436638636 Actually, you keep using this one. You only need to change the target branch here, and then update your `bwsw:bug/ac

[GitHub] rafaelweingartner edited a comment on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
rafaelweingartner edited a comment on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436638636 Actually, you can keep using this one. You only need to change the target branch here, and then update your `

[GitHub] nvazquez closed pull request #3004: [Placeholder] Add new data motion strategy for KVM non managed storage live migrations

2018-11-07 Thread GitBox
nvazquez closed pull request #3004: [Placeholder] Add new data motion strategy for KVM non managed storage live migrations URL: https://github.com/apache/cloudstack/pull/3004 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below fo

[GitHub] bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436640338 @rafaelweingartner Ok. Got it. I'll try. Not a git pro, thought... --

[GitHub] rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436640655 Ping me on Slack if you get lost. This is an au

[GitHub] csquire commented on a change in pull request #3005: Discover tags field on superclass of API responses

2018-11-07 Thread GitBox
csquire commented on a change in pull request #3005: Discover tags field on superclass of API responses URL: https://github.com/apache/cloudstack/pull/3005#discussion_r231526097 ## File path: plugins/api/discovery/src/org/apache/cloudstack/discovery/ApiDiscoveryServiceImpl.java ##

[GitHub] bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436690399 @rafaelweingartner Please, take a look. Looks like I managed it.

[GitHub] bwsw commented on a change in pull request #3000: CLOUDSTACK-2998: Added two more device name patterns to valid bridge slaves (lo* and dummy*)

2018-11-07 Thread GitBox
bwsw commented on a change in pull request #3000: CLOUDSTACK-2998: Added two more device name patterns to valid bridge slaves (lo* and dummy*) URL: https://github.com/apache/cloudstack/pull/3000#discussion_r231597922 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hyp

[GitHub] rafaelweingartner commented on a change in pull request #3000: CLOUDSTACK-2998: Added two more device name patterns to valid bridge slaves (lo* and dummy*)

2018-11-07 Thread GitBox
rafaelweingartner commented on a change in pull request #3000: CLOUDSTACK-2998: Added two more device name patterns to valid bridge slaves (lo* and dummy*) URL: https://github.com/apache/cloudstack/pull/3000#discussion_r231600102 ## File path: plugins/hypervisors/kvm/src/main/java/

[GitHub] bwsw commented on a change in pull request #3000: CLOUDSTACK-2998: Added two more device name patterns to valid bridge slaves (lo* and dummy*)

2018-11-07 Thread GitBox
bwsw commented on a change in pull request #3000: CLOUDSTACK-2998: Added two more device name patterns to valid bridge slaves (lo* and dummy*) URL: https://github.com/apache/cloudstack/pull/3000#discussion_r231603522 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hyp

[GitHub] bwsw commented on a change in pull request #3000: CLOUDSTACK-2998: Added two more device name patterns to valid bridge slaves (lo* and dummy*)

2018-11-07 Thread GitBox
bwsw commented on a change in pull request #3000: CLOUDSTACK-2998: Added two more device name patterns to valid bridge slaves (lo* and dummy*) URL: https://github.com/apache/cloudstack/pull/3000#discussion_r231604385 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hyp

[GitHub] bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
bwsw commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436709978 Looks like Jenkins is unhappy with moving merge branch from master to 4.11. -

[GitHub] rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
rafaelweingartner commented on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436731471 It is a coincidence. It was not a problem because of the changes, but rather a worked that was working as it should.

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

2018-11-07 Thread GitBox
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-436753900 @rhtyd can we merge it? Th

[GitHub] rafaelweingartner closed pull request #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7

2018-11-07 Thread GitBox
rafaelweingartner closed pull request #2896: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 URL: https://github.com/apache/cloudstack/pull/2896 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displaye

[cloudstack] branch master updated: add exitcode / distribution detection and remove export CGROUP_DAEMON for RHEL7/CentOS7 (#2896)

2018-11-07 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 The following commit(s) were added to refs/heads/master by this push: new e44fc4a add exitcode / distribution detecti

[GitHub] rafaelweingartner edited a comment on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts.

2018-11-07 Thread GitBox
rafaelweingartner edited a comment on issue #3012: WIP: Fixed resource calculation CPU, RAM for accounts. URL: https://github.com/apache/cloudstack/pull/3012#issuecomment-436731471 It is a coincidence. It was not a problem because of the changes, but rather a worker that was working as it

[GitHub] nvazquez commented on a change in pull request #3007: Add missing ConfigDrive entries on existing zones after upgrade

2018-11-07 Thread GitBox
nvazquez commented on a change in pull request #3007: Add missing ConfigDrive entries on existing zones after upgrade URL: https://github.com/apache/cloudstack/pull/3007#discussion_r231705014 ## File path: server/src/com/cloud/network/NetworkServiceImpl.java ## @@ -623,11

[GitHub] nvazquez commented on a change in pull request #3007: Add missing ConfigDrive entries on existing zones after upgrade

2018-11-07 Thread GitBox
nvazquez commented on a change in pull request #3007: Add missing ConfigDrive entries on existing zones after upgrade URL: https://github.com/apache/cloudstack/pull/3007#discussion_r231705166 ## File path: server/src/com/cloud/network/NetworkServiceImpl.java ## @@ -623,11

[GitHub] blueorangutan commented on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade

2018-11-07 Thread GitBox
blueorangutan commented on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade URL: https://github.com/apache/cloudstack/pull/3007#issuecomment-436804174 @nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --

[GitHub] nvazquez commented on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade

2018-11-07 Thread GitBox
nvazquez commented on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade URL: https://github.com/apache/cloudstack/pull/3007#issuecomment-436804006 Thanks @rafaelweingartner @GabrielBrascher @PaulAngus for reviewing! @PaulAngus I have pushed the fix moving the l

[GitHub] nvazquez edited a comment on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade

2018-11-07 Thread GitBox
nvazquez edited a comment on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade URL: https://github.com/apache/cloudstack/pull/3007#issuecomment-436804006 Thanks @rafaelweingartner @GabrielBrascher @PaulAngus for reviewing! @PaulAngus I have pushed the fix movin

[GitHub] blueorangutan commented on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade

2018-11-07 Thread GitBox
blueorangutan commented on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade URL: https://github.com/apache/cloudstack/pull/3007#issuecomment-436812236 Packaging result: ✔centos6 ✖centos7 ✔debian. JID-2411 -

[GitHub] nvazquez commented on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade

2018-11-07 Thread GitBox
nvazquez commented on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade URL: https://github.com/apache/cloudstack/pull/3007#issuecomment-436825524 @blueorangutan package This is an automated message

[GitHub] blueorangutan commented on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade

2018-11-07 Thread GitBox
blueorangutan commented on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade URL: https://github.com/apache/cloudstack/pull/3007#issuecomment-436825646 @nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --

[GitHub] blueorangutan commented on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade

2018-11-07 Thread GitBox
blueorangutan commented on issue #3007: Add missing ConfigDrive entries on existing zones after upgrade URL: https://github.com/apache/cloudstack/pull/3007#issuecomment-436832130 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2412 -

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

2018-11-07 Thread GitBox
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-436885607 @blueorangutan test centos7 kvm-centos6 ---