Skip to content
This repository has been archived by the owner on Oct 20, 2023. It is now read-only.

Clarify: scope includes but is not limited to k8s. #220

Merged

Conversation

bridgetkromhout
Copy link
Member

Based on the discussion in today's SMI working group for multi-cluster, we want to clarify that SMI includes but is not limited to Kubernetes.

This changes "a specification for service meshes that run on Kubernetes" to "a specification for service meshes, with a
focus on those that run on Kubernetes".

If we can have maintainers indicate their approval of (or changes to) this PR, that will serve to indicate consensus.

Signed-off-by: Bridget Kromhout [email protected]

Copy link
Member

@mhausenblas mhausenblas left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@leecalcote leecalcote left a comment

Choose a reason for hiding this comment

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

LGTM

@nicholasjackson
Copy link
Collaborator

LGTM

Copy link

@mikemorris mikemorris left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Collaborator

@slack slack left a comment

Choose a reason for hiding this comment

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

LGTM!

@stefanprodan stefanprodan merged commit d17d048 into servicemeshinterface:main Apr 29, 2021
Copy link
Contributor

@michelleN michelleN left a comment

Choose a reason for hiding this comment

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

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants