Skip to content

[Backport 2.x] Traces custom source - Bug Fixes #2307

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

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport a57c10e from #2298.

* fix custom traces filter bug, add default metric selection

Signed-off-by: Adam Tackett <[email protected]>

* helper function for start time, handle date and datenanos

Signed-off-by: Adam Tackett <[email protected]>

* add toast messages for all errors service-traces, add mode to url

Signed-off-by: Adam Tackett <[email protected]>

* fix url re-direction in non custom state

Signed-off-by: Adam Tackett <[email protected]>

* address comments, add testing

Signed-off-by: Adam Tackett <[email protected]>

* update url #, send mode to applications

Signed-off-by: Adam Tackett <[email protected]>

* update helper function

Signed-off-by: Adam Tackett <[email protected]>

* add validation to url mode

Signed-off-by: Adam Tackett <[email protected]>

* fix applications removing traces redirect

Signed-off-by: Adam Tackett <[email protected]>

---------

Signed-off-by: Adam Tackett <[email protected]>
Co-authored-by: Adam Tackett <[email protected]>
(cherry picked from commit a57c10e)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@paulstn paulstn merged commit a1cc0f0 into 2.x Jan 8, 2025
29 of 32 checks passed
@paulstn paulstn deleted the backport/backport-2298-to-2.x branch January 8, 2025 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants