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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Aug 16 2023, 8:50 AM
services/blob/src/s3.rs
110 ↗(On Diff #29965)

Possibly make a helper function for checking this?

services/blob/src/s3.rs
110 ↗(On Diff #29965)

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

services/blob/src/config.rs
7–8 ↗(On Diff #29965)

Can these two constants be cleaned up?

services/blob/src/config.rs
7–8 ↗(On Diff #29965)

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