Provide SENTRY_DSN for Notifications Center
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F37668210: D2944.id7497.diff
	Tue, Jul 21, 19:53

	F37649210: D2944.id7496.diff
	Tue, Jul 21, 18:21

	F37594350: D2944.id.diff
	Tue, Jul 21, 10:44

	Unknown Object (File)
	Mon, Jul 20, 16:57

	Unknown Object (File)
	Mon, Jul 20, 14:12

	Unknown Object (File)
	Mon, Jul 20, 14:06

	Unknown Object (File)
	Mon, Jul 20, 14:04

	Unknown Object (File)
	Sat, Jul 18, 15:13

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