[RFE] Remove the manager.py indirection layer

Bug #1965153 reported by Pedro Castillo
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Keystone Charm
New
Undecided
Unassigned

Bug Description

The manager.py indirection layer acts as a type of proxy for keystone that receives JSON-formatted RPC calls through a UNIX socket and forwards them to the keystone server. This layer was added when there was a Python version mismatch between the running charm operator code and the keystone client code.

Since this version mismatch is not relevant anymore given that both keystone and the charm SDK are fully compatible with Python 3, the manager.py indirection layer can be removed and the calls to keystone can be done directly from the charm code without having to spawn a subprocess.

Removing this layer of indirection will make the code a lot more simple to reason about, and would also remove some code that is hard to reason about (see `KeystoneManagerProxy` in `hooks/keystone_utils.py` line 1218).

Revision history for this message
Corey Bryant (corey.bryant) wrote :

Thank you Pedro. This makes sense. I've subscribed Alex as he developed this and would be the best person to triage this.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.