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.

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

docs: Setting up Cluster Mesh with helm chart / olm #17811

Closed
m-yosefpor opened this issue Nov 8, 2021 · 5 comments
Closed

docs: Setting up Cluster Mesh with helm chart / olm #17811

m-yosefpor opened this issue Nov 8, 2021 · 5 comments
Labels
area/clustermesh Relates to multi-cluster routing functionality in Cilium. area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. area/helm Impacts helm charts and user deployment experience kind/community-report This was reported by a user in the Cilium community, eg via Slack. kind/feature This introduces new functionality.

Comments

@m-yosefpor
Copy link
Contributor

m-yosefpor commented Nov 8, 2021

Proposal / RFE

Is your feature request related to a problem?

Currently there are some docs in cilium website about how to setting up a cluster mesh with cilium cli (https://docs.cilium.io/en/latest/gettingstarted/clustermesh/clustermesh/), however there are no docs which explains how to achieve similar thing via helm charts. Almost all other sections have docs both about CLI solution and helm chart solution, but this part is explained specifically with cli.

The approach explains how to share CA with --inherit-ca flag in installing, and then cilium clustermesh connect --context $CLUSTER1 --destination-context $CLUSTER2 which both of them are not straight forward how can be done in a helm chart deployment.

Describe the solution you'd like

Having a docs about how can we setup cilium in clustermesh mode with helm chart (which would also trivially answers how to do it with OLM, as CilliumConfig corresponds helm chart values.yaml

@m-yosefpor m-yosefpor added the kind/feature This introduces new functionality. label Nov 8, 2021
@pchaigno pchaigno added area/clustermesh Relates to multi-cluster routing functionality in Cilium. area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. area/helm Impacts helm charts and user deployment experience kind/community-report This was reported by a user in the Cilium community, eg via Slack. labels Nov 8, 2021
@samueltorres
Copy link
Contributor

samueltorres commented Nov 10, 2021

I've prepared a PR with the Helm Chart Changes :)
#17851
Tomorrow I'll take care of finishing the PR 👍

@samueltorres
Copy link
Contributor

What do you think about this Helm values schema ?

# -- Clustermesh explicit configuration.
  config:
    # -- Enable the Clustermesh explicit configuration.
    enabled: false
    # -- Default dns domain for the Clustermesh API servers
    # This is used in the case cluster addresses are not provided
    # and IPs are used.
    domain: mesh.cilium.io
    # -- List of clusters to be peered in the mesh.
    clusters: []
    # clusters: 
    # # -- Name of the cluster
    # - name: cluster1
    # # -- Address of the cluster, use this if you created DNS records for
    # # the cluster Clustermesh API server.
    #   address: cluster1.mesh.cilium.io 
    # # -- Port of the cluster Clustermesh API server.
    #   port: 2379
    # # -- IPs of the cluster Clustermesh API server, use multiple ones when
    # # you have multiple IPs to access the Clustermesh API server.
    #   ips:
    #   - 172.18.255.201
    # # -- base64 encoded PEM values for the cluster client certificate, private key and certificate authority.
    #   tls:
    #     cert: ""
    #     key: ""
  • This schema allows a given user to set up the Clustermesh connections manually if needed.
  • It allows each cluster connection to be configured using the Clustermesh load balancer / Node ips or a pre-defined DNS record.
  • The port is configurable
  • The client certificates need to be provided in order to correctly configure the etcd config.
  • It uses the tls.ca.cert CA as the chosen ca for the etcd config.

Is there anything missing in terms of features ?

samueltorres added a commit to samueltorres/cilium that referenced this issue Nov 18, 2021
In order to connect Clustermesh clusters without cilium-cli tool
we would need to manually patch the cilium agent with hostAliases,
configure the cilium-clustermesh secret with mTLS material from
the connected clusters. This commit adds support to connect multiple
Clustermesh clusters using the Helm Chart.

Fixes: cilium#17811

Signed-off-by: Samuel Torres <samuelpirestorres@gmail.com>
@anovv
Copy link

anovv commented Dec 7, 2021

Is it moving?

@github-actions
Copy link

This issue has been automatically marked as stale because it has not
had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Feb 11, 2022
@aanm
Copy link
Member

aanm commented Feb 14, 2022

@dirtyValera yes, in #17851

@github-actions github-actions bot removed the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Feb 15, 2022
nathanjsweet pushed a commit that referenced this issue Feb 23, 2022
In order to connect Clustermesh clusters without cilium-cli tool
we would need to manually patch the cilium agent with hostAliases,
configure the cilium-clustermesh secret with mTLS material from
the connected clusters. This commit adds support to connect multiple
Clustermesh clusters using the Helm Chart.

Fixes: #17811

Signed-off-by: Samuel Torres <samuelpirestorres@gmail.com>
Signed-off-by: Nate Sweet <nathanjsweet@pm.me>
nathanjsweet pushed a commit that referenced this issue Feb 25, 2022
In order to connect Clustermesh clusters without cilium-cli tool
we would need to manually patch the cilium agent with hostAliases,
configure the cilium-clustermesh secret with mTLS material from
the connected clusters. This commit adds support to connect multiple
Clustermesh clusters using the Helm Chart.

Fixes: #17811

Signed-off-by: Samuel Torres <samuelpirestorres@gmail.com>
Signed-off-by: Nate Sweet <nathanjsweet@pm.me>
gandro pushed a commit that referenced this issue Mar 9, 2022
In order to connect Clustermesh clusters without cilium-cli tool
we would need to manually patch the cilium agent with hostAliases,
configure the cilium-clustermesh secret with mTLS material from
the connected clusters. This commit adds support to connect multiple
Clustermesh clusters using the Helm Chart.

Fixes: #17811

Signed-off-by: Samuel Torres <samuelpirestorres@gmail.com>
Signed-off-by: Nate Sweet <nathanjsweet@pm.me>
ti-mo pushed a commit to ti-mo/cilium that referenced this issue Mar 15, 2022
[ upstream commit 4576841 ]

[ Backporter's notes: fixed trailing whitespace in yaml comments ]

In order to connect Clustermesh clusters without cilium-cli tool
we would need to manually patch the cilium agent with hostAliases,
configure the cilium-clustermesh secret with mTLS material from
the connected clusters. This commit adds support to connect multiple
Clustermesh clusters using the Helm Chart.

Fixes: cilium#17811

Signed-off-by: Samuel Torres <samuelpirestorres@gmail.com>
Signed-off-by: Timo Beckers <timo@isovalent.com>
ldelossa pushed a commit that referenced this issue Mar 17, 2022
[ upstream commit 4576841 ]

[ Backporter's notes: fixed trailing whitespace in yaml comments ]

In order to connect Clustermesh clusters without cilium-cli tool
we would need to manually patch the cilium agent with hostAliases,
configure the cilium-clustermesh secret with mTLS material from
the connected clusters. This commit adds support to connect multiple
Clustermesh clusters using the Helm Chart.

Fixes: #17811

Signed-off-by: Samuel Torres <samuelpirestorres@gmail.com>
Signed-off-by: Timo Beckers <timo@isovalent.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/clustermesh Relates to multi-cluster routing functionality in Cilium. area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. area/helm Impacts helm charts and user deployment experience kind/community-report This was reported by a user in the Cilium community, eg via Slack. kind/feature This introduces new functionality.
Projects
None yet
Development

No branches or pull requests

5 participants