Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-02-09 Thread via GitHub
weizhouapache commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1935522444 > @weizhouapache, sorry for the late response! Unfortunately, I don't have an environment with shared mount point but the code LGTM thanks @slavkap -- This is an

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-02-09 Thread via GitHub
weizhouapache merged PR #8481: URL: https://github.com/apache/cloudstack/pull/8481 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-02-08 Thread via GitHub
blueorangutan commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1934380340 Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 8591 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-02-08 Thread via GitHub
blueorangutan commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1934245979 @GutoVeronezi a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-02-08 Thread via GitHub
GutoVeronezi commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1934241947 @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-31 Thread via GitHub
DaanHoogland commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1919272620 > I don´t know, I didn't test it @GutoVeronezi sorry, I didn't read very well and answered hastely. Yes @GutoVeronezi, clgtm -- This is an automated message from the

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-31 Thread via GitHub
weizhouapache commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1919143444 > Aside from the third-party testing, do we still need anything else on this one? > > cc: @DaanHoogland @weizhouapache @GutoVeronezi code looks ok to me.

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-31 Thread via GitHub
DaanHoogland commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1919107302 I don´t know, I didn't test it @GutoVeronezi -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-31 Thread via GitHub
GutoVeronezi commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1919084553 Aside from the third-party testing, do we still need anything else on this one? cc: @DaanHoogland @weizhouapache -- This is an automated message from the Apache Git

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-11 Thread via GitHub
weizhouapache commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1886726914 > > Yes @GutoVeronezi, there are global settings to (1) notify users, or (2) disable new volume allocation, if allocated capacity or actual used capacity reaches a

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-11 Thread via GitHub
DaanHoogland commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1886654348 > > > @GutoVeronezi could you target 4.18.2? > > > > > > @JoaoJandre, done. > > > code looks good. I have a worry though. overprovisioning on storage in general is a

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-10 Thread via GitHub
blueorangutan commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1885744384 Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 8280 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-10 Thread via GitHub
weizhouapache commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1885689032 > > @GutoVeronezi could you target 4.18.2? > > @JoaoJandre, done. > > > > code looks good. I have a worry though. overprovisioning on storage in general is a

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-10 Thread via GitHub
blueorangutan commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1885643612 @GutoVeronezi a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-10 Thread via GitHub
GutoVeronezi commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1885619281 @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-10 Thread via GitHub
GutoVeronezi commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1885617851 > @GutoVeronezi could you target 4.18.2? @JoaoJandre, done. > code looks good. I have a worry though. overprovisioning on storage in general is a dangerous

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-10 Thread via GitHub
weizhouapache commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1884729728 > code looks good. I have a worry though. overprovisioning on storage in general is a dangerous thing. On cpu and memory any errors due to overprovisioning are likely to not be

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-10 Thread via GitHub
JoaoJandre commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1884665993 @GutoVeronezi could you target 4.18.2? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-10 Thread via GitHub
DaanHoogland commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1884424844 code looks good. I have a worry though. overprovisioning on storage in general is a dangerous thing. On cpu and memory any errors due to overprovisioning are likely to not be

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-09 Thread via GitHub
blueorangutan commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1883821515 Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 8257 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

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

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-09 Thread via GitHub
blueorangutan commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1883727927 @GutoVeronezi a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-09 Thread via GitHub
GutoVeronezi commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1883725377 @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-09 Thread via GitHub
GutoVeronezi opened a new pull request, #8481: URL: https://github.com/apache/cloudstack/pull/8481 ### Description ACS does not allow operators to set the configuration `storage.overprovisioning.factor` for `SharedMountPoint` primary storages, presenting the following message when