Skip to content

Multiple RP with scheduler backporting from 1.18.5 - #1097

Merged
centaurus-cloud-bot merged 116 commits into
CentaurusInfra:masterfrom
Sindica:scale-out-poc-530-merge2master-2
Jun 4, 2021
Merged

Multiple RP with scheduler backporting from 1.18.5#1097
centaurus-cloud-bot merged 116 commits into
CentaurusInfra:masterfrom
Sindica:scale-out-poc-530-merge2master-2

Conversation

@Sindica

@Sindica Sindica commented May 29, 2021

Copy link
Copy Markdown
Collaborator

530 release

This code change includes back porting k8s scheduler 1.18.5 and applied the following artkos changes on top of it:
. Multi-tenancy
. VM support & vertical scaling
. Partial runtime readiness
. Scale out support: multiple resource partitions, kubeup/kubemark/proxy/perf test changes, local script for arktos up,

Sindica added 30 commits May 28, 2021 17:38
… and staging/src/k8s.io/component-base/metrics/*_test.go
codec_factory.go, codec_test.go, testing/conversion.go
…bject

Only picked up changes required to compile scheduler
- Due to Scheduler interface change.
. Pick up api violation exception only
TODO: add changes to test/integration/scheduler
Comment thread pkg/scheduler/core/generic_scheduler.go
Comment thread pkg/volume/csi/nodeinfomanager/nodeinfomanager.go
Comment thread test/e2e/framework/ingress/ingress_utils.go
Comment thread test/e2e/storage/csi_mock_volume.go
Comment thread test/e2e/scheduling/priorities.go
Comment thread pkg/scheduler/scheduler.go Outdated
Comment thread cmd/kube-scheduler/app/options/options.go
Comment thread plugin/pkg/auth/authorizer/rbac/bootstrappolicy/policy.go Outdated
@vinaykul

vinaykul commented Jun 3, 2021

Copy link
Copy Markdown
Member

/lgtm

I reviewed specific changes that affect Vertical Scaling and CommonInfo features, and they look fine. I'm counting on fully passing unit tests and E2E tests to ensure this huge change does not regress my earlier work.

@yb01 yb01 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@h-w-chen h-w-chen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@Sindica Sindica closed this Jun 3, 2021
@Sindica Sindica reopened this Jun 3, 2021
@zmn223

zmn223 commented Jun 4, 2021

Copy link
Copy Markdown
Collaborator

/lgtm

@zmn223

zmn223 commented Jun 4, 2021

Copy link
Copy Markdown
Collaborator

/approve

@centaurus-cloud-bot

Copy link
Copy Markdown
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chenqianfzh, h-w-chen, pdgetrf, yb01, zmn223

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

The pull request process is described here

Details 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

@centaurus-cloud-bot
centaurus-cloud-bot merged commit fb66dab into CentaurusInfra:master Jun 4, 2021
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.

9 participants