Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Update kctrl usage docs to use a different namespace and not the default one #592

Closed
praveenrewar opened this issue Nov 11, 2022 · 5 comments
Labels
carvel accepted This issue should be considered for future work and that the triage process has been completed documentation This issue indicates a change to the docs should be considered enhancement This issue is a feature request

Comments

@praveenrewar
Copy link
Member

praveenrewar commented Nov 11, 2022

Describe the problem/challenge you have
From the docs Consuming Packages using CLI, when I copy the command to install the TCE repository and run it, I get an error saying Error: Creating sensitive resources in a shared namespace (default) which is expected but probably not a good experience for someone who is trying it out for the first time.

Describe the solution you'd like
The example usage should use the -n flag to specify a namespace to adhere to the new behaviour of kctrl.

Anything else you would like to add
The same should be done for other examples as well.

@praveenrewar praveenrewar added enhancement This issue is a feature request carvel-triage labels Nov 11, 2022
@carvel-bot carvel-bot moved this to To Triage in Carvel Nov 11, 2022
@praveenrewar praveenrewar added carvel accepted This issue should be considered for future work and that the triage process has been completed and removed carvel-triage labels Nov 11, 2022
@a-mccarthy
Copy link
Contributor

@praveenrewar is this issue still relevant? Is this problem related to TCE specifically or more how the code sample is written? I'd be interested to know how removing TCE references from this page would impact the code references.

@praveenrewar
Copy link
Member Author

@a-mccarthy This issue is not relevant to TCE.
@microwavables brought this up a couple of days back. I will be making some changes to this page to remove TCE references. We could just use some other package repository and that should be fine, but I want to ensure that we can use that same package repository for the end to end workflow described in this page. So I am planning to create a test package repository and use that here.

@a-mccarthy
Copy link
Contributor

Thanks @praveenrewar! Is there a documentation label we can add to this issue?

@praveenrewar praveenrewar added the documentation This issue indicates a change to the docs should be considered label Apr 11, 2024
@praveenrewar
Copy link
Member Author

@a-mccarthy Done!

@praveenrewar
Copy link
Member Author

This was fixed as part of #750

@github-project-automation github-project-automation bot moved this from To Triage to Closed in Carvel Apr 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
carvel accepted This issue should be considered for future work and that the triage process has been completed documentation This issue indicates a change to the docs should be considered enhancement This issue is a feature request
Projects
None yet
Development

No branches or pull requests

2 participants