Federico Simoncelli has uploaded a new change for review.
Change subject: Support sanlock killpath for the SPM
......................................................................
Support sanlock killpath for the SPM
Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com>
Change-Id: I68fecd8dd9753087c4840cb098ba28b10d76d8b8
---
M vdsm.spec.in
M vdsm/Makefile.am
M vdsm/constants.py.in
A vdsm/sanlock-stop-spm
M vdsm/storage/safelease.py
5 files changed, 15 insertions(+), 2 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/85/6385/1
--
To view, visit http://gerrit.ovirt.org/6385
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I68fecd8dd9753087c4840cb098ba28b10d76d8b8
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com>
Federico Simoncelli has uploaded a new change for review.
Change subject: BZ#752470 Related: Don't use SIGTERM in spmprotect.sh
......................................................................
BZ#752470 Related: Don't use SIGTERM in spmprotect.sh
Attempting to stop the SPM (SIGUSR1) and using SIGKILL if that fails is
already enough. In any case giving vdsm only 2 seconds between SIGTERM
and SIGKILL was a too short timeout to hope in a restart.
Change-Id: I83e07f966b69e2f8ba803f0d2a5020d90e6029d8
---
M vdsm/storage/protect/spmprotect.sh
1 file changed, 1 insertion(+), 3 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/65/265/1
--
To view, visit http://gerrit.ovirt.org/265
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I83e07f966b69e2f8ba803f0d2a5020d90e6029d8
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com>
Federico Simoncelli has uploaded a new change for review.
Change subject: disconnectStoragePool must fail if resources are in use
......................................................................
disconnectStoragePool must fail if resources are in use
Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com>
Change-Id: Ic2f4ba01dfd8ba4de0a29a3ca8d0ca26aef83a80
---
M vdsm/storage/safelease.py
M vdsm/storage/sd.py
M vdsm/storage/sp.py
3 files changed, 20 insertions(+), 22 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/45/4845/1
--
To view, visit http://gerrit.ovirt.org/4845
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic2f4ba01dfd8ba4de0a29a3ca8d0ca26aef83a80
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com>
Federico Simoncelli has uploaded a new change for review.
Change subject: Reuse the stored pool host id on reconstructMaster
......................................................................
Reuse the stored pool host id on reconstructMaster
When the SPM is unable to reach the master domain the VDSM process
might be fenced (killed) by safelease or sanlock.
The engine will try to reconstruct the master and VDSM would use
a temporary id (eg: 250) to execute the operation. This patch uses
the information stored by connectStoragePool to acquire the same
host id that was in use before VDSM restarted.
Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com>
Change-Id: I44ff6d04220032ac1f6bd0a7819132103050b516
---
M vdsm/storage/hsm.py
M vdsm/storage/sp.py
2 files changed, 13 insertions(+), 11 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/65/4965/1
--
To view, visit http://gerrit.ovirt.org/4965
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I44ff6d04220032ac1f6bd0a7819132103050b516
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com>
Saggi Mizrahi has uploaded a new change for review.
Change subject: (Schema) Define an error type
......................................................................
(Schema) Define an error type
Change-Id: I89ddcc6f18d982cb1ea5e91a13bcf9f939ae6cb1
Signed-off-by: Saggi Mizrahi <smizrahi(a)redhat.com>
---
M vdsm_api/vdsmapi-schema.json
1 file changed, 21 insertions(+), 17 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/46/7146/1
diff --git a/vdsm_api/vdsmapi-schema.json b/vdsm_api/vdsmapi-schema.json
index f040667..7d54e09 100644
--- a/vdsm_api/vdsmapi-schema.json
+++ b/vdsm_api/vdsmapi-schema.json
@@ -217,6 +217,23 @@
'returns': {'*power': 'FenceNodePowerStatus'}}
##
+# @Error:
+#
+# Represents an error in VDSM.
+#
+# @code: Error identifier.
+#
+# @message: Extra information about the error, should be used for debugging
+# and logging. This only contains auxillery information and
+# shouldn't be used in any logic. The exact phrasing may change at
+# any time.
+#
+# Since: 4.10.0
+#XXX: Create an error list.
+##
+{'type': 'Error', 'data': {'code': 'int', 'message': 'str'}}
+
+##
# @TaskInfo:
#
# Extra information about a task.
@@ -795,14 +812,14 @@
#
# Information about a single block device path.
#
-# @physdev: The physical device name
+# @physdev: The physical device name
#
# @active: Whether the path is active. This will be false if the path has
# failed.
#
-# @lun: The SCSI Logical Unit Number associated with this path
+# @lun: The SCSI Logical Unit Number associated with this path
#
-# @type: The type of path (@BlockDeviceType)
+# @type: The type of path (@BlockDeviceType)
#
# Since: 4.10.0
##
@@ -2410,19 +2427,6 @@
'data': {'type': 'ConnectionRefType', 'params': 'ConnectionRefParameters'}}
##
-# @ConnectionError:
-#
-# A tuple containing @ConnectionRef error information.
-#
-# @code: The error code
-#
-# @message: The error message
-#
-# Since: 4.10.0
-##
-{'type': 'ConnectionError', 'data': {'code': 'int', 'message': 'str'}}
-
-##
# @ConnectionRef:
#
# Contains the details of a registered storage connection.
@@ -2436,7 +2440,7 @@
# Since: 4.10.0
##
{'type': 'ConnectionRef',
- 'data': {'connected': 'bool', 'lastError': 'ConnectionError',
+ 'data': {'connected': 'bool', 'lastError': 'Error',
'connectionInfo': 'ConnectionRefParameters'}}
##
--
To view, visit http://gerrit.ovirt.org/7146
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I89ddcc6f18d982cb1ea5e91a13bcf9f939ae6cb1
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com>
Saggi Mizrahi has uploaded a new change for review.
Change subject: (Schema) Consolidate changeCD and changeFloppy to a single call
......................................................................
(Schema) Consolidate changeCD and changeFloppy to a single call
Change-Id: I10332653af4cebd42bccffe178f84feac4ba5357
Signed-off-by: Saggi Mizrahi <smizrahi(a)redhat.com>
---
M vdsm_api/vdsmapi-schema.json
1 file changed, 4 insertions(+), 4 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/50/7150/1
diff --git a/vdsm_api/vdsmapi-schema.json b/vdsm_api/vdsmapi-schema.json
index 581c5ff..9ee296b 100644
--- a/vdsm_api/vdsmapi-schema.json
+++ b/vdsm_api/vdsmapi-schema.json
@@ -3944,18 +3944,18 @@
'DriveSpecPayload', 'DriveSpecPath']}
##
-# @VM.changeCD:
+# @VM.changeMedia:
#
-# Change the CD in the VM's CD-ROM device.
+# Change the media in a drive.
#
-# @driveSpec: The specification of the new CD
+# @driveSpec: The specification of the new media
#
# Returns:
# @vmList: The VM definition, as updated
#
# Since: 4.10.0
##
-{'command': {'class': 'VM', 'name': 'changeCD'},
+{'command': {'class': 'VM', 'name': 'changeMedia'},
'data': {'driveSpec': 'DriveSpec'},
'returns': {'vmList': 'VmDefinition'}}
--
To view, visit http://gerrit.ovirt.org/7150
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I10332653af4cebd42bccffe178f84feac4ba5357
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com>