Skip to content
This repository has been archived by the owner on Dec 15, 2021. It is now read-only.

Update quick-start.md to use kubeless namespace #1006

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update quick-start.md to use kubeless namespace #1006

wants to merge 1 commit into from

Conversation

jorijinnall
Copy link

You ask at the beggining to create the namespace kubeless, but then it is not used at all.
That makes me some confusion, I advise to use it everywhere

Issue Ref: #1005

Description: Should use kubeless namespace everywhere

[PR Description]

TODOs:

  • [x ] Ready to review
  • Automated Tests
  • [ x] Docs

You ask at the beggining to create the namespace kubeless, but then it is not used at all.
That makes me some confusion, I advise to use it everywhere
@andresmgot
Copy link
Contributor

Hi @jorijinnall,

The kubeless namespace is used at the very beginning. Kubeless controllers are installed, by default, in that namespace. That's why you need to create it beforehand. You can check that the Kubeless controller is there if you execute kubectl get pods -n kubeless.

For the rest of the guide, I'd rather use the default namespace since the commands are simpler and easier to understand.

Thank you for your suggestion! I hope it's more clear now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants