Ian Letourneau letian
  • Joined on 2025-04-08
letian commented on pull request NationTech/harmony#127 2025-08-31 16:22:41 +00:00
feat/discover_inventory

pool is used so we shouldn't need the _

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

Just sharing a thought even if it's not the most representative situation here (considering we're in the infra). Though it still indirectly highlights the underlying issue.

letian commented on pull request NationTech/harmony#127 2025-08-31 15:53:32 +00:00
feat/discover_inventory

A garder un oeil si on aurait besoin de presenter ce summary de manieres differentes (e.g. fichier json). A ce moment-la, il pourrait etre interessant d'y aller avec le pattern de "presenter":

letian commented on pull request NationTech/harmony#125 2025-08-31 15:26:36 +00:00
feat: Inventory PhysicalHost persistence with sqlx and local sqlite db

Do we still need this?

letian commented on pull request NationTech/harmony#125 2025-08-31 15:26:28 +00:00
feat: Inventory PhysicalHost persistence with sqlx and local sqlite db

Could this struct evolve in time and that at some point we update an existing physical host instead of just saving a new one every time?

letian commented on pull request NationTech/harmony#125 2025-08-31 15:12:22 +00:00
feat: Inventory PhysicalHost persistence with sqlx and local sqlite db

should we use the Id type instead of a str here?

letian pushed to doc/pxe_test_setup at NationTech/harmony 2025-08-29 13:52:16 +00:00
b857412151 extract related logic into an OkdIpxeScore
7bb3602ab8 make instrumentation sync instead of async to avoid concurrency issues
Compare 2 commits »
letian pushed to doc/pxe_test_setup at NationTech/harmony 2025-08-29 08:42:30 +00:00
78b80c2169 fix typo in service type
letian pushed to doc/pxe_test_setup at NationTech/harmony 2025-08-28 20:22:02 +00:00
ff2efc0a66 wip: mark DhcpRange fields as optional (to better support OPNSense possible configs)
letian pushed to doc/pxe_test_setup at NationTech/harmony 2025-08-28 18:29:34 +00:00
f180cc4c80 wip: rename harmony-secret* by harmony_secret*
letian created pull request NationTech/harmony#110 2025-08-15 16:50:40 +00:00
fix: bring back the TUI
letian pushed to fix-tui at NationTech/harmony 2025-08-15 16:47:57 +00:00
610ce84280 fix: bring back to TUI
letian created branch fix-tui in NationTech/harmony 2025-08-15 16:47:57 +00:00
letian created pull request NationTech/harmony#109 2025-08-15 15:31:42 +00:00
fix: remove indicatif in harmony_cli to simplify logging and fixing interactions
letian pushed to rip-indicatif at NationTech/harmony 2025-08-15 15:27:08 +00:00
8bb4a9d3f6 fix: remove indicatif in harmony_cli to simplify logging and fixing interactions
letian created branch rip-indicatif in NationTech/harmony 2025-08-15 15:27:07 +00:00
letian pushed to master at NationTech/harmony 2025-08-14 20:44:34 +00:00
67f3a23071 chore: cleanup unused imports
letian deleted branch better-logs from NationTech/harmony 2025-08-14 20:42:19 +00:00
letian pushed to master at NationTech/harmony 2025-08-14 20:42:19 +00:00
d86970f81b fix: make sure demo works on both local & remote target (#107)