NationTech/harmony
5
0
Fork 2
Code Issues 40 Pull Requests 13 Actions Packages Projects Releases Wiki Activity

fix: modified alert receiver trait to allow install plan which provides the topology the ability to apply receiver specfici configurations as required by the underlying alert sender #238

Open
wjro wants to merge 20 commits from fix/refactor_alert_receivers into master
pull from: fix/refactor_alert_receivers
merge into: NationTech:master
Conversation 0 Commits 20 Files Changed 127 +5659 -2740
wjro commented 2026-02-27 16:52:16 +00:00
Owner
Copy Link
No description provided.
wjro added 1 commit 2026-02-27 16:52:16 +00:00
fix: modified alert receiver trait to allow install plan which provides the topology the ability to apply receiver specfici configurations as required by the underlying alert sender
All checks were successful
Run Check Script / check (pull_request) Successful in 1m35s
Details
5c34d81d28
johnride added 43 commits 2026-03-09 21:17:19 +00:00
feat: New harmony node readiness mini project what exposes health of a node on port 25001
All checks were successful
Run Check Script / check (pull_request) Successful in 1m36s
Details
63ef1c0ea7
doc(node-readiness): Fix README
All checks were successful
Run Check Script / check (pull_request) Successful in 1m38s
Details
d084cee8d5
fix: build docker default value
All checks were successful
Run Check Script / check (pull_request) Successful in 1m38s
Details
a1c0e0e246
feat: adding yaml to deploy k8s resources
All checks were successful
Run Check Script / check (pull_request) Successful in 1m37s
Details
b5e971b3b6
feat: Deployment is now happening in harmony-node-healthcheck namespace
All checks were successful
Run Check Script / check (pull_request) Successful in 1m42s
Details
8f8bd34168
feat: node readiness now supports a check query param with node_ready and okd_router_1936 options
All checks were successful
Run Check Script / check (pull_request) Successful in 1m51s
Details
d9a21bf94b
Merge pull request 'feat: New harmony node readiness mini project what exposes health of a node on port 25001' (#237) from feat/harmony-node-health-endpoint into master
Some checks failed
Run Check Script / check (push) Successful in 1m36s
Details
Compile and package harmony_composer / package_harmony_composer (push) Failing after 3m35s
Details
2738985edb 
Reviewed-on: #237
feat(k8s): Can now apply resources of any scope. Kind of a hack leveraging the dynamic type under the hood but this is due to a limitation of kube-rs
Some checks failed
Run Check Script / check (pull_request) Failing after 38s
Details
dca764395d
Merge pull request 'feat(k8s): Can now apply resources of any scope. Kind of a hack leveraging the dynamic type under the hood but this is due to a limitation of kube-rs' (#241) from feat/k8s_apply_any_scope into master
Some checks failed
Run Check Script / check (push) Successful in 2m42s
Details
Compile and package harmony_composer / package_harmony_composer (push) Failing after 4m15s
Details
69c382e8c6 
Reviewed-on: #241
Reviewed-by: stremblay <stremblay@nationtech.io>
feat: implementing node_health 646c5e723e
wip: happy path is working
Some checks failed
Run Check Script / check (pull_request) Failing after 29s
Details
a25ca86bdf
format code, fix interpert name
All checks were successful
Run Check Script / check (pull_request) Successful in 1m33s
Details
d9357adad3
remove useless comment
All checks were successful
Run Check Script / check (pull_request) Successful in 1m43s
Details
6bb33c5845
removing another useless commented line
All checks were successful
Run Check Script / check (pull_request) Successful in 2m17s
Details
20172a7801
Merge pull request 'feat/node-health-score' (#242) from feat/node-health-score into master
Some checks failed
Run Check Script / check (push) Successful in 1m51s
Details
Compile and package harmony_composer / package_harmony_composer (push) Failing after 3m16s
Details
4840c7fdc2 
Reviewed-on: #242
Reviewed-by: johnride <jg@nationtech.io>
wip(secret store): openbao/vault store implementation fd3705e382
wip(secret store): Fix openbao, refactor with rust client ac9fedf853
wip(sso): Openbao deploys fine, not fully tested yet, zitadel wip
All checks were successful
Run Check Script / check (pull_request) Successful in 1m40s
Details
d8338ad12c
Merge pull request 'feat/openbao_secret_manager' (#239) from feat/openbao_secret_manager into master
Some checks failed
Run Check Script / check (push) Successful in 1m35s
Details
Compile and package harmony_composer / package_harmony_composer (push) Failing after 2m36s
Details
fe52f69473 
Reviewed-on: #239
Reviewed-by: stremblay <stremblay@nationtech.io>
wip: zitadel deployment a0c0905c3b
feat: add a background loop checking the node status every X seconds. If NotReady for Y seconds, kill the router pod if there's one 2b157ad7fd
feat: Refactor K8sClient into separate, publishable crate, and add zitadel example 2e1f1b8447
removing bad stuff I did this morning and trying to make it simple, and adding a couple tests d7e5bf11d5
feat: k8s client works fine, added version config in zitadel and fix master key secret existence handling
Some checks failed
Run Check Script / check (pull_request) Failing after 32s
Details
f5aac67af8
... 5db1a31d33
wip: zitadel ingress https not working yet
Some checks failed
Run Check Script / check (pull_request) Failing after 28s
Details
a98113dd40
fix: Node readiness always fails open when kube api call fails on note status check
Some checks failed
Run Check Script / check (pull_request) Failing after 1m54s
Details
f941672662
fix(node health endpoint): Setup sane timeouts for usage as a load balancer health check. The default k8s client timeout of 30 seconds caused haproxy health check to fail even though we still returned 200OK after 30 seconds
Some checks failed
Run Check Script / check (pull_request) Failing after 25s
Details
64893a84f5
fix: change vlan definition from MaybeString to RawXml
All checks were successful
Run Check Script / check (pull_request) Successful in 1m29s
Details
55de206523
fix(zitadel): include admin@zitadel.{host} username, secure password with symbol/number, and cert-manager TLS configuration
Some checks failed
Run Check Script / check (pull_request) Failing after 26s
Details
ce041f495b 
Update Zitadel deployment to use correct username format (admin@zitadel.{host}), generate secure passwords with required complexity (uppercase, lowercase, digit, symbol), configure edge TLS termination for OpenShift, and add cert-manager annotations. Also refactor password generation to ensure all complexity requirements are met.
Fix doc tests for harmony-k8s crate refactoring
All checks were successful
Run Check Script / check (pull_request) Successful in 2m6s
Details
787cc8feab 
- Updated harmony-k8s doc tests to import from harmony_k8s instead of harmony
- Changed CloudNativePgOperatorScore::default() to default_openshift()

This ensures doc tests work correctly after moving K8sClient to the harmony-k8s crate.
Merge pull request 'fix: change vlan definition from MaybeString to RawXml' (#245) from feat/opnsense-config-xml-support-vlan into master
Some checks failed
Run Check Script / check (push) Successful in 1m47s
Details
Compile and package harmony_composer / package_harmony_composer (push) Failing after 2m7s
Details
5412c34957 
Reviewed-on: #245
fix: formatting and check script
All checks were successful
Run Check Script / check (pull_request) Successful in 1m28s
Details
fafca31798
doc: Update node readiness readme and deployed port to 25001
All checks were successful
Run Check Script / check (pull_request) Successful in 1m27s
Details
f532ba2b40
Merge branch 'feat/change-node-readiness-strategy'
Some checks failed
Run Check Script / check (push) Successful in 1m26s
Details
Compile and package harmony_composer / package_harmony_composer (push) Failing after 2m11s
Details
0e9b23a320
okd: add worker nodes to load balancer backend pool
All checks were successful
Run Check Script / check (pull_request) Successful in 1m29s
Details
a93896707f 
Include both control plane and worker nodes in ports 80 and 443 backend pools
okd: include workers in load balancer backend pool + add tests and docs
Some checks failed
Run Check Script / check (pull_request) Failing after 24s
Details
bc2b328296 
- Add nodes_to_backend_server() function to include both control plane and worker nodes
- Update public services (ports 80, 443) to use worker-inclusive backend pool
- Add comprehensive tests covering all backend configurations
- Add documentation with OKD reference link and usage examples
doc: fix example code to use ignore instead of no_run
All checks were successful
Run Check Script / check (pull_request) Successful in 1m43s
Details
d0a1a73710 
-  fails because  cannot be used at module level
- Use  to skip doc compilation while keeping example visible
Merge pull request 'okd: add worker nodes to load balancer backend pool' (#246) from feat/okd-load-balancer-include-workers into master
Some checks failed
Run Check Script / check (push) Successful in 1m30s
Details
Compile and package harmony_composer / package_harmony_composer (push) Failing after 1m53s
Details
e1da7949ec 
Reviewed-on: #246
Fix merge conflict between master and refactor/k8sclient
All checks were successful
Run Check Script / check (pull_request) Successful in 1m28s
Details
f463cd1e94
Merge pull request 'reafactor/k8sclient' (#243) from reafactor/k8sclient into master
Some checks failed
Run Check Script / check (push) Successful in 2m57s
Details
Compile and package harmony_composer / package_harmony_composer (push) Failing after 2m2s
Details
2a7fa466cc 
Reviewed-on: #243
Merge remote-tracking branch 'origin/master' into fix/refactor_alert_receivers 0df4e3cdee
chore: Fix some warnings
Some checks failed
Run Check Script / check (pull_request) Failing after 40s
Details
6db7a780fa
johnride added 1 commit 2026-03-09 21:20:08 +00:00
fix: Finish merging k8s refactoring
All checks were successful
Run Check Script / check (pull_request) Successful in 1m24s
Details
701d86de69
johnride added 1 commit 2026-03-09 22:33:43 +00:00
doc: monitoring module documentation
All checks were successful
Run Check Script / check (pull_request) Successful in 1m23s
Details
af6145afe3
johnride changed target branch from fix/monitoring_abstractions_openshift to master 2026-03-16 18:01:30 +00:00
All checks were successful
Run Check Script / check (pull_request) Successful in 1m23s
Details
This pull request has changes conflicting with the target branch.
  • docs/README.md
  • harmony/src/domain/topology/k8s_anywhere/k8s_anywhere.rs
  • harmony/src/modules/monitoring/grafana/helm/helm_grafana.rs
  • harmony/src/modules/monitoring/kube_prometheus/crd/crd_grafana.rs
  • harmony/src/modules/monitoring/kube_prometheus/crd/rhob_grafana.rs
  • harmony/src/modules/monitoring/okd/mod.rs
  • harmony/src/modules/monitoring/prometheus/prometheus.rs
  • harmony/src/modules/okd/load_balancer.rs
  • harmony/src/modules/postgresql/score_k8s.rs
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/refactor_alert_receivers:fix/refactor_alert_receivers
git checkout fix/refactor_alert_receivers
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Architecture Decision Record
Good first issue
RFC
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
johnride jvtrudel letian stremblay wjro
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: NationTech/harmony#238
No description provided.
Delete Branch "fix/refactor_alert_receivers"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?