Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-6923

Please rename lib/couchbase module so that it doesn't conflict with public sdk

    Details

    • Type: Bug
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: bug-backlog
    • Component/s: test-execution
    • Security Level: Public
    • Labels:
      None

      Description

      Hey Deep, believe you have many tests relying on the couchbase.py module. Problem is whenever I have public couchbase sdk installed and try to use anything from testrunner that relies on rest_client, I get the following error:

      File "../lib/membase/api/rest_client.py", line 8, in <module>
      from couchbase.document import DesignDocument, View
      ImportError: No module named document

      In the past I've done workaround by modifying sys.path to import local couchbase module first. But now I have a complicated situation where this is no longer working.
      Will be great to find another name for this: 'lib/couchbaseinternal' or 'lib/rest_couchbase' ?

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

        Activity

        Hide
        thuan Thuan Nguyen added a comment -

        Integrated in single-node-2.0.x-windows7-64-view #18 (See http://qa.hq.northscale.net/job/single-node-2.0.x-windows7-64-view/18/)
        MB-6923: workaround for couchbase module confilct (Revision a81142c9617483e325e4eab98e8fc92ecae68b5a)

        Result = UNSTABLE
        tmcafee :
        Files :

        • lib/membase/api/rest_client.py
        Show
        thuan Thuan Nguyen added a comment - Integrated in single-node-2.0.x-windows7-64-view #18 (See http://qa.hq.northscale.net/job/single-node-2.0.x-windows7-64-view/18/ ) MB-6923 : workaround for couchbase module confilct (Revision a81142c9617483e325e4eab98e8fc92ecae68b5a) Result = UNSTABLE tmcafee : Files : lib/membase/api/rest_client.py
        Hide
        thuan Thuan Nguyen added a comment -

        Integrated in multi-nodes-2.0.x-windows-64-backup-cli #18 (See http://qa.hq.northscale.net/job/multi-nodes-2.0.x-windows-64-backup-cli/18/)
        MB-6923: workaround for couchbase module confilct (Revision a81142c9617483e325e4eab98e8fc92ecae68b5a)

        Result = UNSTABLE
        tmcafee :
        Files :

        • lib/membase/api/rest_client.py
        Show
        thuan Thuan Nguyen added a comment - Integrated in multi-nodes-2.0.x-windows-64-backup-cli #18 (See http://qa.hq.northscale.net/job/multi-nodes-2.0.x-windows-64-backup-cli/18/ ) MB-6923 : workaround for couchbase module confilct (Revision a81142c9617483e325e4eab98e8fc92ecae68b5a) Result = UNSTABLE tmcafee : Files : lib/membase/api/rest_client.py
        Hide
        thuan Thuan Nguyen added a comment -

        Integrated in single-node-windows-64-install #372 (See http://qa.hq.northscale.net/job/single-node-windows-64-install/372/)
        MB-6923: workaround for couchbase module confilct (Revision a81142c9617483e325e4eab98e8fc92ecae68b5a)

        Result = SUCCESS
        tmcafee :
        Files :

        • lib/membase/api/rest_client.py
        Show
        thuan Thuan Nguyen added a comment - Integrated in single-node-windows-64-install #372 (See http://qa.hq.northscale.net/job/single-node-windows-64-install/372/ ) MB-6923 : workaround for couchbase module confilct (Revision a81142c9617483e325e4eab98e8fc92ecae68b5a) Result = SUCCESS tmcafee : Files : lib/membase/api/rest_client.py
        Hide
        thuan Thuan Nguyen added a comment -

        Integrated in multi-nodes-windows-64-viewtest #20 (See http://qa.hq.northscale.net/job/multi-nodes-windows-64-viewtest/20/)
        MB-6923: workaround for couchbase module confilct (Revision a81142c9617483e325e4eab98e8fc92ecae68b5a)

        Result = SUCCESS
        tmcafee :
        Files :

        • lib/membase/api/rest_client.py
        Show
        thuan Thuan Nguyen added a comment - Integrated in multi-nodes-windows-64-viewtest #20 (See http://qa.hq.northscale.net/job/multi-nodes-windows-64-viewtest/20/ ) MB-6923 : workaround for couchbase module confilct (Revision a81142c9617483e325e4eab98e8fc92ecae68b5a) Result = SUCCESS tmcafee : Files : lib/membase/api/rest_client.py
        Hide
        thuan Thuan Nguyen added a comment -

        Integrated in multi-nodes-2.0.x-windows-64-install #17 (See http://qa.hq.northscale.net/job/multi-nodes-2.0.x-windows-64-install/17/)
        MB-6923: workaround for couchbase module confilct (Revision a81142c9617483e325e4eab98e8fc92ecae68b5a)

        Result = SUCCESS
        tmcafee :
        Files :

        • lib/membase/api/rest_client.py
        Show
        thuan Thuan Nguyen added a comment - Integrated in multi-nodes-2.0.x-windows-64-install #17 (See http://qa.hq.northscale.net/job/multi-nodes-2.0.x-windows-64-install/17/ ) MB-6923 : workaround for couchbase module confilct (Revision a81142c9617483e325e4eab98e8fc92ecae68b5a) Result = SUCCESS tmcafee : Files : lib/membase/api/rest_client.py
        Hide
        maria Maria McDuff (Inactive) added a comment -

        Deep, Tommie,

        is this still a valid request? or we can close this now?

        Show
        maria Maria McDuff (Inactive) added a comment - Deep, Tommie, is this still a valid request? or we can close this now?
        Hide
        deepkaran.salooja Deepkaran Salooja added a comment -

        Tommie,

        Do we need this?

        Show
        deepkaran.salooja Deepkaran Salooja added a comment - Tommie, Do we need this?
        Hide
        tommie Tommie McAfee added a comment -

        I'd still like it.

        the workaround is to remove lib from the sys.path before import
        https://github.com/couchbase/testrunner/blob/master/pysystests/app/sdk_client_tasks.py#L38

        Show
        tommie Tommie McAfee added a comment - I'd still like it. the workaround is to remove lib from the sys.path before import https://github.com/couchbase/testrunner/blob/master/pysystests/app/sdk_client_tasks.py#L38

          People

          • Assignee:
            deepkaran.salooja Deepkaran Salooja
            Reporter:
            tommie Tommie McAfee
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:

              Gerrit Reviews

              There are no open Gerrit changes