search

openstack-k8s-operators / dev-docs

Documentation
9 stars 25 forks source link

[spec] Propose EDPM OpenStack service architecture #41

Closed gibizer closed 1 year ago

dprince commented 1 year ago

@gibizer This came across my path today and I was a bit confused why it exists. We have at least 3 or 4 google docs discussing architectural options for Dataplane services. Is this meant to be a consolidation of the Google docs perhaps and you only want comments here now?

gibizer commented 1 year ago

@gibizer This came across my path today and I was a bit confused why it exists. We have at least 3 or 4 google docs discussing architectural options for Dataplane services. Is this meant to be a consolidation of the Google docs perhaps and you only want comments here now?

I'm trying to follow the pattern started in #31. Where after the initial discussion started to settle in the google docs the actual proposal is cleaned up and moved to this repo. This is intentionally in Draft state as I just finished writing this up as a cleanup of the google doc where the question / comments seems to be settled.

I'm traveling this week. So this and #42 will remain in Draft state at least until next week. If you have comments feel free to provide it here or in the google docs (whichever is more convenient for you) in the meantime. I will sync things up next week and we can also chat about the way forward.

booxter commented 1 year ago

I have nothing to add here except to confirm that this is in line with my thinking on next steps for OVNController and Neutron*Agents (and I believe this is also in line with Networking thinking.)

gibizer commented 1 year ago

Moving this back to draft state as there is a decision forming that service operators should never run ansible via directly creating AnsibleEE CR. So the current proposal contradict with this decision https://github.com/openstack-k8s-operators/docs/pull/41/files#diff-b5fbfde815e7f3476b85b68cc2b58a93dce2cbbe2c94bc68a3e55cea541fdf5aR127-R131

gibizer commented 1 year ago

This is not applicable any more based on the decision that sevice-operators are should not run or trigger any ansible execution, and instead DataPlaneService CRs will do that.