<div dir="ltr">Greetings!<div><br></div><div>I have a patch [0] to deprecate the custom HTTPClient implementation that we have in python-ironicclient. Currently it is used when the 'ironic' command is issued in a standalone mode, that is, with '--ironic-url' and '--os-auth-token' parameters, or when a client is instantiated through the following module [1] directly without passing the session object, or even directly calling HTTPClient constructor at [2]. In other cases, keystoneauth's SessionClient derivative object is used. [0] will basically substitute the HTTPClient with the SessionClient (by using the 'admin_token' auth plugin). It seems like a breaking change, as most likely some HTTP error codes and exceptions thrown may be different, so I think we'll need the major client library version bump. We'll also make it clear in the docs that the only "true" way for instantiating the client is through 'get_client' method in [3].</div><div><br></div><div>After that, we'll need to remove the HTTPClient class completely, and here a question is, whether we should have another major version bump? Or can we remove it right away, and single major version bump should be enough? (as the HTTPClient defined in [2] was not something we advertised as a part of our public python API)</div><div><br></div><div>Any suggestions welcome :)</div><div><br></div><div>-Vlad</div><div><br></div><div>[0] <a href="https://review.openstack.org/359061">https://review.openstack.org/359061</a></div><div>[1] <a href="https://github.com/openstack/python-ironicclient/blob/master/ironicclient/v1/client.py">https://github.com/openstack/python-ironicclient/blob/master/ironicclient/v1/client.py</a></div><div>[2] <a href="https://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/http.py">https://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/http.py</a><br></div><div>[3] <a href="https://github.com/openstack/python-ironicclient/blob/master/ironicclient/client.py">https://github.com/openstack/python-ironicclient/blob/master/ironicclient/client.py</a><br></div></div>