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

CRDs are installed despite of createCRDs: false #1208

Closed
vkryzh opened this issue May 9, 2024 · 4 comments
Closed

CRDs are installed despite of createCRDs: false #1208

vkryzh opened this issue May 9, 2024 · 4 comments

Comments

@vkryzh
Copy link

vkryzh commented May 9, 2024

I believe, issue is introduced here - cce949e

Expected Behaviour

createCRDs: false - no CRDs should be installed

Current Behaviour

createCRDs: false - 3 CRDs are installed (policies, roles, jwtissuers)

Who is this for?

Corva

List All Possible Solutions and Workarounds

  • add flag to new CRDs to skip their installation

Steps to Reproduce (for bugs)

  1. Install latest Openfaas with only createCRDs: false in values override
  2. Have CRDs installed

Context

With Namespace admin permissions we cannot install CRDs, so their installation should be moved to a different place. And, as it stated in chart documentation, we should be able to do this using simple flag.

@alexellis
Copy link
Member

Hi @vkryzh

In order to help you out here quickly, we will need to know what commands you're running (including the skip CRD option)

These items are included in the issue templates in most places, if you don't see them in a repo, go ahead and add in the extra context.

We're missing:

  • Steps to Reproduce (for bugs)
  • Context (i.e. why is this needed for you?)

Alex

@vkryzh
Copy link
Author

vkryzh commented May 10, 2024

Hey @alexellis . Added context, thank you.

@alexellis
Copy link
Member

alexellis commented May 10, 2024

Hi @vkryzh

Those IAM-based CRDs can be suppressed via --skip-crds being passed into Helm

I'm having a look at the docs for "namespaced installation" at the moment so will get this closed.

The full commands you're using would always be appreciated. It will help us avoid this back and forth and guesswork, which ultimately help us to help you.

Hope this helps.

Alex

@alexellis
Copy link
Member

Related to this issue, we've consolidated the way CRDs will be created, feel free to comment:

#1212

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

No branches or pull requests

2 participants