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

Supports regression test in fuzzing_launcher #56

Merged
merged 7 commits into from
Aug 14, 2020

Conversation

tengpengli
Copy link
Contributor

Commit Message:

Supports regression test in fuzzing_launcher

Adds string_flag engine to decide the fuzzing_launcher behavior.

Additional Description:
Testing:
Local testing and CI testing.
Docs Changes: N/A
Fixes #55

Signed-off-by: tengpeng [email protected]

modified:   .bazelrc
modified:   WORKSPACE
modified:   fuzzing/BUILD
modified:   fuzzing/common.bzl
modified:   fuzzing/repositories.bzl
modified:   fuzzing/tools/launcher.py

Adds string_flag engine to decide the fuzzing_launcher behavior.

Signed-off-by: tengpeng <[email protected]>

	modified:   .bazelrc
	modified:   WORKSPACE
	modified:   fuzzing/BUILD
	modified:   fuzzing/common.bzl
	modified:   fuzzing/repositories.bzl
	modified:   fuzzing/tools/launcher.py
fuzzing/common.bzl Outdated Show resolved Hide resolved
fuzzing/common.bzl Show resolved Hide resolved
fuzzing/tools/launcher.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@asraa asraa left a comment

Choose a reason for hiding this comment

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

Just to double-check, is envoyproxy/envoy#12530 on hold until this PR is sorted out?

.bazelrc Outdated Show resolved Hide resolved
fuzzing/common.bzl Outdated Show resolved Hide resolved
@tengpengli
Copy link
Contributor Author

Just to double-check, is envoyproxy/envoy#12530 on hold until this PR is sorted out?

Yes, I plan to go back to the envoy PR as soon as this PR is merged. Can you take a look at the comment question under the PR?

Copy link
Collaborator

@stefanbucur stefanbucur left a comment

Choose a reason for hiding this comment

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

Looks good overall, a few issues remaining.

.bazelrc Outdated Show resolved Hide resolved
.bazelrc Outdated Show resolved Hide resolved
Copy link
Collaborator

@stefanbucur stefanbucur left a comment

Choose a reason for hiding this comment

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

One more issue, and we should be good to go.

fuzzing/common.bzl Outdated Show resolved Hide resolved
fuzzing/common.bzl Outdated Show resolved Hide resolved
Copy link
Collaborator

@stefanbucur stefanbucur left a comment

Choose a reason for hiding this comment

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

LGTM!

@tengpengli tengpengli merged commit a5f1a58 into master Aug 14, 2020
@tengpengli tengpengli deleted the launcher_add_regression branch August 14, 2020 17:16
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.

Enable regression support in the launcher script
3 participants