Skip to content

fix cost leak issue - #104

Merged
InftyAI-Agent merged 2 commits into
InftyAI:mainfrom
kerthcet:fix/cost-tracking-error
Sep 10, 2026
Merged

fix cost leak issue#104
InftyAI-Agent merged 2 commits into
InftyAI:mainfrom
kerthcet:fix/cost-tracking-error

Conversation

@kerthcet

@kerthcet kerthcet commented Sep 10, 2026

Copy link
Copy Markdown
Member

What this PR does / why we need it

Which issue(s) this PR fixes

Fixes #

Special notes for your reviewer

Does this PR introduce a user-facing change?

Copilot AI lite review requested due to automatic review settings September 10, 2026 16:02
Copilot AI reviewed Sep 10, 2026

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@InftyAI-Agent InftyAI-Agent added needs-triage Indicates an issue or PR lacks a label and requires one. needs-priority Indicates a PR lacks a label and requires one. do-not-merge/needs-kind Indicates a PR lacks a label and requires one. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Sep 10, 2026
Copilot AI review requested due to automatic review settings September 10, 2026 17:09
Copilot AI reviewed Sep 10, 2026

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@kerthcet
fix cost leak issue
6864e0a 
Signed-off-by: kerthcet <kerthcet@gmail.com>
@kerthcet
kerthcet force-pushed the fix/cost-tracking-error branch from c5cc7f0 to 6864e0a Compare September 10, 2026 17:10
@kerthcet
fix comments
e39b89e 
Signed-off-by: kerthcet <kerthcet@gmail.com>
Copilot AI review requested due to automatic review settings September 10, 2026 17:19
Copilot AI reviewed Sep 10, 2026

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@kerthcet

kerthcet commented Sep 10, 2026

Copy link
Copy Markdown
Member Author

/lgtm
/kind bug

@InftyAI-Agent InftyAI-Agent added lgtm Looks good to me, indicates that a PR is ready to be merged. bug Categorizes issue or PR as related to a bug. and removed do-not-merge/needs-kind Indicates a PR lacks a label and requires one. labels Sep 10, 2026
@kerthcet

kerthcet commented Sep 10, 2026

Copy link
Copy Markdown
Member Author

/hold

InftyAI-Agent
InftyAI-Agent approved these changes Sep 10, 2026
View reviewed changes

@InftyAI-Agent InftyAI-Agent left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved: PR has both lgtm and approved labels

InftyAI-Agent
InftyAI-Agent approved these changes Sep 10, 2026
View reviewed changes

@InftyAI-Agent InftyAI-Agent left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved: PR has both lgtm and approved labels

@InftyAI-Agent
InftyAI-Agent merged commit ae61684 into InftyAI:main Sep 10, 2026
25 of 26 checks passed
@InftyAI-Agent InftyAI-Agent added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 10, 2026
@kerthcet
kerthcet deleted the fix/cost-tracking-error branch September 10, 2026 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@InftyAI-Agent InftyAI-Agent InftyAI-Agent approved these changes

Assignees

@kerthcet kerthcet

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. bug Categorizes issue or PR as related to a bug. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. lgtm Looks good to me, indicates that a PR is ready to be merged. needs-priority Indicates a PR lacks a label and requires one. needs-triage Indicates an issue or PR lacks a label and requires one.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kerthcet @InftyAI-Agent