fabric8-analytics / fabric8-analytics-worker

fabric8-analytics worker for gathering raw data
GNU General Public License v3.0
8 stars 45 forks source link

fix: Github statistics collection. #946

Closed jparsai closed 3 years ago

jparsai commented 3 years ago

Description

In existing implementation one token is selected and used for all Github API calls so sometimes we end up exceeding rate limit of Github API. To avoid that different token is used for each API call.

Fixes:
APPAI-1725

Changes:

yzainee-zz commented 3 years ago

@jparsai can you also fix the part where the issues / pr created in the last 1 year are correctly captured.

codecov-io commented 3 years ago

Codecov Report

Merging #946 (67321bc) into master (be5c026) will decrease coverage by 0.59%. The diff coverage is 53.62%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #946      +/-   ##
==========================================
- Coverage   61.35%   60.76%   -0.60%     
==========================================
  Files          81       81              
  Lines        5613     5622       +9     
  Branches      735      738       +3     
==========================================
- Hits         3444     3416      -28     
- Misses       1992     2027      +35     
- Partials      177      179       +2     
Impacted Files Coverage Δ
f8a_worker/workers/new_githuber.py 22.22% <37.50%> (+0.38%) :arrow_up:
...a_worker/workers/new_init_package_analysis_flow.py 82.35% <40.00%> (-17.65%) :arrow_down:
f8a_worker/workers/new_init_package_flow.py 83.33% <40.00%> (-16.67%) :arrow_down:
f8a_worker/utils.py 72.75% <53.33%> (-5.49%) :arrow_down:
f8a_worker/defaults.py 76.04% <53.84%> (-4.42%) :arrow_down:
f8a_worker/workers/githuber.py 47.52% <87.50%> (-10.28%) :arrow_down:
f8a_worker/storages/s3.py 41.66% <0.00%> (-1.67%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update be5c026...67321bc. Read the comment docs.