-
Notifications
You must be signed in to change notification settings - Fork 669
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
feat(Cloud Database): support offline restore for MongoDB EE PITR #4601
feat(Cloud Database): support offline restore for MongoDB EE PITR #4601
Conversation
…bee-offline-restore
…bee-offline-restore
…bee-offline-restore
…bee-offline-restore
} | ||
|
||
func Region() string { | ||
region, _ := schema.MultiEnvDefaultFunc([]string{"IC_REGION", "IBMCLOUD_REGION", "BM_REGION", "BLUEMIX_REGION"}, "us-south")() |
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 believe this function is replacing IcdDbRegion
. Previously IcdDbRegion = os.Getenv("ICD_DB_REGION")
.
I don't see that env var in this list. Are we good here?
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.
This function replaces IcdDbRegion
and will use the same defaults the provider uses to configure region.
see https://github.com/IBM-Cloud/terraform-provider-ibm/blob/master/ibm/provider/provider.go#L93
offline_restore
flag for MongoDB EE Point-in-Time Restores, used in a disaster recovery scenario.IcdDBRegion
variable with aRegion
method, to reference the same defaults as the provider configurationCommunity Note
Output from acceptance testing: