Project

General

Profile

Actions
Copy link

Bug #30323

closed

Tracker #27511: ensure foreman-maintain works with RHEL8 Satellite & Capsules

assumes SCL paths for databases on EL8

Added by Evgeni Golov almost 4 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
High
Assignee:
Evgeni Golov
Category:
-
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman_maintain/pull/474, https://github.com/theforeman/foreman_maintain/pull/504
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Ohai

  def config_files
    if check_min_version('foreman', '2.0')
      ['/var/opt/rh/rh-postgresql12/lib/pgsql/data/postgresql.conf']
    else
      ['/var/lib/pgsql/data/postgresql.conf']
    end
  end

this is only true on EL7, on EL8 we don't have an SCL'ed PostgreSQL

This breaks DB backups on EL8 currently :(

Related issues 2 (0 open — 2 closed)

Related to Foreman Maintain - Bug #32649: backup-offline-candlepin-db tries to backup wrong postgresql directory and failsDuplicateActions
Has duplicate Foreman Maintain - Bug #32649: backup-offline-candlepin-db tries to backup wrong postgresql directory and failsDuplicateActions
Actions
Copy link
 #2

Updated by James Jeffers about 3 years ago

Evgeni Golov wrote:

reported in https://community.theforeman.org/t/foreman-maintain-backup-fails-on-centos8-foreman-2-1-0rc-multiple-issues/19412/2

Could we use pg_config to discover these paths?

Actions
Copy link
 #3

Updated by Evgeni Golov about 3 years ago

pg_config is part of postgresql-devel, which we don't install by default (nor should we)

and there is no /usr/bin/pg_config in the SCL'ed packages (neither in their syspaths variants), so we'd still need to know whether to load the SCL or not.

:(

Actions
Copy link
 #4

Updated by Evgeni Golov about 3 years ago

  • Parent task set to #27511
Actions
Copy link
 #5

Updated by Evgeni Golov about 3 years ago

more places where it does this:

% git grep rh-pos
definitions/checks/disk/available_space_postgresql12.rb:            file_exists?('/var/opt/rh/rh-postgresql12'))
definitions/checks/disk/available_space_postgresql12.rb:          '/var/opt/rh/rh-postgresql12/'
definitions/features/foreman_database.rb:      ['/var/opt/rh/rh-postgresql12/lib/pgsql/data/postgresql.conf']
lib/foreman_maintain/concerns/base_database.rb:          '/var/opt/rh/rh-postgresql12/lib/pgsql/data/'
Actions
Copy link
 #6

Updated by The Foreman Bot almost 3 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_maintain/pull/474 added
Actions
Copy link
 #7

Updated by Emmanuel Vanlaar almost 3 years ago

  • Related to Bug #32649: backup-offline-candlepin-db tries to backup wrong postgresql directory and fails added
Actions
Copy link
 #8

Updated by Amit Upadhye almost 3 years ago

  • Status changed from Ready For Testing to Closed
Actions
Copy link
 #9

Updated by The Foreman Bot almost 3 years ago

  • Assignee set to Evgeni Golov
  • Pull request https://github.com/theforeman/foreman_maintain/pull/504 added
Actions
Copy link
 #10

Updated by Evgeni Golov almost 3 years ago

  • Has duplicate Bug #32649: backup-offline-candlepin-db tries to backup wrong postgresql directory and fails added
Actions
Copy link

Also available in: Atom PDF