[INFRA-429] - feat(plane-enterprise): adjust PI_BASE_URL and cross-wire live <-> pi-api envs

[akshat5302]

What

Related plane-enterprise adjustments around the PI and live services:

1. Removes the hardcoded /pi path suffix from PI_BASE_URL in app-vars (the path is already supplied separately via PI_BASE_PATH: "/pi").
2. Cross-wires the live and pi-api services so each can reach the other.

Changes

• templates/config-secrets/app-env.yaml — PI_BASE_URL now renders as http(s)://<licenseDomain> instead of .../pi (both TLS and non-TLS branches). PI_BASE_PATH remains /pi.
• templates/config-secrets/pi-api-env.yaml
  • pi-api-vars ConfigMap: add LIVE_BASE_URL (in-cluster live service) and LIVE_BASE_PATH: "/live", matching app-env.
  • pi-api-secrets Secret: add LIVE_SERVER_SECRET_KEY, sourced from the same env.live_server_secret_key value as live-env so the two stay in sync.
• templates/config-secrets/live-env.yaml — add PI_BASE_URL and PI_BASE_PATH: "/pi" to the live-vars ConfigMap.
• README.md — document LIVE_SERVER_SECRET_KEY as a required key for pi_api_env_existingSecret (must match the value used for the live service).
• Chart.yaml — chart version bumped to 2.6.5 (required for the release workflow).

Verification

• helm lint charts/plane-enterprise → passes
• helm template renders all new envs correctly; LIVE_SERVER_SECRET_KEY matches the live-env value

Work item: INFRA-429

[coderabbitai]

Walkthrough

The PI_BASE_URL value in the plane-enterprise app-env ConfigMap is updated to remove the trailing /pi path segment in both the TLS and non-TLS conditional branches. The Helm chart version is bumped from 2.6.4 to 2.6.5.

Changes

PI_BASE_URL fix and chart version bump

Layer / File(s)	Summary
PI_BASE_URL suffix removal and chart version bump charts/plane-enterprise/templates/config-secrets/app-env.yaml, charts/plane-enterprise/Chart.yaml	PI_BASE_URL no longer appends /pi to the license-domain-based URL in either the TLS or non-TLS branch. Chart version incremented to 2.6.5.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• makeplane/helm-charts#231: Also modifies charts/plane-enterprise/Chart.yaml with a patch version bump alongside changes to app-env.yaml.
• makeplane/helm-charts#239: Bumps the version field in charts/plane-enterprise/Chart.yaml.
• makeplane/helm-charts#247: Bumps charts/plane-enterprise/Chart.yaml version in the same 2.6.x series.

Suggested reviewers

• pratapalakshmi

Poem

🐇 A URL trimmed, no /pi in sight,
The license domain now points just right.
A version bumped from four to five,
Keeps the Helm chart fresh and alive.
Hippity-hop, the fix is deployed! 🚀

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly reflects the main PI_BASE_URL change and related environment wiring update, and is specific enough for history scanning.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch remove-pi-base-url-path

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[makeplane]

Linked to Plane Work Item(s)

• (INFRA-429) Plane-EE: remove /pi base path from PI_BASE_URL

This comment was auto-generated by Plane