[Server-devel] [XSCE] xovis on the XSCE 5.0

[Tim Moody]

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-devel at 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