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

Updated extract to handle http receiver metadata #2

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Commits on Jul 12, 2023

  1. [pkg/ottl] Improve OTTL parser error reporting (open-telemetry#23840)

    **Description:**
    
    I was playing with an erroneous config like the following and noticed
    that the error reporting was confusing:
    
    ```yaml
    processors:
      filter:
        metrics:
          metric:
            - true
            - false
    ```
    
    This PR updates the error messages to print the statements next to the
    error so it's clear which statement caused the error. To speed up
    feedback cycles when changing multiple statements, I've also configured
    the parser to attempt to parse each statement so users can see all
    errors at once.
    
    Errors now look like this:
    ```
    Error: invalid configuration: processors::filter: unable to parse OTTL statement "drop() where 1": statement has invalid syntax: 1:15: unexpected token "<EOF>" (expected <opcomparison> Value); unable to parse OTTL statement "drop() where 0": statement has invalid syntax: 1:15: unexpected token "<EOF>" (expected <opcomparison> Value)
    ```
    
    **Testing:** Unit test
    
    Co-authored-by: Evan Bradley <[email protected]>
    evan-bradley and evan-bradley authored Jul 12, 2023
    Configuration menu
    Copy the full SHA
    a0883a0 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    ac5eb05 View commit details
    Browse the repository at this point in the history
  3. [chore] Update go versions used in GitHub workflows (open-telemetry#2…

    …4213)
    
    **Description:** Update go versions used in GitHub workflows. Also
    changes to a stricter pattern match in some cases to provide more
    awareness on which version will be selected.
    
    ---------
    
    Co-authored-by: Daniel Jaglowski <[email protected]>
    bryan-aguilar and djaglowski authored Jul 12, 2023
    Configuration menu
    Copy the full SHA
    2dde029 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    9d28ac1 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    faa903b View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    1f6d6ea View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    9d22d9b View commit details
    Browse the repository at this point in the history