johnride
  • Joined on 2024-02-06
johnride commented on pull request NationTech/harmony#159 2025-09-29 13:35:44 +00:00
WIP: configure-switch

The clone has been bugging me too. We could use an Rc or Arc too as a balance between usability and performance.

johnride commented on pull request NationTech/harmony#159 2025-09-29 13:35:44 +00:00
WIP: configure-switch

If I understand correctly this has the correct behavior. It is perfectly normal for a host to have mac addresses that are not connected to the switch. Most of them do have disconnected 1gb interfaces.

johnride commented on pull request NationTech/harmony#159 2025-09-29 13:35:44 +00:00
WIP: configure-switch

I understand we don't want to use the "inventory agent" data structures, but I think it's OK to have one base data structure that we try to reuse across crates for basic components like NetworkInterface and these few others.

johnride commented on pull request NationTech/harmony#159 2025-09-29 13:35:44 +00:00
WIP: configure-switch

I did not review very deeply but still a couple comments.

johnride commented on pull request NationTech/harmony#161 2025-09-29 12:55:40 +00:00
feat: K8sFlavour

Should be OpenshiftFamily(Version) as we want the same behavior between okd and openshift.

johnride commented on pull request NationTech/harmony#161 2025-09-29 12:55:40 +00:00
feat: K8sFlavour

That is a clear naming problem. Should be either k3d for real by detecting it another way or k3sFamily just like OpenshiftFamily

johnride commented on pull request NationTech/harmony#161 2025-09-29 12:55:40 +00:00
feat: K8sFlavour

This is more of a fallback to a default than a true way to detect that we're running vanilla k8s (made by kubeadm for example).

johnride suggested changes for NationTech/harmony#161 2025-09-29 12:55:40 +00:00
feat: K8sFlavour
johnride created pull request NationTech/harmony#162 2025-09-25 01:55:08 +00:00
feat: Add openbao example, open-source fork of vault
johnride commented on pull request NationTech/harmony#159 2025-09-17 13:31:31 +00:00
WIP: configure-switch

I think it is fine for now, but that is vendor specific logic. So it should probably be entirely hidden inside the brocade crate's logic.

johnride commented on pull request NationTech/harmony#159 2025-09-17 13:31:31 +00:00
WIP: configure-switch

Je ne met pas la version dans les dependances locales habituellement

johnride commented on pull request NationTech/harmony#159 2025-09-16 18:11:18 +00:00
WIP: configure-switch

speed yes, maybe we will need something else but nothing comes to mind right now.

johnride commented on pull request NationTech/harmony#159 2025-09-16 18:11:18 +00:00
WIP: configure-switch

LGTM

johnride created pull request NationTech/harmony#159 2025-09-16 18:06:36 +00:00
WIP: configure-switch
johnride released Latest Snapshot at NationTech/harmony 2025-09-12 13:57:47 +00:00
johnride released Latest Snapshot at NationTech/harmony 2025-09-10 19:48:39 +00:00
johnride deleted branch fix/add_routes_to_domain from NationTech/harmony 2025-09-10 19:44:59 +00:00
johnride pushed to master at NationTech/harmony 2025-09-10 19:44:56 +00:00
6e6f57e38c Merge pull request 'fix: added routes to domain name for prometheus, grafana, alertmanageradded argo cd to the reporting after successfull build' (#155) from fix/add_routes_to_domain into master
6f55f79281 feat: Update readme with newer UX/DX Rust Leptos app, update slides and misc stuff
19f87fdaf7 fix: added routes to domain name for prometheus, grafana, alertmanageradded argo cd to the reporting after successfull build
Compare 3 commits »
johnride merged pull request NationTech/harmony#155 2025-09-10 19:44:54 +00:00
fix: added routes to domain name for prometheus, grafana, alertmanageradded argo cd to the reporting after successfull build
johnride pushed to fix/add_routes_to_domain at NationTech/harmony 2025-09-10 19:40:51 +00:00
6f55f79281 feat: Update readme with newer UX/DX Rust Leptos app, update slides and misc stuff