<div class="zcontentRow"><p style="font-size:14px;font-family:arial;"><br></p><div><div class="zhistoryRow" style="display:block"><div id="zwriteHistoryContainer"><div class="control-group zhistoryPanel"><div class="zhistoryContent"><div>On 5/15/2019 4:19 AM, li.canwei2@zte.com.cn wrote:<br>> I tried changing nova_client.api_version to a FloatOpt but that gets<br>> messy because of how things like 2.60 are handled (str(2.60) gets turned<br>> into '2.6' which is not what we'd want). I was hoping we could use<br>> FloatOpt with a min version to enforce the minimum required version, but<br>> I guess we could do this other ways in the client helper code itself by<br>> comparing to some minimum required version in the code.<br>> [licanwei]: Maybe we can refer to <br>> https://github.com/openstack/watcher/blob/master/watcher/common/nova_helper.py#L714 <br>> <br><br>I just did this which seems more explicit:<br><br>https://review.opendev.org/#/c/659194/<br><br>That change leaves the default of 2.56 since the 2.56 code does version <br>discovery so it's backward compatible, but I think we can assert that <br>you need at least 2.53 because of how the scoped nova CDM code works <br>(and to support nova deployments with multiple cells properly).<br><br>Also note that 2.53 is pike-era nova and 2.56 is queens-era nova and <br>those seem old enough that it's safe to require 2.53 as a minimum for <br>watcher in train.<br><p>[licanwei]: Because we need to specify the destination host when migration,</p><p>at least 2.56 is required.</p><p><span style="color: rgb(51, 51, 51); font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, Oxygen-Sans, Ubuntu, Cantarell, 'Helvetica Neue', sans-serif; font-variant-ligatures: normal; orphans: 2; widows: 2; background-color: rgb(255, 255, 255);"> <a href="https://github.com/openstack/watcher/blob/master/watcher/common/nova_helper.py#L145">https://github.com/openstack/watcher/blob/master/watcher/common/nova_helper.py#L145</a></span></p>-- <br><br>Thanks,<br><br>Matt<br></div></div></div></div></div></div><p><br></p></div>