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 Docker build #97

Merged
merged 2 commits into from
Apr 30, 2016
Merged

Fix Docker build #97

merged 2 commits into from
Apr 30, 2016

Conversation

pchaigno
Copy link
Collaborator

This pull request fixes the Dockerfile, which contains tests for several Linux distributions. It also replaces Ubuntu 14.10 by Ubuntu 16.04.

Our Docker build has been failing for some time now. Unfortunately, this build is not very useful (even though it runs limited tests on more Linux distributions than any other CI) since we do not receive automatic notifications when it fails. It may not be too long to build an IRC notification system from the webhook feature they offer...

@pchaigno pchaigno added the bug label Apr 29, 2016
@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.342% when pulling 4d07f54 on fix-dockerfile into 3838a44 on master.

@awailly
Copy link
Owner

awailly commented Apr 30, 2016

Why the change towards 16.04? Our playbook and CIS were specifically aiming 14.04. This is a good time to introduce multiple versions support. So far, there is no benchmark available for the LTS 16.04 on https://benchmarks.cisecurity.org/downloads/browse/index.cfm?category=benchmarks.os.linux.ubuntu.

The last Ansible version (2.0.2.0-1) requires
new dependencies depending on the Linux distribution
Replace Ubuntu 14.10 with Ubuntu 16.04 in Dockerfile
@pchaigno
Copy link
Collaborator Author

As discussed offline, this keeps 14.04 in the Dockerfile; it just replaces 14.10 with 16.04.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.818% when pulling d19edbd on fix-dockerfile into bee93d0 on master.

@awailly awailly merged commit 5aada5e into master Apr 30, 2016
@awailly awailly deleted the fix-dockerfile branch April 30, 2016 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants