-
Notifications
You must be signed in to change notification settings - Fork 190
doc: Examples for advanced_cluster preview v2 #3054
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
# Examples for mongodbatlas_advanced_cluster (Preview for MongoDB Atlas Provider v2) | ||
|
||
This directory contains examples of using the `mongodbatlas_advanced_cluster` resource for the **Preview for MongoDB Atlas Provider v2**. | ||
|
||
In order to enable the Preview, you must set the enviroment variable `MONGODB_ATLAS_ADVANCED_CLUSTER_V2_SCHEMA=true`, otherwise the current version will be used. | ||
|
||
You can find more info in the [resource doc page](https://registry.terraform.io/providers/mongodb/mongodbatlas/latest/docs/resources/advanced_cluster%2520%2528preview%2520v2%2529). |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,7 +15,7 @@ Utilize the following to execute a working example, replacing the org id, public | |
|
||
Apply with the following `terraform.tfvars` to first create a shared tier cluster: | ||
``` | ||
atlas_org_id = "627a9687f7f7f7f774de306f14" | ||
atlas_org_id = <REDACTED> | ||
public_key = <REDACTED> | ||
private_key = <REDACTED> | ||
provider_name = "TENANT" | ||
|
@@ -25,8 +25,8 @@ provider_instance_size_name = "M2" | |
|
||
Apply with the following `terraform.tfvars` to upgrade the shared tier cluster you just created to dedicated tier: | ||
``` | ||
atlas_org_id = "627a9687f7f7f7f774de306f14" | ||
atlas_org_id = <REDACTED> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Don't know why we are using There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. it's in the spirit of they can use their own org_id, I think it's confusing to have a hardcoded value, in this way they don't they have to enter their own org_id. not strong opinion about change it to {ORG_ID} although it looks like it's a variable, in current way I think it's more clear that they need to enter the number of the org_id There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. at the end I've changed it here: #3059 |
||
public_key = <REDACTED> | ||
private_key = <REDACTED> | ||
provider_name = "GCP" | ||
provider_instance_size_name = "M10" | ||
provider_name = "AWS" | ||
provider_instance_size_name = "M10" |
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.
Are we locking in on 1.27? Makes sense to me, if there for some reason is another release before, we can always change it
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.
that was my thought, it won't be available in previous version, if not in 1.27 we can change again