[Jobs] invalid parameters (old jobs)

[cocoon]

Hi again,

I think there is a new problem with old jobs, a tleast for "vm.revert".
(build from sources from Apr 9, 2020 )

It looks like there was added a new parameter "timeout" and "snapshotbefore" was removed.

I now tried to edit my old jobs to add the timeout value, but it seems not to remove the old parameter when I save it, and so the old jobs don't work.

I have to create all from scratch now, no big problem for me, but might be better if it would remove invalid parameters when saving or validate if this is the problem.

Cheers!

Update: Might be related:
https://github.com/vatesfr/xen-orchestra/commit/af4904ce8d9937b24654ef83ba138e2d9aaf86e4

0 pdonias committed to vatesfr/xen-orchestra

chore(xo-web,xo-server): move "snapshot before" feature to the client (#4893)

From xo-server's API's point of view, it makes more sense for the "snapshot before" feature to be implemented in xo-web since snapshotting a VM is a whole independent process on its own that requires its own API permissions.

Making this change will be especially useful to not over-complicate the permissions checks for the incoming "Self Service snapshots" feature.

See #3693