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] Configurable AllowableClockDriftSecs / AllowableClockDriftSecs 的值可配置 #5544

Closed
1 of 11 tasks
Tracked by #5580
diwufeiwen opened this issue Dec 7, 2022 · 0 comments
Closed
1 of 11 tasks
Tracked by #5580

[venus] Configurable AllowableClockDriftSecs / AllowableClockDriftSecs 的值可配置 #5544

diwufeiwen opened this issue Dec 7, 2022 · 0 comments
Assignees
@diwufeiwen
Labels
C-enhancement Category: feature enhancement P2 Medium - we should get to this soon
Milestone
Chain Service v1....

Comments

@diwufeiwen
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

AllowableClockDriftSecs=1，目的是为了验证区块的广播时间，维护出块周期合理性，也就是提前1s以上广播到链上的区块不会被承认，但是由于系统时间得误差或有些社区节点自己改了这个时间，经常出现提前5s以上广播的区块最终也被链承认，故：

  1. 允许AllowableClockDriftSecs的值可配置
  2. AllowableClockDriftSecs目前在pkg和venus-shared中都有定义建议统一
@diwufeiwen diwufeiwen added the C-enhancement Category: feature enhancement label Dec 7, 2022
@diwufeiwen diwufeiwen added the P2 Medium - we should get to this soon label Dec 7, 2022
@Fatman13 Fatman13 changed the title [venus] AllowableClockDriftSecs 的值可配置 [venus] Configurable AllowableClockDriftSecs / AllowableClockDriftSecs 的值可配置 Dec 9, 2022
@simlecode simlecode mentioned this issue Dec 19, 2022
Closed
36 tasks
@diwufeiwen diwufeiwen self-assigned this Dec 21, 2022
@diwufeiwen diwufeiwen moved this to In Progress in Venus Project Dec 21, 2022
@diwufeiwen diwufeiwen mentioned this issue Dec 21, 2022
Merged
6 tasks
@diwufeiwen diwufeiwen moved this from In Progress to In Review in Venus Project Dec 21, 2022
Repository owner moved this from In Review to Done in Venus Project Dec 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@diwufeiwen diwufeiwen

Labels
C-enhancement Category: feature enhancement P2 Medium - we should get to this soon
Projects
Venus Project
Archived in project
Milestone
Chain Service v1.10.0 / 链服务v1.10.0
Development

No branches or pull requests
1 participant
@diwufeiwen