Skip to content

Commit

Permalink
tests: Remove --limit when running Ansible localhost CI
Browse files Browse the repository at this point in the history
Some tests were being incorrectly excluded. Including those that use
`add_host`.
refs mitogen-hq#1066, mitogen-hq#1069
  • Loading branch information
moreati committed May 21, 2024
1 parent 0124e11 commit 0fe741c
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .ci/localhost_ansible_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -93,5 +93,5 @@
with ci_lib.Fold('ansible'):
os.chdir(TESTS_DIR)
playbook = os.environ.get('PLAYBOOK', 'all.yml')
ci_lib.run('./run_ansible_playbook.py %s -l target %s',
ci_lib.run('./run_ansible_playbook.py %s %s',
playbook, ' '.join(sys.argv[1:]))
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,19 @@
# since things are ran on localhost; Azure DevOps loses connection and fails
# TODO: do we want to install docker a different way to be able to do this for other tests too
---
- name: regression/issue_655_wait_for_connection_error.yml
- name: regression/issue_655__wait_for_connection_error.yml
hosts: localhost
gather_facts: yes
become: no
tasks:
- meta: end_play
when:
# TODO CI currently runs on macOS 11 images in Azure DevOps. MacOS 11
# is no longer supported by homebrew, so the following install
# task fails.
- ansible_facts.system == 'Darwin'
- ansible_facts.distribution_major_version == '11'

- name: set up test container and run tests inside it
block:
- name: install deps
Expand Down

0 comments on commit 0fe741c

Please sign in to comment.