Bug 764268 (GLUSTER-2536) - gsync service introspection
Summary: gsync service introspection
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: GLUSTER-2536
Product: GlusterFS
Classification: Community
Component: geo-replication
Version: pre-2.0
Hardware: x86_64
OS: Linux
medium
low
Target Milestone: ---
Assignee: Csaba Henk
QA Contact:
URL:
Whiteboard:
: 764965 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-03-16 18:51 UTC by Csaba Henk
Modified: 2015-12-01 16:45 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:


Attachments (Terms of Use)

Description Csaba Henk 2011-03-16 18:51:56 UTC
Glusterd should be aware of the status of configured georeplication sessions. Cli interface should be extended to let users query the status of georeplication services.

Comment 1 Lakshmipathi G 2011-03-23 09:23:24 UTC
I was gsyncing 200gb file from master to slave - initially it sync'ed 3gb to slave and for almost 1 hr slave shows file size as 3gb and then it showed the size as 45gb - Now for more  than 2 hrs file size stays at 45gb. but master and slave process are still syncing.

It would be better to add status of which file is currently getting synced to slave and (if possible) how much of it already gsynced to slave.
seems like rsync itself doesn't have such option. something like lsof + rsync could be used to get such info.

Comment 2 Csaba Henk 2011-03-23 19:15:02 UTC
(In reply to comment #1)
> I was gsyncing 200gb file from master to slave - initially it sync'ed 3gb to
> slave and for almost 1 hr slave shows file size as 3gb and then it showed the
> size as 45gb - Now for more  than 2 hrs file size stays at 45gb. but master and
> slave process are still syncing.
> 
> It would be better to add status of which file is currently getting synced to
> slave and (if possible) how much of it already gsynced to slave.
> seems like rsync itself doesn't have such option. something like lsof + rsync
> could be used to get such info.

Vijay have already proposed this idea. I suggested to postpone it to a later time; being able to retrieve the very basic info of what synchronizations are set up and are functional is part of the soundness of the service. So we try to get this part right first, then we can come to "bit sausages" (do you use this slang for progress bar ;) ?)

Comment 3 Anand Avati 2011-04-14 05:00:58 UTC
PATCH: http://patches.gluster.com/patch/6866 in master (mgmt/glusterd: Implementation of volume gsync status [master [slave]])

Comment 4 Anand Avati 2011-04-14 07:38:35 UTC
PATCH: http://patches.gluster.com/patch/6874 in master (DHT: Add xlator-option assert_no_child_down)

Comment 5 Csaba Henk 2011-04-14 15:20:25 UTC
Kaushik,

please don't mark it resolved until the required further changes get in.

Comment 6 Vijay Bellur 2011-04-15 15:04:14 UTC
(In reply to comment #5)
What are the further changes for this? Any of that getting in to 3.2.0? If not, we can move the target milestone.

Comment 7 kaushik 2011-04-16 04:21:58 UTC
some re-factoring in code is required, will send a patch out today.

Comment 8 Anand Avati 2011-04-18 01:41:01 UTC
PATCH: http://patches.gluster.com/patch/6929 in master (mgmt/glusterd: unify the geo-replication status rpc messages.)

Comment 9 Anand Avati 2011-04-18 01:41:14 UTC
PATCH: http://patches.gluster.com/patch/6933 in master (cli: UI cleanup for geo-replication command)

Comment 10 Csaba Henk 2011-04-22 09:20:10 UTC
I reopen it to make a placeholder for an enhancement which I feel desirable regarding the status.

Along with sessions, either the sugared slave url the user passed on the command line, or a convenient semi-desugared url (like the one you see in gsyncd logs when it clams "syncing from a to b") should be recorded and be used in output.

Having been exposed to it personally, I find it rather unpleasant to see raw canonicalized urls with domain names forcibly resolved to ip addresses and the overwhelming presence of scheme specificators.

Needed changes in python and/or glusterd code are to be discussed, now just make a note so that not to forget of it.

Comment 11 Csaba Henk 2011-08-05 08:22:55 UTC
*** Bug 3233 has been marked as a duplicate of this bug. ***

Comment 12 Anand Avati 2011-08-23 14:45:00 UTC
CHANGE: http://review.gluster.com/79 (Ie. instead of writing out the fully expanded canonical URL like) merged in master by Vijay Bellur (vijay)


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