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

Add workaround for zeek/3534 on Windows #7

Merged
merged 3 commits into from
Jan 16, 2024
Merged

Conversation

philrz
Copy link
Contributor

@philrz philrz commented Jan 15, 2024

zeek/zeek#3534 was the one open issue that was holding me back from packaging a Zeek v6 artifact with Zui. That's now spawned follow-on issue mheily/libkqueue#155 and based on the lack of activity in that libkqueue repo I don't have much faith a fix is coming any time soon. Therefore it seems it's time to make a call on going ahead with shipping a Zeek v6 artifact that disables the packages that are known to trigger that segfault.

The changes in this branch comment out the @load of the two packages that trigger the bad code path as part of the bundling of the Windows artifact. I've made a test Zeek artifact v6.2.0-brim-dev2 based on that and referenced that in test Brimcap artifact v1.6.0-alpha4 and then referenced that Brimcap artifact from the Zui dev build from Actions run https://github.com/brimdata/zui/actions/runs/7534518094. With that Zui I've confirmed that the wrccdc 2018 pcaps that formerly triggered the segfault can loaded into that Zui without error. I've also done side-by-side drags of the 4 GB "all.pcap" and confirmed we see the expected delta in notice events based on the disabling of the detect-MHR package in the Windows case.

macOS

image

Windows

image

@philrz philrz requested a review from nwt January 15, 2024 23:11
@philrz philrz self-assigned this Jan 15, 2024
release.sh Outdated Show resolved Hide resolved
@philrz philrz merged commit c75b920 into main Jan 16, 2024
0 of 3 checks passed
@philrz philrz deleted the workaround-zeek-3534 branch January 16, 2024 20:16
@philrz philrz linked an issue Apr 1, 2024 that may be closed by this pull request
@philrz philrz mentioned this pull request Apr 1, 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.

Release blockers
2 participants