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

Remove Consul from "The Internal flow of processing" section diagram in about.md #3207

Closed
shazron opened this issue Jan 20, 2018 · 5 comments

Comments

@shazron
Copy link
Member

shazron commented Jan 20, 2018

Consul has been removed from OpenWhisk: #2452

The diagram here still shows Consul: https://github.com/apache/incubator-openwhisk/blob/master/docs/about.md#the-internal-flow-of-processing

If there is a source file for the diagram, I can send a pull request.
If just cutting out Consul is acceptable, I can do that also instead of needing the source.

@rabbah
Copy link
Member

rabbah commented Jan 20, 2018

It would be nice to renumber the nodes also. @markusthoemmes might have the source.

@csantanapr
Copy link
Member

@shazron go ahead and create new diagram with draw.io

@shazron
Copy link
Member Author

shazron commented May 10, 2018

How's this for the diagram I created in draw.io? If the numbering is right, I'll send a PR with the source file and PNG (all the logos etc are vector)
openwhisk - the internal flow of processing

@shazron
Copy link
Member Author

shazron commented May 10, 2018

Actually, filed #3638 with the source just in case I don't get back to this issue in a timely manner.

@csantanapr
Copy link
Member

Nice 👍

@shazron shazron closed this as completed May 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants