Change in vdsm[master]: BZ#842771 Fix for exception TypeError: cannot marshal None

2012-08-13 Thread danken
Dan Kenigsberg has submitted this change and it was merged.

Change subject: BZ#842771 Fix for exception TypeError: cannot marshal None
..


BZ#842771 Fix for exception TypeError: cannot marshal None

The VM's mirror and mirrorPath properties had a value of None
rather than an empty string.

Change-Id: I873eff7541016609d81eea0a1b16613f214fcb70
Signed-off-by: Gal Hammer gham...@redhat.com
---
M vdsm/libvirtvm.py
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Gal Hammer: Verified; Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Peter V. Saveliev: Looks good to me, but someone else must approve
  Igor Lvovsky: Looks good to me, but someone else must approve


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I873eff7541016609d81eea0a1b16613f214fcb70
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Gal Hammer gham...@redhat.com
Gerrit-Reviewer: Barak Azulay bazu...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Gal Hammer gham...@redhat.com
Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Michal Skrivanek michal.skriva...@redhat.com
Gerrit-Reviewer: Peter V. Saveliev p...@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


Change in vdsm[master]: BZ#842771 Fix for exception TypeError: cannot marshal None

2012-08-12 Thread danken
Dan Kenigsberg has posted comments on this change.

Change subject: BZ#842771 Fix for exception TypeError: cannot marshal None
..


Patch Set 1: Looks good to me, but someone else must approve

Federico, this seems reasonable enough to me.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I873eff7541016609d81eea0a1b16613f214fcb70
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Gal Hammer gham...@redhat.com
Gerrit-Reviewer: Barak Azulay bazu...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Gal Hammer gham...@redhat.com
Gerrit-Reviewer: Michal Skrivanek michal.skriva...@redhat.com
Gerrit-Reviewer: Peter V. Saveliev p...@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


Change in vdsm[master]: BZ#842771 Fix for exception TypeError: cannot marshal None

2012-08-10 Thread peet
Peter V. Saveliev has posted comments on this change.

Change subject: BZ#842771 Fix for exception TypeError: cannot marshal None
..


Patch Set 1: Looks good to me, but someone else must approve

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I873eff7541016609d81eea0a1b16613f214fcb70
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Gal Hammer gham...@redhat.com
Gerrit-Reviewer: Barak Azulay bazu...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Gal Hammer gham...@redhat.com
Gerrit-Reviewer: Michal Skrivanek michal.skriva...@redhat.com
Gerrit-Reviewer: Peter V. Saveliev p...@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


Change in vdsm[master]: BZ#842771 Fix for exception TypeError: cannot marshal None

2012-08-08 Thread ghammer
Gal Hammer has posted comments on this change.

Change subject: BZ#842771 Fix for exception TypeError: cannot marshal None
..


Patch Set 1: Verified; Looks good to me, but someone else must approve

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I873eff7541016609d81eea0a1b16613f214fcb70
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Gal Hammer gham...@redhat.com
Gerrit-Reviewer: Barak Azulay bazu...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Gal Hammer gham...@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


Change in vdsm[master]: BZ#842771 Fix for exception TypeError: cannot marshal None

2012-08-08 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change.

Change subject: BZ#842771 Fix for exception TypeError: cannot marshal None
..


Patch Set 1: No score

Build Started http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/214/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I873eff7541016609d81eea0a1b16613f214fcb70
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Gal Hammer gham...@redhat.com
Gerrit-Reviewer: Barak Azulay bazu...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Gal Hammer gham...@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


Change in vdsm[master]: BZ#842771 Fix for exception TypeError: cannot marshal None

2012-08-08 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change.

Change subject: BZ#842771 Fix for exception TypeError: cannot marshal None
..


Patch Set 1:

Build Successful 

http://jenkins.ovirt.info/job/vdsm_unit_tests_by_patch/214/ : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I873eff7541016609d81eea0a1b16613f214fcb70
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Gal Hammer gham...@redhat.com
Gerrit-Reviewer: Barak Azulay bazu...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Federico Simoncelli fsimo...@redhat.com
Gerrit-Reviewer: Gal Hammer gham...@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


Change in vdsm[master]: BZ#842771 Fix for exception TypeError: cannot marshal None

2012-08-05 Thread ghammer
Gal Hammer has uploaded a new change for review.

Change subject: BZ#842771 Fix for exception TypeError: cannot marshal None
..

BZ#842771 Fix for exception TypeError: cannot marshal None

The VM's mirror and mirrorPath properties had a value of None
rather than an empty string.

Change-Id: I873eff7541016609d81eea0a1b16613f214fcb70
Signed-off-by: Gal Hammer gham...@redhat.com
---
M vdsm/libvirtvm.py
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/09/6909/1
--
To view, visit http://gerrit.ovirt.org/6909
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I873eff7541016609d81eea0a1b16613f214fcb70
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Gal Hammer gham...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches