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
---
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
-
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
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.
--
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
-
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
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
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.
--
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
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
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
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
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
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
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.
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.
-
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
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
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/
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
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.
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
##
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
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...
--
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
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 `
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
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?
--
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?
-
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
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
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
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
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
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...
--
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.
---
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
##
@@
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
##
@@
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
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
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
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
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
--
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
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
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:
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 (
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
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:
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
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
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-
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
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:
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
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
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
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
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
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
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
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
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
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?
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/
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
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/
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
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:
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
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
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:
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
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
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
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
76 matches
Mail list logo