Bug 1383795 - repmgrd service fails to start
Summary: repmgrd service fails to start
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Appliance
Version: 5.7.0
Hardware: Unspecified
OS: Unspecified
urgent
high
Target Milestone: GA
: 5.8.0
Assignee: Nick Carboni
QA Contact: luke couzens
URL:
Whiteboard: black:replication
Depends On:
Blocks: 1386352
TreeView+ depends on / blocked
 
Reported: 2016-10-11 18:34 UTC by luke couzens
Modified: 2017-06-12 16:16 UTC (History)
6 users (show)

Fixed In Version: 5.8.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1386352 (view as bug list)
Environment:
Last Closed: 2017-06-12 16:16:09 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:


Attachments (Terms of Use)

Description luke couzens 2016-10-11 18:34:36 UTC
Description of problem:repmgrd service fails to start


Version-Release number of selected component (if applicable):5.7.0.4


How reproducible:100%


Steps to Reproduce:
1.setup 2 db's in failover setup
2.setup main application server watching failover service
3.shutdown first db

Actual results:failover to second appliance fails due to repmgrd service not running


Expected results:failover to secondary db successful


Additional info:
quick setup guide on HA
https://mojo.redhat.com/docs/DOC-1097888

Comment 2 Nick Carboni 2016-10-11 18:40:44 UTC
This is because the package got renamed between upstream and downstream. The appliance console code is assuming the repmgrd systemd unit is called rh-repmgr95 when it is really called rh-postgresql95-repmgr due to the SCL naming convention.

This will require the upstream package to be rebuilt with the new name and the appliance console code to change.

Comment 3 Nick Carboni 2016-10-14 13:22:27 UTC
This issue required PRs in three repos to fix properly:

https://github.com/ManageIQ/manageiq-appliance-build/pull/175
  - This picked up the new package with the proper name (https://copr.fedorainfracloud.org/coprs/manageiq/ManageIQ/package/rh-postgresql95-repmgr/)

https://github.com/ManageIQ/manageiq/pull/11885
  - This PR updated the appliance console to use the new service name when starting and stopping repmgrd

https://github.com/ManageIQ/manageiq-appliance/pull/94
  - This PR renamed the systemd "drop-in" file that we use for specifying the config file location and, now, the new PID file location.

Comment 4 Nick Carboni 2016-10-14 16:41:44 UTC
Also, this PR for the container build https://github.com/ManageIQ/manageiq/pull/11951

Comment 5 CFME Bot 2016-10-17 16:22:20 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/04b77d7f4c7942c5735c15f34d364af12973fbfd

commit 04b77d7f4c7942c5735c15f34d364af12973fbfd
Author:     Jason Frey <fryguy9>
AuthorDate: Fri Oct 14 12:02:14 2016 -0400
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Fri Oct 14 12:13:51 2016 -0400

    Merge pull request #11885 from carbonin/change_repmgr_service_name
    
    Update the repmgr service name to match the SCL naming convention
    (cherry picked from commit d500b282d424a60609db6f7c9717475becbb8e33)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1383795

 gems/pending/appliance_console/database_replication_standby.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 6 CFME Bot 2016-10-17 16:22:45 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/fc93fb6e8606a3830f4a5d8b1eebcf14e675131c

commit fc93fb6e8606a3830f4a5d8b1eebcf14e675131c
Author:     Nick Carboni <ncarboni>
AuthorDate: Wed Oct 12 16:19:11 2016 -0400
Commit:     Nick Carboni <ncarboni>
CommitDate: Wed Oct 12 16:19:11 2016 -0400

    Update the repmgr service name to match the SCL naming convention
    
    Requires https://github.com/ManageIQ/manageiq-appliance-build/pull/175
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1383795

 gems/pending/appliance_console/database_replication_standby.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 7 CFME Bot 2016-10-17 16:23:12 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/f7336bc8290778afc24f926b9765ea87b573a63d

commit f7336bc8290778afc24f926b9765ea87b573a63d
Author:     Nick Carboni <ncarboni>
AuthorDate: Fri Oct 14 12:37:37 2016 -0400
Commit:     Nick Carboni <ncarboni>
CommitDate: Fri Oct 14 12:37:37 2016 -0400

    Rename the repmgr package to conform with SCL naming standards
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1383795

 Dockerfile | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comment 8 CFME Bot 2016-10-17 16:29:44 UTC
New commit detected on ManageIQ/manageiq-appliance/master:
https://github.com/ManageIQ/manageiq-appliance/commit/a85592ad37814514693ac399e7924453074b3a21

commit a85592ad37814514693ac399e7924453074b3a21
Author:     Nick Carboni <ncarboni>
AuthorDate: Wed Oct 12 16:23:44 2016 -0400
Commit:     Nick Carboni <ncarboni>
CommitDate: Wed Oct 12 16:23:44 2016 -0400

    Change the repmgr service name
    
    This will allow the drop-in file to be picked up by systemd
    
    Requires https://github.com/ManageIQ/manageiq-appliance-build/pull/175
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1383795

 .../systemd/system/rh-postgresql95-repmgr.service.d/config_file.conf    | 2 ++
 COPY/etc/systemd/system/rh-repmgr95.service.d/config_file.conf          | 2 --
 2 files changed, 2 insertions(+), 2 deletions(-)
 create mode 100644 COPY/etc/systemd/system/rh-postgresql95-repmgr.service.d/config_file.conf
 delete mode 100644 COPY/etc/systemd/system/rh-repmgr95.service.d/config_file.conf

Comment 9 CFME Bot 2016-10-17 16:29:52 UTC
New commit detected on ManageIQ/manageiq-appliance-build/euwe:
https://github.com/ManageIQ/manageiq-appliance-build/commit/651118f79e6d2a57c6828d0b6e996ea6e391ee2b

commit 651118f79e6d2a57c6828d0b6e996ea6e391ee2b
Author:     Jason Frey <fryguy9>
AuthorDate: Fri Oct 14 12:00:39 2016 -0400
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Fri Oct 14 12:11:52 2016 -0400

    Merge pull request #175 from carbonin/change_the_repmgr_package_name
    
    Change the name of the repmgr package
    (cherry picked from commit 47a84313a7aea2320102b56f79a4996d2f5d4397)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1383795

 kickstarts/partials/packages/includes.ks.erb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 11 luke couzens 2017-03-06 12:54:09 UTC
Verified in 5.8.0.3


Note You need to log in before you can comment on or make changes to this bug.