Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Enhancement] Improve CI efficiency by removing redundant mvn test step #8596

Closed
1 task done
chi3316 opened this issue Aug 29, 2024 · 0 comments · Fixed by #8637
Closed
1 task done

[Enhancement] Improve CI efficiency by removing redundant mvn test step #8596

chi3316 opened this issue Aug 29, 2024 · 0 comments · Fixed by #8637

Comments

@chi3316
Copy link
Contributor

chi3316 commented Aug 29, 2024

Before Creating the Enhancement Request

  • I have confirmed that this should be classified as an enhancement rather than a bug/feature.

Summary

Currently, the mvn package phase already includes the test goal, leading to unnecessary repetition when mvn test is explicitly run again. This change aims to simplify the CI process andimproving overall efficiency.

Motivation

The project's unit tests are currently unstable, and running tests multiple times within a single CI job increases the likelihood of CI failures due to the inherent instability. By eliminating the redundant test step, we can reduce the risk of false negatives in CI results, saving developer time and resources.

Describe the Solution You'd Like

  • Remove the explicit mvn test step from the CI workflow.
  • Ensure that the existing mvn package step handles all necessary test executions.

Describe Alternatives You've Considered

Keeping the mvn test step with different configurations (e.g., different JVM options) to attempt to stabilize the tests. However, this approach could still lead to increased CI failures and does not address the root cause of test instability.

Additional Context

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant