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
New interface methods for role list and role argspec #920
Conversation
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 good. Just a bunch of minor things, no blockers.
I verified the tests add 100% coverage to net new code and performed a functional test of the new API endpoints. They work as advertised.
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.
LGTM!
New interface methods for role list and role argspec This adds two new API methods to the runner interface: get_role_list() - Get a list of roles that have defined an argument specification. get_role_argspec() - Get the argument specification for a given role. Reviewed-by: Sam Doran <sdoran@redhat.com> Reviewed-by: David Shrewsbury <None> Reviewed-by: None <None>
I'm interested in this functionality since it is advertised in the documentation but I take it that it has not been released yet. Is there an ETA for that? |
This adds two new API methods to the runner interface: