-
Notifications
You must be signed in to change notification settings - Fork 103
Added support for Host Maintenance Policy features #1961
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
base: proj/maintenance-policy
Are you sure you want to change the base?
Added support for Host Maintenance Policy features #1961
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.
Looks good overall, nice work!
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.
Should we implement maintainacepolicies as the other listing data source and support filtering?
"github.com/hashicorp/terraform-plugin-framework/datasource/schema" | ||
) | ||
|
||
var frameworkDataSourceSchema = 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.
Looks like it's a listing data source, should we implement the schema including filter, order, etc.? Also might need to make some changes to the data source Read to support filtering.
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.
The API Spec did not mention any filtering so I wasn't sure if I should implement it or not.
📝 Description
Added support for Host Maintenance in account settings and instances, and added datasource for maintenance policies.
✔️ How to Test
The following steps assume you have pulled down this PR locally
Unit Tests
make test-unit
Integration Tests
For running the integration tests, you will need to point at DevCloud. Note that the account settings tests will not pass due to limitations with changing account settings for DevCloud accounts.
make test-int PKG_NAME="accountsettings" TEST_CASE="TestAccDataSourceLinodeAccountSettings_basic"
make test-int PKG_NAME="accountsettings" TEST_CASE="TestAccResourceAccountSettings_update"
make test-int PKG_NAME="instance" TEST_CASE="TestAccDataSourceInstances_basic"
make test-int PKG_NAME="instance" TEST_CASE="TestAccResourceInstance_updateMaintenancePolicy"
make test-int PKG_NAME="maintenancepolicies" TEST_CASE="TestAccDataSourceMaintenancePolicies_basic"