Skip to content

gateway api impl#36

Open
KevinGruber2001 wants to merge 1 commit into
mainfrom
feat/gateway-api-implementation
Open

gateway api impl#36
KevinGruber2001 wants to merge 1 commit into
mainfrom
feat/gateway-api-implementation

Conversation

@KevinGruber2001
Copy link
Copy Markdown
Collaborator

@KevinGruber2001 KevinGruber2001 commented May 22, 2026

Summary by CodeRabbit

New Features

  • Added Gateway API HTTPRoute support for automatic cache and repository service routing
  • New configuration options for gateway integration, including hostname customization and routing parent references

Chores

  • Helm chart version updated to 0.5.2

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 22, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 41b61567-060b-4e33-a3c8-a0f6fd8bff3f

📥 Commits

Reviewing files that changed from the base of the PR and between 60e44b1 and 5e4cbf4.

📒 Files selected for processing (5)
  • chart/Chart.yaml
  • chart/templates/_helpers.tpl
  • chart/templates/httproute-cache.yaml
  • chart/templates/httproute-repo.yaml
  • chart/values.yaml

📝 Walkthrough

Walkthrough

This PR adds optional Kubernetes Gateway API HTTPRoute support to the Helm chart for EduIDE shared cache. It introduces gateway configuration values, hostname resolution helpers, conditional HTTPRoute templates for cache and reposilite backends, and bumps the chart version to 0.5.2.

Changes

Gateway API HTTPRoute Integration

Layer / File(s) Summary
Gateway configuration and service naming
chart/values.yaml
New gateway configuration section enables HTTPRoute rendering, specifies base host and per-route parent references, with optional explicit hostname overrides. Reposilite configuration adds fullnameOverride to control the service name used by HTTPRoute backends.
Hostname resolution helpers
chart/templates/_helpers.tpl
Two template helpers (cacheHostname and repoHostname) compute route hostnames from explicit overrides or derive them from a base host, failing template rendering if required values are missing.
Cache and repo HTTPRoute templates
chart/templates/httproute-cache.yaml, chart/templates/httproute-repo.yaml
Conditional Helm templates render Kubernetes HTTPRoute resources. Cache route renders when gateway is enabled; repo route renders when both gateway and reposilite are enabled. Both configure parentRefs, compute hostnames via helpers, and route all traffic to their respective backend services on port 8080.
Chart version release
chart/Chart.yaml
Chart version bumped from 0.5.0 to 0.5.2 for release and Docker image tagging.

🎯 2 (Simple) | ⏱️ ~10 minutes

🐰 A gateway appears, routes unfold,
Cache and repo stories told,
Hostnames compute, templates take flight,
Version bumped: 0.5.2 shines bright! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'gateway api impl' is vague and uses abbreviations. While it relates to the changeset's gateway API implementation, it lacks clarity and specificity needed for meaningful scanning of commit history. Expand the title to be more descriptive, e.g., 'Add Gateway API HTTPRoute implementation for cache and repo services' to clearly convey the scope and purpose of the changes.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/gateway-api-implementation

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

Chart Preview Ready

helm install test oci://ghcr.io/eduide/charts/eduide-shared-cache --version 0.5.2-pr.36

Updated: 10eb257

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant