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

Make candidate validation timeouts configurable #4001

Merged
merged 10 commits into from
Oct 4, 2021

Conversation

rphmeier
Copy link
Contributor

@rphmeier rphmeier commented Oct 3, 2021

and use a shorter timeout for backing as compared to approval and dispute participation.

@rphmeier rphmeier added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Oct 3, 2021
@rphmeier rphmeier requested a review from pepyakin October 3, 2021 20:45
Copy link
Contributor

@pepyakin pepyakin left a comment

Choose a reason for hiding this comment

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

Looks good modulo the test compilation error

@@ -130,6 +133,8 @@ pub enum CandidateValidationMessage {
ValidationCode,
CandidateDescriptor,
Arc<PoV>,
Duration,
/// Execution timeout
Copy link
Contributor

Choose a reason for hiding this comment

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

rustfmt considers comments on the same related to the next line...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I see, so fmt moved this around.

@@ -114,6 +115,8 @@ pub enum CandidateValidationMessage {
ValidateFromChainState(
CandidateDescriptor,
Arc<PoV>,
Duration,
/// Execution timeout
Copy link
Contributor

Choose a reason for hiding this comment

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

.

@jakoblell jakoblell added D1-audited 👍 PR contains changes to critical logic that has been properly reviewed and externally audited. and removed D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Oct 4, 2021
@rphmeier
Copy link
Contributor Author

rphmeier commented Oct 4, 2021

bot merge

@ghost
Copy link

ghost commented Oct 4, 2021

Waiting for commit status.

@rphmeier
Copy link
Contributor Author

rphmeier commented Oct 4, 2021

bot merge cancel

@ghost
Copy link

ghost commented Oct 4, 2021

Merge cancelled.

@rphmeier rphmeier merged commit 8962e04 into master Oct 4, 2021
@rphmeier rphmeier deleted the rh-multiple-validation-timeouts branch October 4, 2021 14:53
ordian added a commit that referenced this pull request Oct 4, 2021
* master:
  chore: update `builder` image (#3884)
  Free disputed cores before processing bitfields (#4008)
  Make candidate validation timeouts configurable (#4001)
ordian added a commit that referenced this pull request Oct 5, 2021
* master: (138 commits)
  Allow an Offset to Lease Periods (#3980)
  Bump quote from 1.0.9 to 1.0.10 (#4013)
  Companion for #9834 (Transaction Priority) (#3901)
  chore: update `builder` image (#3884)
  Free disputed cores before processing bitfields (#4008)
  Make candidate validation timeouts configurable (#4001)
  Add extrinsic ordering filtering (#3631)
  chore: ci list files that spellcheck finds (#3992)
  Use background tasks properly in candidate-validation (#4002)
  Fix unoccupied bitfields (#4004)
  Bump syn from 1.0.77 to 1.0.78 (#4006)
  Bump jsonrpsee-ws-client from 0.3.0 to 0.3.1 (#3931)
  fix clock drift for assignments issued before the block (#3851)
  Remove unoccupied bit check (#3999)
  bump substrate (#4000)
  change genesis authority set for wococo-local, revert rococo-local (#3998)
  ignore irrelevant approvals in logs (#3859)
  avoid expect, on free availability core (#3994)
  preserve finalized block in active leaves (#3997)
  some tweaks to rococo-local (#3996)
  ...
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. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D1-audited 👍 PR contains changes to critical logic that has been properly reviewed and externally audited.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants