Francesco Romani has uploaded a new change for review.
Change subject: janitorial: vm: remove _reportError ......................................................................
janitorial: vm: remove _reportError
Vm._reportError was an early attempt to build what we have now with response.error().
With the new code in place there is no more reason to use it, so this patch removes it.
Change-Id: Ie23167c0289bdb0326dcfe48903cd1bae905e9b5 Signed-off-by: Francesco Romani fromani@redhat.com --- M vdsm/virt/vm.py 1 file changed, 15 insertions(+), 27 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/69/38269/1
diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py index cd392c7..4be63ce 100644 --- a/vdsm/virt/vm.py +++ b/vdsm/virt/vm.py @@ -42,7 +42,7 @@ from vdsm import utils from vdsm.compat import pickle from vdsm.config import config -from vdsm.define import ERROR, NORMAL, doneCode, errCode +from vdsm.define import ERROR, NORMAL, doneCode from vdsm.netinfo import DUMMY_BRIDGE from storage import outOfProcess as oop from storage import sd @@ -2650,15 +2650,15 @@
if not params: self.log.error("updateVmPolicy got an empty policy.") - return self._reportError(key='MissParam', - msg="updateVmPolicy got an empty policy.") + return response.error('MissParam', + 'updateVmPolicy got an empty policy.')
# # Get the current QoS block metadata_modified = False qos = self._getVmPolicy() if qos is None: - return self._reportError(key='updateVmPolicyErr') + return response.error('updateVmPolicyErr')
# # Process provided properties, remove property after it is processed @@ -2712,8 +2712,7 @@ if e.get_error_code() == libvirt.VIR_ERR_NO_DOMAIN: return response.error('noVM') else: - return self._reportError(key='updateVmPolicyErr', - msg=e.message) + return response.error('updateVmPolicyErr', e.message)
return {'status': doneCode}
@@ -2771,9 +2770,9 @@ found_device = self._findDeviceByNameOrPath(device_name, device_path) if found_device is None: - return self._reportError( - key='updateIoTuneErr', - msg="Device {} not found".format(device_name)) + return response.error( + 'updateIoTuneErr', + "Device {} not found".format(device_name))
# Merge the update with current values dom = found_device.getXML() @@ -2799,8 +2798,7 @@ if e.get_error_code() == libvirt.VIR_ERR_NO_DOMAIN: return response.error('noVM') else: - return self._reportError(key='updateIoTuneErr', - msg=e.message) + return response.error('updateIoTuneErr', e.message)
# Update both the ioTune arguments and device xml DOM # so we are still up-to-date @@ -3956,7 +3954,7 @@ def setBalloonTarget(self, target):
if self._dom is None: - return self._reportError(key='balloonErr') + return response.error('balloonErr') try: target = int(target) self._dom.setMemory(target) @@ -3980,8 +3978,8 @@ try: self._dom.setSchedulerParameters({'vcpu_quota': int(quota)}) except ValueError: - return self._reportError(key='cpuTuneErr', - msg='an integer is required for period') + return response.error('cpuTuneErr', + 'an integer is required for period') except libvirt.libvirtError as e: return self._reportException(key='cpuTuneErr', msg=e.message) return {'status': doneCode} @@ -3990,21 +3988,11 @@ try: self._dom.setSchedulerParameters({'vcpu_period': int(period)}) except ValueError: - return self._reportError(key='cpuTuneErr', - msg='an integer is required for period') + return response.error('cpuTuneErr', + 'an integer is required for period') except libvirt.libvirtError as e: return self._reportException(key='cpuTuneErr', msg=e.message) return {'status': doneCode} - - def _reportError(self, key, msg=None): - """ - Produce an error status. - """ - if msg is None: - error = errCode[key] - else: - error = response.error(key, msg) - return error
def _reportException(self, key, msg=None): """ @@ -4012,7 +4000,7 @@ This method should be called only within exception-handling context. """ self.log.exception("Operation failed") - return self._reportError(key, msg) + return response.error(key, msg)
def _getUnderlyingDeviceAddress(self, devXml): """
automation@ovirt.org has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
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.5', 'ovirt-3.4', 'ovirt-3.3'])
oVirt Jenkins CI Server has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_virt_functional_tests_gerrit/2591/ : There was an infra issue, please contact infra@ovirt.org
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16167/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/15367/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16337/ : FAILURE
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created_staging/1143/ : FAILURE
automation@ovirt.org has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 2:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
Francesco Romani has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 2:
rebased
automation@ovirt.org has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 3:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
oVirt Jenkins CI Server has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_virt_functional_tests_gerrit/2606/ : There was an infra issue, please contact infra@ovirt.org
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16246/ : FAILURE
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/15446/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16416/ : SUCCESS
oVirt Jenkins CI Server has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 3:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_virt_functional_tests_gerrit/2610/ : There was an infra issue, please contact infra@ovirt.org
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16250/ : FAILURE
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/15450/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16420/ : SUCCESS
automation@ovirt.org has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 4:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
oVirt Jenkins CI Server has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 4:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_virt_functional_tests_gerrit/2637/ : There was an infra issue, please contact infra@ovirt.org
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16285/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/15485/ : FAILURE
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16455/ : FAILURE
automation@ovirt.org has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 5:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
oVirt Jenkins CI Server has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 5:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_virt_functional_tests_gerrit/2642/ : There was an infra issue, please contact infra@ovirt.org
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16294/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/15494/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16464/ : SUCCESS
Nir Soffer has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 5: Code-Review+1
automation@ovirt.org has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 6:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
Francesco Romani has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 6:
rebased
oVirt Jenkins CI Server has posted comments on this change.
Change subject: janitorial: vm: remove _reportError ......................................................................
Patch Set 6:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16320/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/15520/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16490/ : FAILURE
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 7:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 7:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16330/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/15530/ : FAILURE
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16500/ : SUCCESS
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 8:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
Francesco Romani has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 8:
rebased with no code changes
Nir Soffer has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 8: Code-Review+1
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 8:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16403/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16574/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_el_gerrit/15603/ : FAILURE
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 9:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 10:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
Francesco Romani has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 10:
rebased with no code changes
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 11:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 12:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
Francesco Romani has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 12:
rebased
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 12:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16741/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/16913/ : SUCCESS
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 13:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 13:
Build Started (1/2) -> http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16942/
Francesco Romani has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 13:
rebased
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 13:
Build Started (2/2) -> http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17115/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 13:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/16942/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17115/ : SUCCESS
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 14:
Build Started (1/2) -> http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17021/
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 14:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 15:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 15:
Build Started (1/2) -> http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17032/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 14:
Build Started (2/2) -> http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17195/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 14:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17021/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17195/ : FAILURE
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 15:
Build Started (2/2) -> http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17206/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 15:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17032/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17206/ : SUCCESS
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 16:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 16:
Build Started (1/2) -> http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18228/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 16:
Build Started (2/2)
0 -> http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1458/
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 16:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18228/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1458/ : 0
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 17:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
Vinzenz Feenstra has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 17: Code-Review+1
Nir Soffer has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 17: Code-Review+1
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 18:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 19:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 20:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 21:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 22:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 23:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 24:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 25:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 26:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 27:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 28:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 29:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 30:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 31:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 32:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 33:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
Nir Soffer has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 33: Code-Review+1
Francesco Romani has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 33: Verified-1
let's get rid of this old patch, either merging or abandoning. But first let's make correct. vm.py still needs errCode even after this patch.
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 34:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 35:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 36:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 37:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 38:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 39:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 40:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 41:
* 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.5', 'ovirt-3.4', 'ovirt-3.3'])
Dan Kenigsberg has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 41: Code-Review+2
Francesco Romani has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 41: Verified+1
verified running patched VDSM.
Dan Kenigsberg has submitted this change and it was merged.
Change subject: vm: remove _reportError ......................................................................
vm: remove _reportError
Vm._reportError was an early attempt to build what we have now with response.error().
With the new code in place there is no more reason to use it, so this patch removes it.
Change-Id: Ie23167c0289bdb0326dcfe48903cd1bae905e9b5 Signed-off-by: Francesco Romani fromani@redhat.com Reviewed-on: https://gerrit.ovirt.org/38269 Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg danken@redhat.com --- M vdsm/virt/vm.py 1 file changed, 14 insertions(+), 26 deletions(-)
Approvals: Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified
automation@ovirt.org has posted comments on this change.
Change subject: vm: remove _reportError ......................................................................
Patch Set 42:
* Update tracker::IGNORE, no Bug-Url found * Set MODIFIED::IGNORE, no Bug-Url found.
vdsm-patches@lists.fedorahosted.org