Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Travis CI #23

Closed
l0rd opened this issue Aug 14, 2018 · 4 comments
Closed

Fix Travis CI #23

l0rd opened this issue Aug 14, 2018 · 4 comments

Comments

@l0rd
Copy link
Collaborator

l0rd commented Aug 14, 2018

CI is fails as soon as the command ansible-playbook in the APB image is run:

image

@l0rd
Copy link
Collaborator Author

l0rd commented Aug 14, 2018

We should be able to reproduce this problem locally following these instructions https://github.com/ansibleplaybookbundle/eclipse-che-apb#cli-testing

@ibuziuk
Copy link
Contributor

ibuziuk commented Sep 4, 2018

Eugene figured out that in the latest apb-base the way of passing extra-vars has been changed and now should be done via json e.g. -e "{"namespace": "${APB_NAME}"}" . The build should be fixed by his new PR with tls and external keycloak / postgree #24. In the meantime, I have opened an issue for apb-base since the
help is still saying that setting vars via key=value is a valid option ansibleplaybookbundle/apb-base#46

@ghost
Copy link

ghost commented Sep 5, 2018

@ibuziuk this PR #24 will require json format anyway, since this is the only way to pass booleans according to docs

@ghost
Copy link

ghost commented Sep 5, 2018

Fixed CI in #24
I decided to make lint rules not so strict, and move test commands to a dedicated script to avoid yaml formatting issues in .travis.yaml

@l0rd l0rd closed this as completed Sep 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants