Ensure handlers have proper parent

[sivel]

SUMMARY

This PR ensures that handlers have the correct parent. This change does for handlers what tasks already had done.

This ensures that things like the following apply to handlers:

- import_role:
    name: thing
  delegate_to: localhost

Fixes #36518

Additionally, on a small note, task_list.extend is in place and returns nothing, don't assign to t.

ISSUE TYPE

• Bugfix Pull Request

COMPONENT NAME

ANSIBLE VERSION

ADDITIONAL INFORMATION

[sivel]

FYI, before this change, the new integration test produces:

RUNNING HANDLER [delegated_handler : delegated assert handler] *****************
fatal: [testhost]: FAILED! => {
    "assertion": "ansible_delegated_vars is defined",
    "changed": false,
    "evaluated_to": false
}

[SivaKrishna05]

This issue not resolved yet. I have been facing similar issue with both Ansible 2.6.1 and 2.6.2.
issue delegate_to not working to change the host for include_role.

- include_role:
    name: role_name
  delegate_to: hostname

I have tried same code with 2.4.3 it's working expected.

[sivel]

@SivaKrishna05 delegate_to on an include_role does nothing in Ansible 2.5+. This issue was in regards to using import_role.

Please see: https://docs.ansible.com/ansible/latest/porting_guides/porting_guide_2.5.html#dynamic-includes-and-attribute-inheritance

[SivaKrishna05]

@sivel you mean to say that's expected behavior and nothing to do with next Ansible version updates

[DylanYoung]

I'm still seeing this issue on 2.6.5, but I suspect it's related to a meta: flush_handlers call after the import_role.

That is, the workflow is slightly more complex.

Does meta support delegate_to?

[DylanYoung]

Scratch that. The wrong parent is still being recorded. The import_role is nested under an include_role, if that helps :p