> On Sept. 10, 2014, 1:26 p.m., Rohit Yadav wrote: > > ui/scripts/system.js, lines 15939-15947 > > <https://reviews.apache.org/r/25429/diff/2/?file=684568#file684568line15939> > > > > 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.
This change has been developed by Citrix, but I thought that these new values will be added to the acs master. I will check and update the review request accordingly. - Mihaela ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25429/#review52858 ----------------------------------------------------------- On Sept. 10, 2014, 1: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, 1: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 > >