Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

⚠️ ✨ Make helm chart values extensible per component #638

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

⚠️ ✨ Make helm chart values extensible per component #638

wants to merge 4 commits into from

Commits on Nov 25, 2024

  1. fix: add comments to large scope if & for loops 

    The intention is to reduce time to comprehension of the chart.
    @rbjorklin
    rbjorklin committed Nov 25, 2024
    Configuration menu
    Copy the full SHA
    34fde4e View commit details
    Browse the repository at this point in the history

  2. feat: restructure chart values to allow for extending mappings 

    This commit breaks the colon separated string into multiple key-value
pairs. This is in preparation for extension points to the chart.

BREAKING CHANGE: This changes the format of the values.yaml file.
Chart uses must make adaptations.
    @rbjorklin
    rbjorklin committed Nov 25, 2024
    Configuration menu
    Copy the full SHA
    7b95c1a View commit details
    Browse the repository at this point in the history

  3. feat: extend values.yaml mapping with 'manifestPatches'

    @rbjorklin
    rbjorklin committed Nov 25, 2024
    Configuration menu
    Copy the full SHA
    4b25b86 View commit details
    Browse the repository at this point in the history

  4. feat: extend values.yaml mapping with 'additionalManifests'

    @rbjorklin
    rbjorklin committed Nov 25, 2024
    Configuration menu
    Copy the full SHA
    0eeb087 View commit details
    Browse the repository at this point in the history