Skip to content

Add test case for affix update consumer bug#106284

Merged
mosche merged 1 commit intoelastic:mainfrom
mosche:106283_affix_update_bug
Mar 13, 2024
Merged

Add test case for affix update consumer bug#106284
mosche merged 1 commit intoelastic:mainfrom
mosche:106283_affix_update_bug

Conversation

@mosche
Copy link
Copy Markdown
Contributor

@mosche mosche commented Mar 13, 2024

This test case reproduces #106283 and shows the undesired invocation of the update consumer with the default value.

@mosche mosche added >test Issues or PRs that are addressing/adding tests :Core/Infra/Settings Settings infrastructure and APIs Team:Core/Infra Meta label for core/infra team labels Mar 13, 2024
@mosche mosche requested a review from a team March 13, 2024 10:57
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

Copy link
Copy Markdown
Contributor

@ldematte ldematte left a comment

Choose a reason for hiding this comment

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

LGTM

@mosche
Copy link
Copy Markdown
Contributor Author

mosche commented Mar 13, 2024

Thanks @ldematte

@mosche mosche merged commit 9bf1d5d into elastic:main Mar 13, 2024
@mosche mosche deleted the 106283_affix_update_bug branch March 13, 2024 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Settings Settings infrastructure and APIs Team:Core/Infra Meta label for core/infra team >test Issues or PRs that are addressing/adding tests v8.14.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants