Change in ovirt-engine-cli[master]: cli: Don't require /api

juan.hernandez at redhat.com juan.hernandez at redhat.com
Mon Feb 24 16:18:35 UTC 2014


Juan Hernandez has submitted this change and it was merged.

Change subject: cli: Don't require /api
......................................................................


cli: Don't require /api

Currently we perform an explicit validation of the URL of the API, and
this requires that the path component of the URL is exactly /api. This
doesn't work with the new URL schema of the engine, and won't work
either if the user decides use any other base URL. This patch removes
the validation of the path.

Change-Id: I71fac959498d17e1176ea16584e5b4d2c36ddbf4
Bug-Url: https://bugzilla.redhat.com/1069171
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M src/cli/messages.py
M src/ovirtcli/command/connect.py
M src/ovirtcli/infrastructure/options.py
3 files changed, 3 insertions(+), 4 deletions(-)

Approvals:
  Michael Pasternak: Looks good to me, approved
  Juan Hernandez: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I71fac959498d17e1176ea16584e5b4d2c36ddbf4
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine-cli
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Michael Pasternak <mishka8520 at yahoo.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>



More information about the Engine-commits mailing list