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

chore(docs): 在 code.md 中添加关于 nfs 的描述 #1704

Merged
merged 2 commits into from
Apr 1, 2024

Conversation

BlitherBoom812
Copy link
Contributor

No description provided.

docs/code.md Outdated
@@ -15,6 +15,8 @@ permalink: /code
4. 若语言为解释型语言(`py`),则前端更改数据库`compile_status`为`No Need`(可与第二步合并)
5. 若语言为编译型语言(`cpp`),则前端向后端发请求`/code/compile-start`(见后),使后端开始编译代码
6. 后端下载`cos`上的代码文件,在服务器上启动编译`docker`,并在数据库中更新`compile_status`为`Compiling`
- 选手代码文件在`cos`中,后端需要从`cos`上临时下载队伍的代码到后端服务器上。后端服务器存储空间有限,需要定期清理下载的队伍代码和文件。
Copy link
Contributor

Choose a reason for hiding this comment

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

这两行是不是需要增加缩进呀

Copy link
Contributor Author

Choose a reason for hiding this comment

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

还真是

docs/code.md Outdated
@@ -15,6 +15,8 @@ permalink: /code
4. 若语言为解释型语言(`py`),则前端更改数据库`compile_status`为`No Need`(可与第二步合并)
5. 若语言为编译型语言(`cpp`),则前端向后端发请求`/code/compile-start`(见后),使后端开始编译代码
6. 后端下载`cos`上的代码文件,在服务器上启动编译`docker`,并在数据库中更新`compile_status`为`Compiling`
- 选手代码文件在`cos`中,后端需要从`cos`上临时下载队伍的代码到后端服务器上。后端服务器存储空间有限,需要定期清理下载的队伍代码和文件。
- 后端服务器和`docker`服务器是分离的,`docker`服务器并不能直接获取队伍代码。因此,后端服务器与`docker`服务器之间通过`NFS`进行文件共享,`docker`服务器自动同步了队伍文件。(备注:建议提前服务器之间组内网减少流量费。)
Copy link
Contributor

Choose a reason for hiding this comment

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

建议这边NFS添加链接

@FranGuam FranGuam added this to the Ver. 2024 milestone Apr 1, 2024
@FranGuam FranGuam enabled auto-merge (rebase) April 1, 2024 14:53
@FranGuam FranGuam merged commit ff92570 into eesast:master Apr 1, 2024
2 checks passed
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.

2 participants