[node-patches] Change in ovirt-node[master]: split proxy server:port for rhn conf

jboggs at redhat.com jboggs at redhat.com
Thu Aug 30 16:47:32 UTC 2012


Joey Boggs has posted comments on this change.

Change subject: split proxy server:port for rhn conf
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File scripts/rhn.py
Line 499:             p_server, p_port = self.rv("httpProxy").split(":")
Line 500:             self.proxyhost.set(p_server)
Line 501:             self.proxyport.set(p_port)
Line 502:         except:
Line 503:             pass
Not sure how I didn't catch those line being removed... Will add those back in.

For a proxy there has to be a port specified so it would split properly
Line 504:         self.rhn_actkey = Entry(40, "")
Line 505:         if rhn_check():
Line 506:             if self.rhn_url.value() == ("https://xmlrpc.rhn.redhat.com/XMLRPC"
Line 507:                                         or len(self.rhn_url.value()) == 0):


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id826af6b75f6348254e6b4910351ae8fb625d38f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Joey Boggs <jboggs at redhat.com>
Gerrit-Reviewer: Joey Boggs <jboggs at redhat.com>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>



More information about the node-patches mailing list