The GitHub Actions job "Required Checks" on texera.git/main has failed. Run started by GitHub user github-merge-queue[bot] (triggered by github-merge-queue[bot]).
Head commit for run: e55d1abb07125d6fee698821818cc7785f7c3120 / ali risheh <[email protected]> refactor(k8s): organize Helm templates into base/aws/on-prem (#5757) ### What changes were proposed in this PR? This is a **behavior-neutral refactoring** of the Helm chart under `bin/k8s/` — it reorganizes the flat `templates/` directory into a clear, self-documenting layout with **no value or logic changes**. It is the first in a planned series of small, non-breaking PRs that make the chart cleanly deployable on AWS/EKS while keeping on-prem/local as the unchanged default. Helm renders `templates/**` recursively, so moving files into subdirectories does not change rendered output. Templates are now grouped two ways: - By **where they apply** — `templates/base/` (shared by every deployment), `templates/on-prem/` (`minio-persistence.yaml`), and `templates/aws/` (placeholder for AWS-only, value-gated templates added in later PRs; empty for now). - Within `base/`, by **component** — one subfolder per service holding all of its manifests, e.g. `base/access-control-service/`, `base/gateway/`, `base/workflow-computing-unit-manager/`, `base/workflow-computing-unit-pool/`, etc. A `templates/README.md` documents the `base`/`aws`/`on-prem` and per-component conventions, and `.helmignore` is added so `*.md`/`.gitkeep` are not loaded as manifests. No values were renamed, added, or removed, and `templates/aws/` is empty, so the default on-prem/local install is unchanged. ### Any related issues, documentation, discussions? Closes #5892 (refactor sub-task). Part of #5891 — unify AWS (EKS) and on-premise Kubernetes deployment under `bin/k8s` (parent feature). Follows the design discussion in #5641. This is the first of a planned series of incremental, non-breaking PRs to add AWS/EKS deployment support to the chart; later PRs build on this structure (pluggable object storage, node placement, AWS load balancer, computing-unit warm pool, eksctl/runbook). ### How was this PR tested? This is a pure chart refactor, so it was verified to be a no-op: **Render no-op proof:** `helm template texera bin/k8s` was captured before and after the file moves. The output is byte-identical — same resources both times — with the only differences being the `# Source:` provenance comments (the new paths) and a per-render randomized secret value. `helm lint bin/k8s` passes. No unit tests were added because the change is limited to the organization of Helm chart files, which is validated by the render-diff and `helm lint` above rather than by JVM unit tests. ### 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/28116286818 With regards, GitHub Actions via GitBox
