DaanHoogland commented on PR #8547: URL: https://github.com/apache/cloudstack/pull/8547#issuecomment-1910311087
> -1 (binding) binding does not apply to PRs. That said, if you have objections and a way forward we will not merge (on any solidly argued -1) > @vishesh92 this PR does not fix #8506. The issue and the thread it mentions is related to a generic solution, which was not even discussed. @GutoVeronezi , I suppose you are referring to the solution mentioned in this comment: https://github.com/apache/cloudstack/pull/8502#discussion_r1450506118 ? If so, this is a good idea and would obsolete this PR, but not without implementation. Do you have code ready to replace the code here? -- 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: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
