Skip to content

Conversation

lucacome
Copy link

Proposed changes

Docs changes for v3.3.0

@lucacome lucacome requested review from a team as code owners September 25, 2023 19:15
@lucacome lucacome self-assigned this Sep 25, 2023
@github-actions github-actions bot added documentation Pull requests/issues for documentation helm_chart Pull requests that update the Helm Chart labels Sep 25, 2023
|Debian-based image with NGINX App Protect WAF | ``debian:11-slim`` | NGINX App Protect WAF, NGINX Plus JavaScript and OpenTracing modules, OpenTracing tracers for Jaeger, Zipkin and Datadog | `nginx-ic-nap/nginx-plus-ingress:3.3.0` | amd64 |
|Debian-based image with NGINX App Protect DoS | ``debian:11-slim`` | NGINX App Protect DoS, NGINX Plus JavaScript and OpenTracing modules, OpenTracing tracers for Jaeger, Zipkin and Datadog | `nginx-ic-dos/nginx-plus-ingress:3.3.0` | amd64 |
|Debian-based image with NGINX App Protect WAF and DoS | ``debian:11-slim`` | NGINX App Protect WAF and DoS, NGINX Plus JavaScript and OpenTracing modules, OpenTracing tracers for Jaeger, Zipkin and Datadog | `nginx-ic-nap-dos/nginx-plus-ingress:3.3.0` | amd64 |
|Ubi-based image | ``redhat/ubi9-minimal`` | NGINX Plus JavaScript module | `nginx-ic/nginx-plus-ingress:3.3.0-ubi` | arm64, amd64, s390x |
Copy link
Collaborator

Choose a reason for hiding this comment

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

There should also be a ubi8-minimal option here. Do we have a naming / label pattern to support both?

Copy link
Author

Choose a reason for hiding this comment

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

we don't have a ubi8 based image yet

@codecov
Copy link

codecov bot commented Sep 25, 2023

Codecov Report

Merging #4423 (65cf7e0) into main (a8ba452) will decrease coverage by 0.03%.
Report is 2 commits behind head on main.
The diff coverage is n/a.

❗ Current head 65cf7e0 differs from pull request most recent head a56f56e. Consider uploading reports for the commit a56f56e to get more accurate results

@@            Coverage Diff             @@
##             main    #4423      +/-   ##
==========================================
- Coverage   52.11%   52.08%   -0.03%     
==========================================
  Files          59       59              
  Lines       16956    16956              
==========================================
- Hits         8836     8832       -4     
- Misses       7823     7825       +2     
- Partials      297      299       +2     

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@vepatel vepatel requested a review from ADubhlaoich September 26, 2023 15:31
@lucacome lucacome requested review from a team and brianehlert September 26, 2023 17:23
@lucacome lucacome merged commit f255b03 into main Sep 26, 2023
@lucacome lucacome deleted the docs/release-3.3.0 branch September 26, 2023 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Pull requests/issues for documentation helm_chart Pull requests that update the Helm Chart

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants