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

[fuzz] try fix for oss-fuzz build #19164

Merged
merged 1 commit into from
Dec 2, 2021

Conversation

asraa
Copy link
Contributor

@asraa asraa commented Dec 1, 2021

@adisuissa

Signed-off-by: Asra Ali [email protected]

Commit Message: Try fix for OSS-Fuzz build failure (see bazel-contrib/rules_fuzzing#185)
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Dec 1, 2021
@repokitteh-read-only
Copy link

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).
envoyproxy/dependency-shepherds assignee is @mattklein123

🐱

Caused by: #19164 was opened by asraa.

see: more, trace.

@adisuissa
Copy link
Contributor

Thanks @asraa!
I've tested this on my personal machine with limited disk space, and it didn't crash in the place where it stopped earlier, but I still got an out of disk space error while untaring the output step (https://github.com/google/oss-fuzz/blob/0bed7945b38dfd4173bc4faa6ed59fc1c4c2815f/projects/envoy/build.sh#L142-L144).

I'm not sure if the same issue will happen in oss-fuzz or not, as the disk size limits are different.

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Dec 2, 2021
@mattklein123 mattklein123 merged commit 0e33d22 into envoyproxy:main Dec 2, 2021
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