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
[Closed] Fix handling of inventory and credential options for tower_job_launch #41497
Conversation
Tests are still running as of now, but after they pass... shipit |
It might be nice to add an integration test for this that proves that it works. You could update this playbook to make another Job Template that specified https://docs.ansible.com/ansible/latest/modules/tower_job_template_module.html#tower-job-template-module |
- launch job template without credential and inventory information Signed-off-by: Hideki Saito <saito@fgrep.org>
@AlanCoding @ryanpetrello Thank you for your review! I have added the integration test for this fix :) |
I don't see where the integration test is actually trying this. |
…ential Signed-off-by: Hideki Saito <saito@fgrep.org>
CI failure in integration tests: https://app.shippable.com/github/ansible/ansible/runs/71456/66/tests |
I see a handful of related issues that around tower_job_launch requiring a credential and inventory to be specified. Is someone still working on this fix? I also noticed only the name for a credential/inventory work. You cannot reference by id. I can submit a doc update if this expected. |
The practice in tower-cli is that we will attempt to turn it into an integer, and if that fails, treat it as the name. The tower modules go their own way on this, doing the related lookups itself, so the implementation could differ module-by-module unfortunately. If you have ideas about the direction the modules should go in for related lookups, let us know. |
@AlanCoding That seems like a reasonable approach. I don't mind taking a look at the tower modules to see if there's a simple fix. Otherwise a doc update seems to be in order. |
Closing this PR and crated new PR #54967 based of current "devel" |
SUMMARY
prompt on launch
option), both of them unnecessary to specify whenprompt on launch
is turned offISSUE TYPE
COMPONENT NAME
ANSIBLE VERSION
ADDITIONAL INFORMATION
None