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

malus-collator: implement malicious collator submitting same collation to all backing groups #6924

Merged
merged 29 commits into from
Jan 30, 2025

Conversation

sw10pa
Copy link
Member

@sw10pa sw10pa commented Dec 17, 2024

Issues

Description

Modified the undying collator to include a malus mode, in which it submits the same collation to all assigned backing groups.

TODO

…or submitting same collation to all backing groups
@sw10pa sw10pa added the T10-tests This PR/Issue is related to tests. label Dec 17, 2024
@sw10pa sw10pa requested review from alindima and sandreim December 17, 2024 12:17
@sw10pa sw10pa self-assigned this Dec 17, 2024
Copy link
Contributor

@alindima alindima left a comment

Choose a reason for hiding this comment

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

Looking good overall!

@sw10pa sw10pa changed the title [WIP] malus-collator: implement initial version of malicious collator submitting same collation to all backing groups [WIP] malus-collator: implement malicious collator submitting same collation to all backing groups Jan 27, 2025
@sw10pa sw10pa changed the title [WIP] malus-collator: implement malicious collator submitting same collation to all backing groups malus-collator: implement malicious collator submitting same collation to all backing groups Jan 29, 2025
@sw10pa sw10pa marked this pull request as ready for review January 29, 2025 12:26
@sw10pa sw10pa requested a review from a team as a code owner January 29, 2025 12:26
@paritytech-workflow-stopper
Copy link

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/13031285399
Failed job name: cargo-clippy

Copy link
Contributor

@alindima alindima left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@sandreim sandreim left a comment

Choose a reason for hiding this comment

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

LGTM! Nice work @sw10pa

@sw10pa sw10pa added this pull request to the merge queue Jan 30, 2025
Merged via the queue into master with commit 48f69cc Jan 30, 2025
205 of 207 checks passed
@sw10pa sw10pa deleted the sw10pa/malus-collator branch January 30, 2025 13:20
ordian added a commit that referenced this pull request Feb 3, 2025
* master:
  Remove warnings by cleaning up the `Cargo.toml` (#7416)
  [Backport] Version bumps from stable2412-1 + prdocs reorg (#7401)
  fix pre-dispatch PoV underweight for ParasInherent (#7378)
  malus-collator: implement malicious collator submitting same collation to all backing groups (#6924)
  `fatxpool`: use tracing for logging (#6897)
  Improvements for Weekly bench (#7390)
  Replace derivative dependency with derive-where (#7324)
  Add support for feature `pallet_balances/insecure_zero_ed` in benchmarks and testing (#7379)
  Fix Snowbridge benchmark tests (#7296)
  Bridges small nits/improvements (#7383)
  Migrating cumulus-pallet-session-benchmarking to Benchmarking V2  (#6564)
  [pallet-revive] implement the block author API  (#7198)
  Use checked math in frame-balances named_reserve (#7365)
  move installation of frame-omni-bencher into a cmd.py itself (#7372)
  remove old bench & revert the frame-weight-template (#7362)
  ci: fix workflow permissions (#7366)
  [net/libp2p] Use raw `Identify` observed addresses to discover external addresses (#7338)
  Improve `set_validation_data` error message. (#7359)
  Implement pallet view function queries (#4722)
Ank4n pushed a commit that referenced this pull request Feb 6, 2025
…n to all backing groups (#6924)

## Issues
- [[#5049] Elastic scaling: zombienet
tests](#5049)
- [[#4526] Add zombienet tests for malicious
collators](#4526)

## Description
Modified the undying collator to include a malus mode, in which it
submits the same collation to all assigned backing groups.

## TODO
* [X] Implement malicious collator that submits the same collation to
all backing groups;
* [X] Avoid the core index check in the collation generation subsystem:
https://github.com/paritytech/polkadot-sdk/blob/master/polkadot/node/collation-generation/src/lib.rs#L552-L553;
* [X] Resolve the mismatch between the descriptor and the commitments
core index: #7104
* [X] Implement `duplicate_collations` test with zombienet-sdk;
* [X] Add PRdoc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T10-tests This PR/Issue is related to tests.
Projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

7 participants