Skip to content

Commit 5efd999

Browse files
Merge branch 'main' into docs/jwt-token-update
Signed-off-by: Jason Williams - NGIИX <[email protected]>
2 parents d7a7f7a + 5af0ea5 commit 5efd999

File tree

207 files changed

+8355
-4191
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

207 files changed

+8355
-4191
lines changed

.github/ISSUE_TEMPLATE/bug_report.md

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ A clear and concise description of what the bug is.
1212

1313
**To Reproduce**
1414
Steps to reproduce the behavior:
15+
1516
1. Deploy x to '...' using some.yaml
1617
2. View logs on '....'
1718
3. See error
@@ -20,10 +21,11 @@ Steps to reproduce the behavior:
2021
A clear and concise description of what you expected to happen.
2122

2223
**Your environment**
23-
* Version of the Ingress Controller - release version or a specific commit
24-
* Version of Kubernetes
25-
* Kubernetes platform (e.g. Mini-kube or GCP)
26-
* Using NGINX or NGINX Plus
24+
25+
- Version of the Ingress Controller - release version or a specific commit
26+
- Version of Kubernetes
27+
- Kubernetes platform (e.g. Mini-kube or GCP)
28+
- Using NGINX or NGINX Plus
2729
<!-- output from `docker inspect --format '{{ json .Config.Labels }}' <docker image> | jq` if available -->
2830

2931
**Additional context**

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,10 @@
11
### Proposed changes
2-
Describe the use case and detail of the change. If this PR addresses an issue on GitHub, make sure to include a link to that issue here in this description (not in the title of the PR).
2+
3+
Describe the use case and detail of the change. If this PR addresses an issue on GitHub, make sure to include a link to
4+
that issue here in this description (not in the title of the PR).
35

46
### Checklist
7+
58
Before creating a PR, run through this checklist and mark each as complete.
69

710
- [ ] I have read the [CONTRIBUTING](https://github.com/nginxinc/kubernetes-ingress/blob/main/CONTRIBUTING.md) doc

.github/actions/smoke-tests/action.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ runs:
7474
- name: Build Test-Runner Container
7575
uses: docker/build-push-action@v3
7676
with:
77-
file: tests/docker/Dockerfile
77+
file: tests/Dockerfile
7878
context: '.'
7979
cache-from: type=gha,scope=test-runner
8080
tags: test-runner:${{ github.sha }}

.github/dependabot.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ updates:
1616
interval: daily
1717

1818
- package-ecosystem: docker
19-
directory: /tests/docker
19+
directory: /tests
2020
schedule:
2121
interval: daily
2222

.github/dependency-review-config.yml

Lines changed: 0 additions & 13 deletions
This file was deleted.

.github/labels.yml

Lines changed: 0 additions & 75 deletions
This file was deleted.

.github/workflows/build-oss.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,9 @@ defaults:
1717
run:
1818
shell: bash
1919

20+
permissions:
21+
contents: read
22+
2023
jobs:
2124
build:
2225
runs-on: ubuntu-22.04
@@ -158,7 +161,7 @@ jobs:
158161
ignore-unfixed: "true"
159162

160163
- name: Upload Trivy scan results to GitHub Security tab
161-
uses: github/codeql-action/upload-sarif@489225d82a57396c6f426a40e66d461b16b3461d # v2.20.4
164+
uses: github/codeql-action/upload-sarif@a09933a12a80f87b87005513f0abb1494c27a716 # v2.21.4
162165
continue-on-error: true
163166
with:
164167
sarif_file: "trivy-results-${{ inputs.image }}.sarif"
@@ -179,5 +182,8 @@ jobs:
179182
tag: ${{ inputs.tag }}
180183
version: ${{ needs.build.outputs.version }}
181184
image_digest: ${{ needs.build.outputs.image_digest }}
185+
permissions:
186+
contents: read
187+
actions: read
182188
secrets: inherit
183189
if: ${{ inputs.tag != '' }}

.github/workflows/build-plus.yml

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -76,22 +76,22 @@ jobs:
7676
with:
7777
aws-region: us-east-1
7878
role-to-assume: ${{ secrets.AWS_ROLE_MARKETPLACE }}
79-
if: startsWith(github.ref, 'refs/tags/') && contains(inputs.target, 'aws')
79+
if: github.ref_type == 'tag' && contains(inputs.target, 'aws')
8080

8181
- name: Login to ECR
8282
uses: docker/login-action@465a07811f14bebb1938fbed4728c6a1ff8901fc # v2.2.0
8383
with:
8484
registry: 709825985650.dkr.ecr.us-east-1.amazonaws.com
85-
if: startsWith(github.ref, 'refs/tags/') && contains(inputs.target, 'aws')
85+
if: github.ref_type == 'tag' && contains(inputs.target, 'aws')
8686

8787
- name: Docker meta
8888
id: meta
8989
uses: docker/metadata-action@818d4b7b91585d195f67373fd9cb0332e31a7175 # v4.6.0
9090
with:
9191
images: |
9292
name=gcr.io/f5-gcs-7899-ptg-ingrss-ctlr/dev/nginx-ic${{ contains(inputs.nap_modules, 'dos') && '-dos' || '' }}${{ contains(inputs.nap_modules, 'waf') && '-nap' || '' }}/nginx-plus-ingress
93-
name=gcr.io/f5-gcs-7899-ptg-ingrss-ctlr/release/nginx-ic${{ contains(inputs.nap_modules, 'dos') && '-dos' || '' }}${{ contains(inputs.nap_modules, 'waf') && '-nap' || '' }}/nginx-plus-ingress,enable=${{ startsWith(github.ref, 'refs/tags/') }}
94-
name=709825985650.dkr.ecr.us-east-1.amazonaws.com/nginx/nginx-plus-ingress${{ contains(inputs.nap_modules, 'dos') && '-dos' || '' }}${{ contains(inputs.nap_modules, 'waf') && '-nap' || '' }},enable=${{ startsWith(github.ref, 'refs/tags/') && contains(inputs.target, 'aws') }}
93+
name=gcr.io/f5-gcs-7899-ptg-ingrss-ctlr/release/nginx-ic${{ contains(inputs.nap_modules, 'dos') && '-dos' || '' }}${{ contains(inputs.nap_modules, 'waf') && '-nap' || '' }}/nginx-plus-ingress,enable=${{ github.ref_type == 'tag' }}
94+
name=709825985650.dkr.ecr.us-east-1.amazonaws.com/nginx/nginx-plus-ingress${{ contains(inputs.nap_modules, 'dos') && '-dos' || '' }}${{ contains(inputs.nap_modules, 'waf') && '-nap' || '' }},enable=${{ github.ref_type == 'tag' && contains(inputs.target, 'aws') }}
9595
flavor: |
9696
suffix=${{ contains(inputs.image, 'ubi') && '-ubi' || '' }}${{ contains(inputs.image, 'alpine') && '-alpine' || '' }}${{ contains(inputs.target, 'aws') && '-mktpl' || '' }}${{ contains(inputs.image, 'fips') && '-fips' || ''}},onlatest=true
9797
latest=${{ contains(inputs.target, 'aws') && 'false' || 'auto' }}
@@ -132,7 +132,7 @@ jobs:
132132
provenance: false
133133
build-args: |
134134
BUILD_OS=${{ inputs.image }}
135-
IC_VERSION=${{ startsWith(github.ref, 'refs/tags/') && steps.meta.outputs.version || 'CI' }}
135+
IC_VERSION=${{ github.ref_type == 'tag' && steps.meta.outputs.version || 'CI' }}
136136
${{ inputs.nap_modules != '' && format('NAP_MODULES={0}', inputs.nap_modules) || '' }}
137137
${{ steps.nap_modules.outputs.modules != '' && format('NAP_MODULES_AWS={0}', steps.nap_modules.outputs.modules) || '' }}
138138
secrets: |
@@ -156,7 +156,7 @@ jobs:
156156
echo "version=$version" >> $GITHUB_OUTPUT
157157
echo "product_code=AWS${nap}_PRODUCT_ID" >> $GITHUB_OUTPUT
158158
echo "registry=${aws_registry}" >> $GITHUB_OUTPUT
159-
if: startsWith(github.ref, 'refs/tags/') && contains(inputs.target, 'aws')
159+
if: github.ref_type == 'tag' && contains(inputs.target, 'aws')
160160

161161
- name: Publish to AWS Marketplace
162162
uses: nginxinc/aws-marketplace-publish@93e03c5ce4baa842a8e5baad0a3f35d07b38460c # v0.1.2
@@ -173,7 +173,7 @@ jobs:
173173
This container requires Kubernetes and can be deployed to EKS.
174174
Review the installation instructions https://docs.nginx.com/nginx-ingress-controller/installation/ and utilize the deployment resources available https://github.com/nginxinc/kubernetes-ingress/tree/master/deployments
175175
Use this image instead of building your own.
176-
if: ${{ startsWith(github.ref, 'refs/tags/') && contains(inputs.target, 'aws') }}
176+
if: ${{ github.ref_type == 'tag' && contains(inputs.target, 'aws') }}
177177

178178
- name: Load image for Trivy
179179
uses: docker/build-push-action@2eb1c1961a95fc15694676618e422e8ba1d63825 # v4.1.1
@@ -186,7 +186,7 @@ jobs:
186186
load: true
187187
build-args: |
188188
BUILD_OS=${{ inputs.image }}
189-
IC_VERSION=${{ startsWith(github.ref, 'refs/tags/') && steps.meta.outputs.version || 'CI' }}
189+
IC_VERSION=${{ github.ref_type == 'tag' && steps.meta.outputs.version || 'CI' }}
190190
${{ inputs.nap_modules != '' && format('NAP_MODULES={0}', inputs.nap_modules) || '' }}
191191
${{ steps.nap_modules.outputs.modules != '' && format('NAP_MODULES_AWS={0}', steps.nap_modules.outputs.modules) || '' }}
192192
secrets: |
@@ -204,7 +204,7 @@ jobs:
204204
ignore-unfixed: "true"
205205

206206
- name: Upload Trivy scan results to GitHub Security tab
207-
uses: github/codeql-action/upload-sarif@489225d82a57396c6f426a40e66d461b16b3461d # v2.20.4
207+
uses: github/codeql-action/upload-sarif@a09933a12a80f87b87005513f0abb1494c27a716 # v2.21.4
208208
continue-on-error: true
209209
with:
210210
sarif_file: "trivy-results-${{ inputs.image }}.sarif"

0 commit comments

Comments
 (0)