Dan Kenigsberg has submitted this change and it was merged.

Change subject: xmlrpc: remove useless 'serverPort' attribute
......................................................................


xmlrpc: remove useless 'serverPort' attribute

The BindingXMLRPC class used to have one
'serverPort' attribute, which was used only on the migration
path.

Since commit 3be9ad1f278 was merged some time ago, this
attribute is unused and could be removed.

This patch does that.

Change-Id: I120754b2c934da3393e5ea4323225c14e25b2087
Signed-off-by: Francesco Romani <from...@redhat.com>
Reviewed-on: http://gerrit.ovirt.org/35271
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
Reviewed-by: Piotr Kliczewski <piotr.kliczew...@gmail.com>
Reviewed-by: Yeela Kaplan <ykap...@redhat.com>
Reviewed-by: mooli tayer <mta...@redhat.com>
---
M tests/jsonRpcHelper.py
M vdsm/clientIF.py
M vdsm/rpc/BindingXMLRPC.py
3 files changed, 5 insertions(+), 7 deletions(-)

Approvals:
  Piotr Kliczewski: Looks good to me, but someone else must approve
  Yeela Kaplan: Looks good to me, but someone else must approve
  mooli tayer: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Verified



-- 
To view, visit http://gerrit.ovirt.org/35271
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I120754b2c934da3393e5ea4323225c14e25b2087
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com>
Gerrit-Reviewer: Yeela Kaplan <ykap...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: mooli tayer <mta...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to