weizhouapache commented on PR #7272: URL: https://github.com/apache/cloudstack/pull/7272#issuecomment-1438601830
> @weizhouapache In my opinion, this method `addGuestOsHypervisorMapping` could create issues as it takes the guest OS DB ID. If checks are not done this could cause failures when guest OS entry for that DB ID is not present. This PR just tries to avoid those failures @DaanHoogland thanks. this PR looks good to me. However, do not upgrade cloudstack if guest os and guest os mappings mess up. @borisstoyanov -- 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]
