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 Filter query configuration to Continuous Benchmark Job #2296

Merged
merged 3 commits into from
Jan 19, 2024

Conversation

hlts2
Copy link
Collaborator

@hlts2 hlts2 commented Jan 18, 2024

Description:

WHY

In the following PR, we made a change to pass a query to FilterConfig.
This caused a build error of the Continuous Benchmark Job, but I fixed it in this PR.

WHAT

Related Issue:

Versions:

  • Go Version: 1.21.5
  • Docker Version: 20.10.8
  • Kubernetes Version: v1.28.4
  • NGT Version: 2.1.6

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

@vdaas-ci
Copy link
Collaborator

[WARNING:INTCFG] Changes in interal/config may require you to change Helm charts. Please check.

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

cloudflare-workers-and-pages bot commented Jan 18, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0e6b740
Status: ✅  Deploy successful!
Preview URL: https://9aa83f1c.vald.pages.dev
Branch Preview URL: https://fix-filter-build-error.vald.pages.dev

View logs

Copy link

codecov bot commented Jan 18, 2024

Codecov Report

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

Comparison is base (ae34b8a) 30.28% compared to head (0e6b740) 30.26%.

Files Patch % Lines
internal/config/benchmark.go 0.00% 16 Missing ⚠️
Additional details and impacted files
@@                             Coverage Diff                             @@
##           feature/egress-gateway/add-egress-query    #2296      +/-   ##
===========================================================================
- Coverage                                    30.28%   30.26%   -0.02%     
===========================================================================
  Files                                          378      378              
  Lines                                        37242    37255      +13     
===========================================================================
- Hits                                         11277    11275       -2     
- Misses                                       25407    25422      +15     
  Partials                                       558      558              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@vdaas-ci
Copy link
Collaborator

Profile Report

typevald-agent-ngtvald-lb-gatewayvald-discoverervald-manager-index
cpu
heap
other images

@hlts2 hlts2 marked this pull request as ready for review January 18, 2024 08:02
@hlts2 hlts2 requested review from vankichi, a team and kmrmt and removed request for a team January 18, 2024 08:02
Copy link
Contributor

@vankichi vankichi left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@kmrmt kmrmt left a comment

Choose a reason for hiding this comment

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

LGTM

@kmrmt kmrmt merged commit 16c25cd into feature/egress-gateway/add-egress-query Jan 19, 2024
224 of 228 checks passed
@kmrmt kmrmt deleted the fix/filter/build-error branch January 19, 2024 01:13
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