Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

sc-informant: Do not show Block history if doing major sync #14094

Merged
merged 1 commit into from
May 8, 2023

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented May 8, 2023

After warp syncing a node it still downloads the block history. If the node is stopped and then started later while downloading the block history, the node first needs to do a major sync to sync to the tip of the chain. Before informant was showing Block history as sync state, while we actually were doing a major sync. This pr is fixing this.

After warp syncing a node it still downloads the block history. If the node is stopped and then
started later while downloading the block history, the node first needs to do a major sync to sync
to the tip of the chain. Before informant was showing `Block history` as sync state, while we
actually were doing a major sync. This pr is fixing this.
@bkchr bkchr added A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. B1-note_worthy Changes should be noted in the release notes T0-node This PR/Issue is related to the topic “node”. labels May 8, 2023
@bkchr bkchr requested a review from a team May 8, 2023 13:57
@altonen altonen requested a review from a team May 8, 2023 14:17
@bkchr
Copy link
Member Author

bkchr commented May 8, 2023

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit 33d94c2 into master May 8, 2023
@paritytech-processbot paritytech-processbot bot deleted the bkchr-informant-show-major-sync branch May 8, 2023 14:42
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
…tech#14094)

After warp syncing a node it still downloads the block history. If the node is stopped and then
started later while downloading the block history, the node first needs to do a major sync to sync
to the tip of the chain. Before informant was showing `Block history` as sync state, while we
actually were doing a major sync. This pr is fixing this.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B1-note_worthy Changes should be noted in the release notes C1-low PR touches the given topic and has a low impact on builders. T0-node This PR/Issue is related to the topic “node”.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants