Ian Letourneau letian
  • Joined on 2025-04-08
letian created pull request NationTech/harmony#129 2025-09-01 11:42:56 +00:00
WIP: fix: merge existing services in load balancer config
letian created branch idempotent-load-balancer in NationTech/harmony 2025-09-01 11:39:59 +00:00
letian pushed to idempotent-load-balancer at NationTech/harmony 2025-09-01 11:39:59 +00:00
e9a1aa4831 fix: merge existing services in load balancer config
letian deleted branch feat/inventory_persistence from NationTech/harmony 2025-08-31 22:49:35 +00:00
letian pushed to master at NationTech/harmony 2025-08-31 22:49:31 +00:00
62fa3c2b10 Merge pull request 'feat: Inventory PhysicalHost persistence with sqlx and local sqlite db' (#125) from feat/inventory_persistence into master
ea1380f98a Merge pull request 'refact: Move basic types to harmony_types crate to avoid external dependencies.' (#126) from refact/harmony_types into feat/inventory_persistence
701d8cfab9 feat: automatically discover inventory (#127)
f9906cb419 refact: Move basic types to harmony_types crate to avoid external dependencies.
cb4382fbb5 feat: Inventory PhysicalHost persistence with sqlx and local sqlite db
Compare 5 commits »
letian merged pull request NationTech/harmony#125 2025-08-31 22:49:29 +00:00
feat: Inventory PhysicalHost persistence with sqlx and local sqlite db
letian deleted branch refact/harmony_types from NationTech/harmony 2025-08-31 22:46:02 +00:00
letian pushed to feat/inventory_persistence at NationTech/harmony 2025-08-31 22:45:59 +00:00
ea1380f98a Merge pull request 'refact: Move basic types to harmony_types crate to avoid external dependencies.' (#126) from refact/harmony_types into feat/inventory_persistence
701d8cfab9 feat: automatically discover inventory (#127)
f9906cb419 refact: Move basic types to harmony_types crate to avoid external dependencies.
Compare 3 commits »
letian merged pull request NationTech/harmony#126 2025-08-31 22:45:58 +00:00
refact: Move basic types to harmony_types crate to avoid external dependencies.
letian deleted branch feat/discover_inventory from NationTech/harmony 2025-08-31 22:45:11 +00:00
letian merged pull request NationTech/harmony#127 2025-08-31 22:45:09 +00:00
feat/discover_inventory
letian pushed to refact/harmony_types at NationTech/harmony 2025-08-31 22:45:09 +00:00
701d8cfab9 feat: automatically discover inventory (#127)
letian opened issue NationTech/harmony#128 2025-08-31 22:10:20 +00:00
Better error handling for DiscoverInventoryAgentIntrepret
letian pushed to feat/discover_inventory at NationTech/harmony 2025-08-31 22:07:49 +00:00
c99e9018af adjust logs/errors
letian commented on pull request NationTech/harmony#127 2025-08-31 22:02:55 +00:00
feat/discover_inventory

There's an edge case here that might break the code:

letian pushed to feat/discover_inventory at NationTech/harmony 2025-08-31 20:59:08 +00:00
57c098a6d8 remove extra info log
letian commented on pull request NationTech/harmony#127 2025-08-31 20:56:30 +00:00
feat/discover_inventory

This log and the one above are very similar

letian pushed to feat/discover_inventory at NationTech/harmony 2025-08-31 16:35:26 +00:00
8bcbe7a226 fix var name and format warning
letian commented on pull request NationTech/harmony#127 2025-08-31 16:29:58 +00:00
feat/discover_inventory

It seems useless but I'm not sure they should actually be merged: to me it feels like two different concepts with a different lifecycle. It is highlighted by the fact that the harmony_inventory_agent is a standalone service that could be consumed by other programs and that this inventory module here is just one of them. The inventory has its own representation of a PhysicalHost that might become different than the one from the harmony_inventory_agent.

letian commented on pull request NationTech/harmony#127 2025-08-31 16:25:10 +00:00
feat/discover_inventory

I'll create an issue about this dependency injection problem we have in our Interprets (and at some extend the Scores statelessness).