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

[patch] Release v1.7.13 #2561

Closed
wants to merge 76 commits into from
Closed

[patch] Release v1.7.13 #2561

wants to merge 76 commits into from

Conversation

kpango
Copy link
Collaborator

@kpango kpango commented Aug 2, 2024

[patch] Release v1.7.13

Changelog

[Bugfix]

  • General Fixes

    1. fix: add checkout option #2545 (#2546)
    2. fix: make format #2534 (#2540)
    3. fix conflict bug #2537
    4. Bugfix that caused an error when argument has 3 or more nil arguments #2517 (#2520)
    5. Bugfix recreate benchmark job when operator reboot #2463 (#2464)
    6. Fix agent-faiss build failed #2418 (#2419)
    7. Fix the logic to determine docker image #2410 (#2420)
  • Backport and Release-Related

    1. Fix workflow trigger for backport pr creation #2471 (#2472)
    2. Fix output settings to determine-docker-image-tag action and release branch build tag name #2423 (#2425)
  • E2E and Index

    1. Fix e2e for read replica and add e2e for index operator #2455 (#2459)
    2. Fix index job logic to pass DNS A record #2438 (#2448)
  • Documentation and Other

    1. fix: typo of execution rule #2426 (#2427)
    2. 📝 Fix typo of file name #2413 (#2415)

[Enhancement]

  • General Improvements

    1. Update dependencies, C++ standard, and improve Dockerfiles for better build systems and localization #2549 (#2557)
    2. Implement ngt Statistics API #2539 (#2547)
    3. refactor index manager service add index service API to expose index informations #2525 (#2532)
  • API and Logic Changes

    1. Change default image tag from latest to nightly #2516 (#2518)
    2. update large top-K ratio handling logic #2509 (#2511)
    3. add inner product distance type for ngt #2454 (#2458)
    4. Implement index operator logic for read replica rotation #2444 (#2456)
  • Performance and Optimization

    1. update deps & add validation for Flush API when agent is Read Only #2433 (#2436)
    2. Add index-operator template implementation #2375 (#2424)
  • Testing and Metrics

    1. Implement client metrics interceptor for continuous benchmark job #2477 (#2480)
    2. Add tests for index information export #2412 (#2414)

[CI]

  1. [create-pull-request] automated change #2552 (#2556)
  2. Add workflow to check git conflict for backport PR #2548 (#2550)
  3. [CI] Add workflow to synchronize ubuntu base image #2526 (#2527)
  4. Automatically add backport main label for release-pr #2473 (#2475)

[Backport]

  1. Backport PR Fix info path location #2542, Deleted unnecessary event path for docker build #2538 to release/v1.7 #2543
  2. Backport docs updates to release/v1.7 #2521
  3. Backport Flush API #2434

[Documentation]

  1. capitalize faq #2512 (#2522)
  2. add faiss in values.yaml & valdrelease.yaml #2514 (#2519)
  3. add read replica and rotator docs #2497 (#2499)
  4. Update continuous benchmark docs #2485 (#2486)
  5. docs: add hrichiksite as a contributor for doc #2441 (#2442)

[Other]

  1. Add base of benchmark operator dashboard #2430 (#2453)
  2. Add client metrics panels for continuous benchmark job #2481 (#2483)
  3. Add unit tests for index operator #2460 (#2461)
  4. add reviewer guideline #2507 (#2508)
  5. Sync release/v1.7 to main #2495
  6. Add snapshot timestamp annotations to read replica agent #2428 (#2443)
  7. Update build rule for nightly image #2421 (#2422)

Versions

  • Vald Version: v1.7.12
  • Go Version: v1.22.5
  • Rust Version: v1.80.0
  • Docker Version: v27.1.1
  • Kubernetes Version: v1.30.3
  • Helm Version: v3.15.3
  • NGT Version: v2.2.4
  • Faiss Version: v1.8.0

hlts2 and others added 30 commits February 15, 2024 15:43
* 🔖 🤖 Release v1.7.11

Signed-off-by: Vdaas CI <[email protected]>

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 578746e according to the output
from Gofumpt and Prettier.

Details: #2360

---------

Signed-off-by: Vdaas CI <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
* Create SECURITY.md



* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in e8609b0 according to the output
from Gofumpt and Prettier.

Details: #2367

---------

Signed-off-by: Yusuke Kato <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Signed-off-by: kpango <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Signed-off-by: kpango <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: Yusuke Kadowaki <[email protected]>
* Add config

* Add clusterRole and serviceAccount for agent

* Add more metrics

* Format

* Add export index info duration configuration

* Refactor internal/k8s

* Move apply function to internal/k8s

* Add more config

* Remove unused import and variable

* Use vald errors

* Add pod name and pod namespace to hack e2e agent

* Apply tagalign

* Revert unnecesarry specification of PodName

* Enable export index info to Kubernetes in agent templates only when the config is enabled

* Fix agent clusterrole name

* Update key name

* Update comments

* Add index count as export metrics

* Ignore gomnd

* Add index count entry to exportMetricsOnCreateIndex function

* Update values schema

* Fix formatter job

* Remove unused defines

Co-authored-by: Yusuke Kadowaki <[email protected]>
* improve Backport Pull Request Title & Description Quality (#2373)




* Add readreplica chart release

* update k8s/readreplica

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 70f1dad according to the output
from Gofumpt and Prettier.

Details: #2383

* [bugfix] fix backport github value bypass failure (#2378)



* Revert backport.yaml

---------

Signed-off-by: kpango <[email protected]>
Co-authored-by: Yusuke Kadowaki <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
* fix: delete unnecessary code for mirror



* fix: make deps/install and format



* improve Backport Pull Request Title & Description Quality (#2373)




* fix: naming of variable



---------

Signed-off-by: hlts2 <[email protected]>
Signed-off-by: kpango <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>
Signed-off-by: kpango <[email protected]>
Co-authored-by: taisho <[email protected]>
Co-authored-by: taisuou <[email protected]>
* Add rotate-all option to rotator

* Fix linter warnings

* Make it possible to choose multiple rotation target

* Add tests for parseReplicaID

* Add tests for parseReplicaID

* Hide testify in internal

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in dd84674 according to the output
from Gofumpt and Prettier.

Details: #2305

* Apply format

* Lint

* Lint

---------

Co-authored-by: Yusuke Kadowaki <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: Yusuke Kato <[email protected]>
Signed-off-by: kpango <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: Kiichiro YUKAWA <[email protected]>
)

Signed-off-by: Vdaas CI <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
* improve Backport Pull Request Title & Description Quality (#2373)




* 📝 create continous benchmark doc



* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in ace001a according to the output
from Gofumpt and Prettier.

Details: #2352

* ♻️ rafactor default params



* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 9880dfc according to the output
from Gofumpt and Prettier.

Details: #2352

* ♻️ Fix format



* Revert "improve Backport Pull Request Title & Description Quality (#2373)"

This reverts commit 4246def.

* ♻️ remove dimension from con-bench



* 📝 Add sample benchmark manifest



* Update docs/performance/continuos-benchmatk.md




* Update docs/performance/continuos-benchmatk.md




---------

Signed-off-by: kpango <[email protected]>
Signed-off-by: vankichi <[email protected]>
Signed-off-by: Kiichiro YUKAWA <[email protected]>
Co-authored-by: Kiichiro YUKAWA <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: Hiroto Funakoshi <[email protected]>
Signed-off-by: kpango <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
* 🔖 🤖 Release v1.7.12

Signed-off-by: Vdaas CI <[email protected]>

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 5bd8605 according to the output
from Gofumpt and Prettier.

Details: #2408

---------

Signed-off-by: Vdaas CI <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Signed-off-by: vankichi <[email protected]>
Co-authored-by: Kiichiro YUKAWA <[email protected]>
* 🐳 Fix agent-faiss build failed



* 🐳 Fix COPY binary



* 🐳 Change COPY binary (use ld-linux-x86-64)



---------

Signed-off-by: vankichi <[email protected]>
Co-authored-by: Kiichiro YUKAWA <[email protected]>
* Add tests for index information export

* Add comments

Co-authored-by: Yusuke Kadowaki <[email protected]>
* fix: update action to get tag list



* fix: refactor sub command execution



* fix: use tag list in detect ci container workflow



* fix: name for detected result



* fix: invalid variable bug



* fix: execution error and invalid variable



* fix: deleted unnecessary comment



* fix: deleted unnecessary parameters



* fix: use new determine tag logic in the docker build workflow



* fix: bugfix extra tags handling and small refactor



* fix: build tag rule or release branch based on tag list



* fix: typo



* fix: indent



* fix: refactor syntax



* fix: typo and invalid option error



* fix: use helm extra option



* fix: add comment for tag management



* fix: unsetbug of helm extra option



* fix: build target condition



---------

Signed-off-by: hlts2 <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>
* fix: update build rule for nightly



* fix: indent and deleted unnecessary tag



* fix: update tag name when tag is unknown



* fix: add secondary tags for build performance



* fix: argument order



---------

Signed-off-by: hlts2 <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>
…branch build tag name (#2423) (#2425)

* fix: add outut settings for action



* fix: tag name for release branch build



* fix: refactor



---------

Signed-off-by: hlts2 <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>
vdaas-ci and others added 27 commits April 1, 2024 17:07
* feat: automatically add backport main label for release-pr



* fix: deleted unnecessary quotation



---------

Signed-off-by: Hiroto Funakoshi <[email protected]>
* update deps



* fix server_test.go due to the grpc-go updates



---------

Signed-off-by: kpango <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
) (#2480)

* ♻️ Mount benchmark-operator-config to vald-benchmark-job Pod



* 📈 add framework for send gRPC client metrics



* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in a532b7d according to the output
from Gofumpt and Prettier.

Details: #2477

* 📈 Add client metric interceptor



* ♻️ Change vbor chart for apply general job configuration



* Update pkg/tools/benchmark/job/usecase/benchmarkd.go




* 🐛 Fix build error and add comment



* ♻️ Fix format



---------

Signed-off-by: vankichi <[email protected]>
Signed-off-by: Kiichiro YUKAWA <[email protected]>
Co-authored-by: Kiichiro YUKAWA <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: Hiroto Funakoshi <[email protected]>
* 📝 Update continous benchmark docs



* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in e3ab1d2 according to the output
from Gofumpt and Prettier.

Details: #2485

* Update docs/performance/continuous-benchmark.md



* 📝 Fix



---------

Signed-off-by: vankichi <[email protected]>
Signed-off-by: Kiichiro YUKAWA <[email protected]>
Co-authored-by: Kiichiro YUKAWA <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
* fix: git add chart directory for release (#2356)

Signed-off-by: hlts2 <[email protected]>

* 🔖 🤖 Release v1.7.11 (#2360) (#2361)

* 🔖 🤖 Release v1.7.11



* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 578746e according to the output
from Gofumpt and Prettier.

Details: #2360

---------

Signed-off-by: Vdaas CI <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>

* Change docker scan timeout longer (#2363)

* refactor code using golangci-lint (#2362)

Signed-off-by: kpango <[email protected]>

* Create SECURITY.md (#2367)

* Create SECURITY.md

Signed-off-by: Yusuke Kato <[email protected]>

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in e8609b0 according to the output
from Gofumpt and Prettier.

Details: #2367

---------

Signed-off-by: Yusuke Kato <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>

* add commit hash build image (#2359)

Signed-off-by: kpango <[email protected]>

* Make agent export index metrics to Pod k8s resource (#2319)

* Add config

* Add clusterRole and serviceAccount for agent

* Add more metrics

* Format

* Add export index info duration configuration

* Refactor internal/k8s

* Move apply function to internal/k8s

* Add more config

* Remove unused import and variable

* Use vald errors

* Add pod name and pod namespace to hack e2e agent

* Apply tagalign

* Revert unnecesarry specification of PodName

* Enable export index info to Kubernetes in agent templates only when the config is enabled

* Fix agent clusterrole name

* Update key name

* Update comments

* Add index count as export metrics

* Ignore gomnd

* Add index count entry to exportMetricsOnCreateIndex function

* Update values schema

* Fix formatter job

* Remove unused defines

* update docker build target platform selection rules (#2370)

Signed-off-by: kpango <[email protected]>
Co-authored-by: Yusuke Kadowaki <[email protected]>

* improve Backport Pull Request Title & Description Quality (#2373)

Signed-off-by: kpango <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>

* Update workflow to release readreplica chart (#2383)

* improve Backport Pull Request Title & Description Quality (#2373)

Signed-off-by: kpango <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>

* Add readreplica chart release

* update k8s/readreplica

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 70f1dad according to the output
from Gofumpt and Prettier.

Details: #2383

* [bugfix] fix backport github value bypass failure (#2378)

Signed-off-by: kpango <[email protected]>

* Revert backport.yaml

---------

Signed-off-by: kpango <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>

* 💚 ♻️ Add Con-Bench helm chart to the Vald charts (#2388)

Signed-off-by: vankichi <[email protected]>

* Delete unnecessary code for mirror (#2366)

* fix: delete unnecessary code for mirror

Signed-off-by: hlts2 <[email protected]>

* fix: make deps/install and format

Signed-off-by: hlts2 <[email protected]>

* improve Backport Pull Request Title & Description Quality (#2373)

Signed-off-by: kpango <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>

* fix: naming of variable

Signed-off-by: hlts2 <[email protected]>

---------

Signed-off-by: hlts2 <[email protected]>
Signed-off-by: kpango <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>

* change JP logo to EN logo (#2369)

Signed-off-by: kpango <[email protected]>
Co-authored-by: taisuou <[email protected]>

* Add rotate-all option to rotator (#2305)

* Add rotate-all option to rotator

* Fix linter warnings

* Make it possible to choose multiple rotation target

* Add tests for parseReplicaID

* Add tests for parseReplicaID

* Hide testify in internal

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in dd84674 according to the output
from Gofumpt and Prettier.

Details: #2305

* Apply format

* Lint

* Lint

---------

Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: Yusuke Kato <[email protected]>

* Resolve kvs already closed before last saving (#2390)

Signed-off-by: kpango <[email protected]>

* create continous benchmark doc (#2352)

* improve Backport Pull Request Title & Description Quality (#2373)

Signed-off-by: kpango <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>

* 📝 create continous benchmark doc

Signed-off-by: vankichi <[email protected]>

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in ace001a according to the output
from Gofumpt and Prettier.

Details: #2352

* ♻️ rafactor default params

Signed-off-by: vankichi <[email protected]>

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 9880dfc according to the output
from Gofumpt and Prettier.

Details: #2352

* ♻️ Fix format

Signed-off-by: vankichi <[email protected]>

* Revert "improve Backport Pull Request Title & Description Quality (#2373)"

This reverts commit 4246def.

* ♻️ remove dimension from con-bench

Signed-off-by: vankichi <[email protected]>

* 📝 Add sample benchmark manifest

Signed-off-by: vankichi <[email protected]>

* Update docs/performance/continuos-benchmatk.md

Co-authored-by: Yusuke Kato <[email protected]>
Signed-off-by: Kiichiro YUKAWA <[email protected]>

* Update docs/performance/continuos-benchmatk.md

Co-authored-by: Hiroto Funakoshi <[email protected]>
Signed-off-by: Kiichiro YUKAWA <[email protected]>

---------

Signed-off-by: kpango <[email protected]>
Signed-off-by: vankichi <[email protected]>
Signed-off-by: Kiichiro YUKAWA <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: Hiroto Funakoshi <[email protected]>

* fix: build error of internal kvs test (#2396)

Signed-off-by: hlts2 <[email protected]>

* 🤖 Update license headers / Format Go codes and YAML files (#2397)

Signed-off-by: Vdaas CI <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>

* fix: disable protobuf dispatch for client (#2401)

Signed-off-by: hlts2 <[email protected]>

* update deps (#2404)

Signed-off-by: kpango <[email protected]>

* 🔖 🤖 Release v1.7.12 (#2408) (#2409)

* 🔖 🤖 Release v1.7.12



* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 5bd8605 according to the output
from Gofumpt and Prettier.

Details: #2408

---------

Signed-off-by: Vdaas CI <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>

* Add tests for index information export (#2412)

* Add tests for index information export

* Add comments

* 📝 Fix typo of file name (#2413)

Signed-off-by: vankichi <[email protected]>

* Fix agent-faiss build failed (#2418)

* 🐳 Fix agent-faiss build failed

Signed-off-by: vankichi <[email protected]>

* 🐳 Fix COPY binary

Signed-off-by: vankichi <[email protected]>

* 🐳 Change COPY binary (use ld-linux-x86-64)

Signed-off-by: vankichi <[email protected]>

---------

Signed-off-by: vankichi <[email protected]>

* Fix the logic to determine docker image (#2410)

* fix: update action to get tag list

Signed-off-by: hlts2 <[email protected]>

* fix: refactor sub command execution

Signed-off-by: hlts2 <[email protected]>

* fix: use tag list in detect ci container workflow

Signed-off-by: hlts2 <[email protected]>

* fix: name for detected result

Signed-off-by: hlts2 <[email protected]>

* fix: invalid variable bug

Signed-off-by: hlts2 <[email protected]>

* fix: execution error and invalid variable

Signed-off-by: hlts2 <[email protected]>

* fix: deleted unnecessary comment

Signed-off-by: hlts2 <[email protected]>

* fix: deleted unnecessary parameters

Signed-off-by: hlts2 <[email protected]>

* fix: use new determine tag logic in the docker build workflow

Signed-off-by: hlts2 <[email protected]>

* fix: bugfix extra tags handling and small refactor

Signed-off-by: hlts2 <[email protected]>

* fix: build tag rule or release branch based on tag list

Signed-off-by: hlts2 <[email protected]>

* fix: typo

Signed-off-by: hlts2 <[email protected]>

* fix: indent

Signed-off-by: hlts2 <[email protected]>

* fix: refactor syntax

Signed-off-by: hlts2 <[email protected]>

* fix: typo and invalid option error

Signed-off-by: hlts2 <[email protected]>

* fix: use helm extra option

Signed-off-by: hlts2 <[email protected]>

* fix: add comment for tag management

Signed-off-by: hlts2 <[email protected]>

* fix: unsetbug of helm extra option

Signed-off-by: hlts2 <[email protected]>

* fix: build target condition

Signed-off-by: hlts2 <[email protected]>

---------

Signed-off-by: hlts2 <[email protected]>

* Update build rule for nightly image (#2421)

* fix: update build rule for nightly

Signed-off-by: hlts2 <[email protected]>

* fix: indent and deleted unnecessary tag

Signed-off-by: hlts2 <[email protected]>

* fix: update tag name when tag is unknown

Signed-off-by: hlts2 <[email protected]>

* fix: add secondary tags for build performance

Signed-off-by: hlts2 <[email protected]>

* fix: argument order

Signed-off-by: hlts2 <[email protected]>

---------

Signed-off-by: hlts2 <[email protected]>

* Add `index-operator` template implementation (#2375)

* improve Backport Pull Request Title & Description Quality (#2373)

Signed-off-by: kpango <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>

* [bugfix] fix backport github value bypass failure (#2378)

Signed-off-by: kpango <[email protected]>

* [bugfix] fix backport github value bypass failure (#2381)

Signed-off-by: kpango <[email protected]>

* Add index operator boilerplate

* Add k8s controller

* Add pod and job controllers

* Add annotations in the pod reconciler

* Update auto indexing service configurations

* Update WithOnReconcileFunc signature to include context

* Refactor debugging log

* Refactor reconcile funcs

* add nolint

* remove unused parameters

* lint

* Add dockerfile and build ci

* remove params

* Add index-operator image build command to Makefile

* Add index operator k8s templates

* Update helm schema

* Update readme

* Add internal/k8s as a trigger for images that depends on controller-runtime

* remove comment

* Update pkg/index/operator/service/operator.go

Co-authored-by: Yusuke Kato <[email protected]>
Signed-off-by: Yusuke Kadowaki <[email protected]>

* Update pkg/index/operator/service/operator.go

Co-authored-by: Yusuke Kato <[email protected]>
Signed-off-by: Yusuke Kadowaki <[email protected]>

* Refactor

* Add build commands

* Refactor

Co-authored-by: Yusuke Kato <[email protected]>

---------

Signed-off-by: kpango <[email protected]>
Signed-off-by: Yusuke Kadowaki <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>

* Fix output settings to determine-docker-image-tag action and release branch build tag name (#2423)

* fix: add outut settings for action

Signed-off-by: hlts2 <[email protected]>

* fix: tag name for release branch build

Signed-off-by: hlts2 <[email protected]>

* fix: refactor

Signed-off-by: hlts2 <[email protected]>

---------

Signed-off-by: hlts2 <[email protected]>

* fix: typo of execution rule (#2426)

Signed-off-by: hlts2 <[email protected]>

* Add flush API for deleting all vectors (#2273)

* add flush api

Signed-off-by: aknishid <[email protected]>

* added flush api

* fix typo

* fix pb.go & fix conflict

Signed-off-by: aknishid <[email protected]>

* add flush api

* add handler Flush

Signed-off-by: aknishid <[email protected]>

* add vald.Client flush

* fix filter gateway

Signed-off-by: aknishid <[email protected]>

* fix handler cnts

Signed-off-by: aknishid <[email protected]>

* delete unused var

Signed-off-by: aknishid <[email protected]>

* add author

Signed-off-by: Yamada Takuya <[email protected]>

* fix error

Signed-off-by: aknishid <[email protected]>

* fix flush.go & ngt.go

Signed-off-by: aknishid <[email protected]>

* add flush to example

Signed-off-by: aknishid <[email protected]>

* fix gateway lb

Signed-off-by: aknishid <[email protected]>

* update flush proto

Signed-off-by: aknishid <[email protected]>

* fix docs api README.md

Signed-off-by: aknishid <[email protected]>

* Fix flush ctx

Signed-off-by: aknishid <[email protected]>

* update docs & comment

Signed-off-by: aknishid <[email protected]>

* after make format

Signed-off-by: aknishid <[email protected]>

* add e2e

Signed-off-by: aknishid <[email protected]>

* update example & tutorial

Signed-off-by: aknishid <[email protected]>

* Update internal/errors/ngt.go

Co-authored-by: Kiichiro YUKAWA <[email protected]>

* fix review

Signed-off-by: aknishid <[email protected]>

* modify status.md

Signed-off-by: aknishid <[email protected]>

* add error handling

Signed-off-by: aknishid <[email protected]>

* update proto

Signed-off-by: aknishid <[email protected]>

* fix doc

Signed-off-by: aknishid <[email protected]>

* ♻️ Fix proto

Signed-off-by: vankichi <[email protected]>

* use atomic bool

Signed-off-by: aknishid <[email protected]>

* fix proto

Signed-off-by: aknishid <[email protected]>

* update proto

Signed-off-by: aknishid <[email protected]>

* fix gateway error

Signed-off-by: aknishid <[email protected]>

* ✅ 🚧 Avoid e2e failing due to unimplemented handler and Add unimplmented handler method for the future

Signed-off-by: vankichi <[email protected]>

* Update apis/proto/v1/payload/payload.proto

Co-authored-by: Kiichiro YUKAWA <[email protected]>
Signed-off-by: aknishid <[email protected]>

---------

Signed-off-by: aknishid <[email protected]>
Signed-off-by: Yamada Takuya <[email protected]>
Signed-off-by: vankichi <[email protected]>
Signed-off-by: aknishid <[email protected]>
Co-authored-by: takuyaymd <[email protected]>
Co-authored-by: takuyyam <[email protected]>
Co-authored-by: Kiichiro YUKAWA <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>

* docs: add aknishid as a contributor for code, and maintenance (#2431)

* docs: update README.md [skip ci]

* docs: update .all-contributorsrc [skip ci]

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in abc4c52 according to the output
from Gofumpt and Prettier.

Details: #2431

---------

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>

* update deps & add validation for Flush API when agent is Read Only (#2433)

Signed-off-by: kpango <[email protected]>

* Update README.md (grammer) (#2440)

Signed-off-by: Hrichik Mazumder <[email protected]>

* docs: add hrichiksite as a contributor for doc (#2441)

* docs: update README.md [skip ci]

* docs: update .all-contributorsrc [skip ci]

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in b8054d9 according to the output
from Gofumpt and Prettier.

Details: #2441

---------

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>

* Added snapshot timestamp annotations to read replica agent (#2428)

* Add lastTimeRotateReplicaTimestampAnnotationsKey constant

* Use ptr package

* Update annotations in ngt.go and rotator.go

* Add labels to Pod struct and update annotations handling***

***Introduce vald package for k8s related constants***

***Implement rotateIfNeeded function to check if read replica rotation is needed

* Refactor

* set sample log level debug

* Use k8s timestamp

* Refactor

* Format

* Add MatchingLabels method to ValdK8sClientMock

* Format

* Fix test

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 39b66fa according to the output
from Gofumpt and Prettier.

Details: #2428

* Rename constants

Co-authored-by: Yusuke Kato <[email protected]>

* Refactor rotateIfNeeded function to use label selector for listing deployments

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 6ace761 according to the output
from Gofumpt and Prettier.

Details: #2428

* FIx deepsource

* Revert

---------

Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: Yusuke Kato <[email protected]>

* fix: bugfix version update for docker build (#2445)

Signed-off-by: hlts2 <[email protected]>

* Fix index job logic to pass DNS A record (#2438)

* fix: choose user given target address in priority

Signed-off-by: hlts2 <[email protected]>

* fix: deleted unnecessary code

Signed-off-by: hlts2 <[email protected]>

* fix: add logic to connect the user given target addresses

Signed-off-by: hlts2 <[email protected]>

* fix: add info log about new connection target

Signed-off-by: hlts2 <[email protected]>

* fix: indent of configuration data

Signed-off-by: hlts2 <[email protected]>

* fix: indent bug

Signed-off-by: hlts2 <[email protected]>

* fix: deepsource warning

Signed-off-by: hlts2 <[email protected]>

---------

Signed-off-by: hlts2 <[email protected]>

* Fix operator-sdk version (#2447)

* fix: down grade operator version

Signed-off-by: hlts2 <[email protected]>

* fix: upgrade operator-sdk version

Signed-off-by: hlts2 <[email protected]>

* fix: operator-sdk version

Signed-off-by: hlts2 <[email protected]>

---------

Signed-off-by: hlts2 <[email protected]>

* add file name lint (#2417)

Signed-off-by: kpango <[email protected]>

* fix: add extra option for ci-container build (#2451)

Signed-off-by: hlts2 <[email protected]>

* Add base of benchmark operator dashboard (#2430)

* 📈 Add base of benchmark operator dashboard

Signed-off-by: vankichi <[email protected]>

* 💚 Fix build benchmark images trigger

Signed-off-by: vankichi <[email protected]>

* 💚 Add info tag for info metrics

Signed-off-by: vankichi <[email protected]>

* 📈 showing job image and tags

Signed-off-by: vankichi <[email protected]>

* 📈 add bench job status

Signed-off-by: vankichi <[email protected]>

* ♻️ Update dashboard and rename

Signed-off-by: vankichi <[email protected]>

* ♻️ Fix

Signed-off-by: vankichi <[email protected]>

* Update internal/observability/metrics/tools/benchmark/benchmark.go

Co-authored-by: Hiroto Funakoshi <[email protected]>
Signed-off-by: Kiichiro YUKAWA <[email protected]>

---------

Signed-off-by: vankichi <[email protected]>
Signed-off-by: Kiichiro YUKAWA <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>

* Implement index operator logic for read replica rotation (#2444)

* Remove job reconciler

* Add leader election to index operator

* Add roles for leader election

* Add operator logic for rotation

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 844cca1 according to the output
from Gofumpt and Prettier.

Details: #2444

* Remove unnecesary change

* Fix build

* nits

* Refactor

* Fix clusterrole

* Move to podv2

* Add job concurrency check

* Refactor

* Refactor

* Fix heavy copying

* Update helm schema

* Fix package doc

* Remove unused code and refactor For() method

* Update target read replica ID in config

* Update internal/k8s/podv2/option.go

Co-authored-by: Hiroto Funakoshi <[email protected]>
Signed-off-by: Yusuke Kadowaki <[email protected]>

* Refactor

* rename podv2 to v2/pod

* Add client DI

* Fix envkey helper

* Set namespace from values.yaml

* Use annotations to specify the id

* Update schema

* Update sample

* Update values

* Update schema

* Fix spelling

* Fix package name

* Fix read replica e2e

* Add job templates for rotation, creation, saving, and correction

* Update config.go to decode k8s object

* Update sample.yaml for index operator

* Use job template from cfg

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 584d1f0 according to the output
from Gofumpt and Prettier.

Details: #2444

* Fix tagalign

* Fix config test

* Fix config test

* Disable read replica e2e for now

---------

Signed-off-by: Yusuke Kadowaki <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: Hiroto Funakoshi <[email protected]>

* add inner product distance type for ngt (#2454)

Signed-off-by: kpango <[email protected]>

* Fix e2e for read replica and add e2e for index operator (#2455)

* Fix read replica e2e

* Update default image list in action.yaml

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 6cdac9f according to the output
from Gofumpt and Prettier.

Details: #2455

---------

Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>

* Add unit tests for index operator (#2460)

* Add prototype tests

* Refactor

* Fix not to use kubeconfig

* 🤖 Automatically update PULL_REQUEST_TEMPLATE and ISSUE_TEMPLATE (#2457)

Signed-off-by: Vdaas CI <[email protected]>
Co-authored-by: Kiichiro YUKAWA <[email protected]>

* Bugfix recreate benchmark job when operator reboot (#2463)

* 🐛 Fix recreate benchmark job already runnnig when operator rebooted

Signed-off-by: vankichi <[email protected]>

* 🐛 Fix benchmarkJobReconcile status handling

Signed-off-by: vankichi <[email protected]>

* ✅ Fix test

Signed-off-by: vankichi <[email protected]>

* ♻️ Update k8s dir

Signed-off-by: vankichi <[email protected]>

* ♻️ Fix

Signed-off-by: vankichi <[email protected]>

---------

Signed-off-by: vankichi <[email protected]>

* Refactor k8s types (#2462)

* Refactor types

* Refactor hdf5_test.go

* Format

* Refactor

* add extra flags for NGT build phase (#2467)

* Fix workflow trigger for backport pr creation (#2471)

* Automatically add backport main label for release-pr (#2473)

* feat: automatically add backport main label for release-pr

Signed-off-by: Hiroto Funakoshi <[email protected]>

* fix: deleted unnecessary quotation

Signed-off-by: Hiroto Funakoshi <[email protected]>

---------

Signed-off-by: Hiroto Funakoshi <[email protected]>

* update deps (#2468)

* update deps

Signed-off-by: kpango <[email protected]>

* fix server_test.go due to the grpc-go updates

Signed-off-by: kpango <[email protected]>

---------

Signed-off-by: kpango <[email protected]>

* Update README.md (#2479)

Signed-off-by: Yusuke Kato <[email protected]>

* Implement client metrics interceptor for continuous benchmark job (#2477)

* ♻️ Mount benchmark-operator-config to vald-benchmark-job Pod

Signed-off-by: vankichi <[email protected]>

* 📈 add framework for send gRPC client metrics

Signed-off-by: vankichi <[email protected]>

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in a532b7d according to the output
from Gofumpt and Prettier.

Details: #2477

* 📈 Add client metric interceptor

Signed-off-by: vankichi <[email protected]>

* ♻️ Change vbor chart for apply general job configuration

Signed-off-by: vankichi <[email protected]>

* Update pkg/tools/benchmark/job/usecase/benchmarkd.go

Co-authored-by: Hiroto Funakoshi <[email protected]>
Signed-off-by: Kiichiro YUKAWA <[email protected]>

* 🐛 Fix build error and add comment

Signed-off-by: vankichi <[email protected]>

* ♻️ Fix format

Signed-off-by: vankichi <[email protected]>

---------

Signed-off-by: vankichi <[email protected]>
Signed-off-by: Kiichiro YUKAWA <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: Hiroto Funakoshi <[email protected]>

* implement ngt-rs and its unit test and add GitHub Actions workflows (#2478)

* implement test

* update rust dependencies

* fix

* make format

* add Github Actions workflows

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

---------

Co-authored-by: Yusuke Kato <[email protected]>

* 📈 Add client metrics panels for continuous benchmark job (#2481)

Signed-off-by: vankichi <[email protected]>

* Add rust versions (#2482)

* add rust versions

* fix

* add rust version

* add rust-toolchain.toml

* make format

* Update continuous benchmark docs (#2485)

* 📝 Update continous benchmark docs

Signed-off-by: vankichi <[email protected]>

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in e3ab1d2 according to the output
from Gofumpt and Prettier.

Details: #2485

* Update docs/performance/continuous-benchmark.md

Signed-off-by: Kiichiro YUKAWA <[email protected]>

* 📝 Fix

Signed-off-by: vankichi <[email protected]>

---------

Signed-off-by: vankichi <[email protected]>
Signed-off-by: Kiichiro YUKAWA <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>

* update NGT version (#2484)

* fix sed line (#2488)

* 🤖 Automatically update PULL_REQUEST_TEMPLATE and ISSUE_TEMPLATE (#2487)

Signed-off-by: Vdaas CI <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>

* Add efficient search logic for large top-k users (#2491)

* add efficient search logic for large top-k users

Signed-off-by: kpango <[email protected]>

* fix bug

Signed-off-by: kpango <[email protected]>

---------

Signed-off-by: kpango <[email protected]>

---------

Signed-off-by: hlts2 <[email protected]>
Signed-off-by: Vdaas CI <[email protected]>
Signed-off-by: kpango <[email protected]>
Signed-off-by: Yusuke Kato <[email protected]>
Signed-off-by: vankichi <[email protected]>
Signed-off-by: Kiichiro YUKAWA <[email protected]>
Signed-off-by: Yusuke Kadowaki <[email protected]>
Signed-off-by: aknishid <[email protected]>
Signed-off-by: Yamada Takuya <[email protected]>
Signed-off-by: aknishid <[email protected]>
Signed-off-by: Hrichik Mazumder <[email protected]>
Signed-off-by: Hiroto Funakoshi <[email protected]>
Co-authored-by: vdaas-ci <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: Yusuke Kadowaki <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>
Co-authored-by: Kiichiro YUKAWA <[email protected]>
Co-authored-by: taisho <[email protected]>
Co-authored-by: taisuou <[email protected]>
Co-authored-by: aknishid <[email protected]>
Co-authored-by: takuyaymd <[email protected]>
Co-authored-by: takuyyam <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Hrichik Mazumder <[email protected]>
* add read replica and rotator docs



* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in bc69bc5 according to the output
from Gofumpt and Prettier.

Details: #2497

* Update docs/user-guides/read-replica-and-rotator.md




* Update docs/user-guides/read-replica-and-rotator.md




* Update docs/user-guides/read-replica-and-rotator.md




* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 49a3155 according to the output
from Gofumpt and Prettier.

Details: #2497

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 93ed666 according to the output
from Gofumpt and Prettier.

Details: #2497

* Update docs/user-guides/read-replica-and-rotator.md




* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 3644007 according to the output
from Gofumpt and Prettier.

Details: #2497

* Update docs/user-guides/read-replica-and-rotator.md




* Update docs/user-guides/read-replica-and-rotator.md




* Update docs/user-guides/read-replica-and-rotator.md




* Apply suggestions from code review




* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 8f82b96 according to the output
from Gofumpt and Prettier.

Details: #2497

* Apply suggestions from code review




* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 9d125bc according to the output
from Gofumpt and Prettier.

Details: #2497

* fix linter suggestions



* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in a8ee4ab according to the output
from Gofumpt and Prettier.

Details: #2497

---------

Signed-off-by: Kosuke Morimoto <[email protected]>
Signed-off-by: Kosuke Morimoto <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
* add reviewer guideline



* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in 97aa0a6 according to the output
from Gofumpt and Prettier.

Details: #2507

* fix typo



* Apply suggestions from code review




* Apply suggestions from code review




* fix



* fix



* fix review request



---------

Signed-off-by: Kosuke Morimoto <[email protected]>
Signed-off-by: Kosuke Morimoto <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Signed-off-by: Kosuke Morimoto <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>
* fix: change default image tag for example



* fix: change default image tag for values



---------

Signed-off-by: hlts2 <[email protected]>
…#2517) (#2520)

* fix: bugfix a problem that caused an error when argument has 3 or more nil values



* fix: apply coderabbit suggestion



* Update internal/errors/errors.go




* fix: create error if e is nil



---------

Signed-off-by: hlts2 <[email protected]>
Signed-off-by: Hiroto Funakoshi <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
* add faiss in values.yaml, fix valdrelease.yaml

---------

Co-authored-by: datelier <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>
* capitalize faq



* fix link



---------

Signed-off-by: Kosuke Morimoto <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>
* fix links (#2500)

Signed-off-by: Kosuke Morimoto <[email protected]>

* fix indent and link (#2501)

Signed-off-by: Kosuke Morimoto <[email protected]>

* fix link (#2502)

Signed-off-by: Kosuke Morimoto <[email protected]>

* fix typo (#2503)

* fix typo (#2504)

* change fixed version to replace string (#2506)

* change fixed version to replace string

Signed-off-by: Kosuke Morimoto <[email protected]>

* Update docs/overview/component/agent.md

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Signed-off-by: Kosuke Morimoto <[email protected]>

---------

Signed-off-by: Kosuke Morimoto <[email protected]>
Signed-off-by: Kosuke Morimoto <[email protected]>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

---------

Signed-off-by: Kosuke Morimoto <[email protected]>
Signed-off-by: Kosuke Morimoto <[email protected]>
Co-authored-by: Kosuke Morimoto <[email protected]>
Co-authored-by: datelier <[email protected]>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
* feat: add sync workflow for ubuntu base image



* fix: add todo comment



* fix: docker pull step



* fix: invalid command



* fix: add docker tagging



* fix: invalid config



* fix: enable arm



* fix: add tag



* fix: invalid command



* fix: add debug job



* fix: invalid command



* fix: invalid command



* fix: invalid command



* fix: refactor



* fix: add push image



* fix: image not found error



* fix: use docker build action



* fix: add docker file



* fix: add docker meta



* fix: invalid command



* fix: add build command for buildbase



* fix: deleted unnecessary workflow



* fix: file name



* fix: image name



* fix: invalid name



* fix: resolve warning



* fix: deleted unnecessary file



* fix: add file to event path



* fix: add schedule event



---------

Signed-off-by: hlts2 <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>
Signed-off-by: hlts2 <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>
* fix: deleted unnecessary event path for docker build (#2538)

Signed-off-by: hlts2 <[email protected]>

* fix: fix info path location (#2542)

Signed-off-by: hlts2 <[email protected]>

---------

Signed-off-by: hlts2 <[email protected]>
Signed-off-by: hlts2 <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>
Signed-off-by: kpango <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
* feat: add workflow to check conflict



* Apply suggestions from code review



* feat: send conflict warning comment



* fix: verify for sending comment



* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in aee3f32 according to the output
from Gofumpt and Prettier.

Details: #2548

* fix: verify for sending comment



* fix: debug body



* fix: add log



* feat: add debug code



* fix: add debug



* fix: deleted unnecessary debug code



* fix: fails comment for sending comment



* fix: deleted unnecessary changes



* feat: fix grep condition



* fix: add changes for verify



* Revert "fix: add changes for verify"

This reverts commit 98f4c44.

* fix: add changes for verify



* Revert "fix: add changes for verify"

This reverts commit 31ed2a8.

---------

Signed-off-by: hlts2 <[email protected]>
Signed-off-by: Yusuke Kato <[email protected]>
Co-authored-by: Hiroto Funakoshi <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
… build systems and localization (#2549) (#2557)

Signed-off-by: kpango <[email protected]>
Co-authored-by: Yusuke Kato <[email protected]>
Signed-off-by: kpango <[email protected]>
Copy link
Contributor

coderabbitai bot commented Aug 2, 2024

Important

Review skipped

More than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review.

100 files out of 256 files are above the max files limit of 50. Please upgrade to Pro plan to get higher limits.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

Share
Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai generate interesting stats about this repository and render them as a table.
    • @coderabbitai show all the console.log statements in this repository.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (invoked as PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Additionally, you can add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants