Differential D4798

Don't log throttle config passthrough to Sentry.
ClosedPublic
Actions

Authored by vlorentz on Dec 30 2020, 11:39 AM.

Details

Reviewers

olasd

Group Reviewers

Reviewers

Commits

rDWAPPSe131fa7ae236: Don't log throttle config passthrough to Sentry.

Summary

It's not an actual error, it doesn't deserve to be logged.

Replaces D4794.

Diff Detail

Repository

rDWAPPS Web applications

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 18115
Build 27969: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 27968: arc lint + arc unit

Event Timeline

vlorentz created this revision.Dec 30 2020, 11:39 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptDec 30 2020, 11:39 AM

Build is green

Patch application report for D4798 (id=16978)

Rebasing onto 2c9c8527be...

Current branch diff-target is up to date.

Changes applied before test

commit e131fa7ae2364aa3f5bc4138b411324c61b7d006
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Wed Dec 30 11:39:10 2020 +0100

    Don't log throttle config passthrough to Sentry.
    
    It's not an actual error, it doesn't deserve to be logged.

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/538/ for more details.

Harbormaster completed remote builds in B18115: Diff 16978.Dec 30 2020, 11:46 AM

vlorentz requested review of this revision.Dec 30 2020, 11:46 AM