Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

listing region availability zones? #320

Closed
zetaab opened this issue Apr 22, 2017 · 8 comments
Closed

listing region availability zones? #320

zetaab opened this issue Apr 22, 2017 · 8 comments

Comments

@zetaab
Copy link

zetaab commented Apr 22, 2017

Is there method for listing availability zones? https://developer.openstack.org/api-ref/compute/?expanded=get-availability-zone-information-detail#get-availability-zone-information

if no, could it be implemented?
@dixudx
Copy link
Contributor

dixudx commented Apr 25, 2017

@zetaab It has already been implemented. Please refer to availabilityzones/requests.go.

@dixudx
Copy link
Contributor

dixudx commented Apr 25, 2017

It is missing in gophercloud godoc. Will add it later.

@dixudx dixudx mentioned this issue Apr 25, 2017
Closed
@dixudx
Copy link
Contributor

dixudx commented Apr 26, 2017

@zetaab availabilityzones/requests.go is shared file system availabilityzones, not compute availabilityzones.

I'll submit a new PR on this new feature. Sorry for misleading you.

@dixudx dixudx mentioned this issue May 2, 2017
Closed
@yongfengdu
Copy link
Contributor

Should this issue being closed since this is tracked in #327?

@jtopjian
Copy link
Contributor

@yongfengdu No, this needs to stay open for two reasons:

  1. All Gophercloud Pull Requests must have a corresponding issue.
  2. compute: Lists and gets detailed availability zone information #327 seems to have gone stale. Likely someone else could/should follow-up with a new PR.

@dstdfx
Copy link
Contributor

dstdfx commented Jan 3, 2018

@jtopjian I'd like to finish this

@jtopjian
Copy link
Contributor

jtopjian commented Jan 3, 2018

@dstdfx Sure, go for it :)

dstdfx added a commit to dstdfx/gophercloud that referenced this issue Jan 3, 2018
@dstdfx
Availability zones info support (gophercloud#320)
a430976 
Add ability to get list and detailed AZs info
@dstdfx dstdfx mentioned this issue Jan 3, 2018
Merged
jtopjian pushed a commit that referenced this issue Jan 8, 2018
@dstdfx @jtopjian
Compute availability zones info support (#704)
b2bf8a6 
* Availability zones info support (#320)

Add ability to get list and detailed AZs info

* Style fixes

* Change test fixture to proper

* Delete useless method of `ServerAvailabilityZoneExt`

* Code improvements

* Change `AvailabilityZoneInfo` type from `struct` to `slice`
* Add custom `UnmarshalJSON` to `AvailabilityZoneInfo`
* Get rid of 'OS' prefixes in variable's names

* Update doc.go

* Code refactoring

* Rename `ServiceOfState` to `ServiceState`
* Change type `Services` from `struct` to `map[string]ServiceState`
* Delete useless type `AvailabilityZoneInfo`
* Update doc.go

* Acceptance tests for #704

* Change struct tag `updated_at` to `-`
@jtopjian jtopjian mentioned this issue Jan 8, 2018
Merged
@jtopjian
Copy link
Contributor

Implemented in #704

@dstdfx Thank you again for your work on this.

cardoe pushed a commit to cardoe/gophercloud that referenced this issue Aug 27, 2020
@jtopjian
Merge pull request gophercloud#320 from jtopjian/vendor-identityv3-en…
a630fb0 
…dpoint-fix

vendor: Updating Gophercloud for Identity endpoint fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@jtopjian @dixudx @yongfengdu @dstdfx @zetaab