-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12284/#review22889
-----------------------------------------------------------



File Attachment: After commit, slave ID - 
Screen_Shot_2013-07-09_at_12.17.35_AM.png
<https://reviews.apache.org//r/12284/#fcomment4>
    slave.html shows a single slave, not a single host. The final breadcrumb 
should be the slave's ID not the slave's host's name.

- Ross Allen


On July 9, 2013, 7:19 a.m., Ross Allen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12284/
> -----------------------------------------------------------
> 
> (Updated July 9, 2013, 7:19 a.m.)
> 
> 
> Review request for mesos.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Ending crumb was the host, but it should be the ID of the slave.
> 
> 
> Diffs
> -----
> 
>   src/webui/master/static/slave.html 9a52f907271e362d3dac127fc74e8c4a8581e43d 
> 
> Diff: https://reviews.apache.org/r/12284/diff/
> 
> 
> Testing
> -------
> 
> Rendered the "/slaves/:id" route to confirm page renders as expected.
> 
> 
> File Attachments
> ----------------
> 
> Before commit, no ID
>   
> https://reviews.apache.org/media/uploaded/files/2013/07/09/Screen_Shot_2013-07-09_at_12.18.15_AM.png
> After commit, slave ID
>   
> https://reviews.apache.org/media/uploaded/files/2013/07/09/Screen_Shot_2013-07-09_at_12.17.35_AM.png
> 
> 
> Thanks,
> 
> Ross Allen
> 
>

Reply via email to