Skip to content

Make runner progress output configurable via system property#312

Merged
basilevs merged 4 commits into
eclipse-rcptt:masterfrom
rolandthoma:fix/runner-progress-logging
May 5, 2026
Merged

Make runner progress output configurable via system property#312
basilevs merged 4 commits into
eclipse-rcptt:masterfrom
rolandthoma:fix/runner-progress-logging

Conversation

@rolandthoma
Copy link
Copy Markdown
Contributor

@rolandthoma rolandthoma commented May 4, 2026

RCPTT runner currently prints progress monitor output ("Begin task", "progress: ...")
unconditionally to stdout via PrintStreamMonitor.

This floods CI logs when resolving large AUT target platforms.

This PR introduces a system property:

-Drcptt.runner.target_progress=false

to disable this output.

Default Runner behavior remains unchanged.
Rcptt Maven plugin is updated to only show target platform resolution progress in debug mode.

@basilevs
Copy link
Copy Markdown
Contributor

basilevs commented May 4, 2026

This change has to be coordinated with RCPTT Maven plugin. In particular, we want the output to be verbose when Maven is started with -X. This can be done in another PR though.

@rolandthoma rolandthoma force-pushed the fix/runner-progress-logging branch from 1efcbc4 to d45b469 Compare May 4, 2026 14:21
@rolandthoma rolandthoma force-pushed the fix/runner-progress-logging branch from d45b469 to 0fd40b6 Compare May 4, 2026 14:23
@basilevs
Copy link
Copy Markdown
Contributor

basilevs commented May 4, 2026

Make sure to sign Eclipse Contributor Agreement.

@rolandthoma
Copy link
Copy Markdown
Contributor Author

Applied the suggestion and aligned the property name between runner and Maven plugin. Thanks!

@basilevs
Copy link
Copy Markdown
Contributor

basilevs commented May 5, 2026

The test CloseUnrelatedProjectsInQ7Explorer fails because of an incorrect fix for eclipse-platform/eclipse.platform.ui#2636 it is not related to this PR. I'm squashing the changes in the master.

Thank you for the contribution!

Consider suggesting a change to https://github.com/eclipse-rcptt/rcptt-website/blob/main/content/userguide/runner/arguments.md if you see a need.

@basilevs basilevs merged commit 615775b into eclipse-rcptt:master May 5, 2026
8 of 9 checks passed
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