search

GoogleCloudPlatform / k8s-multicluster-ingress

kubemci: Command line tool to configure L7 load balancers using multiple kubernetes clusters
Apache License 2.0
376 stars 68 forks source link

consolidate demo/README.md and examples/zone-printer/README.md #161

Open ahmetb opened 6 years ago

ahmetb commented 6 years ago

I think you should move zone-printer/README.md to demo/.

We can easily change zone-printer with a different app that no longer uses nginx+configmap and the demo/sample/example isn't much about the app itself, but rather instructions.

ahmetb commented 6 years ago

I understand demo/ is for bash prompt automation. Consider moving it out of project root. It's not an important component and users will not use it.

nikhiljindal commented 6 years ago

Yes demo/ has scripts that people can use to demo kubemci. We could move it out of the repo, it is there for discoverability.

I dont see this as an important issue.