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

[venus-messager] venus-messager中错误信息和返回值都是保存在return字段里面的,现在需要吧这两个信息分离开。 #5197

Closed
1 of 11 tasks
Tracked by #5449
hunjixin opened this issue Aug 17, 2022 · 1 comment
Assignees
Labels
C-enhancement Category: feature enhancement C-need-testing Category: Venus testing related CU-chain-service Venus chain service related issues 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

增加一个字段用于保存错误信息

  1. 一个字段保存两种信息有歧义
  2. 用户需要查看(包括命令,页面监控工具)错误信息的时候还需要额外的判断,在有的工具(grafana)下无法完成这种判断
@hunjixin hunjixin added C-enhancement Category: feature enhancement P2 Medium - we should get to this soon V-venus-messager venus-messager component CU-chain-service Venus chain service related issues labels Aug 17, 2022
@LinZexiao LinZexiao self-assigned this Oct 18, 2022
@LinZexiao LinZexiao moved this to In Progress in Venus Project Oct 20, 2022
@LinZexiao LinZexiao added the C-Blocked Category: temporarily stuck without a good solution label Oct 24, 2022
@simlecode simlecode added this to the 链服务 v1.9.0 milestone Oct 26, 2022
@LinZexiao LinZexiao removed this from Venus Project Nov 3, 2022
@simlecode simlecode added C-need-testing Category: Venus testing related and removed C-Blocked Category: temporarily stuck without a good solution labels Nov 21, 2022
@YiRuiLiher
Copy link
Collaborator

YiRuiLiher commented Dec 26, 2022

ErrorMsg 没有被定义到消息的结构体中,所以只有 以 table 形式输出的情况下次才可以看到 ErrorMsg 这个字段,search 消息的时候 不显示具体错误,这里应该要完善一下

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: feature enhancement C-need-testing Category: Venus testing related CU-chain-service Venus chain service related issues P2 Medium - we should get to this soon V-venus-messager venus-messager component
Projects
None yet
Development

No branches or pull requests

4 participants