Project

General

Profile

Actions
Copy link

Feature #3757

closed

osd: Watch/Notify cleanup

Added by Samuel Just over 11 years ago. Updated about 11 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Samuel Just
Category:
-
Target version:
v0.59
% Done:

0%

Source:
Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

The current design is rather fragile particularly with respect to the locking and ref counting.

The result of this task should be a design which avoids the watch_lock ordering problems, uses smart pointers with clear reference ownership to avoid leaks/corruption, and allows multiple watches on the same object from the same client.

Testing will be accomplished via existing existing watch/notify and rbd teuthology tasks.

Related issues 2 (0 open — 2 closed)

Related to Ceph - Feature #3403: librados: expose a list of watchers on an objectResolvedDavid Zafman10/24/2012

Actions
Has duplicate Ceph - Bug #2533: osd: watchers tracked by entity_name_t, not by cookieDuplicateSamuel Just06/07/2012

Actions
Actions
Copy link
 #1

Updated by Samuel Just over 11 years ago

  • Translation missing: en.field_position set to 15
Actions
Copy link
 #2

Updated by Samuel Just over 11 years ago

  • Translation missing: en.field_position deleted (15)
  • Translation missing: en.field_position set to 17
Actions
Copy link
 #3

Updated by Sage Weil over 11 years ago

  • Assignee set to Samuel Just
Actions
Copy link
 #4

Updated by Sage Weil over 11 years ago

  • Translation missing: en.field_position deleted (23)
  • Translation missing: en.field_position set to 8
Actions
Copy link
 #5

Updated by Sage Weil over 11 years ago

  • Translation missing: en.field_position deleted (7)
  • Translation missing: en.field_position set to 3
Actions
Copy link
 #6

Updated by Sage Weil over 11 years ago

  • Translation missing: en.field_position deleted (3)
  • Translation missing: en.field_position set to 5
Actions
Copy link
 #7

Updated by Samuel Just about 11 years ago

  • Target version set to v0.57c
  • Translation missing: en.field_position deleted (12)
  • Translation missing: en.field_position set to 8
Actions
Copy link
 #8

Updated by Ian Colle about 11 years ago

  • Target version changed from v0.57c to v0.58
  • Translation missing: en.field_position deleted (8)
  • Translation missing: en.field_position set to 2
Actions
Copy link
 #9

Updated by Ian Colle about 11 years ago

  • Translation missing: en.field_story_points set to 13
  • Translation missing: en.field_position deleted (2)
  • Translation missing: en.field_position set to 2
Actions
Copy link
 #10

Updated by Sage Weil about 11 years ago

  • Status changed from New to In Progress
  • Translation missing: en.field_position deleted (2)
  • Translation missing: en.field_position set to 2
Actions
Copy link
 #11

Updated by Sage Weil about 11 years ago

  • Status changed from In Progress to Fix Under Review
Actions
Copy link
 #12

Updated by Sage Weil about 11 years ago

  • Target version changed from v0.58 to v0.59
Actions
Copy link
 #13

Updated by Samuel Just about 11 years ago

  • Status changed from Fix Under Review to Resolved

b531aa3688d9e8831837c23abac0bdaba04ec793

Actions
Copy link

Also available in: Atom PDF