Mark Goddard

Results 79 comments of Mark Goddard

Fixed an issue with the change to the free strategy plugin caught by the handlers integration test.

Dependent change #82407 has now merged, so I've marked this as ready for review. I've included a fix for the issue mentioned [here](https://github.com/ansible/ansible/pull/82407#pullrequestreview-1823324949) by @mkrizek.

Updated after conflicting change was made in the callbacks tests.

> @markgoddard the CI issues appear to be related to the patch in this PR. The same error output is present in different test runtimes: https://dev.azure.com/ansible/ansible/_build/results?buildId=102426&view=logs&j=b8b182ae-7382-5e2d-7321-805eb8f377b5&t=61efecb5-7da7-54db-c13c-3d39685f67a7&l=5606. Thanks for spotting that,...

Hi @webknjaz, how is it looking now?

Hi @webknjaz, @bcoca. What can I do to help move this forward? I realise you must be busy (I maintain some OpenStack projects, so I've been on both sides of...

> > Hi @webknjaz, @bcoca. What can I do to help move this forward? > > Thank you for your contribution. Apologies this hasn't had much attention. For this particular...

Also noticed this. Using offline as an override for immediate seems useful & reasonable to me, i.e. setting offline, immediate & permanent all to true should mean "set this permanently....

Hi @b1r63r, will you provide a PR?

Hi @btravouillon, thanks for the patch. I think the code is working as expected - it should be possible to start the units without a listen address. Perhaps for some...