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

[GEN-1931]: add Highlight destination support and documentation #2088

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

Conversation

BenElferink
Copy link
Contributor

@BenElferink BenElferink commented Dec 27, 2024

This pull request introduces support for the Highlight observability backend. The changes include updates to configuration files, documentation, and the addition of new code to handle the Highlight integration.

Key changes:

Configuration and Code Updates:

  • Added Highlight struct and methods in common/config/highlight.go to configure Highlight as a destination.
  • Included Highlight in the list of available configers in common/config/root.go.
  • Defined HighlightDestinationType in common/dests.go.

Documentation Updates:

  • Added Highlight as a supported backend in README.md and docs/backends-overview.mdx. [1] [2]
  • Created a new documentation page for Highlight in docs/backends/highlight.mdx.
  • Updated docs/mint.json and docs/quickstart/next-steps.mdx to include Highlight references. [1] [2]

YAML Configuration:

  • Added a new YAML configuration file for Highlight in destinations/data/highlight.yaml.

Copy link

@BenElferink BenElferink marked this pull request as draft December 27, 2024 15:06
@BenElferink
Copy link
Contributor Author

Getting “frame too large” errors with exporter:

{
  "kind": "exporter",
  "data_type": "traces",
  "name": "otlp/highlight-odigos.io.dest.highlight-fprsm",
  "error": "rpc error: code = Unavailable desc = connection error: desc = \"error reading server preface: http2: frame too large\"",
  "interval": "24.376872645s"
}

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.

1 participant