Skip to content

[WIP] Azure Resource Manager: Load Balancers#8130

Closed
tombuildsstuff wants to merge 14 commits into
hashicorp:masterfrom
tombuildsstuff:arm-load-balancers
Closed

[WIP] Azure Resource Manager: Load Balancers#8130
tombuildsstuff wants to merge 14 commits into
hashicorp:masterfrom
tombuildsstuff:arm-load-balancers

Conversation

@tombuildsstuff

@tombuildsstuff tombuildsstuff commented Aug 11, 2016

Copy link
Copy Markdown
Contributor

This is a complete work in progress - opening this so I can keep track of tasks & get some feedback. Load Balancers are available in the Azure SDK for Go - so I'm trying to follow the convention of the other components from this module.

  • Create Load Balancer
  • Get Load Balancer
  • Delete Load Balancer
  • Update the Docs
  • Tests
    • Internal Load Balancer
    • Public Load Balancer
  • Linting / Whitespace (I didn't realise my Atom settings were so far out, I'll fix that in a bit..)

Questions:

  • Given that the internal and external LB's have completely different properties associated with them, is it worth making them separate resources, so we can strongly type them?

Just noticed #8007 after opening this - let me know if one of the team would rather pick this up instead?

@stack72

stack72 commented Oct 3, 2016

Copy link
Copy Markdown
Contributor

Hi @tombuildsstuff

this is going to be seen through to completion by @jen20 and I in #9199

@stack72 stack72 closed this Oct 3, 2016
@tombuildsstuff tombuildsstuff deleted the arm-load-balancers branch October 4, 2016 07:29
@ghost

ghost commented Apr 21, 2020

Copy link
Copy Markdown

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 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants