aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Suman Karumuri" <ma...@apache.org>
Subject Re: Review Request 19323: Added jobStats on old role page. Disabled job status badge.
Date Wed, 19 Mar 2014 18:30:52 GMT

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

(Updated March 19, 2014, 6:30 p.m.)


Review request for Aurora, Kevin Sweeney and Bill Farner.


Changes
-------

Updated code per code review comments.


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


Repository: aurora


Description
-------

Added jobStats on role page. 

We have stopped counting recently failed task count which is used to display the health badge.
Since this badge is being removed from the new UI, removed it from the old UI as well.


Diffs (updated)
-----

  build.gradle f38888beb00f69f40e81c1be80f5f6ade84720ae 
  src/main/java/org/apache/aurora/scheduler/base/Jobs.java 3f027032a785810bddfb3ff4dd4e472989c9493f

  src/main/java/org/apache/aurora/scheduler/http/SchedulerzRole.java b3f3124a73b56e48339f89dbd3b1067527757d04

  src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java ee81b494b9b4dff9839a9e32c9b81fa8ef898e1b

  src/main/python/apache/aurora/tools/java/thrift_wrapper_codegen.py 3143c6afe8adeb18fac44e92f94b9860753c3eb0

  src/main/resources/org/apache/aurora/scheduler/http/schedulerzrole.st 39c1d9527bff88ff191233629babfd6e6035d4e7

  src/test/java/org/apache/aurora/scheduler/base/JobsTest.java 6524e2441d25de6573190866a8559be620961330


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


Testing
-------

./gradlew clean build
gradlew run


File Attachments
----------------

job stats
  https://reviews.apache.org/media/uploaded/files/2014/03/17/65da4a77-4c82-46f8-93d1-34fd319ef60e__Screen_Shot_2014-03-17_at_3.13.36_PM.png


Thanks,

Suman Karumuri


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message