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

📖 Add alternative communication patterns feature group #7902

Merged
merged 1 commit into from
Jan 26, 2023

Conversation

richardcase
Copy link
Member

What this PR does / why we need it:

This change adds a doc to cover the new "alternative communication patterns" feature group. This covers the scope and use cases that the group will be focused on.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Relates to #6520

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jan 11, 2023
@k8s-ci-robot k8s-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jan 11, 2023
@richardcase richardcase force-pushed the feat_group_proxying branch 2 times, most recently from 023e341 to 71b708f Compare January 11, 2023 15:02
---
# Alternative Communication Patterns Feature Group

This document briefly outlines the scope, communication media, and stakeholders for a formal Feature Group dedicated to defining a Cluster API-approved solution for supporting alternative communication patterns between the workload clusters and the management clusters as alternatives to the current model which is management cluster initiated direct connections.
Copy link
Contributor

Choose a reason for hiding this comment

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

Shall we also include the communication from providers to the infrastructure control plane in a disjoint network?
E.g. CAPV wants to create a cluster in a private vSphere, where the vSphere endpoint is not directly reachable from the management cluster.

Copy link
Member Author

Choose a reason for hiding this comment

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

I agree and this was one of the main use cases from the original issue. I have changed the summary. This use case is listed in the user stories? Are the user stories sufficient to cover this?


1. A solution for managed clusters only, which does not require API-server connectivity from CAPI at all.
1. The "common cluster as a service offering" scenario where only the workload clusters' worker nodes are in a different network.
1. The scenario where a management cluster manages workload clusters (including control plane nodes), which are in a different network than the management cluster.
Copy link
Contributor

Choose a reason for hiding this comment

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

This is an interesting use case for us. We were able to successfully run a POC last year with unmodified CAPI controllers to create a workload cluster with a unidirectional network setup where the workload cluster machines had to establish the connection with the management cluster.
It relied on a SOCKS5 proxy and some additional config changes to generate the correct kubeconfig, but it would be awesome to have a simpler support for this.

I will join the meetings once those start.

Copy link
Member Author

Choose a reason for hiding this comment

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

First meeting is planned for tomorrow (27th Jan 2023 @ 09:00 PT). It would be great to have your experience included in this 😄

@fabriziopandini
Copy link
Member

praise for a well written charter, ping me when you want to merge it

This change adds a doc to cover the new "alternative communication
patterns" feature group.

Signed-off-by: Richard Case <richard.case@suse.com>
@richardcase
Copy link
Member Author

I have made some slight alterations and added the meeting time to the doc. Feel free to comment here or we can discuss in the first meeting tomorrow.

@fabriziopandini - i think this is good to go.

@richardcase richardcase changed the title [WIP] 📖 Add alternative communication patterns feature group 📖 Add alternative communication patterns feature group Jan 26, 2023
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 26, 2023
@fabriziopandini
Copy link
Member

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 26, 2023
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: e62f2bb1a6acef5af9cc95dcabd0823f255fcf6c

@sbueringer
Copy link
Member

Very interesting topic!

/lgtm

@fabriziopandini
Copy link
Member

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fabriziopandini

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 26, 2023
@k8s-ci-robot k8s-ci-robot merged commit 6a600bd into kubernetes-sigs:main Jan 26, 2023
@k8s-ci-robot k8s-ci-robot added this to the v1.4 milestone Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants