Change in vdsm[master]: Fix security option for attachSD and detachSD

[Federico Simoncelli]

Federico Simoncelli has uploaded a new change for review.

Change subject: Fix security option for attachSD and detachSD
......................................................................

Fix security option for attachSD and detachSD

Change-Id: I617f9a63f9b788f030aa6ad3751922db570b6d6d
---
M vdsm/storage/sp.py
1 file changed, 4 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/03/203/1
--
To view, visit http://gerrit.ovirt.org/203
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I617f9a63f9b788f030aa6ad3751922db570b6d6d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com&gt;

[Federico Simoncelli]

Federico Simoncelli has abandoned this change.

Change subject: Fix security option for attachSD and detachSD
......................................................................


Patch Set 2: Abandoned

Fixed in an other patch: http://gerrit.ovirt.org/587

--
To view, visit http://gerrit.ovirt.org/203
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: abandon
Gerrit-Change-Id: I617f9a63f9b788f030aa6ad3751922db570b6d6d
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com&gt;

Saggi Mizrahi has posted comments on this change.

Change subject: Fix security option for attachSD and detachSD
......................................................................


Patch Set 2: Do not submit

Security override doesn't trickle down so using it should not be as commons as it
already is. Because you know that while creating the pool object and taking the lock and
the master domain you effectively have SPM lock you should just set safe after acquiring
the cluster lock

--
To view, visit http://gerrit.ovirt.org/203
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I617f9a63f9b788f030aa6ad3751922db570b6d6d
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com&gt;