Page MenuHomePhabricator
Differential D4272

[services] Add teardown script
ClosedPublic
Actions

Authored by • karol on Jun 15 2022, 3:09 AM.
Tags
None
Referenced Files
F3492207: D4272.id14022.diff
Wed, Dec 18, 10:29 PM
F3492101: D4272.id13883.diff
Wed, Dec 18, 10:11 PM
Unknown Object (File)
Sat, Dec 7, 7:51 AM
Unknown Object (File)
Sun, Nov 24, 3:42 AM
Unknown Object (File)
Sun, Nov 24, 3:42 AM
Unknown Object (File)
Sun, Nov 24, 3:41 AM
Unknown Object (File)
Sat, Nov 23, 11:39 PM
Unknown Object (File)
Sat, Nov 23, 11:38 PM
View All Files
Subscribers
• abosh
• adrian
ashoat
atul
tomek

Details

Reviewers
tomek
varun
ashoat
Commits
rCOMM21c2899f7001: [services] Add teardown script
Summary

Depends on D4271

Sometimes we may want to quickly delete all the containers and data connected to the local cloud. This may come useful when we do some tests that perform the following:

  • add
  • get
  • remove

and the test fails on get. The remove will never be performed, so next time we run this test, add may fail.

That's just one example, I decided this may be useful in other situations as well.

Test Plan
cd services
yarn run-local-cloud

then do some operations on the local cloud and see the records in the dynamo db or objects on the s3

yarn tear-down-local-cloud
yarn run-local-cloud

see that the objects/records are gone

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• karol created this revision.Jun 15 2022, 3:09 AM
Herald added subscribers: • abosh, atul, • adrian and 2 others. · View Herald TranscriptJun 15 2022, 3:09 AM
Harbormaster completed remote builds in B9763: Diff 13515.Jun 15 2022, 3:23 AM
• karol requested review of this revision.Jun 15 2022, 3:23 AM
• karol edited the summary of this revision. (Show Details)Jun 15 2022, 3:27 AM
• karol edited the test plan for this revision. (Show Details)
• karol added reviewers: tomek, varun, ashoat.
• karol added a parent revision: D4271: [services] Blob - use terminated clb instead of done clb.
• karol added a child revision: D4273: [services] Tests - Blob - Add blob utils.
ashoat added inline comments.Jun 15 2022, 4:37 PM
services/package.json
22 ↗(On Diff #13515)

Since this deletes volumes, I wonder if we should have stronger language... eg. delete-local-cloud or something

ashoat requested changes to this revision.Jun 15 2022, 5:55 PM

Just a rename request, feel free to push back

This revision now requires changes to proceed.Jun 15 2022, 5:55 PM
• karol added inline comments.Jun 20 2022, 4:17 AM
services/package.json
22 ↗(On Diff #13515)

Doesn't matter to me

• karol updated this revision to Diff 13607.EditedJun 20 2022, 4:18 AM
• karol edited the summary of this revision. (Show Details)

rename

Harbormaster completed remote builds in B9835: Diff 13607.Jun 20 2022, 4:26 AM
tomek accepted this revision.Jun 20 2022, 5:41 AM
ashoat accepted this revision.Jun 20 2022, 10:49 AM
This revision is now accepted and ready to land.Jun 20 2022, 10:49 AM
• karol updated this revision to Diff 13883.Jun 28 2022, 4:58 AM

fix invoking line

Harbormaster failed remote builds in B10043: Diff 13883!Jun 28 2022, 4:58 AM
• karol updated this revision to Diff 14022.Jun 30 2022, 3:01 AM

retrigger CI

Harbormaster failed remote builds in B10152: Diff 14022!Jun 30 2022, 8:16 AM
Harbormaster completed remote builds in B10152: Diff 14022.Jun 30 2022, 8:49 AM
• karol updated this revision to Diff 14322.Jul 8 2022, 2:26 AM

rebase

This revision was landed with ongoing or failed builds.Jul 8 2022, 2:30 AM
Closed by commit rCOMM21c2899f7001: [services] Add teardown script (authored by • karol). · Explain Why
This revision was automatically updated to reflect the committed changes.
• karol added a commit: rCOMM21c2899f7001: [services] Add teardown script.
Harbormaster completed remote builds in B10412: Diff 14322.Jul 8 2022, 2:37 AM

Revision Contents
Changeset List

PathSize
services/
3 lines

Diff 14335

View Options

services/package.json

Loading...