Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

WIP: Run Cloud Controller Manager on AWS #707

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

WIP: Run Cloud Controller Manager on AWS #707

wants to merge 3 commits into from

Conversation

BrainBlasted
Copy link
Contributor

Closes #145

@BrainBlasted BrainBlasted marked this pull request as draft July 7, 2020 20:47
@BrainBlasted
assets: Change hostname for AWS nodes in ignition config
7b43184 
Changes the hostname for AWS clusters to the naming scheme
preferred by Cloud Controller Manager in order to allow us
to set up LoadBalancer services on AWS.

Required for #145
@BrainBlasted
Conditionally set cloud provider flag for CCM setup
c6831d3
@BrainBlasted
Add helm chart for Cloud Controller Manager to AWS config
c282acd
@BrainBlasted BrainBlasted force-pushed the cdavis/aws-load-balancer branch from 3d49618 to c282acd Compare July 7, 2020 20:49
invidian
invidian reviewed Jul 15, 2020
View reviewed changes
Copy link
Member

@invidian invidian left a comment

Choose a reason for hiding this comment

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

We also need CCM as static controlplane pod, otherwise node will never become untained and cluster will not converge.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@invidian invidian invidian left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Run cloud-controller-manager (CCM) on AWS
2 participants
@BrainBlasted @invidian