Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(events)!: add config.merge option #2643

Merged
merged 1 commit into from
Nov 25, 2022
Merged

Conversation

swiatekm
Copy link

@swiatekm swiatekm commented Nov 25, 2022

Description

Logs and metrics metadata config has merge and override properties to directly modify otel config. Make this so for events as well. This is a breaking change, as events already had override, but the meaning of it was identical to merge. We can handle this via an automatic migration.

Migration PR: SumoLogic/sumologic-kubernetes-tools#389


Checklist
  • Changelog updated

@swiatekm swiatekm marked this pull request as ready for review November 25, 2022 07:51
@swiatekm swiatekm requested a review from a team as a code owner November 25, 2022 07:51
@github-actions github-actions bot added the documentation documentation label Nov 25, 2022
@sumo-drosiek
Copy link
Contributor

Please remove tmpcharts

@swiatekm
Copy link
Author

Please remove tmpcharts

Whoops, done. I also added the directory to .gitignore.

.gitignore Outdated Show resolved Hide resolved
Copy link
Contributor

@sumo-drosiek sumo-drosiek left a comment

Choose a reason for hiding this comment

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

one nit regarding .gitignore

@swiatekm swiatekm enabled auto-merge (rebase) November 25, 2022 11:09
@swiatekm swiatekm merged commit 66a6428 into main Nov 25, 2022
@swiatekm swiatekm deleted the feat/events/add-merge branch November 25, 2022 11:18
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.

2 participants