Huge progress here, will do a final full review in the skeleton branch with everything together.
● All three deliverables done
feat/iot-helm — 3 new commits on top of the existing PR
┌──────────┬─────────────────────────────────────…
The feature is correct but the implementation of the nats node primitive is wrong, see previous comment.
That should be renamed to something harmony-reconcile-operator or fleet manager or distributed deployment reconcile operator . Capture the essence of the decentralized fleet (iot or datacenters or whatever) management.
This is completely wrong. Do not build a completely new way to deploy nats. Just use the existing deployment method in the module and create a score on top of the same method for the simple deployment.
This one looks good, we might be able to improve type safety a bit but it's already looking good to me.