kubernetes
/
kops
mirror of https://github.com/kubernetes/kops.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Don't try building TLS for etcd-manager if not using etcd-manager

This commit is contained in:
John Gardiner Myers 2020-06-08 10:01:13 -07:00
parent 7650a988ce
commit bacaf87f0e
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
nodeup/pkg/model/etcd_manager_tls.go
View File

@ -18,8 +18,8 @@ package model
import ( import (
"crypto/x509/pkix" "crypto/x509/pkix"
"fmt"
"k8s.io/klog"
"k8s.io/kops/upup/pkg/fi" "k8s.io/kops/upup/pkg/fi"
"k8s.io/kops/upup/pkg/fi/nodeup/nodetasks" "k8s.io/kops/upup/pkg/fi/nodeup/nodetasks"
) )
@ -33,7 +33,7 @@ var _ fi.ModelBuilder = &EtcdManagerTLSBuilder{}
// Build is responsible for TLS configuration for etcd-manager // Build is responsible for TLS configuration for etcd-manager
func (b *EtcdManagerTLSBuilder) Build(ctx *fi.ModelBuilderContext) error { func (b *EtcdManagerTLSBuilder) Build(ctx *fi.ModelBuilderContext) error {
if !b.IsMaster { if !b.IsMaster || !b.UseEtcdManager() {
return nil return nil
} }
@ -52,8 +52,7 @@ func (b *EtcdManagerTLSBuilder) Build(ctx *fi.ModelBuilderContext) error {
return err return err
} }
if cert == nil { if cert == nil {
klog.Warningf("keypair %q not found, won't configure", keystoreName) return fmt.Errorf("keypair %q not found", keystoreName)
continue
} }
if err := b.BuildCertificateTask(ctx, keystoreName, d+"/"+fileName+".crt"); err != nil { if err := b.BuildCertificateTask(ctx, keystoreName, d+"/"+fileName+".crt"); err != nil {