Skip to content

Support Monitor Client and Fetch Entity Metrics #569

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

Open
wants to merge 8 commits into
base: dev
Choose a base branch
from

Conversation

yec-akamai
Copy link
Contributor

@yec-akamai yec-akamai commented Jul 10, 2025

📝 Description

This PR introduced a new client MonitorClient, to make calls to a different host monitor-api. Also refactor the LinodeClient and have them share a BaseClient. The endpoint for this client is fetching metrics for a list of entities. Also, updated test suite for the new client and test the fetching metrics endpoint.

✔️ How to Test

Unit test:

make test-unit

Integration test:

make test-int TEST_SUITE=monitor_api

@yec-akamai yec-akamai changed the title init Support Monitor Client and Fetch Entity Metrics Jul 10, 2025
@yec-akamai yec-akamai marked this pull request as ready for review July 15, 2025 13:40
@yec-akamai yec-akamai requested a review from a team as a code owner July 15, 2025 13:40
@yec-akamai yec-akamai requested review from jriddle-linode and zliang-akamai and removed request for a team July 15, 2025 13:40
@yec-akamai yec-akamai added new-feature for new features in the changelog. do-not-merge PRs that should not be merged until the commented issue is resolved labels Jul 15, 2025
@yec-akamai yec-akamai removed the do-not-merge PRs that should not be merged until the commented issue is resolved label Jul 23, 2025
@yec-akamai yec-akamai requested a review from zliang-akamai July 23, 2025 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-feature for new features in the changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants