had to create a new PR to merge previous PRs. I added a var to control the chart heading.
https://github.com/XSCE/xsce/pull/145 Tim From: Tim Moody Sent: Saturday, June 28, 2014 6:33 PM To: Martin Dluhos ; xsce-de...@googlegroups.com ; Tim Moody Cc: XS Devel Subject: Re: [XSCE] xovis on the XSCE 5.0 https://github.com/XSCE/xsce/pull/144 has been created. Martin, please review. Tim -----Original Message----- From: Tim Moody Sent: Saturday, June 28, 2014 12:13 PM To: Martin Dluhos ; xsce-de...@googlegroups.com ; Tim Moody Cc: XS Devel Subject: Re: [XSCE] xovis on the XSCE 5.0 In the latest version of xovis Martin has a node js package for creating the couchdb database. While this introduces another package on the server (node js), I think it is preferable to downloading a dummy database from aws, so I plan to follow this approach. Anyone object? Tim -----Original Message----- From: Tim Moody Sent: Saturday, June 28, 2014 11:18 AM To: Martin Dluhos ; xsce-de...@googlegroups.com ; Tim Moody Cc: XS Devel Subject: Re: [XSCE] xovis on the XSCE 5.0 I will create a new PR based on the below. Martin, perhaps you can review it before we merge. Tim -----Original Message----- From: Martin Dluhos Sent: Friday, June 27, 2014 4:55 AM To: xsce-de...@googlegroups.com ; Tim Moody Cc: XS Devel Subject: Re: [XSCE] xovis on the XSCE 5.0 On 06/24/2014 08:35 PM, Tim Moody wrote: > Here's where I'm stuck. There is > https://github.com/martasd/xsce/tree/feature_xovis and there is > https://github.com/martasd/xovis/tree/master (and several other branches). > There is also https://github.com/martasd/xo-stats (with several branches). > > Martin created a PR for the first , but there have been additions to the > second, > as Sameer noted, since the PR and I don’t think they are reflected in the > xsce > branch. I think Martin or someone familiar with the changes needs to > modify the > xsce branch so it can be tested and merged. In fact I think the third is > what > is cloned by the existing PR. Should tasks/main.yml be based on > https://github.com/martasd/xovis/blob/master/scripts/install_xovis.sh? It > clones the xovis repo, not xo-stats. Yes, tasks/main.yml should be based on install_xovis.sh, but it needs to be updated to reflect the changes in that script. Since xo-stats repo was creating a confusion I have integrated that with xovis repo, so xo-stats is now superfluous- I have just added a comment to xo-stats README explaining that xo-stats in now deprecated. I will try to find some time to update the ansible script asap, but can't promise when that will be right now.:) > In terms of the configuration for xovis on xsce, Martin included a > vars/main.yml > to hold values like the name of the deployment (which I think is supposed > to > appear on the charts) and user name and password for admin. Right, based on the documentation I thought that's the ansible way to do it. Cheers, Martin
_______________________________________________ Server-devel mailing list Server-devel@lists.laptop.org http://lists.laptop.org/listinfo/server-devel
