Provide SENTRY_DSN for Notifications Center
ClosedPublic
Actions

Authored by dereckson on Mar 30 2023, 01:07.

Tags

None

Referenced Files

	F11625145: D2944.diff
	Tue, Sep 2, 12:49

	F11620274: D2944.id7498.diff
	Tue, Sep 2, 02:59

	F11619109: D2944.id7497.diff
	Tue, Sep 2, 01:04

	F11617623: D2944.id7496.diff
	Mon, Sep 1, 22:22

	Unknown Object (File)
	Wed, Aug 27, 03:23

	Unknown Object (File)
	Fri, Aug 22, 11:41

	Unknown Object (File)
	Wed, Aug 20, 21:15

	Unknown Object (File)
	Tue, Aug 19, 10:54

View All Files

Subscribers

None

Details

Reviewers

dereckson

Maniphest Tasks

T1791: Refresh Sentry installation

Commits

rOPSd7e6cddec8b7: Provide SENTRY_DSN for Notifications Center

Summary

33fcf3f4 introduced SENTRY_DSN but it has been removed afterwards,
while Sentry wasn't currently in use.

Restore back using credential.sentry_dsn with new pillar syntax.

Ref T1791

Test Plan

Deploy container on docker-002

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Mar 30 2023, 01:07

dereckson created this revision.

Harbormaster completed remote builds in B4644: Diff 7496.Mar 30 2023, 01:07

Fix DSN

Harbormaster completed remote builds in B4645: Diff 7497.Mar 30 2023, 01:17

dereckson accepted this revision.Mar 30 2023, 01:18

This revision is now accepted and ready to land.Mar 30 2023, 01:18

Closed by commit rOPSd7e6cddec8b7: Provide SENTRY_DSN for Notifications Center (authored by dereckson). · Explain WhyMar 30 2023, 01:19

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rOPSd7e6cddec8b7: Provide SENTRY_DSN for Notifications Center.

Revision Contents
Changeset List

Path

Size

_modules/

credentials.py

9 lines

roles/

paas-docker/

containers/

notifications.sls

2 lines

Diff 7498

View Options

_modules/credentials.py