Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[venus-messager] 推送过程中出现的错误也更新到数据库当中 #5381

Closed
1 of 11 tasks
Tracked by #5979
hunjixin opened this issue Oct 17, 2022 · 3 comments
Closed
1 of 11 tasks
Tracked by #5979
Labels
C-enhancement Category: feature enhancement P2 Medium - we should get to this soon V-venus-messager venus-messager component

Comments

@hunjixin
Copy link
Contributor

链服务模块 / 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

把推送时候发生的错误更新到数据库当中,这样在命令行,监控面板中可以更直观看到错误原因
需注意是否对mark-bad/list-fail有影响,

@hunjixin hunjixin added C-enhancement Category: feature enhancement P2 Medium - we should get to this soon V-venus-messager venus-messager component labels Oct 17, 2022
@simlecode simlecode added this to the 链服务 v1.9.0 milestone Oct 26, 2022
@hunjixin hunjixin mentioned this issue Nov 7, 2022
45 tasks
@hunjixin
Copy link
Contributor Author

hunjixin commented Nov 7, 2022

#5197

@hunjixin hunjixin added the C-duplicate Category: duplicate issue label Nov 7, 2022
@hunjixin hunjixin closed this as completed Nov 7, 2022
@hunjixin hunjixin moved this to Done in Venus Project Nov 7, 2022
@simlecode simlecode added C-need-testing Category: Venus testing related and removed C-need-testing Category: Venus testing related labels Nov 21, 2022
@hunjixin hunjixin reopened this Dec 19, 2022
Repository owner moved this from Done to In Progress in Venus Project Dec 19, 2022
@hunjixin
Copy link
Contributor Author

目前应该没有完成这个工作,推送多个节点的时候可能有的有错误有的没有错误,这时候记录数据库感觉也不是十分合理。可能需要一个推送记录的功能, 手机推送日志并打印,同时可以供外部接口调用监控。

@Fatman13
Copy link
Contributor

关联 issue 完成了,这个是 dup。

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
C-enhancement Category: feature enhancement P2 Medium - we should get to this soon V-venus-messager venus-messager component
Projects
Archived in project
Development

No branches or pull requests

3 participants