<div class="zcontentRow"><p style="font-size:14px;font-family:arial;"><br></p><div><div class="zhistoryRow" style="display:block"><br><div id="zwriteHistoryContainer"><div class="control-group zhistoryPanel"><div class="zhistoryContent"><div>On 5/14/2019 3:34 PM, Matt Riedemann wrote:<br>> 1. In general, a lot of this code for building the compute node model is <br>> based on at least using the 2.53 microversion (Pike) in nova where the <br>> hypervisor.id is a UUID - this is actually necessary for a multi-cell <br>> environment like CERN. The nova_client.api_version config option already <br>> defaults to 2.56 which was in Queens. I'm not sure what the <br>> compatibility matrix looks like for Watcher, but would it be possible <br>> for us to say that Watcher requires nova at least at Queens level API <br>> (so nova_client.api_version >= 2.60), add a release note and a <br>> "watcher-status upgrade check" if necessary. This might make things a <br>> bit cleaner in the nova CDM code to know we can rely on a given minimum <br>> version.<br><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 <a href="https://github.com/openstack/watcher/blob/master/watcher/common/nova_helper.py#L714">https://github.com/openstack/watcher/blob/master/watcher/common/nova_helper.py#L714</a> <br><br>Thanks,<br><br>Matt<br></div></div></div></div></div></div><p><br></p></div>