You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@aurora.apache.org by se...@apache.org on 2017/02/01 07:40:10 UTC

aurora git commit: Suppress role deprecation warning as replacement is not yet ready.

Repository: aurora
Updated Branches:
  refs/heads/master efffd85f6 -> fb7ab0938


Suppress role deprecation warning as replacement is not yet ready.

The role field was prematurely deprecated in the Mesos project.
https://github.com/apache/mesos/blob/master/include/mesos/mesos.proto#L257

Reviewed at https://reviews.apache.org/r/56131/


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

Branch: refs/heads/master
Commit: fb7ab093827830936a00450f46bba16df60eb670
Parents: efffd85
Author: David McLaughlin <da...@dmclaughlin.com>
Authored: Wed Feb 1 08:38:54 2017 +0100
Committer: Stephan Erb <se...@apache.org>
Committed: Wed Feb 1 08:38:54 2017 +0100

----------------------------------------------------------------------
 .../aurora/scheduler/mesos/CommandLineDriverSettingsModule.java  | 4 ++++
 1 file changed, 4 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/aurora/blob/fb7ab093/src/main/java/org/apache/aurora/scheduler/mesos/CommandLineDriverSettingsModule.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/aurora/scheduler/mesos/CommandLineDriverSettingsModule.java b/src/main/java/org/apache/aurora/scheduler/mesos/CommandLineDriverSettingsModule.java
index 4987768..36c69be 100644
--- a/src/main/java/org/apache/aurora/scheduler/mesos/CommandLineDriverSettingsModule.java
+++ b/src/main/java/org/apache/aurora/scheduler/mesos/CommandLineDriverSettingsModule.java
@@ -152,6 +152,10 @@ public class CommandLineDriverSettingsModule extends AbstractModule {
   }
 
   @VisibleForTesting
+  // See: https://github.com/apache/mesos/commit/d06d05c76eca13745ca73039b93ad684b9d07196
+  // The role field has been deprecated but the replacement is not ready. We'll also have to
+  // turn on MULTI_ROLES capability before we can use the roles field.
+  @SuppressWarnings("deprecation")
   static FrameworkInfo buildFrameworkInfo(
       String frameworkName,
       String executorUser,