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.

Sign up for GitHub

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

Jump to bottom

[Venus]：more sanity check before upping / 对于特殊配置项在服务启动时进行验证 #5566

Open
3 of 11 tasks
Tracked by #5979
diwufeiwen opened this issue Dec 14, 2022 · 2 comments
Open
3 of 11 tasks
Tracked by #5979

[Venus]：more sanity check before upping / 对于特殊配置项在服务启动时进行验证 #5566

diwufeiwen opened this issue Dec 14, 2022 · 2 comments
Labels
C-enhancement Category: feature enhancement P2 Medium - we should get to this soon

Comments

@diwufeiwen
Copy link
Contributor

diwufeiwen commented Dec 14, 2022 •
edited
Loading

链服务模块 / Chain Service Components

  • venus
  • venus-auth
  • venus-gateway
  • venus-messager
  • venus-miner
  • 文档 / docs

订单服务模块 / Deal Service Components

  • venus-market
  • 文档 / docs

算力服务模块 / Storage Power Service Components

  • venus-sector-manager
  • venus-worker
  • 文档 / docs

描述 / Description

（1）有些配置有多个选择项，比如venus-market的签名配置；
（2）有些配置可以有多项，比如venus-miner或venus-auth配置的gateway；
在服务启动时进行验证，以免后续执行到具体逻辑时失败，即让因为配置错误可能造成的错误在启动时就暴露出来
@diwufeiwen diwufeiwen added C-enhancement Category: feature enhancement P1 High - we should be working on this now or in the immediate future labels Dec 14, 2022
@hunjixin
Copy link
Contributor

  • market-client 在不指定auth地址的情况下，默认本地部署的情况，禁用gateway类型的签名方式
  • venus-market至少需要一个piece目录才能启动。 但是由于默认的是没有piece目录，这样会导致默认的无法启动，解决办法是venus-market初始化的时候在home目录下默认增加一个fs类型的， 文件夹名称是piecestore，name是local+随机数的piece存储目录。

@hunjixin
Copy link
Contributor

hunjixin commented Dec 14, 2022 •
edited
Loading

@cloudxin @waleyhi

@Fatman13 Fatman13 changed the title [Venus]：对于特殊配置项在服务启动时进行验证 [Venus]：more sanity check before upping / 对于特殊配置项在服务启动时进行验证 Dec 16, 2022
@hunjixin hunjixin added P2 Medium - we should get to this soon and removed P1 High - we should be working on this now or in the immediate future labels Dec 19, 2022
@Fatman13 Fatman13 mentioned this issue May 22, 2023
Closed
26 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-enhancement Category: feature enhancement P2 Medium - we should get to this soon
Projects
Venus Project
Status: No status
Milestone
No milestone
Development

No branches or pull requests
2 participants
@diwufeiwen @hunjixin