aerospike / aerospike-client-python

Aerospike Python Client
Apache License 2.0
134 stars 111 forks source link

[DEVOPS-234] CI/CD: Reuse existing feature branch builds for valgrind runs #670

Closed juliannguyen4 closed 1 week ago

juliannguyen4 commented 2 months ago

No merge conflicts with CLIENT-3027* PR

Reasons for this change: It will be a while to clean up all the memory leaks while running the Python client regression tests, and we normally run valgrind manually. We won't be running valgrind as a CI/CD check for a while because of this. Often times, we run valgrind twice, so the second valgrind run builds a wheel when a wheel was already built for the same commit. This change forces valgrind to reuse a wheel (from JFrog) if it already exists

Known issues Running the valgrind workflow on central branches (e.g dev) will build and upload the wheels to the generic JFrog repo. This shouldn't cause any issues but this isn't intended behavior.

codecov-commenter commented 2 months ago

Codecov Report

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

Project coverage is 80.78%. Comparing base (d8eff6e) to head (c2f438c). Report is 4 commits behind head on dev.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #670 +/- ## ========================================== - Coverage 80.79% 80.78% -0.01% ========================================== Files 102 102 Lines 15173 15236 +63 ========================================== + Hits 12259 12309 +50 - Misses 2914 2927 +13 ```

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


🚨 Try these New Features: