Skip to content

Conversation

@nginx-bot
Copy link
Contributor

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

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

* 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 requested review from a team as code owners May 7, 2025 16:02
@nginx-bot nginx-bot added documentation Pull requests/issues for documentation needs cherry pick Cherry pick this PR into a release branch labels May 7, 2025
@github-actions
Copy link
Contributor

github-actions bot commented May 7, 2025

Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/nginx-ingress-controller/7763/

@vepatel vepatel enabled auto-merge (squash) May 7, 2025 16:10
@vepatel vepatel changed the title [cherry-pick] docs: NIC+NAP wafv5 install guide perm fix NIC+NAP wafv5 install guide perm fix May 7, 2025
@vepatel vepatel merged commit 63a9b30 into release-5.0 May 8, 2025
23 checks passed
@vepatel vepatel deleted the cherry-pick-release-5.0-fb446888b76eaa5b9496f6c81b1bf289e5e32656 branch May 8, 2025 08:59
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 needs cherry pick Cherry pick this PR into a release branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants