Bug 1340247

Summary: Replication subscription update fails
Product: Red Hat CloudForms Management Engine Reporter: Nick Carboni <ncarboni>
Component: ReplicationAssignee: Nick Carboni <ncarboni>
Status: CLOSED ERRATA QA Contact: Alex Newman <anewman>
Severity: high Docs Contact:
Priority: high    
Version: 5.6.0CC: cpelland, gtanzill, jhardy, mfeifer, obarenbo, simaishi
Target Milestone: GA   
Target Release: 5.6.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.6.0.10 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-06-29 16:06:32 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Nick Carboni 2016-05-26 20:21:22 UTC 
Description of problem:

Updating a subscription fails with the message:
Error during replication configuration save: Failed to save subscription to <hostname>: fe_sendauth: no password supplied

This is being raised by the process that validates the remote schema against the schema.yml file.

The model doesn't have to password field set until we actually attempt the update, which is too late for the schema validity check.

Version-Release number of selected component (if applicable):
5.6.0.8-rc1-nightly

How reproducible:
Always

Steps to Reproduce:
1. Deploy two regions
2. Set up replication (remote region + subscription)
3. Edit the subscription
  - For this I set the hostname by setting an alias in /etc/hosts to the original IP of the regional appliance and using that hostname in the update.

Actual results:
Error is raised and save fails

Expected results:
Subscription info is updated.

Additional info:
Comment 2 CFME Bot 2016-05-26 20:55:46 UTC 
https://github.com/ManageIQ/manageiq/pull/8997
Comment 3 CFME Bot 2016-05-31 19:20:53 UTC 
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/e6ea49cb43f165b0c74b29e6a874a78a0ec5a86d

commit e6ea49cb43f165b0c74b29e6a874a78a0ec5a86d
Author:     Nick Carboni <ncarboni>
AuthorDate: Thu May 26 16:46:27 2016 -0400
Commit:     Nick Carboni <ncarboni>
CommitDate: Thu May 26 16:46:27 2016 -0400

    Get subscription password before attempting to verify remote schema
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1340247

 app/models/pglogical_subscription.rb       |  1 +
 spec/models/pglogical_subscription_spec.rb | 16 ++++++++--------
 2 files changed, 9 insertions(+), 8 deletions(-)
Comment 5 CFME Bot 2016-06-15 12:47:06 UTC 
https://github.com/ManageIQ/manageiq/pull/9133
Comment 7 errata-xmlrpc 2016-06-29 16:06:32 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2016:1348