Skip to content

NFR Test Results for NGF version edge #3391

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

Closed
wants to merge 5 commits into from
Closed

Conversation

nginx-bot
Copy link
Contributor

Update with NFR test results for NGF version edge ["oss","plus"]

  • Auto-generated by the NFR tests workflow run 15143764098
  • Tests ran using Docker image tag edge
  • all test(s) ran

@github-actions github-actions bot added documentation Improvements or additions to documentation tests Pull requests that update tests labels May 20, 2025
Copy link
Contributor

@bjee19 bjee19 May 20, 2025

Choose a reason for hiding this comment

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

Seems like we are encountering a little bit of dropped traffic in some of our zero-downtime-scale tests.


### NGINX Error Logs
2025/05/20 19:47:56 [emerg] 8#8: unexpected end of file, expecting ";" or "}" in /etc/nginx/conf.d/http.conf:1608
Copy link
Contributor

Choose a reason for hiding this comment

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

These weird errors are showing up still, which we have communicated with the nginx agent team.

@bjee19 bjee19 force-pushed the chore/fix-nfr-workflow branch from 66d6f36 to f3c93e8 Compare May 20, 2025 23:05
Base automatically changed from chore/fix-nfr-workflow to main May 20, 2025 23:30
@bjee19 bjee19 closed this May 20, 2025
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in NGINX Gateway Fabric May 20, 2025
Copy link

codecov bot commented May 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.75%. Comparing base (b6ff28c) to head (b00314e).
Report is 11 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3391      +/-   ##
==========================================
- Coverage   86.78%   86.75%   -0.04%     
==========================================
  Files         129      129              
  Lines       14862    14862              
  Branches       62       62              
==========================================
- Hits        12898    12893       -5     
- Misses       1816     1820       +4     
- Partials      148      149       +1     

☔ 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.

@sjberman sjberman deleted the tests/nfr-tests-edge branch May 28, 2025 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

2 participants