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

[newrelic-logging] Adjust order of env list in daemonset to fix dependency reference #1441

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

[newrelic-logging] Adjust order of env list in daemonset to fix dependency reference #1441

wants to merge 6 commits into from

Commits on Jul 25, 2024

  1. [newrelic-logging] Adjust order of env list in daemonset 

    When using persistentVolume mode, the env variable FB_DB references NODE_NAME. This reference is currently not resolved, because NODE_NAME is defined after FB_DB. This commit fixes this, by moving the definition of NODE_NAME before the definition of FB_DB.
    @nluedema
    nluedema authored Jul 25, 2024
    Configuration menu
    Copy the full SHA
    192cba5 View commit details
    Browse the repository at this point in the history

  2. [newrelic-logging] Bump chart version

    @nluedema
    nluedema authored Jul 25, 2024
    Configuration menu
    Copy the full SHA
    3b7a1b5 View commit details
    Browse the repository at this point in the history

Commits on Oct 15, 2024

  1. Revert "[newrelic-logging] Bump chart version" 

    This reverts commit 3b7a1b5.
    @nluedema
    nluedema committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    38f3137 View commit details
    Browse the repository at this point in the history

  2. Revert "[newrelic-logging] Adjust order of env list in daemonset" 

    This reverts commit 192cba5.
    @nluedema
    nluedema committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    648c0a9 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'master' into master

    @nluedema
    nluedema authored Oct 15, 2024
    Configuration menu
    Copy the full SHA
    6e49309 View commit details
    Browse the repository at this point in the history

  4. [newrelic-logging] Fix FB_DB env variable being dependent on definiti… 

    …on order

The FB_DB environment variable depends on the environment variable
NODE_NAME. This variable is currently resolved by Helm, which makes
FB_DB depend on the environment variable definition order.

To fix this, we make NODE_NAME being resolved by Fluent Bit instead of
Helm.
    @nluedema
    nluedema committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    d8cc15a View commit details
    Browse the repository at this point in the history