Ian Letourneau

letian commented on pull request NationTech/harmony#86

2025-08-02 04:17:49 +00:00

Monitor an application within a tenant

this Prometheus sender doesn't seem to be used

letian commented on pull request NationTech/harmony#86

2025-08-02 04:09:38 +00:00

Monitor an application within a tenant

install_prometheus seems a bit inaccurate: we're not installing prometheus here, we're installing the receivers to run on prometheus

letian commented on pull request NationTech/harmony#86

2025-08-02 03:54:16 +00:00

Monitor an application within a tenant

I'm not sure it's really a good thing that we try to Serialize the K8sClient 🤔

letian pushed to feat/crd-alertmanager-configs at NationTech/harmony

2025-08-02 03:52:39 +00:00

056152a1e5 remove comment

letian pushed to feat/crd-alertmanager-configs at NationTech/harmony

2025-08-02 03:52:20 +00:00

8d4e06ced1 remove comment

letian pushed to feat/crd-alertmanager-configs at NationTech/harmony

2025-08-02 03:09:27 +00:00

c6b255d0bd merge configure_receiver with AlertReceiver::install & cleanup unused stuff

4b6bebcaf1 remove unnecessary configure_receivers method from trait

961a300154 cleanup unused k3d prometheus monitoring score & simplify design

Compare 3 commits »

letian commented on pull request NationTech/harmony#86

2025-08-01 21:46:51 +00:00

Monitor an application within a tenant

This trait shouldn't be extending PrometheusMonitoring as they're not really related to each other. It's 2 different capabilities.

letian commented on pull request NationTech/harmony#86

2025-08-01 21:38:04 +00:00

Monitor an application within a tenant

This file is almost identical to k8s_prometheus_alerting_score, the only difference being that this one has a namespace attribute whereas the k8s one has a sender that is used to retrieve the namespace.

letian created pull request NationTech/harmony#93

2025-07-31 19:37:41 +00:00

fix(cli): simplify running the CLI by hiding the maestro inside the implemtation

letian pushed to simplify-cli at NationTech/harmony

2025-07-31 19:37:16 +00:00

4ac74bc862 fix(cli): simplify running the CLI by hiding the maestro init inside the implemtation

letian created branch simplify-cli in NationTech/harmony

2025-07-31 19:37:15 +00:00

letian pushed to master at NationTech/harmony

2025-07-31 19:35:39 +00:00

06aab1f57f fix(cli): reduce noise & better track progress within Harmony (#91)

letian merged pull request NationTech/harmony#91

2025-07-31 19:35:36 +00:00

fix(cli): reduce noise & better track progress within Harmony

letian commented on pull request NationTech/harmony#91

2025-07-31 13:32:03 +00:00

fix(cli): reduce noise & better track progress within Harmony

@johnride About this, could we actually reverse things a little bit?

As of now, when we initialize the maestro, it tries to prepare the Topology right away. But because of this, when later on…

letian pushed to better-cli at NationTech/harmony

2025-07-31 13:24:43 +00:00

39208c5e86 Merge branch 'master' into better-cli

1ab66af718 Merge pull request 'refactor(topo/k8s_anywhere): simplify local installation of k3d' (#90) from simply-k3d-installation into master

Compare 2 commits »

letian pushed to better-cli at NationTech/harmony

2025-07-31 13:24:21 +00:00

6b36b1c7e9 rename some event attributes

letian deleted branch simply-k3d-installation from NationTech/harmony

2025-07-31 13:23:16 +00:00

letian pushed to master at NationTech/harmony

2025-07-31 13:22:30 +00:00

1ab66af718 Merge pull request 'refactor(topo/k8s_anywhere): simplify local installation of k3d' (#90) from simply-k3d-installation into master

0fff4ef566 refactor(topo/k8s_anywhere): simplify local installation of k3d

Compare 2 commits »

letian merged pull request NationTech/harmony#90

2025-07-31 13:22:29 +00:00

refactor(topo/k8s_anywhere): simplify local installation of k3d

letian pushed to better-cli at NationTech/harmony

2025-07-31 12:05:22 +00:00

507556969a add k3d todo