Skip to content

Conversation

@sstanculeanu
Copy link
Collaborator

Reasoning behind the pull request

Proposed changes

Testing procedure

Pre-requisites

Based on the Contributing Guidelines the PR author and the reviewers must check the following requirements are met:

  • was the PR targeted to the correct branch?
  • if this is a larger feature that probably needs more than one PR, is there a feat branch created?
  • if this is a feat branch merging, do all satellite projects have a proper tag inside go.mod?

@sstanculeanu sstanculeanu added the ignore-for-release-notes Do not include item in release notes label Aug 8, 2024
@sstanculeanu sstanculeanu self-assigned this Aug 8, 2024
sasurobert
sasurobert previously approved these changes Aug 8, 2024
miiu96
miiu96 previously approved these changes Aug 8, 2024
@sstanculeanu sstanculeanu dismissed stale reviews from miiu96 and sasurobert via f894d55 August 8, 2024 10:51
Copy link
Contributor

@elvisboghiu elvisboghiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

System test results
Normal allin test: master-7cab1a501e -> merge_master_into_rc17next-a94af2a9e6

--- Specific errors ---

block hash does not match 303
wrong nonce in block 234
miniblocks does not match 0
num miniblocks does not match 0
miniblock hash does not match 0
block bodies does not match 0
receipts hash missmatch 0

/------/

--- Statistics ---

Nr. of all ERRORS: 1
Nr. of all WARNS: 266
Nr. of new ERRORS: 1
Nr. of new WARNS: 21
Nr. of PANICS: 0

/------/

--- ERRORS ---

ovh-p04-observer-2 :
Error: error calling SaveBlock, will retry driver = *dataindexer.dataIndexer retrial in = error = { "index": "id": "statusCode": "errorType": "version_conflict_engine_exception", "reason": version conflict, required seqNo primary term current document has seqNo and primary term "causedBy": { "type": "", "reason": "" }}\n{ "index": "id": "statusCode": "errorType": "version_conflict_engine_exception", "reason": version conflict, required seqNo primary term current document has seqNo and primary term "causedBy": { "type": "", "reason": "" }}\n when saving transactions, block hash nonce 1

/------/

--- WARNINGS ---

ovh-p04-validator-28 :
Warn: shardProcessor.createMiniBlocks error = shard is stuck shard = 2

ovh-p04-validator-32 :
Warn: shardProcessor.createMiniBlocks error = shard is stuck shard = 2

ovh-p04-validator-31 :
Warn: shardProcessor.createMiniBlocks error = shard is stuck shard = 1

ovh-p04-validator-24 :
Warn: shardProcessor.createMiniBlocks error = shard is stuck shard = 2

ovh-p04-validator-7 :
Warn: shardProcessor.createMiniBlocks error = shard is stuck shard = 1

ovh-p04-validator-2 :
Warn: shardProcessor.createMiniBlocks error = shard is stuck shard = 2

ovh-p04-validator-26 :
Warn: shardProcessor.createMiniBlocks error = shard is stuck shard = 2

ovh-p04-validator-35 :
Warn: shardProcessor.createMiniBlocks error = shard is stuck shard = 3

ovh-p04-validator-30 :
Warn: shardProcessor.createMiniBlocks error = shard is stuck shard = 3

ovh-p04-validator-14 :
Warn: shardProcessor.createMiniBlocks error = shard is stuck shard = 1

ovh-p04-validator-10 :
Warn: shardProcessor.createMiniBlocks error = shard is stuck shard = 1

ovh-p04-validator-9 :
Warn: shardProcessor.createMiniBlocks error = shard is stuck shard = 1

/------/

@sstanculeanu sstanculeanu merged commit 6671aa7 into rc/v1.7.next1 Aug 9, 2024
@sstanculeanu sstanculeanu deleted the merge_master_into_rc17next1_2024.08.08 branch August 9, 2024 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore-for-release-notes Do not include item in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants