[asterisk-scf-commits] asterisk-scf/release/slice.git branch "master" updated.

[Commits to the Asterisk SCF project code repositories]

branch "master" has been updated
       via  89d5514fadac44681f7a38a627aa2728c7bb16c1 (commit)
      from  1940c4d44f63e8818702868ba2144c3867ac3e3c (commit)

Summary of changes:
 .../Core/Discovery/ServiceLocatorIf.ice            |   46 ++++++++++++++++++-
 slice/AsteriskSCF/Media/RTP/MediaRTPIf.ice         |    5 ++
 .../SessionCommunicationsIf.ice                    |    9 +++-
 .../System/Component/ComponentServiceIf.ice        |    2 +-
 4 files changed, 56 insertions(+), 6 deletions(-)


- Log -----------------------------------------------------------------
commit 89d5514fadac44681f7a38a627aa2728c7bb16c1
Author: Ken Hunt <ken.hunt at digium.com>
Date:   Mon Aug 8 03:12:56 2011 -0500

    Changes for base Component class (including expanded ServiceLocatorParams)

diff --git a/slice/AsteriskSCF/Core/Discovery/ServiceLocatorIf.ice b/slice/AsteriskSCF/Core/Discovery/ServiceLocatorIf.ice
index 44fab7f..e8d5f8a 100644
--- a/slice/AsteriskSCF/Core/Discovery/ServiceLocatorIf.ice
+++ b/slice/AsteriskSCF/Core/Discovery/ServiceLocatorIf.ice
@@ -52,13 +52,53 @@ module V1
 
    /**
     * Generic service locator parameters class that more specific parameter classes can extend.
+    * These parameters are used to register a service with the ServiceLocator, and to 
+    * locate (i.e. query) the service. 
+    * 
+    * Note: The identity of the service is the unique combination of the service field and the
+    * id field, and the id field may be left empty. A service can be part of multiple categories. 
     */
    unsliceable class ServiceLocatorParams
    {
+      /** 
+       * Category for the service.
+       *
+       * When registering a service: 
+       *  - An optional category that the service
+       *    should be considered a member of. The service
+       *    can be added to multiple categories via multiple
+       *    calls to addLocatorParams. 
+       * @see ServiceManagement
+       *
+       * When locating:
+       * - An optional category to filter the results on.
+       */
+      string category; 
+
+      /**
+       * Identifier for the service. 
+       *
+       * When registering a service: 
+       *  - Identifier for the service. Can't be empty. 
+       *
+       * When locating:
+       * - If left empty, locate will return all services within a specified category.
+       * - If category and service are both empty, all services registered with the given
+       *   ServiceLocator are returned. In this case, the id field is ignored. 
+       */
+      string service;
+
       /**
-       * Basic category for the service, such as bridge or channel service.
+       * Instance id within a replica group. 
+       *
+       * When registering a service:
+       *  - Identifier of a specific service instance. 
+       * 
+       * When locating: 
+       *  - Identifier of a specific instance. If left empty the default 
+       *    comparator will match on service (and category, if set) only. 
        */
-      string category;
+       string id;
    };
 
    /**
@@ -67,7 +107,7 @@ module V1
    interface ServiceLocator
    {
       /**
-       * Method which performs a locator request using provided parameters but only returns a single
+       * Method which performs a locator " using provided parameters but only returns a single
        * proxy.
        *
        * @param params A concrete class containing parameters describing the service that is trying to be found.
diff --git a/slice/AsteriskSCF/Media/RTP/MediaRTPIf.ice b/slice/AsteriskSCF/Media/RTP/MediaRTPIf.ice
index 535a729..5794b31 100644
--- a/slice/AsteriskSCF/Media/RTP/MediaRTPIf.ice
+++ b/slice/AsteriskSCF/Media/RTP/MediaRTPIf.ice
@@ -30,6 +30,11 @@ module RTP
 ["suppress"]
 module V1
 {
+   /**
+    * String name of the service locator category for the RTP component service interface.
+    */
+   const string ComponentServiceDiscoveryCategory = "MediaRTPService";
+
     /**
      * String representation of the version of this interface
      */
diff --git a/slice/AsteriskSCF/SessionCommunications/SessionCommunicationsIf.ice b/slice/AsteriskSCF/SessionCommunications/SessionCommunicationsIf.ice
index 1f16f64..615a6ce 100644
--- a/slice/AsteriskSCF/SessionCommunications/SessionCommunicationsIf.ice
+++ b/slice/AsteriskSCF/SessionCommunications/SessionCommunicationsIf.ice
@@ -758,9 +758,14 @@ module V1
     };
 
     /**
-     * Category used for the bridge service.
+     * Category used to publish the Bridge Service's ComponentService interface.
      */
-    const string BridgeServiceDiscoveryCategory = "BridgeService";
+    const string BridgeComponentServiceCategory = "BridgingComponentService";
+
+    /**
+     * Category used for the bridge manager.
+     */
+    const string BridgeManagerDiscoveryCategory = "BridgeManager";
 
     /**
      * The bridge throws an InvalidSessions exception if any of the sessions
diff --git a/slice/AsteriskSCF/System/Component/ComponentServiceIf.ice b/slice/AsteriskSCF/System/Component/ComponentServiceIf.ice
index b6db122..e3fe84f 100644
--- a/slice/AsteriskSCF/System/Component/ComponentServiceIf.ice
+++ b/slice/AsteriskSCF/System/Component/ComponentServiceIf.ice
@@ -83,7 +83,7 @@ module V1
     /** 
      * A base class to enable arbitary parameter passing to the ComponentTest inteface.
      */
-    class ComponentTestParam {};
+    unsliceable class ComponentTestParam {};
     sequence<ComponentTestParam> ComponentTestParamSeq;
 
     const string ComponentTestFacet = "TestFacet";

-----------------------------------------------------------------------


-- 
asterisk-scf/release/slice.git