Skip to content
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

Workflow Prompt on Launch Causes Inventory to be required #8933

Closed
tstahnke2 opened this issue Dec 16, 2020 · 3 comments
Closed

Workflow Prompt on Launch Causes Inventory to be required #8933

tstahnke2 opened this issue Dec 16, 2020 · 3 comments

Comments

@tstahnke2
Copy link

ISSUE TYPE
  • Bug Report
SUMMARY

We have workflows that we've been using in previous AWX builds (15.0.1 and earlier) that use Prompt on Launch for variables. As of AWX 16, these prompted launches are failing with unresolved errors. These errors go away if we add an inventory to the workflow.

fyi, recreated it so awxbot properly tags

ENVIRONMENT
  • AWX version: 16.0.0
  • AWX install method: docker on linux
  • Ansible version: 2.9.15
  • Operating System: Windows 10 (client)
  • Web Browser: Brave, Chrome, Edge, etc.
STEPS TO REPRODUCE
  1. Set up a workflow that prompts on launch for variables, but has no inventory defined.
  2. Launch the workflow
  3. Enter custom variables and click Next
  4. Non-existent validation errors will be flagged. Will be unable to finalize launching the workflow
EXPECTED RESULTS

I would expect to be able to launch a workflow with Prompt on Launch for variables without having to define an inventory for the workflow.

ACTUAL RESULTS

I am unable to launch workflows with prompt on launch enabled without an inventory selected

ADDITIONAL INFORMATION

Not working without an inventory:
image
image

Showing it working with an Inventory added to the workflow:
image

@jmhbnz
Copy link

jmhbnz commented Dec 21, 2020

Running into the same issue. These workflows with prompts for vars ran fine in 15.01.

Edit: Looks like this error includes workflows that prompt for a limit as well.

@mabashian
Copy link
Member

This will be fixed by #9028

@unlikelyzero
Copy link

covered in #9028 (review)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants