Skip to content

Conversation

sjberman
Copy link
Collaborator

cherry-pick R33 PR into release 1.5 branch

Adding support for NGINX Plus R33. The major change with this release is that NGINX Plus now requires a JWT in order to run. A user must create a Secret with this JWT and supply the secret name to NGF when installing. A user can also create client SSL and CA Secrets for NIM connections. All of these Secrets are mounted to the nginx container.

Because of the new usage reporting method, the old usage reporting method has been removed and CLI arguments have been altered. Since this release is a breaking change for N+ users, the choice was made to remove the unused usage reporting flags instead of deprecating them.

Updated documentation to describe this process, while also cleaning up the JWT docker registry process for N+.
@sjberman sjberman requested review from a team as code owners November 19, 2024 19:39
@github-actions github-actions bot added documentation Improvements or additions to documentation tests Pull requests that update tests helm-chart Relates to helm chart labels Nov 19, 2024
@sjberman sjberman added the enhancement New feature or request label Nov 19, 2024
@codecov
Copy link

codecov bot commented Nov 19, 2024

Codecov Report

Attention: Patch coverage is 83.45865% with 66 lines in your changes missing coverage. Please review.

Project coverage is 89.86%. Comparing base (1f60f9e) to head (4da04a6).

Files with missing lines Patch % Lines
cmd/gateway/commands.go 74.52% 24 Missing and 3 partials ⚠️
internal/mode/static/manager.go 74.44% 23 Missing ⚠️
internal/mode/static/handler.go 86.04% 4 Missing and 2 partials ⚠️
cmd/gateway/validation.go 91.42% 2 Missing and 1 partial ⚠️
internal/mode/static/nginx/config/main_config.go 95.58% 1 Missing and 2 partials ⚠️
internal/mode/static/nginx/file/folders.go 33.33% 1 Missing and 1 partial ⚠️
internal/mode/static/state/graph/graph.go 86.66% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##           release-1.5    #2782      +/-   ##
===============================================
+ Coverage        89.40%   89.86%   +0.46%     
===============================================
  Files              110      107       -3     
  Lines            10913    10997      +84     
  Branches            50       50              
===============================================
+ Hits              9757     9883     +126     
+ Misses            1098     1054      -44     
- Partials            58       60       +2     

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


🚨 Try these New Features:

@sjberman sjberman closed this Nov 19, 2024
@sjberman sjberman deleted the cherry-pick/nplus-r33 branch November 19, 2024 20:20
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 enhancement New feature or request helm-chart Relates to helm chart tests Pull requests that update tests

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants