clusterfuzz icon indicating copy to clipboard operation
clusterfuzz copied to clipboard

Optimize get_artifacts_for_build() with regex filtering

Open aditya-wazir opened this issue 1 year ago • 3 comments

Introduces regex-based filtering directly in the API to improve performance and reduce number of calls to ab api server.

aditya-wazir avatar Oct 08 '24 08:10 aditya-wazir

/gcbrun

svasudevprasad avatar Oct 08 '24 19:10 svasudevprasad

This change looks good.

@aditya-wazir it would be really interesting to see about adding a test for files located in the src/clusterfuzz/_internal/platforms/android/ directory. As far as I can tell, there are not any (at least, not in src/clusterfuzz/_internal/tests/core).

Could you please investigate the level of effort to add a new test file for android here? Maybe we could just start with a single fetch_artifact_test.py file, and build on the structure thereafter.

Hi Mark, We will start looking into feasibility of adding tests for src/clusterfuzz/_internal/platforms/android/ directory. We will start with fetch_artifact_test.py and then upload tests for other python components as part of different PR

aditya-wazir avatar Oct 15 '24 16:10 aditya-wazir

/gcbrun

svasudevprasad avatar Oct 15 '24 16:10 svasudevprasad