From daa111156f1a4b6f9b97750e02ccff23578293d9 Mon Sep 17 00:00:00 2001 From: Ed Bartosh Date: Sun, 21 Oct 2018 10:27:47 +0300 Subject: [PATCH] kubeadm: fix missing error handling Handled error returned from CertificateSigningRequests().Get() --- cmd/kubeadm/app/phases/certs/renewal/certsapi.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/cmd/kubeadm/app/phases/certs/renewal/certsapi.go b/cmd/kubeadm/app/phases/certs/renewal/certsapi.go index f49ddf69ca6..1d5e45516e2 100644 --- a/cmd/kubeadm/app/phases/certs/renewal/certsapi.go +++ b/cmd/kubeadm/app/phases/certs/renewal/certsapi.go @@ -119,6 +119,9 @@ func (r *CertsAPIRenewal) Renew(cfg *certutil.Config) (*x509.Certificate, *rsa.P } req, err = r.client.CertificateSigningRequests().Get(req.Name, metav1.GetOptions{}) + if err != nil { + return nil, nil, errors.Wrap(err, "couldn't get certificate signing request") + } if len(req.Status.Conditions) < 1 { return nil, nil, errors.New("certificate signing request has no statuses") }