dheeraj12347 commented on PR #12860: URL: https://github.com/apache/cloudstack/pull/12860#issuecomment-4105844714
> @dheeraj12347 > > Normally, PR authors test their changes before requesting peer review. However, I understand that in some cases it may not be possible to reproduce the issue due to differences in hardware or environment, or because the issue itself is difficult to reproduce. > > That said, I don’t think it’s necessary to acquire new hardware to reproduce and verify the fix in this case. It would be helpful if @UAnton could test the changes. > > However, as mentioned in my previous comment, I don’t think this PR will fix the issue #12815. We need to understand why the expected `OvsFetchInterfaceAnswer` is not being returned, and why an `UnsupportedAnswer` is returned instead. Thanks for the review and explanation. This PR mainly makes the management server handle the UnsupportedAnswer safely and log a clear warning instead of throwing a ClassCastException. Right now I don’t have a CloudStack 4.22 + XCP‑ng 8.3 + OVS/GRE lab, so I can’t fully reproduce and verify the GRE isolation behavior myself. If you feel it’s important, I can try to build such an environment, and I’m also happy to support @UAnton who can test it in their setup. -- 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]
