Opened 2 years ago

Closed 2 years ago

#23255 closed defect (fixed)

Fix a bug while sanitizing bridge network statuses without entries

Reported by: karsten Owned by: metrics-team
Priority: Very High Milestone:
Component: Metrics/CollecTor Version:
Severity: Normal Keywords:
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Due to an error on the bridge authority, it produced statuses without any entries, just with a header. Those caused a NullPointerException in CollecTor's bridgedescs module. I found the bug and will provide a branch for review in a second. We should merge and deploy really soon (tonight or early tomorrow).

Child Tickets

Change History (4)

comment:1 Changed 2 years ago by karsten

Status: newneeds_review

Please review my task-23255 branch.

comment:2 Changed 2 years ago by iwakeh

Status: needs_reviewmerge_ready

Test in 44bcd9 passes in f6e9d6. Other checks and tests pass, too. The fix looks fine.
Ready for merge!

comment:3 Changed 2 years ago by karsten

Thanks for the quick review! Merged to master and deployed on colchicifolium. I'll report how that works out. We should consider putting out 1.2.1.

comment:4 Changed 2 years ago by karsten

Resolution: fixed
Status: merge_readyclosed

CollecTor 1.2.1 was released a few days ago and contains this fix. Closing.

Note: See TracTickets for help on using tickets.