Skip to content

Update multicluster-using-statefulsets#1945

Merged
kflynn merged 4 commits intomainfrom
alpeb/mc-statefulsets-2.18
Apr 22, 2025
Merged

Update multicluster-using-statefulsets#1945
kflynn merged 4 commits intomainfrom
alpeb/mc-statefulsets-2.18

Conversation

@alpeb
Copy link
Member

@alpeb alpeb commented Mar 28, 2025

Changed demo repo to https://github.com/linkerd/l2d-k3d-statefulset, which has been updated (linkerdll2d-k3d-statefulset#1) to fit the new multicluster link management in 2.18.

Also fixed a couple other errors.

Changed demo repo to https://github.com/linkerd/l2d-k3d-statefulset,
which has been updated (linkerdll2d-k3d-statefulset#1) to fit the new
multicluster link management in 2.18.

Also fixed a couple other errors.
alpeb added a commit to linkerd/l2d-k3d-statefulset that referenced this pull request Mar 28, 2025
chore: update for linkerd v2.18

- Install GatewayAPI CRDs before installing linkerd
- Added multicluster config (east/mv-values.yml and west/mc-values.yml) to be fed into `linkerd mc install`
- Use `linkerd mc link-gen` instead of `linkerd mc link`
- Other various fixes

Tested against linkerd/website#1945
Once our clusters are created, we will install Linkerd and the multi-cluster
extension. Finally, once both are installed, we need to link the two clusters
together so their services may be mirrored. To enable support for headless
services, we will pass an additional `--set "enableHeadlessServices=true` flag
Copy link
Member

Choose a reason for hiding this comment

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

For my understanding, is this flag no longer required?

Copy link
Member Author

Choose a reason for hiding this comment

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

That setting is a controller setting, so it has been moved to the controllers section in the multicluster extension values.yaml:

E.g. for east, https://github.com/linkerd/l2d-k3d-statefulset/blob/main/east/mc-values.yml :

controllers:
- link:
    ref:
      name: k3d-west
  logLevel: debug
  enableHeadlessServices: true

Copy link
Member

@kflynn kflynn left a comment

Choose a reason for hiding this comment

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

One question but looks good other than that!

Copy link
Member

@kflynn kflynn left a comment

Choose a reason for hiding this comment

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

This looks OK, I think?

@kflynn kflynn merged commit deefbec into main Apr 22, 2025
7 checks passed
@kflynn kflynn deleted the alpeb/mc-statefulsets-2.18 branch April 22, 2025 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants