Uploaded image for project: 'Couchbase Kubernetes'
  1. Couchbase Kubernetes
  2. K8S-1073

Simplify Pod Wait Logic

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Major
    • 1.2.2, 2.0.0
    • None
    • operator
    • None

    Description

      Feedback from a user suggested the following.  We create 5 PVCs, waiting for 10s for each to become ready then create the pod.  10s expiry isn't a failure as WaitForFirstConsumer volumes will fall into this trap.

      Sadly on Azure some non-lazy volumes still aren't bound when the pod is created and it fails scheduling.  Now we do have a Watch that polls the pod for readiness, but at the first sign of trouble it bails out.  In this case it doesn't wait for Kubernetes to complete the bind and schedule the pod as intended.

      What we should be doing is the following:

      • Don't wait for PVCs, this is an unnecessary performance bottleneck that will affect both customers and our testing.
      • Waiting for the pod to become ready should be an infinite loop (cancelled by the context), thus always honours the pod creation timeout - no short cuts that defy logic!
      • When the pod enters the Running phase we are done.

      Simple, performant, infallible (in theory)

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              simon.murray Simon Murray
              simon.murray Simon Murray
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty