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

Fix incorrect tag provided flag in tx pool #1786

Open
kamilsa opened this issue Sep 5, 2023 · 0 comments
Open

Fix incorrect tag provided flag in tx pool #1786

kamilsa opened this issue Sep 5, 2023 · 0 comments
Assignees
Labels
bug Something isn't working Transaction pool

Comments

@kamilsa
Copy link
Contributor

kamilsa commented Sep 5, 2023

Sometimes, during execution of zombienet-test-6 KAGOME crashes with with this assert being not satisfied.
The only place where tag_provided flag is set is here. It is not clear, how it could become false

@kamilsa kamilsa added this to KAGOME Sep 5, 2023
@kamilsa kamilsa converted this from a draft issue Sep 5, 2023
@kamilsa kamilsa added this to the Kusama treasury 2 milestone Sep 5, 2023
@kamilsa kamilsa added Transaction pool bug Something isn't working labels Sep 5, 2023
kamilsa added a commit that referenced this issue Sep 5, 2023
This commit includes changes for dispute finality handling and experimentation on transaction pool implementation. The changes implemented in the core/dispute_coordinator/impl/dispute_coordinator_impl.cpp file remove all debugging statements to improve readability and ensure cleaner code. Some adjustments have also been made to improve efficiency and avoid unnecessary computations.

In the file core/transaction_pool/impl/transaction_pool_impl.cpp, a commented assertion has been added, signifying a pending fix on issue #1786. Furthermore, configuration files for the zombienet had been deployed focusing on dispute resilience within the parachain validators' network layer. These configurations will prompt validation nodes to report any discrepancies on the chain, ensuring the integrity of the block records.

These modifications are crucial for maintaining the stability, security, and efficiency of the blockchain system.
kamilsa added a commit that referenced this issue Sep 5, 2023
This commit includes changes for dispute finality handling and experimentation on transaction pool implementation. The changes implemented in the core/dispute_coordinator/impl/dispute_coordinator_impl.cpp file remove all debugging statements to improve readability and ensure cleaner code. Some adjustments have also been made to improve efficiency and avoid unnecessary computations.

In the file core/transaction_pool/impl/transaction_pool_impl.cpp, a commented assertion has been added, signifying a pending fix on issue #1786. Furthermore, configuration files for the zombienet had been deployed focusing on dispute resilience within the parachain validators' network layer. These configurations will prompt validation nodes to report any discrepancies on the chain, ensuring the integrity of the block records.

These modifications are crucial for maintaining the stability, security, and efficiency of the blockchain system.
kamilsa added a commit that referenced this issue Sep 5, 2023
* Comment assert, add zombienet test

This commit includes changes for dispute finality handling and experimentation on transaction pool implementation. The changes implemented in the core/dispute_coordinator/impl/dispute_coordinator_impl.cpp file remove all debugging statements to improve readability and ensure cleaner code. Some adjustments have also been made to improve efficiency and avoid unnecessary computations.

In the file core/transaction_pool/impl/transaction_pool_impl.cpp, a commented assertion has been added, signifying a pending fix on issue #1786. Furthermore, configuration files for the zombienet had been deployed focusing on dispute resilience within the parachain validators' network layer. These configurations will prompt validation nodes to report any discrepancies on the chain, ensuring the integrity of the block records.

These modifications are crucial for maintaining the stability, security, and efficiency of the blockchain system.

* Update and rename 0004-parachains-garbage-candidate.zndsl to 0007-parachains-garbage-candidate.zndsl

Rename 1

* Rename 0004-parachains-garbage-candidate.toml to 0007-parachains-garbage-candidate.toml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Transaction pool
Projects
None yet
Development

No branches or pull requests

2 participants