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

cbopinfo: all flag not collecting all available custom resource defenitions

    XMLWordPrintable

Details

    Description

       

      Created a crd for portworx storage deployment. This info is missing in the collected log file.

      Test case: TestLogCollectClusterWithPVC

       

      couchbase-operator]$ ./build/bin/cbopinfo -operator-image couchbase/couchbase-operator-internal:1.0.0-448 -kubeconfig ~/.kube/config_NewCluster1 -namespace default -all
      Wrote cluster information to cbopinfo-20180926T152736+0530.tar.gz

      couchbase-operator]$ export KUBECONFIG=~/.kube/config_NewCluster1
      Ashwins-MacBook-Pro:couchbase-operator]$ kubectl get crd -n default
      NAME AGE
      couchbaseclusters.couchbase.com 22m
      etcdclusters.etcd.database.coreos.com 19m
      couchbase-operator]${noformat}
       

      Attachments

        For Gerrit Dashboard: K8S-600
        # Subject Branch Project Status CR V

        Activity

          Updated the test case to fetch only the couchbase CRD for verification.

          Hence closing this ticket

          ashwin.govindarajulu Ashwin Govindarajulu added a comment - Updated the test case to fetch only the couchbase CRD for verification. Hence closing this ticket
          ashwin.govindarajulu Ashwin Govindarajulu added a comment - - edited

          Fine. I will update the test case and send it for review

          ashwin.govindarajulu Ashwin Govindarajulu added a comment - - edited Fine. I will update the test case and send it for review
          simon.murray Simon Murray added a comment -

          For now yes, until Product Management or Support start moaning for whatever reason

          simon.murray Simon Murray added a comment - For now yes, until Product Management or Support start moaning for whatever reason

          So in this case, should we only look for "couchbase.com" crds for our verification ?

          ashwin.govindarajulu Ashwin Govindarajulu added a comment - So in this case, should we only look for "couchbase.com" crds for our verification ?
          simon.murray Simon Murray added a comment -

          This is true, but the General Data Protection Act Article 5 1b states we should only collect potentially personal information for explicit purposes.  If we can omit it we should.  I'm not saying every one will create CRDs called or containing "john-smith.company.com" revealing a name, but legally we need to be as restrictive as possible for safety.  In this case we know explicitly what CRDs we can and should collect.

          simon.murray Simon Murray added a comment - This is true, but the General Data Protection Act Article 5 1b states we should only collect potentially personal information for explicit purposes.  If we can omit it we should.  I'm not saying every one will create CRDs called or containing "john-smith.company.com" revealing a name, but legally we need to be as restrictive as possible for safety.  In this case we know explicitly what CRDs we can and should collect.

          But that is what cbopinfo without '-all' flag does.

          So by giving '-all' we are supposed to get all available resource irrespective of type of resources.

          ashwin.govindarajulu Ashwin Govindarajulu added a comment - But that is what cbopinfo without '-all' flag does. So by giving '-all' we are supposed to get all available resource irrespective of type of resources.
          simon.murray Simon Murray added a comment -

          Not a bug, we only collect things in the namespace ".couchbase.com" so we don't collect things we neither need nor care about, nor reveal too much about what the customer is running.

          simon.murray Simon Murray added a comment - Not a bug, we only collect things in the namespace ".couchbase.com" so we don't collect things we neither need nor care about, nor reveal too much about what the customer is running.

          People

            ashwin.govindarajulu Ashwin Govindarajulu
            ashwin.govindarajulu Ashwin Govindarajulu
            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