ManageIQ / manageiq-automation_engine

Automation engine for ManageIQ
Apache License 2.0
11 stars 74 forks source link

Support for AWX service provision #499

Closed nasark closed 2 years ago

nasark commented 2 years ago

Depends on:

@miq-bot assign @agrare @miq-bot add_labels enhancement

nasark commented 2 years ago

Looking for some advice, I did not find it necessary to add the AWX counterparts for the following files in order to provision:

./lib/miq_automation_engine/service_models/miq_ae_service_manageiq-providers-ansible_tower-automation_manager-template_runner.rb
./lib/miq_automation_engine/service_models/miq_ae_service_manageiq-providers-ansible_tower-automation_manager-configuration_workflow.rb
./lib/miq_automation_engine/service_models/miq_ae_service_manageiq-providers-ansible_tower-automation_manager-configuration_script.rb
./lib/miq_automation_engine/service_models/miq_ae_service_manageiq-providers-ansible_tower-automation_manager-workflow_job.rb

Should they be added anyway?

agrare commented 2 years ago

@nasark yes I think we should add those, they aren't for provisioning via the UI they are for customers to call from their own automate scripts as far as I know

miq-bot commented 2 years ago

Checked commit https://github.com/nasark/manageiq-automation_engine/commit/ef68b477122a4f7316daf2887420001bef1acb5b with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint 12 files checked, 0 offenses detected Everything looks fine. :+1: