-
Notifications
You must be signed in to change notification settings - Fork 140
fix(postgresql): [117252443] support params db_major_version
create resource
#2618
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
Conversation
db_major_version
create resourcedb_major_version
create resource
Optional: true, | ||
Default: "10.4", |
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.
这里去掉 default 会有不兼容变更问题吗
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.
如果用户engine_version和db_major_version都不填写 会在代码中手动设置engine_version:10.4
@@ -438,8 +442,8 @@ func resourceTencentCloudPostgresqlInstanceCreate(d *schema.ResourceData, meta i | |||
return outErr | |||
} | |||
|
|||
if specVersion == "" { | |||
return fmt.Errorf(`The "engine_version" value: "%s" is invalid, Valid values are one of: "%s"`, dbVersion, strings.Join(allowVersion, `", "`)) | |||
if majorVersion == "" && specVersion == "" { |
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.
majorVersion 和 specVersion 参数必须传一个是吗
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.
必须传一个或者都传入
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.
majorVersion和specVersion参数主要对应的是客户传入的engine_version和db_major_version
db_major_version
create resourcedb_major_version
create resource
db_major_version
create resourcedb_major_version
create resource
db_major_version
create resourcedb_major_version
create resource
db_major_version
create resourcedb_major_version
create resource
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
… resource (#2618) * add * add * add * add * add * add * add * add * add
add