Signed-off-by: Lukas Wagner
---
proxmox-notify/examples/render.rs | 63 +++
1 file changed, 63 insertions(+)
create mode 100644 proxmox-notify/examples/render.rs
diff --git a/proxmox-notify/examples/render.rs
b/proxmox-notify/examples/render.rs
new file mode 100644
Signed-off-by: Lukas Wagner
---
Cargo.toml | 1 +
proxmox-notify/Cargo.toml | 9 +
proxmox-notify/src/config.rs| 51 +
proxmox-notify/src/endpoints/mod.rs | 0
proxmox-notify/src/lib.rs | 311
proxmox
The proxy configuration will be read from datacenter.cfg via
a new method of the `Context` trait.
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/context.rs | 1 +
proxmox-notify/src/endpoints/gotify.rs | 22 --
2 files changed, 17 insertions(+), 6 deletions
Signed-off-by: Lukas Wagner
---
pve-rs/src/notify.rs | 34 --
1 file changed, 32 insertions(+), 2 deletions(-)
diff --git a/pve-rs/src/notify.rs b/pve-rs/src/notify.rs
index 9677d8b..74a872b 100644
--- a/pve-rs/src/notify.rs
+++ b/pve-rs/src/notify.rs
@@ -2,10
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/api/group.rs | 264
proxmox-notify/src/api/mod.rs | 1 +
2 files changed, 265 insertions(+)
create mode 100644 proxmox-notify/src/api/group.rs
diff --git a/proxmox-notify/src/api/group.rs b/proxmox-notify
Signed-off-by: Lukas Wagner
---
pve-rs/src/notify.rs | 70
1 file changed, 70 insertions(+)
diff --git a/pve-rs/src/notify.rs b/pve-rs/src/notify.rs
index 74a872b..cac233a 100644
--- a/pve-rs/src/notify.rs
+++ b/pve-rs/src/notify.rs
@@ -5,6 +5,7
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/api/common.rs | 11 +++
proxmox-notify/src/api/mod.rs| 125 +++
2 files changed, 136 insertions(+)
diff --git a/proxmox-notify/src/api/common.rs b/proxmox-notify/src/api/common.rs
index 518caa8f..48761fbb 100644
Signed-off-by: Lukas Wagner
---
pve-rs/src/notify.rs | 83
1 file changed, 83 insertions(+)
diff --git a/pve-rs/src/notify.rs b/pve-rs/src/notify.rs
index 9490ea8..aa2c312 100644
--- a/pve-rs/src/notify.rs
+++ b/pve-rs/src/notify.rs
@@ -5,6 +5,10
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/api/filter.rs | 231 +
proxmox-notify/src/api/gotify.rs | 12 ++
proxmox-notify/src/api/group.rs| 7 +
proxmox-notify/src/api/mod.rs | 1 +
proxmox-notify/src/api/sendmail.rs | 10 ++
5 files
When notifying via a group, all endpoints contained in that group
will send out the notification.
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/config.rs | 9 ++
proxmox-notify/src/group.rs | 40 +
proxmox-notify/src/lib.rs| 170 ---
3 files
The new notification backend is implemented in Rust where we use SHA256
for config digests.
Signed-off-by: Lukas Wagner
---
src/PVE/JSONSchema.pm | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm
index 7589bba..49e0d7a
This parameter allows to send mails to the email address configured
for users from the product's user database.
`proxmox-notify` now has a `Context` that must be set via
`proxmox_notify::context::set_context` before the crate is used.
Signed-off-by: Lukas Wagner
---
pve-rs/Cargo.toml
They need to have the same name as the target.
Took the opportunity to move the .PHONY right next to the target recipe,
so that mistakes like these are hopefully easier caught.
Signed-off-by: Lukas Wagner
---
test/Makefile | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff
ACL paths for notification targets can become quite long, e.g.:
/mappings/notifications/
Signed-off-by: Lukas Wagner
---
www/manager6/form/PermPathSelector.js | 1 +
1 file changed, 1 insertion(+)
diff --git a/www/manager6/form/PermPathSelector.js
b/www/manager6/form/PermPathSelector.js
index
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/api/mod.rs | 94 +++
proxmox-notify/src/lib.rs | 1 +
2 files changed, 95 insertions(+)
create mode 100644 proxmox-notify/src/api/mod.rs
diff --git a/proxmox-notify/src/api/mod.rs b/proxmox-notify/src/api
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/api/mod.rs | 7 +
proxmox-notify/src/api/sendmail.rs | 254 +
2 files changed, 261 insertions(+)
create mode 100644 proxmox-notify/src/api/sendmail.rs
diff --git a/proxmox-notify/src/api/mod.rs b/proxmox
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/context.rs| 2 ++
proxmox-notify/src/endpoints/sendmail.rs | 18 +-
2 files changed, 15 insertions(+), 5 deletions(-)
diff --git a/proxmox-notify/src/context.rs b/proxmox-notify/src/context.rs
index 25be949a
Signed-off-by: Lukas Wagner
---
PVE/API2/Cluster/Notifications.pm | 254 ++
1 file changed, 254 insertions(+)
diff --git a/PVE/API2/Cluster/Notifications.pm
b/PVE/API2/Cluster/Notifications.pm
index 1efebbc1..b1971911 100644
--- a/PVE/API2/Cluster/Notifications.pm
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/lib.rs | 21 ++---
1 file changed, 14 insertions(+), 7 deletions(-)
diff --git a/proxmox-notify/src/lib.rs b/proxmox-notify/src/lib.rs
index 3c2b6d55..d2f15496 100644
--- a/proxmox-notify/src/lib.rs
+++ b/proxmox-notify/src
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/api/gotify.rs | 284 +++
proxmox-notify/src/api/mod.rs| 6 +
2 files changed, 290 insertions(+)
create mode 100644 proxmox-notify/src/api/gotify.rs
diff --git a/proxmox-notify/src/api/gotify.rs b/proxmox
Signed-off-by: Lukas Wagner
---
www/manager6/Makefile | 1 +
www/manager6/dc/Config.js | 12 ++
www/manager6/dc/NotificationEvents.js | 238 ++
3 files changed, 251 insertions(+)
create mode 100644 www/manager6/dc/NotificationEvents.js
Signed-off-by: Lukas Wagner
---
pve-rs/Cargo.toml| 1 +
pve-rs/Makefile | 1 +
pve-rs/src/lib.rs| 1 +
pve-rs/src/notify.rs | 71
4 files changed, 74 insertions(+)
create mode 100644 pve-rs/src/notify.rs
diff --git a/pve-rs
Signed-off-by: Lukas Wagner
---
pve-rs/src/notify.rs | 33 -
1 file changed, 32 insertions(+), 1 deletion(-)
diff --git a/pve-rs/src/notify.rs b/pve-rs/src/notify.rs
index ea34bfe..04e902c 100644
--- a/pve-rs/src/notify.rs
+++ b/pve-rs/src/notify.rs
@@ -34,6
This introduces a new configuration parameter `mailto-user`.
A user's email address will be looked up in the product-specific
user database.
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/api/sendmail.rs | 32 ---
proxmox-notify/src/context.rs
Signed-off-by: Lukas Wagner
---
Notes:
Did not add version number since I do not know which it will be yet.
debian/control | 2 ++
1 file changed, 2 insertions(+)
diff --git a/debian/control b/debian/control
index 3206b514..b807dbfe 100644
--- a/debian/control
+++ b/debian/control
dds the 'never' option.
- Mark 'mailnotification' as deprecated in favor of 'notification-policy'
- Clarify that 'mailto' is ignored if 'notification-target' is set
Signed-off-by: Lukas Wagner
---
src/PVE/VZDump/Common.pm | 21 +++-
Signed-off-by: Lukas Wagner
---
www/manager6/dc/Config.js | 16
1 file changed, 16 insertions(+)
diff --git a/www/manager6/dc/Config.js b/www/manager6/dc/Config.js
index aa025c8d..9ba7b301 100644
--- a/www/manager6/dc/Config.js
+++ b/www/manager6/dc/Config.js
@@ -329,6 +329,22
These options allow setting the notification target for package update
notifications, node fencing notifications and replication notifications.
Also, fencing and replication has now new options that allow disabling
notifications altogether.
Signed-off-by: Lukas Wagner
---
src/PVE
ls whether
to send a notification at all.
Signed-off-by: Lukas Wagner
---
PVE/API2/Replication.pm | 63 -
1 file changed, 43 insertions(+), 20 deletions(-)
diff --git a/PVE/API2/Replication.pm b/PVE/API2/Replication.pm
index 89c5a802..d61518ba 100644
--- a/PVE/API2/
In essence the same change as for backup jobs.
Signed-off-by: Lukas Wagner
---
www/manager6/window/Backup.js | 35 ++-
1 file changed, 34 insertions(+), 1 deletion(-)
diff --git a/www/manager6/window/Backup.js b/www/manager6/window/Backup.js
index 4b21c746
Signed-off-by: Lukas Wagner
---
PVE/API2/Cluster.pm | 7 +++
PVE/API2/Cluster/Makefile | 1 +
PVE/API2/Cluster/Notifications.pm | 71 +++
3 files changed, 79 insertions(+)
create mode 100644 PVE/API2/Cluster/Notifications.pm
diff --git a/PVE
in `proxmox-perl-rs` and handler in
`pve-manager`)
- Integrated new notification channels in backup jobs/one-off backups (repo
`pve-manager`)
- Replication/APT/Fencing use an 'anonymous' channel with a temporary
sendmail endpoint, sending mails to `root`
- Added n
... instead of using sendmail directly
If the new 'target-package-updates' is set, we send a
notification to this target. If not, we continue to send
a mail to root@pam (if the mail address is configured)
Signed-off-by: Lukas Wagner
---
PVE/API2/A
The package contains the PVE::Notify. It is a very thin wrapper
around the Proxmox::RS::Notify module, feeding the configuration
from the new 'notifications.cfg' and 'priv/notifications.cfg' files
into it.
Signed-off-by: Lukas Wagner
---
debian/control
Signed-off-by: Lukas Wagner
---
src/Makefile | 1 +
src/Schema.js| 5
src/panel/GotifyEditPanel.js | 52
3 files changed, 58 insertions(+)
create mode 100644 src/panel/GotifyEditPanel.js
diff --git a/src/Makefile b/src
#x27; paramter which controls if
notifications should be sent at all. If it is not set, we
default to the old behavior, which is to send.
Also add dependency to the `libpve-notify-perl` package to d/control.
Signed-off-by: Lukas Wagner
---
debian/control | 2 ++
src/PVE/HA/Env.pm|
Signed-off-by: Lukas Wagner
---
PVE/API2/Cluster/Notifications.pm | 271 ++
1 file changed, 271 insertions(+)
diff --git a/PVE/API2/Cluster/Notifications.pm
b/PVE/API2/Cluster/Notifications.pm
index aea571f0..8f0b6429 100644
--- a/PVE/API2/Cluster/Notifications.pm
Signed-off-by: Lukas Wagner
---
src/Makefile| 1 +
src/form/NotificationFilterSelector.js | 58 +
src/panel/GotifyEditPanel.js| 9
src/panel/NotificationConfigView.js | 4 ++
src/panel/NotificationGroupEditPanel.js | 9
Signed-off-by: Lukas Wagner
---
src/Makefile | 3 +-
src/data/model/NotificationConfig.js | 9 ++
src/panel/NotificationConfigView.js | 119 +++
src/window/NotificationFilterEdit.js | 115 ++
4 files changed, 245
backup jobs. Some logic which automatically migrates from
'mailnotification' has been added.
Signed-off-by: Lukas Wagner
---
www/manager6/Makefile | 4 +-
www/manager6/dc/Backup.js | 84 +--
www/manager6/form/Notifica
Signed-off-by: Lukas Wagner
---
notifications.adoc | 159 +++
pve-admin-guide.adoc | 2 +
pve-gui.adoc | 2 +
vzdump.adoc | 5 ++
4 files changed, 168 insertions(+)
create mode 100644 notifications.adoc
diff --git a
rbatim-monospaced}}{{#verbatim-monospaced}}
Do not reflow text. NOP for plain text, but for HTML output the text
will be contained in a with a monospaced font.
Signed-off-by: Lukas Wagner
---
Cargo.toml | 1 +
proxmox-notify/Cargo.toml| 6 +-
Signed-off-by: Lukas Wagner
---
src/Makefile | 4 +
src/Schema.js| 8 ++
src/data/model/NotificationConfig.js | 8 ++
src/panel/NotificationConfigView.js | 192 +++
src/panel/SendmailEditPanel.js | 140
Signed-off-by: Lukas Wagner
---
pve-rs/src/notify.rs | 11 +++
1 file changed, 11 insertions(+)
diff --git a/pve-rs/src/notify.rs b/pve-rs/src/notify.rs
index 04e902c..0c00b03 100644
--- a/pve-rs/src/notify.rs
+++ b/pve-rs/src/notify.rs
@@ -60,6 +60,11 @@ impl Context for PVEContext
Signed-off-by: Lukas Wagner
---
src/PVE/Cluster.pm | 2 ++
src/pmxcfs/status.c | 2 ++
2 files changed, 4 insertions(+)
diff --git a/src/PVE/Cluster.pm b/src/PVE/Cluster.pm
index c310a67..e3705b6 100644
--- a/src/PVE/Cluster.pm
+++ b/src/PVE/Cluster.pm
@@ -55,6 +55,8 @@ my $observed
Signed-off-by: Lukas Wagner
---
PVE/API2/Cluster/Notifications.pm | 263 ++
1 file changed, 263 insertions(+)
diff --git a/PVE/API2/Cluster/Notifications.pm
b/PVE/API2/Cluster/Notifications.pm
index 8f0b6429..e358573c 100644
--- a/PVE/API2/Cluster/Notifications.pm
Signed-off-by: Lukas Wagner
---
pve-rs/src/notify.rs | 83
1 file changed, 83 insertions(+)
diff --git a/pve-rs/src/notify.rs b/pve-rs/src/notify.rs
index aa2c312..a6143fc 100644
--- a/pve-rs/src/notify.rs
+++ b/pve-rs/src/notify.rs
@@ -12,6 +12,9
The function returns all other entities referenced by a filter/target.
This is useful for permission checks, where the user must have the
appropriate permissions for all entities.
Signed-off-by: Lukas Wagner
---
pve-rs/src/notify.rs | 9 +
1 file changed, 9 insertions(+)
diff --git a
Check notification targets configured in datacenter.cfg and jobs.cfg,
failing if the group/endpoint to be removed is still in use there.
Signed-off-by: Lukas Wagner
---
PVE/API2/Cluster/Notifications.pm | 44 ++-
1 file changed, 43 insertions(+), 1 deletion(-)
diff
Signed-off-by: Lukas Wagner
---
Notes:
I'm not sure if I like this solution, but adding notification targets to
the resources API endpoint would not have make sense.
Maybe we could create a new API endpoint that returns all possible ACL
paths and then use a normal store fo
Signed-off-by: Lukas Wagner
---
pve-rs/src/notify.rs | 88
1 file changed, 88 insertions(+)
diff --git a/pve-rs/src/notify.rs b/pve-rs/src/notify.rs
index cac233a..9490ea8 100644
--- a/pve-rs/src/notify.rs
+++ b/pve-rs/src/notify.rs
@@ -5,6 +5,9
Signed-off-by: Lukas Wagner
---
PVE/API2/Cluster/Notifications.pm | 40 +++
1 file changed, 40 insertions(+)
diff --git a/PVE/API2/Cluster/Notifications.pm
b/PVE/API2/Cluster/Notifications.pm
index 32a873a7..fa2c1d9d 100644
--- a/PVE/API2/Cluster/Notifications.pm
Signed-off-by: Lukas Wagner
---
src/Makefile| 1 +
src/Schema.js | 5 +
src/panel/NotificationGroupEditPanel.js | 177
src/window/EndpointEditBase.js | 6 +-
4 files changed, 188 insertions(+), 1
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/api/filter.rs | 1 +
proxmox-notify/src/api/gotify.rs | 1 +
proxmox-notify/src/api/mod.rs | 113 ++---
proxmox-notify/src/api/sendmail.rs | 1 +
4 files changed, 106 insertions(+), 10 deletions(-)
diff
Otherwise, a filter with the same name as an already existing
endpoint or group can overwrite it.
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/api/filter.rs | 7 +
proxmox-notify/src/api/gotify.rs | 10 +--
proxmox-notify/src/api/group.rs| 24 ++-
proxmox
Signed-off-by: Lukas Wagner
---
test/Makefile | 8
test/{mail_test.pl => vzdump_notification_test.pl} | 0
2 files changed, 4 insertions(+), 4 deletions(-)
rename test/{mail_test.pl => vzdump_notification_test.pl} (100%)
diff --git a/test/Ma
Signed-off-by: Lukas Wagner
---
www/manager6/dc/BackupJobDetail.js | 20
1 file changed, 16 insertions(+), 4 deletions(-)
diff --git a/www/manager6/dc/BackupJobDetail.js
b/www/manager6/dc/BackupJobDetail.js
index c4683a47..8b9bb749 100644
--- a/www/manager6/dc
Signed-off-by: Lukas Wagner
---
proxmox-notify/debian/changelog | 5 ++
proxmox-notify/debian/control | 108
proxmox-notify/debian/copyright | 16 +
proxmox-notify/debian/debcargo.toml | 7 ++
4 files changed, 136 insertions(+)
create mode
end_notification`
- Breaking out some of the code into helper subs, hopefully
reducing the spaghetti factor a bit
Signed-off-by: Lukas Wagner
---
PVE/API2/VZDump.pm | 10 +-
PVE/VZDump.pm | 335 +
test/mail_test.pl | 36 ++---
3 files c
This commit adds a way to filter notifications based on severity. The
filter module also has the necessary foundation work for more complex
filters, e.g. matching on properties or for creating arbitarily complex
filter structures using nested sub-filters.
Signed-off-by: Lukas Wagner
---
proxmox
Signed-off-by: Lukas Wagner
---
PVE/API2/Cluster/Notifications.pm | 315 ++
1 file changed, 315 insertions(+)
diff --git a/PVE/API2/Cluster/Notifications.pm
b/PVE/API2/Cluster/Notifications.pm
index b1971911..aea571f0 100644
--- a/PVE/API2/Cluster/Notifications.pm
Signed-off-by: Lukas Wagner
---
PVE/API2/Cluster/Notifications.pm | 100 ++
1 file changed, 100 insertions(+)
diff --git a/PVE/API2/Cluster/Notifications.pm
b/PVE/API2/Cluster/Notifications.pm
index e358573c..32a873a7 100644
--- a/PVE/API2/Cluster/Notifications.pm
Thanks for the review!
On 7/17/23 17:48, Maximiliano Sandoval wrote:
Lukas Wagner writes:
Signed-off-by: Lukas Wagner
---
Cargo.toml | 1 +
proxmox-notify/Cargo.toml | 9 +
proxmox-notify/src/config.rs| 51 +
proxmox-notify/src
Signed-off-by: Lukas Wagner
---
www/manager6/dc/Backup.js| 3 +--
www/manager6/grid/Replication.js | 3 +--
2 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/www/manager6/dc/Backup.js b/www/manager6/dc/Backup.js
index 03a02651..e662dd36 100644
--- a/www/manager6/dc/Backup.js
On 7/18/23 14:34, Dominik Csapak wrote:
gave the series a quick spin, review of the gui patches comes later ;)
a few high level comments from a user perspective:
* the node fencing/replication edit windows always shows the warning that it
shouldn't be
disabled, that should imo only be there
On 7/18/23 14:44, Wolfgang Bumiller wrote:
+}
+
+if super::endpoint_exists(config, &endpoint_config.name) {
(*could* dedup the whole if into a helper in `super` so we don't need to
copy-pasta the message to every new endpoint - we already have this
twice now ;-) )
Yup, a later com
On 7/18/23 15:58, Dominik Csapak wrote:
I already have patches laying around that implement the additional filter
matchers, but
decided to not include them in the patch series yet. Before the new matchers
are merged,
I would need to 'stabilize' the properties associate with every single
not
Hi again,
On 7/18/23 14:34, Dominik Csapak wrote:
* i found one bug, but not quite sure yet where it comes from exactly,
putting in emojis into a field (e.g. a comment or author) it's accepted,
but editing a different entry fails with:
--->8---
could not serialize configuration: writing '
On 7/19/23 12:23, Wolfgang Bumiller wrote:
+#[export(raw_return)]
+fn parse_config(
+#[raw] class: Value,
+raw_config: &str,
+raw_private_config: &str,
I half-suspect that using &[u8] here - since that's what perl *actually*
gives us if we just read the data out
On 7/18/23 14:36, Wolfgang Bumiller wrote:
+
+/// Get sendmail endpoint with given `name`.
+///
+/// The caller is responsible for any needed permission checks.
+/// Returns the endpoint or an `ApiError` if the endpoint was not found.
+pub fn get_endpoint(config: &Config, name: &str) -> Result
On 7/19/23 14:11, Wolfgang Bumiller wrote:
The String -> &[u8] modification in the parse method may be a follow-up
or a v4 - depending on how we proceed with applying this the whole
thing.
Discussed this with Thomas yesterday - I'll post a v4 once all reviews are done.
Though I'll leave some th
On 7/18/23 16:37, Thomas Lamprecht wrote:
Am 18/07/2023 um 14:34 schrieb Dominik Csapak:
* the backup edit window is rather tall at this point, and if we assume a
minimum
screen size of 1280x720 there is not quite enough space when you subtract
the (typical) os bar and browser window deco
Hi,
On 7/19/23 14:45, Dominik Csapak wrote:
+
+Ext.define('PVE.dc.NotificationEventsTargetSelector', {
+ alias: ['widget.pveNotificationEventsTargetSelector'],
+ extend: 'PVE.form.NotificationTargetSelector',
+ fieldLabel: gettext('Notification Target'),
+ allowBlank: true,
+ edit
On 7/19/23 14:53, Dominik Csapak wrote:
On 7/17/23 17:00, Lukas Wagner wrote:
Signed-off-by: Lukas Wagner
---
Notes:
I'm not sure if I like this solution, but adding notification targets to
the resources API endpoint would not have make sense.
Maybe we could create a ne
On 7/20/23 09:54, Dominik Csapak wrote:
i'ts even easier, the user can simply edit the field manually ;)
i know this trips up many people, but we often have dropdown fields
that are actually manually editable (like the permpathselector)
but it's only hinted at by the blinking cursor in the fi
On 7/19/23 15:32, Dominik Csapak wrote:
+ mixins: {
+ field: 'Ext.form.field.Field',
+ },
when implementing the field mixin you have to (quote from the extjs docs)
---8<---
You will also need to make sure that initField is called during the component's
initialization.
--->8---
so yo
Signed-off-by: Lukas Wagner
---
Notes:
Changes since v3:
- New in v4
proxmox-schema/src/api_types.rs | 8
1 file changed, 8 insertions(+)
diff --git a/proxmox-schema/src/api_types.rs b/proxmox-schema/src/api_types.rs
index ba6f02df..0cec043c 100644
--- a/proxmox-schema/src
Signed-off-by: Lukas Wagner
---
Notes:
Changes since v3:
- New in v4
proxmox-section-config/src/lib.rs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/proxmox-section-config/src/lib.rs
b/proxmox-section-config/src/lib.rs
index a86692a8..4441df12 100644
--- a
Signed-off-by: Lukas Wagner
---
Notes:
Changes v1 -> v2:
- Renamed crate from 'proxmox-notification' to 'proxmox-notify'
Cargo.toml| 1 +
proxmox-notify/Cargo.toml | 10 ++
proxmox-notify/src/lib.rs | 0
3 files changed, 11 ins
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/api/mod.rs | 94 +++
proxmox-notify/src/lib.rs | 1 +
2 files changed, 95 insertions(+)
create mode 100644 proxmox-notify/src/api/mod.rs
diff --git a/proxmox-notify/src/api/mod.rs b/proxmox-notify/src/api
Signed-off-by: Lukas Wagner
---
proxmox-notify/examples/render.rs | 63 +++
1 file changed, 63 insertions(+)
create mode 100644 proxmox-notify/examples/render.rs
diff --git a/proxmox-notify/examples/render.rs
b/proxmox-notify/examples/render.rs
new file mode 100644
This plugin uses the 'sendmail' command to send an email
to one or more recipients.
Signed-off-by: Lukas Wagner
---
proxmox-notify/Cargo.toml| 9 ++-
proxmox-notify/src/config.rs | 12
proxmox-notify/src/endpoints/mod.rs | 2 +
proxmox-
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/api/mod.rs | 7 +
proxmox-notify/src/api/sendmail.rs | 254 +
2 files changed, 261 insertions(+)
create mode 100644 proxmox-notify/src/api/sendmail.rs
diff --git a/proxmox-notify/src/api/mod.rs b/proxmox
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/api/group.rs | 264
proxmox-notify/src/api/mod.rs | 1 +
2 files changed, 265 insertions(+)
create mode 100644 proxmox-notify/src/api/group.rs
diff --git a/proxmox-notify/src/api/group.rs b/proxmox-notify
Signed-off-by: Lukas Wagner
---
pve-rs/src/notify.rs | 34 --
1 file changed, 32 insertions(+), 2 deletions(-)
diff --git a/pve-rs/src/notify.rs b/pve-rs/src/notify.rs
index 6ea9b78..cff1b44 100644
--- a/pve-rs/src/notify.rs
+++ b/pve-rs/src/notify.rs
@@ -2,10
Signed-off-by: Lukas Wagner
---
pve-rs/src/notify.rs | 11 +++
1 file changed, 11 insertions(+)
diff --git a/pve-rs/src/notify.rs b/pve-rs/src/notify.rs
index 5fc11b2..5ab0ef5 100644
--- a/pve-rs/src/notify.rs
+++ b/pve-rs/src/notify.rs
@@ -62,6 +62,11 @@ impl Context for PVEContext
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/api/common.rs | 44
proxmox-notify/src/api/mod.rs| 2 ++
2 files changed, 46 insertions(+)
create mode 100644 proxmox-notify/src/api/common.rs
diff --git a/proxmox-notify/src/api/common.rs b/proxmox-notify
Signed-off-by: Lukas Wagner
---
Notes:
Changes since v3:
- Removed unneeded drain() call
proxmox-notify/src/api/common.rs | 11 +++
proxmox-notify/src/api/mod.rs| 125 +++
2 files changed, 136 insertions(+)
diff --git a/proxmox-notify/src/api
Signed-off-by: Lukas Wagner
---
Notes:
Did not add version number since I do not know which it will be yet.
debian/control | 2 ++
1 file changed, 2 insertions(+)
diff --git a/debian/control b/debian/control
index 3206b514..b807dbfe 100644
--- a/debian/control
+++ b/debian/control
When notifying via a group, all endpoints contained in that group
will send out the notification.
Signed-off-by: Lukas Wagner
---
proxmox-notify/src/config.rs | 9 ++
proxmox-notify/src/group.rs | 41 +
proxmox-notify/src/lib.rs| 170 ---
3 files
They need to have the same name as the target.
Took the opportunity to move the .PHONY right next to the target recipe,
so that mistakes like these are hopefully easier caught.
Signed-off-by: Lukas Wagner
---
test/Makefile | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff
Signed-off-by: Lukas Wagner
---
pve-rs/src/notify.rs | 83
1 file changed, 83 insertions(+)
diff --git a/pve-rs/src/notify.rs b/pve-rs/src/notify.rs
index 1d612f1..08726e5 100644
--- a/pve-rs/src/notify.rs
+++ b/pve-rs/src/notify.rs
@@ -12,6 +12,9
Signed-off-by: Lukas Wagner
---
PVE/API2/Cluster.pm | 7 +++
PVE/API2/Cluster/Makefile | 1 +
PVE/API2/Cluster/Notifications.pm | 71 +++
3 files changed, 79 insertions(+)
create mode 100644 PVE/API2/Cluster/Notifications.pm
diff --git a/PVE
Signed-off-by: Lukas Wagner
---
www/manager6/dc/BackupJobDetail.js | 20
1 file changed, 16 insertions(+), 4 deletions(-)
diff --git a/www/manager6/dc/BackupJobDetail.js
b/www/manager6/dc/BackupJobDetail.js
index c4683a47..8b9bb749 100644
--- a/www/manager6/dc
Check notification targets configured in datacenter.cfg and jobs.cfg,
failing if the group/endpoint to be removed is still in use there.
Signed-off-by: Lukas Wagner
---
PVE/API2/Cluster/Notifications.pm | 44 ++-
1 file changed, 43 insertions(+), 1 deletion(-)
diff
In essence the same change as for backup jobs.
Signed-off-by: Lukas Wagner
---
www/manager6/window/Backup.js | 35 ++-
1 file changed, 34 insertions(+), 1 deletion(-)
diff --git a/www/manager6/window/Backup.js b/www/manager6/window/Backup.js
index 4b21c746
ls whether
to send a notification at all.
Signed-off-by: Lukas Wagner
---
PVE/API2/Replication.pm | 63 -
1 file changed, 43 insertions(+), 20 deletions(-)
diff --git a/PVE/API2/Replication.pm b/PVE/API2/Replication.pm
index 89c5a802..d61518ba 100644
--- a/PVE/API2/
Signed-off-by: Lukas Wagner
---
PVE/API2/Cluster/Notifications.pm | 100 ++
1 file changed, 100 insertions(+)
diff --git a/PVE/API2/Cluster/Notifications.pm
b/PVE/API2/Cluster/Notifications.pm
index e358573c..32a873a7 100644
--- a/PVE/API2/Cluster/Notifications.pm
Settings for notifications have been moved to their own view.
Signed-off-by: Lukas Wagner
---
Notes:
Changes since v3:
- New in v4
www/manager6/dc/OptionView.js | 20
1 file changed, 20 deletions(-)
diff --git a/www/manager6/dc/OptionView.js b/www/manager6/dc
ACL paths for notification targets can become quite long, e.g.:
/mappings/notifications/
Signed-off-by: Lukas Wagner
---
www/manager6/form/PermPathSelector.js | 1 +
1 file changed, 1 insertion(+)
diff --git a/www/manager6/form/PermPathSelector.js
b/www/manager6/form/PermPathSelector.js
index
501 - 600 of 1023 matches
Mail list logo
