weizhouapache commented on PR #7150: URL: https://github.com/apache/cloudstack/pull/7150#issuecomment-1816382493
> > @BryanMLima , one last thing; Do you think it makes sense to add the integration test to the test matrix somewhere? (in .github/workflows/ci.yaml) > > To be honest, I do not know the criteria for adding integration tests to the test matrix. From my understanding, all smoke tests need to pass for a PR to be merged, thus, I do not think it is necessary to add the integration test to the test matrix as well. the tests in `test/integration/smoke/` should be executed in trillian test currently there are some issues with our yum repository. fixing it -- 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]
