camel-quarkus icon indicating copy to clipboard operation
camel-quarkus copied to clipboard

perf-regression: add integration tests #3982

Open aldettinger opened this issue 3 years ago • 1 comments

aldettinger avatar Aug 11 '22 12:08 aldettinger

Please do not merge. At this stage, this PR should check whether it build on CI and how to run mvn clean verify on perf-regression.

aldettinger avatar Aug 11 '22 12:08 aldettinger

@jamesnetherton @ppalaga With mvn clean verify, we could early detect potential breakage in the perf-regression tool before using it during the release.

How could we run those integration tests for each PR during CI checks ? I guess trying to list perf-regression in a test group is a bad idea ?

aldettinger avatar Aug 16 '22 09:08 aldettinger

How could we run those integration tests for each PR during CI checks

Probably best to add it as another task here:

https://github.com/apache/camel-quarkus/blob/main/.github/workflows/ci-build.yaml#L196

The overhead is probably minimal.

jamesnetherton avatar Aug 16 '22 10:08 jamesnetherton

Looks good to me, it's ready for review.

aldettinger avatar Aug 17 '22 07:08 aldettinger