Closed dwu359 closed 1 year ago
Hello @dwu359! Thank you for submitting the Feature Request Form. We appreciate your contribution. :wave:
We will look into it and provide a response as soon as possible.
To work on this feature request, you can follow these branch setup instructions:
Checkout the main branch:
git checkout nextjs
Pull the latest changes from the remote main branch:
git pull origin nextjs
Create a new branch specific to this feature request using the issue number:
git checkout -b feature-912
Feel free to make the necessary changes in this branch and submit a pull request when you're ready.
Best regards, Deep Learning Playground (DLP) Team
@dwu359 approach for this ticket updated?
Stuff we talked about yesterday for next steps. Let me know if I missed the mark anywhere @karkir0003 @dwu359 .
DLP-CLI COMMAND SCRIPTS FOR THE FOLLOWING
Will probably really get most of the progress done for this Wednesday night and onwards (I have a linear algebra test on Wed). Hoping for a really productive and almost-able-to-merge PR in the next week.
Yes LGTM
Fantastic detailed guide @codingwithsurya !
hey everyone, i have a draft PR up for this issue. Let's move all our communication for this issue there so we can have everything in one place.
@codingwithsurya I left some PR comments in your PR under dlp-cli
Feature Name
Centralize envs with AWS Secrets Manager and AWS Parameter Store
Your Name
Daniel Wu
Description
Step 1: Write a cli cmd that pulls values from parameter store/secrets manager and writes it in .env files in /training and /frontend, perhaps adding/updating an .env.example also.
Step 2: Making sure that when we deploy to prod, we delete that .env file before copying it into docker container. Whenever a container is assigned to ec2 instance, we run the cli cmd to pull the prod configs and add the env file back.