opensearch-project / performance-analyzer-rca

The Performance Analyzer RCA is a framework that builds on the Performance Analyzer engine to support root cause analysis (RCA) of performance and reliability problems for OpenSearch instances.
https://opensearch.org/docs/latest/monitoring-plugins/pa/rca/index/
Apache License 2.0
30 stars 56 forks source link

Fix Java CI Check by using Amazon Linux 2 instead of Centos7 which is EOL #580

Open cwperks opened 2 weeks ago

cwperks commented 2 weeks ago

Is your feature request related to a problem? Please provide an existing Issue # , or describe.

This PR does an upgrade of the actions/upload-artifact github action from v2 to v3 to resolve errors seen in CI checks like https://github.com/opensearch-project/performance-analyzer-rca/actions/runs/11485849764/job/31966863799?pr=577

Check List

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

cwperks commented 2 weeks ago

CI failures appear to be related to CentOS7 having archived the repositories in a vault: https://github.com/cloudspinx/centos7-vault-repositories

https://www.redhat.com/en/blog/centos-linux-going-end-life-what-does-mean-me

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.39%. Comparing base (3c92f12) to head (daaad12). Report is 4 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #580 +/- ## ============================================ - Coverage 74.42% 74.39% -0.03% Complexity 2665 2665 ============================================ Files 317 317 Lines 16377 16377 Branches 1282 1282 ============================================ - Hits 12188 12184 -4 - Misses 3674 3678 +4 Partials 515 515 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

cwperks commented 2 weeks ago

Got the Java CI / Building RCA package check to pass.

cwperks commented 2 weeks ago

Including changes from https://github.com/opensearch-project/performance-analyzer-rca/pull/579