partially close #438: vzdump: support setting notes-template

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: restore: fix syntax error introduced in previous commit

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: restore overrides: switch from column to hbox+panel layout

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

ui: restore: display "Hostname" for container

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: restore: group override settings in a fieldset

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: restore: allow override of some settings

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: restore: disallow empty storage selection if it wouldn't work

Namely, if there is a storage in the backup configuration that's not
available on the current node.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui; datacenter options: add next-id editor

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

fix replication fail count tests & bump libpve-guest-common-perl b-d

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

api2: network: anybridge: re-add regular bridges

commit 89d146f207225bb8ca2e01d7e79000bb37a227d1 introduced permission
checks here that caused all regular bridges to be removed from the
returned list as soon as the SDN package is installed, unless the user
is root@pam or there exists a VNET with the same ID.

this is arguably a breaking change, so limit the priv check to actually
defined VNETs for the time being, and add ALL regular bridges
uncondtionally like before.

get_local_vnets already filters by the same prvs, so we need to get the
full config to find out which IDs are VNETs and which are not.

once/iff we introduce ACL paths for *all* bridges in the future, we can
limit accordingly here.

CC: Alexandre Derumier <aderumier@odiso.com>
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

api: next-id: honor new datacenter.cfg upper/lower range option

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

vzdump: support setting protected status

Check the number of protected backups early if the protected flag
is set.

Suggested-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

fix #3955: vzdump: new: also send mail for error during job-init hook

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

vzdump: new: add add_error helper

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: sdn: zone evpn: delete exitnodes-primary if empty

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

replication: split out error handling and include more info in mail

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

replication: sent always mail for first three tries and move helper

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

api: replication: don't send mails about failed replication only once

but rather multiple times becoming exponentially less frequent.

Suggested-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

d/control: bump proxmox-widget-toolkit dependency to 3.4-9

to have the moved markdown notes component available, among other
things

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

ui: dont show '__default__' in renderer

with the recent rework of the render/maps/arrays, we now
show 'Default (__default__)'. Since '__default__' is only an internal
value in the gui, don't expose it by explicitely checking for it

in the other render functions it works already because we either construct
the text differently (console_map) or we check the result from
parsing (vga_driver)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

ui: realm sync: use fieldset for remove-vanished & ux/wording

Use fieldsets, which are just way nicer for grouping these things.

Disable the "remove properties" checkbox if we'd remove the whole
user anyway.

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

ui: realm sync: replace 'full' and 'purge' with 'remove-vanished'

in default sync options and the sync window. We do this by exposing
the individual flags as checkboxes. We get the mapped value from the
backend so we do not have to handle 'old' values here.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

d/control: drop unused liblocale-po-perl build dependency

Became unused with commit 2bb03197d3f7c33d19f0979eb08e76d5814be162
that moved the locale stuff to its own repo proxmox-i18n

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

ui: utils: drop some useles _array helpers, just use Object.entries

if we need to enforce ordering it may help to reintroduce them
though.

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

ui: utils: rework kvm-vga/keymaps and console-viewer schema handling

shave off some bloat

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

ui: Utils: remove unnecessary render_kvm_vga_driver

the 'value' here is already the rendered text, passing it through
render again does not make sense

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

ceph: services: broadcast versions: improve requesting old version

to avoid a "malformed JSON string" warning when there is no old
version present (e.g. after starting a cluster).

get_node_kv will always return something that evaluates to true, so
instead, test if the result has an entry for the current node. Also,
it's enough to request the kv for the current node only.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: fix adding hostpci > 5

by using the limits from PVE.Utils

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

ui: vm display: refactor and cleanup

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

ui: backup restore: add hint to window title on overwrite restore

we prompt for confirmation in that case anyway, but some subtle, but
extra difference may still help

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

ui: backup restore: show "make unique" field also for self-overrride-existing

For example, relevant if the filterVMID checkbox got turned off. But
a user may also want to make the restored VM unique in other cases
too.

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

ui: backup restore: use displayEdit field for guest selector

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

ui: backup restore: rework to leverage more viewcontroller and cleanup

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

ui: disk storage selector: support storage with multiple formats without qcow2

For future or external storages where the assumption "multiple formats implies
qcow2 is supported" doesn't hold.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: disk storage selector: never send format when hideFormat is true

The backend will pick an appropriate format when nothing is specified. The
comment made it sound like 'raw' would be sent, but that didn't actually happen
on file-based storages, and now no format is sent, so adapt the comment too.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: lxc resources: modernize & simplify

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

ui: move NotesView panel and NotesEdit window to widget kit

this removes the NotesView panel and NotesEdit and replaces them with
with the version from the widget kit. requires a bump of the widget
toolkit.

Signed-off-by: Stefan Sterz <s.sterz@proxmox.com>

ui: disk reasign: condense code, some style improvements

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

ui: disk reassign: improve title and field-label

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

ui: disk reassign: drop useless intermediate form

The form is basically a 1:1 copy from what the edit window gives us
for free already anyway, so just drop it.

best viewed with `git show -w`

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

ui: disk reassign: improve default focus behavior

avoid that opening the edit window makes the guest-selectors combobox
"jump" in ones face.

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

ui: move disk reassign into window/ folder

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Fabian Ebner <f.ebner@proxmox.com>

ui: hdmove: modernize/refactor

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
Reviewed-by: Fabian Ebner <f.ebner@proxmox.com>

ui: BusTypeSelector: change noVirtIO to withVirtIO

Double negated properties make it harder than necessary to parse
conditions.

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
Reviewed-by: Fabian Ebner <f.ebner@proxmox.com>

ui: lxc/qemu: add disk reassign and action submenu

For the new HDReassign component, we follow the approach of HDMove to
have one componend for qemu and lxc.

To avoid button clutter, a new "Disk/Volume action" button is
introduced. It holds the Move, Reassign and Resize buttons in a
sub-menu.

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-by: Fabian Ebner <f.ebner@proxmox.com>

ui: utils: add nextFreeMP

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
Reviewed-by: Fabian Ebner <f.ebner@proxmox.com>

ui: utils: refactor mps to mp

Using the actual config key instead of the pluralization, makes it
easier in the situations where we need to match against it.

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
Reviewed-by: Fabian Ebner <f.ebner@proxmox.com>

ui: lxc: resources: consider rootfs as a disk again

Commit 809f6b6e ("ui: lxc resources: switch to vector based font
awesome icons") seems like an innocent change, but it broke the
(very brittle) logic here by removing the tdCls for rootfs.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: storage edit: retention: add max-protected-backups setting

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

vzdump: always pass full list to prune marking

else this single call site is subtly different from all the rest, which
could cause problems further down the line if we ever change the prune
logic.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

vzdump: backup limit: only count unprotected backups

since they are the ones relevant for pruning and protected backups
have their own separate limit.

Since get_backup_file_list is only used in places where the
unprotected backups are needed, adapt the helper accordingly.

If there is a storage, use PVE::Storage::volume_list to count the
unprotected backups. This avoids a direct invocation of the
proxmox-backup-client for PBS and the limit check can also work for
external storage plugins which might not be dir-based or name the
backups differently.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

vzdump: backup file list: drop unused parameter

It's not used anymore since 77c7945123724564ea909439ea51b537a5d0b6c2
aligned pruning in a dumpdir with pruning on a storage.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

fix #3976: api/backup: make schedule/starttime truly optional on update

on create we require either starttime (+dow) or a schedule, but when
updating an existing job, this is not necessary

before we changed to schedules, the starttime was not optional either on
update, but i think there is no reason to require the user to send the
schedule/startime along every time.

the gui will send all values every time, so that was never a problem there

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

ui: render storage type: improve fallback

We support external storage plugins, e.g., for proprietary
technology, so a "unknown" value here may not actually mean that we
missed anything, so just return the type value 1:1 as fallback, that
gives the user a better idea about such a storage entry.

Reported-by: Joshua Huber <jhuber@blockbridge.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

Jobs: don't schedule jobs with no computable next event

if we have a schedule that has no 'next event' we should skip the scheduling
instead of schedule every round

this can happen if someone sets an schedule that has no next match.
some examples:
* 2-31 00:00 (there is not February 31st)
* mon 2022-04-02 (this would be a saturday, not monday)
* 1970-1-1 (or every other exact date in the past)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

ui: vm hardware: module wide refactoring/cleanups/style changes/fixes

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

ui: vm hardware: fix add-menu itemId casing

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

ui: vm: factor out add-HW menu handler to higher order fn

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

ui: lxc resources: refactor add menu handler

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

ui: lxc resources: switch to vector based font awesome icons

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

ui: lxc resources: support font awesome icons directly

like we do for the VMs hardware view

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

ui: icons: swap pci bitmap to svg

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

api: vzdump: extract config: add content type check

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

pveam: remove: add content type check

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

api: vzdump: extract config: check for VM.Backup privilege

In preparation to have check_volume_access() always allow access for
users with Datastore.Allocate privilege. As to not automatically give
all such users permission to extract the config too.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

ui: css: set correct font awesome grid font-size

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

ui: qemu: switch CD drive icon to SVG

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

ui: qemu: switch memory icon to SVG

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

ui: cpu icon: enable crispEdges and increase stroke width slightly

crisp edges avoids antialiasing, which is rather counterproductive on
this icon with straight lines that are in 90° to each other.

The higher stroke width makes it more similar to the previously used
bitmap one.

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

ui: qemu: switch processor icon to SVG

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

css: fix parenthesis opening style

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

report: lsblk: merge/group parents of sub-trees

can cut down lines outputted, especially on lvm-thin or other systems
that have volumes listed duplicated. E.g.,

Note, if nothing can be merged the output is 1:1 the same as before,
so seems like a win-win to me.

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

report: lsblk: output more columns

Output columns that can help on debugging (e.g., physical sector
size, hot-pluggable, rotational, transport) or make it easier to
cross-correlate (model, fs-type)

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

report: report FS type in df call

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

ui: sdn: options: flex all and drop collapsible

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

bump version to 7.1-12

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

ui: sdn: fix options panel headings case

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

apl info: update index to current repository state

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

ui: sdn : controllers: bgp : add bgp-multipath-as-path-relax

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

ui: sdn: zones: evpn : add rt-import

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

ui: sdn: zones: evpn : move controller option on top of the form

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

ui: sdn: zones: evpn : add exitnodes-primary

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

ui: sdn: zones: evpn : add disable-arp-nd-suppression

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

apl info: trailing whitespace cleanups

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

api: cluster: refactor and cleanup

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

d/control: add missing libproxmox-rs-perl dependency

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

api: fill in nodeconfig schema

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

nodeconfig: use common config parser

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

d/control: bump versioned dependency for pve-common

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

bump version to 7.1-11

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

switch to Proxmox::RS::CalendarEvent

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

api: sdn: code/whitespace refactor/fixes

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

api2 : network: anybridge: don't display bridges if user have access to vnets.

This remove vmbr* from bridgeselector if user have access to vnets.
if user need to have also access to vmbr, we can add a permission
in path "/sdn/vnets/vmbrX"

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

permpathstore: add sdn zones

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

ui: sdn: zones: evpn : add exitnodes-local-routing && advertise-subnets

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

ui: lxc options: disable features edit as non-root when container is privileged

The backend won't allow any edits in this case, so better just disable
the edit button altogether.

Co-developed-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

gui: add help button to qm guest agent window

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>

ui: fw grid: eslint auto-fixes

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

ui: firwall: change icmp type selector to a combogrid

The combogrid contains all valid icmp types that iptables accepts. In
addition to the names, the Type[/Code] value is shown as well. But
specifying Type[/Code] is not supported.

As the simple solution with setStore() does not work to change the
store for the combogrid, we simply have 2 different
ICMPTypeSelectors, one for IPv4, the other for IPv6, which are
switched depending on the specified protocol.

As disabled fields don't submit their value, we can keep the values
for the dport and both icmp-type fields when switching protocols.

Signed-off-by: Mira Limbeck <m.limbeck@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

daily update timer: start already on 01:00 to avoid dst change issue

see https://github.com/systemd/systemd/issues/5595

Has also the side effect of spreading out the updates for our most
used product over a longer time span, reducing load on the CDN.

Note, this is now the same timer setting as PBS used from the start.

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