search

ManageIQ / manageiq

ManageIQ Open-Source Management Platform
https://manageiq.org
Apache License 2.0
1.35k stars 900 forks source link

Fix MiqServer WorkerManager Process with Non Rails workers #23117

Closed agrare closed 3 months ago

agrare commented 4 months ago

We only added logic to set non-rails workers to started/stopped for kubernetes and systemd worker manager subclasses. This meant if you ran rake evm:start locally with a non-rails worker it wouldn't ever be marked as started.

Also updated the systemd side to be consistent with kubernetes in not returning a worker that was just marked as started.

Depends on:

miq-bot commented 3 months ago

Checked commits https://github.com/agrare/manageiq/compare/b20a2d6dd8e3780aac43e239b459d4a3aaee804f~...217068ba0305b9e2ab2f42140bd3c5d84035697a with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.51.0, and yamllint 4 files checked, 0 offenses detected Everything looks fine. :trophy:

Fryguy commented 3 months ago

Backported to radjabov in commit 2075352834988b549ae2697961bbda14bfab458e.

commit 2075352834988b549ae2697961bbda14bfab458e
Author: Joe Rafaniello <jrafanie@users.noreply.github.com>
Date:   Fri Aug 16 15:53:44 2024 -0400

    Merge pull request #23117 from agrare/non_rails_worker_process_starting

    Fix MiqServer WorkerManager Process with Non Rails workers

    (cherry picked from commit cffb072312449f4c922ca6d62755d7ecb479ab99)