-
Notifications
You must be signed in to change notification settings - Fork 666
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
Migrate to the new linode_v4 module #1678
Comments
I've started a discussion on |
FYI, the discussion is underway ... tokens / CI/CD account access is being arranged 🚀 |
|
OK, I'll try to take a stab at the new module. No functional testing for now but that can come later on when the plugin architecture comes into place. Since we're aiming for a 3.0 release, I won't handle backwards support for the new modules. |
Issue Type
Issue Description
Since #1458 is now merged, there are 2 remaining tasks to do.
Since Implement the linode_v4 module. ansible#46151 is merged (to arrive in 2.8), we can drop the deprecated linode module being used here. That can't happen until 2.8 is out but should happen before Deprecate the linode module for the linode_v4 module ansible#51060 happens :) AFAIK, there is already enough functionality in the new module to make this driver work.
Currently, the functional tests for the linode module are skipped. There is a nice recommendation to use the linode_v4 client fixtures (seen in https://github.com/linode/linode_api4-python/tree/master/test/fixtures) to mock the network connection necessary for running the functional tests. Any test server developed for this work would be of general use to other test drivers who want to mock the network as well.This work mostly waits on the release of 2.8.
The text was updated successfully, but these errors were encountered: