Skip to content

feat(models): Implement collection of plugin-internal statistics for all types - #17749

Merged
skartikey merged 3 commits into
influxdata:masterfrom
Super-Visions:feat/models/selfstat
Nov 6, 2025
Merged

feat(models): Implement collection of plugin-internal statistics for all types#17749
skartikey merged 3 commits into
influxdata:masterfrom
Super-Visions:feat/models/selfstat

Conversation

@Hipska

@Hipska Hipska commented Oct 2, 2025

Copy link
Copy Markdown
Contributor

Summary

While #17277 implemented the selfstat.Collector, it only added support for output plugins. This PR adds support for aggregators, inputs, parsers, processors, serializers and secret-stores.

Checklist

  • No AI generated code was used in this PR

Related issues

based on #17344
based on #17277
required for #17733

@telegraf-tiger telegraf-tiger Bot added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Oct 2, 2025
@srebhan srebhan self-assigned this Oct 6, 2025

@srebhan srebhan 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.

Thanks @Hipska for your contribution! One small request/question and a wishful thinking comment. Please ignore the latter! ;-)

Comment thread config/config.go
Comment thread models/running_input.go Outdated
@Hipska

This comment was marked as outdated.

@Hipska
Hipska requested a review from srebhan October 20, 2025 08:49
@srebhan srebhan added the waiting for response waiting for response from contributor label Oct 23, 2025
@Hipska

This comment was marked as outdated.

@telegraf-tiger telegraf-tiger Bot removed the waiting for response waiting for response from contributor label Nov 3, 2025
@Hipska

This comment was marked as outdated.

1 similar comment
@Hipska

This comment was marked as outdated.

@Hipska

Hipska commented Nov 4, 2025

Copy link
Copy Markdown
Contributor Author

@srebhan it also looks like the @influx-telegraf-tiger-bot command !retry-failed doesn't seem to work.

@srebhan srebhan 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.

Thanks @Hipska!

@srebhan srebhan changed the title feat(selfstat): Implement collection of plugin-internal statistics for all types Nov 5, 2025
@srebhan srebhan added area/agent ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. labels Nov 5, 2025
@srebhan srebhan assigned skartikey and mstrandboge and unassigned srebhan Nov 5, 2025

@skartikey skartikey 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.

@Hipska Thanks for the contribution!

@skartikey
skartikey merged commit 932b053 into influxdata:master Nov 6, 2025
29 checks passed
@github-actions github-actions Bot added this to the v1.37.0 milestone Nov 6, 2025
@Hipska
Hipska deleted the feat/models/selfstat branch November 18, 2025 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/agent feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.

4 participants