bugGluster - Bugs: bug #25207, an rm of a file should not cause...

 
 

bug #25207: an rm of a file should not cause that file to be replicated with afr self-heal.

Submitter:  swankier <swankier>
Submitted:  Mon 29 Dec 2008 09:44:00 PM UTC
   
 
Category:  GlusterFS Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Improper behaviour
Status:  Wont Fix Privacy:  Public
Assigned to:  None Originator Name:  * swankier
Open/Closed:  Closed Release: 
Operating System:  GNU/Linux Reproducibility:  Every Time
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 23 Jun 2009 06:27:22 PM UTC, comment #5: 

Self-heal happens in lookup(). And the current frame-work does not notify the glusterfs about the intent of doing lookup(), meaning glusterfs would not know during lookup() that the file will deleted. So, self-heal is done.

--
Gowda

Basavanagowda Kanur <gowda>
Group Member
Mon 05 Jan 2009 09:39:47 AM UTC, comment #4: 

Krishna, leave it as is. Once self-heal ensures that the volumes are intact, rm will remove both the copies anyways. It is inefficient, but optimizing it the current framework will be hacky.

Swaniker, We are ditching the current self-healing framework with an active healing tool. We can take care of it then.

Anand Babu Periasamy <ab>
Group administrator
Mon 05 Jan 2009 09:20:36 AM UTC, comment #3: 

this begs the question,

if the file is removed from the brick on the "lookup" afr...

will the file remain on the other bricks indefinitely?

swankier <swankier>
Mon 05 Jan 2009 06:49:49 AM UTC, comment #2: 

I am:

1) delete file from posix system beneath afr on one side
2) run rm on gluster file system

file is then replicated followed by deletion

swankier <swankier>
Wed 31 Dec 2008 06:09:27 AM UTC, comment #1: 

Hi,

How are you trying to self-heal? currently self-heal relies on lookup being sent on the parent directory to properly heal.

proper way to heal would be something like,

# rm -f file
# cd ../
# cd -
# ls

This procedure should not cause the deleted files to reappear.

Raghavendra <raghavendra>
Group Member
Mon 29 Dec 2008 09:44:00 PM UTC, original submission:  

it currently does.

it's silly to replicate the file only to delete it.

swankier <swankier>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by gowda (Posted a comment)
  • -email is unavailable- added by raghavendra (Posted a comment)
  • -email is unavailable- added by swankier (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2009-06-23 gowda StatusNone Wont Fix
        Open/ClosedOpen Closed
    2008-12-31 raghavendra Originator Name swankier

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code