Joe Topjian

Results 140 comments of Joe Topjian

@kayrus Understood. I don't think there's an opportunity to turn this into a utility function, do you? If not, do you think this would be useful as documentation or should...

Just to make sure it's asked and a lot of in-depth architectural changes aren't made, what is the use-case that this will solve that can't be solved by a RoundTripper?...

> The difference is when user can inject the context (or provide a RoundTripper) So can a RoundTripper be used? If so, that is entirely possible to do right now....

err - my copying and pasting was incorrect and I pulled the wrong example. I was focusing on Alternative B being discussed in https://github.com/gophercloud/gophercloud/pull/2274: ``` allPages, err := servers.List(client, servers.ListOpts{Context:...

Understood - thanks! I think there are a number of ideas happening in this issue (and in #2274) and it might be worth isolating them for simplicity. 1. Persistent/Shared token...

The Heat/Orchestration code isn't commonly used or well supported nowadays, so there isn't a lot of knowledge about how to apply it to different use-cases. I know Terraform has support...

@jichenjc Thanks! This looks good, but Travis is reporting go format issues. You'll need to run `gofmt` on the modified files. Please let me know if you have any questions.

@jichenjc Thanks, but it looks like there are still failures. Please see the Travis CI logs for more details.

@jichenjc Just a heads-up that the way microversion fields are handled in Gophercloud has changed since this PR was first opened. Please see https://github.com/gophercloud/gophercloud/blob/master/docs/MICROVERSIONS.md for more information and let me...

@jichenjc Thanks. I've just opened an issue about it here: https://github.com/theopenlab/openlab/issues/751