search for: virtualnetworkinterfac

...884, in get_networks > dev = parse_network(guest, netstr, mac=mac) > File "/usr/local/lib/python2.6/dist-packages/virtinst/cli.py", line > 1642, in parse_network > bridge=opts.get("bridge")) > File "/usr/local/lib/python2.6/dist-packages/virtinst/VirtualNetworkInterface.py", > line 153, in __init__ > self.network = network > File "/usr/local/lib/python2.6/dist-packages/virtinst/XMLBuilderDomain.py", > line 309, in new_setter > fset(self, val, *args, **kwargs) > File "/usr/local/lib/python2.6/dist-packages/virtinst...

..._default_url() logging.info("Connecting to libvirt: %s" % url) + self.__url = None + self.__conn = None self.open_connection(url) self.__capabilities = virtinst.CapabilitiesParser.parse(self.__conn.getCapabilities()) self.__net = virtinst.VirtualNetworkInterface(conn = self.__conn) @@ -84,9 +86,21 @@ class LibvirtWorker: '''Returns the underlying connection.''' return self.__conn + def get_url(self): + return self.__url + def open_connection(self, url): '''Lets the user change...

This version of the patch includes feedback from jboggs at redhat.com, including fixes to the BuildRequires and Requires in the spec file.

This patch obsoletes any previous ones. This is an upstream candidate, so I'm looking for feedback so we can push this and start using it.

This version fixes a few bugs found by jboggs. It also includes some logging facilities that need to be fleshed out more.

This patch is ready for pushing upstream.

...__(self, url = None): + if url is None: url = get_default_url() + logging.info("Connecting to libvirt: %s" % url) + self.open_connection(url) self.__capabilities = virtinst.CapabilitiesParser.parse(self.__conn.getCapabilities()) self.__net = virtinst.VirtualNetworkInterface(conn = self.__conn) self.__net.setup(self.__conn) (self.__new_guest, self.__new_domain) = virtinst.CapabilitiesParser.guest_lookup(conn = self.__conn) + def open_connection(self, url): + '''Lets the user change the url for the connection.''' +...

The previous patch was missing this file. Resending.

This set of patches have been rebased since the originals have grown stale.

This patch set includes both the host configuration patch and the node migration patch which depends on it. And the latter has a fix for missing files.

...__(self, url = None): + if url is None: url = get_default_url() + logging.info("Connecting to libvirt: %s" % url) + self.open_connection(url) self.__capabilities = virtinst.CapabilitiesParser.parse(self.__conn.getCapabilities()) self.__net = virtinst.VirtualNetworkInterface(conn = self.__conn) self.__net.setup(self.__conn) (self.__new_guest, self.__new_domain) = virtinst.CapabilitiesParser.guest_lookup(conn = self.__conn) + def open_connection(self, url): + '''Lets the user change the url for the connection.''' +...

The previous patch set had an error and one development branch's changes showed up twice, and two different change sets. So it was missing the bulk of the configuration work. This patch set fixes that.

...just looking for some feedback on the direction I'm going. The code won't get all the way to the define stage since I'm in the middle of retrofitting it to use virtinst instead of a home spun node definition.