Bug 1540688

Summary: Subscription page fails when a remote database is down
Product: Red Hat CloudForms Management Engine Reporter: luke couzens <lcouzens>
Component: ApplianceAssignee: Yuri Rudman <yrudman>
Status: CLOSED CURRENTRELEASE QA Contact: luke couzens <lcouzens>
Severity: high Docs Contact:
Priority: high    
Version: 5.9.0CC: abellott, cpelland, lcouzens, ncarboni, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly
Target Release: 5.10.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: black:replication
Fixed In Version: 5.10.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1543543 1550116 (view as bug list) Environment:
Last Closed: 2019-02-11 14:06:54 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1543543, 1550116    
Attachments:
Description Flags
screenshot of error none

Description luke couzens 2018-01-31 18:44:24 UTC
Created attachment 1389134 [details]
screenshot of error

Description of problem:Can't add second pglogical subscription or remove current ones


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


How reproducible:100%


Steps to Reproduce:
1.configure appliances
2.set replications on appliance 1 to remote region
3.set replication to global on second appliance
4.add subscription to first appliance
5.navigate away from replication in the ui
6.navigate back to configuration-settings-region-replication

Actual results:
Error message appears and cant access subscriptions

Expected results:
page loads correctly and you can add remove subscriptions

Additional info:
see screen shot

Comment 8 CFME Bot 2018-02-02 22:08:41 UTC
New commit detected on ManageIQ/pg-pglogical/master:
https://github.com/ManageIQ/pg-pglogical/commit/fcf2fe4b21ecb2778b80c40e879d4ad3017644b8

commit fcf2fe4b21ecb2778b80c40e879d4ad3017644b8
Author:     Yuri Rudman <yrudman>
AuthorDate: Fri Feb 2 08:37:00 2018 -0500
Commit:     Yuri Rudman <yrudman>
CommitDate: Fri Feb 2 08:37:00 2018 -0500

    show all subscriptions defined on global region, even if no corresponding node exists on remote
    https://bugzilla.redhat.com/show_bug.cgi?id=1540688

 lib/pg/pglogical/client.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 10 CFME Bot 2018-02-05 09:08:51 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/9700419ab6fe87939b884aeb0dfee46f1a8d4123

commit 9700419ab6fe87939b884aeb0dfee46f1a8d4123
Author:     Yuri Rudman <yrudman>
AuthorDate: Fri Feb 2 14:43:59 2018 -0500
Commit:     Yuri Rudman <yrudman>
CommitDate: Fri Feb 2 17:00:16 2018 -0500

    add column 'status' to replication screen
    https://bugzilla.redhat.com/show_bug.cgi?id=1540688

 app/controllers/ops_controller/settings/common.rb | 3 ++-
 app/views/ops/_settings_replication_tab.html.haml | 4 ++++
 2 files changed, 6 insertions(+), 1 deletion(-)

Comment 11 Yuri Rudman 2018-02-05 12:06:00 UTC
First PR: show all subscriptions (good ones and corrupted). It would prevent UI screen from blowing-up if one of remote subscription is not good.
Second PR: added new column 'status' to replication management screen.

Comment 14 CFME Bot 2018-02-07 22:36:49 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/32584e92367709eb4c45f0be5dac26c2aed85e87

commit 32584e92367709eb4c45f0be5dac26c2aed85e87
Author:     Nick Carboni <ncarboni>
AuthorDate: Wed Feb 7 16:18:42 2018 -0500
Commit:     Nick Carboni <ncarboni>
CommitDate: Wed Feb 7 16:18:42 2018 -0500

    Update pg-pglogical gem to 2.1.2
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1540528
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1540688

 Gemfile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 18 luke couzens 2018-06-20 08:41:12 UTC
Verified in 5.10