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

[pull] main from googleforgames:main #258

Open
wants to merge 54 commits into
base: main
Choose a base branch
from
Open

Conversation

pull[bot]
Copy link

@pull pull bot commented Feb 5, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

@pull pull bot added the ⤵️ pull label Feb 5, 2025
Jake-Shadle and others added 12 commits February 6, 2025 08:56
* Update crates

* Fix bench

* Fix test
* Don't check external urls

* Skip test-docs if no changes
* Defer shutdown until actives sessions are 0

* Remove unconditional RUSTFLAGS

This is already handled in config.toml, and only adds the flags for the x86 arch so that warnings aren't emitted

* Fix non-linux
* Update to xdp 0.2.0

* Use InterfaceIter from xdp

* Change to use wakable rings

Also, correct huge screw up and only poll for _read_ operations, making the CPU usage when not actually receiving packets got to "0"

* Fix tests
XAMPPRocky and others added 15 commits February 13, 2025 13:42
* Remove nmap-service-probe

This doesn't compile so makes workspace level changes annoying to check

* Remove Config::version

This isn't used anywhere

* Begin config migration

* Add version
* chore: Update to Rust 1.85 and 2024 edition

* Regen protos

* Remove workspace dep check for now

* Use 2024 formatting

* Format

---------

Co-authored-by: Jake Shadle <[email protected]>
* Update to xdp 0.5.0

* Fixup tests

* Fix off by 1 bug

* Update crates/test/tests/xdp.rs

Co-authored-by: XAMPPRocky <[email protected]>

---------

Co-authored-by: XAMPPRocky <[email protected]>
* Add more comprehensive lints

* Fix lints
Jake-Shadle and others added 26 commits February 28, 2025 13:12
* Fix bug in remove_head

* Cleanup filter tests
* Remove error

If XDP was not requested we'd get a large error stacktrace, this removes it as it doesn't help, an error is only shown if XDP was actually requested but failed

* Handle terminated connections

* Specialize datacenter

Datacenter was not removing resources correctly, the generic xDS stuff just sends a type and value, but elements in the datacenter are keyed off of the agent's IP
Co-authored-by: XAMPPRocky <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants