Skip to content
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

Add index correction metrics #2215

Merged
merged 13 commits into from
Oct 30, 2023
Merged

Add index correction metrics #2215

merged 13 commits into from
Oct 30, 2023

Conversation

ykadowak
Copy link
Contributor

@ykadowak ykadowak commented Oct 20, 2023

Description:

This PR adds metrics for index-correction job. Also added grafana dashboard for index-correction.
Added metrics:

checkedIndexCount     = "index_job_correction_checked_index_count"
checkedIndexCountDesc = "The number of checked indexes while index correction job"

correctedOldIndexCount     = "index_job_correction_corrected_old_index_count"
correctedOldIndexCountDesc = "The number of corrected old indexes while index correction job"

correctedReplicationCount     = "index_job_correction_corrected_replication_count"
correctedReplicationCountDesc = "The number of operation happend to correct replication number while index correction job"

Related Issue:

Versions:

  • Go Version: 1.21.3
  • Docker Version: 20.10.8
  • Kubernetes Version: v1.28.2
  • NGT Version: 2.1.3

Checklist:

Special notes for your reviewer:

@vdaas-ci
Copy link
Collaborator

[CHATOPS:HELP] ChatOps commands.

  • 🙆‍♀️ /approve - approve
  • 🍱 /format - format codes and add licenses
  • /gen-test - generate test codes
  • 🏷️ /label - add labels
  • 🔚 2️⃣ 🔚 /label actions/e2e-deploy - run E2E deploy & integration test

correctedOldIndexCountDesc = "The number of corrected old indexes while index correction job"

correctedReplicationCount = "index_job_correction_corrected_replication_count"
correctedReplicationCountDesc = "The number of operation happend to correct replication number while index correction job"
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [golangci] reported by reviewdog 🐶
happend is a misspelling of happened (misspell)

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 20, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2e3d072
Status: ✅  Deploy successful!
Preview URL: https://f19d8a55.vald.pages.dev
Branch Preview URL: https://feature-corrector-metrics.vald.pages.dev

View logs

@codecov
Copy link

codecov bot commented Oct 20, 2023

Codecov Report

Attention: 11 lines in your changes are missing coverage. Please review.

Files Coverage Δ
pkg/index/job/correction/usecase/corrector.go 0.00% <0.00%> (ø)
pkg/index/job/correction/service/corrector.go 18.46% <30.00%> (+0.28%) ⬆️

... and 2 files with indirect coverage changes

📢 Thoughts on this report? Let us know!.

@ykadowak ykadowak changed the title [WIP] Add index correction metrics Add index correction metrics Oct 25, 2023
@ykadowak ykadowak requested review from a team, kmrmt and datelier and removed request for a team October 25, 2023 08:10
@ykadowak ykadowak merged commit ca118d3 into main Oct 30, 2023
83 of 84 checks passed
@ykadowak ykadowak deleted the feature/corrector/metrics branch October 30, 2023 04:32
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
@vankichi vankichi mentioned this pull request Dec 4, 2023
kmrmt pushed a commit that referenced this pull request Dec 12, 2023
* Add observability pkg for index correction

* Add template grafana dashboard page of index correction

* Update label

* Add checked index count

* Fix o11y service name

* Add correction metrics fields in the grafana

* Init correction metrics

* Organized grafana dashboard for index correction

* Fix misspell

* Remove unused method receiver
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants