Skip to content

feat: [DPS-44447] Add endpoint to expose streams limit per account - #1019

Open
odyszews wants to merge 1 commit into
linode:mainfrom
odyszews:feat/DPS-44447-Add-endpoint-to-expose-streams-limit-per-account
Open

feat: [DPS-44447] Add endpoint to expose streams limit per account#1019
odyszews wants to merge 1 commit into
linode:mainfrom
odyszews:feat/DPS-44447-Add-endpoint-to-expose-streams-limit-per-account

Conversation

@odyszews

Copy link
Copy Markdown

📝 Description

Adds support for the GET /monitor/streams/quotas endpoint, which exposes stream quota definitions for the authenticated account.

✔️ How to Test

make test-unit

@odyszews
odyszews requested review from a team as code owners August 10, 2026 11:31
@odyszews
odyszews requested review from mawilk90 and zliang-akamai and a lite review from Copilot and removed request for a team August 10, 2026 11:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds client + unit-test support for the new GET /monitor/streams/quotas endpoint so users can retrieve stream quota definitions for the authenticated account.

Changes:

  • Introduces a StreamQuota model and a Client.ListStreamQuotas list method for /monitor/streams/quotas.
  • Adds unit test coverage for listing stream quotas using a new JSON fixture.
  • Adds a new unit-test fixture representing a paginated quotas response.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
monitor_log_streams.go Adds StreamQuota type and a client method to list quotas from monitor/streams/quotas.
test/unit/monitor_logs_test.go Adds a unit test validating ListStreamQuotas parsing and behavior.
test/unit/fixtures/monitor_log_streams_quotas_list.json Adds fixture data for the quotas list endpoint response.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread monitor_log_streams.go Outdated
@odyszews
odyszews force-pushed the feat/DPS-44447-Add-endpoint-to-expose-streams-limit-per-account branch from 3bc1f96 to f0a2062 Compare August 10, 2026 11:43
@zliang-akamai
zliang-akamai requested a balanced review from Copilot August 17, 2026 00:05

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

@zliang-akamai zliang-akamai added the community-contribution contributions from the community. label Aug 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution contributions from the community.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants