-
Notifications
You must be signed in to change notification settings - Fork 100
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
Implement linode_database_mysql_v2
data source
#1717
Conversation
linode_database_mysql_v2
data source
85203ce
to
d373f54
Compare
Got a test failure locally:
Looks like you just need to update the test case 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.
LGTM, just a small update required in the assertions. Great job!
Looks like another test case needs to be tweaked:
|
Not sure how I mismatched all of those test assertions, but regardless they should all be fixed now haha |
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.
Everything is passing now. Nice work!
📝 Description
This pull request introduces a new
linode_database_mysql_v2
data source with schema matching the schema of thelinode_database_mysql_v2
resource.Depends on #1701
✔️ How to Test
The following test steps assume you have pulled down this PR locally.
Integration Testing
Manual Testing