Ian Letourneau letian
  • Joined on 2025-04-08
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
letian opened issue NationTech/harmony#92 2025-07-31 12:03:56 +00:00
K3D: better verification that the K8S Client was successfully created
letian pushed to better-cli at NationTech/harmony 2025-07-31 01:42:19 +00:00
68fde23f2c remove unused inquire dependency for Harmony
letian commented on pull request NationTech/harmony#91 2025-07-31 01:39:31 +00:00
fix(cli): reduce noise & better track progress within Harmony

some kind of retry mechanism could be useful here to make sure no events are lost

letian pushed to better-cli at NationTech/harmony 2025-07-31 01:34:47 +00:00
49f1e56599 add event to track progress of interprets, change a bunch of info! to debug!
letian pushed to better-cli at NationTech/harmony 2025-07-30 16:22:19 +00:00
ff7801a7c1 ensure event handlers are properly subscribed on init, extract duplicated progress functions, cleanup duplication
letian commented on pull request NationTech/harmony#91 2025-07-28 02:12:07 +00:00
fix(cli): reduce noise & better track progress within Harmony

cargo prints warnings and other stuff in stderr, meaning it can get really noisy. So we'll silence those warnings unless we're asking for debug logs (RUST_LOG=debug)

letian commented on pull request NationTech/harmony#91 2025-07-28 02:10:19 +00:00
fix(cli): reduce noise & better track progress within Harmony

Some instrumentations are still missing, but this PR is mostly to get early feedback on the approach.

letian commented on pull request NationTech/harmony#91 2025-07-28 02:09:29 +00:00
fix(cli): reduce noise & better track progress within Harmony

I'm not a big fan of "duplicating" all the stuff below. It is for different concerns than the ones defined in harmony_cli, but it follows the same pattern.

letian commented on pull request NationTech/harmony#91 2025-07-28 02:07:12 +00:00
fix(cli): reduce noise & better track progress within Harmony

See the PR comment above (in examples/rust/main.rs) for more info

letian commented on pull request NationTech/harmony#91 2025-07-28 02:05:55 +00:00
fix(cli): reduce noise & better track progress within Harmony

Ideally, this shouldn't have to be handled by the end user. But in this case, because Maestro::initialize prepares the Topology (which might execute Scores), we have to register the cli_logger early.

letian pushed to better-cli at NationTech/harmony 2025-07-28 00:52:40 +00:00
8fae9cf8c8 split instrumentation in 2 different places: harmony domain (for domain observability) & harmoy composer (for build/commands observability)
letian created pull request NationTech/harmony#91 2025-07-27 21:43:04 +00:00
WIP: fix(cli): reduce noise & better track progress within Harmony
letian created branch better-cli in NationTech/harmony 2025-07-27 21:41:53 +00:00
letian pushed to better-cli at NationTech/harmony 2025-07-27 21:41:53 +00:00
6f7e1640c1 fix(cli): reduce noise & better track progress within Harmony
letian pushed to simply-k3d-installation at NationTech/harmony 2025-07-27 15:51:01 +00:00
0fff4ef566 refactor(topo/k8s_anywhere): simplify local installation of k3d