Shahar Havivi has posted comments on this change.

Change subject: v2v: use new jobs infrastructure
......................................................................


Patch Set 3:

(2 comments)

https://gerrit.ovirt.org/#/c/52858/3/lib/vdsm/v2v.py
File lib/vdsm/v2v.py:

Line 80
Line 81
Line 82
Line 83
Line 84
> This does not comply with the schema of getJobs - there are no such statuse
I will remove the status class and will work with the default jobs 
infrastructure


Line 82
Line 83
Line 84
Line 85
Line 86
> This is now "failed", so you will have to change the engine side to look fo
?
its the removed entries ie we don't use it


-- 
To view, visit https://gerrit.ovirt.org/52858
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iac71559e02502580de9c2e537733ec7286682050
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Shahar Havivi <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to