Skip to content

Conversation

@timorl
Copy link
Contributor

@timorl timorl commented Nov 29, 2022

  • Limit nonfinalized block production

  • Better saturation handling

  • Add log when delaying block production

  • Disable two tests that were operating under outdated assumptions

  • Disable one more test

Description

Cherry-pick from the main branch.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • I have added tests
  • I have made neccessary updates to the Infrastructure
  • I have made corresponding changes to the existing documentation
  • I have created new documentation
  • I have bumped spec_version and transaction_version
  • I have bumped aleph-client version if relevant

* Limit nonfinalized block production

* Better saturation handling

* Add log when delaying block production

* Disable two tests that were operating under outdated assumptions

* Disable one more test
@timorl timorl merged commit 285c13d into Cardinal-Cryptography:release-8 Nov 29, 2022
Marcin-Radecki added a commit that referenced this pull request Dec 1, 2022
* Request justification from forks

* Reverted changes to cargo lock introduced in 6ae50ca (#773)

* A0-1635: Limit nonfinalized block production (#769) (#774)

* A0-1635: Limit nonfinalized block production (#769)

* Limit nonfinalized block production

* Better saturation handling

* Add log when delaying block production

* Disable two tests that were operating under outdated assumptions

* Disable one more test

* Turn off the likely broken test

* Changed node version

* Fixed leftover conflicted line in lock file

* Reverted unwanted change to runtime/Cargo.toml

* Updated Cargo.lock

Co-authored-by: Michal Swietek <[email protected]>
Co-authored-by: timorl <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants