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

feat(kuma-cp) serve gui server #410

Merged
merged 4 commits into from
Nov 13, 2019
Merged

Conversation

jakubdyszkiewicz
Copy link
Contributor

Summary

Server for Kuma GUI. It is exposed on 5683 port and by default expose stuff under gui directory in the directory of kuma-cp binary.

Questions

How to deploy this?
On Universal, I guess we will build files from kuma-gui and then include gui directory in final package.
What about K8S? Should we build docker image with kuma-gui and then start it in the same pod as kuma-cp?

@jakubdyszkiewicz jakubdyszkiewicz requested a review from a team November 7, 2019 11:58
pkg/config/gui-server/config.go Outdated Show resolved Hide resolved
pkg/config/app/kuma-cp/kuma-cp.defaults.yaml Outdated Show resolved Hide resolved
pkg/config/app/kuma-cp/default_config_test.go Outdated Show resolved Hide resolved
@jakubdyszkiewicz jakubdyszkiewicz force-pushed the feature/static-server branch 2 times, most recently from 1a1b31f to 8c9483e Compare November 12, 2019 11:55
Makefile Outdated Show resolved Hide resolved
pkg/gui/guidir.go Outdated Show resolved Hide resolved
pkg/gui/guidir_dev.go Outdated Show resolved Hide resolved
pkg/gui/server.go Outdated Show resolved Hide resolved
gui/data.js Outdated Show resolved Hide resolved
gui/index.html Outdated Show resolved Hide resolved
@@ -232,6 +232,9 @@ generate/kumactl/install/control-plane:
go generate ./app/kumactl/pkg/install/k8s/control-plane/...
go generate ./app/kumactl/pkg/install/universal/control-plane/postgres/...

generate/gui: ## Generate go files with GUI static files to embed it into binary
Copy link
Contributor

Choose a reason for hiding this comment

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

add generate/gui to the .PHONY list at the top

@jakubdyszkiewicz jakubdyszkiewicz merged commit 366106e into master Nov 13, 2019
@jakubdyszkiewicz jakubdyszkiewicz deleted the feature/static-server branch December 23, 2019 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants