Skip to content

[Centipede] Implement corpus prune for centipede on corpus prune task - #4707

Merged
javanlacerda merged 14 commits into
masterfrom
javan.prune-cent
Mar 7, 2025
Merged

[Centipede] Implement corpus prune for centipede on corpus prune task#4707
javanlacerda merged 14 commits into
masterfrom
javan.prune-cent

Conversation

@javanlacerda

@javanlacerda javanlacerda commented Feb 27, 2025

Copy link
Copy Markdown
Collaborator

It implements corpus pruning for centipede fuzzer.

@javanlacerda javanlacerda changed the title Javan.prune cent [Centipede] Implement corpus prune for centipede on corpus prune task Feb 27, 2025
@javanlacerda
javanlacerda marked this pull request as ready for review February 27, 2025 20:27
Comment thread src/clusterfuzz/_internal/bot/tasks/utasks/corpus_pruning_task.py
Comment thread src/clusterfuzz/_internal/bot/tasks/utasks/corpus_pruning_task.py Outdated
Comment thread src/clusterfuzz/_internal/bot/tasks/utasks/corpus_pruning_task.py Outdated
@alhijazi

Copy link
Copy Markdown
Contributor

Seems that you have some formatting/linting errors, let's try to get the basic tests running

Comment thread .style.yapf Outdated
@javanlacerda
javanlacerda changed the base branch from master to javan.bump-yapf March 1, 2025 23:20
@javanlacerda
javanlacerda changed the base branch from javan.bump-yapf to master March 1, 2025 23:21
@javanlacerda javanlacerda mentioned this pull request Mar 5, 2025
alhijazi and others added 14 commits March 6, 2025 17:33
This change call's centipede's minimize_corpus method on each fuzzing
round. This allows us to distill the corpus and only add useful units.

On a local run, this allowed to decrease the number of units added from
2000+ to 5 with the same coverage.

The cl also extracts some common functionality into engine_common.
Signed-off-by: Javan Lacerda <javanlacerda@google.com>
Signed-off-by: Javan Lacerda <javanlacerda@google.com>
Signed-off-by: Javan Lacerda <javanlacerda@google.com>
Signed-off-by: Javan Lacerda <javanlacerda@google.com>
Signed-off-by: Javan Lacerda <javanlacerda@google.com>
Signed-off-by: Javan Lacerda <javanlacerda@google.com>
Signed-off-by: Javan Lacerda <javanlacerda@google.com>
Signed-off-by: Javan Lacerda <javanlacerda@google.com>
Signed-off-by: Javan Lacerda <javanlacerda@google.com>
Signed-off-by: Javan Lacerda <javanlacerda@google.com>
@javanlacerda

Copy link
Copy Markdown
Collaborator Author

@jonathanmetzman @alhijazi I just removed the match case usage for simplicity and avoid any other frictions for now.

@jonathanmetzman jonathanmetzman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@alhijazi alhijazi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, let's rebase and get this landed and see if this works as expected.

@javanlacerda
javanlacerda merged commit 1d2c8d1 into master Mar 7, 2025
@javanlacerda
javanlacerda deleted the javan.prune-cent branch March 7, 2025 15:15
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.

3 participants