Skip to content

hive,metrics: Add metrics for pkg/hive/job#26077

Merged
dylandreimerink merged 1 commit intocilium:mainfrom
dylandreimerink:feature/hive-job-metrics
Jul 3, 2023
Merged

hive,metrics: Add metrics for pkg/hive/job#26077
dylandreimerink merged 1 commit intocilium:mainfrom
dylandreimerink:feature/hive-job-metrics

Conversation

@dylandreimerink
Copy link
Copy Markdown
Member

@dylandreimerink dylandreimerink commented Jun 9, 2023

This PR adds a metric to that tracks error counts for job runs, a metric to track timer run performance and observer run performance.

For one shot jobs the metric collection is disabled by default, only jobs with the WithMetrics option will be measured. This way we can collect metrics for short running jobs and skip collection for high cardinality jobs or jobs that run forever.

Added metrics for jobs

@dylandreimerink dylandreimerink added kind/enhancement This would improve or streamline existing functionality. area/metrics Impacts statistics / metrics gathering, eg via Prometheus. release-note/misc This PR makes changes that have no direct user impact. labels Jun 9, 2023
@dylandreimerink dylandreimerink force-pushed the feature/hive-job-metrics branch 2 times, most recently from 4b01365 to 32a9fba Compare June 9, 2023 12:59
@dylandreimerink dylandreimerink force-pushed the feature/hive-job-metrics branch 4 times, most recently from ae67055 to d02ba53 Compare June 14, 2023 09:03
@dylandreimerink dylandreimerink marked this pull request as ready for review June 14, 2023 09:20
@dylandreimerink dylandreimerink requested review from a team as code owners June 14, 2023 09:20
Copy link
Copy Markdown
Contributor

@zacharysarah zacharysarah left a comment

Choose a reason for hiding this comment

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

LGTM

@dylandreimerink dylandreimerink force-pushed the feature/hive-job-metrics branch 2 times, most recently from fc3e9fd to 6fc509e Compare June 16, 2023 11:24
Copy link
Copy Markdown
Member

@pippolo84 pippolo84 left a comment

Choose a reason for hiding this comment

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

Awesome! 🚀

@dylandreimerink
Copy link
Copy Markdown
Member Author

/test

@dylandreimerink dylandreimerink force-pushed the feature/hive-job-metrics branch from 6fc509e to 0dcebb5 Compare June 27, 2023 10:08
This commit adds a metric to that tracks error counts for job runs, a
metric to track timer run performance and observer run performance.

For one shot jobs the metric collection is disabled by default, only
jobs with the `WithMetrics` option will be measured. This way we can
collect metrics for short running jobs and skip collection for
high cardinality jobs or jobs that run forever.

Signed-off-by: Dylan Reimerink <dylan.reimerink@isovalent.com>
@dylandreimerink dylandreimerink force-pushed the feature/hive-job-metrics branch from 0dcebb5 to e056d7a Compare July 3, 2023 07:53
@dylandreimerink
Copy link
Copy Markdown
Member Author

/test

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jul 3, 2023
@dylandreimerink dylandreimerink merged commit d102dbc into cilium:main Jul 3, 2023
mhofstetter added a commit to mhofstetter/cilium that referenced this pull request Dec 15, 2025
This commit updates the documentation about hive job metrics.

* Initially introduced and documented with cilium#26077
* Deleted when cilium/hive has been extracted with cilium#32020
* Re-introduced via integration with cilium#42762 (not aware that they existed in the past)

Therefore this commit updates the docs to reflect the current implementation.

Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
github-merge-queue bot pushed a commit that referenced this pull request Dec 16, 2025
This commit updates the documentation about hive job metrics.

* Initially introduced and documented with #26077
* Deleted when cilium/hive has been extracted with #32020
* Re-introduced via integration with #42762 (not aware that they existed in the past)

Therefore this commit updates the docs to reflect the current implementation.

Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/metrics Impacts statistics / metrics gathering, eg via Prometheus. kind/enhancement This would improve or streamline existing functionality. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants