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.

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

Heroku Spaces Support #7059

Closed
ethangunderson opened this issue Jun 7, 2016 · 6 comments · Fixed by #13921
Closed

Heroku Spaces Support #7059

ethangunderson opened this issue Jun 7, 2016 · 6 comments · Fixed by #13921

Comments

@ethangunderson
Copy link

Is support for Heroku Spaces in the works? I searched open issues, but couldn't find anything. I've started work on this already but figured I would check to see if this is on someone else's radar being going any further.

@ethangunderson
Copy link
Author

I started to work on this, and got pretty far. However, the issue I found is that the Heroku API client that Terraform uses appears to be abandoned. More so, the tool that's used to generate the client can no longer handle the schema provided by Heroku. interagent/schematic#29

I've hand edited the client code so that I can continue working on this provider enhancement, but is this a dependency that the Terraform team wishes to keep around? Should I look at replacing it?

@phinze
Copy link
Contributor

phinze commented Jun 30, 2016

Hey hey @ethangunderson! 👋

We are not at all committed to the current dependency for any other reason than it is currently implemented and it passes the tests 😁 . I've bumped into its abandoned-ness before trying to figure out how to get a bug fixed or feature added to the Heroku provider. It's definitely preventing us from making more forward progress on the provider.

If you find a better client for us to lean on, I'd be all for replacing it. 👍

@ethangunderson
Copy link
Author

From what I can find, all of them are abandoned. 😆 😢

@phinze
Copy link
Contributor

phinze commented Jun 30, 2016

Ha welp!

So Heroku publishes a "JSON Schema"

http://json-schema.org/

Is https://github.com/xeipuuv/gojsonschema any help?

@OJFord
Copy link
Contributor

OJFord commented Jan 21, 2017

@phinze That's essentially just a validator as far as I can tell? The broken dependency actually generates the client, but it's choking on (valid) syntax that must be new since whenever it was last run.

This will block other Heroku FRs (#5649; #10350; #11326) - as I said in the latter:

Given that the library's generated from a schema, and that generator has been broken since 2014 (cf. heroku/heroku-go#9; interagent/schematic#29) might it be worth abandoning it in favour of HTTP requests at this point?

Or for new features, at least? I'd be happy to work on one or two if so.

@ghost
Copy link

ghost commented Apr 13, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants