Review Request 26391: Remove special casing of 'rack' attribute.

[Zameer Manji]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26391/
---

Review request for Aurora, Maxim Khutornenko and Bill Farner.


Bugs: AURORA-662
https://issues.apache.org/jira/browse/AURORA-662


Repository: aurora


Description
---

This patch removes the auto injection of the 'rack' attribute in the constraint 
of every job. This will allow for jobs without the 'rack' attribute to be 
scheduled and allow Aurora to schedule jobs on Mesos slaves that don't have the 
'rack' attribute.


Diffs
-

  
src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java
 865742171c11fbe5cf1469a69dd7258ec1be28c2 
  
src/test/java/org/apache/aurora/scheduler/app/local/simulator/ClusterSimulatorModule.java
 9ee4fe2e76d36e17f8de2ab3eb714a6aae52c09c 
  
src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java
 02cd8f712fff3d283abf8e3eb1b4dcab1e762ac2 

Diff: https://reviews.apache.org/r/26391/diff/


Testing
---

./gradlew clean build -Pq


Thanks,

Zameer Manji

Re: Review Request 26391: Remove special casing of 'rack' attribute.

[Maxim Khutornenko]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26391/#review55606
---

Ship it!


Ship It!

- Maxim Khutornenko


On Oct. 6, 2014, 11:21 p.m., Zameer Manji wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/26391/
 ---
 
 (Updated Oct. 6, 2014, 11:21 p.m.)
 
 
 Review request for Aurora, Maxim Khutornenko and Bill Farner.
 
 
 Bugs: AURORA-662
 https://issues.apache.org/jira/browse/AURORA-662
 
 
 Repository: aurora
 
 
 Description
 ---
 
 This patch removes the auto injection of the 'rack' attribute in the 
 constraint of every job. This will allow for jobs without the 'rack' 
 attribute to be scheduled and allow Aurora to schedule jobs on Mesos slaves 
 that don't have the 'rack' attribute.
 
 
 Diffs
 -
 
   
 src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java
  865742171c11fbe5cf1469a69dd7258ec1be28c2 
   
 src/test/java/org/apache/aurora/scheduler/app/local/simulator/ClusterSimulatorModule.java
  9ee4fe2e76d36e17f8de2ab3eb714a6aae52c09c 
   
 src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java
  02cd8f712fff3d283abf8e3eb1b4dcab1e762ac2 
 
 Diff: https://reviews.apache.org/r/26391/diff/
 
 
 Testing
 ---
 
 ./gradlew clean build -Pq
 
 
 Thanks,
 
 Zameer Manji