You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ch...@apache.org on 2018/04/18 01:47:39 UTC

[1/3] mesos git commit: Marked volume/block creation and destroy operations as experimental.

Repository: mesos
Updated Branches:
  refs/heads/master 5f53ddc99 -> 705bf1dfa


Marked volume/block creation and destroy operations as experimental.

This patch marks the `CREATE_VOLUME`, `DESTROY_VOLUME`, `CREATE_BLOCK`
and `DESTROY_BLOCK` as experimental APIs. It also unifies the way we
mark experimental APIs.

Review: https://reviews.apache.org/r/66616/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/cbb947d5
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/cbb947d5
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/cbb947d5

Branch: refs/heads/master
Commit: cbb947d50adeb1d76768910367cd973712b06583
Parents: 5f53ddc
Author: Chun-Hung Hsiao <ch...@apache.org>
Authored: Tue Apr 17 18:38:38 2018 -0700
Committer: Chun-Hung Hsiao <ch...@mesosphere.io>
Committed: Tue Apr 17 18:38:38 2018 -0700

----------------------------------------------------------------------
 include/mesos/mesos.proto    | 22 +++++++++++++++-------
 include/mesos/v1/mesos.proto | 22 +++++++++++++++-------
 2 files changed, 30 insertions(+), 14 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/cbb947d5/include/mesos/mesos.proto
----------------------------------------------------------------------
diff --git a/include/mesos/mesos.proto b/include/mesos/mesos.proto
index a3ee005..d480ec4 100644
--- a/include/mesos/mesos.proto
+++ b/include/mesos/mesos.proto
@@ -419,7 +419,7 @@ message FrameworkInfo {
  * specifying one of the optional fields. Specifying more than one type
  * is an error.
  *
- * NOTE: This API is unstable and the related feature is experimental.
+ * NOTE: This API is subject to change and the related feature is experimental.
  */
 message CheckInfo {
   enum Type {
@@ -1919,10 +1919,10 @@ message Offer {
       UNRESERVE = 3;
       CREATE = 4;
       DESTROY = 5;
-      CREATE_VOLUME = 7;
-      DESTROY_VOLUME = 8;
-      CREATE_BLOCK = 9;
-      DESTROY_BLOCK = 10;
+      CREATE_VOLUME = 7;  // EXPERIMENTAL.
+      DESTROY_VOLUME = 8; // EXPERIMENTAL.
+      CREATE_BLOCK = 9;   // EXPERIMENTAL.
+      DESTROY_BLOCK = 10; // EXPERIMENTAL.
     }
 
     // TODO(vinod): Deprecate this in favor of `LaunchGroup` below.
@@ -1959,19 +1959,27 @@ message Offer {
       repeated Resource volumes = 1;
     }
 
+    // NOTE: For the time being, this API is subject to change and the related
+    // feature is experimental.
     message CreateVolume {
       required Resource source = 1;
       required Resource.DiskInfo.Source.Type target_type = 2;
     }
 
+    // NOTE: For the time being, this API is subject to change and the related
+    // feature is experimental.
     message DestroyVolume {
       required Resource volume = 1;
     }
 
+    // NOTE: For the time being, this API is subject to change and the related
+    // feature is experimental.
     message CreateBlock {
       required Resource source = 1;
     }
 
+    // NOTE: For the time being, this API is subject to change and the related
+    // feature is experimental.
     message DestroyBlock {
       required Resource block = 1;
     }
@@ -1981,7 +1989,7 @@ message Offer {
     // NOTE: The `id` field will allow frameworks to indicate that they wish to
     // receive feedback about an operation. Since this feature is not yet
     // implemented, the `id` field should NOT be set at present. See MESOS-8054.
-    optional OperationID id = 12; // Experimental.
+    optional OperationID id = 12; // EXPERIMENTAL.
 
     optional Launch launch = 2;
     optional LaunchGroup launch_group = 7;
@@ -2334,7 +2342,7 @@ message OperationStatus {
 * (e.g., the check timed out), these fields must contain empty messages, i.e.,
 * `exit_code` or `status_code` will be unset.
 *
-* NOTE: This API is unstable and the related feature is experimental.
+* NOTE: This API is subject to change and the related feature is experimental.
 */
 message CheckStatusInfo {
   message Command {

http://git-wip-us.apache.org/repos/asf/mesos/blob/cbb947d5/include/mesos/v1/mesos.proto
----------------------------------------------------------------------
diff --git a/include/mesos/v1/mesos.proto b/include/mesos/v1/mesos.proto
index 26fc57e..775c7eb 100644
--- a/include/mesos/v1/mesos.proto
+++ b/include/mesos/v1/mesos.proto
@@ -417,7 +417,7 @@ message FrameworkInfo {
  * specifying one of the optional fields. Specifying more than one type
  * is an error.
  *
- * NOTE: This API is unstable and the related feature is experimental.
+ * NOTE: This API is subject to change and the related feature is experimental.
  */
 message CheckInfo {
   enum Type {
@@ -1911,10 +1911,10 @@ message Offer {
       UNRESERVE = 3;
       CREATE = 4;
       DESTROY = 5;
-      CREATE_VOLUME = 7;
-      DESTROY_VOLUME = 8;
-      CREATE_BLOCK = 9;
-      DESTROY_BLOCK = 10;
+      CREATE_VOLUME = 7;  // EXPERIMENTAL.
+      DESTROY_VOLUME = 8; // EXPERIMENTAL.
+      CREATE_BLOCK = 9;   // EXPERIMENTAL.
+      DESTROY_BLOCK = 10; // EXPERIMENTAL.
     }
 
     // TODO(vinod): Deprecate this in favor of `LaunchGroup` below.
@@ -1951,19 +1951,27 @@ message Offer {
       repeated Resource volumes = 1;
     }
 
+    // NOTE: For the time being, this API is subject to change and the related
+    // feature is experimental.
     message CreateVolume {
       required Resource source = 1;
       required Resource.DiskInfo.Source.Type target_type = 2;
     }
 
+    // NOTE: For the time being, this API is subject to change and the related
+    // feature is experimental.
     message DestroyVolume {
       required Resource volume = 1;
     }
 
+    // NOTE: For the time being, this API is subject to change and the related
+    // feature is experimental.
     message CreateBlock {
       required Resource source = 1;
     }
 
+    // NOTE: For the time being, this API is subject to change and the related
+    // feature is experimental.
     message DestroyBlock {
       required Resource block = 1;
     }
@@ -1973,7 +1981,7 @@ message Offer {
     // NOTE: The `id` field will allow frameworks to indicate that they wish to
     // receive feedback about an operation. Since this feature is not yet
     // implemented, the `id` field should NOT be set at present. See MESOS-8054.
-    optional OperationID id = 12; // Experimental.
+    optional OperationID id = 12; // EXPERIMENTAL.
 
     optional Launch launch = 2;
     optional LaunchGroup launch_group = 7;
@@ -2326,7 +2334,7 @@ message OperationStatus {
 * (e.g., the check timed out), these fields must contain empty messages, i.e.,
 * `exit_code` or `status_code` will be unset.
 *
-* NOTE: This API is unstable and the related feature is experimental.
+* NOTE: This API is subject to change and the related feature is experimental.
 */
 message CheckStatusInfo {
   message Command {


[2/3] mesos git commit: Marked the resource provider API as experimental.

Posted by ch...@apache.org.
Marked the resource provider API as experimental.

This patch marks the resource provider API as experimental, and make the
v0 and v1 protos consistent.

Review: https://reviews.apache.org/r/66648/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/97cc6656
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/97cc6656
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/97cc6656

Branch: refs/heads/master
Commit: 97cc6656279ed0b785226b3f5166e955f399a1e7
Parents: cbb947d
Author: Chun-Hung Hsiao <ch...@apache.org>
Authored: Tue Apr 17 18:38:52 2018 -0700
Committer: Chun-Hung Hsiao <ch...@mesosphere.io>
Committed: Tue Apr 17 18:38:52 2018 -0700

----------------------------------------------------------------------
 include/mesos/resource_provider/resource_provider.proto |  5 +++++
 .../mesos/v1/resource_provider/resource_provider.proto  | 12 ++++++++----
 2 files changed, 13 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/97cc6656/include/mesos/resource_provider/resource_provider.proto
----------------------------------------------------------------------
diff --git a/include/mesos/resource_provider/resource_provider.proto b/include/mesos/resource_provider/resource_provider.proto
index db7c751..7c68333 100644
--- a/include/mesos/resource_provider/resource_provider.proto
+++ b/include/mesos/resource_provider/resource_provider.proto
@@ -23,6 +23,9 @@ package mesos.resource_provider;
 option java_package = "org.apache.mesos.resource_provider";
 option java_outer_classname = "Protos";
 
+
+// NOTE: For the time being, this API is subject to change and the related
+// feature is experimental.
 message Event {
   enum Type {
     // This must be the first enum value in this list, to
@@ -104,6 +107,8 @@ message Event {
 }
 
 
+// NOTE: For the time being, this API is subject to change and the related
+// feature is experimental.
 message Call {
   enum Type {
     // This must be the first enum value in this list, to

http://git-wip-us.apache.org/repos/asf/mesos/blob/97cc6656/include/mesos/v1/resource_provider/resource_provider.proto
----------------------------------------------------------------------
diff --git a/include/mesos/v1/resource_provider/resource_provider.proto b/include/mesos/v1/resource_provider/resource_provider.proto
index 42bc050..535b898 100644
--- a/include/mesos/v1/resource_provider/resource_provider.proto
+++ b/include/mesos/v1/resource_provider/resource_provider.proto
@@ -24,6 +24,8 @@ option java_package = "org.apache.mesos.v1.resource_provider";
 option java_outer_classname = "Protos";
 
 
+// NOTE: For the time being, this API is subject to change and the related
+// feature is experimental.
 message Event {
   enum Type {
     // This must be the first enum value in this list, to
@@ -52,8 +54,8 @@ message Event {
     required Offer.Operation info = 2;
 
     // This is the internal UUID for the operation, which is kept
-    // independently from the framework specified operation id, which is
-    // optional.
+    // independently from the framework-specified operation ID, which
+    // is optional.
     required UUID operation_uuid = 3;
 
     // Used to establish the relationship between the operation and
@@ -105,6 +107,8 @@ message Event {
 }
 
 
+// NOTE: For the time being, this API is subject to change and the related
+// feature is experimental.
 message Call {
   enum Type {
     // This must be the first enum value in this list, to
@@ -131,8 +135,8 @@ message Call {
     optional OperationStatus latest_status = 3;
 
     // This is the internal UUID for the operation, which is kept
-    // independently from the framework specified operation id, which is
-    // optional.
+    // independently from the framework-specified operation ID, which
+    // is optional.
     required UUID operation_uuid = 4;
   }
 


[3/3] mesos git commit: Added MESOS-8787 to the 1.5.1 CHANGELOG.

Posted by ch...@apache.org.
Added MESOS-8787 to the 1.5.1 CHANGELOG.


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/705bf1df
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/705bf1df
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/705bf1df

Branch: refs/heads/master
Commit: 705bf1dfad090c22e58edfe4a53c7ebd6a75fd8f
Parents: 97cc665
Author: Chun-Hung Hsiao <ch...@mesosphere.io>
Authored: Tue Apr 17 18:42:05 2018 -0700
Committer: Chun-Hung Hsiao <ch...@mesosphere.io>
Committed: Tue Apr 17 18:42:05 2018 -0700

----------------------------------------------------------------------
 CHANGELOG | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/705bf1df/CHANGELOG
----------------------------------------------------------------------
diff --git a/CHANGELOG b/CHANGELOG
index 4a2185e..d3809b2 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -38,6 +38,7 @@ Release Notes - Mesos - Version 1.5.1 (WIP)
   * [MESOS-8646] - Agent should be able to resolve file names on open files.
   * [MESOS-8651] - Potential memory leaks in the `volume/sandbox_path` isolator.
   * [MESOS-8742] - Agent resource provider config API calls should be idempotent.
+  * [MESOS-8787] - RP-related API should be experimental.
 
 
 Release Notes - Mesos - Version 1.5.0