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

Update lb_loadbalancer_v3.md #2385

Merged
merged 6 commits into from
Dec 5, 2023
Merged

Update lb_loadbalancer_v3.md #2385

merged 6 commits into from
Dec 5, 2023

Conversation

dombisza
Copy link
Contributor

@dombisza dombisza commented Dec 1, 2023

Summary of the Pull Request

hello there,

I have added a hint about l4_flavor and l7_flavor, because the behaviour was really confusing at first.
How it is actually working is:

l4_flavor and l7_flavor empty

  • both will be attached with the default flavor (doubles your ELB cost)

l4_flavor or l7_flavor is specified

  • only the one you specify gets attached to the ELB

PR Checklist

  • Refers to: #xxx
  • Tests added/passed.
  • Documentation updated.
  • Schema updated.
  • Release notes added.

Acceptance Steps Performed

=== RUN   TestAccSomethingV0_basic
--- PASS: TestAccSomethingV0_basic (101.71s)
=== RUN   TestAccSomethingV0_timeout
--- PASS: TestAccSomethingV0_timeout (128.67s)
PASS

Process finished with exit code 0

@anton-sidelnikov anton-sidelnikov added the gate Merge PR label Dec 5, 2023
Copy link

otc-zuul bot commented Dec 5, 2023

Build succeeded.
https://zuul.otc-service.com/t/eco/buildset/70ca7b9000f3403c9ce711afd9a6cda7

✔️ build-otc-releasenotes SUCCESS in 6m 33s
✔️ noop SUCCESS in 0s
✔️ otc-golangci-lint SUCCESS in 3m 06s
✔️ golang-make-test SUCCESS in 5m 01s
✔️ golang-make-vet SUCCESS in 3m 52s
✔️ tflint SUCCESS in 3m 10s
✔️ goreleaser-build SUCCESS in 7m 15s
✔️ otc-terraform-visualize-main SUCCESS in 6m 23s

@otc-zuul otc-zuul bot merged commit 8c4a2b2 into opentelekomcloud:devel Dec 5, 2023
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gate Merge PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants