Page MenuHomePhabricator
Differential D3523

[services] Backup/Blob - fix test path
AbandonedPublic
Actions

Authored by • karol on Mar 25 2022, 5:02 AM.
Tags
None
Referenced Files
F3377084: D3523.diff
Wed, Nov 27, 3:39 AM
Unknown Object (File)
Fri, Nov 15, 9:02 AM
Unknown Object (File)
Fri, Nov 8, 10:40 PM
Unknown Object (File)
Fri, Nov 8, 10:40 PM
Unknown Object (File)
Fri, Nov 8, 10:40 PM
Unknown Object (File)
Thu, Nov 7, 1:23 AM
Unknown Object (File)
Wed, Nov 6, 8:17 AM
Unknown Object (File)
Tue, Nov 5, 4:32 PM
View All Files
Subscribers
• adrian
ashoat
atul
• benschac
tomek

Details

Reviewers
tomek
• max
varun
• jim
ashoat
Summary

Depends on D3498

This fixes the test path for the backup service. I changed the path to absolute for the blob service to keep it consistent.

Test Plan
cd services
yarn test-backup-service
yarn test-backup-service-dev-mode
yarn test-blob-service
yarn test-blob-service-dev-mode

Diff Detail

Repository
rCOMM Comm
Branch
arcpatch-D3498
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

• karol created this revision.Mar 25 2022, 5:02 AM
Herald added subscribers: • benschac, atul, • adrian and 2 others. · View Herald TranscriptMar 25 2022, 5:02 AM
• karol edited the summary of this revision. (Show Details)Mar 25 2022, 5:04 AM
• karol edited the test plan for this revision. (Show Details)
• karol added reviewers: tomek, • max, varun, • jim.
• karol added a parent revision: D3498: [services] Dev Mode - Apply local cloud to the Backup service.
Harbormaster completed remote builds in B7599: Diff 10674.Mar 25 2022, 5:10 AM
• karol requested review of this revision.Mar 25 2022, 5:10 AM
ashoat accepted this revision.Mar 25 2022, 12:56 PM

In which diff was this changed?

This revision is now accepted and ready to land.Mar 25 2022, 12:56 PM
• karol mentioned this in D3652: [services] Backup - fix test script.Apr 8 2022, 1:03 AM
• karol updated this revision to Diff 11210.Apr 8 2022, 1:05 AM

rebase

Harbormaster failed remote builds in B8005: Diff 11210!Apr 8 2022, 1:07 AM
• karol abandoned this revision.Apr 11 2022, 6:44 AM

I changed my approach, now I used terraform for this. I'm abandoning this one. I decided that it will be faster to set up a new stack as I reordered the diffs and a lot of the differed from the ones from this stack. Let's follow up in the stack beginning at D3695.

In which diff was this changed?

Sorry, IDK :/

• karol mentioned this in D3695: [services] Dev mode - Fix Backup service.Apr 11 2022, 6:49 AM
In D3523#96086, @ashoat wrote:

In which diff was this changed?

Added a note about this in the new stack.

Revision Contents
Changeset List

PathSize
services/
backup/
docker-server/
contents/
4 lines
blob/
docker-server/
contents/
4 lines

Diff 10674

View Options

services/backup/docker-server/contents/run_tests.sh

Loading...
View Options

services/blob/docker-server/contents/run_tests.sh

Loading...