Skip to content

Conversation

@colinmollenhour
Copy link
Contributor

If the config file is this:

env:
  ENVIRONMENT: x
  INSTANCE_UID: x

# Main Fluent Bit Configuration
service:
  flush: 1
  grace: 0
  daemon: off
  log_level: debug
  parsers_file: parsers.yaml

# Include input configurations
includes:
  - inputs/*.yaml

# Output to stdout for testing
pipeline:
  filters:
    - name: modify
      match: "*"
      add:
        - environment ${ENVIRONMENT}
        - instance_uid ${INSTANCE_UID}

  outputs:
    - name: stdout
      match: "*"

If I run this with docker run -e ENVIRONMENT=dev ... it is still reported as "x"

If the config file is this:

```
env:
  ENVIRONMENT: x
  INSTANCE_UID: x

# Main Fluent Bit Configuration
service:
  flush: 1
  grace: 0
  daemon: off
  log_level: debug
  parsers_file: parsers.yaml

# Include input configurations
includes:
  - inputs/*.yaml

# Output to stdout for testing
pipeline:
  filters:
    - name: modify
      match: "*"
      add:
        - environment ${ENVIRONMENT}
        - instance_uid ${INSTANCE_UID}

  outputs:
    - name: stdout
      match: "*"
```

If I run this with `docker run -e ENVIRONMENT=dev ...` it is still reported as "x"

Signed-off-by: Colin Mollenhour <[email protected]>
@colinmollenhour colinmollenhour requested review from a team as code owners May 29, 2025 14:00
Copy link
Contributor

@esmerel esmerel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two stylistic changes, otherwise looks good to me.

@esmerel
Copy link
Contributor

esmerel commented May 29, 2025

Made a couple of minor changes and merging. Thanks @colinmollenhour !

@esmerel esmerel merged commit c349252 into fluent:master May 29, 2025
5 checks passed
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.

3 participants