Sandro Bonazzola has uploaded a new change for review. Change subject: upload: remove nossl option ......................................................................
upload: remove nossl option Rest APIs doesn't support plain http connection so nossl doesn't make sense. Removing it. Change-Id: I04934a3a2c7b004c94692d5b163a73746c29d38c Bug-Url: https://bugzilla.redhat.com/1100825 Signed-off-by: Simone Tiraboschi <[email protected]> (cherry picked from commit 2619a581f6c0e00688a1d49618f28b53da83b884) --- M src/__main__.py 1 file changed, 1 insertion(+), 9 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-iso-uploader refs/changes/16/29716/1 diff --git a/src/__main__.py b/src/__main__.py index f17d609..1399293 100644 --- a/src/__main__.py +++ b/src/__main__.py @@ -487,8 +487,7 @@ "the oVirt Engine REST API." ) - url = ("http://"; if self.configuration.get("nossl") - else "https://";) + self.configuration.get("engine") + "/api" + url = "https://"; + self.configuration.get("engine") + "/api" try: # If "insecure" option was provided, use it during API creation @@ -1296,13 +1295,6 @@ parser.add_option( "", "--insecure", dest="insecure", help="Do not make an attempt to verify the engine.", - action="store_true", - default=False - ) - - parser.add_option( - "", "--nossl", dest="nossl", - help="Do not use ssl to connect to the engine.", action="store_true", default=False ) -- To view, visit http://gerrit.ovirt.org/29716 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I04934a3a2c7b004c94692d5b163a73746c29d38c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: ovirt-iso-uploader-3.5 Gerrit-Owner: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Simone Tiraboschi <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
