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

Add outbound index metrics to the policy controller #12429

Merged
merged 4 commits into from
May 10, 2024

Conversation

adleong
Copy link
Member

@adleong adleong commented Apr 13, 2024

Adds index metrics to the outbound policy index.

# HELP outbound_index_service_index_size The number of entires in service index
# TYPE outbound_index_service_index_size gauge
outbound_index_service_index_size 20
# HELP outbound_index_service_info_index_size The number of entires in the service info index
# TYPE outbound_index_service_info_index_size gauge
outbound_index_service_info_index_size 23
# HELP outbound_index_service_route_index_size The number of entires in the service route index
# TYPE outbound_index_service_route_index_size gauge
outbound_index_service_route_index_size{namespace="kube-system"} 0
outbound_index_service_route_index_size{namespace="cert-manager"} 0
outbound_index_service_route_index_size{namespace="default"} 0
outbound_index_service_route_index_size{namespace="linkerd"} 0
outbound_index_service_route_index_size{namespace="emojivoto"} 0
outbound_index_service_route_index_size{namespace="linkerd-viz"} 0
# HELP outbound_index_service_port_route_index_size The number of entires in the service port route index
# TYPE outbound_index_service_port_route_index_size gauge
outbound_index_service_port_route_index_size{namespace="kube-system"} 0
outbound_index_service_port_route_index_size{namespace="cert-manager"} 0
outbound_index_service_port_route_index_size{namespace="default"} 1
outbound_index_service_port_route_index_size{namespace="linkerd"} 0
outbound_index_service_port_route_index_size{namespace="emojivoto"} 3
outbound_index_service_port_route_index_size{namespace="linkerd-viz"} 0

Signed-off-by: Alex Leong <alex@buoyant.io>
@adleong adleong requested a review from a team as a code owner April 13, 2024 00:08
Signed-off-by: Alex Leong <alex@buoyant.io>
Signed-off-by: Alex Leong <alex@buoyant.io>
@adleong adleong merged commit 405aabb into main May 10, 2024
41 checks passed
@adleong adleong deleted the alex/outbound-index-metrics branch May 10, 2024 16:22
the-wondersmith pushed a commit to the-wondersmith/linkerd2 that referenced this pull request May 15, 2024
Adds index metrics to the outbound policy index.

```
# HELP outbound_index_service_index_size The number of entires in service index
# TYPE outbound_index_service_index_size gauge
outbound_index_service_index_size 20
# HELP outbound_index_service_info_index_size The number of entires in the service info index
# TYPE outbound_index_service_info_index_size gauge
outbound_index_service_info_index_size 23
# HELP outbound_index_service_route_index_size The number of entires in the service route index
# TYPE outbound_index_service_route_index_size gauge
outbound_index_service_route_index_size{namespace="kube-system"} 0
outbound_index_service_route_index_size{namespace="cert-manager"} 0
outbound_index_service_route_index_size{namespace="default"} 0
outbound_index_service_route_index_size{namespace="linkerd"} 0
outbound_index_service_route_index_size{namespace="emojivoto"} 0
outbound_index_service_route_index_size{namespace="linkerd-viz"} 0
# HELP outbound_index_service_port_route_index_size The number of entires in the service port route index
# TYPE outbound_index_service_port_route_index_size gauge
outbound_index_service_port_route_index_size{namespace="kube-system"} 0
outbound_index_service_port_route_index_size{namespace="cert-manager"} 0
outbound_index_service_port_route_index_size{namespace="default"} 1
outbound_index_service_port_route_index_size{namespace="linkerd"} 0
outbound_index_service_port_route_index_size{namespace="emojivoto"} 3
outbound_index_service_port_route_index_size{namespace="linkerd-viz"} 0
```

Signed-off-by: Alex Leong <alex@buoyant.io>
(cherry picked from commit 405aabb)
the-wondersmith added a commit to the-wondersmith/linkerd2 that referenced this pull request May 15, 2024
* origin/policy-feat-grpcroute-status-support:
  chore(ci): merge fixes from origin/main
  build(deps): bump tj-actions/changed-files from 44.3.0 to 44.4.0 (linkerd#12588)
  build(deps): bump github.com/fatih/color from 1.16.0 to 1.17.0 (linkerd#12590)
  chore(ci): Remove conditional integration testing (linkerd#12591)
  build(deps-dev): bump sinon from 17.0.1 to 17.0.2 in /web/app (linkerd#12587)
  build(deps): bump github.com/prometheus/client_golang (linkerd#12586)
  build(deps): bump thiserror from 1.0.59 to 1.0.60 (linkerd#12585)
  Revert "Fix destination staleness issue when adding EndpointSlices (linkerd#12427)" (linkerd#12589)
  Add outbound index metrics to the policy controller (linkerd#12429)
  build(deps): bump tj-actions/changed-files from 44.3.0 to 44.4.0 (linkerd#12588)
  build(deps): bump github.com/fatih/color from 1.16.0 to 1.17.0 (linkerd#12590)
  chore(ci): Remove conditional integration testing (linkerd#12591)
  build(deps-dev): bump sinon from 17.0.1 to 17.0.2 in /web/app (linkerd#12587)
  build(deps): bump github.com/prometheus/client_golang (linkerd#12586)
  build(deps): bump thiserror from 1.0.59 to 1.0.60 (linkerd#12585)
  Revert "Fix destination staleness issue when adding EndpointSlices (linkerd#12427)" (linkerd#12589)
  Set backend_not_found route status when any backends are not found (linkerd#12565)
  Add outbound index metrics to the policy controller (linkerd#12429)

Signed-off-by: Mark S <the@wondersmith.dev>
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.

None yet

2 participants