Provide SENTRY_DSN for Notifications Center
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F7662206: D2944.id7496.diff
	Wed, Apr 30, 14:54

	F7621220: D2944.id7498.diff
	Tue, Apr 29, 21:29

	Unknown Object (File)
	Tue, Apr 29, 19:02

	Unknown Object (File)
	Tue, Apr 29, 00:40

	Unknown Object (File)
	Mon, Apr 28, 15:19

	Unknown Object (File)
	Mon, Apr 28, 12:49

	Unknown Object (File)
	Mon, Apr 28, 03:43

	Unknown Object (File)
	Sun, Apr 27, 12:30

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