----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25429/#review52858 -----------------------------------------------------------
ui/scripts/system.js <https://reviews.apache.org/r/25429/#comment92019> I need to test this, I think we don't have CCP.* keys in listHosts outputs, perhaps it's from Citrix's commercial distribution? Are you using Apache CloudStack or not? In case this has nothing to do with Apache CloudStack, we cannot accept this for ACS. - Rohit Yadav On Sept. 10, 2014, 12:19 p.m., Mihaela Stoica wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25429/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2014, 12:19 p.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 ed53fd2 > > 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 > >
