Bug 764765 (GLUSTER-3033) - Changes to replace-brick and syntask interface.
Summary: Changes to replace-brick and syntask interface.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: GLUSTER-3033
Product: GlusterFS
Classification: Community
Component: unclassified
Version: mainline
Hardware: x86_64
OS: Linux
medium
medium
Target Milestone: ---
Assignee: krishnan parthasarathi
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-06-15 13:59 UTC by krishnan parthasarathi
Modified: 2015-12-01 16:45 UTC (History)
4 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)
Add an attachment (proposed patch, testcase, etc.)

Description krishnan parthasarathi 2011-06-15 13:59:56 UTC 
This bug tracks the following changes,
- On completion of replace-brick (abort/commit), cleaning up of 
  xattrs that were used internally to maintain some state.
- Commit command to be sent via setxattr command as in the case of 
  abort.
- Fail commit operation if migration is not yet 'complete'.
- Change synctask interface to be 'universal' than just using it
  internally in pump xlator.
Comment 1 Anand Avati 2011-06-16 13:15:05 UTC 
PATCH: http://patches.gluster.com/patch/7514 in master (syncop: Modified to accept one 'non-frame' arg.)
Comment 2 Anand Avati 2011-06-16 13:15:10 UTC 
PATCH: http://patches.gluster.com/patch/7515 in master (afr: holding stack var via dict_set_static_bin corrupts.)
Comment 3 Anand Avati 2011-06-16 13:15:15 UTC 
PATCH: http://patches.gluster.com/patch/7516 in master (glusterd: do_operation_abort and then kill dst_brick.)
Comment 4 Anand Avati 2011-06-16 13:15:21 UTC 
PATCH: http://patches.gluster.com/patch/7517 in master (pump: cleanup xattrs on both commit and abort path.)
Comment 5 krishnan parthasarathi 2011-06-17 02:43:03 UTC 
Steps to verify:
- Issue a replace brick (start) command from gluster CLI.
- On completion of the data migration (observed using status command), 
  issue a commit or abort command. 
- No pump related extended attributes should be see on '/' of the 'src'
  and 'dst' brick.
Comment 6 Saurabh 2011-07-01 02:47:13 UTC 
please update the target milestone.
Comment 7 Shwetha H Panduranga 2011-09-05 06:30:29 UTC 
This bug is verified as per the steps mentioned by krishnan parthasarathi and works fine.
Note You need to log in before you can comment on or make changes to this bug.