feat: add start_service API

changes/479.feature

@@ -0,0 +1 @@
+Add `session.start_service` API

src/ai/backend/manager/api/session.py

@@ -28,6 +28,7 @@
     TYPE_CHECKING,
     cast,
 )
+from urllib.parse import urlparse
 import uuid
 
 import aiohttp
@@ -44,6 +45,8 @@
 import sqlalchemy as sa
 from sqlalchemy.sql.expression import true, null
 import trafaret as t
+
+from ai.backend.manager.models.scaling_group import scaling_groups
 if TYPE_CHECKING:
     from sqlalchemy.ext.asyncio import AsyncConnection as SAConnection
 
@@ -76,6 +79,7 @@
 from ai.backend.common.logging import BraceStyleAdapter
 from ai.backend.common.utils import cancel_tasks, str_to_timedelta
 from ai.backend.common.types import (
+    AccessKey,
     AgentId,
     KernelId,
     ClusterMode,
@@ -103,10 +107,12 @@
 from ..models.kernel import match_session_ids
 from ..models.utils import execute_with_retry
 from .exceptions import (
+    AppNotFound,
     InvalidAPIParameters,
     GenericNotFound,
     ImageNotFound,
     InsufficientPrivilege,
+    ServiceUnavailable,
     SessionNotFound,
     SessionAlreadyExists,
     TooManySessionsMatched,
@@ -1048,6 +1054,98 @@ async def create_cluster(request: web.Request, params: Any) -> web.Response:
     return web.json_response(resp, status=201)
 
 
+@server_status_required(READ_ALLOWED)
+@auth_required
+@check_api_params(
+    t.Dict({
+        tx.AliasedKey(['app', 'service']): t.String,
+        # The port argument is only required to use secondary ports
+        # when the target app listens multiple TCP ports.
+        # Otherwise it should be omitted or set to the same value of
+        # the actual port number used by the app.
+        tx.AliasedKey(['port'], default=None): t.Null | t.Int[1024:65535],
+        tx.AliasedKey(['envs'], default=None): t.Null | t.String,  # stringified JSON
+                                                                   # e.g., '{"PASSWORD": "12345"}'
+        tx.AliasedKey(['arguments'], default=None): t.Null | t.String,  # stringified JSON
+                                                                        # e.g., '{"-P": "12345"}'
+                                                                        # The value can be one of:
+                                                                        # None, str, List[str]
+    }))
+async def start_service(request: web.Request, params: Mapping[str, Any]) -> web.Response:
+    root_ctx: RootContext = request.app['_root.context']
+    session_name: str = request.match_info['session_name']
+    access_key: AccessKey = request['keypair']['access_key']
+    service: str = params['app']
+    myself = asyncio.current_task()
+    assert myself is not None
+    try:
+        kernel = await asyncio.shield(root_ctx.registry.get_session(session_name, access_key))
+    except (SessionNotFound, TooManySessionsMatched):
+        raise
+
+    query = (sa.select([scaling_groups.c.wsproxy_address])
+               .select_from(scaling_groups)
+               .where((scaling_groups.c.name == kernel['scaling_group'])))
+
+    async with root_ctx.db.begin() as conn:
+        result = await conn.execute(query)
+        sgroup = result.first()
+    wsproxy_address = sgroup['wsproxy_address']
+    if not wsproxy_address:
+        raise ServiceUnavailable('No coordinator configured for this resource group')
+
+    if kernel['kernel_host'] is None:
+        kernel_host = urlparse(kernel['agent_addr']).hostname
+    else:
+        kernel_host = kernel['kernel_host']
+    for sport in kernel['service_ports']:
+        if sport['name'] == service:
+            if params['port']:
+                # using one of the primary/secondary ports of the app
+                try:
+                    hport_idx = sport['container_ports'].index(params['port'])
+                except ValueError:
+                    raise InvalidAPIParameters(
+                        f"Service {service} does not open the port number {params['port']}.")
+                host_port = sport['host_ports'][hport_idx]
+            else:
+                # using the default (primary) port of the app
+                if 'host_ports' not in sport:
+                    host_port = sport['host_port']  # legacy kernels
+                else:
+                    host_port = sport['host_ports'][0]
+            break
+    else:
+        raise AppNotFound(f'{session_name}:{service}')
+
+    await asyncio.shield(root_ctx.registry.increment_session_usage(session_name, access_key))
+
+    opts: MutableMapping[str, Union[None, str, List[str]]] = {}
+    if params['arguments'] is not None:
+        opts['arguments'] = json.loads(params['arguments'])
+    if params['envs'] is not None:
+        opts['envs'] = json.loads(params['envs'])
+
+    result = await asyncio.shield(
+        root_ctx.registry.start_service(session_name, access_key, service, opts)
+    )
+    if result['status'] == 'failed':
+        raise InternalServerError(
+            "Failed to launch the app service",
+            extra_data=result['error'])
+
+    async with aiohttp.ClientSession() as session:
+        async with session.post(f'{wsproxy_address}/v2/conf', json={
+            'kernel_host': kernel_host,
+            'kernel_port': host_port,
+        }) as resp:
+            token_json = await resp.json()
+            return web.json_response({
+                'token': token_json['token'],
+                'wsproxy_address': wsproxy_address,
+            })
+
+
 async def handle_kernel_creation_lifecycle(
     app: web.Application,
     source: AgentId,
@@ -1992,4 +2090,5 @@ def create_app(default_cors_options: CORSOptions) -> Tuple[web.Application, Iter
     cors.add(app.router.add_route('GET',  '/{session_name}/download', download_files))
     cors.add(app.router.add_route('GET',  '/{session_name}/download_single', download_single))
     cors.add(app.router.add_route('GET',  '/{session_name}/files', list_files))
+    cors.add(app.router.add_route('POST', '/{session_name}/start-service', start_service))
     return app, []

src/ai/backend/manager/models/alembic/versions/60a1effa77d2_add_coordinator_address_column_on_.py

@@ -0,0 +1,24 @@
+"""Add wsproxy_address column on scaling_group
+
+Revision ID: 60a1effa77d2
+Revises: 8679d0a7e22b
+Create Date: 2021-09-17 13:19:57.525513
+
+"""
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision = '60a1effa77d2'
+down_revision = '8679d0a7e22b'
+branch_labels = None
+depends_on = None
+
+
+def upgrade():
+    op.add_column('scaling_groups', sa.Column('wsproxy_address', sa.String(length=1024), nullable=True))
+
+
+def downgrade():
+    op.drop_colu('scaling_groups', 'wsproxy_address')

src/ai/backend/manager/models/scaling_group.py

@@ -59,6 +59,7 @@
     sa.Column('is_active', sa.Boolean, index=True, default=True),
     sa.Column('created_at', sa.DateTime(timezone=True),
               server_default=sa.func.now()),
+    sa.Column('wsproxy_address', sa.String(length=1024), nullable=True),
     sa.Column('driver', sa.String(length=64), nullable=False),
     sa.Column('driver_opts', pgsql.JSONB(), nullable=False, default={}),
     sa.Column('scheduler', sa.String(length=64), nullable=False),
@@ -168,6 +169,7 @@ class ScalingGroup(graphene.ObjectType):
     description = graphene.String()
     is_active = graphene.Boolean()
     created_at = GQLDateTime()
+    wsproxy_address = graphene.String()
     driver = graphene.String()
     driver_opts = graphene.JSONString()
     scheduler = graphene.String()
@@ -186,6 +188,7 @@ def from_row(
             description=row['description'],
             is_active=row['is_active'],
             created_at=row['created_at'],
+            wsproxy_address=row['wsproxy_address'],
             driver=row['driver'],
             driver_opts=row['driver_opts'],
             scheduler=row['scheduler'],
@@ -324,6 +327,7 @@ async def batch_load_by_name(
 class CreateScalingGroupInput(graphene.InputObjectType):
     description = graphene.String(required=False, default='')
     is_active = graphene.Boolean(required=False, default=True)
+    wsproxy_address = graphene.String(required=False)
     driver = graphene.String(required=True)
     driver_opts = graphene.JSONString(required=False, default={})
     scheduler = graphene.String(required=True)
@@ -333,6 +337,7 @@ class CreateScalingGroupInput(graphene.InputObjectType):
 class ModifyScalingGroupInput(graphene.InputObjectType):
     description = graphene.String(required=False)
     is_active = graphene.Boolean(required=False)
+    wsproxy_address = graphene.String(required=False)
     driver = graphene.String(required=False)
     driver_opts = graphene.JSONString(required=False)
     scheduler = graphene.String(required=False)
@@ -363,6 +368,7 @@ async def mutate(
             'name': name,
             'description': props.description,
             'is_active': bool(props.is_active),
+            'wsproxy_address': props.wsproxy_address,
             'driver': props.driver,
             'driver_opts': props.driver_opts,
             'scheduler': props.scheduler,
@@ -399,6 +405,7 @@ async def mutate(
         set_if_set(props, data, 'description')
         set_if_set(props, data, 'is_active')
         set_if_set(props, data, 'driver')
+        set_if_set(props, data, 'wsproxy_address')
         set_if_set(props, data, 'driver_opts')
         set_if_set(props, data, 'scheduler')
         set_if_set(props, data, 'scheduler_opts')