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

低版本docker命令 #113

Open
wants to merge 4 commits into
base: feature
Choose a base branch
from

Conversation

CharlieKoo
Copy link
Contributor

低版本docker命令

README_zh.md Outdated
- Tips:
低版本的docker
- 需要添加参数```--security-opt seccomp:unconfined```这是因为numpy在低版本会报Operation not permitted的错误
- 将```-c http://host.docker.internal:8001/```改为```-c -c http://172.17.0.1:8001/```这是因为低版本docker不支持```host.docker.internal```访问宿主机ip
Copy link
Contributor

Choose a reason for hiding this comment

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

有两个-c,是不是笔误了?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

是的,这里我再改一下。

README.md Outdated
```
docker run -p 8002:8002 -d mybigpai-public-registry.cn-beijing.cr.aliyuncs.com/mybigpai/pairag:0.0.2_ui pai_rag ui -p 8002 -c http://host.docker.internal:8001/
```
- Tips: For lower versions of Docker
Copy link
Contributor

Choose a reason for hiding this comment

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

lower version具体是指什么版本?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

我们的机器比较老,目前测试的是20.10.7

@@ -178,21 +178,25 @@ To make it easier to use and save time on environment installation, we also prov
```

2. RAG UI
Linux:

- Linux:
Copy link
Contributor

Choose a reason for hiding this comment

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

这些代码都有缩近,和原来的md编写风格改变了,如果要改,建议整体全改,或者另外起一个PR, @wwxxzz 帮忙一起看下

Copy link
Contributor Author

Choose a reason for hiding this comment

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

我看了一下排版
image
为了迎合这个排版所以改动了缩进

Copy link
Contributor Author

Choose a reason for hiding this comment

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

新的排版我本地渲染的效果是这样的
image

README_zh.md Outdated
- Tips:
低版本的docker
- 需要添加参数```--security-opt seccomp:unconfined```这是因为numpy在低版本会报Operation not permitted的错误
- 将```-c http://host.docker.internal:8001/```改为```-c -c http://172.17.0.1:8001/```这是因为低版本docker不支持```host.docker.internal```访问宿主机ip
Copy link
Contributor

Choose a reason for hiding this comment

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

172.17.0.1这个是标准的内网ip地址吗?还是和环境有关

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这个地址是老版本docker的默认网桥,高版本的host.docker.internal->对应低版本的172.17.0.1,如果用户不做修改的话,这个就是用来访问宿主机的

Copy link
Contributor Author

Choose a reason for hiding this comment

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

实际上高版本也能访问172.17.0.1,你可以ip addr docker0看一下

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.

3 participants