Skip to content

Conversation

@rawadhossain
Copy link

What this PR does:
Applies defaults.Set() when decoding stage configurations in the wait and analysis stage plugins, ensuring default values are initialized correctly during config loading.

Why we need it:

  • Stage plugins need to apply defaults during config decoding.
  • wait and analysis were unmarshalled without calling defaults.Set().
  • Hence config defaults were not being applied.

Part of Issue #6174

Does this PR introduce a user-facing change?:
No

  • How are users affected by this change: Stage configurations now correctly apply default values when fields are omitted, as expected behavior.
  • Is this breaking change: No
  • How to migrate (if breaking change): N/A

Signed-off-by: Rawad Hossain <rawad.hossain00@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant