Paths

Table of Contentst

-
services/terraform/self-host/
-
terraform/
-
self-host/
-
aws-deploy.sh

Differential D13093

[terraform] fix indents
ClosedPublic
Actions

Authored by will on Aug 15 2024, 8:05 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Apr 19, 8:20 AM

	Unknown Object (File)
	Thu, Apr 17, 1:15 AM

	Unknown Object (File)
	Wed, Apr 16, 4:25 PM

	Unknown Object (File)
	Wed, Apr 16, 3:34 PM

	Unknown Object (File)
	Wed, Apr 16, 6:39 AM

	Unknown Object (File)
	Wed, Apr 16, 2:52 AM

	Unknown Object (File)
	Tue, Apr 15, 1:18 AM

	Unknown Object (File)
	Sun, Mar 30, 9:17 PM

View All 89 Files

Subscribers

Details

Reviewers

varun
bartek

Commits

rCOMM73f6a4f35181: [terraform] fix indents

Summary

This fixes the indents in the aws-deploy script

Test Plan

ran the script. Changing the indents did not affect script behavior

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

will created this revision.Aug 15 2024, 8:05 PM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 15 2024, 8:05 PM

rebase

will added a child revision: D13094: [terraform] increase max time for curl.Aug 15 2024, 8:09 PM

Harbormaster completed remote builds in B31117: Diff 43414.Aug 15 2024, 8:23 PM

will requested review of this revision.Aug 15 2024, 8:23 PM

Harbormaster completed remote builds in B31116: Diff 43413.Aug 15 2024, 8:26 PM

varun accepted this revision.Aug 15 2024, 10:46 PM

This revision is now accepted and ready to land.Aug 15 2024, 10:46 PM

Closed by commit rCOMM73f6a4f35181: [terraform] fix indents (authored by will). · Explain WhyAug 16 2024, 12:05 PM

This revision was automatically updated to reflect the committed changes.

will added a commit: rCOMM73f6a4f35181: [terraform] fix indents.

Revision Contents
Changeset List

Path

Size

services/

terraform/

self-host/

38 lines

Diff 43414

services/terraform/self-host/aws-deploy.sh

This file was added.

Property	Old Value	New Value
File Mode	null	100755

				#!/bin/bash

				set -e

				docker-compose up -d --force-recreate localstack
				tomekUnsubmitted Not Done Inline Actions What is the result of including `--force-recreate`? Does that mean that the data will be gone after restarting a container? tomek: What is the result of including `--force-recreate`? Does that mean that the data will be gone…
				karolAuthorUnsubmitted Done Inline Actions Yes, I think we can remove this or make it optional at least. I'm not recalling right now what exactly wasn't working properly without recreating the container. We can always add it later. karol: Yes, I think we can remove this or make it optional at least. I'm not recalling right now what…

				pushd terraform

				./run.sh

				popd # terraform