Skip to content

Remove items that only contain space in gcb commands - #1861

Merged
DonggeLiu merged 1 commit into
masterfrom
filter_spaces_in_gcb_command
Jun 13, 2023
Merged

DonggeLiu merged 1 commit into
masterfrom
filter_spaces_in_gcb_command

Conversation

@DonggeLiu

@DonggeLiu DonggeLiu commented Jun 13, 2023 •

Copy link
Copy Markdown
Contributor

Fixes a known issues that affects many experiment requests, e.g., this one in #1859.

@DonggeLiu

DonggeLiu commented Jun 13, 2023 •

Copy link
Copy Markdown
Contributor Author

Intentionally added redundant spaces between fuzzers to check if they are removed as expected. ⬇️

@DonggeLiu

Copy link
Copy Markdown
Contributor Author

/gcbrun run_experiment.py -a --experiment-config /opt/fuzzbench/service/experiment-config.yaml --experiment-name 2023-06-13-space-1 --fuzzers libfuzzer honggfuzz

@jonathanmetzman jonathanmetzman 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

@DonggeLiu
DonggeLiu merged commit 59c0c17 into master Jun 13, 2023
@DonggeLiu
DonggeLiu deleted the filter_spaces_in_gcb_command branch June 13, 2023 02:46
@jonathanmetzman

Copy link
Copy Markdown
Contributor

I wouldn't run an experiment for this in the future. If it's broken we will notice right away

@DonggeLiu

Copy link
Copy Markdown
Contributor Author

I wouldn't run an experiment for this in the future. If it's broken we will notice right away

Ah sorry, I was not intended to leave the experiment running.
I only wanted to use the GCB to verify that the extra spaces I added between libfuzzer and honggfuzz won't affect the experiment like in #1859 (the spaces are only visible if you edit my comment).

The experiment has been terminated now : )

SPINO-11 pushed a commit to SPINO-11/green_fuzzing_evaluation that referenced this pull request Aug 9, 2025
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