GutoVeronezi commented on PR #8362: URL: https://github.com/apache/cloudstack/pull/8362#issuecomment-1924491548
> @JoaoJandre Description in the PR specifically mentions limits for tagged resources. I'll review and make further changes to make it clearer. Regarding listDiskOfferings change I would prefer it in the same PR as it has already been tested internally. We will add relevant test reports. Also, diskoffering suitability for a VM is quite related to offering and storage pool tags. I'll update the title and description for the PR if needed. @shwstppr, does the change on the list disk offering depends on the other change you are proposing? Because if one does not affect or depends on the other, they are different contexts, even if related to the same set of functionalities (storage's and host's tags, in this case). Mixing contexts in a PR is not encouraged. It makes it difficult for others to review the changes and understand what is being done, especially in these cases with thousands of changes. Furthermore, unrelated changes being tested internally should not be used as reason to open a PR with all of them. While contributing to a community, we should structure our changes and proposals in a way that others can easily understand what is being done/proposed; thus, communicating without noise. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
