Skip to content

Implement region aware event/metric ingest URIs#2749

Merged
jasonjkeller merged 2 commits intomainfrom
add-region-aware-ingest-uris
Feb 25, 2026
Merged

Implement region aware event/metric ingest URIs#2749
jasonjkeller merged 2 commits intomainfrom
add-region-aware-ingest-uris

Conversation

@jasonjkeller
Copy link
Copy Markdown
Contributor

@jasonjkeller jasonjkeller commented Feb 10, 2026

Resolves: #2748

We weren't parsing the region from the license_key and applying it to the metric and event ingest URIs. This PR fixes that.

Do not merge until after the 9.1.0 release.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.37%. Comparing base (f88d2dd) to head (7ce0b4e).
⚠️ Report is 33 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2749      +/-   ##
============================================
+ Coverage     70.35%   70.37%   +0.02%     
- Complexity    10365    10379      +14     
============================================
  Files           869      869              
  Lines         41985    42006      +21     
  Branches       6381     6385       +4     
============================================
+ Hits          29538    29562      +24     
+ Misses         9568     9563       -5     
- Partials       2879     2881       +2     

☔ 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.

@jeffalder
Copy link
Copy Markdown
Contributor

Can you revise one of the tests to have a shorter prefix? I've been using czx as in the cross-agent tests.

@jasonjkeller jasonjkeller merged commit cb7a43e into main Feb 25, 2026
123 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Review to Code Complete/Done in Java Engineering Board Feb 25, 2026
@jasonjkeller jasonjkeller deleted the add-region-aware-ingest-uris branch February 25, 2026 17:16
@jeffalder
Copy link
Copy Markdown
Contributor

Thanks @jasonjkeller !!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Generalize metric and event ingest URIs.

4 participants