Skip to content

ci: speed up PR compatibility checks - #2567

Open
zeitlinger wants to merge 1 commit into
mainfrom
ci/jmx-exporter-pr-smoke
Open

zeitlinger wants to merge 1 commit into
mainfrom
ci/jmx-exporter-pr-smoke

Conversation

@zeitlinger

@zeitlinger zeitlinger commented Oct 16, 2026 •

Copy link
Copy Markdown
Member

I am reducing PR feedback time for compatibility checks.

  • Run one representative JMX Exporter Paramixel smoke test instead of the full compatibility suite. This reduced the JMX Exporter check from about 18m46s to 3m56s in observed runs.
  • Build only the native-test reactor and its dependencies instead of the full Maven reactor. This reduced the native-tests job from about 8m29s to 5m35s in observed runs.

The full JMX Exporter compatibility matrix is intentionally not run as part of the PR check.

Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
@zeitlinger
zeitlinger force-pushed the ci/jmx-exporter-pr-smoke branch from 74312c6 to a813a03 Compare October 16, 2026 12:15
@zeitlinger zeitlinger changed the title ci: run a JMX exporter smoke test on PRs ci: speed up PR compatibility checks Oct 16, 2026
@zeitlinger
zeitlinger marked this pull request as ready for review October 16, 2026 12:34
Comment thread mise.toml
tools.java = "oracle-graalvm-25"
# Native Test uses Surefire, so explicitly select the Failsafe-named test class.
run = '''
./mvnw -pl integration-tests/it-spring-boot-smoke-test -am install -DskipTests -Dcoverage.skip=true

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this use mise like below?

Suggested change
./mvnw -pl integration-tests/it-spring-boot-smoke-test -am install -DskipTests -Dcoverage.skip=true
"$MISE_CONFIG_ROOT/mvnw" -pl integration-tests/it-spring-boot-smoke-test -am install -DskipTests -Dcoverage.skip=true

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the working dir is the config root - so I don't see how this is making it easier

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gotcha

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants