Page MenuHomeDevCentral

Fix credential helper and sentry wrapper
ClosedPublic

Authored by dereckson on Mar 6 2023, 00:16.
Tags
None
Referenced Files
F24670335: D2848.id7256.diff
Wed, Mar 4, 09:05
F24670334: D2848.id7260.diff
Wed, Mar 4, 09:05
F24668948: D2848.diff
Wed, Mar 4, 07:21
F24668760: D2848.id7256.diff
Wed, Mar 4, 07:05
F24658341: D2848.diff
Tue, Mar 3, 15:58
F24658076: D2848.diff
Tue, Mar 3, 15:46
F24658075: D2848.diff
Tue, Mar 3, 15:46
Unknown Object (File)
Wed, Feb 11, 08:04
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.