On 02/17/22 16:05, Richard W.M. Jones wrote:
On Thu, Feb 17, 2022 at 03:56:38PM +0100, Laszlo Ersek wrote:
> The functions "remote_file_exists" and
"scp_from_remote_to_temporary"
> [input/parse_domain_from_vmx.ml] rely on "ssh" and "scp" shell
commands,
> which intentionally only take a password from a tty. This means that for
> vmx+ssh, "-ip" is incomplete.
>
> Completing the feature would require a lot of work; we'd have to
> reimplement "remote_file_exists" and
"scp_from_remote_to_temporary" in C,
> using libssh (an sftp session). For "remote_file_exists", we'd have to
> replace "test -f" with sftp_lstat(). For
"scp_from_remote_to_temporary",
> we'd need to write an SFTP download loop. vmx+ssh is too niche for this,
> so let's just document the limitation.
>
> Extends: 784be60842d088596d7af938f90c689083677dca
> Bugzilla:
https://bugzilla.redhat.com/show_bug.cgi?id=1854275
> Signed-off-by: Laszlo Ersek <lersek(a)redhat.com>
> ---
> docs/virt-v2v-input-vmware.pod | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/docs/virt-v2v-input-vmware.pod b/docs/virt-v2v-input-vmware.pod
> index f13861339034..599a45cbfb63 100644
> --- a/docs/virt-v2v-input-vmware.pod
> +++ b/docs/virt-v2v-input-vmware.pod
> @@ -139,6 +139,11 @@ virt-v2v server to the ESXi hypervisor. For example:
> $ ssh root(a)esxi.example.com
> [ logs straight into the shell, no password is requested ]
>
> +Note that support for non-interactive authentication via the I<-ip>
> +option is incomplete. Some operations remain that still require the
> +user to enter the password manually. Therefore ssh-agent is recommended
> +over the I<-ip> option.
> +
Yes this is fine as it is, but you might consider adding:
See
L<https://bugzilla.redhat.com/1854275>
ACK
Merged as commit 3172a509bb43, with the following updates:
1: 5339e3e534d7 ! 1: 3172a509bb43 docs: highlight that
"-ip" with vmx+ssh still requires user interaction
@@ -17,6 +17,9 @@
Extends: 784be60842d088596d7af938f90c689083677dca
Bugzilla:
https://bugzilla.redhat.com/show_bug.cgi?id=1854275
Signed-off-by: Laszlo Ersek <lersek(a)redhat.com>
+ Message-Id: <20220217145638.9324-1-lersek(a)redhat.com>
+ [lersek(a)redhat.com: add docs link to RHBZ#1854275 (Rich)]
+ Acked-by: Richard W.M. Jones <rjones(a)redhat.com>
diff --git a/docs/virt-v2v-input-vmware.pod b/docs/virt-v2v-input-vmware.pod
--- a/docs/virt-v2v-input-vmware.pod
@@ -28,7 +31,7 @@
+Note that support for non-interactive authentication via the I<-ip>
+option is incomplete. Some operations remain that still require the
+user to enter the password manually. Therefore ssh-agent is recommended
-+over the I<-ip> option.
++over the I<-ip> option. See
L<https://bugzilla.redhat.com/1854275>.
+
=head3 VMX: Construct the SSH URI
Thanks,
Laszlo