Source Vars are marked as required on Constructed Inventory UI

Jaapis commented 1 year ago

Please confirm the following

[X] I agree to follow this project's code of conduct.
[X] I have checked the current issues for duplicates.
[X] I understand that AWX is open source software provided for free and that I might not receive a timely response.
[X] I am NOT reporting a (potential) security vulnerability. (These should be emailed to security@ansible.com instead.)

Bug Summary

On the Constructed Inventories UI there is an asterisk on the source_vars implying that it is required to build the inventory even though this is not correct.

This was not tested on the Tech Preview UI as the Constructed Inventory page is not available at the time.

AWX version

4.4

Select the relevant components

[X] UI
[ ] UI (tech preview)
[ ] API
[ ] Docs
[ ] Collection
[ ] CLI
[ ] Other

Installation method

docker development environment

Modifications

no

Ansible version

2.4

Operating system

fedora

Web browser

Firefox, Chrome

Steps to reproduce

Go to Inventories>Add>Add constructed inventory On the bottom of the page there's the Source vars field

Expected results

Source vars is not required to create the constructed inventory so the asterisk shouldn't be there

Actual results

The asterisk is there, implying that the source var is required

Additional information

Bug found during testathon Jira Issue: https://issues.redhat.com/browse/AAP-12581

shanemcd commented 1 year ago

Is this actually an API issue?

Jaapis commented 1 year ago

@kialam showed me that this is actually an issue related to the UI requiring source_vars and not the other way around. Closing this issue.

shanemcd commented 1 year ago

If another issue exists it would be good to link it here for the sake of having a paper trail.

Jaapis commented 1 year ago

There wasn´t any other issue for that so I reopened and edited to be suitable to the related issue.

ansible / awx