Skip to content

Conversation

@jsoriano
Copy link
Member

We are logging now Package does not embed ECS mappings every time we check if we should embed ECS mappings because they are referenced in the package.
This is not so relevant because the final reason to import ECS schema during validation depends on two other variables at the moment, but it sometimes floods the logs. For example when running the pipeline tests of the Azure package, this line is printed 33 times in a row, and is not deterministic on its own for the actual result.

This change removes this debug log, and when the mappings are effectively considered for fields validation, the possible reasons for importing are printed too.

@jsoriano jsoriano requested a review from a team April 16, 2024 10:58
@jsoriano jsoriano self-assigned this Apr 16, 2024
@jsoriano jsoriano requested a review from mrodm April 16, 2024 11:55
@elasticmachine
Copy link
Collaborator

elasticmachine commented Apr 16, 2024

💛 Build succeeded, but was flaky

Failed CI Steps

History

cc @jsoriano

@jsoriano jsoriano enabled auto-merge (squash) April 16, 2024 14:34
@jsoriano jsoriano merged commit 0647d80 into elastic:main Apr 16, 2024
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