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

Docker image updates for integration tests. #26054

Merged
merged 7 commits into from
Jun 23, 2017

Conversation

mattclay
Copy link
Member

SUMMARY

Docker image updates for integration tests:

  • Sort packages to install.
  • Add python-argparse to centos6 docker image.
  • Add gcc and python dev lib to docker images.
  • Add python cryptography to docker images.
  • Add coverage using pip instead of OS packages.
  • Update old pip versions in docker images.

Also exclude */pyshared/* from coverage reporting now that coverage has been updated.

ISSUE TYPE

Feature Pull Request

COMPONENT NAME

docker images

ANSIBLE VERSION
ansible 2.4.0 (docker-update 6c555ba263) last updated 2017/06/23 12:02:42 (GMT -700)
  config file = None
  configured module search path = [u'/Users/mclay/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /Users/mclay/code/mattclay/ansible/lib/ansible
  executable location = /Users/mclay/code/mattclay/ansible/bin/ansible
  python version = 2.7.11 (default, Jan 22 2016, 08:29:18) [GCC 4.2.1 Compatible Apple LLVM 7.0.2 (clang-700.1.81)]

mattclay added 7 commits June 23, 2017 12:02
This avoids a pip install of argparse when running ansible-test.
This permits pip installs of packages which require compiling.
This avoids having to install cryptography during test runs.
Installation via pip is done when OS packages are unavailable.
This will ensure the latest version of coverage is used.
We require coverage 4.2+ which some OS packages don't provide.
Other OS packages don't provide coverage with the C extension.
These old versions do not support the --disable-pip-version-check
option, which causes ansible-test to upgrade pip during testing.
@ansibot ansibot added affects_2.4 This issue/PR affects Ansible v2.4 cloud docker feature_pull_request needs_triage Needs a first human triage before being processed. labels Jun 23, 2017
@mattclay mattclay merged commit 4530eea into ansible:devel Jun 23, 2017
@mattclay mattclay deleted the docker-update branch June 23, 2017 19:45
@s-hertel s-hertel removed the needs_triage Needs a first human triage before being processed. label Jun 23, 2017
@ansibot ansibot added feature This issue/PR relates to a feature request. and removed feature_pull_request labels Mar 5, 2018
@ansible ansible locked and limited conversation to collaborators Apr 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.4 This issue/PR affects Ansible v2.4 cloud docker feature This issue/PR relates to a feature request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants