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

Adjust APS MONIT metrics streaming #33

Open
vkuznet opened this issue Sep 24, 2024 · 2 comments
Open

Adjust APS MONIT metrics streaming #33

vkuznet opened this issue Sep 24, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@vkuznet
Copy link
Collaborator

vkuznet commented Sep 24, 2024

Based on Nikodemas report:

We have created a dashboard with the new APS dashboard (original dashboard is here). A few things about the fields in the new dashboard/logs with the examples:

  • missing data.cmsweb_env - k8s-preprod value in the original dashboard, no alternatives in the new dashboard
  • missing data.cmsweb_cluster - cmsweb-testbed value in the original dashboard, but data.x_forwarded_host could work as it currently has the value of cmsweb-test10.cern.ch in the new dashboard
  • missing data.client_version - v002 value in the original dashboard, but we could use data.user_agent in the new dashboard that has a value of WMCore.Services.Requests/v002
  • missing data.proxy_server - x509-proxy-server or aps-proxy-server, no alternatives in the new dashboard

We need to implement those to be compatible with APS+filebeat+logstash approach used in APS timber dashboard.

@vkuznet vkuznet added the enhancement New feature or request label Sep 24, 2024
@vkuznet
Copy link
Collaborator Author

vkuznet commented Sep 24, 2024

Added in this commit: 791702a

@vkuznet
Copy link
Collaborator Author

vkuznet commented Sep 24, 2024

@arooshap , @nikodemas : I added missing fields for sending APS logs directly to MONIT in this ticket. The cmsweb_env, cmsweb_cluster, and proxy_server will be set if we setup CMSWEB_ENV, CMSWEB_CLUSTER, and APS_SERVER in aps k8s yaml files. And, client_version will be set via user-agent of HTTP request.

Once this issue is committed, APS go.mod and go.sum will need an appropriate update and new APS image should be built (I'll do it later). After that you may test new feature in cmsweb clusters and decide should it be used. For that a set of stress tests will be required to determine the latency of APS+MONIT vs APS+filebeat+logstash+MONIT routes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant