Eric Helms

The previous code was creating a new instance of the UpdateRunner class every time and therefore throwing away the saved exit code.

Contains https://github.com/theforeman/foreman_maintain/pull/837

Before this change a user would just see the following if a service was down: ``` Nothing to update, can't find new version of satellite-maintain. Running preparation steps required to...

This may only be when running from git, but whenever `packages update` is run on Debian it will create two files depending on if `assumeyes` is present: ``` Untracked files:...

Migrated from https://projects.theforeman.org/issues/37182 Submitted based on community 36512: https://community.theforeman.org/t/pulpcore-coredumps-during-stopping-services/36512 A user is doing some sort of "offline backup" operation which shuts down the database. Afterwards, the user calls 'foreman-maintain service...

Migrated from https://projects.theforeman.org/issues/37166 Sometimes this happens during deleting a host: PG::ForeignKeyViolation: ERROR: update or delete on table "hosts" violates foreign key constraint "host_status_hosts_host_id_fk" on table "host_status" DETAIL: Key (id)=(HOST_ID) is...

Migrated from https://projects.theforeman.org/issues/36264

Migrated from https://projects.theforeman.org/issues/34431 ``` [root@centos8-stream-katello-nightly ~]# foreman-maintain service restart Running Restart Services ================================================================================ Check if command is run as root user: [OK] -------------------------------------------------------------------------------- Restart applicable services: Stopping the following service(s):...