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

[BACKPORT/23.0.x] 5426 #5431

Merged
merged 12 commits into from
Nov 7, 2023
Merged

Conversation

peternose
Copy link
Contributor

Backports #5426

The periodic re-attestation happens every hour, while explicitly requested
re-attestation takes place approximately every 1 to 2 hours. This results
in both re-attestations occurring closely together. To avoid unnecessary
re-attestation and re-registration, we extend the periodic re-attestation
interval to encompass the worst-case scenario of explicit attestation.
When a node is not a member of the committee, events, commitments,
and transactions accumulate in the channels. When the node gets
re-elected, it must process these channels first, potentially
resulting in delayed submission of its commitments or incorrect
discrepancy detection.
@peternose peternose added c:backport Category: backport to stable branch c:bug Category: bug labels Nov 7, 2023
@peternose peternose enabled auto-merge November 7, 2023 14:17
Copy link

codecov bot commented Nov 7, 2023

Codecov Report

Merging #5431 (fc8e4e9) into stable/23.0.x (49f958b) will increase coverage by 0.24%.
Report is 1 commits behind head on stable/23.0.x.
The diff coverage is 85.49%.

@@                Coverage Diff                @@
##           stable/23.0.x    #5431      +/-   ##
=================================================
+ Coverage          66.29%   66.54%   +0.24%     
=================================================
  Files                533      533              
  Lines              56352    56365      +13     
=================================================
+ Hits               37361    37509     +148     
+ Misses             14569    14466     -103     
+ Partials            4422     4390      -32     
Files Coverage Δ
go/consensus/cometbft/apps/roothash/roothash.go 73.91% <100.00%> (+0.11%) ⬆️
go/roothash/api/api.go 72.97% <ø> (ø)
go/roothash/tests/tester.go 92.98% <100.00%> (+0.05%) ⬆️
go/runtime/host/sgx/sgx.go 68.75% <ø> (ø)
go/runtime/txpool/txpool.go 83.37% <100.00%> (-0.20%) ⬇️
go/worker/client/committee/node.go 67.35% <ø> (-1.53%) ⬇️
go/worker/common/committee/group.go 82.85% <ø> (-1.56%) ⬇️
go/worker/common/committee/node.go 62.91% <100.00%> (+0.72%) ⬆️
go/worker/common/committee/p2p.go 88.88% <ø> (+5.55%) ⬆️
go/worker/compute/executor/committee/hooks.go 87.50% <ø> (-3.41%) ⬇️
... and 7 more

... and 41 files with indirect coverage changes

@peternose peternose merged commit bb385af into stable/23.0.x Nov 7, 2023
6 checks passed
@peternose peternose deleted the peternose/stable/23.0.x/backport-5426 branch November 7, 2023 15:28
pro-wh added a commit to oasisprotocol/nexus that referenced this pull request Feb 26, 2024
cross-reference: oasisprotocol/oasis-core#5431

Adding this as nilable so we can detect versions below 23.0.3 leaving it out.
pro-wh added a commit to oasisprotocol/nexus that referenced this pull request Feb 27, 2024
cross-reference: oasisprotocol/oasis-core#5431

Adding this as nilable so we can detect versions below 23.0.3 leaving it out.
pro-wh added a commit to oasisprotocol/nexus that referenced this pull request Feb 27, 2024
cross-reference: oasisprotocol/oasis-core#5431

Adding this as nilable so we can detect versions below 23.0.3 leaving it out.
pro-wh added a commit to oasisprotocol/nexus that referenced this pull request Feb 27, 2024
cross-reference: oasisprotocol/oasis-core#5431

Adding this as nilable so we can detect versions below 23.0.3 leaving it out.
pro-wh added a commit to oasisprotocol/nexus that referenced this pull request Feb 27, 2024
cross-reference: oasisprotocol/oasis-core#5431

Adding this as nilable so we can detect versions below 23.0.3 leaving it out.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:backport Category: backport to stable branch c:bug Category: bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants