Skip to content
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

use efs instead of ebs to store mongodb data #13

Merged
merged 1 commit into from
Mar 26, 2021
Merged

use efs instead of ebs to store mongodb data #13

merged 1 commit into from
Mar 26, 2021

Conversation

mglotov
Copy link
Contributor

@mglotov mglotov commented Mar 26, 2021

The old version used EBS storage to store MongoDB's data. That solution isn't very good. If AZ with EBS volume will be unavailable we lose our VPN server for some amount of time.

In the new version, I use EFS (Elastic File System) to store MongoDB's data. It helps us to avoid a situation when one AZ is down and we can't connect to the VPN server.

@mglotov mglotov requested review from p1gmale0n and halfb00t March 26, 2021 06:49
@guardrails
Copy link

guardrails bot commented Mar 26, 2021

⚠️ We detected security issues in this pull request:

Hard-Coded Secrets (1)

More info on how to fix Hard-Coded Secrets in Terraform.


👉 Go to the dashboard for detailed results.

📥 Happy? Share your feedback with us.

Copy link

@bridgecrew bridgecrew bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bridgecrew has found 5 infrastructure configuration errors in this PR ⬇️

terraform/modules/aws-ec2-pritunl/main.tf Show resolved Hide resolved
terraform/modules/aws-ec2-pritunl/main.tf Show resolved Hide resolved
terraform/modules/aws-ec2-pritunl/main.tf Show resolved Hide resolved
terraform/modules/aws-ec2-pritunl/backup.tf Show resolved Hide resolved
terraform/modules/aws-ec2-pritunl/backup.tf Show resolved Hide resolved
@mglotov mglotov merged commit 9c94f16 into main Mar 26, 2021
@halfb00t halfb00t deleted the pritunl_efs branch April 6, 2021 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants