Skip to content

fix(config): ensured env vars take precedence over in-code config#2409

Open
aryamohanan wants to merge 8 commits intofix-config-precedencefrom
fix-config-order
Open

fix(config): ensured env vars take precedence over in-code config#2409
aryamohanan wants to merge 8 commits intofix-config-precedencefrom
fix-config-order

Conversation

@aryamohanan
Copy link
Contributor

@aryamohanan aryamohanan commented Mar 16, 2026

This is the first PR addressing configuration precedence.

The goal of this change is to ensure the correct order between environment variables and in-code configuration:

env > in-code

Currently, the precedence between these two sources is mixed. This PR corrects that behavior.

Added missing logging in the configuration file when a config is set.

Scope of This PR

This PR only fixes the precedence between:

  • Environment variables
  • In-code configuration

Agent configuration precedence will be addressed in separate follow-up PRs.

Future Work

Additional PRs will follow to correct the agent configuration order.

Migration Plan

This change will be released in the next major version (v6), planned for Q2 2026.

ref https://jsw.ibm.com/browse/INSTA-80965

Epic : https://jsw.ibm.com/browse/INSTA-817

@aryamohanan aryamohanan changed the title Fix config order fix(config): ensure env vars take precedence over in-code config Mar 16, 2026
@aryamohanan aryamohanan marked this pull request as ready for review March 18, 2026 07:11
@aryamohanan aryamohanan requested a review from a team as a code owner March 18, 2026 07:11
@aryamohanan aryamohanan added feature-branch Target is a feat branch and removed merge later labels Mar 18, 2026
@aryamohanan aryamohanan changed the title fix(config): ensure env vars take precedence over in-code config fix(config): ensured env vars take precedence over in-code config Mar 18, 2026
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature-branch Target is a feat branch v6

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant