----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69662/#review211802 -----------------------------------------------------------
Fix it, then Ship it! It looks like a great start. Thanks for adding the UI! src/webui/app/controllers.js Lines 697 (patched) <https://reviews.apache.org/r/69662/#comment297396> Instead of setting `agent.resource_providers` with `state.resource_providers` and mutating each item, how about initializing it to `{}` and constructing each item one by one, like what we do for `agent.frameworks`, for consistency? - Chun-Hung Hsiao On Jan. 7, 2019, 10:33 p.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69662/ > ----------------------------------------------------------- > > (Updated Jan. 7, 2019, 10:33 p.m.) > > > Review request for mesos, Armand Grillet, Benjamin Mahler, and Chun-Hung > Hsiao. > > > Bugs: MESOS-8380 > https://issues.apache.org/jira/browse/MESOS-8380 > > > Repository: mesos > > > Description > ------- > > Displayed resource provider information in the Mesos webui. > > > Diffs > ----- > > src/webui/app/agents/agent.html a101a93dcdb95f257fe0ee967c92d2cdc1c84f84 > src/webui/app/controllers.js 8049cf611895edea7c54b3c58d71e00d823a1fd3 > > > Diff: https://reviews.apache.org/r/69662/diff/4/ > > > Testing > ------- > > `make check` > > Ran a local test with a `./src/test-csi-plugin`. > > > File Attachments > ---------------- > > Screenshot Agent screen > > https://reviews.apache.org/media/uploaded/files/2019/01/04/ed920e7b-4072-49be-8801-3b875d529fad__Screen_Shot_2019-01-04_at_11.11.51_AM.png > Screenshot Agent screen > > https://reviews.apache.org/media/uploaded/files/2019/01/07/8f494c0f-1c76-4734-9aae-7fb899589120__Screen_Shot_2019-01-07_at_9.42.51_PM.png > Screenshot Agent screen > > https://reviews.apache.org/media/uploaded/files/2019/01/07/8a850bcd-dd30-4d25-bb37-60cd872ddd62__Screen_Shot_2019-01-07_at_11.27.49_PM.png > > > Thanks, > > Benjamin Bannier > >