• Joined on 2024-02-08
wjro pushed to feat/crd-alertmanager-configs at NationTech/harmony 2025-07-16 19:56:21 +00:00
114219385f wip:added impl for prometheuses, alertmanagers, prometheusrules, servicemonitors, and some default rules that are deployed for application monitor
wjro pushed to feat/crd-alertmanager-configs at NationTech/harmony 2025-07-14 18:35:16 +00:00
1525ac2226 fix: git conflict
537da5800f uncomment docker image build
3be2fa246c fix: unjank the demo (#85)
Compare 3 commits »
wjro pushed to feat/crd-alertmanager-configs at NationTech/harmony 2025-07-14 18:18:57 +00:00
55a4e79ec4 fix: added updated Cargo
wjro pushed to feat/crd-alertmanager-configs at NationTech/harmony 2025-07-14 17:42:10 +00:00
7b91088828 feat: added impl for webhook receiver for crd alertmanagerconfigs
wjro pushed to feat/crd-alertmanager-configs at NationTech/harmony 2025-07-14 17:07:30 +00:00
e61ec015ab feat: added impl for Discordwebhook receiver to receive application alerts from namespaces from application feature
wjro created pull request NationTech/harmony#86 2025-07-11 20:03:41 +00:00
wip: added an implementation of CRDalertmanagerconfigs that can be used to add a discord webhook receiver, currently the namespace is hard coded and there are a bunch of todos!() that need to be cleaned up, and flags need to be added so that alertmanag…
wjro pushed to feat/crd-alertmanager-configs at NationTech/harmony 2025-07-11 20:02:27 +00:00
819f4a32fd wip: added an implementation of CRDalertmanagerconfigs that can be used to add a discord webhook receiver, currently the namespace is hard coded and there are a bunch of todos!() that need to be cleaned up, and flags need to be added so that alertmanager will automatically register the crd
wjro created branch feat/crd-alertmanager-configs in NationTech/harmony 2025-07-11 20:02:26 +00:00
wjro commented on pull request NationTech/harmony#85 2025-07-10 18:14:14 +00:00
fix: unjank the demo

if we add pub application: Arc to monitoring we can use the application name as namespace so that it is consistent with the way continuous _delivery feature gets the namespace

wjro commented on pull request NationTech/harmony#85 2025-07-10 18:14:14 +00:00
fix: unjank the demo

rather than hardcoding namespace here

wjro approved NationTech/harmony#85 2025-07-10 18:14:14 +00:00
fix: unjank the demo
wjro suggested changes for NationTech/harmony#85 2025-07-10 15:43:39 +00:00
fix: unjank the demo

Just needs some documentation to justify some choices but overall looks good to me

wjro commented on pull request NationTech/harmony#85 2025-07-10 15:42:29 +00:00
fix: unjank the demo

add a comment to justify using bollard vs docker via the cli

wjro commented on pull request NationTech/harmony#85 2025-07-10 15:41:10 +00:00
fix: unjank the demo

I would write some documentation for why you replaced url with string

wjro pushed to feat/monitoring-application-feature at NationTech/harmony 2025-07-09 19:09:49 +00:00
5c628b37b7 wip:added alertreceiver and alert rules which are built and added to the yaml before deploying prometheus, added a few dashboards to grafana. Trying to fix prometheus-server clusterrole/role/serviceaccount so that it can discover targets and kubernetes in a namespaced release where it does not have access to clusterrole
wjro pushed to feat/monitoring-application-feature at NationTech/harmony 2025-07-07 18:33:20 +00:00
31661aaaf1 fix: prometheus deploys as namespaced resource without prometheus-server clusterrole and clusterrolebinding
wjro created branch feat/monitoring-application-feature in NationTech/harmony 2025-07-07 17:25:40 +00:00
wjro pushed to feat/monitoring-application-feature at NationTech/harmony 2025-07-07 17:25:40 +00:00
2c208df143 fix: deploys by default in the application name namespace
wjro pushed to feat/monitoring-application-feature at NationTech/harmony 2025-07-04 20:30:45 +00:00
1a6d72dc17 fix: uncommented example
wjro pushed to feat/monitoring-application-feature at NationTech/harmony 2025-07-04 20:23:11 +00:00
df9e21807e fix: git conflict
a19b52e690 fix: properly append YAML in correct places in argoapplication (#80)
Compare 2 commits »