search

clearlinux / clear-config-management

Clear Config Management Project
Apache License 2.0
8 stars 4 forks source link

ciao: ciao-webui might not be compatible with the ciao core code in developer mode #41

Open markdryan opened 8 years ago

markdryan commented 8 years ago

When we run the ansible playbook in developer mode we build the ciao middleware sources, i.e., the part written in go, from source. The web ui is downloaded from docker. The docker web-ui image is currently only updated when the corresponding ansible role gets updated and so can be considerably older than the main ciao source. This can lead to compatibility issues.

In my current cluster the web ui is over 2 weeks older than the ciao source and I have issues creating instances.

obedmr commented 8 years ago

@markdryan it's fixed, we're on same version as ciao-webui project. https://hub.docker.com/r/clearlinux/ciao-webui/~/dockerfile/

ciao-webui team will be taking care of keeping this docker image up-to-date.

markdryan commented 8 years ago

@obedmr is it fixed if I do a build from source, i.e., if I have ciao_dev=true? In this case I think the only way to really fix it would be to have the ansible scripts build the webui as well. Is this possible? If not we should mention this in the docs, i.e., say that the UI is not guaranteed to work when you specify ciao_dev=true.

obedmr commented 8 years ago

@markdryan, as far I know, development mode will only compile ciao project not ciao-webui. @albertomurillo could you confirm?. We may do some changes on ciao-webui Docker container for accepting from master's branch. Now, it only accepts ciao-webui official releases , and this is on container build time, not in run time. Do you want from master ciao-webui builds? now, we're in sync with ciao-webui team for updating docker container on every release they generate to update Dockerfile.

What we could do in playbooks is to make sure we always do a docker pull clearlinux/ciao-webuito make sure that your ciao-webui container is up-to-date. @albertomurillo, could we verify we're doing that?

albertomurillo commented 8 years ago

@obedmr thats correct.

So we have two options here. 1) make ansible do docker pull clearlinux/ciao-webui This will ensure you are running the latest released webui but will not allow you to modify ciao-webui code and test it in the same way ciao does.

2) Expand ciao-webui role to read ciao_dev and build the container from the github master branch of ciao-webui.