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

e2e: multiple gc test #2707

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cnvergence
Copy link
Member

@cnvergence cnvergence commented Feb 26, 2024

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #3364

Copy link

codecov bot commented Feb 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.12%. Comparing base (b90f81c) to head (57d216a).
Report is 26 commits behind head on main.

❗ Current head 57d216a differs from pull request most recent head 3ae2615. Consider uploading reports for the commit 3ae2615 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2707      +/-   ##
==========================================
- Coverage   67.11%   63.12%   -4.00%     
==========================================
  Files         164      121      -43     
  Lines       23773    19599    -4174     
==========================================
- Hits        15955    12371    -3584     
+ Misses       6898     6431     -467     
+ Partials      920      797     -123     

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

Copy link

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. Please feel free to give a status update now, ping for review, when it's ready. Thank you for your contributions!

@github-actions github-actions bot added stale and removed stale labels Mar 27, 2024
Copy link

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. Please feel free to give a status update now, ping for review, when it's ready. Thank you for your contributions!

@github-actions github-actions bot added the stale label Apr 27, 2024
@arkodg
Copy link
Contributor

arkodg commented May 6, 2024

@cnvergence we could circumvent this issue by following what @shawnh2 did in decefcc where he manually creates and applies the resources via the client API 😆

@github-actions github-actions bot removed the stale label May 7, 2024
@cnvergence
Copy link
Member Author

@cnvergence we could circumvent this issue by following what @shawnh2 did in decefcc where he manually creates and applies the resources via the client API 😆

I see 😄 and still ended up using suite client, I will then try this as well

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
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.

E2E: multiple GatewayClass test
2 participants