search

gophercloud / utils

Apache License 2.0
20 stars 55 forks source link

compute: add helper to return available AZs #130

Closed EmilienM closed 3 years ago

EmilienM commented 3 years ago

Add a new function, named ListAvailableAvailabilityZones, that return a slice of AZs that are actually available.

Note: it uses availabilityzones.List() which doesn't require admin privileges (comparing to availabilityzones.ListDetail() which does).

EmilienM commented 3 years ago

@jtopjian hey, do you think this one would be fine?

jtopjian commented 3 years ago

@EmilienM Yeah, this gophercloud/utils repo is pretty flexible with what is accepted. Let me know if/when this is ready to be merged 🙂

EmilienM commented 3 years ago

@EmilienM Yeah, this gophercloud/utils repo is pretty flexible with what is accepted. Let me know if/when this is ready to be merged slightly_smiling_face

Thanks ! yes this is ready in my opinion, unless someone thinks the function should work otherwise. I'll try to get someone else to review it before EOD but if you don't see any comment feel free to land it.

pierreprinetti commented 3 years ago

:+1: LGTM!

EmilienM commented 3 years ago

Thanks for your quick review @jtopjian