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

envoy: exclude no_fuzz tagged targets. #1712

Merged
merged 1 commit into from
Aug 15, 2018
Merged

Conversation

htuch
Copy link
Contributor

@htuch htuch commented Aug 15, 2018

This will make it easier for us to control individual test targets and
exclude trivials/bad fuzzers from the Envoy GH repo.

Signed-off-by: Harvey Tuch [email protected]

This will make it easier for us to control individual test targets and
exclude trivials/bad fuzzers from the Envoy GH repo.

Signed-off-by: Harvey Tuch <[email protected]>
htuch added a commit to htuch/envoy that referenced this pull request Aug 15, 2018
In conjunction with google/oss-fuzz#1712, this allows us to elide trivial
fuzzers on ClusterFuzz. This should save CPU cycles and project quota (depending on the nuances of
how they schedule).

Risk level: Low
Testing: Building with oss-fuzz Docker image.

Signed-off-by: Harvey Tuch <[email protected]>
@Dor1s Dor1s merged commit 870f29b into google:master Aug 15, 2018
htuch added a commit to envoyproxy/envoy that referenced this pull request Aug 15, 2018
In conjunction with google/oss-fuzz#1712, this allows us to elide trivial
fuzzers on ClusterFuzz. This should save CPU cycles and project quota (depending on the nuances of
how they schedule).

Risk level: Low
Testing: Building with oss-fuzz Docker image.

Signed-off-by: Harvey Tuch <[email protected]>
tmatth pushed a commit to tmatth/oss-fuzz that referenced this pull request Oct 22, 2018
This will make it easier for us to control individual test targets and
exclude trivials/bad fuzzers from the Envoy GH repo.

Signed-off-by: Harvey Tuch <[email protected]>
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