Re: Review Request 16528: Scheduler home page new using AngularJS and UI client

2014-01-13 Thread Suman Karumuri
://reviews.apache.org/r/16528/#review31389 --- On Jan. 13, 2014, 7:32 p.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 16528: Scheduler home page new using AngularJS and UI client

2014-01-13 Thread Suman Karumuri
https://reviews.apache.org/media/uploaded/files/2013/12/31/03920805-bba2-4160-90fb-f137b79cde20__Screen_Shot_2013-12-31_at_2.28.42_AM.png Thanks, Suman Karumuri

Re: Review Request 16528: Scheduler home page new using AngularJS and UI client

2014-01-13 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16528/#review31390 --- On Jan. 7, 2014, 8:10 a.m., Suman Karumuri wrote

Re: Review Request 16528: Scheduler home page new using AngularJS and UI client

2014-01-13 Thread Suman Karumuri
-f137b79cde20__Screen_Shot_2013-12-31_at_2.28.42_AM.png Thanks, Suman Karumuri

Re: Review Request 16710: Support multiple simulataneous auth mechanisms.

2014-01-14 Thread Suman Karumuri
checks inside the scheduler may be a better way to implement this. PS: I may have understood this entirely wrong, if so, please feel free to ignore this. - Suman Karumuri On Jan. 8, 2014, 12:45 a.m., Kevin Sweeney wrote

Re: Review Request 16528: Scheduler home page new using AngularJS and UI client

2014-01-15 Thread Suman Karumuri
generated e-mail. To reply, visit: https://reviews.apache.org/r/16528/#review31922 --- On Jan. 15, 2014, 7:08 a.m., Suman Karumuri wrote: --- This is an automatically generated e

Re: Review Request 16995: Only make the SchedulerDriver available after registered() has been called.

2014-01-16 Thread Suman Karumuri
/ --- (Updated Jan. 16, 2014, 11 p.m.) Review request for Aurora, Kevin Sweeney and Suman Karumuri. Bugs: AURORA-45 https://issues.apache.org/jira/browse/AURORA-45 Repository: aurora Description --- I've also removed some functions from

Re: Review Request 16528: Scheduler home page new using AngularJS and UI client

2014-01-16 Thread Suman Karumuri
, Suman Karumuri

Re: Review Request 17068: Added docs about git setup.

2014-01-17 Thread Suman Karumuri
, Suman Karumuri

Re: Review Request 17131: Improve test coverage for CronJobManager.

2014-01-21 Thread Suman Karumuri
inline this variable? src/test/java/org/apache/aurora/scheduler/state/CronJobManagerTest.java https://reviews.apache.org/r/17131/#comment61166 Would it be better if these comments are changed to JavaDoc comments? - Suman Karumuri On Jan. 20, 2014, 9:01 p.m., Bill Farner wrote

Re: Review Request 17133: Improve test coverage for TaskAssignerImpl.

2014-01-21 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17133/#review32369 --- Ship it! Looks good! - Suman Karumuri On Jan. 20, 2014, 10:12

Re: Review Request 16995: Only make the SchedulerDriver available after registered() has been called.

2014-01-21 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16995/#review32372 --- Ship it! Ship It! - Suman Karumuri On Jan. 20, 2014, 10:24 p.m

Re: Review Request 17095: Only export counters in TaskVars after SchedulerActive event.

2014-01-21 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17095/#review32373 --- Ship it! Ship It! - Suman Karumuri On Jan. 18, 2014, 9:01 p.m

Re: Review Request 17068: Added docs about git setup.

2014-01-21 Thread Suman Karumuri
--- On Jan. 17, 2014, 11:40 p.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17068/ --- (Updated Jan. 17, 2014

Re: Review Request 17068: Added docs about git setup.

2014-01-21 Thread Suman Karumuri
and Bill Farner. Repository: aurora Description --- Added docs about git setup. Diffs (updated) - docs/contributing.md d337da2be9b182a0c872574c3587d414c566331b Diff: https://reviews.apache.org/r/17068/diff/ Testing --- Thanks, Suman Karumuri

Re: Review Request 17014: Show an error message in the UI when scheduler returns an invalid response.

2014-01-21 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17014/#review32442 --- ping. - Suman Karumuri On Jan. 21, 2014, 9:28 p.m., Suman

Re: Review Request 17131: Improve test coverage for CronJobManager.

2014-01-22 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17131/#review32516 --- Ship it! Ship It! - Suman Karumuri On Jan. 20, 2014, 9:01 p.m

Re: Review Request 17014: Show an error message in the UI when scheduler returns an invalid response.

2014-01-22 Thread Suman Karumuri
16f22a700abb7ba51559e3cfbb04d9218bf4f98a Diff: https://reviews.apache.org/r/17014/diff/ Testing --- gradle clean build. Tested with local scheduler. Scheduler shows an error message (ex: when storage is not ready) Thanks, Suman Karumuri

Re: Review Request 17014: Show an error message in the UI when scheduler returns an invalid response.

2014-01-22 Thread Suman Karumuri
. - Suman --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17014/#review32444 --- On Jan. 22, 2014, 7:16 p.m., Suman Karumuri wrote

Re: Review Request 17205: Moved readonly thrift API calls into ReadOnlyScheduler.

2014-01-23 Thread Suman Karumuri
. - Suman --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17205/#review32638 --- On Jan. 22, 2014, 9:02 p.m., Suman Karumuri wrote

Review Request 17303: Updated getJobs API to return task stats and latest task config

2014-01-23 Thread Suman Karumuri
to test local UI. Thanks, Suman Karumuri

Re: Review Request 17303: Updated getJobs API to return task stats and latest task config

2014-01-24 Thread Suman Karumuri
struct. Agreed. - Suman --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17303/#review32720 --- On Jan. 24, 2014, 7:13 a.m., Suman Karumuri

Review Request 17347: Added non-prod consumption info to getQuota call.

2014-01-24 Thread Suman Karumuri
42fdca2759f15d007bee058485c237268c57597a Diff: https://reviews.apache.org/r/17347/diff/ Testing --- gradle clean build. Thanks, Suman Karumuri

Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

2014-01-27 Thread Suman Karumuri
6cefdfad469a9b69a5291ad46be1df14b443472e Diff: https://reviews.apache.org/r/17430/diff/ Testing --- gradle clean build gradle run - tested in local UI. Thanks, Suman Karumuri

Re: Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

2014-01-28 Thread Suman Karumuri
run - tested in local UI. Thanks, Suman Karumuri

Re: Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

2014-01-28 Thread Suman Karumuri
--- On Jan. 28, 2014, 8:51 p.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17430

Re: Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

2014-01-28 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17430/#review32945 --- On Jan. 28, 2014, 8:51 p.m., Suman Karumuri wrote

Re: Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

2014-01-28 Thread Suman Karumuri
/resources/org/apache/aurora/gen/api.thrift.md5 42fdca2759f15d007bee058485c237268c57597a Diff: https://reviews.apache.org/r/17430/diff/ Testing --- gradle clean build gradle run - tested in local UI. Thanks, Suman Karumuri

Re: Review Request 17430: Bug fix: Return job count instead of task count on GetRoleSummary() call.

2014-01-28 Thread Suman Karumuri
/#review33041 --- On Jan. 28, 2014, 10:17 p.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17430

Review Request 17562: Added cluster name to page title.

2014-01-30 Thread Suman Karumuri
--- gradle clean build. gradle run. Thanks, Suman Karumuri

Re: Review Request 17562: Added cluster name to page title.

2014-01-30 Thread Suman Karumuri
://reviews.apache.org/r/17562/diff/ Testing --- gradle clean build. gradle run. Thanks, Suman Karumuri

Re: Review Request 17562: Added cluster name to page title.

2014-01-30 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17562/#review33266 --- On Jan. 30, 2014, 10:27 p.m., Suman Karumuri wrote

Re: Review Request 17562: Added cluster name to page title.

2014-02-03 Thread Suman Karumuri
On Jan. 31, 2014, 12:56 a.m., Kevin Sweeney wrote: Suman Karumuri wrote: Spoke to Bill before making these changes. We decided to use getInfo API and deprecate getAPIVersion. The getInfo will return with the clusterName and APIVersion for now. In future it may include other

Re: Review Request 17562: Added cluster name to page title.

2014-02-04 Thread Suman Karumuri
e88d29732ee8652ede6c7ee13513122086385646 Diff: https://reviews.apache.org/r/17562/diff/ Testing --- gradle clean build. gradle run. Thanks, Suman Karumuri

Review Request 17715: Added THROTTLED state to scheduler UI.

2014-02-04 Thread Suman Karumuri
/17715/diff/ Testing --- gradle clean build gradle run Thanks, Suman Karumuri

Re: Review Request 17672: Add indices to OfferQueueImpl to improve resource offer handling performance.

2014-02-04 Thread Suman Karumuri
, it would be nice to leave a TODO here. - Suman Karumuri On Feb. 4, 2014, 6:41 p.m., Bill Farner wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17672

Review Request 17730: Now sorting roles before displaying them on home page.

2014-02-04 Thread Suman Karumuri
0abd3e737e901f08d18c3ceb55ea2f94847cc2e1 Diff: https://reviews.apache.org/r/17730/diff/ Testing --- gradle clean build. gradle run with fake summary data. Thanks, Suman Karumuri

Re: Review Request 17730: Now sorting roles before displaying them on home page.

2014-02-06 Thread Suman Karumuri
/#review33719 --- On Feb. 5, 2014, 7:14 a.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17730

Re: Review Request 17730: Now sorting roles before displaying them on home page.

2014-02-06 Thread Suman Karumuri
0abd3e737e901f08d18c3ceb55ea2f94847cc2e1 Diff: https://reviews.apache.org/r/17730/diff/ Testing --- gradle clean build. gradle run with fake summary data. Thanks, Suman Karumuri

Re: Review Request 17823: Add a utility to list missing shipits.

2014-02-06 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17823/#review33910 --- +1 - Suman Karumuri On Feb. 6, 2014, 11:52 p.m., Kevin Sweeney

Review Request 18036: Added LeaderRedirectFilter to scheduler home page.

2014-02-12 Thread Suman Karumuri
, Suman Karumuri

Re: Review Request 18036: Added LeaderRedirectFilter to scheduler home page.

2014-02-13 Thread Suman Karumuri
prefer to have this since it provides visibility into the logic mentioned in the previous comment. - Suman Karumuri On Feb. 12, 2014, 11:26 p.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit

Re: Review Request 18036: Added LeaderRedirectFilter to scheduler home page.

2014-02-13 Thread Suman Karumuri
/scheduler/http/ServletModule.java ba37e26bba4e1ad4c546e73748182deb2d5942a7 Diff: https://reviews.apache.org/r/18036/diff/ Testing --- gradle clean build tested on localhost, verified behavior via log statements. Thanks, Suman Karumuri

Re: Review Request 18036: Added LeaderRedirectFilter to scheduler home page.

2014-02-13 Thread Suman Karumuri
On Feb. 13, 2014, 6:29 p.m., Suman Karumuri wrote: src/main/java/org/apache/aurora/scheduler/http/UIRedirectFilter.java, line 40 https://reviews.apache.org/r/18036/diff/1/?file=483735#file483735line40 It is not logged by the library. I prefer to have this since it provides

Review Request 18300: Updated FakeSchedulerDriver to work with delayed driver registration semantics of SchedulerLifeCycle.

2014-02-19 Thread Suman Karumuri
Diff: https://reviews.apache.org/r/18300/diff/ Testing --- gradle clean build. rm /tmp/testing_log_file gradle run Thanks, Suman Karumuri

Re: Review Request 18297: Clean up idea module configuration, make sure generated sources are available.

2014-02-20 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18297/#review35046 --- Ship it! Ship It! - Suman Karumuri On Feb. 20, 2014, 6:41 p.m

Re: Review Request 17562: Added cluster name to page title.

2014-02-20 Thread Suman Karumuri
/AopModuleTest.java d93ac53ec15863f2ee0e0ebc4737d8f4f8e4be9d src/test/resources/org/apache/aurora/gen/api.thrift.md5 10f5cdab16230e66ac7c9fab199e2dd2564248da Diff: https://reviews.apache.org/r/17562/diff/ Testing --- gradle clean build. gradle run. Thanks, Suman Karumuri

Re: Review Request 17562: Added cluster name to page title.

2014-02-20 Thread Suman Karumuri
--- On Feb. 20, 2014, 9:53 p.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17562

Review Request 18340: Renamed getJobSummary call to getRoleSummary

2014-02-20 Thread Suman Karumuri
clean build gradle run and tested on my laptop Thanks, Suman Karumuri

Re: Review Request 17562: Added cluster name to page title.

2014-02-24 Thread Suman Karumuri
/aurora/gen/api.thrift.md5 fafb5100443482e662db453429c5259f2ab80ae5 Diff: https://reviews.apache.org/r/17562/diff/ Testing --- gradle clean build. gradle run. Thanks, Suman Karumuri

Re: Review Request 17303: Updated getJobs API to return task stats and latest task config

2014-02-25 Thread Suman Karumuri
--- On Jan. 24, 2014, 7:13 a.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17303/ --- (Updated Jan

Re: Review Request 17303: Updated getJobs API to return task stats and latest task config

2014-02-25 Thread Suman Karumuri
generated e-mail. To reply, visit: https://reviews.apache.org/r/17303/#review33522 --- On Jan. 24, 2014, 7:13 a.m., Suman Karumuri wrote: --- This is an automatically generated e-mail

Re: Review Request 17303: Added getJobSummary API

2014-02-25 Thread Suman Karumuri
/ForwardingThrift.java 1edc0d7b224cc477ea6e8873e76ee8c70c6b4d50 src/test/resources/org/apache/aurora/gen/api.thrift.md5 fafb5100443482e662db453429c5259f2ab80ae5 Diff: https://reviews.apache.org/r/17303/diff/ Testing --- gradle clean build gradle run to test local UI. Thanks, Suman Karumuri

Re: Review Request 17303: Added getJobSummary API

2014-02-25 Thread Suman Karumuri
fafb5100443482e662db453429c5259f2ab80ae5 Diff: https://reviews.apache.org/r/17303/diff/ Testing --- gradle clean build gradle run to test local UI. Thanks, Suman Karumuri

Review Request 18546: Renamed Quota to ResourceAggregate.

2014-02-26 Thread Suman Karumuri
end points. Will run end to end test tonight. Thanks, Suman Karumuri

Re: Review Request 17347: Added non-prod consumption info to getQuota call.

2014-02-27 Thread Suman Karumuri
this something that doesn't already have an implied meaning about production=True. Mind taking a detour to address this TODO? grep -R ResourceAggregate src/main/java/ src/main/java//org/apache/aurora/scheduler/quota/Quotas.java: // TODO(Suman Karumuri): Rename Quota

Re: Review Request 18546: Renamed Quota to ResourceAggregate.

2014-02-27 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18546/#review35590 --- On Feb. 27, 2014, 2:44 a.m., Suman Karumuri wrote

Review Request 18598: Updated test to use environment variable if it exits.

2014-02-27 Thread Suman Karumuri
/test_diff.py 339459e2f563083ddbf01cadd4e938970d6ea388 Diff: https://reviews.apache.org/r/18598/diff/ Testing --- ./build-support/jenkins/build.sh runs successfully Thanks, Suman Karumuri

Re: Review Request 18546: Renamed Quota to ResourceAggregate.

2014-02-27 Thread Suman Karumuri
--- On Feb. 27, 2014, 2:44 a.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18546

Re: Review Request 18546: Renamed Quota to ResourceAggregate.

2014-02-27 Thread Suman Karumuri
://reviews.apache.org/r/18546/diff/ Testing --- Successfully ran: gradle clean build ./pants src/test/python/apache/aurora/client:all On a local scheduler tested /scheduler* and /quotas end points. Will run end to end test tonight. Thanks, Suman Karumuri

Re: Review Request 18546: Renamed Quota to ResourceAggregate.

2014-02-28 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18546/#review35751 --- On Feb. 28, 2014, 1:32 a.m., Suman Karumuri wrote

Re: Review Request 18546: Renamed Quota to ResourceAggregate.

2014-02-28 Thread Suman Karumuri
://reviews.apache.org/r/18546/diff/ Testing --- Successfully ran: gradle clean build ./pants src/test/python/apache/aurora/client:all On a local scheduler tested /scheduler* and /quotas end points. Will run end to end test tonight. Thanks, Suman Karumuri

Re: Review Request 17347: Added non-prod consumption info to getQuota call.

2014-02-28 Thread Suman Karumuri
/SchedulerThriftInterfaceTest.java dc557718269064a202c3e4eb1272ff2b9f209ad9 src/test/resources/org/apache/aurora/gen/api.thrift.md5 4e6c51d9298bf6fc1935ec9080f38726f79e7959 Diff: https://reviews.apache.org/r/17347/diff/ Testing (updated) --- ./gradlew clean build. Thanks, Suman Karumuri

Re: Review Request 17303: Added getJobSummary API

2014-02-28 Thread Suman Karumuri
run to test local UI. Thanks, Suman Karumuri

Re: Review Request 17303: Added getJobSummary API

2014-02-28 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17303/#review35831 --- On March 1, 2014, 1:26 a.m., Suman Karumuri wrote

Re: Review Request 17347: Added non-prod consumption info to getQuota call.

2014-02-28 Thread Suman Karumuri
, 2014, 11:32 p.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17347/ --- (Updated Feb. 28, 2014, 11:32 p.m

Re: Review Request 17347: Added non-prod consumption info to getQuota call.

2014-02-28 Thread Suman Karumuri
dc557718269064a202c3e4eb1272ff2b9f209ad9 src/test/resources/org/apache/aurora/gen/api.thrift.md5 4e6c51d9298bf6fc1935ec9080f38726f79e7959 Diff: https://reviews.apache.org/r/17347/diff/ Testing --- ./gradlew clean build. Thanks, Suman Karumuri

Re: Review Request 17347: Added non-prod consumption info to getQuota call.

2014-02-28 Thread Suman Karumuri
src/test/resources/org/apache/aurora/gen/api.thrift.md5 08c4c2a4717a1c827f4cef6405d78e7db5046a00 Diff: https://reviews.apache.org/r/17347/diff/ Testing --- ./gradlew clean build. Thanks, Suman Karumuri

Re: Review Request 17347: Added non-prod consumption info to getQuota call.

2014-02-28 Thread Suman Karumuri
src/test/python/apache/aurora/client/api/test_quota_check.py 4dff351ef031d9cdfadbcea5666f4e995a89cce2 Diff: https://reviews.apache.org/r/17347/diff/ Testing --- ./gradlew clean build. Thanks, Suman Karumuri

Re: Review Request 18484: Count on task timeouts and task pruning to be idempotent, simplifying handling code.

2014-03-06 Thread Suman Karumuri
/HistoryPrunerTest.java https://reviews.apache.org/r/18484/#comment67332 This function can be replaced by Tasks.SCHEDULED_TO_JOB_KEY - Suman Karumuri On Feb. 25, 2014, 10:25 p.m., Bill Farner wrote: --- This is an automatically generated e

Re: Review Request 18714: Disable log and initiate shutdown upon log storage failures.

2014-03-06 Thread Suman Karumuri
, 2014, 11:35 p.m.) Review request for Aurora, Kevin Sweeney and Suman Karumuri. Bugs: AURORA-221 https://issues.apache.org/jira/browse/AURORA-221 Repository: aurora Description --- Disable log and initiate shutdown upon log storage failures. Diffs - src

Re: Review Request 18658: GZIP HTTP thrift API responses

2014-03-06 Thread Suman Karumuri
/ --- (Updated March 1, 2014, 1:27 a.m.) Review request for Aurora, Suman Karumuri and Bill Farner. Repository: aurora Description --- GZIP HTTP thrift API responses Diffs - src/main/java/org/apache/aurora/scheduler/thrift/ThriftModule.java

Re: Review Request 18713: Initiate a teardown of scheduler lifecycle upon application exit.

2014-03-06 Thread Suman Karumuri
/ --- (Updated March 3, 2014, 11:31 p.m.) Review request for Aurora, Suman Karumuri and Maxim Khutornenko. Repository: aurora Description --- AFAICT there currently isn't wiring for this, so application exit not initiated by scheduler lifecycle

Re: Review Request 17562: Added cluster name to page title.

2014-03-06 Thread Suman Karumuri
. 25, 2014, 1:22 a.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17562/ --- (Updated Feb. 25, 2014, 1:22

Re: Review Request 17562: Added cluster name to page title.

2014-03-06 Thread Suman Karumuri
/org/apache/aurora/gen/api.thrift.md5 5cfa330f54c0297c84b8f41ee550a7bc6f163010 Diff: https://reviews.apache.org/r/17562/diff/ Testing --- gradle clean build. gradle run. Thanks, Suman Karumuri

Re: Review Request 17303: Added getJobSummary API

2014-03-09 Thread Suman Karumuri
. - Suman --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17303/#review36023 --- On March 1, 2014, 1:26 a.m., Suman Karumuri wrote

Re: Review Request 17303: Added getJobSummary API

2014-03-09 Thread Suman Karumuri
build gradle run to test local UI. Thanks, Suman Karumuri

Re: Review Request 17303: Added getJobSummary API

2014-03-11 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17303/#review36735 --- On March 9, 2014, 10:17 a.m., Suman Karumuri wrote

Re: Review Request 17303: Added getJobSummary API

2014-03-11 Thread Suman Karumuri
build gradle run to test local UI. Thanks, Suman Karumuri

Re: Review Request 18935: Part 1 of converting from Packages to Metadata.

2014-03-11 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18935/#review36817 --- Ship it! Ship It! - Suman Karumuri On March 11, 2014, 12:09 a.m

Re: Review Request 18713: Initiate a teardown of scheduler lifecycle upon application exit.

2014-03-11 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18713/#review36890 --- Ship it! Ship It! - Suman Karumuri On March 12, 2014, 1:26 a.m

Review Request 19323: [BugFix] Added jobStats on old role page. Disabled job status badge.

2014-03-17 Thread Suman Karumuri
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

Re: Review Request 19323: Added jobStats on old role page. Disabled job status badge.

2014-03-17 Thread Suman Karumuri
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

Re: Review Request 19323: Added jobStats on old role page. Disabled job status badge.

2014-03-17 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19323/#review37493 --- On March 17, 2014, 11:57 p.m., Suman Karumuri wrote: --- This is an automatically

Re: Review Request 19323: Added jobStats on old role page. Disabled job status badge.

2014-03-19 Thread Suman Karumuri
://reviews.apache.org/r/19323/#review37592 --- On March 17, 2014, 11:57 p.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 19323: Added jobStats on old role page. Disabled job status badge.

2014-03-19 Thread Suman Karumuri
/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

Review Request 19450: Added nextCronRunMs field to JobSummary.

2014-03-20 Thread Suman Karumuri
--- ./gradlew clean build run on laptop. Thanks, Suman Karumuri

Re: Review Request 18714: Disable log and initiate shutdown upon log storage failures.

2014-03-20 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18714/#review37939 --- Ship it! Ship It! - Suman Karumuri On March 3, 2014, 11:35 p.m

Re: Review Request 19450: Added nextCronRunMs field to JobSummary.

2014-03-20 Thread Suman Karumuri
run on laptop. Thanks, Suman Karumuri

Re: Review Request 19450: Added nextCronRunMs field to JobSummary.

2014-03-20 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19450/#review37925 --- On March 20, 2014, 10:22 p.m., Suman Karumuri wrote

Re: Review Request 19450: Added nextCronRunMs field to JobSummary.

2014-03-20 Thread Suman Karumuri
generated e-mail. To reply, visit: https://reviews.apache.org/r/19450/#review37929 --- On March 20, 2014, 10:22 p.m., Suman Karumuri wrote: --- This is an automatically generated e

Review Request 19509: Now consuming nonProd consumption from the new getQuota API

2014-03-20 Thread Suman Karumuri
-246_getQuotaAPI) ~/workspace/incubator-aurora Thanks, Suman Karumuri

Re: Review Request 19509: Now consuming nonProd consumption from the new getQuota API

2014-03-20 Thread Suman Karumuri
:response.result.getQuotaResult.nonProdConsumption.ramMb = 1024 59:response.result.getQuotaResult.nonProdConsumption.diskMb = 2048 ? git:(mansu/AURORA-246_getQuotaAPI) ~/workspace/incubator-aurora Thanks, Suman Karumuri

Re: Review Request 19450: Added nextCronRunMs field to JobSummary.

2014-03-20 Thread Suman Karumuri
/ThriftIT.java e78ee7b8051bc080dbb3ed2daf72171bffe3915a src/test/resources/org/apache/aurora/gen/api.thrift.md5 456634c76b9993119af2d841073c31a4a1c82ab7 Diff: https://reviews.apache.org/r/19450/diff/ Testing --- ./gradlew clean build run on laptop. Thanks, Suman Karumuri

Re: Review Request 19509: Now consuming nonProd consumption from the new getQuota API

2014-03-21 Thread Suman Karumuri
--- On March 21, 2014, 2:39 a.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19509

Re: Review Request 19509: Now consuming nonProd consumption from the new getQuota API

2014-03-21 Thread Suman Karumuri
--- On March 21, 2014, 2:39 a.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19509

Re: Review Request 19509: Now consuming nonProd consumption from the new getQuota API

2014-03-21 Thread Suman Karumuri
) ~/workspace/incubator-aurora Thanks, Suman Karumuri

Review Request 19565: Role and Role/env pages for Scheduler

2014-03-22 Thread Suman Karumuri
/19565/diff/ Testing --- ./gradlew clean build run on local laptop. Thanks, Suman Karumuri

Re: Review Request 19509: Now consuming nonProd consumption from the new getQuota API

2014-03-24 Thread Suman Karumuri
--- On March 21, 2014, 10:14 p.m., Suman Karumuri wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19509

Re: Review Request 19509: Now consuming nonProd consumption from the new getQuota API

2014-03-24 Thread Suman Karumuri
) ~/workspace/incubator-aurora Thanks, Suman Karumuri

  1   2   3   >