Skip to content

Commit d4ce16b

Browse files
authored
Remove njs-modules configmap (#959)
Problem: PR #934 did not remove the njs-modules ConfigMap and they are some lingering references to it. Solution: Remove the njs modules ConfigMap and remove references.
1 parent 1f47bc0 commit d4ce16b

File tree

3 files changed

+0
-222
lines changed

3 files changed

+0
-222
lines changed

.yamllint.yaml

-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ rules:
3030
check-multi-line-strings: true
3131
ignore: |
3232
deploy/manifests/nginx-gateway.yaml
33-
deploy/manifests/njs-modules.yaml
3433
key-duplicates: enable
3534
key-ordering: disable
3635
line-length:

deploy/manifests/njs-modules.yaml

-214
This file was deleted.

internal/mode/static/nginx/modules/README.md

-7
Original file line numberDiff line numberDiff line change
@@ -111,10 +111,3 @@ This project uses [prettier](https://prettier.io/) to lint and format the JavaSc
111111
```shell
112112
npm run format
113113
```
114-
115-
## Generate the ConfigMap yaml
116-
117-
The NJS modules are mounted to the NGINX container using a ConfigMap. Once you have completed your code changes, ensure
118-
the [njs-modules.yaml](/deploy/manifests/njs-modules.yaml) and
119-
[NJS ConfigMap Helm template](/deploy/helm-chart/templates/njs-modules.yaml) are updated to reflect the new code. See
120-
[Update NJS module ConfigMaps](/docs/developer/quickstart.md#update-njs-module-configmaps).

0 commit comments

Comments
 (0)