Repository navigation
test(gateway): cover all APIs with real-service E2E - #241
Conversation
Exercise all 45 Gateway API operations with both policy engines and five storage profiles, including two complete shared-state replicas. Guard route/documentation coverage and refresh the API guide against live protocols. Fix missing KBS management authentication, AS policy-list routing, wide MySQL audit table initialization and burst-write transaction retry starvation exposed by the suite. Signed-off-by: Jiale Zhang <zhangjiale@linux.alibaba.com>
|
@jialez0 ,您好,您的请求已接收,请耐心等待结果。 |
|
@jialez0 ,您好,未检测到有镜像需要构建,如需重新检测请评论 /start 。 |
jialez0
left a comment
There was a problem hiding this comment.
Self-review of a8d1740 completed: checked route/contract/docs parity, real upstream processes, isolated loopback-only database ownership and cleanup, independent JWT verification and JWE decryption, shared sessions and persistence, cache invalidation, and bounded deadlock backoff without CAS changes. Gateway DYNAMIC table options affect new tables only. Artifacts contain operation counters only. Local E2E passed across five profiles with both engines (three full VM repetitions); 32-writer regression passed 20 rounds. No blocking findings. Hardware/external-service and deployment-auth boundaries are documented. Waiting for all CI before rebase-only merge.
Signed-off-by: Jiale Zhang <zhangjiale@linux.alibaba.com>
|
@jialez0 ,您好,您的请求已接收,请耐心等待结果。 |
|
@jialez0 ,您好,未检测到有镜像需要构建,如需重新检测请评论 /start 。 |
jialez0
left a comment
There was a problem hiding this comment.
Follow-up self-review: 682610d changes only Actionlint reporting. The earlier failure was reviewdog PATCH /check-runs returning 404, with zero lint annotations. Local reporting plus fail_level=error and nofilter preserves full workflow validation while removing check-run write dependence; contents remain read-only. All functional source/tests are unchanged from the locally verified revision.
Signed-off-by: Jiale Zhang <zhangjiale@linux.alibaba.com>
|
@jialez0 ,您好,您的请求已接收,请耐心等待结果。 |
|
@jialez0 ,您好,未检测到有镜像需要构建,如需重新检测请评论 /start 。 |
jialez0
left a comment
There was a problem hiding this comment.
Final workflow self-review at 5faf4cf: upgraded the existing setup-go action to v6 because full Actionlint correctly rejects deprecated Node20 actions. Workflow Files and Go Test/Vet are now green. Re-ran KV Clippy with -D warnings and all 14 KV tests successfully. No functional source changed after the local E2E matrix. Awaiting remaining checks before rebase merge.
Summary
Add real-service, contract-gated E2E coverage for all 45 logical Trustee Gateway API operations. The suite starts Gateway, KBS, gRPC AS, RESTful AS and RVPS production binaries; no upstream HTTP/gRPC services are mocked.
Defects exposed and fixed
Validation
CI creates isolated test databases, deliberately exercises MySQL COMPACT defaults, and uploads only credential-free API operation counters. Test keys/configuration/tokens/cookies/evidence and service logs are not uploaded.
Validation boundaries
The software sample verifier exercises the real protocol/policy/storage chain and report-data binding, not physical TDX/SNP/CSV/TPM authenticity. This does not claim hardware quote generation, production PKI, live cloud KMS/HSM or external OCI/Rekor validation. There is no generic Gateway cache-management endpoint: the suite checks observable policy/RVPS/key-cache invalidation. WASM in-memory persistence and existing WASM checks are retained.
Merge using rebase and merge only, after review and all CI checks pass.