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

Add mobile-service to ns-server

    XMLWordPrintable

Details

    • Task
    • Status: Reopened
    • Major
    • Resolution: Unresolved
    • None
    • backlog
    • ns_server
    • None

    Description

      Server build integration: This task is currently in progress in CBG-75 (gerrit review)

      Binary name: mobile-service

      Api listening ports: see design doc under Architecture / Mobile Service / Api ports

       

      Port Port Number Bound Interface Externally Accessible Auth Description
      Mobile Service REST HTTP :8097 localhost No Cbauth based authentication Expected clients:
      • Javascript code running in the UI
      • CBcollectinfo
      • CBBackup
      Mobile Service REST TLS :18097 Same as Mobile Service REST HTTP
      Mobile Service GRPC TLS :18098 0.0.0.0 (all interfaces) Yes – users would need to update their firewall rules since connections might be coming in from different subnets. Basic Auth + TLS.  Requires “Mobile Admin” role under Roles and Permissions Expected clients:
      • Gateway nodes

       

      Command line arguments: see design doc under Architecture / Mobile Service / Command line arguments

       

      Name Description
      --dataDir The path where the mobile service can store local data
      --uuid The node UUID
      --server The node’s REST host/port (result of misc:node_rest_port(Config, node()))

       

      Stats endpoints: There are stats endpoints spec'd out in the design doc, however, they have not been implemented in the prototype yet. Can we hold off on that until a later phase, or should I implement those in the prototype before this work can start?

       

      Option to run under the delve debugger: Here is the patch I've been using for local prototyping – one thing you'll notice is in the src/ns_ports_setup.erl file there is an option to run under the delve debugger. As a quick hack, I just had it kick in if a special port was specified. Is there a way to put this behind a config setting, environment variable, or command line argument? It's useful for development, since it allows remote debugging of the golang process. Also, ignore the "port 4986", that was never being used.

      Attachments

        Issue Links

          For Gerrit Dashboard: MB-31682
          # Subject Branch Project Status CR V

          Activity

            traun Traun Leyden (Inactive) created issue -
            traun Traun Leyden (Inactive) made changes -
            Field Original Value New Value
            Link This issue blocks CBG-73 [ CBG-73 ]
            traun Traun Leyden (Inactive) made changes -
            Link This issue blocks GOCBC-365 [ GOCBC-365 ]
            traun Traun Leyden (Inactive) made changes -
            Link This issue blocks CBG-76 [ CBG-76 ]

            Build couchbase-server-6.5.0-1636 contains ns_server commit f878ca4 with commit message:
            MB-31682 basic UI integration of mobile into ns_server

            build-team Couchbase Build Team added a comment - Build couchbase-server-6.5.0-1636 contains ns_server commit f878ca4 with commit message: MB-31682 basic UI integration of mobile into ns_server

            Artem Stemkovski I'm closing this based on https://github.com/couchbase/ns_server/commit/f878ca4d1e607c5f9ab4b78308809470039dd043.  Please re-open if you know of any pending work needed.

            traun Traun Leyden (Inactive) added a comment - Artem Stemkovski I'm closing this based on https://github.com/couchbase/ns_server/commit/f878ca4d1e607c5f9ab4b78308809470039dd043 .  Please re-open if you know of any pending work needed.
            traun Traun Leyden (Inactive) made changes -
            Resolution Fixed [ 1 ]
            Status Open [ 1 ] Closed [ 6 ]
            traun Traun Leyden (Inactive) made changes -
            Actual End 2018-11-15 12:52 (issue has been closed)
            traun Traun Leyden (Inactive) made changes -
            Link This issue blocks CBG-73 [ CBG-73 ]
            traun Traun Leyden (Inactive) added a comment - - edited

            I'm getting an "Unknown services - [mobile]" error.  Here is a screenshot:

             

             

            Steps to reproduce:

             

             

             

            traun Traun Leyden (Inactive) added a comment - - edited I'm getting an "Unknown services - [mobile] " error.  Here is a screenshot:     Steps to reproduce: Kick off toy build with MANIFEST_REPO=git://github.com/couchbase/manifest, MANIFEST_FILE=couchbase-server/mad-hatter.xml Install http://server.jenkins.couchbase.com/view/Toys/job/toy-unix/3596/artifact/couchbase-server-enterprise-6.5.0-10014-centos7.x86_64.rpm  on a VirtualBox VM Setup New Cluster Leave all defaults and hit Next      
            traun Traun Leyden (Inactive) made changes -
            Resolution Fixed [ 1 ]
            Status Closed [ 6 ] Reopened [ 4 ]
            traun Traun Leyden (Inactive) made changes -

            This is because the integration is not fully committed, and Mike committed UI commit before the ns_server one.

            artem Artem Stemkovski added a comment - This is because the integration is not fully committed, and Mike committed UI commit before the ns_server one.

            Build couchbase-server-6.5.0-1687 contains ns_server commit f3a7507 with commit message:
            MB-31682 basic integration of mobile into ns_server

            build-team Couchbase Build Team added a comment - Build couchbase-server-6.5.0-1687 contains ns_server commit f3a7507 with commit message: MB-31682 basic integration of mobile into ns_server

            I just did the same test drive on a recent toy build, no errors this time.

            traun Traun Leyden (Inactive) added a comment - I just did the same test drive on a recent toy build, no errors this time.
            traun Traun Leyden (Inactive) made changes -
            Resolution Fixed [ 1 ]
            Status Reopened [ 4 ] Resolved [ 5 ]
            traun Traun Leyden (Inactive) made changes -
            Actual End 2018-11-15 12:52 2018-11-20 17:59 (issue has been resolved)
            traun Traun Leyden (Inactive) made changes -
            Status Resolved [ 5 ] Closed [ 6 ]

            Reopening due to MB-33742

            artem Artem Stemkovski added a comment - Reopening due to MB-33742
            artem Artem Stemkovski made changes -
            Resolution Fixed [ 1 ]
            Status Closed [ 6 ] Reopened [ 4 ]
            artem Artem Stemkovski made changes -
            Fix Version/s Cheshire-Cat [ 15915 ]
            Fix Version/s Mad-Hatter [ 15037 ]

            Build couchbase-server-6.5.0-3038 contains ns_server commit ccacdd0 with commit message:
            MB-33742: Revert "MB-31682 basic integration of mobile into ns_server"

            build-team Couchbase Build Team added a comment - Build couchbase-server-6.5.0-3038 contains ns_server commit ccacdd0 with commit message: MB-33742 : Revert " MB-31682 basic integration of mobile into ns_server"
            meni.hillel Meni Hillel (Inactive) made changes -
            Fix Version/s CheshireCat.Next [ 16908 ]
            Fix Version/s Cheshire-Cat [ 15915 ]
            meni.hillel Meni Hillel (Inactive) made changes -
            Fix Version/s backlog [ 15925 ]
            Fix Version/s CheshireCat.Next [ 16908 ]

            People

              artem Artem Stemkovski
              traun Traun Leyden (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty