Ian Letourneau letian
  • Joined on 2025-04-08
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 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 created branch better-cli in NationTech/harmony 2025-07-27 21:41:53 +00:00
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
letian created pull request NationTech/harmony#90 2025-07-27 15:08:35 +00:00
refactor(topo/k8s_anywhere): simplify local installation of k3d
letian pushed to simply-k3d-installation at NationTech/harmony 2025-07-27 15:05:46 +00:00
9d16e60285 refactor(topo/k8s_anywhere): simplify local installation of k3d
letian created branch simply-k3d-installation in NationTech/harmony 2025-07-27 15:05:46 +00:00
letian deleted branch fix-image-tag from NationTech/harmony 2025-07-27 14:10:45 +00:00
letian pushed to master at NationTech/harmony 2025-07-27 14:10:29 +00:00
d95e84d6fc Merge pull request 'fix(apps/rust): build & push using image tag instead of local VS remote image name' (#87) from fix-image-tag into master
a47be890de Merge branch 'master' into fix-image-tag
f7625f0484 fix(rust): push only the actual image tag
Compare 3 commits »