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

field name compatibility for SCK #258

Merged
merged 6 commits into from
Nov 3, 2021

Conversation

rockb1017
Copy link
Contributor

@rockb1017 rockb1017 commented Oct 29, 2021

backward compatible field names for container metadata.

@rockb1017 rockb1017 requested review from a team as code owners October 29, 2021 23:47
@rockb1017 rockb1017 force-pushed the field_name_compat branch 3 times, most recently from 0fd335d to e17c5ff Compare October 29, 2021 23:55
@rockb1017 rockb1017 force-pushed the field_name_compat branch 4 times, most recently from 0d4ea2c to 0606bbf Compare November 2, 2021 01:59
CHANGELOG.md Outdated Show resolved Hide resolved
helm-charts/splunk-otel-collector/values.yaml Outdated Show resolved Hide resolved
helm-charts/splunk-otel-collector/values.yaml Outdated Show resolved Hide resolved
helm-charts/splunk-otel-collector/values.yaml Outdated Show resolved Hide resolved
@rockb1017 rockb1017 force-pushed the field_name_compat branch 2 times, most recently from fdd50a2 to 5d9d816 Compare November 2, 2021 06:28
@rockb1017 rockb1017 changed the title field name compatibility for SCK and index routing support field name compatibility for SCK Nov 2, 2021
Copy link
Contributor

@dmitryax dmitryax left a comment

Choose a reason for hiding this comment

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

One last correction. Otherwise looks good

CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Dmitrii Anoshin <[email protected]>
@rockb1017
Copy link
Contributor Author

Thank you so much for detailed review!

@dmitryax dmitryax merged commit a02dda1 into signalfx:main Nov 3, 2021
@rockb1017 rockb1017 deleted the field_name_compat branch November 9, 2021 23:54
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.

2 participants