----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22842/#review46479 -----------------------------------------------------------
Hm... can we write these unsafe hosts to a file? That'd be very helpful to enable us to come back to it later-on. Right now I'd need to remember to copy-paste the log output (if I'm following right) into another file and re-run src/main/python/apache/aurora/admin/admin_util.py <https://reviews.apache.org/r/22842/#comment81863> maybe..? "a tuple containing: 1) list of output strings 2) set of hostnames included in output" - Joe Smith On June 20, 2014, 2:40 p.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22842/ > ----------------------------------------------------------- > > (Updated June 20, 2014, 2:40 p.m.) > > > Review request for Aurora, David McLaughlin and Joe Smith. > > > Bugs: AURORA-542 > https://issues.apache.org/jira/browse/AURORA-542 > > > Repository: aurora > > > Description > ------- > > Changing maintenance command to drain SLA-compliant hosts in a group. > > > Diffs > ----- > > src/main/python/apache/aurora/admin/admin_util.py > d8517e93c9f2d62703b32c304f4b0c415f6343d7 > src/main/python/apache/aurora/admin/host_maintenance.py > 97d484fc29b8cd0ff1f2c7ae6624d598a0b7f703 > src/main/python/apache/aurora/client/commands/admin.py > 02d055f58e6a9bf906bc65a8cc4899e653c1679f > src/main/python/apache/aurora/client/commands/maintenance.py > e2ac59a60b5ac99cad464b1b4a74082514a1d9e0 > src/test/python/apache/aurora/admin/test_host_maintenance.py > 8abce0e69d391950f7134af68307308743b9b405 > src/test/python/apache/aurora/client/commands/test_admin_sla.py > be380dfbbc2feed9b366305cdbae48929b34d2d3 > src/test/python/apache/aurora/client/commands/test_maintenance.py > 642c235ac820803ef70de409979552d67f47fd53 > src/test/python/apache/aurora/client/commands/util.py > b1822f27d2890469efefc5bfa0878f29163c167a > > Diff: https://reviews.apache.org/r/22842/diff/ > > > Testing > ------- > > ./pants src/test/python:all > > > Thanks, > > Maxim Khutornenko > >