Skip to content

Helm Pinot chart: add optional namespace, workload annotations, and minion image override #18794

@davidballano

Description

@davidballano

Summary

I opened a PR to upstream a small set of backward-compatible Helm chart improvements from our production usage.

Related PR

#18793

What this issue tracks

This issue tracks review and merge of the following chart improvements:

  • Optional namespace creation via a values flag
  • Workload-level annotations for controller, broker, server, minion, and stateless minion workloads
  • Optional minion-specific image override with fallback to the main image values
  • Minor template whitespace cleanup in server configmap rendering

Why this is needed

These changes improve operational flexibility (namespace management, annotations for platform integrations, independent minion image control) while preserving current behavior by default.

Backward compatibility

All new knobs default to no-op behavior:

Namespace creation remains disabled unless explicitly enabled
Annotation maps default to empty
Minion image override falls back to existing image settings when not set

Thanks guys!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions