jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Zack Shoylev <notificati...@github.com>
Subject Re: [jclouds] Adding missing "hosts" field to OpenStack Nova AvailabilityZone (#806)
Date Tue, 14 Jul 2015 01:22:07 GMT
Ok. Thanks, ran it and looks good, but I have some questions:

It seems you need to deprecate, because you are using a new AvailabilityZone class. Why not
just update the old availability zone so you don't have to deprecate it? It seems the new
one is just adding stuff, so it should be backwards compatible.

Also, while the rest of nova might not use AutoValue yet, maybe it makes sense to start using
it here.

Let me know if I missed something!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/806#issuecomment-121105564
Mime
View raw message