The GitHub Actions job "Benchmarks PR Comment" on texera.git/main has succeeded. Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).
Head commit for run: ee062a20d3777d1409508766dba18187e3719035 / ali risheh <[email protected]> refactor(k8s): consolidate gateway resources under the gateway folder (#6472) ### What changes were proposed in this PR? A small, behavior-preserving refactor of the Helm chart layout under `bin/k8s`. The agent-service **`BackendTrafficPolicy`** is a gateway-layer [Envoy Gateway](https://gateway.envoyproxy.io/) resource — it configures consistent-hash load balancing on the gateway for the agent-service `HTTPRoute` (so all requests for a given workflow pin to the same replica). Despite being a gateway concern, it was living under `templates/base/agent-service/` next to the app-tier `Deployment`/`Service`/`Secret`. It is the same class of resource as the `SecurityPolicy` (`gateway-security-policy.yaml`) that already lives in `templates/base/gateway/`, so this PR moves it there and renames it to the folder's `gateway-*` convention: ``` templates/base/agent-service/agent-service-backend-traffic-policy.yaml -> templates/base/gateway/gateway-agent-traffic-policy.yaml ``` After this move, **every** gateway-related resource (`GatewayClass`, `Gateway`, `HTTPRoute`, `Backend`, `SecurityPolicy`, `BackendTrafficPolicy`) lives under `templates/base/gateway/`, and no gateway resource remains scattered in a service folder (verified by grepping all templates for the `gateway.envoyproxy.io` / `gateway.networking.k8s.io` API groups — this was the only stray one). This is purely organizational: no values, resource names, gating conditions, or rendered output change. It continues the `base`/`aws`/`on-prem` template reorganization from #5757. ### Any related issues, documentation, discussions? Part of the deployment-unification effort tracked in #5891 (housekeeping follow-up to the template reorg in #5757). No functional dependency on the object-storage work (#5932, #6295). ### How was this PR tested? Verified the move is byte-for-byte behavior-preserving by diffing the rendered manifests before and after: ```bash cd bin/k8s helm dependency build # fetch subcharts # render with the file in its OLD location helm template test . > /tmp/before.txt # render with the file in its NEW location (this PR) helm template test . > /tmp/after.txt diff /tmp/before.txt /tmp/after.txt ``` The only differences are: 1. The `# Source:` provenance comment path (`.../agent-service/...` → `.../gateway/...`) — expected, cosmetic. 2. The auto-generated `encryptionKey` value — Helm regenerates this random secret on every invocation; unrelated to this change. The `BackendTrafficPolicy` resource itself (name `<release>-agent-service-traffic-policy`, its `targetRefs`, and the `X-Agent-Workflow-Id` consistent-hash config) renders identically. `helm template` succeeds for the default value set. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Claude Opus 4.8) Co-authored-by: Claude Opus 4.8 (1M context) <[email protected]> Report URL: https://github.com/apache/texera/actions/runs/29564319489 With regards, GitHub Actions via GitBox
