Skip to content

[WIP] reword/improve ingress documentation#3173

Closed
jschaul wants to merge 2 commits intodevelopfrom
docs-ingress-update
Closed

[WIP] reword/improve ingress documentation#3173
jschaul wants to merge 2 commits intodevelopfrom
docs-ingress-update

Conversation

@jschaul
Copy link
Member

@jschaul jschaul commented Mar 22, 2023

WIP

Based on:

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Mar 22, 2023
@jschaul jschaul mentioned this pull request Mar 22, 2023
2 tasks
Most managed K8s clusters have support for LoadBalancers. Kubernetes clusters that are manually set up can also support this, by using a provider/environment-specific [CCM (Cloud Controller Manager)](https://kubernetes.io/docs/concepts/architecture/cloud-controller/) (see hcloud and digitalocean for examples).

In case you're provisioning on bare metal, without any hardware load balancer support in front,
you might be using MetalLB, which supports BGP or Failover [L2 ARP announcements](https://metallb.universe.tf/configuration/_advanced_l2_configuration/).
Copy link
Contributor

Choose a reason for hiding this comment

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

Looking at the ticket description, we probably shouldn't be linking to any of this repositories, but rather to https://kubernetes.github.io/ingress-nginx/deploy/baremetal/. We can still mention these as examples, but that should be it.

Copy link
Contributor

Choose a reason for hiding this comment

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

Can you remove this paragraph describing MetalLB specifically? That's all included in https://kubernetes.github.io/ingress-nginx/deploy/baremetal/.

@CLAassistant
Copy link

CLAassistant commented May 10, 2023

CLA assistant check
All committers have signed the CLA.

@jschaul jschaul closed this Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments