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

LBAAS-3620: support load balancers network_stack #1648

Merged

Conversation

jvasilevsky
Copy link
Contributor

support load balancers network_stack

@jvasilevsky jvasilevsky force-pushed the jvasilevsky/LBAAS-3620-doctl branch from a206d2c to f118030 Compare January 27, 2025 20:19
@jvasilevsky jvasilevsky marked this pull request as ready for review January 27, 2025 20:22
@jvasilevsky jvasilevsky force-pushed the jvasilevsky/LBAAS-3620-doctl branch 3 times, most recently from 37c55a2 to 6d19080 Compare January 27, 2025 20:35
@jvasilevsky jvasilevsky force-pushed the jvasilevsky/LBAAS-3620-doctl branch from 6d19080 to f8958e1 Compare January 27, 2025 21:31
Copy link
Member

@bentranter bentranter left a comment

Choose a reason for hiding this comment

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

Change looks good overall, just one nitpick about the updated product lifecycle language, after that I'll merge.

@@ -101,6 +101,8 @@ With the load-balancer command, you can list, create, or delete load balancers,
AddStringSliceFlag(cmdLoadBalancerCreate, doctl.ArgTargetLoadBalancerIDs, "", []string{},
"A comma-separated list of Load Balancer IDs to add as target to the global load balancer ")
AddStringFlag(cmdLoadBalancerCreate, doctl.ArgLoadBalancerNetwork, "", "", "The type of network the load balancer is accessible from, e.g.: `EXTERNAL` or `INTERNAL`")
AddStringFlag(cmdLoadBalancerCreate, doctl.ArgLoadBalancerNetworkStack, "", "", "The network stack type determines the allocation of ipv4/ipv6 addresses to the load balancer, e.g.: `IPV4` or `DUALSTACK`"+
" (NOTE: this is a closed beta feature, contact DigitalOcean support to review its public availability.)")
Copy link
Member

Choose a reason for hiding this comment

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

Should we use the updated product lifecycle language here?

Suggested change
" (NOTE: this is a closed beta feature, contact DigitalOcean support to review its public availability.)")
" (NOTE: this is feature is in private preview, contact DigitalOcean support to review its public availability.)")

@bentranter bentranter merged commit aac3476 into digitalocean:main Jan 28, 2025
8 checks passed
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