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

[GEP-20] Make MCM and CCM components HA #490

Closed
wants to merge 1 commit into from
Closed

[GEP-20] Make MCM and CCM components HA #490

wants to merge 1 commit into from

Conversation

shreyas-s-rao
Copy link

Signed-off-by: Shreyas Rao shreyas.sriganesh.rao@sap.com

How to categorize this PR?

/area control-plane
/area high-availability
/kind enhancement
/platform gcp

What this PR does / why we need it:
This PR makes the machine-controller-manager and cloud-controller-manager extension components highly available, based on the failureToleranceType.

Which issue(s) this PR fixes:
Fixes partially gardener/gardener#6529

Special notes for your reviewer:
/invite @unmarshall @timuthy

This PR needs to be merged only after gardener/gardener#6646 is merged and released and vendored into this repo.
/hold

Release note:

Introduce [TSC](https://kubernetes.io/docs/concepts/scheduling-eviction/topology-spread-constraints/) for `machine-controller-manager` and `cloud-controller-manager` extension components to make them highly available.

@shreyas-s-rao
Make MCM and CCM components HA
370769f 
Signed-off-by: Shreyas Rao <shreyas.sriganesh.rao@sap.com>
@shreyas-s-rao shreyas-s-rao requested review from a team as code owners September 7, 2022 20:41
@gardener-robot gardener-robot added area/control-plane Control plane related area/high-availability High availability related kind/enhancement Enhancement, improvement, extension needs/review Needs review platform/gcp Google cloud platform/infrastructure reviewed/do-not-merge Has no approval for merging as it may break things, be of poor quality or have (ext.) dependencies size/s Size of pull request is small (see gardener-robot robot/bots/size.py) labels Sep 7, 2022
@gardener-robot-ci-2 gardener-robot-ci-2 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Sep 7, 2022
@rfranzke rfranzke marked this pull request as draft September 8, 2022 03:25
@ashwani2k ashwani2k mentioned this pull request Sep 10, 2022
Closed
56 tasks
@timuthy
Copy link
Contributor

timuthy commented Sep 13, 2022

/close
see gardener/gardener#6646 (comment)

@gardener-robot gardener-robot added the status/closed Issue is closed (either delivered or triaged) label Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@timuthy timuthy Awaiting requested review from timuthy

@unmarshall unmarshall Awaiting requested review from unmarshall

Assignees
No one assigned
Labels
area/control-plane Control plane related area/high-availability High availability related kind/enhancement Enhancement, improvement, extension needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) needs/review Needs review platform/gcp Google cloud platform/infrastructure reviewed/do-not-merge Has no approval for merging as it may break things, be of poor quality or have (ext.) dependencies size/s Size of pull request is small (see gardener-robot robot/bots/size.py) status/closed Issue is closed (either delivered or triaged)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@shreyas-s-rao @timuthy @gardener-robot-ci-2 @gardener-robot