----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39366/#review103021 -----------------------------------------------------------
src/main/python/apache/aurora/client/cli/jobs.py (line 281) <https://reviews.apache.org/r/39366/#comment160870> How does this play with an N-1 version scheduler that doesn't support `get_job_update_diff`? src/main/python/apache/aurora/client/cli/jobs.py (line 287) <https://reviews.apache.org/r/39366/#comment160868> From the output you pasted in the review, shouldn't this read "This job update will:"? i.e. the wording sounds like you might not actually be doing it, or i have to do something else to actually do it. Make it clear that it _is_ happening. src/main/python/apache/aurora/client/cli/jobs.py (line 288) <https://reviews.apache.org/r/39366/#comment160869> Please omit lines for fields that are blank to avoid confusion. - Bill Farner On Oct. 16, 2015, 11:03 a.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39366/ > ----------------------------------------------------------- > > (Updated Oct. 16, 2015, 11:03 a.m.) > > > Review request for Aurora, Joshua Cohen and Bill Farner. > > > Bugs: AURORA-1516 > https://issues.apache.org/jira/browse/AURORA-1516 > > > Repository: aurora > > > Description > ------- > > "aurora job diff" now prints update sequence along with diff details when > applicable (updated instances). > > > Diffs > ----- > > src/main/python/apache/aurora/client/api/__init__.py > 4b9c48e84bc203fc7b28d7efd0b2e6b8a6f18302 > src/main/python/apache/aurora/client/cli/jobs.py > 6d15f1e8558f9df5f00994d20949f19b0de1ad32 > src/test/python/apache/aurora/client/api/test_api.py > b56e35265b1e32d61a13fe72734cae824737278d > src/test/python/apache/aurora/client/cli/test_diff.py > 753a0417caf29249cbc2fca5a9fc41b1ce535c92 > src/test/python/apache/aurora/client/cli/util.py > b03148b4edaa334620a72bee1937c8c16e19f23e > > Diff: https://reviews.apache.org/r/39366/diff/ > > > Testing > ------- > > ./pants test.pytest --no-fast src/test/python:: > > Also tested manually in vagrant: > > Upscale and update: > ``` > $ aurora job diff devcluster/www-data/prod/hello > aurora/examples/jobs/hello_world.aurora > INFO] Starting update for: hello > A job update with this config will: > ----> add instances: [100-119] > ----> update instances: [0-9], [21-29], [41-44], [61-99] > with diff: > 64c64 > < "cpu": 2.0, > --- > > "cpu": 3.0, > 75c75 > < 'numCpus': 2.0, > --- > > 'numCpus': 3.0, > ----> update instances: [30-40], [45-60] > with diff: > 64c64 > < "cpu": 4.0, > --- > > "cpu": 3.0, > 75c75 > < 'numCpus': 4.0, > --- > > 'numCpus': 3.0, > ----> not change instances: [10-20] > ``` > > Diff with instance spec: > ``` > $ aurora job diff devcluster/www-data/prod/hello/0-10 > aurora/examples/jobs/hello_world.aurora > INFO] Starting update for: hello > A job update with this config will: > ----> remove instances: [10] > ----> update instances: [0-9] > with diff: > 64c64 > < "cpu": 2.0, > --- > > "cpu": 3.0, > 75c75 > < 'numCpus': 2.0, > --- > > 'numCpus': 3.0, > ----> not change instances: [11-20], [21-29], [30-40], [41-44], [45-60], > [61-99] > ``` > > Downscale and update: > ``` > $ aurora job diff devcluster/www-data/prod/hello > aurora/examples/jobs/hello_world.aurora > INFO] Starting update for: hello > A job update with this config will: > ----> remove instances: [10-20], [21-29], [30-40], [41-44], [45-60], [61-99] > ----> update instances: [0-9] > with diff: > 64c64 > < "cpu": 2.0, > --- > > "cpu": 3.0, > 75c75 > < 'numCpus': 2.0, > --- > > 'numCpus': 3.0, > ``` > > Update non-existent job: > ``` > $ aurora job diff devcluster/www-data/prod/hello1 > aurora/examples/jobs/hello_world.aurora > INFO] Starting update for: hello1 > A job update with this config will: > ----> add instances: [0-9] > ``` > > > Thanks, > > Maxim Khutornenko > >
