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

Remove storage class from logging tutorial

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.2.0
    • documentation
    • None
    • 1

    Description

      Specifying the storage class in the logging tutorial triggers a failure on platforms where it does not exist. We should remove it so it just uses the default.

      https://docs-staging.couchbase.com/operator/current/tutorial-couchbase-log-forwarding.html#configure-the-couchbase-cluster

        volumeClaimTemplates:
        - metadata:
            name: couchbase
          spec:
            storageClassName: standard
            resources:
              requests:
                storage: 1Gi

      Remove that `storageClassName`. Add a call out to storage class docs as well.

      Attachments

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

        Activity

          Please merge this change as soon as possible.

          roshani.sanghavi Roshani Sanghavi (Inactive) added a comment - Please merge this change as soon as possible.

          Re-assigning back to assignee. Patrick Stephens and I have chatted about this issue already, and he is going to make the change himself tomorrow UK time. My guess is that he still may do some verification first before making the change.

          eric.schneider Eric Schneider (Inactive) added a comment - Re-assigning back to assignee. Patrick Stephens and I have chatted about this issue already, and he is going to make the change himself tomorrow UK time. My guess is that he still may do some verification first before making the change.

          I've also fixed the non-ANSI quotes around the secret name.

          patrick.stephens Patrick Stephens (Inactive) added a comment - I've also fixed the non-ANSI quotes around the secret name.

          The secret configuration seems to be causing issues when copy-pasted from the HTML. If I copy-paste the text directly from the Asciidoc it is fine but something seems to be triggering problems in the parsing of the version from the HTML.

          patrick.stephens Patrick Stephens (Inactive) added a comment - The secret configuration seems to be causing issues when copy-pasted from the HTML. If I copy-paste the text directly from the Asciidoc it is fine but something seems to be triggering problems in the parsing of the version from the HTML.

          It seems to be the call outs triggering it. Removing the spaces seems to help but I've added a quick note about checking formatting as well.

          patrick.stephens Patrick Stephens (Inactive) added a comment - It seems to be the call outs triggering it. Removing the spaces seems to help but I've added a quick note about checking formatting as well.

          Patrick Stephens Thank you so much for quickly fixing the changes required and actively following up on the issues. Really appreciate it!

           

          roshani.sanghavi Roshani Sanghavi (Inactive) added a comment - Patrick Stephens  Thank you so much for quickly fixing the changes required and actively following up on the issues. Really appreciate it!  

          As a follow up, we've tried to implement some extra docs checks to catch this: https://issues.couchbase.com/browse/K8S-2251

          patrick.stephens Patrick Stephens (Inactive) added a comment - As a follow up, we've tried to implement some extra docs checks to catch this: https://issues.couchbase.com/browse/K8S-2251

          People

            eric.schneider Eric Schneider (Inactive)
            patrick.stephens Patrick Stephens (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty