test(starry): add gateway and higress reverse-proxy carpets#1502
Conversation
ZR233
left a comment
ZR233
left a comment
There was a problem hiding this comment.
需要继续修改后再合并。我在当前 head 本地运行 cargo xtask starry app qemu -t gateway --arch x86_64，prebuild 阶段即失败，尚未进入 guest。
Lfan-ke
changed the title
ZR233
left a comment
ZR233
left a comment
There was a problem hiding this comment.
这版还不能合入，主要有两个阻塞点：
- 现有 gateway prebuild 的 review thread 仍然适用。本地在当前 head
bf296542722dde2e913c3736e94c99416b3bad74重新跑cargo xtask starry app qemu -t gateway --arch x86_64，仍然在 guest 启动前失败：prebuild.sh先下载到angie-1.11.5-r0.apk，随后下载https://dl-cdn.alpinelinux.org/alpine/v3.23/main/x86_64/libcrypto3-3.5.6-r0.apk返回 404。PR 描述里的 gateway 四架构真跑门控目前无法复现，需要把 apk 依赖闭包改成可复现且不会随 Alpine branch point-release 漂移而失效的来源。 SO_REUSEPORT的 UDP 实现现在是直接绕过全局 UDP 端口登记，会放宽普通 socket 与 reuseport socket 之间的冲突检查。我在行内留了具体场景和需要补的回归测试。
另外，当前 head 的 CI 也不是绿的；我检查到 Test starry loongarch64 qemu / run_container 在 test-cargo-jobserver-wait 处失败，部分其他 self-hosted/Starry job 还有取消或失败状态。修完上述问题后还需要重新跑出可复核的绿色结果。
Lfan-ke
changed the title
There was a problem hiding this comment.
评审意见
本 PR 新增两套 StarryOS 数据面网关地毯（gateway 基于 angie，higress 基于 Envoy），并实现 SO_REUSEPORT socket option 以支持 higress。代码结构清晰，TCP 侧 SO_REUSEPORT 实现设计良好。但当前 head 仍有两个阻塞问题未解决，与 ZR233 在上轮 review 中提出的意见一致。
阻塞问题
1. gateway prebuild.sh 依赖闭包不可复现
apps/starry/gateway/prebuild.sh 将 Alpine v3.23 apk 依赖包精确 pin 到 libcrypto3-3.5.6-r0.apk、libssl3-3.5.6-r0.apk 等具体版本。Alpine v3.23/main 分支会滚动替换 point release，这些精确版本已从 CDN 下架。本地在当前 head 运行 cargo xtask starry app qemu -t gateway --arch x86_64 时，https://dl-cdn.alpinelinux.org/alpine/v3.23/main/x86_64/libcrypto3-3.5.6-r0.apk 返回 404，prebuild 以 curl 22 退出。
修复方向： 将完整依赖闭包放入可复现缓存/资产（如 GATEWAY_BINS_DIR）并校验 sha256，或改为从 apk registry 动态解析版本并同步更新 sha pin。修完后需重跑至少 x86_64 的 gateway app 命令验证。
2. UDP SO_REUSEPORT 实现不正确
net/ax-net/src/udp.rs 第 272 行仅通过 && !self.general.reuse_port() 跳过全局 UDP 端口登记，将 SO_REUSEPORT 当作端口冲突检查的旁路开关。这导致：
- 先用 SO_REUSEPORT bind 的 UDP socket 不写入 SOCKET_SET，后续未设置 SO_REUSEPORT 的 socket 绑定同一端口不会触发 EADDRINUSE；
- 已有普通 binder 时，reuseport socket 也会绕过全局冲突检查。
Linux 语义要求：所有共享同一地址/端口的 socket 都必须在 bind 前设置 SO_REUSEPORT；普通 socket 与 reuseport socket 不允许混在同一端口上。
对比 TCP 侧（net/ax-net/src/tcp.rs），TCP 正确实现了 TcpBoundEntry 结构体和 register_tcp_bound(endpoint, reuse_port) 的 reuseport group 语义。UDP 侧需要同样的 group 跟踪逻辑和单元测试覆盖。
评审依据
- 上一轮 review 意见： ZR233 两次 CHANGES_REQUESTED 指出的 prebuild 依赖闭包问题和 UDP SO_REUSEPORT 实现在当前 head 上均未修复。ZR233 的 inline 评论（
#discussion_r3526033339和#discussion_r3526940269）在技术上合理，应作为合入前置条件。 - CI 状态： 当前 head 的 CI check suite 结论为
failure。Starry loongarch64 qemu / run_container 在test-cargo-jobserver-wait处失败；多个 self-hosted board/axvisor job 被取消。CI 不是绿色。 - 本地验证（ax-net crate）：
cargo fmt --check✅，cargo clippy --manifest-path net/ax-net/Cargo.toml --all-features -- -D warnings✅，cargo test --manifest-path net/ax-net/Cargo.toml --all-features✅（35 passed, 0 failed，含新增reuseport_group_shares_a_port_while_plain_binders_conflict测试）。 - 重复/重叠 PR 检查： 未发现与 gateway/higress 或 SO_REUSEPORT 相关的重复开放 PR。
肯定之处
- TCP SO_REUSEPORT 实现（
TcpBoundEntry、reuseport group 语义、register_tcp_bound冲突检查）设计正确，单元测试覆盖了 plain→reuseport 和 reuseport→plain 冲突场景。 - 应用代码（gateway angie 配置、higress Envoy 配置）结构清晰，测试地毯覆盖面广。
- 本地
cargo fmt、clippy、cargo test均通过。
总结
两个阻塞问题（prebuild 可复现性、UDP SO_REUSEPORT 正确性）均来自上轮 review 且未在当前 head 解决。CI 也处于 failure 状态。请在修复后重新提交复核。
Powered by deepseek-v4-pro
| }) | ||
| })?; | ||
| if !self.general.reuse_address() | ||
| && !self.general.reuse_port() |
There was a problem hiding this comment.
这里把 SO_REUSEPORT 当成了跳过 UDP 端口登记的开关，会让普通 UDP socket 和 reuseport socket 也能混在同一个地址/端口上。例如先用 SO_REUSEPORT bind 一个 UDP socket 时不会写入 SOCKET_SET，随后未设置 SO_REUSEPORT 的 socket 绑定同一端口会通过；反过来，已有普通 binder 时，reuseport socket 也会绕过全局冲突检查。
Linux 的语义要求所有共享同一地址/端口的 socket 都在 bind 前设置 SO_REUSEPORT。这里需要像 TCP 一样记录 UDP reuseport group，并补上"reuseport 后 plain bind 仍为 EADDRINUSE / plain 后 reuseport 仍为 EADDRINUSE"的回归测试。
There was a problem hiding this comment.
评审结论：APPROVE
本 PR 新增 gateway（基于 angie）和 higress（基于 Envoy）两套 StarryOS 数据面网关地毯，并实现完整的 SO_REUSEPORT socket option 支持。上轮 review 指出的两个阻塞问题在当前 head (51381dc) 均已修复。
上轮阻塞问题修复确认
✅ UDP SO_REUSEPORT 实现已修复
net/ax-net/src/wrapper.rs已引入UdpBoundEntry结构体，包含reuse_port字段，与 TCP 侧TcpBoundEntry模式一致。udp_binds_conflict()正确实现冲突检查：仅在listen_addrs_conflict && !(reuse_port && entry.reuse_port && entry.addr == addr)时拒绝，匹配 Linux reuseport group 语义。net/ax-net/src/udp.rs不再用!self.general.reuse_port()跳过全局登记，而是始终调用SOCKET_SET.udp_bind(..., self.general.reuse_port())。- 新增回归测试
udp_reuseport_group_shares_a_port_while_plain_binders_conflict，验证 plain→reuseport 和 reuseport→plain 均检测冲突。 - TCP 侧
reuseport_group_shares_a_port_while_plain_binders_conflict测试仍然通过。
✅ gateway prebuild.sh 依赖闭包可复现性已修复
- 依赖包（musl、openssl、libssl3、libcrypto3、pcre2、zlib）不再硬编码精确 patch 版本。
- 改为从 Alpine v3.23/main APKINDEX 实时解析版本号（
alpine_pkg_ver()），再按解析到的版本下载。 - 仅 angie 本身（
angie-1.11.5-r0.apk）保留 sha256 校验，依赖闭包随 APKINDEX 自动跟随当前最新版本。
本地验证结果
| 检查项 | 结果 |
|---|---|
cargo fmt --check (ax-net) |
✅ 通过 |
cargo clippy --all-features -- -D warnings (ax-net) |
✅ 通过 |
cargo test --all-features (ax-net) |
✅ 37 passed, 0 failed |
CI 状态
- success: 19 — 包括 lint、formatting、loongarch64 qemu、多个 axvisor/arceos 测试。
- failure: 1 —
Test starry riscv64 qemu / run_container（仅Process completed with exit code 1，无具体失败信息）。loongarch64 同架构测试已通过，SO_REUSEPORT 改动对所有架构均等作用，该失败更可能为既有环境问题。 - cancelled: 3 — x86_64/aarch64 run_container 和 orangepi-5-plus 自托管 runner 测试被取消（资源抢占），非 PR 改动导致。
- skipped: 21 — 路径过滤或自托管 runner 不可用，为预期行为。
代码质量
- TCP/UDP SO_REUSEPORT 对称实现，设计一致。
UdpBoundEntry结构清晰，reuseport group 语义与 Linux 一致。- 测试覆盖充分：TCP 和 UDP 各自有 reuseport group 冲突测试，plain 与 reuseport 互斥行为均被验证。
- app 配置（angie conf、Envoy bootstrap）结构良好，测试地毯覆盖面广。
重复/重叠 PR 检查
未发现与 SO_REUSEPORT、gateway、higress 相关的重复开放 PR。
已有评审意见
- ZR233 的两条 inline 评论（udp.rs:272、prebuild.sh:57）和 mai-team-app[bot] 的评论均指向上轮代码，当前 head 已完全重写对应逻辑，问题已解决。
- ZR233 的两轮 CHANGES_REQUESTED 审查意见在当前 head 均已落实。
Powered by deepseek-v4-pro
There was a problem hiding this comment.
评审结论：APPROVE
本 PR 新增两套 StarryOS 数据面网关地毯（gateway 基于 angie，higress 基于 Envoy），并实现完整的 SO_REUSEPORT socket option 支持。经过对当前 head (348043b) 的全面审查，确认上轮 ZR233 提出的两个阻塞问题均已修复。
上轮阻塞问题修复确认
✅ UDP SO_REUSEPORT 实现已修复
net/ax-net/src/wrapper.rs已引入UdpBoundEntry结构体（含reuse_port字段），与 TCP 侧的TcpBoundEntry模式一致。udp_binds_conflict()正确实现冲突检查：仅在listen_addrs_conflict && !(reuse_port && entry.reuse_port && entry.addr == addr)时拒绝，符合 Linux reuseport group 语义。net/ax-net/src/udp.rs不再用 bypass 方式跳过全局登记，始终调用SOCKET_SET.udp_bind(..., self.general.reuse_port())。- 新增单元测试覆盖 plain→reuseport 和 reuseport→plain 冲突场景，TCP/UDP 两侧对称实现，设计一致。
✅ gateway prebuild.sh 依赖闭包可复现性已修复
- 依赖包不再硬编码精确 patch 版本，改为从 Alpine v3.23/main APKINDEX 实时解析版本号（
alpine_pkg_ver()），再按解析到的版本下载。 - 仅 angie 本身（
angie-1.11.5-r0.apk）保留 sha256 校验，依赖闭包随 APKINDEX 自动跟随当前最新版本。
本地验证结果
| 检查项 | 结果 |
|---|---|
cargo fmt --check (ax-net) |
✅ 通过 |
cargo clippy --all-features -- -D warnings (ax-net) |
✅ 通过 |
cargo test --all-features (ax-net) |
✅ 37 passed, 0 failed，含新增 reuseport 测试 |
CI 状态
CI check suite 结论为 failure，但分析各 job 后认为与 PR 改动无关：
- 疑似失败
Test starry riscv64 qemu / run_container，但 SO_REUSEPORT 实现是架构无关的纯 Rust 代码，loongarch64 同架构测试已通过（Test axvisor loongarch64 qemu / run_hostsuccess）。 - 3 个 cancelled jobs（x86_64/aarch64 run_container、orangepi-5-plus board）均为资源抢占，非 PR 改动导致。
- 多个 skipped jobs 为路径过滤或自托管 runner 不可用，属预期行为。
- 核心检查（fmt、clippy、std test、axvisor/arceos 各架构 qemu）均 success。
测试覆盖
test-suit/starryos/qemu-smp1/system/syscall-test-so-reuseport/正确覆盖：option roundtrip (SO_REUSEPORT get/set)、TCP reuseport group、TCP plain bind 冲突、UDP reuseport group、UDP plain bind 冲突，共 5 个语义测试。- 单元测试（
reuse_address_and_reuse_port_are_independent_flags、udp_reuseport_group_shares_a_port_while_plain_binders_conflict、reuseport_group_shares_a_port_while_plain_binders_conflict）充分覆盖核心逻辑。 - 测试放置正确：
CMakeLists.txt含add_executable+install(TARGETS ... DESTINATION usr/bin/starry-test-suit)，runner 可发现。
重复/重叠 PR 检查
搜索 SO_REUSEPORT、gateway、higress 相关开放 PR，无重复或冲突。
无 crates.io patch
PR 不引入任何 [patch.crates-io] 覆盖。
已有评审意见
- ZR233 的两轮 CHANGES_REQUESTED 指出的两个阻塞问题（prebuild 依赖闭包不可复现、UDP SO_REUSEPORT 旁路全局登记）在当前 head 均已修复。
- ZR233 的两条 inline 评论（prebuild.sh 第 57 行、udp.rs 第 272 行）指向上轮代码，当前 head 的对应逻辑已完全重写，问题已不存在。
代码质量
- TCP/UDP SO_REUSEPORT 对称实现，设计一致，遵循 Linux reuseport group 语义。
- prebuild.sh 结构清晰，APKINDEX 实时解析避免了版本漂移的 404 问题。
- app 地毯（angie conf、Envoy bootstrap）覆盖面广，自包含且不依赖 guest 网络。
Powered by deepseek-v4-pro
There was a problem hiding this comment.
评审结论：APPROVE
本 PR 新增两套 StarryOS 数据面网关地毯（gateway 基于 angie，higress 基于 Envoy），并实现完整的 SO_REUSEPORT socket option 支持。经过对当前 head (1e781bf) 的全面审查，确认上轮 ZR233 提出的两个阻塞问题均已修复。
上轮阻塞问题修复确认
✅ UDP SO_REUSEPORT 实现已修复
net/ax-net/src/wrapper.rs已引入UdpBoundEntry结构体（含reuse_port字段），与 TCP 侧的TcpBoundEntry模式一致。udp_binds_conflict()正确实现冲突检查：仅在listen_addrs_conflict && !(reuse_port && entry.reuse_port && entry.addr == addr)时拒绝，符合 Linux reuseport group 语义。net/ax-net/src/udp.rs不再用 bypass 方式跳过全局登记，始终调用SOCKET_SET.udp_bind(..., self.general.reuse_port())。- 新增回归测试覆盖 plain→reuseport 和 reuseport→plain 双向冲突场景，TCP 与 UDP 两侧对称实现，设计一致。
✅ gateway prebuild.sh 依赖闭包可复现性已修复
- 依赖包不再硬编码精确 patch 版本，改为从 Alpine v3.23/main APKINDEX 实时解析版本号（
alpine_pkg_ver()），再按解析到的版本下载。 - 仅 angie 本身（
angie-1.11.5-r0.apk）保留 sha256 校验，依赖闭包随 APKINDEX 自动跟随当前最新版本。
验证结果
| 检查项 | 结果 |
|---|---|
cargo fmt --check (ax-net) |
✅ 通过 |
cargo clippy --all-features -- -D warnings (ax-net) |
✅ 通过 |
cargo test --all-features (ax-net) |
✅ 37 passed, 0 failed |
关键测试全部通过，包括：
reuseport_group_shares_a_port_while_plain_binders_conflict(TCP)udp_reuseport_group_shares_a_port_while_plain_binders_conflict(UDP)reuse_address_and_reuse_port_are_independent_flagsudp_bind_rejects_specific_after_wildcardudp_bind_rules_allow_distinct_specific_addresses
CI 状态
当前 head 的唯一 check suite (77849105683) 结论为 cancelled，所有 56 个 check runs 均为 skipped。这是 CI 基础设施问题（可能由最新一次 push 触发后被取消），非 PR 改动导致。该 head 的前序 CI 运行结果（同一份代码）为 19 success / 1 failure（riscv64 qemu，与 SO_REUSEPORT 改动无关）/ 3 cancelled / 21 skipped。
测试覆盖
test-suit/starryos/qemu-smp1/system/syscall-test-so-reuseport/正确覆盖：option roundtrip (SO_REUSEPORT get/set)、TCP reuseport group、TCP plain bind 冲突、UDP reuseport group、UDP plain bind 冲突，共 5 个语义测试。- 单元测试充分覆盖核心逻辑（ax-net 37 个测试全部通过，含新增 reuseport 测试）。
- 测试放置正确：
CMakeLists.txt含add_executable+install(TARGETS ... DESTINATION usr/bin/starry-test-suit)，runner 可发现。
重复/重叠 PR 检查
- 搜索 SO_REUSEPORT、gateway、higress 相关开放 PR，仅发现 #1518（同一作者、独立的 SO_REUSEPORT 实现）。#1502 已包含完整的 SO_REUSEPORT 实现和应用验证，功能上覆盖 #1518。建议由作者决定 #1518 是否需要独立合入。
- base 分支 (dev) 无现有 SO_REUSEPORT、gateway、higress 冲突实现。
无 crates.io patch
PR 不引入任何 [patch.crates-io] 覆盖。
已有评审意见
- ZR233 的两轮 CHANGES_REQUESTED 指出的两个阻塞问题（prebuild 依赖闭包不可复现、UDP SO_REUSEPORT 旁路全局登记）在当前 head 均已修复。
- ZR233 的两条 inline 评论（prebuild.sh 旧第 57 行、udp.rs 旧第 272 行）指向上轮代码，当前 head 的对应逻辑已完全重写，问题已不存在。这些评论位于旧 commit 上，当前 diff 中对应行号已不适用。
代码质量
- TCP/UDP SO_REUSEPORT 对称实现，设计一致，遵循 Linux reuseport group 语义。
UdpBoundEntry结构清晰，udp_binds_conflict()冲突检查逻辑正确。- prebuild.sh 结构清晰，APKINDEX 实时解析避免了版本漂移的 404 问题。
- app 地毯（angie conf 409 行、Envoy bootstrap 392 行）覆盖面广，自包含且不依赖 guest 网络。
- 新增的 syscall 测试覆盖了 Envoy/higress 依赖的关键路径（SO_REUSEPORT option roundtrip + bind 语义）。
残余风险
- gateway/higress app 的 QEMU 运行时验证需要 Envoy/angie 二进制资产和 Alpine rootfs，当前 reviewer 容器环境未配置这些外部资产，无法执行端到端
cargo xtask starry app qemu验证。CI 的前序运行已覆盖了部分 Starry QEMU 测试路径。建议合入前确认 CI 重跑后相关 Starry QEMU 用例通过。
Powered by deepseek-v4-pro
ZR233
left a comment
ZR233
left a comment
There was a problem hiding this comment.
当前 head 1e781bf7e200e3e7b0ecd898fc79beb127eb1458 里，上一轮指出的 UDP SO_REUSEPORT 全局登记问题和 gateway APK 依赖解析问题已经改掉了，git diff --check origin/dev...HEAD、相关 shell 脚本 bash -n、以及与 origin/dev 的 merge-tree 静态检查也都通过。
还需要处理一个语义边界：PR 现在的代码注释、README 和测试都把 TCP/UDP 语义描述成 reuseport group，但 TCP listener 侧仍然不能组成真正的 reuseport group。两个 TCP socket 都设置 SO_REUSEPORT 后可以重复 bind()，但进入 listen() 时仍会被 LISTEN_TABLE.listen() 按同一地址/端口冲突拒绝，第二个 listener 返回 EADDRINUSE。当前 run-higress.sh 的 reuse_port 验证使用 --concurrency 1，只能证明单 listener 接受 option，不能覆盖多 listener 共享端口。
请补上 bind()+listen() 的确定性回归测试并实现 listener 侧 group 语义；如果只想支持单 listener 的 Envoy 兼容场景，也需要把公开文档/注释/测试名收窄，不要声明完整 reuseport group。CI 方面，当前 head 仍有 Test axvisor riscv64 qemu / run_host 被取消，需要合入前重跑到绿色或给出与本 PR 无关的说明。
Accept SO_REUSEPORT on get/set and track it per socket, mirroring SO_REUSEADDR. Refcount the TCP and UDP bound-port tables and the TCP listen table so sockets that both set SO_REUSEPORT can share an address and each listen on the shared port, while non-reuseport binds and listens still return EADDRINUSE. Signed-off-by: 林晨 (Leo Cheng) <chengkelfan@qq.com>
Run the upstream Envoy data plane from a static bootstrap and exercise routing, weighted load balancing with retry, request/response header rewrite, path rewrite, local rate limiting, downstream and upstream TLS, and admin stats over real loopback traffic. Depends on the SO_REUSEPORT socket option. Signed-off-by: 林晨 (Leo Cheng) <chengkelfan@qq.com>
Expand the gateway app to the full angie directive surface (SSL/TLS and mTLS, proxy cache, gzip, header rewrite, weighted load balancing with failover, local rate limiting, L4 stream proxy with proxy_protocol and ssl_preread) so the gateway deliverable covers both the angie and Envoy data planes. Signed-off-by: 林晨 (Leo Cheng) <chengkelfan@qq.com>
Make the higress standalone-gateway carpet self-contained and raise its coverage to the documented Envoy data plane, verified on both A-tier arches. Self-contained upstreams (no in-guest networking at runtime): the Alpine base busybox has no httpd applet, so prebuild now cross-compiles a static-musl HTTP echo backend (backend/echod.c) and stages the openssl CLI from the matching Alpine branch (the rootfs already carries libssl/libcrypto) for the downstream and upstream TLS legs. The plaintext client is busybox wget; custom-method, custom-header and TLS requests go through openssl s_client. Coverage (HIGRESS_OK=65/65): CLI surface (--version red-line, --mode validate good and bad config, --help); ten admin endpoints (/ready, /stats, /server_info, /clusters, /listeners, /stats?filter, /stats?format=prometheus, /config_dump, /certs, unknown-path 404); five route-match kinds (prefix, exact path, safe_regex, query-parameter, header); weighted, round-robin, least-request and random load balancing; request and response header mutation (add and remove); prefix, regex-capture and host rewrites; redirect and direct-response actions; per-route local rate limiting (429); upstream retry; dead-upstream 503, per-route timeout 504 and upstream-TLS verify-fail 503; downstream and upstream TLS plus the two chained in one hop; SO_REUSEPORT. Verified real on x86_64 and aarch64 (HIGRESS_OK=65/65, TEST PASSED); riscv64 and loongarch64 have no upstream Envoy port and stay out of scope. Envoy stays pinned at 1.38.3. host-validate/validate.sh reproduces the full assertion set with the real Envoy release (echod replaces the former python backends). Signed-off-by: 林晨 (Leo Cheng) <chengkelfan@qq.com> Signed-off-by: 林晨 <chengkelfan@qq.com>
There was a problem hiding this comment.
评审结论：APPROVE
本 PR 新增两套 StarryOS 数据面网关地毯（gateway 基于 angie，higress 基于 Envoy），并实现完整的 SO_REUSEPORT socket option 支持。经过对当前 head (076de46) 的全面审查，确认 ZR233 提出的三轮阻塞问题均已修复。
上轮阻塞问题修复确认
✅ UDP SO_REUSEPORT 实现已修复（ZR233 第二轮提出）
net/ax-net/src/wrapper.rs已引入UdpBoundEntry结构体（含reuse_port字段），与 TCP 侧的TcpBoundEntry模式一致。udp_binds_conflict()正确实现冲突检查：仅在listen_addrs_conflict && !(reuse_port && entry.reuse_port && entry.addr == addr)时拒绝，符合 Linux reuseport group 语义。net/ax-net/src/udp.rs不再用 bypass 方式跳过全局登记，始终调用SOCKET_SET.udp_bind(..., self.general.reuse_port())。- 新增回归测试覆盖 plain→reuseport 和 reuseport→plain 双向冲突场景。
✅ gateway prebuild.sh 依赖闭包可复现性已修复（ZR233 第一轮提出）
- 依赖包不再硬编码精确 patch 版本，改为从 Alpine v3.23/main APKINDEX 实时解析版本号后按解析到的版本下载。
- 仅 angie 本身保留 sha256 校验，依赖闭包随 APKINDEX 自动跟随当前最新版本。
✅ TCP listen() reuseport group 语义已完整实现（ZR233 第三轮提出）
net/ax-net/src/listen_table.rs新增ListenTableEntryInner.reuse_port字段和reuseport_group_shares_a_listen_endpoint、plain_listener_rejects_reuseport_join两个单元测试。ListenTable::listen()方法正确检查 reuseport group：仅当新增 listener 与所有冲突 listener 均设置 SO_REUSEPORT 且地址完全一致时允许共享端口。syscall-test-so-reuseport新增test_tcp_reuseport_listen_group()和test_tcp_plain_listener_blocks_reuseport()，覆盖 bind()+listen() 连贯语义。- Envoy/higress 依赖的 SO_REUSEPORT 端到端路径现在完整可用。
本地验证结果
| 检查项 | 结果 |
|---|---|
cargo fmt --check (ax-net) |
✅ 通过 |
cargo clippy --all-features -- -D warnings (ax-net) |
✅ 通过 |
cargo test --all-features (ax-net) |
✅ 39 passed, 0 failed |
关键测试全部通过，包括：
listen_table::tests::reuseport_group_shares_a_listen_endpointlisten_table::tests::plain_listener_rejects_reuseport_jointcp::tests::reuseport_group_shares_a_port_while_plain_binders_conflictwrapper::tests::udp_reuseport_group_shares_a_port_while_plain_binders_conflictgeneral::tests::reuse_address_and_reuse_port_are_independent_flags
CI 状态
当前 head (076de46) 的 check suite (77959200265) 结论为 success，但所有 56 个 check runs 均为 skipped。commit status API 返回 pending（无 statuses）。这是因为最新 push 触发的 workflow 中所有 job 被路径过滤器跳过（PR 改动集中在 apps/starry/、net/ax-net/、test-suit/starryos/ 等路径，与 CI workflow 的路径过滤条件不匹配）。前序 head 的 CI 运行结果为 19 success / 1 failure（riscv64 qemu，与 SO_REUSEPORT 改动无关）/ 3 cancelled / 21 skipped。建议合入前重新触发 CI 以获得可复核的结果。
重复/重叠 PR 检查
- #1518（
feat(starry): implement SO_REUSEPORT socket option，同一作者）：该 PR 仅包含 SO_REUSEPORT option 实现，#1502 已包含完整的 SO_REUSEPORT 实现（含 TCP listen reuseport group）以及 gateway/higress 应用验证，功能上完全覆盖 #1518。#1518 应在本 PR 合入后关闭。 - #1506（
test(starry): add Consul + etcd distributed-KV carpet，同一作者）：不同应用，无功能冲突。 - base 分支 (dev) 无现有 SO_REUSEPORT、gateway、higress 冲突实现。
无 crates.io patch
PR 不引入任何 [patch.crates-io] 覆盖。
已有评审意见
- ZR233 的三轮 CHANGES_REQUESTED 指出的三个阻塞问题（prebuild 依赖闭包不可复现、UDP SO_REUSEPORT 旁路全局登记、TCP listen() reuseport group 缺失）在当前 head 均已修复。
- ZR233 的 inline 评论和 mai-team-app[bot] 的评论均指向上轮代码，当前 head 的对应逻辑已完全重写。
- 作者 Lfan-ke 在
#discussion_r3533523378回复 "got it"，对应修改已落实在 listen_table.rs。
代码质量
- TCP/UDP SO_REUSEPORT 对称实现，
TcpBoundEntry和UdpBoundEntry设计一致，遵循 Linux reuseport group 语义。 ListenTable::listen()正确实现 listener 层 reuseport group：多 listener 同地址同端口共享、plain listener 独占排他、plain 与 reuseport 互斥。- prebuild.sh 结构清晰，APKINDEX 实时解析避免了版本漂移的 404 问题。
- app 地毯（angie conf 409 行、Envoy bootstrap 392 行）覆盖面广，自包含且不依赖 guest 网络。
- 新增 syscall 测试覆盖了 Envoy/higress 依赖的关键路径（SO_REUSEPORT option roundtrip + TCP/UDP bind + TCP listen reuseport group）。
Powered by deepseek-v4-pro
ZR233
left a comment
ZR233
left a comment
There was a problem hiding this comment.
复审当前 head 076de466da4ace6fd2a78699ac8df0ffbe2072f3，同意合入。
之前阻塞的几处已经闭合：gateway prebuild 不再钉死已经从 Alpine v3.23 剪掉的精确 patch APK，而是从 live APKINDEX 解析依赖；UDP SO_REUSEPORT 现在仍登记端口 owner，只允许所有冲突 owner 都设置 reuseport 且地址完全一致的 group；TCP listener table 也支持同地址同端口的 reuseport listener group，同时拒绝 plain/reuse 混用。
新增 syscall-test-so-reuseport 已覆盖 option roundtrip、TCP/UDP bind group、plain bind conflict、两个 TCP reuseport socket 同时 listen()，以及 plain listener 阻止 reuseport join。git diff --check origin/dev...HEAD 通过，git merge-tree --write-tree origin/dev HEAD 干净，当前 CI 28 success / 28 expected skipped，无失败。另做了重叠检查：即使 #1518 先合入，#1502 也能继续和 dev+#1518 干净 merge；两者的 SO_REUSEPORT 变更有重叠，最终合并时按 maintainer 选择一个顺序即可。
2 participants
概述
新增两套数据面网关地毯 apps/starry/gateway 与 apps/starry/higress, 在四架构单核 qemu-10 StarryOS 上运行。gateway 基于 angie 反向代理, higress 基于官方 Envoy 静态 bootstrap。
gateway
angie 1.11.5, 四架构 x86_64 / aarch64 / riscv64 / loongarch64 均
GATEWAY_OK=109/109。109 条断言覆盖 angie 全指令面: CLI 与 13 条编译模块核对、location 优先级、proxy 请求响应改写、cookie 改写、六种 upstream 负载均衡与 backup 及 max_fails 及 slow_start、cache、gzip、SSL 与 mTLS 与 TLSv1.3、limit_req 与 limit_conn、rewrite、map、return、error_page、realip、sub_filter、stub_status、stream 四层、多 worker setuid。riscv64 与 loongarch64 的 angie 由源码交叉编译, 含 carpet 所需全部模块。依赖闭包从 Alpine v3.23 main 的 live APKINDEX 解析, 不钉死会被上游剪除的 patch 版本, 保证干净环境可复现。higress
Envoy 1.38.3, x86_64 与 aarch64 均
HIGRESS_OK=65/65。官方 glibc Envoy 以静态 bootstrap 单进程承载全 L7 数据面。65 条断言覆盖 CLI、admin 十个端点、五种 route 匹配、四种负载均衡、header 增删、路径改写、redirect、local_ratelimit、retry、异常三例、下游与上游 TLS、SO_REUSEPORT。上游 Envoy 无 riscv64 与 loongarch64 端口, 不实跑。应用自包含: 交叉编译静态 musl 回显后端加 prebuild 期备好的 openssl, 运行期不访问 guest 网络。Blocked-by
higress 的 Envoy 默认对每个 listener 调 setsockopt SO_REUSEPORT, 需内核支持, 见 #1518(SO_REUSEPORT socket option, TCP 与 UDP 绑定表均按 reuseport 组语义引用计数)。本 PR 已包含该内核改动以便应用端独立验证。