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] Optimize CI pipeline stability by enhancing log management and fixing test issues #8668

Closed
1 task done
chi3316 opened this issue Sep 9, 2024 · 0 comments · Fixed by #8667
Closed
1 task done

Comments

@chi3316
Copy link
Contributor

chi3316 commented Sep 9, 2024

Before Creating the Enhancement Request

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

Summary

This enhancement proposes a series of improvements aimed at increasing the stability and reliability of our CI pipelines. The changes include enhancements to test reporting and resolution of issues related to test stability.

Motivation

  • Added a step to upload test reports in the integration pipeline due to large log files that required manual downloads for review.

    image

  • Resolved issues causing instability in unit tests on macOS CI environments.

    image

Describe the Solution You'd Like

  • Ensure that all test logs are automatically uploaded to facilitate easier access and review.
  • Implement fixes and adjustments to reduce test flakiness and ensure consistent behavior across different CI environments.

Describe Alternatives You've Considered

  • Manual Log Management: Continuing with the current manual process for log handling, which is less efficient and prone to errors.
  • Partial Fixes: Addressing only specific issues without a comprehensive approach to improve overall pipeline stability.

Additional Context

No response

RongtongJin pushed a commit that referenced this issue Sep 9, 2024
… and test fixes (#8667)

* Add sleep 3s after consumer shutdown to ensure queue rebanlance

* Add a step to upload test report

* Unified code style

* Unified mockito runner configuration to suppress UnnecessaryStubbingException
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant