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

Null pointer from deleting non-existent backup

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 2.3.0
    • operator
    • None
    • 48
    • 1

    Description

      when deleting a CouchbacBackup resource instantly after creating it, operator segfaults.

      Reproduction steps:

      1. create a cluster with back up enabled
      2. apply a backup configuration as per https://docs.couchbase.com/operator/current/howto-backup.html
      3. delete the yaml
      4. check operator logs for segfault

      see attached logs for example

      Attachments

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

        Activity

          Alex.emery Alex Emery created issue -
          Alex.emery Alex Emery made changes -
          Field Original Value New Value
          Attachment logs.txt [ 167114 ]
          Description Reproduction steps:
           # create a cluster with back up enabled
           # apply a backup configuration as per [https://docs.couchbase.com/operator/current/howto-backup.html]
           # delete the yaml
           # check logs for segfault
          when deleting a CouchbacBackup resource instantly after creating it, operator segfaults.

          Reproduction steps:
           # create a cluster with back up enabled
           # apply a backup configuration as per [https://docs.couchbase.com/operator/current/howto-backup.html]
           # delete the yaml
           # check operator logs for segfault

          see attached logs for example
          Summary Null pointer from deleting Null pointer from deleting non-existent backup
          simon.murray Simon Murray added a comment -

          Smells a lot like a lack of test coverage...

          simon.murray Simon Murray added a comment - Smells a lot like a lack of test coverage...
          ingenthr Matt Ingenthron made changes -
          Fix Version/s 2.3.0 [ 17600 ]
          ingenthr Matt Ingenthron made changes -
          Rank Ranked higher
          ingenthr Matt Ingenthron made changes -
          Rank Ranked lower
          ingenthr Matt Ingenthron made changes -
          Sprint 48 [ 1889 ]
          ingenthr Matt Ingenthron made changes -
          Assignee Simon Murray [ simon.murray ] Alex Emery [ JIRAUSER26037 ]
          Alex.emery Alex Emery made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Alex.emery Alex Emery made changes -
          Status In Progress [ 3 ] Open [ 1 ]
          simon.murray Simon Murray made changes -
          Rank Ranked higher
          Alex.emery Alex Emery made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Alex.emery Alex Emery made changes -
          Resolution Fixed [ 1 ]
          Status In Progress [ 3 ] Resolved [ 5 ]

          Build couchbase-operator-2.3.0-200 contains couchbase-operator commit feefd71 with commit message:
          K8S-2507: Backup cronjob removal segfaults

          build-team Couchbase Build Team added a comment - Build couchbase-operator-2.3.0-200 contains couchbase-operator commit feefd71 with commit message: K8S-2507 : Backup cronjob removal segfaults
          arunkumar Arunkumar Senthilnathan (Inactive) made changes -
          Assignee Alex Emery [ JIRAUSER26037 ] Prateek Kumar [ prateek.kumar ]

          Prateek Kumar Please verify and close this issue

          arunkumar Arunkumar Senthilnathan (Inactive) added a comment - Prateek Kumar Please verify and close this issue

          Test added by Dev passes with backup image.

          prateek.kumar Prateek Kumar (Inactive) added a comment - Test added by Dev passes with backup image.
          prateek.kumar Prateek Kumar (Inactive) made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

          People

            prateek.kumar Prateek Kumar (Inactive)
            Alex.emery Alex Emery
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty