- don't listen to ip address, but use interface= instead
- generate 1 config file by vnet instead 1 by subnet
- enable-ra is global to server, enable it in default conf
Signed-off-by: Alexandre Derumier
---
src/PVE/Network/SDN/Dhcp.pm | 33 +-
Hi,
Apologies in advance if this is not the right place to post this. Please
redirect me to the appropriate forum if not. I'm also happy to discuss
off list if that is deemed more appropriate.
My name is Jeremy and I work with TurnKey Linux.
As a housekeeping matter, we're looking to update
Am 17/11/2023 um 12:39 schrieb Stefan Hanreich:
> Signed-off-by: Stefan Hanreich
> ---
> PVE/API2/Network.pm | 1 +
> 1 file changed, 1 insertion(+)
>
>
applied the manager ones, thanks!
___
pve-devel mailing list
pve-devel@lists.proxmox.com
Am 21/11/2023 um 20:46 schrieb Stefan Hanreich:
> Signed-off-by: Stefan Hanreich
> ---
> www/manager6/sdn/SubnetEdit.js | 51 +-
> 1 file changed, 38 insertions(+), 13 deletions(-)
>
>
applied, thanks!
___
pve-devel
Am 20/11/2023 um 17:28 schrieb Stefan Hanreich:
> The IPAM-related API endpoints were moved, reflect those changes in
> the UI as well.
>
> Signed-off-by: Stefan Hanreich
> ---
> www/manager6/sdn/IpamEdit.js | 4 +++-
> www/manager6/tree/DhcpTree.js | 15 +++
> 2 files changed, 14
Am 21/11/2023 um 21:19 schrieb Stoiko Ivanov:
> seems adding `--removable` makes grub install ignore most other
> information - e.g. the bootloader-id (guessed based on [0]).
> add dedicated call with out `--removable` in addition
>
> Seems that's the reason why our installer also 'rolls its own
seems adding `--removable` makes grub install ignore most other
information - e.g. the bootloader-id (guessed based on [0]).
add dedicated call with out `--removable` in addition
Seems that's the reason why our installer also 'rolls its own
removable' [1]
minimally tested with an ISO with this
Am 21/11/2023 um 15:55 schrieb Stefan Lendl:
>
> 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
Am 21/11/2023 um 18:50 schrieb Stefan Hanreich:
> Signed-off-by: Stefan Hanreich
> ---
> src/lxc-pve-prestart-hook | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>
applied, thanks!
___
pve-devel mailing list
pve-devel@lists.proxmox.com
Am 21/11/2023 um 15:47 schrieb Lukas Wagner:
> 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 +
>
Am 21/11/2023 um 17:01 schrieb Lukas Wagner:
> This is the severity used for forwarded mails, since we cannot
> reasonably infer a priority here.
>
>
>
> pve-docs:
>
> Lukas Wagner (1):
> notification: document 'unknown' severity
>
> notifications.adoc | 2 +-
> 1 file changed, 1
Signed-off-by: Stefan Hanreich
---
www/manager6/sdn/SubnetEdit.js | 51 +-
1 file changed, 38 insertions(+), 13 deletions(-)
diff --git a/www/manager6/sdn/SubnetEdit.js b/www/manager6/sdn/SubnetEdit.js
index 4fe16ab92..8851b013a 100644
---
Am 17/11/2023 um 12:40 schrieb Stefan Hanreich:
> Signed-off-by: Stefan Hanreich
> ---
> pvesdn.adoc | 122
> 1 file changed, 122 insertions(+)
>
>
applied, with some merge conflicts resolved, maybe you could also
recheck to ensure nothing
Signed-off-by: Stefan Hanreich
---
src/PVE/Network/SDN/SubnetPlugin.pm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/PVE/Network/SDN/SubnetPlugin.pm
b/src/PVE/Network/SDN/SubnetPlugin.pm
index 88933f5..37b6b2b 100644
--- a/src/PVE/Network/SDN/SubnetPlugin.pm
+++
"DERUMIER, Alexandre" writes:
> Another example with custom dhcpv6 options + client script
>
> https://antiphishing.cetsi.fr/proxy/v3?i=Z09lbVhROVBaOHd3NjZZMVfTogSU
> UCLtvQYv_TI7-
> Qg=eVVJWkdic3NtZTdZV2pycd0wEJhcfg3n60sFMbDyx7fGbFFjGiZx-
>
Signed-off-by: Stefan Hanreich
---
src/lxc-pve-prestart-hook | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/lxc-pve-prestart-hook b/src/lxc-pve-prestart-hook
index 4f5c7e2..6a60856 100755
--- a/src/lxc-pve-prestart-hook
+++ b/src/lxc-pve-prestart-hook
@@ -210,7 +210,7 @@
see below
On 11/21/23 15:55, Stefan Lendl wrote:
> 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
Am 21/11/2023 um 17:56 schrieb Stoiko Ivanov:
> noticed while installing with secure-boot enabled on ZFS RAID1:
> The system has no entry to boot from in the efi-vars and
> the entry for the first disk simply does not boot (I assume OVMF tries
> the default bootx64.efi.
>
> Since
noticed while installing with secure-boot enabled on ZFS RAID1:
The system has no entry to boot from in the efi-vars and
the entry for the first disk simply does not boot (I assume OVMF tries
the default bootx64.efi.
Since `proxmox-boot-tool init` should only be done for ESPs, which are
dedicated
Am 21/11/2023 um 15:47 schrieb Lukas Wagner:
> These have been renamed in:
> bcb72b 'sdn: overall language and consistency rework' in pve-docs
I reverted that part in the docs before I saw you patch and will keep it that
way, IMO one should have good reasons to break links than just vnet vs
Am 21/11/2023 um 15:47 schrieb Lukas Wagner:
> 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(-)
>
>
applied, thanks!
Am 21/11/2023 um 15:47 schrieb Lukas Wagner:
> 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(+)
>
>
applied this one, needing a bump for the other ones, thanks!
"DERUMIER, Alexandre" writes:
> Another example with custom dhcpv6 options + client script
>
> https://serverfault.com/a/528425
Thank you very much for clarifying. So the /128 is fine.
I also tried to ping6 dead:beef::1 to reach the gateway or from the host
to reach the VM and that did not
Am 21/11/2023 um 13:52 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.
>
>
Am 21/11/2023 um 16:53 schrieb Lukas Wagner:
> With the newly built-in targets/matchers, we should not add
> a target/matcher manually any more. In fact, this broke mail
> forwarding on PBS because 'default-matcher' already existed as a
> built-in and could thus not be created.
> We now simply do
This is the severity used for forwarded mails, since we cannot
reasonably infer a priority here.
Signed-off-by: Lukas Wagner
---
notifications.adoc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/notifications.adoc b/notifications.adoc
index 74447e5..8059953 100644
---
This is the severity used for forwarded mails, since we cannot
reasonably infer a priority here.
pve-docs:
Lukas Wagner (1):
notification: document 'unknown' severity
notifications.adoc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
proxmox-widget-toolkit:
Lukas Wagner (1):
This is the severity used for forwarded mails, since we cannot
reasonably infer a priority here.
Signed-off-by: Lukas Wagner
---
src/window/NotificationMatcherEdit.js | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/window/NotificationMatcherEdit.js
With the newly built-in targets/matchers, we should not add
a target/matcher manually any more. In fact, this broke mail
forwarding on PBS because 'default-matcher' already existed as a
built-in and could thus not be created.
We now simply do an early return.
Also initialize notify-context before
Signed-off-by: Maximiliano Sandoval
---
proxinstall | 34 --
1 file changed, 12 insertions(+), 22 deletions(-)
diff --git a/proxinstall b/proxinstall
index 69c48b4..bdff22d 100755
--- a/proxinstall
+++ b/proxinstall
@@ -648,12 +648,12 @@ sub update_layout {
my
For consistency sake, all colons and trailing spaces in labels that were
followed with an entry were removed, this matches other panels such as
the password and country/timezone panels.
Signed-off-by: Maximiliano Sandoval
---
proxinstall | 12 ++--
1 file changed, 6 insertions(+), 6
The extra 10px margin on the email row was added to account for the
removed line:
$vbox->pack_start($hbox3, 0, 0, 15);
Signed-off-by: Maximiliano Sandoval
---
proxinstall | 29 +++--
1 file changed, 11 insertions(+), 18 deletions(-)
diff --git a/proxinstall
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
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
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
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
---
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
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
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
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
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
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
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
+++
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
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)
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
---
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
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
applied, 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 both, thanks
___
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 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
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
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
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
---
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
---
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
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
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.
>
>
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
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
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 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
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 @@
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
+++
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
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
+++
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
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
>>
>>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
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
> > +++
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:
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:
-
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.
... 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
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
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
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
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
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.
*
On 11/21/23 13:38, Philipp Hufnagl wrote:
+ let notificationModeSelector = Ext.create({
+ xtype: 'proxmoxKVComboBox',
+ comboItems: [
+ ['auto', gettext('Auto')],
+ ['legacy-sendmail', gettext('Email (legacy)')],
+
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 |
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
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
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
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
>>>
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
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:
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
'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 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:
-
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 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
---
... 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
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
---
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
>
>
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
> 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
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
1 - 100 of 132 matches
Mail list logo
