> On Jan. 3, 2014, 9:29 p.m., Brian Wickman wrote: > > src/main/python/apache/aurora/client/api/quota_check.py, line 13 > > <https://reviews.apache.org/r/16444/diff/4/?file=414953#file414953line13> > > > > quota=None
Done. > On Jan. 3, 2014, 9:29 p.m., Brian Wickman wrote: > > src/main/python/apache/aurora/client/api/quota_check.py, lines 43-45 > > <https://reviews.apache.org/r/16444/diff/4/?file=414953#file414953line43> > > > > @classmethods at top before __init__ Moved. > On Jan. 3, 2014, 9:29 p.m., Brian Wickman wrote: > > src/main/python/apache/aurora/client/api/updater.py, line 283 > > <https://reviews.apache.org/r/16444/diff/4/?file=414954#file414954line283> > > > > or CapacityRequest() > > Done. - Maxim ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16444/#review31162 ----------------------------------------------------------- On Jan. 3, 2014, 9:21 p.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16444/ > ----------------------------------------------------------- > > (Updated Jan. 3, 2014, 9:21 p.m.) > > > Review request for Aurora, Mark Chu-Carroll and Brian Wickman. > > > Repository: aurora > > > Description > ------- > > Quota checks should be done outside of populateJobConfig(). This is needed to > properly account for the released resources to prevent failures in job > updates approaching their quota limits. > > Part 1: schema and client related changes. > > > Diffs > ----- > > > src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java > c1a11bdb91c5e764864324d26248d1783af8048b > src/main/python/apache/aurora/client/api/BUILD > 9af74e9746500e7b388fc0b7fe7186fca0daefd3 > src/main/python/apache/aurora/client/api/quota_check.py PRE-CREATION > src/main/python/apache/aurora/client/api/updater.py > e364f26005d5194f2c8fea8c267fe2708c59d50b > src/main/python/apache/aurora/client/commands/BUILD > 556d00aab633106a98cbfb4b1ee2d8e755627c04 > src/main/python/apache/aurora/client/commands/core.py > 45204b58ed14fdb83d31afe0fa4598f1b539062b > src/main/thrift/org/apache/aurora/gen/api.thrift > 480b8f472bcfbe547a91c41638250350a0110334 > src/test/java/org/apache/aurora/scheduler/thrift/aop/ForwardingThrift.java > 62fc8045f6a5fda234df73452685bd04e3142aaf > src/test/python/apache/aurora/client/api/BUILD > c5326752c9841a3f3cb8c9cc57f29f675e345788 > src/test/python/apache/aurora/client/api/test_quota_check.py PRE-CREATION > src/test/python/apache/aurora/client/api/test_updater.py > 5843bb4d3698abc5c4a48ee6eaf2be85e9415a06 > src/test/python/apache/aurora/client/commands/test_update.py > ccc3ee9d97de4373b05ad2de8d88939510c2f052 > > Diff: https://reviews.apache.org/r/16444/diff/ > > > Testing > ------- > > ./pants src/test/python/twitter/aurora/client:all > > > Thanks, > > Maxim Khutornenko > >
