Skip to content

Commit dc21b45

Browse files
moreatijadacyrus
authored andcommitted
tests: Remove --limit when running Ansible localhost CI
Some tests were being incorrectly excluded. Including those that use `add_host`. refs mitogen-hq#1066, mitogen-hq#1069
1 parent 686adef commit dc21b45

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

.ci/localhost_ansible_tests.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -93,5 +93,5 @@
9393
with ci_lib.Fold('ansible'):
9494
os.chdir(TESTS_DIR)
9595
playbook = os.environ.get('PLAYBOOK', 'all.yml')
96-
ci_lib.run('./run_ansible_playbook.py %s -l target %s',
96+
ci_lib.run('./run_ansible_playbook.py %s %s',
9797
playbook, ' '.join(sys.argv[1:]))

tests/ansible/regression/issue_655__wait_for_connection_error.yml

+9-1
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,19 @@
44
# since things are ran on localhost; Azure DevOps loses connection and fails
55
# TODO: do we want to install docker a different way to be able to do this for other tests too
66
---
7-
- name: regression/issue_655_wait_for_connection_error.yml
7+
- name: regression/issue_655__wait_for_connection_error.yml
88
hosts: localhost
99
gather_facts: yes
1010
become: no
1111
tasks:
12+
- meta: end_play
13+
when:
14+
# TODO CI currently runs on macOS 11 images in Azure DevOps. MacOS 11
15+
# is no longer supported by homebrew, so the following install
16+
# task fails.
17+
- ansible_facts.system == 'Darwin'
18+
- ansible_facts.distribution_major_version == '11'
19+
1220
- name: set up test container and run tests inside it
1321
block:
1422
- name: install deps

0 commit comments

Comments
 (0)