-
Notifications
You must be signed in to change notification settings - Fork 167
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
INTMDB-211: Add new advanced shard key options in global cluster resource #562
Conversation
mongodbatlas/resource_mongodbatlas_global_cluster_config_test.go
Outdated
Show resolved
Hide resolved
addManagedNamespace.IsCustomShardKeyHashed = pointy.Bool(isCustomShardKeyHashed.(bool)) | ||
} | ||
|
||
if isShardKeyUnique, okShard := mn["is_shard_"]; okShard { |
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 is incomplete right? is_shard_
addManagedNamespace.IsCustomShardKeyHashed = pointy.Bool(isCustomShardKeyHashed.(bool)) | ||
} | ||
|
||
if isShardKeyUnique, okShard := mn["is_shard_"]; okShard { |
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.
is this being tested? it seems it is incomplete
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.
yes, It was incomplete
mongodbatlas/resource_mongodbatlas_global_cluster_config_test.go
Outdated
Show resolved
Hide resolved
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.
LGTM!
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.
LGTM
…urce (#562) * Added is_custom_shard_key_hashed and is_shard_key_unique to cluster config * fixed tests * added documentation * fix dep * fixes * go.mod changed * fixed test format * added tests * fix lint
Description
Added
isCustomShardKeyHashed
andisShardKeyUnique
properties to RS and DS for Global_clusterLink to any related issue(s):INTMDB-211
Type of change:
Required Checklist:
Further comments
Tests:
--- PASS: TestAccResourceMongoDBAtlasGlobalCluster_basic (662.26s)
PASS
coverage: 8.3% of statements
ok github.com/mongodb/terraform-provider-mongodbatlas/mongodbatlas 664.729s coverage: 8.3% of statements
? github.com/mongodb/terraform-provider-mongodbatlas/version [no test files]