Skip to content

Conversation

@kayx23
Copy link
Member

@kayx23 kayx23 commented Sep 9, 2025

Type of change:

  • Bugfix
  • New feature provided
  • Improve performance
  • Backport patches
  • Documentation
  • Refactor
  • Chore
  • CI/CD or Tests

What this PR does / why we need it:

Currently the doc explicitly sets the preferred namespace with kubectl config set-context --current --namespace=ingress-apisix and leaves out the namespace specification in manifest file. However if users did not read the docs sequentially and set the preferred namespace, resources could be applied to the wrong namespace, leading to downstream issues.

This PR explicitly sets namespace in metadata.

Pre-submission checklist:

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

@kayx23 kayx23 merged commit ebaed22 into master Sep 10, 2025
21 checks passed
@kayx23 kayx23 deleted the docs-specify-ns branch September 10, 2025 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants