search

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: Sentry retry errors. #962

Closed jparsai closed 3 years ago

jparsai commented 3 years ago

Description

Fix for sentry errors caused by retrying GH data collection for packages with incorrect Git URL.
https://sentry.devshift.net/sentry/fabric8-analytics-production/issues/271709/ https://sentry.devshift.net/sentry/fabric8-analytics-production/issues/271737/ https://sentry.devshift.net/sentry/fabric8-analytics-production/issues/275495/

codecov-io commented 3 years ago

Codecov Report

Merging #962 (21d956a) into master (c74145b) will decrease coverage by 0.19%. The diff coverage is 21.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #962      +/-   ##
==========================================
- Coverage   61.97%   61.77%   -0.20%     
==========================================
  Files          85       85              
  Lines        5759     5771      +12     
  Branches      739      745       +6     
==========================================
- Hits         3569     3565       -4     
- Misses       2013     2026      +13     
- Partials      177      180       +3
Impacted Files Coverage Δ
f8a_worker/workers/githuber.py 55.23% <0.00%> (-1.08%) :arrow_down:
f8a_worker/workers/new_githuber.py 21.17% <16.66%> (-1.05%) :arrow_down:
f8a_worker/utils.py 77.68% <33.33%> (-1.83%) :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 c74145b...21d956a. Read the comment docs.