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

feat(monitoring): Filter active sessions by session variables #1573

Merged
merged 1 commit into from
May 27, 2024

Conversation

MoritzWeber0
Copy link
Member

@MoritzWeber0 MoritzWeber0 commented May 17, 2024

Filter active sessions by tool, tool version, connection method and session type.

The values are available as annotations on the Kubernetes service, deployment and pod. Prometheus relabels the annotations to labels.

In Grafana, a dropdown menu allows filtering:
image

In addition, the dashboard also shows sessions with failed activity monitoring.

Also, a small bug that tools couldn't be updated without the ID attribute is fixed.

Copy link

github-actions bot commented May 17, 2024

A Storybook preview is available for commit a7014db.
View Storybook
View Chromatic build

@MoritzWeber0 MoritzWeber0 changed the title feat(monitoring): Filter active sessions for session variables feat(monitoring): Filter active sessions by session variables May 17, 2024
Filter active sessions by tool, tool version, connection method
and session type.

The value are available as annotations on the Kubernetes service,
deployment and pod. Prometheus relabels the annotations to labels.

In Grafana, a dropdown menu allows filtering.

In addition, the dashboard also shows sessions with failed activity
monitoring.
Copy link

sonarcloud bot commented May 21, 2024

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link

codecov bot commented May 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.05%. Comparing base (ff32d5f) to head (da1c8c8).
Report is 161 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1573   +/-   ##
=======================================
  Coverage   79.04%   79.05%           
=======================================
  Files         172      172           
  Lines        5766     5767    +1     
  Branches      665      665           
=======================================
+ Hits         4558     4559    +1     
  Misses       1054     1054           
  Partials      154      154           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MoritzWeber0 MoritzWeber0 merged commit 52f059a into main May 27, 2024
30 checks passed
@MoritzWeber0 MoritzWeber0 deleted the session-annotations branch May 27, 2024 09:41
MoritzWeber0 added a commit that referenced this pull request May 28, 2024
The automatic termination of idle sessions stopped working with #1573.
Reason is that the app label was removed from Prometheus.

This commit migrates the old app references to the new session_id label.
MoritzWeber0 added a commit that referenced this pull request May 28, 2024
Unfortunately, #1573 was merged in an incomplete state due to a missing push.
This commit adds the missing changes so that the dashboard works as described in the PR.
MoritzWeber0 added a commit that referenced this pull request May 28, 2024
Unfortunately, #1573 was merged in an incomplete state due to a missing push.
This commit adds the missing changes so that the dashboard works as described in the PR.
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.

1 participant