Skip to content

Update the kibana link code to use the new, cleaner Kibana URL#717

Open
BenGalewsky wants to merge 1 commit intomasterfrom
1111_log_links
Open

Update the kibana link code to use the new, cleaner Kibana URL#717
BenGalewsky wants to merge 1 commit intomasterfrom
1111_log_links

Conversation

@BenGalewsky
Copy link
Contributor

Companion to ServiceX PR 1295 and part of fix for Issue 1111

Problem

The default Kibana URL we've been using in ServiceX deployments is more complicated than it needs to be and makes manipulating the filters a heroic task.

Approach

We devised a cleaner URL which will now be vended by ServiceX and included in the status messages. This code matches the code in the backend for producing a link that shows just messages associated with a particular transaction on a particular ServiceX deployment.

@codecov
Copy link

codecov bot commented Feb 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.40%. Comparing base (e761557) to head (4a5d3b8).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #717      +/-   ##
==========================================
+ Coverage   98.35%   98.40%   +0.04%     
==========================================
  Files          30       30              
  Lines        2191     2188       -3     
==========================================
- Hits         2155     2153       -2     
+ Misses         36       35       -1     
Flag Coverage Δ
unittests 98.40% <100.00%> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@BenGalewsky BenGalewsky requested a review from ponyisi February 23, 2026 14:39
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