Change in vdsm[master]: yml: return type fixes for StoragePool.getSpmStatus
Nir Soffer has submitted this change and it was merged. Change subject: yml: return type fixes for StoragePool.getSpmStatus .. yml: return type fixes for StoragePool.getSpmStatus Change-Id: Ieb9fbae250507d4cf02ef9b73eee6403a06cd9d2 Signed-off-by: Piotr KliczewskiReviewed-on: https://gerrit.ovirt.org/59706 Continuous-Integration: Jenkins CI Reviewed-by: Yaniv Bronhaim Reviewed-by: Adam Litke --- M lib/api/vdsm-api.yml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Piotr Kliczewski: Verified Adam Litke: Looks good to me, approved Yaniv Bronhaim: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59706 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb9fbae250507d4cf02ef9b73eee6403a06cd9d2 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Freddy Rolland Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks ___ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
Change in vdsm[master]: yml: return type fixes for StoragePool.getSpmStatus
gerrit-hooks has posted comments on this change. Change subject: yml: return type fixes for StoragePool.getSpmStatus .. Patch Set 3: * #59706::Update tracker: OK * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit https://gerrit.ovirt.org/59706 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ieb9fbae250507d4cf02ef9b73eee6403a06cd9d2 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr KliczewskiGerrit-Reviewer: Adam Litke Gerrit-Reviewer: Freddy Rolland Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
Change in vdsm[master]: yml: return type fixes for StoragePool.getSpmStatus
Adam Litke has posted comments on this change. Change subject: yml: return type fixes for StoragePool.getSpmStatus .. Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/59706 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ieb9fbae250507d4cf02ef9b73eee6403a06cd9d2 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr KliczewskiGerrit-Reviewer: Adam Litke Gerrit-Reviewer: Freddy Rolland Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
Change in vdsm[master]: yml: return type fixes for StoragePool.getSpmStatus
Yaniv Bronhaim has posted comments on this change. Change subject: yml: return type fixes for StoragePool.getSpmStatus .. Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/59706 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ieb9fbae250507d4cf02ef9b73eee6403a06cd9d2 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr KliczewskiGerrit-Reviewer: Adam Litke Gerrit-Reviewer: Freddy Rolland Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
Change in vdsm[master]: yml: return type fixes for StoragePool.getSpmStatus
Piotr Kliczewski has posted comments on this change. Change subject: yml: return type fixes for StoragePool.getSpmStatus .. Patch Set 2: Let's keep this patch for some time longer -- To view, visit https://gerrit.ovirt.org/59706 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ieb9fbae250507d4cf02ef9b73eee6403a06cd9d2 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr KliczewskiGerrit-Reviewer: Adam Litke Gerrit-Reviewer: Freddy Rolland Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
Change in vdsm[master]: yml: return type fixes for StoragePool.getSpmStatus
Piotr Kliczewski has posted comments on this change. Change subject: yml: return type fixes for StoragePool.getSpmStatus .. Patch Set 2: Verified+1 Verified by running schema verification tests -- To view, visit https://gerrit.ovirt.org/59706 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ieb9fbae250507d4cf02ef9b73eee6403a06cd9d2 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr KliczewskiGerrit-Reviewer: Adam Litke Gerrit-Reviewer: Freddy Rolland Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
Change in vdsm[master]: yml: return type fixes for StoragePool.getSpmStatus
Freddy Rolland has posted comments on this change. Change subject: yml: return type fixes for StoragePool.getSpmStatus .. Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/59706 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ieb9fbae250507d4cf02ef9b73eee6403a06cd9d2 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr KliczewskiGerrit-Reviewer: Adam Litke Gerrit-Reviewer: Freddy Rolland Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
Change in vdsm[master]: yml: return type fixes for StoragePool.getSpmStatus
Piotr Kliczewski has posted comments on this change. Change subject: yml: return type fixes for StoragePool.getSpmStatus .. Patch Set 1: Verified+1 Verified by running unit test. The message was copied from Nir's vdsm log. -- To view, visit https://gerrit.ovirt.org/59706 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ieb9fbae250507d4cf02ef9b73eee6403a06cd9d2 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr KliczewskiGerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
Change in vdsm[master]: yml: return type fixes for StoragePool.getSpmStatus
Piotr Kliczewski has uploaded a new change for review. Change subject: yml: return type fixes for StoragePool.getSpmStatus .. yml: return type fixes for StoragePool.getSpmStatus Change-Id: Ieb9fbae250507d4cf02ef9b73eee6403a06cd9d2 Signed-off-by: Piotr Kliczewski--- M lib/api/vdsm-api.yml M tests/vdsmapi_test.py 2 files changed, 7 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/06/59706/1 diff --git a/lib/api/vdsm-api.yml b/lib/api/vdsm-api.yml index 20de0a6..7dd95d1 100644 --- a/lib/api/vdsm-api.yml +++ b/lib/api/vdsm-api.yml @@ -5468,7 +5468,7 @@ - description: The lock version of the Storage Pool name: spmLver -type: int +type: long type: object StorageDomainCreateArgumentsBlock: diff --git a/tests/vdsmapi_test.py b/tests/vdsmapi_test.py index c3cacf0..eddb735 100644 --- a/tests/vdsmapi_test.py +++ b/tests/vdsmapi_test.py @@ -601,3 +601,9 @@ _schema.schema().verify_retval( vdsmapi.MethodRep('Task', 'getStatus'), ret) + +def test_spm_status(self): +ret = {'spmId': 1, 'spmStatus': 'SPM', 'spmLver': 10} + +_schema.schema().verify_retval( +vdsmapi.MethodRep('StoragePool', 'getSpmStatus'), ret) -- To view, visit https://gerrit.ovirt.org/59706 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ieb9fbae250507d4cf02ef9b73eee6403a06cd9d2 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
Change in vdsm[master]: yml: return type fixes for StoragePool.getSpmStatus
gerrit-hooks has posted comments on this change. Change subject: yml: return type fixes for StoragePool.getSpmStatus .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url. * Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 'ovirt-4.0']) -- To view, visit https://gerrit.ovirt.org/59706 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ieb9fbae250507d4cf02ef9b73eee6403a06cd9d2 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr KliczewskiGerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org