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

docs(s2n-quic): add a comment about docke+GSO #2487

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

Conversation

dougch
Copy link
Contributor

@dougch dougch commented Feb 25, 2025

Release Summary:

Related issues:

related #2485

Description of changes:

Add a comment to the user guide about Docker and GSO.

Call-outs:

Docker host networking works fine with GSO, should I add that as an additional workaround?

Testing:

AWS CodeBuild

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@dougch dougch marked this pull request as ready for review February 25, 2025 23:06
```

In the case of running tests in broken GSO environments, setting `S2N_QUIC_PLATFORM_FEATURES_OVERRIDE=""` as an environment variable will disable the feature.
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure we want to document this right now. It's a bit of a blunt instrument and will disable all feature probing. I think we could improve usability, where you can specify something like S2N_QUIC_PLATFORM_FEATURES_OVERRIDE=default,-gso,-gro

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