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

Start all automation runners in AutomationWorker #23151

Merged

Conversation

agrare
Copy link
Member

@agrare agrare commented Aug 15, 2024

Add a way for plugins to register automation_runners, currently Workflows is the only one but embedded ansible / terraform could have dedicated runners in the future

Required for ManageIQ/manageiq-providers-workflows#102

@agrare agrare force-pushed the start_automation_runners_in_automation_worker branch from 0f1e608 to 59bcade Compare August 15, 2024 16:12
@miq-bot
Copy link
Member

miq-bot commented Aug 15, 2024

Checked commit agrare@59bcade with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.51.0, and yamllint
2 files checked, 0 offenses detected
Everything looks fine. 🍪

@agrare
Copy link
Member Author

agrare commented Aug 15, 2024

Test failures look unrelated, #23152

@Fryguy Fryguy merged commit 9343b8b into ManageIQ:master Aug 15, 2024
12 of 14 checks passed
@Fryguy Fryguy self-assigned this Aug 15, 2024
@agrare agrare deleted the start_automation_runners_in_automation_worker branch August 15, 2024 19:11
@Fryguy
Copy link
Member

Fryguy commented Aug 22, 2024

Backported to radjabov in commit 81aac39.

commit 81aac3984d22ed4776a43a55f11b613cba2c996b
Author: Jason Frey <fryguy9@gmail.com>
Date:   Thu Aug 15 15:10:00 2024 -0400

    Merge pull request #23151 from agrare/start_automation_runners_in_automation_worker
    
    Start all automation runners in AutomationWorker
    
    (cherry picked from commit 9343b8bab7925f8c806e7b30a62c58c94d405ce8)

Fryguy added a commit that referenced this pull request Aug 22, 2024
…omation_worker

Start all automation runners in AutomationWorker

(cherry picked from commit 9343b8b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants