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

[Log Parsing Pipelines] Testing TraceParsing preview from JSON doesn't work #3795

Closed
raj-k-singh opened this issue Oct 25, 2023 · 1 comment · Fixed by #3822
Closed

[Log Parsing Pipelines] Testing TraceParsing preview from JSON doesn't work #3795

raj-k-singh opened this issue Oct 25, 2023 · 1 comment · Fixed by #3822
Assignees

Comments

@raj-k-singh
Copy link
Collaborator

raj-k-singh commented Oct 25, 2023

  • Payload while saving a trace_parser is wrong.
    • Update traceParser to trace_parser here

    • Current/Wrong Payload

      {
          "id": "trace",
          "orderId": 2,
          "type": "trace_parser",
          "enabled": true,
          "name": "trace",
          "traceId": "attributes.temp.trace_id",
          "spanId": "attributes.temp.span_id",
          "traceFlags": "attributes.temp.trace_flags",
          "output": "removetemp"
      }
    • Correct payload

      {
            "orderId": 3,
            "enabled": true,
            "type": "trace_parser", // all three cannot be empty
            "name": "Parse trace details",
            "id": "parsetracedetails",
            "trace_id": { // Remove the keys if the user left them empty
                "parse_from": "attributes.xtrace_id"
            },
            "span_id": { // Remove the keys if the user left them empty
                "parse_from": "attributes.xspan_id"
            },
            "trace_flags": { // Remove the keys if the user left them empty
                "parse_from": "attributes.xtrace_flag"
            },
            "output": "removebytes" // required
        },
@raj-k-singh raj-k-singh self-assigned this Oct 25, 2023
@request-info
Copy link

request-info bot commented Oct 25, 2023

We would appreciate it if you could provide us with more info about this issue/pr!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant