-
Notifications
You must be signed in to change notification settings - Fork 672
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
Add new service Db2 SaaS to terraform provider #5801
Conversation
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.
Please have a look at these suggested changes. Thanks!
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.
Only one small change. This serves as my approval but we will need approval from the community I believe as well.
} | ||
|
||
return &schema.Resource{ | ||
Read: resourcecontroller.DataSourceIBMResourceInstanceRead, |
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.
Here we are referring to use existing datasource DataSourceIBMResourceInstanceRead to fetch and set all schema attributes but in schema we have new fields like "high_availability","instance_type", "backup_location" how are we handling either they are n't used as input or output attributes
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.
done required changes @hkantare
func ResourceIBMDb2Instance() *schema.Resource { | ||
riSchema := resourcecontroller.ResourceIBMResourceInstance().Schema | ||
|
||
riSchema["high_availability"] = &schema.Schema{ |
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.
Since we don't have any update support do we need set ForceNew : true
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 currently no update support; but these parmas are optional (by default values been taken if we didn't pass value)
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 will be another commits for operations once our API's and SDK done at that time will have support for update.
We can take this up for next beta release. |
Community Note
Output from acceptance testing: