fix #4643: show a confirmation dialog when clicking abort

Note that unlike the rest of the file, we connect to the response signal
instead of using Gtk3::Dialog->run, the reason is that run blocks the
main loop used by GTK and this undesirable to the point where
Gtk3::Dialog->run was removed for GTK 4.

Signed-off-by: Maximiliano Sandoval <m.sandoval@proxmox.com>

tui: refresh summary view if it is switched back to

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: make HVM extension not being present a non-fatal warning

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: assume that the installer is run from the repo root in debug mode

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: do not fail completely on invalid domain names

Since we already handly non-present domain names, invalid names can be
handled the same way, as not to completely fail the installation.

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: allow one-part FQDNs/domains

Otherwise, the user would be blocked from continuing the the installer,
instead just being presented with an error.

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

gtk: return truthy for rescheduling the timer for autoreboot

This definitively worked without that value when the functionality
was first added in dfc02f3 ("fix #3093: allow to automatically reboot
on installation success"), so possibly something new with the glib,
gtk, or the perl bindings in bookworm..

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

run env: make addresses optional, include all interfaces

Similar to our actual battle proven get_ip_config, but as the TUI
uses this one, and switching to it rather more risk, make just
addresses optional, add skip for "lo" loopback and don't skip ifaces
that aren't UP in the current helper.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

tui: slight FloatEditView improvement

with this you can type without removing the decimal places, but if
you type inside the decimal places it'll just move the cursor to the
end and do nothing

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

config: avoid warning if no country was detected

e.g., due to having no network (no DHCP lease)

Reported-by: Christian Ebner <c.ebner@proxmox.com>
Reported-by: Stefan Hanreich <s.hanreich@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

config: fix missing include of Log module

Reported-by: Stefan Hanreich <s.hanreich@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

tui: only focus summary table if it can be scrolled

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: set detected keyboard layout at installer launch

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: make auto-deteced routes optional

If, for some reason, the network is not properly configured (due to
e.g. no DHCP server being present on the network), there will be no
routes in the runtime environment as well. So do not depend on that,
otherwise the installer fails at the start.

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: skip serializing empty fields for install config

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: make disk swap-, max-root-, max-data- and min-lvm-free-size optional

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: allow numeric & disk size edit views to be empty

Bit of a retro-fit I guess, could maybe get improved in the future.

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: run rustfmt

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

low level: fix test-session progress

In a8fbe0ff, the protocol was changed; such that the TUI now multiplies
the value by 100. Thus divide it here by 100 before sending it off.

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: simplify test-mode evaluation

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: drop unneeded parameter from InstallerBackgroundView::new()

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: drop useless gitattributes file

Since the cargo lockfile was removed from the repo in 74e3a21, this
became useless.

Fixes: 74e3a21 ("tui: remove cargo lockfile")
Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

gitignore: merge TUI .gitignore into top-level

Since the TUI was put into a top-level workspace, the `target` folder
and lockfile exist too on that level.

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

bump version to 8.0.4

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

proxinstall: re-add Encode module

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 8.0.3

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

tui: avoid using domain as FQDN, hard-code product as workaround

Using the product is actually fine, but we should derive it from the
ISO env, not hard-coding "pve", but for now ok enough.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

gtk: fix initial default for FQDN

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

low level: drop extra exit

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

tui: fix passing disk selection, use index

Currently the installer depends on the index to derive the selected
disks, so use that instead of a wishing-how-it-could-be approach.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

install: fix passing missing root volume name

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add Finished UI² message

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

install module: move over ZFS and BTRFS device getters

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

tui: disk view: partition sizes are in GB

The proxinstall takes a GB float here and multiplies it up to KB for
most parts, which is then passed to various tools, passing bytes here
makes this go bonkers and try to create a 4 EiB swap partition.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

tui: lvm: crudely fix calculation mismatch

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

tui: use plain Proxmox ascii figlet in header

If, one would need to write the full product name, and as that is
already written in the "inner" dialogue header, lets just drop it
from the outer.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

low-level: report errors and implement autor-reboot

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

tui: add initial implementation of low-level installer transport

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: make yes_no_dialog() accept a callback for negative responses too

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

UI: turn on autoflush for STDOUT

Otherwise, when STDOUT is connected to a pipe, it will not get flushed
completely after each message, which might than hangs the installer UI.

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: add back disk sorting

This was accidentally removed with below commit.

Fixes: 4532cff ("run env: replace lsblk with hd_list but fix it up a bit")
Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: add "abort" button to installation progress screen

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: adapt install config serialization to proper format

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

proxinstall: further clean ups

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

low level: support actual installation

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

proxinstall: cleanup imports

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

sys command: add missing imports, switch to UI for event processing

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

factor out installation into own module

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

storage config: use strictures

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

move ipconf into run-env

This adds some duplication there, but much safer than restructuring
stuff at this point in time, we can deal with unification later.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

config: always parse kernel cmdline on initialization

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

move assembling target and base directory into extract_data

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

run env: use single source of truth for block devices

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

config: note that we want to remove mngmt_nic_id in the future

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

tui: include disk model in disk list

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

run env: replace lsblk with hd_list but fix it up a bit

Instead of lsblk we use the original `hd_list` to be closer
to what we originally had.
However, from the PVE codebase I copied over the use of the
`E: DEVNAME` property to find the `/dev` node like (since
some weird devices can have '!' in /sys with '/' in /dev),
drop the check for `/sys/block/$d/dev`, and provide both the
`/dev` and `/sys/block` path in the output array.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

d/rules: actually build in release mode

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

d/control: add missing libjson-perl dependency

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

low-level: add dummy install with prompt and message test

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

unconfigured: honor proxtui cmndline flag

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

uncofigured: prepare for TUI and reworked installer config handling

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

write out install config as json on ack step for debugging

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "target_hd" & "disk_selection" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

tui: use total memory info from runtime env

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: use network domain as default administrator email domain

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: fix parsing of FQDNs directly from runtime env info

The JSON deserializer always expects to return an owned string.

Fixes: 7762d12 ("tui: directly parse domain name from runtime env info")
Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: add better error handling to BootdiskOptions::get_values()

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: fix unbounded size of advanced bootdisk dialog

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: use country from runtime env info for default timezone and layout

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: use available network interfaces from runtime env info

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: directly parse gateway IP address from runtime env info

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: bootdisk: sort disks alphabetically

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: directly parse disks from runtime env info

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: directly parse domain name from runtime env info

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: use `IpAddr` type for deserializing ip addresses directly

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: directly parse network interface addresses from runtime env info

This then allows to skip 2 levels of if-lets later on.

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

tui: move installer logo into background and to the top

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

switch "zfs_opts" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "cidr", "gateway" & "dns" over to central config

allows to drop quite a few module wide variables for GUI widgets,
which haven been re-created everytime the step got visited anyway.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "hostname" & "domain" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "mngmt_nic" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "mailto" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "password" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "keymap" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "timezone" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "country" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "maxvz" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "minfree" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "maxroot" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "swapsize" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "hdsize" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "filesys" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch "autoreboot" over to central config

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add beginnings of central config module with singleton

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

run env: add kernel cmdline info

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

provide a global singleton for RunEnv

we should only use the exact same set of values everywhere any way
for this, a good hint that a singleton is a good fit and this is
global info.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

buildsys: fix clean target

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

factor out storage config generation and handling

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

iso env: add singleton getter

this is basically global state, so don't bend overt backwards in
passing it around, just access it wherever..

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

move setting global environment variables into ISO env

could be also done in RunEnv, as one could argue both, this is
set at runtime, and this is for the ISO installer environment.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>