Re: [PR] Restores value of config vm_memballoon_stats_period back to 60 [cloudstack]

2024-01-17 Thread via GitHub
andrijapanicsb commented on PR #8520: URL: https://github.com/apache/cloudstack/pull/8520#issuecomment-1896545334 (For what is worth it - I've been told by a RH engineer (in 2019) that memory auto-ballooning is an abandoned project - so be careful with the feature) -- This is an

Re: [PR] Restores value of config vm_memballoon_stats_period back to 60 [cloudstack]

2024-01-17 Thread via GitHub
weizhouapache commented on PR #8520: URL: https://github.com/apache/cloudstack/pull/8520#issuecomment-1896069953 > BTW: using the value 0 makes the monitoring stop working in KVM. worth to mention that I have faced an issue caused by it in the past .

Re: [PR] Restores value of config vm_memballoon_stats_period back to 60 [cloudstack]

2024-01-17 Thread via GitHub
weizhouapache commented on PR #8520: URL: https://github.com/apache/cloudstack/pull/8520#issuecomment-1895976343 > The value of this configuration was originally proposed with `60` to avoid misunderstandings as the one that are happening with operators. However, this value has been

Re: [PR] Restores value of config vm_memballoon_stats_period back to 60 [cloudstack]

2024-01-17 Thread via GitHub
RodrigoDLopez commented on PR #8520: URL: https://github.com/apache/cloudstack/pull/8520#issuecomment-1895942157 The value of this configuration was originally proposed with `60` to avoid misunderstandings as the one that are happening with operators. However, this value has been modified,

Re: [PR] Restores value of config vm_memballoon_stats_period back to 60 [cloudstack]

2024-01-17 Thread via GitHub
andrijapanicsb commented on PR #8520: URL: https://github.com/apache/cloudstack/pull/8520#issuecomment-1895686704 many of the global settings are set to a certain value, many of them to 0 - and can be easily changed (like most of the other global settings). I prefer to keep it as it is,

Re: [PR] Restores value of config vm_memballoon_stats_period back to 60 [cloudstack]

2024-01-17 Thread via GitHub
weizhouapache commented on PR #8520: URL: https://github.com/apache/cloudstack/pull/8520#issuecomment-1895509170 this value can be set manually, or by ansible/chef/puppet. I prefer to leave users to decide. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] Restores value of config vm_memballoon_stats_period back to 60 [cloudstack]

2024-01-17 Thread via GitHub
DaanHoogland commented on PR #8520: URL: https://github.com/apache/cloudstack/pull/8520#issuecomment-1895395130 @andrijapanicsb @alexandremattioli any ideas on this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] Restores value of config vm_memballoon_stats_period back to 60 [cloudstack]

2024-01-17 Thread via GitHub
codecov[bot] commented on PR #8520: URL: https://github.com/apache/cloudstack/pull/8520#issuecomment-1895393860 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8520?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report All

[PR] Restores value of config vm_memballoon_stats_period back to 60 [cloudstack]

2024-01-16 Thread via GitHub
RodrigoDLopez opened a new pull request, #8520: URL: https://github.com/apache/cloudstack/pull/8520 ### Description Currently, the default value for the `vm_memballoon_stats_period` configuration is set to `0`. As a result, out of the box in ACS, memory metrics for Windows instances