volcano icon indicating copy to clipboard operation
volcano copied to clipboard

feat:adding sidecar

Open 7h3-3mp7y-m4n opened this issue 1 year ago • 5 comments

This added a feature of sidecar Fixes #3513

7h3-3mp7y-m4n avatar Jul 14 '24 19:07 7h3-3mp7y-m4n

Welcome @7h3-3mp7y-m4n!

It looks like this is your first PR to volcano-sh/volcano.

Thank you, and welcome to Volcano. :smiley:

volcano-sh-bot avatar Jul 14 '24 19:07 volcano-sh-bot

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: To complete the pull request process, please assign wpeng102 You can assign the PR to them by writing /assign @wpeng102 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

volcano-sh-bot avatar Jul 14 '24 19:07 volcano-sh-bot

Unit test is need!

lowang-bh avatar Jul 15 '24 02:07 lowang-bh

We should also add filter logic like https://github.com/kubernetes/kubernetes/blob/f8d5b2074c9b3a1629ef085b9e17a5370f393732/pkg/scheduler/framework/plugins/noderesources/fit.go#L217.

Monokaix avatar Aug 01 '24 01:08 Monokaix

We should also add filter logic like https://github.com/kubernetes/kubernetes/blob/f8d5b2074c9b3a1629ef085b9e17a5370f393732/pkg/scheduler/framework/plugins/noderesources/fit.go#L217.

okay :)

7h3-3mp7y-m4n avatar Aug 04 '24 20:08 7h3-3mp7y-m4n

Hi, since it's not active and we will release v1.10 soon so I fire a new PR based on your work and your commit is also inclueded, thanks for your contribution!

Monokaix avatar Sep 03 '24 12:09 Monokaix

@7h3-3mp7y-m4n: PR needs rebase.

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.

volcano-sh-bot avatar Sep 07 '24 06:09 volcano-sh-bot

/close

lowang-bh avatar Sep 07 '24 07:09 lowang-bh

@lowang-bh: Closed this PR.

In response to this:

/close

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.

volcano-sh-bot avatar Sep 07 '24 07:09 volcano-sh-bot