Change in ovirt-engine-sdk[sdk_4.0]: Convert 'all_content' parameter to 'All-Content' header

omachace at redhat.com omachace at redhat.com
Thu Oct 20 07:01:00 UTC 2016


Ondra Machacek has submitted this change and it was merged.

Change subject: Convert 'all_content' parameter to 'All-Content' header
......................................................................


Convert 'all_content' parameter to 'All-Content' header

Older versions of the engine (before 4.1) only accepted the
'all_content' parameter as an HTTP header. To better support those
versions of the engine this patch changes the SDK so that when the
'all_content' parameter is specified its value is copied to the
'All-Content' request header. So a request like this:

  GET /ovirt-engine/api/vms/123?all_content=true

Will be translated into this:

  GET /ovirt-engine/api/vms/123?all_content=true
  All-Content: true

Change-Id: I5f4be9028d50a1571a938956d960d7c299400515
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
Signed-off-by: Ondra Machacek <omachace at redhat.com>
---
M sdk/lib/ovirtsdk4/__init__.py
1 file changed, 10 insertions(+), 0 deletions(-)

Approvals:
  Ondra Machacek: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5f4be9028d50a1571a938956d960d7c299400515
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-sdk
Gerrit-Branch: sdk_4.0
Gerrit-Owner: Ondra Machacek <omachace at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Ondra Machacek <omachace at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list