Change in vdsm[master]: clusterlock: Remove unneeded workaround

ishaby at redhat.com ishaby at redhat.com
Wed Nov 11 07:12:06 UTC 2015


Idan Shaby has posted comments on this change.

Change subject: clusterlock: Remove unneeded workaround
......................................................................


Patch Set 4:

Sure:
I had an environment of three hosts.
I blocked the outgoing packets from one of the hosts (id=3) to the storage server, and watched its status changing from live, to fail, and eventually to dead.
To do that, I used manhole:

>>> irs.getHostLeaseStatus({'2b95e786-1d9f-4567-8133-3fc350d8cea4':3})
{'status': {'message': 'OK', 'code': 0}, 'domains': {'2b95e786-1d9f-4567-8133-3fc350d8cea4': 'live'}}

>>> irs.getHostLeaseStatus({'2b95e786-1d9f-4567-8133-3fc350d8cea4':3})
{'status': {'message': 'OK', 'code': 0}, 'domains': {'2b95e786-1d9f-4567-8133-3fc350d8cea4': 'fail'}}

>>> irs.getHostLeaseStatus({'2b95e786-1d9f-4567-8133-3fc350d8cea4':3})
{'status': {'message': 'OK', 'code': 0}, 'domains': {'2b95e786-1d9f-4567-8133-3fc350d8cea4': 'dead'}}


To verify that only this host's (id=3) status has changed, I ran the command:
>>> sanlock.get_hosts('c8d24dc0-dfa6-4140-a11f-8b943f2dae0d')

And watched its status changing (where the others stayed the same):
[{'generation': 12, 'host_id': 1, 'flags': 3, 'io_timeout': 10, 'timestamp': 452680}, {'generation': 7, 'host_id': 2, 'flags': 3, 'io_timeout': 10, 'timestamp': 11591}, {'generation': 10, 'host_id': 3, 'flags': 3, 'io_timeout': 10, 'timestamp': 430448}]
[{'generation': 12, 'host_id': 1, 'flags': 3, 'io_timeout': 10, 'timestamp': 450199}, {'generation': 7, 'host_id': 2, 'flags': 3, 'io_timeout': 10, 'timestamp': 9101}, {'generation': 9, 'host_id': 3, 'flags': 4, 'io_timeout': 10, 'timestamp': 427850}]

-- 
To view, visit https://gerrit.ovirt.org/31162
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ide75e749fbc2916540c2b526b78fedc247b5c6f9
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: No


More information about the vdsm-patches mailing list