lhotari commented on pull request #10148: URL: https://github.com/apache/pulsar/pull/10148#issuecomment-814685496
> We do need the "excludedGroups" we already have group separation of broker tests running in gitlab workflows. They do no conflict with each other. I have used the extra group successfully in our internal branch to exclude tests from all runs, it does not require any changes in pom.xml. In regards to the "BeforeMethod" more most of the them extra in already in place if it's missing we can add them. When mixing class level @Test group definitions and method level @Test group definitions, it's necessary. My previous comment explains it. Perhaps you didn't have that type of use case? -- 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. For queries about this service, please contact Infrastructure at: [email protected]
