Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.Sign up
Allow tower inventory plugin to accept integer inventory_id #61338
Before 2.9, the tower inventory plugin would happily accept
It was part of our intention to accept inventory primary key values (which could be integers) or strings (which could be from the Tower feature of named URLs).
It seems that the old config parser would automatically convert the integers to strings (thus the added
We never documented this functionality with Named URLs, so that really muddied the waters.
This will maintain all functionality from Ansible 2.8. We could do additional custom type validation, but it's not high priority for me.
Pushed to address comment that we validate type.
The only case I can easily think of where it's not an integer or string is to make it a list. I copied heavily from the base plugin logic for type checking:
We allow 2 types, so I'm trying to make only the modifications needed for this case.
Confirmed the integer & string case still work.