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

Multiple Google Cloud Platform improvements and new resources #588

Closed
wants to merge 14 commits into from

Conversation

sparkprime
Copy link
Contributor

Added: HttpHealthCheck, ForwardingRule, TargetPool (together they provide network / layer3 load balancing)
Modified Instance: added service account + scopes
Modified Address: added selfUrl

There is currently no documentation, I'd like to get some eyeballs on this first.

@@ -57,3 +60,5 @@ func providerConfigure(d *schema.ResourceData) (interface{}, error) {

return &config, nil
}

// vim: ts=4:sw=4:noet
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please avoid adding these to the source

@armon
Copy link
Member

armon commented Nov 20, 2014

Overall this is coming together nicely!

Stylistically:

  • Prefer to use lowercase with underscore instead of camalCase
  • Remove the "// vim:" lines

Missing:

  • Acceptance tests for all the new resources would be great
  • Documentation for the new resources and new attributes of existing providers

Keep up the good work!

@sparkprime
Copy link
Contributor Author

Does this need anything more? I'd like to start working on #599 and #609 which would clash. I could keep adding to this PR, or create a new one.

@phinze phinze self-assigned this Jan 28, 2015
@phinze
Copy link
Contributor

phinze commented Jan 30, 2015

Closing in favor of the repack in #895 👍

@ghost
Copy link

ghost commented May 3, 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 May 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants