----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25429/#review52850 -----------------------------------------------------------
ui/scripts/system.js <https://reviews.apache.org/r/25429/#comment92015> This probably does not apply for ACS? Is this for CCP? Can you fix/test with ACS master? - Rohit Yadav On Sept. 10, 2014, 10:52 a.m., Mihaela Stoica wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25429/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2014, 10:52 a.m.) > > > Review request for cloudstack, Brian Federle and Jessica Wang. > > > Bugs: CLOUDSTACK-7502 > https://issues.apache.org/jira/browse/CLOUDSTACK-7502 > > > Repository: cloudstack-git > > > Description > ------- > > - Added Agent version and Qemu version fields to the Host detail page, with > the values from the host's details field > - Only displayed for KVM hosts, as the details field is not defined for other > hypervisors > > > Diffs > ----- > > client/WEB-INF/classes/resources/messages.properties 4655973 > ui/dictionary.jsp 0102144 > ui/scripts/system.js cf3b9dc > > Diff: https://reviews.apache.org/r/25429/diff/ > > > Testing > ------- > > Tested with a KVM host: the two fields are displayed. > Tested with a XenServer host: fields not shown. > > Screenshots attached to the ticket. > > > Thanks, > > Mihaela Stoica > >