Foreman » Installer

Fixes #30193 - move version unlock to main pre_validations hooks

hooks/pre_validations/31-puppet-agent-oauth.rb tries to re-install
puppet-agent-oauth, but for that to work the packages need to be
unlocked first

it's not sufficient to move it to katello/hooks/pre_validations as those
are executed after all main pre_validations

Fixes #30193 - execute rpm commands outside the SCL

otherwise Ruby finds the SCL oauth gem and does not re-install the one
for the AIO Puppet Ruby

Fixes #30193 - status is now a boolean

Refs #30193 - always execute commands outside the SCL context

Refs #30193 - Correct version locking

6e1a07bc35bfe97927dfac6477a1faf53092c58f moved version locking from
Katello to the common hooks. This was unintentional and should have been
kept for Katello, at least for now.

However, it did try to fix a real issue where yum was locked while the
hooks tried to make changes. However, during pre_validations it may
still find an issue and roll back so no changes should be made. To solve
this, the hook is moved to pre. This is more correct because at this
point Kafo, has committed all the answers and is going to make changes
to the system using Puppet.