Skip to content

NFR Test Results for NGF version 2.1.0 #3727

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

Merged
merged 4 commits into from
Aug 14, 2025
Merged

Conversation

bjee19
Copy link
Contributor

@bjee19 bjee19 commented Aug 13, 2025

Add the NFR and Longevity test results.

@bjee19 bjee19 requested a review from a team as a code owner August 13, 2025 17:39
@github-actions github-actions bot added documentation Improvements or additions to documentation chore Pull requests for routine tasks tests Pull requests that update tests labels Aug 13, 2025
@bjee19 bjee19 changed the title Add NFR test results NFR Test Results for NGF version 2.1.0 Aug 13, 2025
Copy link

codecov bot commented Aug 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.93%. Comparing base (33d442b) to head (2991def).
⚠️ Report is 1 commits behind head on release-2.1.

Additional details and impacted files
@@               Coverage Diff               @@
##           release-2.1    #3727      +/-   ##
===============================================
- Coverage        86.95%   86.93%   -0.02%     
===============================================
  Files              128      128              
  Lines            15986    15986              
  Branches            62       62              
===============================================
- Hits             13900    13898       -2     
- Misses            1919     1921       +2     
  Partials           167      167              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bjee19
Copy link
Contributor Author

bjee19 commented Aug 13, 2025

I want to highlight the errors seen in the Longevity tests, and the downtime seen in the upgrade test. Do these things pose any high concern to the upcoming release?

@sjberman
Copy link
Collaborator

The upgrade errors are a bit concerning. Longevity I just don't know. It could be due to automatic cloud upgrades. I think for next time we increase the number of nodes, and maybe run over 3 days instead of 4.

@salonichf5
Copy link
Contributor

I'll open up a story to go through these issues for scale, upgrade and longevity tests, if you would like @bjee19 ?

@bjee19
Copy link
Contributor Author

bjee19 commented Aug 13, 2025

I'll open up a story to go through these issues for scale, upgrade and longevity tests, if you would like @bjee19 ?

Yea sure if you'd like to that would be great, thanks @salonichf5

@bjee19 bjee19 force-pushed the chore/nfr-test-results branch from 74404c8 to 2991def Compare August 14, 2025 16:47
@bjee19 bjee19 enabled auto-merge (squash) August 14, 2025 16:54
@bjee19 bjee19 merged commit fd94860 into release-2.1 Aug 14, 2025
64 of 66 checks passed
@bjee19 bjee19 deleted the chore/nfr-test-results branch August 14, 2025 17:26
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in NGINX Gateway Fabric Aug 14, 2025
bjee19 added a commit that referenced this pull request Aug 14, 2025
Add the NFR and Longevity test results.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Pull requests for routine tasks documentation Improvements or additions to documentation tests Pull requests that update tests
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants