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

proposal: add dynamic edge load balancer support for openyurt (#400) #528

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

proposal: add dynamic edge load balancer support for openyurt (#400) #528

wants to merge 1 commit into from

Conversation

LindaYu17
Copy link
Member

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage
/sig storage

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

@openyurt-bot
Copy link
Collaborator

@LindaYu17: GitHub didn't allow me to assign the following users: your_reviewer.

Note that only openyurtio members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time.
For more information please see the contributor guide

In response to this:

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage
/sig storage

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openyurt-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: LindaYu17
To complete the pull request process, please assign fei-guo
You can assign the PR to them by writing /assign @fei-guo in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openyurt-bot openyurt-bot added the size/L size/L: 100-499 label Oct 21, 2021
rambohe-ch
rambohe-ch reviewed Oct 21, 2021
View reviewed changes
docs/proposals/20211017-adding-dynamic-load-balancer.md Outdated
type UseCase struct {
UseCaseClass string `json:"useCaseClass"` //AI, media, gaming...
UseCaseName string `json:"useCaseName"` //detect, classify...
DeploymentName string `json:"deploymentName"`
Copy link
Member

Choose a reason for hiding this comment

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

I think one DLB instance is used for a kind of pods controlled by Deployment. and StatefulSet or DeamonSet are not supported?

Copy link
Member Author

Choose a reason for hiding this comment

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

StatefulSet and DaemonSet should be supported as well, I will update this parameter.

@rambohe-ch
Copy link
Member

@LindaYu17 please fix markdown lint error.

@LindaYu17 LindaYu17 force-pushed the dlb branch 11 times, most recently from e463a79 to e770010 Compare November 10, 2021 02:16
Merged
6 tasks
@stale
Copy link

stale bot commented Feb 8, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Feb 8, 2022
@rambohe-ch
Copy link
Member

/pinned

@stale stale bot removed the wontfix label Feb 8, 2022
@stale
Copy link

stale bot commented May 9, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label May 9, 2022
@rambohe-ch
Copy link
Member

/pinned

@stale stale bot removed the wontfix label May 10, 2022
@LindaYu17 LindaYu17 changed the title proposal: add dynamic load balancer support for openyurt (#400) proposal: add dynamic edge load balancer support for openyurt (#400) Jun 22, 2022
@LindaYu17 LindaYu17 force-pushed the dlb branch 2 times, most recently from 66ab872 to 8f66453 Compare June 23, 2022 04:41
@stale
Copy link

stale bot commented Sep 21, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Sep 21, 2022
@stale stale bot removed the wontfix label Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rambohe-ch rambohe-ch rambohe-ch left review comments

@Peeknut Peeknut Awaiting requested review from Peeknut

@SataQiu SataQiu Awaiting requested review from SataQiu

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
pinned size/L size/L: 100-499
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@LindaYu17 @openyurt-bot @rambohe-ch