Bug 1762220

Summary: [geo-rep] sync_method showing rsync instead of tarssh post in-service upgrade
Product: [Community] GlusterFS Reporter: Kotresh HR <khiremat>
Component: geo-replicationAssignee: bugs <bugs>
Status: CLOSED NEXTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: mainlineCC: bugs
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1763028 1764028 (view as bug list) Environment:
Last Closed: 2019-10-17 04:04:41 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1763028, 1764028    

Comment 1 Worker Ant 2019-10-16 09:04:32 UTC
REVIEW: https://review.gluster.org/23555 (geo-rep: Fix config upgrade on non-participating node) posted (#1) for review on master by Kotresh HR

Comment 2 Kotresh HR 2019-10-16 11:20:58 UTC
Description of problem:
While performing an in-service upgrade from 5.x to 6.0 on a non-root geo-rep setup with one volume of each type and use_tarssh set to enabled. It was observed that for a 1x3 replica volume the sync_method was showing rsync instead of tarssh. 

Before upgrade:

[root@dhcp43-49 ~]# gluster v geo-rep masterarb geoaccount.43.153::slavearb config use_tarssh; gluster v geo-rep mastertdisp geoaccount.43.153::slavetdisp config use_tarssh;gluster v geo-rep mastertdistdisp geoaccount.43.153::slavetdistdisp config use_tarssh; gluster v geo-rep masterrep geoaccount.43.153::slaverep config use_tarssh;gluster v geo-rep masterdistrep geoaccount.43.153::slavedistrep config use_tarssh
true
true
true
true
true

After upgrade:

[root@dhcp43-49 ~]# gluster v geo-rep masterarb geoaccount.43.153::slavearb config sync_method; gluster v geo-rep mastertdisp geoaccount.43.153::slavetdisp config sync_method;gluster v geo-rep mastertdistdisp geoaccount.43.153::slavetdistdisp config sync_method; gluster v geo-rep masterrep geoaccount.43.153::slaverep config sync_method;gluster v geo-rep masterdistrep geoaccount.43.153::slavedistrep config sync_method
tarssh
tarssh
tarssh
rsync
tarssh

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

How reproducible:
1/1

Steps to Reproduce:
1.Created a non-root geo-rep setup.
2.Set use_tarssh to true.
3.Performed an in-service upgrade to 3.5.0.
4.Checked the sync method.

Actual results:
sync_method was showing rsync.

Expected results:
sync_method should show tarssh.

Comment 3 Worker Ant 2019-10-17 04:04:41 UTC
REVIEW: https://review.gluster.org/23555 (geo-rep: Fix config upgrade on non-participating node) merged (#5) on master by Amar Tumballi