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

build: Set up nightly runs of table evolution fuzzer #12037

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kagamiori
Copy link
Contributor

Differential Revision: D67921566

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jan 8, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D67921566

Copy link

netlify bot commented Jan 8, 2025

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit ed4af9b
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/677dec67f7c56b0008ce2a94

@@ -202,6 +202,7 @@ jobs:
run: |
EXTRA_CMAKE_FLAGS="-DPYTHON_EXECUTABLE=$(which python3) $EXTRA_CMAKE_FLAGS"
make debug
make release
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why? We run all other fuzzers debug only.

Copy link
Contributor

Choose a reason for hiding this comment

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

Release build runs faster and could cover more cases, we probably should run all fuzzers in release build

Copy link
Collaborator

@assignUser assignUser Jan 8, 2025

Choose a reason for hiding this comment

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

Release makes sense, I would be fine with either or but running both seems redundant.
Also should we use release with debug symbols for better stacktraces or do the fuzzers have their own thing?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah we should include the full symbol in these builds for fuzzers

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D67921566

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants