-
Notifications
You must be signed in to change notification settings - Fork 23
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
Implemented changes for Linode Disk Encryption #550
Implemented changes for Linode Disk Encryption #550
Conversation
93c829b
to
8a3a1d0
Compare
8a3a1d0
to
08d2d4e
Compare
64cd869
to
855a6ed
Compare
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.
Probably need to change a bit the SpecFields based on the API spec doc:
I can't share the spec doc on github for security reason. Let's take the discussion offline! |
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 great! Tests all passed locally 🎉
bbf73d8
to
0cfc7d6
Compare
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.
Tests are passing locally and I haven't been able to break it in my manual testing. Excellent work!
This reverts commit 19fb287.
📝 Description
Updated ansible modules to consume changes according to the API Spec (can be found in the ticket).
✔️ How to Test
Integration Tests
make TEST_ARGS="-v instance_disk_encryption" test
make TEST_ARGS="-v lke_cluster_basic" test
make TEST_ARGS="-v lke_node_pool_basic" test