diff --git a/hooks/boot/01-kafo-hook-extensions.rb b/hooks/boot/01-kafo-hook-extensions.rb index 21c596fe..a8c11d90 100644 --- a/hooks/boot/01-kafo-hook-extensions.rb +++ b/hooks/boot/01-kafo-hook-extensions.rb @@ -164,6 +164,11 @@ def execute_as!(user, command, do_say = true, do_log = true, extra_env = {}) execute!(runuser_command, do_say, do_log, extra_env) end + def execute_as_formatted!(user, command, do_say = true, do_log = true, extra_env = {}) + runuser_command = ['runuser', '-l', user, '-c', command] + execute_command(runuser_command, do_say, do_log, extra_env) + end + def execute(command, do_say, do_log, extra_env = {}) _stdout_stderr, status = execute_command(command, do_say, do_log, extra_env) status diff --git a/hooks/pre/35-change-evr-extension-ownership.rb b/hooks/pre/35-change-evr-extension-ownership.rb new file mode 100644 index 00000000..fdcf8d64 --- /dev/null +++ b/hooks/pre/35-change-evr-extension-ownership.rb @@ -0,0 +1,21 @@ +# In Katello 4.16, the 'evr' extension is removed from PostgreSQL and integrated into the Katello database via a migration. +# This hook ensures the 'evr' extension's ownership is transferred to the 'foreman' user so migrations can act on it. + +if (local_foreman_db? || devel_scenario?) && execute("rpm -q postgresql-evr", false, false) + if app_value(:noop) + logger.debug("Would start postgresql service") + else + start_services(['postgresql']) + end + + # Update the ownership of the evr extension + database = param_value('foreman', 'db_database') || 'foreman' + username = param_value('foreman', 'db_username') || 'foreman' + sql = "UPDATE pg_extension SET extowner = (SELECT oid FROM pg_authid WHERE rolname='#{username}') WHERE extname='evr'" + if app_value(:noop) + logger.debug("Would execute: #{sql} to update ownership of the evr extension") + else + logger.debug("Updating ownership of the evr extension") + execute_as_formatted!('postgres', "psql -d #{database} -c \"#{sql}\"", false, true) + end +end