Skip to content

feat: add profile by service name endpoint #403

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
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

feat: add profile by service name endpoint #403

wants to merge 3 commits into from

Conversation

shaheerem
Copy link

  • Add GET /v1/cover/profile/?name= endpoint
  • Allows retrieving merged coverage profiles for all instances of a specific service
  • Includes comprehensive error handling for missing services and failed connections
  • Add unit tests covering various error scenarios
  • Returns appropriate HTTP status codes for different error conditions

shaheerem added 2 commits June 27, 2025 16:07
@shaheerem
feat: add profile by service name endpoint
94e9472 
- Add GET /v1/cover/profile/?name=<service-name> endpoint
- Allows retrieving merged coverage profiles for all instances of a specific service
- Includes comprehensive error handling for missing services and failed connections
- Add unit tests covering various error scenarios
- Returns appropriate HTTP status codes for different error conditions
@shaheerem
Update: Update mod with latest go version
972934c
@qiniu-prow qiniu-prow
Copy link

qiniu-prow bot commented Jun 27, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: shaheerem
Once this PR has been reviewed and has the lgtm label, please assign e06084 for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@qiniu-prow qiniu-prow bot added the size/L label Jun 27, 2025
@shaheerem shaheerem mentioned this pull request Jun 27, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
size/L
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@shaheerem