Nir Soffer has submitted this change and it was merged.

Change subject: lib: response: helper to detect valid responses

lib: response: helper to detect valid responses

We should build responses in the API layer, and the responses
should be built from what the application layer gives back,
be that return values, or exceptions

We need a gradual upgrade path due to huge amount of legacy code,
so we need to handle the response objects built in the
application layer. For the short/mid-term, the API layer
needs to cope with both flows, modern and old.

So we need an utility to detect which kind the response
the application layer returns to the API layer.

This patch adds this utility, which will be erased once
the transition is complete, with the other scaffolding.

Change-Id: Ifab6a25bea1b4a187d8425275e86bdb2fecf4c7d
Signed-off-by: Francesco Romani <>
Continuous-Integration: Jenkins CI
Reviewed-by: Piotr Kliczewski <>
M lib/vdsm/common/
M tests/
2 files changed, 46 insertions(+), 0 deletions(-)

  Piotr Kliczewski: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Francesco Romani: Verified

To view, visit
To unsubscribe, visit

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifab6a25bea1b4a187d8425275e86bdb2fecf4c7d
Gerrit-PatchSet: 15
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <>
Gerrit-Reviewer: Francesco Romani <>
Gerrit-Reviewer: Irit Goihman <>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Milan Zamazal <>
Gerrit-Reviewer: Nir Soffer <>
Gerrit-Reviewer: Piotr Kliczewski <>
Gerrit-Reviewer: gerrit-hooks <>
vdsm-patches mailing list --
To unsubscribe send an email to

Reply via email to