Page MenuHomePhabricator
Differential D12417

[terraform] Disable CSAT verification on prod
ClosedPublic
Actions

Authored by bartek on Thu, Jun 13, 1:25 AM.
Tags
None
Referenced Files
F2102363: D12417.id41435.diff
Tue, Jun 25, 12:11 AM
Unknown Object (File)
Mon, Jun 24, 12:06 AM
Unknown Object (File)
Sun, Jun 23, 9:14 PM
Unknown Object (File)
Sat, Jun 22, 1:42 PM
Unknown Object (File)
Sat, Jun 22, 10:03 AM
Unknown Object (File)
Sat, Jun 22, 9:57 AM
Unknown Object (File)
Fri, Jun 21, 7:17 PM
Unknown Object (File)
Fri, Jun 21, 10:37 AM
View All 22 Files
Subscribers
ashoat
tomek

Details

Reviewers
will
varun
kamil
Commits
rCOMM7c14a71b9904: [terraform] Disable CSAT verification on prod
Summary

Updated Terraform config for HTTP services to require CSAT only on staging

Depends on D12448

Test Plan

TBD - test on staging

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek created this revision.Thu, Jun 13, 1:25 AM
bartek held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptThu, Jun 13, 1:25 AM
Harbormaster failed remote builds in B29645: Diff 41268!Thu, Jun 13, 1:49 AM
bartek updated this revision to Diff 41272.Thu, Jun 13, 1:55 AM

Rebase

Harbormaster completed remote builds in B29649: Diff 41272.Thu, Jun 13, 2:17 AM
bartek edited the summary of this revision. (Show Details)Mon, Jun 17, 3:51 AM
bartek added a parent revision: D12448: [reports] Verify CSAT if present.
bartek removed a parent revision: D12416: [reports] Require auth for viewing reports.
bartek updated this revision to Diff 41433.Tue, Jun 18, 12:40 AM

Update docker images

bartek published this revision for review.Tue, Jun 18, 12:41 AM
Harbormaster completed remote builds in B29738: Diff 41433.Tue, Jun 18, 12:57 AM
kamil accepted this revision.Tue, Jun 18, 2:06 AM
This revision is now accepted and ready to land.Tue, Jun 18, 2:06 AM
Closed by commit rCOMM7c14a71b9904: [terraform] Disable CSAT verification on prod (authored by bartek). · Explain WhyTue, Jun 18, 4:59 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMM7c14a71b9904: [terraform] Disable CSAT verification on prod.

Revision Contents
Changeset List

PathSize
services/
terraform/
remote/
10 lines
10 lines
10 lines

Diff 41449

View Options

services/terraform/remote/service_backup.tf

Loading...
View Options

services/terraform/remote/service_blob.tf

Loading...
View Options

services/terraform/remote/service_reports.tf

Loading...