Am 21/11/2023 um 08:53 schrieb Fabian Grünbichler:
> these can happen for example if the system
> - is set up to boot using systemd-boot
> - but grub updates trigger a call to "grub-install"
> - and systemd-boot is not installed
>
> in this case, "proxmox-boot-tool reinit" will fail because of the
in the case we add a second cd drive (for windows), we don't want the
backend logic to only include the first one, since we cannot know
which is bootable and which is (probably) the virtio iso.
so instead, emulate the backend logic for the wizard but include both cd
drives in that case, otherwise
>
> > > For the code it would be nice to add a helper that asserts that
> > > frr
> > > is installed
> > > for all API endpoints that need it, that way user get informed
> > > upfront, similar
> > > to how we do it for ceph (but no full web installer for now ^^)
>
> Yes good idea.
>
> I don't k
SI units are used for storage size information, as typically used by
hard disk manufacturers.
Change the root filesystem usage values in the node summary to be
consistent.
Signed-off-by: Christian Ebner
---
www/manager6/node/StatusView.js | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
In the default NixOS configuration (e.g. when using prebuilt container
images from the Hydra pipeline [0]), getty is only configured for
/dev/console, but not any TTY ports. This results in the user just
getting a blank, black screen when opening the console in the web UI.
With this, the mentioned
On 11/21/23 10:00, Christian Ebner wrote:
> SI units are used for storage size information, as typically used by
> hard disk manufacturers.
>
> Change the root filesystem usage values in the node summary to be
> consistent.
>
> Signed-off-by: Christian Ebner
> ---
> www/manager6/node/StatusView
Am 21.11.23 um 10:00 schrieb Christian Ebner:
> diff --git a/www/manager6/node/StatusView.js b/www/manager6/node/StatusView.js
> index d34724f7..a6975b6e 100644
> --- a/www/manager6/node/StatusView.js
> +++ b/www/manager6/node/StatusView.js
> @@ -72,7 +72,9 @@ Ext.define('PVE.node.StatusView', {
>
Signed-off-by: Filip Schauer
---
Changes since v1:
* Remove usb mapping
* Add mode, uid and gid fields
To get this to build on top of the current master cd731902b7a7,
comment out the call to $notification_config->add_matcher
in PVE/VZDump.pm for now.
www/manager6/Makefile | 1 +
www/
This patch series adds the 'notification-mode' setting for backup jobs.
It allows users to choose between the 'old-style' notifications
(mail to configured address, directly via a call to sendmail) or
the 'new-style' notification system.
notification-mode has three possible values:
- legacy-se
This selector allows one to selected between the 'old' (send email
directly via sendmail) or the 'new' notification system.
The default is 'auto', which sends and email if one is configured,
and uses the notification system if no email address is set.
Signed-off-by: Lukas Wagner
---
www/manager
This parameter lets us choose between the 'legacy' notification
system (sendmail to some email addresses) and the 'new' notification
system (pub-sub based system with targets and matchers).
'auto' (default) will use the 'legacy' system if a mail address is
provided and the 'new' system if not.
This
This selector allows one to selected between the 'old' (send email
directly via sendmail) or the 'new' notification system.
The default is 'auto', which sends and email if one is configured,
and uses the notification system if no email address is set.
Signed-off-by: Lukas Wagner
---
www/manager
'legacy-sendmail': Use mailto/mailnotification parameters and send
emails directly.
'notification-system': Always notify via notification system
'auto': Notify via mail if mailto is set, otherwise use notification
system.
Signed-off-by: Lukas Wagner
---
src/PVE/VZDump/Common.pm | 14
This is actually v1, not v2. My bad.
--
- Lukas
___
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
Patch v2 available:
https://lists.proxmox.com/pipermail/pve-devel/2023-November/060583.html
On 30/10/2023 12:27, Filip Schauer wrote:
Signed-off-by: Filip Schauer
---
Depends on:
https://lists.proxmox.com/pipermail/pve-devel/2023-October/059616.html
www/manager6/Makefile | 1 +
Christoph Heiss (4):
tui: do not center EULA text
tui: preserve autoreboot checkbox state when switching views
tui: add missing argument for low-level installer test-session
common: enforce even number of disks for ZFS RAID-10
proxmox-installer-common/src/disk_checks.rs | 8 ++
proxm
This broke running the TUI installer in debug mode, does not effect
release builds in any way.
Fixes: 4b4dfa1 ("low level: testmode: take path to disk image instead of using
/dev/null")
Signed-off-by: Christoph Heiss
---
proxmox-installer-common/src/setup.rs | 2 +-
1 file changed, 1 insertion(
Brings it in line with the GUI installer.
Reported-by: Fiona Ebner
Signed-off-by: Christoph Heiss
---
proxmox-tui-installer/src/main.rs | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/proxmox-tui-installer/src/main.rs
b/proxmox-tui-installer/src/main.rs
index 4b6b5b2.
Instead of reading the checkbox when continuing to the next screen, save
its toggle status to the installer state instead on change.
Reported-by: Fiona Ebner
Signed-off-by: Christoph Heiss
---
proxmox-tui-installer/src/main.rs | 25 +
1 file changed, 13 insertions(+), 12
An uneven number of disks otherwise causes a panic due to an
out-of-bounds array access in the loop below.
Reported-by: Fiona Ebner
Signed-off-by: Christoph Heiss
---
proxmox-installer-common/src/disk_checks.rs | 8
1 file changed, 8 insertions(+)
diff --git a/proxmox-installer-common
Am 21/11/2023 um 09:54 schrieb DERUMIER, Alexandre:
>>
For the code it would be nice to add a helper that asserts that
frr
is installed
for all API endpoints that need it, that way user get informed
upfront, similar
to how we do it for ceph (but no full web installer f
Sent bit to quickly and missed the proper prefix; this fixes #5050 [0].
[0] https://bugzilla.proxmox.com/show_bug.cgi?id=5050
On Tue, Nov 21, 2023 at 11:45:51AM +0100, Christoph Heiss wrote:
>
> An uneven number of disks otherwise causes a panic due to an
> out-of-bounds array access in the loo
Am 21.11.23 um 11:22 schrieb Lukas Wagner:
> This patch series adds the 'notification-mode' setting for backup jobs.
> It allows users to choose between the 'old-style' notifications
> (mail to configured address, directly via a call to sendmail) or
> the 'new-style' notification system.
>
> not
> On 21.11.2023 11:05 CET Fiona Ebner wrote:
>
>
> Am 21.11.23 um 10:00 schrieb Christian Ebner:
> > diff --git a/www/manager6/node/StatusView.js
> > b/www/manager6/node/StatusView.js
> > index d34724f7..a6975b6e 100644
> > --- a/www/manager6/node/StatusView.js
> > +++ b/www/manager6/node/St
Matching if a serial will be needed for grub is based on the target
commandline - the speed is also read from there. The unit is based
on the ttyS device - although I'd assume that this might not always
match up.
Signed-off-by: Stoiko Ivanov
---
Proxmox/Install.pm | 8 +++-
1 file changed, 7
both issues are unrelated, and can be applied individually
first is that with the recent creation of /var/lib/vz as dedicated dataset,
systems yielded a failed service for var.mount upon shutdown. The fix
simply sets the canmount property for the intermediate dataset var and
var/lib to off - follo
as explained in zfsprops(4) setting canmount to off is similar to
setting mountpoint to none - except that you can still use the dataset
for storing properties to be inherited to children (and we want
/var/lib/vz to have a mountpoint set)
Follows recommendations from upstreams ZFS on / guide:
http
> On 21.11.2023 12:03 CET Christian Ebner wrote:
>
>
> > On 21.11.2023 11:05 CET Fiona Ebner wrote:
> >
> >
> > Am 21.11.23 um 10:00 schrieb Christian Ebner:
> > > diff --git a/www/manager6/node/StatusView.js
> > > b/www/manager6/node/StatusView.js
> > > index d34724f7..a6975b6e 100644
>
"DERUMIER, Alexandre" writes:
>>> * dnsmasq and IPv6 (and DHCP in general) do not really play well
>>>together,
>>> so using subnets with IPv6 configured is wonky
>
> I didn't have tested yet, but it's seem that dnsmasq only support old
> classic duid reservation and not mac ?
>
I tested the
SI units are used for storage size information, as typically used by
hard disk manufacturers.
Change the root filesystem usage values in the node summary to be
consistent.
Signed-off-by: Christian Ebner
---
Will also send a patch for PMG to be consistent across all of them.
changes since v1:
-
On 11/21/23 12:36, Christian Ebner wrote:
> SI units are used for storage size information, as typically used by
> hard disk manufacturers.
>
> Change the root filesystem usage values in the node summary to be
> consistent.
>
> Signed-off-by: Christian Ebner
> ---
>
> Will also send a patch
> Stoiko Ivanov hat am 21.11.2023 12:09 CET geschrieben:
>
>
> as explained in zfsprops(4) setting canmount to off is similar to
> setting mountpoint to none - except that you can still use the dataset
> for storing properties to be inherited to children (and we want
> /var/lib/vz to have a m
On 11/21/23 11:55, Fiona Ebner wrote:
Am 21.11.23 um 11:22 schrieb Lukas Wagner:
This patch series adds the 'notification-mode' setting for backup jobs.
It allows users to choose between the 'old-style' notifications
(mail to configured address, directly via a call to sendmail) or
the 'new-style
> On 16.11.2023 11:35 CET Philipp Hufnagl wrote:
>
>
> Currently, when adding a PBS storage with a namespace that does not
> exist, the storage gets added normally, but browsing/using it only
> returns a cryptic error message.
>
> This change checks if the namespace entered when adding is vali
Am 21/11/2023 um 12:09 schrieb Stoiko Ivanov:
> Matching if a serial will be needed for grub is based on the target
> commandline - the speed is also read from there. The unit is based
> on the ttyS device - although I'd assume that this might not always
> match up.
>
> Signed-off-by: Stoiko Ivano
Am 21/11/2023 um 11:45 schrieb Christoph Heiss:
> Christoph Heiss (4):
> tui: do not center EULA text
> tui: preserve autoreboot checkbox state when switching views
> tui: add missing argument for low-level installer test-session
> common: enforce even number of disks for ZFS RAID-10
>
>
This selector allows one to selected between the 'old' (send email
directly via sendmail) or the 'new' notification system.
The default is 'auto', which sends and email if one is configured,
and uses the notification system if no email address is set.
Signed-off-by: Lukas Wagner
---
www/manager
... to highlight that this setting only affects the 'legacy-sendmail'
mail notifications.
Signed-off-by: Lukas Wagner
---
Notes:
New in v3.
www/manager6/dc/Backup.js | 2 +-
www/manager6/form/NotificationPolicySelector.js | 2 +-
2 files changed, 2 insertions(+), 2 de
This selector allows one to selected between the 'old' (send email
directly via sendmail) or the 'new' notification system.
The default is 'auto', which sends and email if one is configured,
and uses the notification system if no email address is set.
Signed-off-by: Lukas Wagner
---
www/manager
This parameter lets us choose between the 'legacy' notification
system (sendmail to some email addresses) and the 'new' notification
system (pub-sub based system with targets and matchers).
'auto' (default) will use the 'legacy' system if a mail address is
provided and the 'new' system if not.
This
This patch series adds the 'notification-mode' setting for backup jobs.
It allows users to choose between the 'old-style' notifications
(mail to configured address, directly via a call to sendmail) or
the 'new-style' notification system.
notification-mode has three possible values:
- legacy-se
'legacy-sendmail': Use mailto/mailnotification parameters and send
emails directly.
'notification-system': Always notify via notification system
'auto': Notify via mail if mailto is set, otherwise use notification
system.
Signed-off-by: Lukas Wagner
---
src/PVE/VZDump/Common.pm | 14
The build in mail-to-root target can be disabled but not be deleted.
Is this on purpose?
___
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
Am 21/11/2023 um 11:22 schrieb Lukas Wagner:
> 'legacy-sendmail': Use mailto/mailnotification parameters and send
> emails directly.
> 'notification-system': Always notify via notification system
> 'auto': Notify via mail if mailto is set, otherwise use notification
> system.
>
> Signed-off-by: Lu
Am 21/11/2023 um 13:29 schrieb Philipp Hufnagl:
> The build in mail-to-root target can be disabled but not be deleted.
> Is this on purpose?
>
it's built-in after all (hardcoded in the code), so yes.
___
pve-devel mailing list
pve-devel@lists.proxmox
Am 21.11.23 um 13:13 schrieb Lukas Wagner:
> On 11/21/23 11:55, Fiona Ebner wrote:
>> Am 21.11.23 um 11:22 schrieb Lukas Wagner:
>>> This should provide a fix/workaround for the users' reports of
>>> - double notifications (these happened in case mailto was set to
>>> the same address
>>> a
Am 21/11/2023 um 13:13 schrieb Lukas Wagner:
> On 11/21/23 11:55, Fiona Ebner wrote:
>> Can't we just treat 'failure' mode as always defaulting to legacy
>> sendmail? And properly deprecate the setting, showing a warning/info
>> that new notification system is not used if set to 'failure' for both
Signed-off-by: Markus Frank
Reviewed-by: Dominik Csapak
Tested-by: Dominik Csapak
---
qm.adoc | 18 ++
1 file changed, 18 insertions(+)
diff --git a/qm.adoc b/qm.adoc
index 55a4728..cd0d907 100644
--- a/qm.adoc
+++ b/qm.adoc
@@ -817,6 +817,24 @@ Selecting `serialX` as display '
Due to the wrong check an IP was allocated for every DHCP range,
instead of only allocating an IP in the first free DHCP range.
Suggested-By: Stefan Lendl
Signed-off-by: Stefan Hanreich
---
The fix I talked about off-list was actually unrelated to this. It
prevented allocating IP addresses in mu
For SPICE and VNC, a different message is displayed.
Save config in DisplayEdit so that the clipboard setting persist.
Signed-off-by: Markus Frank
Reviewed-by: Dominik Csapak
Tested-by: Dominik Csapak
---
v15:
* changed style of line break in vncHint field
www/manager6/qemu/DisplayEdit.js |
On 11/21/23 13:38, Philipp Hufnagl wrote:
+ let notificationModeSelector = Ext.create({
+ xtype: 'proxmoxKVComboBox',
+ comboItems: [
+ ['auto', gettext('Auto')],
+ ['legacy-sendmail', gettext('Email (legacy)')],
+ ['notificat
Signed-off-by: Dominik Csapak
---
pve-gui.adoc | 1 +
1 file changed, 1 insertion(+)
diff --git a/pve-gui.adoc b/pve-gui.adoc
index bda370f..9e4650d 100644
--- a/pve-gui.adoc
+++ b/pve-gui.adoc
@@ -383,6 +383,7 @@ and the corresponding interfaces for each menu item on the
right.
* *Permissions
in the tag view, we have a custom 'full' style in a place where we
can have another tagstyle class above. to compensate for that, we have
to add another condition to those styles, namely that there is not the
'proxmox-tags-full' in between.
Signed-off-by: Dominik Csapak
---
src/css/ext6-pmx.css
this adds a 'tagview' to the web ui, organizing guests by their tags
(for details see the pve-manager patch)
changes from v1:
* rebase on master
* adapt to recent tooltip changes
* add a comment to TagConfig class to better explain what it does
pve-docs:
Dominik Csapak (1):
gui: add anchor for
and keep the functionality in ResourceTree as generic as possible.
We achieve this by having an 'itemMap' function that can split one item
from the store into multiple to add to the tree.
for the updates, we have to have an 'idMapFn' (to get the original id
back)
we also have to modify how the m
On 11/21/23 13:23, Lukas Wagner wrote:
> This selector allows one to selected between the 'old' (send email
> directly via sendmail) or the 'new' notification system.
>
> The default is 'auto', which sends and email if one is configured,
> and uses the notification system if no email address is
... to highlight that this setting only affects the 'legacy-sendmail'
mail notifications.
Signed-off-by: Lukas Wagner
---
Notes:
New in v3.
www/manager6/dc/Backup.js | 2 +-
www/manager6/form/NotificationPolicySelector.js | 2 +-
2 files changed, 2 insertions(+), 2 de
This parameter lets us choose between the 'legacy' notification
system (sendmail to some email addresses) and the 'new' notification
system (pub-sub based system with targets and matchers).
'auto' (default) will use the 'legacy' system if a mail address is
provided and the 'new' system if not.
This
This patch series adds the 'notification-mode' setting for backup jobs.
It allows users to choose between the 'old-style' notifications
(mail to configured address, directly via a call to sendmail) or
the 'new-style' notification system.
notification-mode has three possible values:
- legacy-se
This selector allows one to selected between the 'old' (send email
directly via sendmail) or the 'new' notification system.
The default is 'auto', which sends and email if one is configured,
and uses the notification system if no email address is set.
Signed-off-by: Lukas Wagner
---
Notes:
This selector allows one to selected between the 'old' (send email
directly via sendmail) or the 'new' notification system.
The default is 'auto', which sends and email if one is configured,
and uses the notification system if no email address is set.
Signed-off-by: Lukas Wagner
---
Notes:
On Wed, Nov 15, 2023 at 02:30:06PM +0100, Thomas Lamprecht wrote:
>
> Am 15/11/2023 um 13:23 schrieb Markus Frank:
> > Change regex from "m/^[a-zA-Z0-9]+$/" to "m/^[a-zA-Z0-9\-]+$/"
> > to allow hyphen in ldap attribute names for pve & pmg.
> > [..]
> > --- a/src/PVE/JSONSchema.pm
> > +++ b/src/P
>>
>>I tested the DHCPv6 setup now and unfortunately it does not work
>>fully.
>>
>>I have a VNet with both IPv4 and IPv6 subnets.
>>The IPAM part works correctly, registering both IPv4 and IPv6.
>>In the VM I can get an IPv4 from the DHCP but IPv6 does not work.
>>
>>I only get a /128 address but
Creating rpool/var/lib/vz and all intermediate datasets causes a
service-failure of `var.mount` upon shutdown.
creating the dataset for /var/lib/vz directly at the rpool and setting
its mountpoint property seems the most robust way to address this.
The alternative approach of setting `canmount=of
On 11/21/23 13:16, Christian Ebner wrote:
>> On 16.11.2023 11:35 CET Philipp Hufnagl wrote:
>>
>>
>> Currently, when adding a PBS storage with a namespace that does not
>> exist, the storage gets added normally, but browsing/using it only
>> returns a cryptic error message.
>>
>> This change
and switch the ESP to grub if it is enabled.
Signed-off-by: Fabian Grünbichler
---
Proxmox/Install.pm | 17 +++--
1 file changed, 15 insertions(+), 2 deletions(-)
diff --git a/Proxmox/Install.pm b/Proxmox/Install.pm
index 1a4ee93..a2f35ae 100644
--- a/Proxmox/Install.pm
+++ b/Proxmo
previously, when opening the dialog multiple times, the maximum was
determined by the previous set value, not the maxium possible for the
storage
Signed-off-by: Folke Gleumes
---
proxinstall | 25 +
1 file changed, 17 insertions(+), 8 deletions(-)
diff --git a/proxinstal
this prevents a lower hdsize to be set, when intermittently adding a
smaller storage device.
Signed-off-by: Folke Gleumes
---
proxinstall | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/proxinstall b/proxinstall
index 4fc31f8..695826d 100755
--- a/proxinstall
+++ b/proxin
until now it was only checked at install time, failing the whole
installation
Signed-off-by: Folke Gleumes
---
proxinstall | 6 ++
1 file changed, 6 insertions(+)
diff --git a/proxinstall b/proxinstall
index 01d4cfe..cf8f510 100755
--- a/proxinstall
+++ b/proxinstall
@@ -1526,6 +1526,12 @@
Am 21/11/2023 um 14:11 schrieb Stoiko Ivanov:
> Creating rpool/var/lib/vz and all intermediate datasets causes a
> service-failure of `var.mount` upon shutdown.
>
> creating the dataset for /var/lib/vz directly at the rpool and setting
> its mountpoint property seems the most robust way to address
Am 21/11/2023 um 14:13 schrieb Fabian Grünbichler:
> and switch the ESP to grub if it is enabled.
>
> Signed-off-by: Fabian Grünbichler
> ---
> Proxmox/Install.pm | 17 +++--
> 1 file changed, 15 insertions(+), 2 deletions(-)
>
>
applied, thanks!
_
Am 20/11/2023 um 16:45 schrieb Dominik Csapak:
> and make it into a proper field
> it's intended to be used like a single field and exactly as before
>
> Signed-off-by: Dominik Csapak
> ---
> no changes
> www/manager6/Makefile| 1 +
> www/manager6/form/IsoSelector.js | 107
see this old article:
https://www.isc.org/blogs/routing-configuration-over-dhcpv6-2/
for the custom dhcp option + script on client side
___
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
On 11/21/23 13:52, Lukas Wagner wrote:
> This patch series adds the 'notification-mode' setting for backup jobs.
> It allows users to choose between the 'old-style' notifications
> (mail to configured address, directly via a call to sendmail) or
> the 'new-style' notification system.
>
> noti
Another example with custom dhcpv6 options + client script
https://serverfault.com/a/528425
___
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
Am 21/11/2023 um 13:39 schrieb Markus Frank:
> Signed-off-by: Markus Frank
> Reviewed-by: Dominik Csapak
> Tested-by: Dominik Csapak
> ---
> qm.adoc | 18 ++
> 1 file changed, 18 insertions(+)
>
>
applied, thanks!
___
pve-devel mai
Signed-off-by: Stefan Hanreich
---
src/PVE/Network/SDN/Zones/SimplePlugin.pm | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/PVE/Network/SDN/Zones/SimplePlugin.pm
b/src/PVE/Network/SDN/Zones/SimplePlugin.pm
index f4525d5..7cb65c2 100644
--- a/src/PVE/Network/SDN/Zon
Suggested-By: Wolfgang Bumiller
Signed-off-by: Stefan Hanreich
---
src/PVE/Network/SDN/Zones/SimplePlugin.pm | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/PVE/Network/SDN/Zones/SimplePlugin.pm
b/src/PVE/Network/SDN/Zones/SimplePlugin.pm
index f30278c..f4525d5 100644
--- a/src/PVE/Netw
Am 21/11/2023 um 09:35 schrieb Dominik Csapak:
> in the case we add a second cd drive (for windows), we don't want the
> backend logic to only include the first one, since we cannot know
> which is bootable and which is (probably) the virtio iso.
>
> so instead, emulate the backend logic for the w
Am 29/09/2023 um 15:02 schrieb Aaron Lauterer:
> The main goal of this series is to improve the handling of configured
> default size & min_size values when creating a new Ceph Pool in the GUI.
>
> A new Ceph API endpoint, 'cfg/value', is added. It allows us to fetch
> values for config keys that
> On 21.11.2023 14:12 CET Philipp Hufnagl wrote:
>
>
> Hmmm... thats very curios. I tried that on my end and it worked fine.
>
> What datastore did you try to delete? Did you have a valid
> configuration for it?
Not deleting a datastore (that works fine), I was testing if I am able
to deact
applied both, thanks
___
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
applied container series with a small `$have_sdn` guard fixup
___
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
applied, thanks
___
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
applied qemu-server series (also with a small $have_sdn guard fixup ;-)
)
thanks
___
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
On 11/21/23 14:40, Christian Ebner wrote:
>
>> On 21.11.2023 14:12 CET Philipp Hufnagl wrote:
>>
>>
>> Hmmm... thats very curios. I tried that on my end and it worked fine.
>>
>> What datastore did you try to delete? Did you have a valid
>> configuration for it?
>
> Not deleting a datastore
The 'onlineHelp' is now set in the component definition in
'proxmox-widget-toolkit'.
Signed-off-by: Lukas Wagner
---
www/manager6/dc/Config.js | 1 -
1 file changed, 1 deletion(-)
diff --git a/www/manager6/dc/Config.js b/www/manager6/dc/Config.js
index 74a84e91..f22688f8 100644
--- a/www/manage
Add help buttons for notification related UIs, linking to the appropriate
section in the documentation.
First patch for pve-manager fixes the build against the latest
pve-doc-generator package from the current git master.
Requires pve-doc packages to be bumped (due to new section anchors)
pve
This links the dialog windows to the correct help section (different
target types, matchers).
Signed-off-by: Lukas Wagner
---
src/panel/GotifyEditPanel.js | 1 +
src/panel/NotificationConfigView.js | 1 +
src/panel/SendmailEditPanel.js| 1 +
src/panel/SmtpEditPanel.js
This enables us to directly link to the appropriate section from the
UI.
Signed-off-by: Lukas Wagner
---
notifications.adoc | 6 ++
1 file changed, 6 insertions(+)
diff --git a/notifications.adoc b/notifications.adoc
index 74447e5..21e13c4 100644
--- a/notifications.adoc
+++ b/notifications
These have been renamed in:
bcb72b 'sdn: overall language and consistency rework' in pve-docs
Signed-off-by: Lukas Wagner
---
www/manager6/sdn/VnetPanel.js| 2 +-
www/manager6/sdn/VnetView.js | 6 +++---
www/manager6/sdn/ZoneContentPanel.js | 2 +-
3 files changed, 5 insertions(+
If a VM or LXC starts and no IP was found in the IPAM, register a new one.
This is very seful as a fallback if for some reason an IP mapping was deleted
or there is a bug somewhere that does not register an IP.
This acts more like DHCP to allocate an IP on demand.
Special case that is not cove
if inside add_dhcp_mapping, which is called at VM or LCX start, we do
not find an IP in IPAM, register the MAC.
This is very useful as a fallback if for some reason an IP mapping was
deleted or there is a bug somewhere that does not register an IP.
This acts more like DHCP to allocate an IP on de
if no DHCP mapping was found in IPAM it will request a new IP which
requires these values.
Signed-off-by: Stefan Lendl
---
PVE/QemuServer.pm | 2 +-
vm-network-scripts/pve-bridge | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/PVE/QemuServer.pm b/PVE/QemuServ
If no DHCP mapping was found in IPAM it will request a new IP.
In order to register an IPAM mapping it requires these values.
Signed-off-by: Stefan Lendl
---
src/PVE/LXC.pm| 4 ++--
src/lxc-pve-prestart-hook | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/P
We replace the use of nested boxes with grids when possible to ensure correct
margins and alignment of widgets.
At the end we also remove trailing spaces and colons from labels preceding an
input widget, e.g. we replace `DNS Server: ` with `DNS Server`.
Differences from v1:
- Rebased on top of
Previously the grids were inserted in a succession of boxes each with
its own set of margins and spacing. We define the margins now
exclusively in the grid and account for previous values.
Note that we match the top and bottom margins of the 'Target Harddisk'
panel which does not need to use a gri
This will be used in future commits to create grids so we need it to be defined.
Signed-off-by: Maximiliano Sandoval
---
proxinstall | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/proxinstall b/proxinstall
index 01d4cfe..8f40234 100755
--- a/
Using boxes causes the labels to not align correctly in certain
circumstances. In the following commits we replace the use of boxes with
grids and set the margins and spacing directly on the respective grid.
Signed-off-by: Maximiliano Sandoval
---
proxinstall | 52 ---
This accounts for the different layout set in the previous commit.
Signed-off-by: Maximiliano Sandoval
---
proxinstall | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/proxinstall b/proxinstall
index 8f40234..a5de06c 100755
--- a/proxinstall
+++ b/proxinstall
@@ -308,7 +308,7
1 - 100 of 132 matches
Mail list logo
