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

Initial ansible-test support for collections. #59197

Merged
merged 13 commits into from
Jul 23, 2019

Conversation

mattclay
Copy link
Member

SUMMARY

Initial ansible-test support for collections.

This maintains existing support for using ansible-test on the Ansible source while providing the initial refactoring needed to support testing of Ansible collections.

Collections testing support is still an early preview. Many features for testing collections are incomplete, have bugs or are completely missing. Some existing features will change significantly in subsequent PRs.

The primary change in existing behavior is that ansible-test can no longer be invoked from a current working directory outside of the content root being tested.

ISSUE TYPE

Feature Pull Request

COMPONENT NAME

ansible-test

@ansibot ansibot added affects_2.9 This issue/PR affects Ansible v2.9 core_review In order to be merged, this PR must follow the core review workflow. feature This issue/PR relates to a feature request. needs_triage Needs a first human triage before being processed. new_plugin This PR includes a new plugin. support:core This issue/PR relates to code supported by the Ansible Engineering Team. test This PR relates to tests. labels Jul 17, 2019
@mattclay mattclay removed the needs_triage Needs a first human triage before being processed. label Jul 17, 2019
@ansibot ansibot added needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed core_review In order to be merged, this PR must follow the core review workflow. labels Jul 17, 2019
@ansibot ansibot removed the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. label Jul 18, 2019
ci_complete
ci_coverage
@mattclay
Copy link
Member Author

Full coverage run: https://app.shippable.com/github/ansible/ansible/runs/132593/

The timeouts are unrelated to the changes.

@ansibot ansibot removed the core_review In order to be merged, this PR must follow the core review workflow. label Jul 18, 2019
pabelanger added a commit to pabelanger/ansible that referenced this pull request Jul 18, 2019
Depends-On: ansible#59197
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
@mattclay mattclay added this to In progress in Testing Collections Jul 19, 2019
Copy link
Contributor

@jborean93 jborean93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry it took so long, everything seems to be good I just had a few questions/queries about some of the changes.

test/runner/lib/cli.py Show resolved Hide resolved
test/runner/lib/cli.py Outdated Show resolved Hide resolved
test/runner/lib/coverage_util.py Show resolved Hide resolved
test/runner/lib/powershell_import_analysis.py Outdated Show resolved Hide resolved
test/runner/lib/sanity/__init__.py Outdated Show resolved Hide resolved
test/runner/lib/sanity/ansible_doc.py Show resolved Hide resolved
test/runner/lib/util.py Show resolved Hide resolved
The module option can be plural but powershell must be singular.
This makes it more obvious what the variables are referencing and keeps them consistent.

- `is_install` -> `is_ansible`
- `INSTALL_ROOT` -> `ANSIBLE_ROOT`
@mattclay mattclay merged commit 79eca9c into ansible:devel Jul 23, 2019
Testing Collections automation moved this from In progress to Done Jul 23, 2019
@mattclay mattclay deleted the at-collections-preview branch July 23, 2019 02:24
@mattclay mattclay self-assigned this Jul 23, 2019
@mattclay mattclay removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. new_plugin This PR includes a new plugin. labels Jul 23, 2019
@ansible ansible locked and limited conversation to collaborators Aug 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.9 This issue/PR affects Ansible v2.9 feature This issue/PR relates to a feature request. support:core This issue/PR relates to code supported by the Ansible Engineering Team. test This PR relates to tests.
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants