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

Add SMI spec integration - service mesh hub #57

Merged
merged 5 commits into from
Mar 6, 2020

Conversation

bettyjunod
Copy link
Contributor

Page to explain service mesh hub, a reference implementation of SMI spec, how to try it and links to more information on Hub

Page to explain service mesh hub, a reference implementation of SMI spec, how to try it and links to more information on Hub
@msftclas
Copy link

msftclas commented Jun 21, 2019

CLA assistant check
All CLA requirements met.

Copy link
Collaborator

@grampelberg grampelberg left a comment

Choose a reason for hiding this comment

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

Couple requests:

  • Please make this match the other documents (all lower case, markdown, service-mesh-hub.md for example).
  • I know we don't have CI running here yet, but it would be awesome if you passed this through markdownlint.
  • Please add a section and link for this from README.md so that it shows up on the main page!

As a high level, will service mesh hub become part of SMI? We'll need to think about how to message third party solutions if that's not the case.


##Instructions to install Service Mesh Hub##
Install Service Mesh Hub on your kubernetes cluster by running the following command:
```
Copy link
Collaborator

Choose a reason for hiding this comment

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

It'd be great to add language to these (bash) and remove the $.


The Service Mesh Hub is a mesh agnostic dashboard that implements the SMI specification for users to install and operate meshes. A read only version of the Service Mesh Hub can be found on https://servicemeshhub.io

Service Mesh Hub does
Copy link
Collaborator

Choose a reason for hiding this comment

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

Service Mesh Hub does:

$ kubectl port-forward -n sm-marketplace deploy/smm-apiserver 8080
```

Then navigate your browser to:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Then navigate your browser to localhost:8080.

Then navigate your browser to:
```localhost:8080```


Copy link
Contributor Author

Choose a reason for hiding this comment

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

Navigate to Extensions tab and install the SMI Spec extension
http://servicemeshhub.io/extensions/default/smi

@bettyjunod
Copy link
Contributor Author

@grampelberg not so much that Hub becomes a part of SMI but it is an example implementation. Maybe we should consider making an Examples folder so that others in the ecosystem can put in there...WDYT

addressed comments and made edits
removed authoring extensions
added installing the SMI extension
@grampelberg
Copy link
Collaborator

Hmm, couple comments:

  • A folder with just a single file in it feels sad right now. My personal preference would be to keep it to a single file for now until there’s a whole bunch.
  • This feels more like an integration than example to me. Examples feel like some code/manifests that I can run myself and integrations feel more like products that leverage functionality. I don’t feel super passionately about either though and it isn’t hard to fix in the future.

@bettyjunod
Copy link
Contributor Author

@grampelberg for now are we good if I rename from "example" to "integration"

@grampelberg
Copy link
Collaborator

@bettyjunod integrations.md sounds good. Now that lint has merged, that'll hopefully complain about the markdown issues that are there =)

@bettyjunod bettyjunod changed the title Add SMI spec example - service mesh hub Add SMI spec integration - service mesh hub Jul 19, 2019
@bettyjunod
Copy link
Contributor Author

@grampelberg which markdown issues? I need a little help here -- can take to slack if that's easier for chatting

@grampelberg
Copy link
Collaborator

@bettyjunod just that it won't pass lint. It looks like there hasn't been a run yet though, mind pushing a new commit to kick CI off? It should give you all the issues there.

Copy link
Contributor Author

@bettyjunod bettyjunod left a comment

Choose a reason for hiding this comment

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

trying to kick off a new commit

@bettyjunod
Copy link
Contributor Author

/kick

@bridgetkromhout
Copy link
Member

While we're waiting for one more reviewer, let's consider getting this linked from somewhere as well.

@lachie83 lachie83 merged commit b916c12 into servicemeshinterface:master Mar 6, 2020
This was referenced Mar 9, 2020
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.

5 participants