rhtyd commented on pull request #5082: URL: https://github.com/apache/cloudstack/pull/5082#issuecomment-889738885
Will review soon, but +1 great work @DaanHoogland we've always had components test but never run them. With your work, we finally wil have verified list of tests/cases that we expect to work which will further help with running such component tests on weekly basis to review/monitor branch health. I suppose there are still some work left but I would want to get something merged soon that could help 4.16, for example: - limit scope of the remaining component tests and fix that limited scope of tests, such that they pass for kvm/vmware/xenserver at least with adv zone (preferably the case of adv zone + SG too) - move all failing tests that can't be fixed (takes another couple of years :) ) with a different attribute to be fixed in future - add/update component tests that missing to Travis - integrate fix BO to run component tests (that are known to pass), something like @bo test componentmatrix to kick component only tests across kvm/vmware/xenserver? -- 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]
