[lvm-devel] master - lvmdbusd: Remove TODO on concurrent access to properties

[tasleson]

Gitweb:        http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=1d520909534e1a4e1adf2513a5061adfe631d6af
Commit:        1d520909534e1a4e1adf2513a5061adfe631d6af
Parent:        37f05ccab1a576bdd71be1f4f80ab6f621042d71
Author:        Tony Asleson <tasleson at redhat.com>
AuthorDate:    Wed Nov 30 13:03:25 2016 -0600
Committer:     Tony Asleson <tasleson at redhat.com>
CommitterDate: Wed Nov 30 15:59:06 2016 -0600

lvmdbusd: Remove TODO on concurrent access to properties

As the code now uses a single thread to handle all changes to the
dbus model we no longer need to handle this potential race
condition.
---
 daemons/lvmdbusd/automatedproperties.py |    5 +----
 1 files changed, 1 insertions(+), 4 deletions(-)

diff --git a/daemons/lvmdbusd/automatedproperties.py b/daemons/lvmdbusd/automatedproperties.py
index 5e14d61..7b22d19 100644
--- a/daemons/lvmdbusd/automatedproperties.py
+++ b/daemons/lvmdbusd/automatedproperties.py
@@ -159,10 +159,7 @@ class AutomatedProperties(dbus.service.Object):
 			cfg.om.lookup_update(self, new_id[0], new_id[1])
 
 		# Grab the properties values, then replace the state of the object
-		# and retrieve the new values
-		# TODO: We need to add locking to prevent concurrent access to the
-		# properties so that a client is not accessing while we are
-		# replacing.
+		# and retrieve the new values.
 		o_prop = get_properties(self)
 		self.state = new_state
 		n_prop = get_properties(self)