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

[ELBv3] ability to change eip bandwidth inopentelekomcloud_lb_loadbalancer_v3 #1991

Merged
merged 4 commits into from Nov 15, 2022

Conversation

anton-sidelnikov
Copy link
Member

@anton-sidelnikov anton-sidelnikov commented Nov 15, 2022

Summary of the Pull Request

added suppress funcs to size and type to have ability change eip bandwidth in opentelekomcloud_lb_loadbalancer_v3
removed obsolete tests
possible it is breaking change

PR Checklist

Acceptance Steps Performed


=== RUN   TestAccLBV3LoadBalancer_basic
=== PAUSE TestAccLBV3LoadBalancer_basic
=== CONT  TestAccLBV3LoadBalancer_basic
--- PASS: TestAccLBV3LoadBalancer_basic (146.58s)
=== RUN   TestAccLBV3LoadBalancer_bandwidth
=== PAUSE TestAccLBV3LoadBalancer_bandwidth
=== CONT  TestAccLBV3LoadBalancer_bandwidth
--- PASS: TestAccLBV3LoadBalancer_bandwidth (97.73s)
=== RUN   TestAccLBV3LoadBalancer_import
=== PAUSE TestAccLBV3LoadBalancer_import
=== CONT  TestAccLBV3LoadBalancer_import
--- PASS: TestAccLBV3LoadBalancer_import (123.32s)
PASS


Process finished with the exit code 0

Aloento
Aloento previously approved these changes Nov 15, 2022
@anton-sidelnikov anton-sidelnikov added the gate Merge PR label Nov 15, 2022
@otc-zuul
Copy link

otc-zuul bot commented Nov 15, 2022

Build succeeded.
https://zuul.otc-service.com/t/eco/buildset/1746d078667a459fb9004ef8d0282409

✔️ build-otc-releasenotes SUCCESS in 2m 59s
✔️ otc-golangci-lint SUCCESS in 2m 47s
✔️ golang-make-test SUCCESS in 2m 58s
✔️ golang-make-vet SUCCESS in 2m 25s
✔️ tflint SUCCESS in 1m 41s
✔️ goreleaser-build SUCCESS in 7m 54s

@otc-zuul otc-zuul bot merged commit a165ebf into devel Nov 15, 2022
@otc-zuul otc-zuul bot deleted the elb-v3-bandwidth-recreation branch November 15, 2022 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants