Ian Letourneau letian
  • Joined on 2025-04-08
letian opened issue NationTech/harmony#94 2025-08-04 19:26:26 +00:00
Introduire un K3d/sTenantManager
letian commented on pull request NationTech/harmony#86 2025-08-04 19:26:02 +00:00
Monitor an application within a tenant

Selon le contexte (e.g. K3dLocal), introduire une nouvelle implementation K3d/sTenantManager qui permet d'aller chercher dans docker l'adresse ip a definir ici.

letian pushed to feat/crd-alertmanager-configs at NationTech/harmony 2025-08-02 15:58:50 +00:00
a8394cda47 simpler check to see if a crd exists + cleanup
letian pushed to feat/crd-alertmanager-configs at NationTech/harmony 2025-08-02 15:11:03 +00:00
064f6d88ba quick cleanup
9403581be5 fix check to ensure prometheus operator is installed
Compare 2 commits »
letian commented on pull request NationTech/harmony#86 2025-08-02 14:08:24 +00:00
Monitor an application within a tenant

is there a reason why these double clone().clone()? there's a lot of them in this file

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 »