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.

Sign up for GitHub

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

Jump to bottom

[Metrics] Track latency under Search endpoint #3850

Merged
merged 2 commits into from
May 13, 2024
Merged

[Metrics] Track latency under Search endpoint #3850

merged 2 commits into from
May 13, 2024

Conversation

fhibf
Copy link
Contributor

@fhibf fhibf commented May 10, 2024 •
edited
Loading

Track latency under Search endpoint.

Related issues

AB#119528

Testing

Under FHIR Controller there are multiple endpoints handling search.
In my original PR I missed one endpoint.
In this PR I'm fixing that.

  • CI is green before merge Build Status

@fhibf fhibf added Enhancement Enhancement on existing functionality. Azure API for FHIR Label denotes that the issue or PR is relevant to the Azure API for FHIR Azure Healthcare APIs Label denotes that the issue or PR is relevant to the FHIR service in the Azure Healthcare APIs labels May 10, 2024
@fhibf fhibf added this to the S140 milestone May 10, 2024
@fhibf fhibf requested a review from a team as a code owner May 10, 2024 17:16
@fhibf fhibf changed the title [Metrics] Track latency under "Search" endpoint [Metrics] Track latency under Search endpoint May 10, 2024
@fhibf fhibf enabled auto-merge (squash) May 13, 2024 17:38
mikaelweave
mikaelweave approved these changes May 13, 2024
View reviewed changes
Copy link
Contributor

@mikaelweave mikaelweave left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦾

@fhibf fhibf merged commit a26b9b3 into main May 13, 2024
47 of 59 checks passed
@fhibf fhibf deleted the user/fhibf/extendSearchLatencyToEndpoint branch May 13, 2024 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mikaelweave mikaelweave mikaelweave approved these changes

Assignees
No one assigned
Labels
Azure API for FHIR Label denotes that the issue or PR is relevant to the Azure API for FHIR Azure Healthcare APIs Label denotes that the issue or PR is relevant to the FHIR service in the Azure Healthcare APIs Enhancement Enhancement on existing functionality.
Projects
None yet
Milestone
S140
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@fhibf @mikaelweave