lp://staging/~niedbalski/uvtool/fixes-lp-1428674

Branch merges

Propose for merging

No branches dependent on this one.

Rejected for merging into lp://staging/~uvtool-dev/uvtool/trunk

Robie Basak: Needs Fixing on 2015-03-16

uvtool development: Pending requested 2015-03-05

Diff: 36 lines (+16/-5)

1 file modified

uvtool/libvirt/__init__.py (+16/-5)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1428674: uvt-kvm: error: no IP address found for libvirt machine

High

Fix Released

Link a bug report

Related blueprints

94. By Jorge Niedbalski on 2015-03-05

Fixed file size

93. By Jorge Niedbalski on 2015-03-05

[niedbalski, r=] Fixes bugs LP #1428674 and LP #1420142
 - Since libvirt v1.2.11 though, libvirt adds --leasefile-ro which stops it
   updating the dnsmasq lease file and makes libvirt store the leases itself.
 - This patch uses /proc/net/arp as a fallback in case no leasefile exist.

92. By Robie Basak on 2014-04-01

Correctly handle wait with custom ssh keys

When "uvt-kvm create" is called with --ssh-public-key-file, "uvt-kvm
wait" is no longer able to ssh into the guest by default, because it
can no longer authenticate. This breaks "wait" in this case, when it
would still otherwise be useful to abstract away the remote wait case.

Add options to accomodate these cases:

--without-ssh to explicitly skip the remote wait case, in the case that
the user doesn't want a remote wait, or will arrange the remote wait
himself.

--ssh-private-key-file to specify a private key, in the case that the
user does want a remote wait and can supply a key that can be used.

LP: #1287140

91. By Robie Basak on 2014-04-01

Fix manpage to reflect correct "create" default

"uvt-kvm create <name>" defaults to using a filter or the current LTS
release, but does not currently default to the host architecture.

90. By Robie Basak on 2014-03-12

Add manpages

89. By Robie Basak on 2014-03-12

Correctly print missing key path when not found

When I refactored the ssh public key file discovery handling, I broke
the warning message generated when the file is not found, since the
actual filename used was no longer available at the time of printing the
message.

Return the actual path that was not found, so that the warning message
can notify the user of this path.

88. By Robie Basak on 2014-02-19

Add copyright and licence notice

87. By Robie Basak on 2014-02-19

Add uvtool.libvirt.kvm to packaging

86. By Robie Basak on 2014-02-19

Drop experimental CLI warnings

I now consider the CLI provisionally stable.

85. By Robie Basak on 2014-02-19

Drop "uvt-kvm import"

This is a relic that was useful prior to simplestreams support. I don't
intend to support this subcommand as part of a stable API, since I don't
think anybody will need it in the common case. In particular, the uvtool
libvirt volume storage pool namespace belongs to uvtool, so shouldn't be
messed with by users directly, to prevent future features causing later
collisions. Therefore, I'm dropping this subcommand prior to declaring
the CLI provisionally stable and documenting the remaining subcommands.

We can reimplement it if necessary though. In the meantime, see
http://askubuntu.com/q/299570/7808 for the shell equivalent using virsh
instead.