Ian Letourneau letian
  • Joined on 2025-04-08
letian created pull request NationTech/harmony#174 2025-10-22 19:15:31 +00:00
fix(ha_cluster): inject switch client for better testability
letian pushed to switch-client at NationTech/harmony 2025-10-22 19:12:57 +00:00
7368184917 fix(ha_cluster): inject switch client for better testability
letian created branch switch-client in NationTech/harmony 2025-10-22 19:12:56 +00:00
letian commented on pull request NationTech/harmony#173 2025-10-22 18:55:43 +00:00
WIP: feat: implementation for opnsense os-node_exporter

I think we should leave it to the implementers to define their Debug impl. So on line 6 we should have pub trait NodeExporter: std::fmt::Debug + Send + Sync instead.

letian commented on pull request NationTech/harmony#173 2025-10-22 15:37:15 +00:00
WIP: feat: implementation for opnsense os-node_exporter

Looking good

letian commented on pull request NationTech/harmony#173 2025-10-22 15:34:54 +00:00
WIP: feat: implementation for opnsense os-node_exporter

is it expected to have a line return here?

letian commented on pull request NationTech/harmony#173 2025-10-22 15:34:42 +00:00
WIP: feat: implementation for opnsense os-node_exporter

white line missing

letian commented on pull request NationTech/harmony#173 2025-10-22 15:32:14 +00:00
WIP: feat: implementation for opnsense os-node_exporter

Could you make a quick screen recording of this change in action? 📹

letian pushed to nmstate at NationTech/harmony 2025-10-21 17:42:23 +00:00
035de57925 add missing field kubeconfig
letian closed pull request NationTech/harmony#160 2025-10-21 13:48:50 +00:00
okd_enable_user_workload_monitoring
letian pushed to okd_enable_user_workload_monitoring at NationTech/harmony 2025-10-21 13:48:35 +00:00
05f4769ffe Merge branch 'master' into okd_enable_user_workload_monitoring
ed7f81aa1f fix(opnsense-config): ensure load balancer service configuration is idempotent (#129)
2d891e4463 Merge pull request 'feat(host_network): configure bonds and port channels' (#169) from config-host-network into master
f66e58b9ca Merge branch 'master' into config-host-network
ea39d93aa7 feat(host_network): configure bonds on the host and switch port channels
Compare 12 commits »
letian closed pull request NationTech/harmony#122 2025-10-21 13:39:39 +00:00
WIP: rook-ceph install score
letian closed pull request NationTech/harmony#164 2025-10-21 13:37:37 +00:00
fix: clippy
letian deleted branch idempotent-load-balancer from NationTech/harmony 2025-10-20 19:19:03 +00:00
letian pushed to master at NationTech/harmony 2025-10-20 19:18:56 +00:00
ed7f81aa1f fix(opnsense-config): ensure load balancer service configuration is idempotent (#129)
letian merged pull request NationTech/harmony#129 2025-10-20 19:18:50 +00:00
fix(opnsense-config): ensure load balancer service configuration is idempotent
letian pushed to idempotent-load-balancer at NationTech/harmony 2025-10-20 19:18:11 +00:00
7e70ece201 Merge branch 'master' into idempotent-load-balancer
2d891e4463 Merge pull request 'feat(host_network): configure bonds and port channels' (#169) from config-host-network into master
f66e58b9ca Merge branch 'master' into config-host-network
ea39d93aa7 feat(host_network): configure bonds on the host and switch port channels
6989d208cf Merge pull request 'feat(switch/brocade): Implement client to interact with Brocade switches' (#168) from brocade-switch-client into master
Compare 45 commits »
letian commented on pull request NationTech/harmony#167 2025-10-20 19:10:28 +00:00
feat(application): Webapp feature with production dns

Left a few comments here and there that could be small improvements for a future PR

letian commented on pull request NationTech/harmony#167 2025-10-20 18:59:04 +00:00
feat(application): Webapp feature with production dns

a quick cleanup/refactoring for this function could make things easier to read/understand, for example by breaking down the different steps into different (module private) functions

letian commented on pull request NationTech/harmony#167 2025-10-20 18:55:45 +00:00
feat(application): Webapp feature with production dns

outdated comment -> to be removed