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(operator): Provide default OTLP attribute configuration #14410

Draft
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

xperimental
Copy link
Collaborator

What this PR does / why we need it:

The main purpose of this PR is to provide a default set of OTLP attributes that will be persisted when using LokiStack in openshift-logging tenancy mode.

This PR also changes the API for specifying an OTLP attribute configuration in LokiStack to make it easier to use.

Which issue(s) this PR fixes:

LOG-6147

Special notes for your reviewer:

Checklist

  • Reviewed the CONTRIBUTING.md guide (required)
  • Documentation added
  • Tests updated
  • Title matches the required conventional commits format, see here

… openshift-logging tenancy

- Extend API for default OTLP label set
- Disable autodetection of service.name
- Update non-otlp config tests
- Simplify OTLP attribute configuration
- Remove validator code
- Content-assist for runtime config
- Add openshift-logging defaults
- Only add ignore_defaults to config if needed
- Use Loki default OTLP attributes when none are specified
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