Skip to content

Fix ASan build #13757

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

Closed
wants to merge 1 commit into from
Closed

Fix ASan build #13757

wants to merge 1 commit into from

Conversation

iluuu1994
Copy link
Member

@iluuu1994 iluuu1994 commented Mar 19, 2024

See actions/runner-images#9491 (comment)

The mentioned workaround doesn't work for us because we run ASan inside Docker.
Instead, we switch to ubuntu-20.04 as the host. The docker setup itself remains
the same.

@iluuu1994 iluuu1994 requested a review from TimWolla as a code owner March 19, 2024 11:47
Copy link
Member

@TimWolla TimWolla left a comment

Choose a reason for hiding this comment

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

see: actions/runner-images#9491

We can either wait until the end of the week where the fix will be rolled out or sudo sysctl vm.mmap_rnd_bits=28.

@iluuu1994 iluuu1994 force-pushed the fix-asan branch 2 times, most recently from 69901f9 to 4d0596e Compare March 19, 2024 12:36
@iluuu1994
Copy link
Member Author

@TimWolla Good find, thank you! Let's add this line and drop it once the issue is resolved.

@iluuu1994 iluuu1994 changed the title Switch to Ubuntu 20.04 for asan Fix ASan build Mar 19, 2024
See actions/runner-images#9491 (comment)

The mentioned workaround doesn't work for us because we run ASan inside Docker.
Instead, we switch to ubuntu-20.04 as the host. The docker setup itself remains
the same.
@iluuu1994
Copy link
Member Author

iluuu1994 commented Mar 19, 2024

@TimWolla Actually, the previous fix was accidentally correct. The mentioned workaround doesn't work for us because we run the ASan build inside Docker, and thus the command runs it in the container rather than the host.

@iluuu1994 iluuu1994 closed this in 697d1a1 Mar 19, 2024
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