marvel-nccr / ansible-role-bigdft

Other
0 stars 1 forks source link

switch to molecule for testing #1

Closed ltalirz closed 5 years ago

ltalirz commented 5 years ago

@adegomme The travis build currently fails

I suggest you still merge this PR and then make a new one to fix the issue(s).

adegomme commented 5 years ago

The problem is that bigdft refuses to be built by the "root" user. For the full image everything is done by the "max" user, so it does work, but molecule uses root by default, and the docker image only has an "ubuntu" user. So I'm not sure how to deal with this properly to have a playbook that works in both situations. Any idea ?

ltalirz commented 5 years ago

Interesting - I've never seen something like this with any of the other codes (and there we are, thinking the root user can do whatever he wants ;-) ).

The problems seems that it isn't creating a necessary directory https://travis-ci.org/marvel-nccr/ansible-role-bigdft/jobs/492851916#L759

I would suggest two things:

Edit: correction, rather try become_user and become: True like here

adegomme commented 5 years ago

Yes, I wasn't clear, I tried locally using the docker image. The folder is not there because the previous step exits prematurely saying it doesn't want to be root. It doesn't fail properly, which is a problem by itself. I will try tomorrow to find a good way to fix the build here, while also not breaking the quantum-mobile build, which uses another user.