jmsperu commented on PR #13074: URL: https://github.com/apache/cloudstack/pull/13074#issuecomment-4411313104
@harikrishna-patnala the unit test NPE is fixed in [72f967aa6d](https://github.com/apache/cloudstack/pull/13074/commits/72f967aa6d) (mocked `BackupDetailsDao` so `testDeleteBackup` doesn't dereference null). The main `build` check is now passing on the latest revision; the remaining failures (`test_acl_*`, `test_routers`, `test_mm_domain_limits`) are unrelated integration smoke tests — happy to re-run if you want. @bernardodemarco the RFC has been moved out of the source tree in [9764025358](https://github.com/apache/cloudstack/pull/13074/commits/9764025358); content is now in the PR description and I'll mirror it on the wiki once we settle on final scope. @weizhouapache yes, this is ready for review. -- 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]
