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

Add Vip addresses & InstanceEndpoints to VMs #18

Merged
merged 1 commit into from
Jan 29, 2015
Merged

Add Vip addresses & InstanceEndpoints to VMs #18

merged 1 commit into from
Jan 29, 2015

Conversation

keithchambers
Copy link
Contributor

Credit to @frodenas for inspiration.

CLA has been signed.

@@ -171,6 +186,14 @@ type RoleSize struct {
VirtualMachineResourceDiskSizeInMb int
}

type VirtualIPs struct {
VirtualIP []*VirtualIP
Copy link
Contributor

Choose a reason for hiding this comment

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

why *?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks ... removed.

@ahmetb
Copy link
Contributor

ahmetb commented Jan 29, 2015

@keithchambers please squash the additional fixes into one commit (in this case your initial commit) and git push -f to your branch.

cc: @hopetobelievein @paulmey @jeffmendoza for review

@keithchambers
Copy link
Contributor Author

Thank you Ahmet.

I have now squashed the commits and pushed to my branch.

@jeffmendoza
Copy link

Looks good

@ahmetb
Copy link
Contributor

ahmetb commented Jan 29, 2015

Lgtm as well

jeffmendoza pushed a commit that referenced this pull request Jan 29, 2015
Add Vip addresses & InstanceEndpoints to VMs
@jeffmendoza jeffmendoza merged commit 930a296 into Azure:master Jan 29, 2015
richardpark-msft pushed a commit to richardpark-msft/azure-sdk-for-go that referenced this pull request Aug 5, 2021
add travis integration build and badges
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants