- 
                Notifications
    
You must be signed in to change notification settings  - Fork 1k
 
Kustomization #608
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
Kustomization #608
Conversation
| 
           Thanks for the contributions. Another K8s tool, I've never heard about ^^  | 
    
| 
           I like it. Seems pretty slim. Maybe it can be reduced to the instructions how to install with kubectl, incl. brief min version? Maybe we can also slim down the Helm section to the most basic instructions and not too many words about helm.  | 
    
          
 It is a built-in plugin in recent Kubectl release. For quickstart, one line instruction on the version constraint and some link might be good enough.  | 
    
| 
           By the way, the manifests is kind of out of date comparing to helm chart output  | 
    
| 
           It's now mentioned in the Quickstart. Thanks again @jakub-roman for pointing us to this feature :)  | 
    
| 
           @FxKu the main value of this PR was the  The idea was that you manage the kustomization file in this repository - you are the ones who know how to install the operator properly after all - so people doesn't have to know which manifests to use. The alternative is that people make their own kustomization files (which is already redundant), but then if you decide to add another manifest file necessary to install the operator everyone that uses kustomization will have to know about that change and make amendments in their kustomization files and that's just painful... P.S. sorry I'm coming back to this so late, I didn't have access to internet last week  | 
    
| 
           We will add the file ;)  | 
    
          
 Should I just remove the quickstart docs since you already have a note about it so you can merge in this one?  | 
    
| 
           Yes. Or you can add one more sentence with a link to the provided Kustomization file. Then i'll merge it.  | 
    
Add ability to install the operator via kustomization.
| 
           👍  | 
    
    
      
        1 similar comment
      
    
  
    | 
           👍  | 
    
| 
           @jakub-roman thanks for your contribution :)  | 
    
| 
           Thanks for including this, it makes the installation way easier for us :)  | 
    
Add ability to install the operator via kustomization as proposed in #607