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

profiles: Avoid creating a default route stack #1223

Merged
merged 6 commits into from
Aug 26, 2021
Merged

Commits on Aug 25, 2021

  1. Configuration menu
    Copy the full SHA
    95587f9 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    90fdd82 View commit details
    Browse the repository at this point in the history
  3. profiles: Avoid creating a default route stack

    When the proxy resolves a service profile with routes that do not match
    any requests, we handle these requests on a 'default' route. This route
    does not have any labels and is generally superflous, inflating our
    metrics export unnecessarily.
    
    This change removes this behavior by instead skipping the route stack
    entirely when no routes are matched. In order to do this, we add some
    extra boxes so that we can handle the request & response types
    transparently.
    olix0r committed Aug 25, 2021
    Configuration menu
    Copy the full SHA
    3e5c983 View commit details
    Browse the repository at this point in the history

Commits on Aug 26, 2021

  1. Configuration menu
    Copy the full SHA
    120a33b View commit details
    Browse the repository at this point in the history
  2. fix weird indentation

    olix0r committed Aug 26, 2021
    Configuration menu
    Copy the full SHA
    038c4a2 View commit details
    Browse the repository at this point in the history
  3. avoid boxed future

    olix0r committed Aug 26, 2021
    Configuration menu
    Copy the full SHA
    7ecd9c7 View commit details
    Browse the repository at this point in the history