Repository: mesos
Updated Branches:
  refs/heads/master 7a98a0139 -> d1c6173ab


Removed NOTE about incomplete implementation of FrameworkInfo.roles.

Now that the support for frameworks with multiple roles has been
implemented, we can remove the note about the `FrameworkInfo.roles`
field having an incomplete implementation.

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


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

Branch: refs/heads/master
Commit: d1c6173abdc12e464d0911eb01e5e2d39599fd80
Parents: 7a98a01
Author: Benjamin Mahler <[email protected]>
Authored: Wed Apr 19 15:28:14 2017 -0700
Committer: Benjamin Mahler <[email protected]>
Committed: Mon Apr 24 14:24:38 2017 -0700

----------------------------------------------------------------------
 include/mesos/mesos.proto    | 5 +----
 include/mesos/v1/mesos.proto | 5 +----
 2 files changed, 2 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/d1c6173a/include/mesos/mesos.proto
----------------------------------------------------------------------
diff --git a/include/mesos/mesos.proto b/include/mesos/mesos.proto
index fbb491c..0087343 100644
--- a/include/mesos/mesos.proto
+++ b/include/mesos/mesos.proto
@@ -262,11 +262,8 @@ message FrameworkInfo {
   // `roles` and the default unset `role`, we require that
   // frameworks set the `MULTI_ROLE` capability if
   // setting the `roles` field.
-  //
-  // NOTE: The implmentation for supporting `roles`
-  // is not complete, DO NOT USE the `roles` field.
   optional string role = 6 [default = "*", deprecated=true];
-  repeated string roles = 12; // EXPERIMENTAL.
+  repeated string roles = 12;
 
   // Used to indicate the current host from which the scheduler is
   // registered in the Mesos Web UI. If set to an empty string Mesos

http://git-wip-us.apache.org/repos/asf/mesos/blob/d1c6173a/include/mesos/v1/mesos.proto
----------------------------------------------------------------------
diff --git a/include/mesos/v1/mesos.proto b/include/mesos/v1/mesos.proto
index 54a9efc..a4426c4 100644
--- a/include/mesos/v1/mesos.proto
+++ b/include/mesos/v1/mesos.proto
@@ -262,11 +262,8 @@ message FrameworkInfo {
   // `roles` and the default unset `role`, we require that
   // frameworks set the `MULTI_ROLE` capability if
   // setting the `roles` field.
-  //
-  // NOTE: The implmentation for supporting `roles`
-  // is not complete, DO NOT USE the `roles` field.
   optional string role = 6 [default = "*", deprecated=true];
-  repeated string roles = 12; // EXPERIMENTAL.
+  repeated string roles = 12;
 
   // Used to indicate the current host from which the scheduler is
   // registered in the Mesos Web UI. If set to an empty string Mesos

Reply via email to