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

all: prepare for v0.18.0-rc1 release #498

Merged
merged 10 commits into from
Apr 30, 2024
Merged

all: prepare for v0.18.0-rc1 release #498

merged 10 commits into from
Apr 30, 2024

Conversation

xenoscopic
Copy link
Member

What does this pull request do and why is it needed?

This PR prepares for the v0.18.0-rc1 release. It primarily includes dependency updates, but also a few changes to scan preemption.

When using recursive watching, a very large synchronization root could
block endpoint scan calls from preempting because they would be blocked
trying to acquire the scan lock, which would be held by the watching
Goroutine while it tried to perform its initial scan. This could make
session termination hang. To fix this, we switch to a semaphore-based
scan lock that allows for preemption. Termination of the session still
behaves correctly because the watching Goroutine context is still
cancelled and synchronization/core.Scan will preempt correctly.

Signed-off-by: Jacob Howard <[email protected]>
Signed-off-by: Jacob Howard <[email protected]>
This doesn't have a practical impact, since scan cancellation indicates
that an endpoint is shutting down anyway, but it's a more "correct"
return value.

Signed-off-by: Jacob Howard <[email protected]>
Something seems to have changed with macOS' FAT32 implementation with
macOS 14 that is causing FAT32-related tests to fail. We need to
investigate further, but this shouldn't affect most users.

Moreover, something also seems broken with macOS 13's base64 command (it
doesn't like long-form flags like --output).

Signed-off-by: Jacob Howard <[email protected]>
@xenoscopic xenoscopic merged commit cc411d5 into master Apr 30, 2024
14 checks passed
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.

1 participant