Provide SENTRY_DSN for Notifications Center
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F25062195: D2944.id7497.diff
	Sat, Mar 28, 03:28

	F25062191: D2944.id7496.diff
	Sat, Mar 28, 03:28

	F25056574: D2944.id7498.diff
	Fri, Mar 27, 18:30

	F25054640: D2944.id7498.diff
	Fri, Mar 27, 14:52

	Unknown Object (File)
	Thu, Mar 26, 16:26

	Unknown Object (File)
	Mon, Mar 23, 13:08

	Unknown Object (File)
	Mon, Mar 23, 13:08

	Unknown Object (File)
	Mon, Mar 23, 10:56

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