<div dir="ltr">On Wed, Mar 8, 2017 at 9:05 AM, Mario Villaplana <span dir="ltr"><<a href="mailto:mario.villaplana@gmail.com" target="_blank">mario.villaplana@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">We want to deprecate ironic CLI soon, but I would prefer if that were<br>
discussed on a separate thread if possible, aside from concerns about<br>
versioning in ironic CLI. Feature parity should exist in Pike, then we<br>
can issue a warning in Queens and deprecate the cycle after. More<br>
information is on L56:<br>
<a href="https://etherpad.openstack.org/p/ironic-pike-ptg-operations" rel="noreferrer" target="_blank">https://etherpad.openstack.<wbr>org/p/ironic-pike-ptg-<wbr>operations</a><br>
<br>
I'm a bit torn on whether to use the API version coded in the OSC<br>
plugin or not. On one hand, it'd be good to be able to test out new<br>
features as soon as they're available. On the other hand, it's<br>
possible that the client won't know how to parse certain items after a<br>
microversion bump. I think I prefer using the hard-coded version to<br>
avoid breakage, but we'd have to be disciplined about updating the<br>
client when the API version is bumped (if needed). Opinions on this<br>
are welcome. In either case, I think the deprecation warning could<br>
land without specifying that.<br></blockquote><div><br></div><div>I agree, I think we should pin it, otherwise it's one more hump to<br></div><div>overcome when we do want to make a breaking change.<br><br></div><div>FWIW, nova pins (both clients) to the max the client knows about,<br>specifically for this reason:<br></div><div><a href="https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/client.py#L52-L57">https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/client.py#L52-L57</a><br><a href="https://github.com/openstack/python-novaclient/blob/master/novaclient/__init__.py#L23-L28">https://github.com/openstack/python-novaclient/blob/master/novaclient/__init__.py#L23-L28</a><br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I'll certainly make an RFE when I update the patch later this week,<br>
great suggestion.<br>
<br>
I can make a spec, but it might be mostly empty except for the client<br>
impact section. Also, this is a < 40 line change. :)<br></blockquote><div><br></div><div>I tend to think a spec is a bit overkill for this, but I won't deny Ruby's request.<br></div><div>Ping me when it's up and I'm happy to review it ASAP. <br><br></div><div>// jim <br></div></div><br></div></div>