----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71184/#review216956 -----------------------------------------------------------
src/webui/app/agents/agent.html Line 64 (original), 64 (patched) <https://reviews.apache.org/r/71184/#comment304191> Why "grace time" instead of "grace period"? src/webui/app/controllers.js Lines 725 (patched) <https://reviews.apache.org/r/71184/#comment304190> Is this leftover from debugging? - Greg Mann On July 29, 2019, 9:18 p.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71184/ > ----------------------------------------------------------- > > (Updated July 29, 2019, 9:18 p.m.) > > > Review request for mesos and Greg Mann. > > > Bugs: MESOS-9907 > https://issues.apache.org/jira/browse/MESOS-9907 > > > Repository: mesos > > > Description > ------- > > Added expected drain end time to webui's agent screen. > > > Diffs > ----- > > src/webui/app/agents/agent.html 25d233ba5fd7253e04a7c9cf222f7649bab75baf > src/webui/app/controllers.js 725230f415b66166543c132a4409d2da303cc1e6 > > > Diff: https://reviews.apache.org/r/71184/diff/1/ > > > Testing > ------- > > * `make check` > * patched agent to not discard `drainConfig` and inspected screen for > `drained` agent run via `mesos-local` > > > Thanks, > > Benjamin Bannier > >
