Page MenuHomeDevCentral

Fix credential helper and sentry wrapper
ClosedPublic

Authored by dereckson on Mar 6 2023, 00:16.
Tags
None
Referenced Files
F34903351: D2848.diff
Fri, Jul 3, 20:29
F34903299: D2848.id.diff
Fri, Jul 3, 20:28
F34892858: D2848.diff
Fri, Jul 3, 16:40
F34851820: D2848.id7260.diff
Fri, Jul 3, 11:50
Unknown Object (File)
Thu, Jul 2, 18:57
Unknown Object (File)
Thu, Jul 2, 17:33
Unknown Object (File)
Thu, Jul 2, 16:29
Unknown Object (File)
Thu, Jul 2, 04:23
Subscribers
None

Details

Summary

The credential helper should use jq -r (--raw-output) to avoid quotes.
It needs jq installed, which is not the case by default.

The sentry wrapper needs the correct credential name.

This is a follow-up for 0e4eca7aea58.

Fixes T1788.

Test Plan

Fixed on docker-002

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Passed
Unit
No Test Coverage
Branch
fix-sentry-wrapper
Build Status
Buildable 4497
Build 4764: arc lint + arc unit

Event Timeline

dereckson created this revision.
This revision is now accepted and ready to land.Mar 7 2023, 17:19
This revision was automatically updated to reflect the committed changes.