Re: [PATCH] tests/vm: update openbsd to release 7.0

[Brad Smith]

On 10/18/2021 2:28 AM, Thomas Huth wrote:
> On 17/10/2021 00.56, Brad Smith wrote:
> > tests/vm: update openbsd to release 7.0
> >
> > Signed-off-by: Brad Smith <brad@comstyle.com>
> > ---
> >   tests/vm/openbsd | 4 ++--
> >   1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/tests/vm/openbsd b/tests/vm/openbsd
> > index c4c78a80f1..abf510e117 100755
> > --- a/tests/vm/openbsd
> > +++ b/tests/vm/openbsd
> > @@ -22,8 +22,8 @@ class OpenBSDVM(basevm.BaseVM):
> >       name = "openbsd"
> >       arch = "x86_64"
> >   -    link = 
> > "https://cdn.openbsd.org/pub/OpenBSD/6.9/amd64/install69.iso";
> > -    csum = 
> > "140d26548aec680e34bb5f82295414228e7f61e4f5e7951af066014fda2d6e43"
> > +    link = 
> > "https://cdn.openbsd.org/pub/OpenBSD/7.0/amd64/install70.iso";
> > +    csum = 
> > "1882f9a23c9800e5dba3dbd2cf0126f552605c915433ef4c5bb672610a4ca3a4"
> >       size = "20G"
> >       pkgs = [
> >           # tools
>
> That does not work for me. It hangs in the "DNS domain ..." line
> until it times out. When I disable that line with:
>
> diff --git a/tests/vm/openbsd b/tests/vm/openbsd
> index abf510e117..dfefb6bba3 100755
> --- a/tests/vm/openbsd
> +++ b/tests/vm/openbsd
> @@ -98,7 +100,7 @@ class OpenBSDVM(basevm.BaseVM):
>          self.console_wait_send("IPv4 address", "dhcp\n")
>          self.console_wait_send("IPv6 address", "none\n")
>          self.console_wait_send("Which network interface", "done\n")
> -        self.console_wait_send("DNS domain name", "localnet\n")
> +        #self.console_wait_send("DNS domain name", "localnet\n")
>          self.console_wait("Password for root account")
>          self.console_send("%s\n" % self._config["root_pass"])
>          self.console_wait("Password for root account")
>
> ... then it succeeds the installation... could you please check
> with a fresh install on your side whether you could reproduce
> this problem?

I was actually looking at that particular line when looking at the other 
diff
Richard sent out.

With the installer sub-routine the question is only asked under a set of
conditions and since there is only one physical interface as far as I know
of and using DHCP then that question is never asked. I don't see it when
I was verifying the steps.

    # Get & apply fqdn to hostname. Don't ask if there's only one 
configured
    # interface and if it's managed by dhclient and if the domain name is
    # configured via dhclient too.
    resp="${_dn:-$(get_fqdn)}"
    if ifconfig dhcp >/dev/null 2>&1 && [[ $NIFS == 1 && -z $_dn ]]; then
        # If we have a 'domain-name' option in the lease file use that.
        # It might *NOT* not be the same as the first domain in any
        # 'domain-search' option.
        set -- $(get_ifs dhcp)
        set -- $(lease_value /var/db/dhcpleased/$1 domain-name)
        [[ -n $1 ]] && resp=$1
        echo "Using DNS domainname $resp"
    else
        ask "DNS domain name? (e.g. 'example.com')" "$resp"
    fi
    hostname "$(hostname -s).$resp"

>  Thanks,
>   Thomas