Uploaded image for project: 'OpenShift Bugs'
  1. OpenShift Bugs
  2. OCPBUGS-19052

Avoid caching etcdctl on cluster-backup.sh

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done-Errata
    • Icon: Normal Normal
    • 4.15.0
    • 4.13, 4.12, 4.11, 4.14, 4.15
    • Etcd
    • None
    • Low
    • No
    • 1
    • ETCD Sprint 242, ETCD Sprint 243
    • 2
    • False
    • Hide

      None

      Show
      None
    • Hide
      * Previously, the `cluster-backup.sh` script cached the `etcdctl` binary on the local machine indefinitely, making updates impossible. Now, the `cluster-backup.sh` script pulls the latest `etcdctl` binary each time it is run. (link:https://issues.redhat.com/browse/OCPBUGS-19052[*OSDOCS-19052*])
      Show
      * Previously, the `cluster-backup.sh` script cached the `etcdctl` binary on the local machine indefinitely, making updates impossible. Now, the `cluster-backup.sh` script pulls the latest `etcdctl` binary each time it is run. (link: https://issues.redhat.com/browse/OCPBUGS-19052 [*OSDOCS-19052*])
    • Bug Fix
    • Done

      Description of problem:

      With OCPBUGS-18274 we had to update the etcdctl binary. Unfortunately the script does not attempt to update the binary if it's found in the path already:
      
      https://github.com/openshift/cluster-etcd-operator/blob/master/bindata/etcd/etcd-common-tools#L16-L24
      
      This causes confusion as the binary might not be the latest that we're shipping with etcd.
      
      Pulling the binary shouldn't be a big deal, etcd is running locally anyway and the local image should be cached already just fine. We should always replace the binary

      Version-Release number of selected component (if applicable):

      any currently supported release

      How reproducible:

      always

      Steps to Reproduce:

      1. run cluster-backup.sh to download the binary
      2. update the etcd image (take a different version or so)
      3. run cluster-backup.sh again 

      Actual results:

      cluster-backup.sh will simply print "etcdctl is already installed"

      Expected results:

      etcdctl should always be pulled

      Additional info:

       

              tjungblu@redhat.com Thomas Jungblut
              tjungblu@redhat.com Thomas Jungblut
              Ge Liu Ge Liu
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Created:
                Updated:
                Resolved: