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

[SGW] TLS required by default from Sync Gateway 3.0

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Major
    • 2.3.0
    • None
    • testing
    • 1

    Description

      Sync-Gateway test case(s) affected: TestSyncGatewayCreateRemote

      Refer: https://docs.couchbase.com/sync-gateway/current/secure-sgw-access.html#overview

      TLS not used in test case mentioned above: https://github.com/couchbase/couchbase-operator/blob/master/test/e2e/sync_gateway_test.go#L83

      Error: 

      2022-02-28T10:58:21.455Z [ERR] Couldn't start Sync Gateway: 1 errors:
      Must use secure scheme in Couchbase Server URL, or opt out by setting bootstrap.use_tls_server to false. Current URL: couchbase://test-couchbase-2shsz-srv.test-hqbzd?network=default -- rest.ServerMain() at main.go:26 

      Resolution: Skip TestSyncGatewayCreateRemote against Sync Gateway Images >= 3.0.0

      as we will be already testing the required scenario in TestSyncGatewayCreateRemoteTLS.  (https://github.com/couchbase/couchbase-operator/blob/master/test/e2e/sync_gateway_test.go#L87 )

       

      If this is not acceptable, we would need to set the bootstrap.use_tls_server to false in sync_gateway.json (https://github.com/couchbase/couchbase-operator/blob/515dfb9b8b2575d2f46715a33473380e1e06de21/test/e2e/e2eutil/sync-gateway.go#L113 ) according to different SGW image versions.

       

      Skipping makes more sense without cluttering the codebase, but I would leave it to you for the final decision.

      CC: Simon Murray 

      Attachments

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

        Activity

          People

            prateek.kumar Prateek Kumar (Inactive)
            prateek.kumar Prateek Kumar (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty