----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47550/#review134045 -----------------------------------------------------------
src/main/python/apache/aurora/client/cli/diff_formatter.py (line 83) <https://reviews.apache.org/r/47550/#comment198687> Mark all non-public methods with underscore prefix. src/main/python/apache/aurora/client/cli/diff_formatter.py (lines 142 - 145) <https://reviews.apache.org/r/47550/#comment198686> This was added in one of the previous releases to ensure backwards compatibility with older scheduler that didn't have the jobUpdateDiff API. You can drop this now. src/test/python/apache/aurora/client/cli/test_supdate.py (line 247) <https://reviews.apache.org/r/47550/#comment198688> Since DiffFormatter is a new component, please move all relevant tests into the new test_diff_formatter.py. It's sufficient to mock out any diff_formatter entry points in diff and update command tests. - Maxim Khutornenko On May 19, 2016, 7:28 p.m., Mehrdad Nurolahzade wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47550/ > ----------------------------------------------------------- > > (Updated May 19, 2016, 7:28 p.m.) > > > Review request for Aurora. > > > Repository: aurora > > > Description > ------- > > AURORA-1492 Improve "aurora update start" command output > > > Diffs > ----- > > src/main/python/apache/aurora/client/cli/diff_formatter.py PRE-CREATION > src/main/python/apache/aurora/client/cli/jobs.py > e8bc38aaff42579419130b116389d9b9e09122a9 > src/main/python/apache/aurora/client/cli/update.py > eb7e9b0799bb25af652a0d5bde231cfca5bc1510 > src/test/python/apache/aurora/client/cli/test_supdate.py > 2135ca988ce2f260262145a2168849acd8d14874 > > Diff: https://reviews.apache.org/r/47550/diff/ > > > Testing > ------- > > Automated: ./pants test.pytest --no-fast src/test/python:: > Manual: using vagrant > e2e: src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh > Style: ./build-support/hooks/pre-commit > > Example output: > > aurora update start devcluster/www-data/prod/hello > ./aurora/examples/jobs/updated_hello_world.aurora > INFO] Starting update for: hello > Job update has started. View your update progress at > http://aurora.local:8081/scheduler/www-data/prod/hello/update/df5ba4d7-6a32-42ca-8453-1b172f61a23b > This job update will: > add instances: [2-3] > update instances: [0-1] > > > Thanks, > > Mehrdad Nurolahzade > >