Skip to content

upstream_ha: added environment variable support - #9255

Merged
edsiper merged 2 commits into
masterfrom
leonardo-master-upstream_environment_variable_support
Aug 21, 2024
Merged

upstream_ha: added environment variable support#9255
edsiper merged 2 commits into
masterfrom
leonardo-master-upstream_environment_variable_support

Conversation

@leonardo-albertovich

@leonardo-albertovich leonardo-albertovich commented Aug 21, 2024

Copy link
Copy Markdown
Contributor

No description provided.

Leonardo Alminana
upstream_ht: added environment variable support
b17d357 
Signed-off-by: Leonardo Alminana <leonardo.alminana@chronosphere.io>
@leonardo-albertovich leonardo-albertovich mentioned this pull request Aug 21, 2024
Closed
Leonardo Alminana
upstream_ht: added support for inherited property lookup
feb67e2 
Signed-off-by: Leonardo Alminana <leonardo.alminana@chronosphere.io>
@edsiper

edsiper commented Aug 21, 2024

Copy link
Copy Markdown
Member

thanks for the PR.

For v3.2 we should port this interface to use a config_map.

@edsiper edsiper added this to the Fluent Bit v3.1.7 milestone Aug 21, 2024
@edsiper
edsiper merged commit d25efca into master Aug 21, 2024
@edsiper
edsiper deleted the leonardo-master-upstream_environment_variable_support branch August 21, 2024 23:15
@BrewTestBot BrewTestBot mentioned this pull request Sep 2, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@edsiper edsiper Awaiting requested review from edsiper edsiper is a code owner
@fujimotos fujimotos Awaiting requested review from fujimotos
@koleini koleini Awaiting requested review from koleini

Assignees

No one assigned

Labels

docs-required ok-to-test

Projects

None yet

Milestone

Fluent Bit v3.1.7

Development

Successfully merging this pull request may close these issues.

2 participants

@leonardo-albertovich @edsiper