-
Notifications
You must be signed in to change notification settings - Fork 57
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
Pass only prefixed attributes to filters #102
Pass only prefixed attributes to filters #102
Conversation
Codecov Report
@@ Coverage Diff @@
## main #102 +/- ##
==========================================
+ Coverage 63.75% 71.73% +7.98%
==========================================
Files 30 1 -29
Lines 2163 92 -2071
Branches 417 17 -400
==========================================
- Hits 1379 66 -1313
+ Misses 551 24 -527
+ Partials 233 2 -231 Continue to review full report at Codecov.
|
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.
I think this PR is superior compared to #103 because it preserves the consistency of behavior. And although this is indeed an API break, I would say that the original behavior was flawed, and thus this is a fix that sets things straight. Plus, the Ansible PyPI package did not include the "broken" version of the plugin yet, so not many users were exposed to it.
But this PR does need a documentation fragment that describes the change, this is why I am requesting changes in my review.
In contrast to the jinja expressions in `compose`, `groups` and `keyed_groups`, `filters` was given all droplet attributes without a prefix.
4e0f91d
to
6c000d2
Compare
@tadeboro Thanks for the feedback! I added the missing changelog fragment. |
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.
Thank you very much for this, want to close #103?
Closed #103. Thank you! |
SUMMARY
This PR is mutually exclusive with #103 .
In contrast to the jinja expressions in
compose
,groups
andkeyed_groups
,filters
was given all droplet attributes without a prefix. My bad for not catching that.Note that this will break anyone who started using this feature since the release of version 1.5.0. If that is a no-go, I can amend the PR to keep the previous inconsistent behavior as well.
ISSUE TYPE
COMPONENT NAME
Inventory plugin
ADDITIONAL INFORMATION
Before:
or even without the attributes:
After: