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.

Sign up for GitHub

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

Jump to bottom

[ELB] Reduce requests load in lb_member_v2 #1678

Merged
merged 1 commit into from Mar 25, 2022
Merged

[ELB] Reduce requests load in lb_member_v2 #1678

merged 1 commit into from Mar 25, 2022

Conversation

outcatcher
Copy link
Contributor

@outcatcher outcatcher commented Mar 25, 2022 •
edited

Summary of the Pull Request

Reduce number of request set to LB service during member creation and deletion

Part of #1671

PR Checklist

Acceptance Steps Performed

Added tests produces more load than actual configuration, making orphan resources. Manual tests were performed for the most recent changes.

lego963
lego963 previously approved these changes Mar 25, 2022
View reviewed changes
@outcatcher outcatcher added gate Merge PR and removed gate Merge PR labels Mar 25, 2022
@outcatcher outcatcher requested a review from lego963 March 25, 2022 16:08
@outcatcher outcatcher marked this pull request as draft March 25, 2022 16:08
@outcatcher outcatcher marked this pull request as ready for review March 25, 2022 17:10
@outcatcher outcatcher added the gate Merge PR label Mar 25, 2022
@outcatcher
Copy link
Contributor Author

regate

@otc-zuul
Copy link

otc-zuul bot commented Mar 25, 2022

Build succeeded.

✔️ build-otc-releasenotes SUCCESS in 8m 44s
✔️ otc-golangci-lint SUCCESS in 8m 23s
✔️ golang-make-test SUCCESS in 8m 34s
✔️ golang-make-vet SUCCESS in 7m 26s
✔️ tflint SUCCESS in 7m 22s
✔️ goreleaser-build SUCCESS in 7m 16s

@otc-zuul otc-zuul bot merged commit 94badfd into devel Mar 25, 2022
@otc-zuul otc-zuul bot deleted the throttle branch March 25, 2022 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lego963 lego963 lego963 approved these changes

@vladimirvshivkov vladimirvshivkov vladimirvshivkov approved these changes

@anton-sidelnikov anton-sidelnikov Awaiting requested review from anton-sidelnikov

Assignees
No one assigned
Labels
gate Merge PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@outcatcher @lego963 @vladimirvshivkov