Page MenuHomePhabricator

[blob-service] Standardize localstack config
ClosedPublic

Authored by bartek on Aug 16 2023, 8:12 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 18, 2:40 PM
Unknown Object (File)
Sun, Dec 15, 1:56 AM
Unknown Object (File)
Sun, Dec 15, 1:56 AM
Unknown Object (File)
Sun, Dec 15, 1:56 AM
Unknown Object (File)
Sun, Dec 15, 1:54 AM
Unknown Object (File)
Sun, Dec 15, 1:29 AM
Unknown Object (File)
Sun, Dec 1, 2:59 PM
Unknown Object (File)
Fri, Nov 29, 9:02 AM
Subscribers

Details

Summary

We should have one standard way of configuring localstack endpoint for all services. This diff standardizes the localstack config for blob-service.
I consider Tunnelbroker as state-of-the-art solution for localstack endpoint configuration.

TL;DR

cargo run -- --sandbox --localstack-url "http://localhost:4566"

-->

cargo run

The LOCALSTACK_ENDPOINT env variable is automatically provided by Nix so no need to specify --localstack-endpoint explicitly.

Test Plan

Ran the above command and it worked. Blob service integration tests are working.

Diff Detail

Repository
rCOMM Comm
Branch
barthap/blob-config
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

This revision is now accepted and ready to land.Aug 16 2023, 8:50 AM
services/blob/src/s3.rs
110

Possibly make a helper function for checking this?

services/blob/src/s3.rs
110

I don't think that's really useful just for two places

services/blob/src/config.rs
7–8

Can these two constants be cleaned up?

services/blob/src/config.rs
7–8

lol their removal didn't commit, good catch, I was about to land this