[NEW PATCH] Related to BZ#705058 - Stop all repoStats threads that were started during reconstructMaster (via gerrit-bot)
7:08 a.m.
New patch submitted by Igor Lvovsky (ilvovsky(a)redhat.com)
You can review this change at: http://gerrit.usersys.redhat.com/894
commit 8dd633ccdc1f81c0adf5d2185b7dfad17b4ff75c
Author: Igor Lvovsky <ilvovsky(a)redhat.com>
Date: Thu Sep 1 11:20:26 2011 +0300
Related to BZ#705058 - Stop all repoStats threads that were started during
reconstructMaster
Change-Id: I8e1539ca7ebe75b1d47ef27ace8325265372fae6
diff --git a/vdsm/storage/sp.py b/vdsm/storage/sp.py
index 440a86a..014da1e 100644
--- a/vdsm/storage/sp.py
+++ b/vdsm/storage/sp.py
@@ -482,6 +482,8 @@ class StoragePool:
self.setMetaParam(PMDK_DOMAINS, domains)
self.log.info("Set storage pool domains: %s", domains)
finally:
+ # We need stop all repoStats threads that were started during
reconstructMaster
+ self.disconnectDomains()
futureMaster.releaseClusterLock()
finally:
self.id = None
3:59 a.m.
New subject: Change in vdsm[master]: BZ#732275 - Stop all repoStats threads that were started dur...
Dan Kenigsberg has submitted this change and it was merged.
Change subject: BZ#732275 - Stop all repoStats threads that were started during
reconstructMaster.
......................................................................
BZ#732275 - Stop all repoStats threads that were started during reconstructMaster.
Change-Id: I8e1539ca7ebe75b1d47ef27ace8325265372fae6
---
M vdsm/storage/sp.py
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Dan Kenigsberg: Looks good to me, approved
Igor Lvovsky: Verified
Ayal Baron: Looks good to me, but someone else must approve
Saggi Mizrahi: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.usersys.redhat.com/894
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8e1539ca7ebe75b1d47ef27ace8325265372fae6
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky <ilvovsky(a)redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
3:59 a.m.
New subject: Change in vdsm[master]: BZ#732275 - Stop all repoStats threads that were started dur...
Dan Kenigsberg has posted comments on this change.
Change subject: BZ#732275 - Stop all repoStats threads that were started during
reconstructMaster.
......................................................................
Patch Set 2: Looks good to me, approved
--
To view, visit http://gerrit.usersys.redhat.com/894
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8e1539ca7ebe75b1d47ef27ace8325265372fae6
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky <ilvovsky(a)redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
9:09 a.m.
New subject: Change in vdsm[master]: BZ#732275 - Stop all repoStats threads that were started dur...
Saggi Mizrahi has posted comments on this change.
Change subject: BZ#732275 - Stop all repoStats threads that were started during
reconstructMaster.
......................................................................
Patch Set 2: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.usersys.redhat.com/894
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8e1539ca7ebe75b1d47ef27ace8325265372fae6
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky <ilvovsky(a)redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
9:09 a.m.
New subject: Change in vdsm[master]: BZ#732275 - Stop all repoStats threads that were started dur...
Igor Lvovsky has posted comments on this change.
Change subject: BZ#732275 - Stop all repoStats threads that were started during
reconstructMaster.
......................................................................
Patch Set 2: Verified
Verified by Paikov
--
To view, visit http://gerrit.usersys.redhat.com/894
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8e1539ca7ebe75b1d47ef27ace8325265372fae6
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky <ilvovsky(a)redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
8:05 a.m.
New subject: Change in vdsm[master]: Related to BZ#705058 - Stop all repoStats threads that were ...
Saggi Mizrahi has posted comments on this change.
Change subject: Related to BZ#705058 - Stop all repoStats threads that were started during
reconstructMaster
......................................................................
Patch Set 1: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.usersys.redhat.com/894
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8e1539ca7ebe75b1d47ef27ace8325265372fae6
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky <ilvovsky(a)redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
4623
days inactive
4624
days old
vdsm-patches@lists.fedorahosted.org
5 comments
3 participants
participants (3)
-
Dan Kenigsberg -
Igor Lvovsky -
smizrahi＠redhat.com