Change in vdsm[master]: BZ#809012 - Calculate metadata offset from logical block siz...

Nigel Jones has uploaded a new change for review.

Change subject: BZ#809012 - Calculate metadata offset from logical block size
......................................................................

BZ#809012 - Calculate metadata offset from logical block size

Changes to ensure that V1 storage domain metadata area is not made dirty due to
7e79153705891a91a06eb31cd642fb209d10ff86.

Signed-off-by: Nigel Jones <nigjones(a)redhat.com&gt;
Change-Id: I0dd89a5d2a2f24e55e777529d5e667517d7c0983
---
M vdsm/storage/blockSD.py
1 file changed, 1 insertion(+), 7 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0dd89a5d2a2f24e55e777529d5e667517d7c0983
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nigel Jones <nigelj(a)gmail.com&gt;

[Federico Simoncelli]

Federico Simoncelli has posted comments on this change.

Change subject: BZ#809012 - Calculate metadata offset from logical block size
......................................................................


Patch Set 1: Looks good to me, but someone else must approve

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0dd89a5d2a2f24e55e777529d5e667517d7c0983
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nigel Jones <nigjones(a)redhat.com&gt;
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Lee Yarwood <lyarwood(a)redhat.com&gt;

[Federico Simoncelli]

Federico Simoncelli has posted comments on this change.

Change subject: BZ#809012 - Calculate metadata offset from logical block size
......................................................................


Patch Set 1: Looks good to me, approved

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0dd89a5d2a2f24e55e777529d5e667517d7c0983
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nigel Jones <nigjones(a)redhat.com&gt;
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Lee Yarwood <lyarwood(a)redhat.com&gt;

Lee Yarwood has posted comments on this change.

Change subject: BZ#809012 - Calculate metadata offset from logical block size
......................................................................


Patch Set 1: Looks good to me, but someone else must approve

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0dd89a5d2a2f24e55e777529d5e667517d7c0983
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nigel Jones <nigjones(a)redhat.com&gt;
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Lee Yarwood <lyarwood(a)redhat.com&gt;

[Nigel Jones]

Nigel Jones has posted comments on this change.

Change subject: BZ#809012 - Calculate metadata offset from logical block size
......................................................................


Patch Set 1: Verified

Tests over last 36 hours have been successful using 4k physical block iSCSI targets
(Open-E)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0dd89a5d2a2f24e55e777529d5e667517d7c0983
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nigel Jones <nigjones(a)redhat.com&gt;
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Lee Yarwood <lyarwood(a)redhat.com&gt;
Gerrit-Reviewer: Nigel Jones <nigjones(a)redhat.com&gt;

[Dan Kenigsberg]

Dan Kenigsberg has submitted this change and it was merged.

Change subject: BZ#809012 - Calculate metadata offset from logical block size
......................................................................


BZ#809012 - Calculate metadata offset from logical block size

Changes to ensure that V1 storage domain metadata area is not made dirty due to
7e79153705891a91a06eb31cd642fb209d10ff86.

Signed-off-by: Nigel Jones <nigjones(a)redhat.com&gt;
Change-Id: I0dd89a5d2a2f24e55e777529d5e667517d7c0983
---
M vdsm/storage/blockSD.py
1 file changed, 1 insertion(+), 7 deletions(-)

Approvals:
  Lee Yarwood: Looks good to me, but someone else must approve
  Federico Simoncelli: Looks good to me, approved
  Dan Kenigsberg: 
  Nigel Jones: Verified


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0dd89a5d2a2f24e55e777529d5e667517d7c0983
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nigel Jones <nigjones(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Lee Yarwood <lyarwood(a)redhat.com&gt;
Gerrit-Reviewer: Nigel Jones <nigjones(a)redhat.com&gt;