Provide SENTRY_DSN for Notifications Center
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F3607332: D2944.id7498.diff
	Mon, Sep 30, 08:31

	F3607327: D2944.id7497.diff
	Mon, Sep 30, 08:30

	Unknown Object (File)
	Sat, Sep 28, 18:31

	Unknown Object (File)
	Sat, Sep 28, 10:06

	Unknown Object (File)
	Wed, Sep 25, 13:14

	Unknown Object (File)
	Fri, Sep 20, 06:32

	Unknown Object (File)
	Sun, Sep 15, 08:39

	Unknown Object (File)
	Sat, Sep 14, 09:58

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