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

added tests for correctly handling handlers w free #44175

Open
wants to merge 3 commits into
base: devel
from

Conversation

@bcoca
Copy link
Member

@bcoca bcoca commented Aug 15, 2018

SUMMARY

confirms #33011 fix

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

strategy

ANSIBLE VERSION
2.7
@mattclay
Copy link
Member

@mattclay mattclay commented Aug 28, 2018

@mattclay
Copy link
Member

@mattclay mattclay commented Aug 31, 2018

@mattclay
Copy link
Member

@mattclay mattclay commented Nov 7, 2018

CI failure in integration tests: https://app.shippable.com/github/ansible/ansible/runs/92369/56/console

03:18 ++ ansible-playbook test_handlers_include_role.yml -i ../../inventory -v -vvvvvv
03:18 ++ egrep -o 'RUNNING HANDLER \[test_handlers_include_role : .*?]'
03:19 + '[' 'RUNNING HANDLER [test_handlers_include_role : test handler]' = 'RUNNING HANDLER [test_handlers_include_role : test handler]' ']'
03:19 ++ ansible-playbook free_strategy.yml -i ../../inventory -v -vvvvvv
03:19 ++ grep testhost
03:19 ++ grep -c -v testhost2
04:40 + '[' 12 = 9 ']'
@bcoca bcoca force-pushed the bcoca:test_free_handlers branch to 82d5f94 Dec 11, 2018
@mattclay
Copy link
Member

@mattclay mattclay commented Dec 19, 2018

@Shrews
Copy link
Contributor

@Shrews Shrews commented Sep 11, 2020

Is this something we still want?

@samdoran
Copy link
Member

@samdoran samdoran commented Mar 15, 2021

/azp run

@azure-pipelines
Copy link

@azure-pipelines azure-pipelines bot commented Mar 15, 2021

Azure Pipelines successfully started running 1 pipeline(s).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants