Page MenuHomePhabricator

[keyserver] Fix update-geoip call in Docker environment
ClosedPublic

Authored by ashoat on Mar 14 2023, 3:24 PM.
Tags
None
Referenced Files
F3356526: D7071.diff
Sat, Nov 23, 7:02 PM
Unknown Object (File)
Fri, Nov 22, 12:19 AM
Unknown Object (File)
Fri, Nov 22, 12:19 AM
Unknown Object (File)
Fri, Nov 22, 12:19 AM
Unknown Object (File)
Fri, Nov 22, 12:18 AM
Unknown Object (File)
Mon, Nov 4, 11:33 PM
Unknown Object (File)
Mon, Nov 4, 11:33 PM
Unknown Object (File)
Mon, Nov 4, 11:33 PM
Subscribers

Details

Summary

yarn prod-build in keyserver is supposed to call yarn update-geoip, but this call fails in the Docker environment beacuse that call doesn't have access to the GeoIP license. This diff makes sure we forward that environment variable to the Docker build environment (instead of only being available in the production environment).

Test Plan

cd keyserver && ./bash/dc.sh up --build, with keyserver/.env populated with COMM_JSONCONFIG_secrets_geoip_license env var set. Monitor output and make sure update-geoip runs. It took about 3 minutes on my MBP, probably will take 5-6 minutes in prod

Diff Detail

Repository
rCOMM Comm
Branch
ashoat/geoip
Lint
No Lint Coverage
Unit
No Test Coverage