passportxyz / passport

Passport allows users to prove their identity through a secure, decentralized UI
Other
975 stars 455 forks source link

Infra Week: Create Utility Script for Exporting ENV Variables and Update Pulumi Scripts #2628

Open erichfi opened 4 months ago

erichfi commented 4 months ago

User Story: As a developer, I want to create a utility script that pulls and exports all relevant ENV variables for any environment to local ENVs and update Pulumi scripts to read these variables, so that I can run pulumi up from my local environment for review, staging, and production using the correct variables.

!! Note from Lucian !! : I'll update the pulumi scripts as part of https://github.com/gitcoinco/passport/issues/2627, so this ticket can just handle the script for local development. Or really, setting up a .env.local with the right 1password paths and then altering the package.json commands to use the 1password utility, and making sure 1password has good default entries for local dev. See https://developer.1password.com/docs/cli/secrets-environment-variables/

Acceptance Criteria: GIVEN the utility script and updated Pulumi scripts, WHEN I run the script, THEN all relevant environment variables should be pulled and exported to my local environment, and the Pulumi scripts should use these variables.

Tech Details:

Open Questions:

Notes/Assumptions:

lucianHymer commented 4 months ago

We probably want to use the methods described here? https://developer.1password.com/docs/cli/secrets-environment-variables

nutrina commented 4 months ago

The repos affected

Exceptions to be handled

lucianHymer commented 3 months ago

Note: I'll update the pulumi scripts as part of #2627, so this ticket can just handle the script for local development

tim-schultz commented 3 months ago

There are POCs for loading the env variables while running pulumi up locally in the linked PRs. Moving to blocked until 2627 is finished up since there is a decent amount of overlap

tim-schultz commented 3 months ago

This is no longer needed