Make Zulip's alertmanager integration closer to matrix-hookshot-adapter's formatting #287

Open
raito wants to merge 1 commit from alertmanager-zulip-ng into main
Owner

As @delroth asked for, the last commit contains a patch to Zulip to enable an alerting formatting closer to the one we had on Matrix.

I plan to send this change to upstream and discuss a bit first, I surmise they will want to make some parts of it configurable.

As @delroth asked for, the last commit contains a patch to Zulip to enable an alerting formatting closer to the one we had on Matrix. I plan to send this change to upstream and discuss a bit first, I surmise they will want to make some parts of it configurable.
We have VMs sitting on the AFNix hypervisor for Lix secret tenancy.
Let the VM pick their tenant themselves.

Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
The `hostId` concept is akin to ZFS' `hostId` and aim to enable VM
identity across virtual machines renames, e.g. ZFS datasets consistency.

In addition, it ensure clash-free network interface names and VSOCK
CIDs.

Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
https://github.com/ryantm/agenix/pull/255
Otherwise, VMs cannot decrypt their secrets at all.

Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
This matches the matrix-hookshot-adapter's formatting made by @K900 more
or less.

Signed-off-by: Raito Bezarius <masterancpp@gmail.com>
raito force-pushed alertmanager-zulip-ng from e397bf005c to a65af4cc0c 2025-08-27 00:38:27 +00:00 Compare
requested review from delroth 2025-08-27 00:38:34 +00:00
raito changed title from WIP: Make Zulip's alertmanager integration closer to matrix-hookshot-adapter's formatting to Make Zulip's alertmanager integration closer to matrix-hookshot-adapter's formatting 2025-08-27 00:38:37 +00:00
delroth approved these changes 2025-08-27 00:43:36 +00:00
delroth left a comment
Owner

Approval conditional on actual testing to make sure the formatting is good - feel free to apply minor changes without asking me for another look.

Approval conditional on actual testing to make sure the formatting is good - feel free to apply minor changes without asking me for another look.
raito force-pushed alertmanager-zulip-ng from a65af4cc0c to f340a3767e 2025-08-27 02:26:32 +00:00 Compare
Author
Owner

Pending final formatting validation

Pending final formatting validation
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin alertmanager-zulip-ng:alertmanager-zulip-ng
git switch alertmanager-zulip-ng

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff alertmanager-zulip-ng
git switch alertmanager-zulip-ng
git rebase main
git switch main
git merge --ff-only alertmanager-zulip-ng
git switch alertmanager-zulip-ng
git rebase main
git switch main
git merge --no-ff alertmanager-zulip-ng
git switch main
git merge --squash alertmanager-zulip-ng
git switch main
git merge --ff-only alertmanager-zulip-ng
git switch main
git merge alertmanager-zulip-ng
git push origin main
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: the-distro/infra#287
No description provided.