-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Linode dynamic inventory plugin fails with ip_style=api due to missing ipv6 pools #8213
Comments
Files identified in the description: If these files are incorrect, please update the |
cc @Charliekenney23 @InTheCloudDan @LBGarber @decentral1se @displague @rmcintosh |
It looks like (I really wonder what kind of versioning scheme they use? Breaking changes in a patch version doesn't sound like a good idea to me... Also the PR doesn't mention any kind of motivation for the rename, which doesn't make it any better...) |
I created a PR to fix this, could you please test it? #8248 (I have no way to test this.) |
I get: |
Can you paste the full stack trace? Thanks. |
|
I guess in that case further things have changed in a backwards incompatible way. It's probably best if someone who can actually run the plugin against the real service continues debugging and fixing this. |
Summary
The linode dynamic inventory plugin fails to parse the inventory when ip_style is set to "api", with an error about a missing attribute
instance.ips.ipv6.pools
This appears to be due to line 221:
I don't see
Issue Type
Bug Report
Component Name
linode
Ansible Version
Community.general Version
Configuration
$ ansible-config dump --only-changed
OS / Environment
Ubuntu 23.04
Steps to Reproduce
Expected Results
Would expect the inventory plugin to correctly parse the response from the linode API
Actual Results
Code of Conduct
The text was updated successfully, but these errors were encountered: