stacklok / stacklok/mecatl

mecak8s chart: close invalid rendering states and scope local Redis resources

Open
#1,519 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Go
Stars
152
Forks
16
Avg merge
14h 48m
Merged PRs (30d)
536

Description

Problem

Several invalid or collision-prone value combinations pass Helm schema/template validation:

  • Local Redis can render --redis-url="".
  • Local Redis Service and StatefulSet names are hard-coded as redis, colliding across releases.
  • A ClusterIP Service can render explicit nodePort fields.
  • Invalid image digests and Kubernetes fullnameOverride values are accepted.
  • Redis URL syntax and learning-store ports above 65535 render successfully but fail at runtime.
  • Extra volumes/mounts can collide with chart-owned names such as tmp.

Proposed direction

Make local Redis release-scoped and derive its endpoint from the helper-generated Service name. Tighten JSON Schema and template-side cross-field validation for Kubernetes names, OCI digests, host/port syntax, Service type/nodePort combinations, and reserved resource names.

Acceptance criteria

  • Local mode always renders a usable release-scoped Redis endpoint.
  • Multiple local releases can coexist in one namespace.
  • Every listed invalid value fails before manifests reach the API server or process startup.
  • Negative render tests pin each rejected case.
  • The local Redis image is configurable and can be digest-pinned.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the Helm chart's JSON Schema, templates, helper-generated Service naming, and existing render-test harness. Add negative render coverage for each listed invalid combination and verify that local Redis uses a release-scoped endpoint, supports configurable digest-pinned images, and allows multiple releases to coexist.

Written by the indexing model from the issue text.

Assessment

Tech stack
helm, kubernetes, redis
Domain
databases, infrastructure
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.