Change in vdsm[master]: Change mom policy related interface to support multi-policy
Itamar Heim has abandoned this change. Change subject: Change mom policy related interface to support multi-policy .. Abandoned no reply - abandoning - please restore if still relevant -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: abandon Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 14 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Better Saggi Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Itamar Heim Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: Saggi Mizrahi Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Itamar Heim has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 14: ping -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 14 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Better Saggi Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Itamar Heim Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: Saggi Mizrahi Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Saggi Mizrahi has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 14: One of SaggiMizrahi's automated scripts discovered this patch might require his approval. Please wait until he had time to check it out. -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 14 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Better Saggi Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Itamar Heim Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: Saggi Mizrahi Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Itamar Heim has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 14: ping -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 14 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Better Saggi Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Itamar Heim Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 14: Verified-1 Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4984/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/4098/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/4908/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_network_functional_tests/693/ : FAILURE -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 14 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Better Saggi Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 13: Verified-1 Build Failed http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4983/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/4097/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/4907/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_network_functional_tests/692/ : FAILURE -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 13 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Better Saggi Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Dan Kenigsberg has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 12: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 12 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Better Saggi Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Dan Kenigsberg has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 12: Code-Review-1 Please rebase an retest. -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 12 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Better Saggi Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Dan Kenigsberg has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 12: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 12 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Better Saggi Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: Saggi Mizrahi Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 12: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/4548/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/3651/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/4470/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 12 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Better Saggi Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: Saggi Mizrahi Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Dan Kenigsberg has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 11: Code-Review-1 Mei Liu, I am sorry that it takes so long, but an introduction of a new API requires much more detailed explanation. Please improve the commit message, include link to feature page if existing, etc etc. -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 11 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Better Saggi Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: Saggi Mizrahi Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Saggi Mizrahi has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 11: I reviewed *only* the API change and I approve. I have not looked at the implementation! -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 11 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Better Saggi Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: Saggi Mizrahi Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Dan Kenigsberg has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 11: Is this urgently needed for ovirt-3.3? I would like to avoid needless destabilization. -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 11 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 11: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/2585/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3392/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3476/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 11 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Dan Kenigsberg has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 10: (1 inline comment) I suspect that this is an ovirt-3.3 blocker? Nonetheless could you explain why resetMomPolicy is helpful? What is its expected use case? File tests/functional/Makefile.am Line 29:xmlrpcTests.py \ Line 30:$(NULL) Line 31: Line 32: nodist_vdsmfunctests_PYTHON = \ Line 33: momTests.py Tab inconsistency Line 34: Line 35: EXTRA_DIST = \ Line 36:momTests.py.in Line 37: -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 10 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Doron Fediuck has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 10: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 10 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Martin Sivák has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 10: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 10 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 10: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/2540/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3347/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3430/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 10 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Martin Sivák has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 9: I would prefer that you didn't submit this (1 inline comment) If we can't use default arguments because of our domain specific json descriptions, at least make momIf use it and convert the data in API. File vdsm/momIF.py Line 58: ret['ksmCpu'] = stats['ksmd_cpu_usage'] Line 59: return ret Line 60: Line 61: def setPolicy(self, policyParam): Line 62: if isinstance(policyParam, dict) and len(policyParam) == 1: I do not like this much, I would prefer having setPolicy(self, policyStr, name = None) over the dictionary logic here.. Line 63: policy = policyParam.popitem() Line 64: self._mom.setNamedPolicy(policy[0], policy[1]) Line 65: else: Line 66: # mom.setPolicy will raise an exception on failure. -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 9 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Martin Sivák Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 9: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/2538/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3345/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3428/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 9 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 8: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/2435/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/3245/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/3324/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Royce Lv has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 7: Thanks, Doron! Currently this patch series are waiting for some patch of MOM to be merged so that it won't break vdsm installation. I'll rebase this for your request as soon as the MOM side is ready. -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Doron Fediuck has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 7: Looks good to me, but someone else must approve Royce, good job! You have my ack, but there's one thing I'd like to see changed (in this patch or later on); errCode['momErr'] is returned for most actions / functions, with absolutely no logging and/or some message that could help us identify issues. Both get and reset policy in this case may end up with the same error and no clue what and why it happened. So my requests are: 1. Consider adding meaningful error codes (momPolicyErr, momConfigErr, ..). 2. (Starting now) log every error we hit in the code. Thanks! Doron -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 7: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1582/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1545/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 7: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1582/ (2/2) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 7: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1545/ (1/2) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Royce Lv has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 6: (1 inline comment) File tests/functional/momTests.py.in Line 69: @testValidation.ValidateRunningAsRoot Line 70: def testDefaultPolicy(self): Line 71: r = self.s.getMOMPolicy() Line 72: Line 73: policy_dir = '@CONFDIR@/mom-policy-dir' Done Line 74: names = sorted(os.listdir(policy_dir)) Line 75: for name in names: Line 76: if not name.startswith('.') and name.endswith('.policy'): Line 77: fname = os.path.join(policy_dir, name) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Laszlo Hornyak Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 6: Build Successful http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1510/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1473/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 6: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1510/ (2/2) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 6: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1473/ (1/2) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Adam Litke Gerrit-Reviewer: Doron Fediuck Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 5: Fails Build Failed http://alterway01.ovirt.org/job/vdsm_unit_tests_gerrit/1449/ : FAILURE http://alterway01.ovirt.org/job/vdsm_pep8_gerrit/1410/ : FAILURE -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 5: Build Started http://alterway01.ovirt.org/job/vdsm_pep8_gerrit/1410/ (2/2) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 5: Build Started http://alterway01.ovirt.org/job/vdsm_unit_tests_gerrit/1449/ (1/2) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Mei Liu Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 5: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1433/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1472/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/580/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 5: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/580/ (3/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 5: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1472/ (1/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 5: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1433/ (2/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 4: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1432/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1471/ : FAILURE http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/579/ : FAILURE -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 4: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1471/ (2/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 4: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1432/ (1/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 4: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/579/ (3/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 3: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1431/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1470/ : FAILURE http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/578/ : FAILURE -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 3: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/578/ (3/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 3: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1470/ (2/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 3: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1431/ (1/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 2: Build Successful http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1395/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1434/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/542/ : SUCCESS -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 2: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1395/ (3/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 2: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1434/ (2/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 2: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/542/ (1/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 1: Fails Build Failed http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1389/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1428/ : SUCCESS http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/536/ : FAILURE -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 1: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/536/ (2/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 1: Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1428/ (1/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
oVirt Jenkins CI Server has posted comments on this change. Change subject: Change mom policy related interface to support multi-policy .. Patch Set 1: Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1389/ (3/3) -- To view, visit http://gerrit.ovirt.org/12466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv Gerrit-Reviewer: oVirt Jenkins CI Server ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
Change in vdsm[master]: Change mom policy related interface to support multi-policy
Royce Lv has uploaded a new change for review. Change subject: Change mom policy related interface to support multi-policy .. Change mom policy related interface to support multi-policy Change-Id: I09c40fee74b10d3eb41f4ec3ca18096a9b20dfcd Signed-off-by: Royce Lv --- M .gitignore M tests/functional/Makefile.am R tests/functional/momTests.py.in M vdsm/API.py M vdsm/BindingXMLRPC.py M vdsm/momIF.py M vdsm_cli/vdsClient.py 7 files changed, 56 insertions(+), 4 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/66/12466/1 diff --git a/.gitignore b/.gitignore index 41066dd..b9b992c 100644 --- a/.gitignore +++ b/.gitignore @@ -19,6 +19,7 @@ config.status configure results.log +tests/functional/momTests.py tests/run_tests.sh tests/run_tests_local.sh vdsm-*.tar.gz diff --git a/tests/functional/Makefile.am b/tests/functional/Makefile.am index 030242b..fe4f276 100644 --- a/tests/functional/Makefile.am +++ b/tests/functional/Makefile.am @@ -17,12 +17,21 @@ # # Refer to the README and COPYING files for full details of the license # +include $(top_srcdir)/build-aux/Makefile.subs vdsmfunctestsdir = ${vdsmtestsdir}/functional dist_vdsmfunctests_PYTHON = \ - momTests.py \ sosPluginTests.py \ xmlrpcTests.py \ $(NULL) +nodist_vdsmfunctests_PYTHON = \ +momTests.py + +EXTRA_DIST = \ + momTests.py.in + +CLEANFILES = \ + $(nodist_vdsmfunctests_PYTHON) + diff --git a/tests/functional/momTests.py b/tests/functional/momTests.py.in similarity index 76% rename from tests/functional/momTests.py rename to tests/functional/momTests.py.in index 1981e05..c54b427 100644 --- a/tests/functional/momTests.py +++ b/tests/functional/momTests.py.in @@ -18,6 +18,7 @@ # Refer to the README and COPYING files for full details of the license # import imp +import os import random import time @@ -54,3 +55,17 @@ hostStats = s.getVdsStats()['info'] self.assertEqual(bool(run), hostStats['ksmState']) self.assertEqual(pages_to_scan, hostStats['ksmPages']) + +@testValidation.ValidateRunningAsRoot +def testDefaultPolicy(self): +s = vdscli.connect() +r = s.getMOMPolicy() + +policy_dir = '@CONFDIR@/mom-policy-dir' +names = sorted(os.listdir(policy_dir)) +for name in names: +if name.startswith('.') or not name.endswith('.policy'): +fname = os.path.join(policy_dir, name) +with open(fname, 'r') as f: +policyStr = f.read() +self.assertEqual(policyStr, r['policyDict'].get(name)) diff --git a/vdsm/API.py b/vdsm/API.py index 0046b57..0f02fde 100644 --- a/vdsm/API.py +++ b/vdsm/API.py @@ -1379,6 +1379,12 @@ except: return errCode['momErr'] +def getMOMPolicy(self): +try: +return dict(status=doneCode, +policyDict=self._cif.mom.getPolicy()) +except: +return errCode['momErr'] # take a rough estimate on how much free mem is available for new vm # memTotal = memFree + memCached + mem_used_by_non_qemu + resident . # simply returning (memFree + memCached) is not good enough, as the diff --git a/vdsm/BindingXMLRPC.py b/vdsm/BindingXMLRPC.py index 9a4db12..a540900 100644 --- a/vdsm/BindingXMLRPC.py +++ b/vdsm/BindingXMLRPC.py @@ -401,6 +401,10 @@ api = API.Global() return api.setMOMPolicy(policy) +def getMOMPolicy(self): +api = API.Global() +return api.getMOMPolicy() + def domainActivate(self, sdUUID, spUUID, options=None): domain = API.StorageDomain(sdUUID) return domain.activate(spUUID) @@ -807,6 +811,7 @@ (self.fenceNode, 'fenceNode'), (self.prepareForShutdown, 'prepareForShutdown'), (self.setLogLevel, 'setLogLevel'), +(self.getMOMPolicy, 'getMOMPolicy'), (self.setMOMPolicy, 'setMOMPolicy'), (self.vmHotplugDisk, 'hotplugDisk'), (self.vmHotunplugDisk, 'hotunplugDisk'), diff --git a/vdsm/momIF.py b/vdsm/momIF.py index d0f63d1..831b649 100644 --- a/vdsm/momIF.py +++ b/vdsm/momIF.py @@ -56,9 +56,16 @@ ret['ksmCpu'] = stats['ksmd_cpu_usage'] return ret -def setPolicy(self, policyStr): -# mom.setPolicy will raise an exception on failure. -self._mom.setPolicy(policyStr) +def setPolicy(self, policyParam): +if isinstance(policyParam, dict) and len(policyParam) == 1: +policy = policyParam.popItem() +self._mom.setNamedPolicy(policy[0], policy[1]) +else: +# mom.setPolicy will raise an exception on failure. +self._mom.setPolicy(policyParam) + +def getPolicy(self): +return self._mom.getNamedPolicies() def stop(self): if self._mom is not None: diff --git