search

wrfly / container-web-tty

Connect your containers via a web-tty
https://container-web-tty.kfd.me/
Apache License 2.0
246 stars 45 forks source link

[refactor] remove docker & kubectl client #8

Closed wrfly closed 6 years ago

wrfly commented 6 years ago

use docker API and kubectl API instead of cli execution

wrfly commented 6 years ago

emmm，到底是remove mount进去的client，还是重构一下代码，缩减二进制文件大小呢

wrfly commented 6 years ago

反正都挺烦的

wrfly commented 6 years ago

可以尝试重构一下kubernetes的实现，这货太大了，从12M变成了28M。 也可以把go-gin也干掉，毕竟只是一个框架，完全可以用更小的mux替代，gin的优点不明显。 docker的client就留着吧，反正也不大，而且API实现起来比较困难，可以考虑在Makefile中删除不需要的文件，只留下有用的（ls，inspect等）。

尝试把镜像做小。

wrfly commented 6 years ago

需要抓紧了，因为alpine里面挂在进去的docker cli不能用，缺少库

wrfly commented 6 years ago

70fc15680b7106100304e77b60baa65a2a6a8f74

wrfly commented 6 years ago

k8s 的调用的确很烦, 最后是用了一个pipe实现的劫持