-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FIX: Update Deployment documentation #622
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
Internally we will need a list of permissions a person need to update both envs.
|
||
## Environments: | ||
- Testnet (Production) | ||
- Dev (Development) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we still have partner and testnet folders?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because partner is a codebase for our partners to use this code to deploy their environment. Testnet is our personal environment.
1. Make sure you have [terraform installed](https://developer.hashicorp.com/terraform/tutorials/aws-get-started/install-cli) on your local machine | ||
2. Navigate to the `infra` directory, and then the `multichain-dev` directory | ||
3. Verify the variables in both `variables.tf` and `terraform-dev.tfvars` are up to date | ||
4. Verify the environment variables (`main.tf lines 17-61`, `variables.tf lines 87-150`) for the container are as desired |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would not mention lines here, we will forget to update it. I think it's better to mention specific fields you need to update.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Feel free to add those, I am not sure what fields get updated frequently.
- partner | ||
- internal | ||
description: Do you want to deploy the Parner nodes or internal Pagoda nodes? | ||
default: us-east1-docker.pkg.dev/pagoda-discovery-platform-prod/multichain-public/multichain-testnet |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we also need to build it from the main branch instead of develop branch.Can we add that as well? Thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can select which branch to build from in the workflow dispatch run, it defaults to main branch.
Terraform Feature Environment Destroy (dev-622)Terraform Initialization ⚙️
|
Closes #599
Please let me know if there is anything else that needs to be added to this doc.