Skip to content

[Bug]: napv5 + nic install doc fails for docker engine #7757

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
javorszky opened this issue May 7, 2025 · 1 comment · Fixed by #7760
Closed

[Bug]: napv5 + nic install doc fails for docker engine #7757

javorszky opened this issue May 7, 2025 · 1 comment · Fixed by #7760
Assignees
@javorszky
Labels
bug An issue reporting a potential bug documentation Pull requests/issues for documentation needs triage An issue that needs to be triaged

Comments

@javorszky
Copy link
Contributor

Version

edge

What Kubernetes platforms are you running on?

Kind

Steps to reproduce

Following the documentation on https://docs.nginx.com/nginx-ingress-controller/installation/installing-nic/deploy-with-nap-using-helm/ while installing using docker engine, rather than docker desktop, fails in certain places.
@javorszky javorszky self-assigned this May 7, 2025
@javorszky javorszky added bug An issue reporting a potential bug documentation Pull requests/issues for documentation needs triage An issue that needs to be triaged labels May 7, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented May 7, 2025

Hi @javorszky thanks for reporting!

Be sure to check out the docs and the Contributing Guidelines while you wait for a human to take a look at this 🙂

Cheers!

javorszky added a commit that referenced this issue May 7, 2025
@javorszky
docs: NIC+NAP wafv5 install guide perm fix
d5d0378 
Closes #7757

* Use `/tmp` folder on both the host and the docker image when compiling
  the waf policy bundle due to file permission handling differences
  between docker engine and docker desktop
* Update the installation guide to reflect the new location of the
  waf policy bundle and downloaded source JSON
@javorszky javorszky mentioned this issue May 7, 2025
Merged
6 tasks
javorszky added a commit that referenced this issue May 7, 2025
@javorszky
docs: NIC+NAP wafv5 install guide perm fix (#7760)
fb44688 
* docs: NIC+NAP wafv5 install guide perm fix

Closes #7757

* Use `/tmp` folder on both the host and the docker image when compiling
  the waf policy bundle due to file permission handling differences
  between docker engine and docker desktop
* Update the installation guide to reflect the new location of the
  waf policy bundle and downloaded source JSON

* Change the destination of an mv to /Users/g.javorszky/Projects/NIC/kubernetes-ingress-docs instead of .

* Fix a typo for the downloaded file name
nginx-bot pushed a commit that referenced this issue May 7, 2025
@javorszky @web-flow
docs: NIC+NAP wafv5 install guide perm fix (#7760)
c0085c9 
* docs: NIC+NAP wafv5 install guide perm fix

Closes #7757

* Use `/tmp` folder on both the host and the docker image when compiling
  the waf policy bundle due to file permission handling differences
  between docker engine and docker desktop
* Update the installation guide to reflect the new location of the
  waf policy bundle and downloaded source JSON

* Change the destination of an mv to /Users/g.javorszky/Projects/NIC/kubernetes-ingress-docs instead of .

* Fix a typo for the downloaded file name
@nginx-bot nginx-bot mentioned this issue May 7, 2025
Merged
6 tasks
haywoodsh pushed a commit that referenced this issue May 15, 2025
@javorszky @haywoodsh
docs: NIC+NAP wafv5 install guide perm fix (#7760)
549bbf6 
* docs: NIC+NAP wafv5 install guide perm fix

Closes #7757

* Use `/tmp` folder on both the host and the docker image when compiling
  the waf policy bundle due to file permission handling differences
  between docker engine and docker desktop
* Update the installation guide to reflect the new location of the
  waf policy bundle and downloaded source JSON

* Change the destination of an mv to /Users/g.javorszky/Projects/NIC/kubernetes-ingress-docs instead of .

* Fix a typo for the downloaded file name
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@javorszky javorszky

Labels
bug An issue reporting a potential bug documentation Pull requests/issues for documentation needs triage An issue that needs to be triaged
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs: NIC+NAP wafv5 install guide perm fix nginx/kubernetes-ingress
1 participant
@javorszky