Comment 2 for bug 1790779

Discussion from the bug triage meeting today. I am seeing it on Pike, logs included:

[16:39:10] <mnaser> #link https://bugs.launchpad.net/openstack-ansible/+bug/1790779
[16:39:11] <openstack> Launchpad bug 1790779 in openstack-ansible "delegate_to on deploying host uses dns?" [Undecided,New]
[16:39:58] <mnaser> hwoarang: or logan- might chime on this?
[16:40:17] <hwoarang> hmm
[16:40:23] <mnaser> afaik this should use the physical_mapping thing no?
[16:40:43] <logan-> yes one sec
[16:41:00] <logan-> i hit this on pike last week
[16:41:02] <evrardjp> interesting, is it the case of delegation to something not in the inventory that triggers?
[16:41:03] <logan-> it is broken as of https://github.com/openstack/openstack-ansible/commit/a8a809839484105d9cd27463defc19a8a617c64b#diff-db999e390dd84f2a8c2a48b19aa9533f
[16:41:31] <logan-> you can see it on a prod deploy here: http://logs.openstack.org/e9/e957a7dc2fd07f86bc2ab60dd25e00504c705eca/post/limestone-ci-deploy/705562f/
[16:41:41] <evrardjp> oh god it's loop_var that causing the issue?
[16:41:43] <hwoarang> reporter does not say what version it is
[16:41:48] <logan-> http://logs.openstack.org/e9/e957a7dc2fd07f86bc2ab60dd25e00504c705eca/post/limestone-ci-deploy/705562f/logs/ara-report/result/45ed4626-65ca-4b65-a4c4-d39cf32eaadf/
[16:41:57] <jamesdenton> mnaser pong
[16:42:02] <hwoarang> i thought the interpolation of loop_var was fixed by the recent commit in the plugins repo
[16:42:23] <logan-> hwoarang: i was hoping that backporting your change to the other branches would solve this, but it did not in pike
[16:42:30] <openstackgerrit> Jacob Wagner proposed openstack/openstack-ansible-ops master: Add ability to deploy Ceph into a Multinode AIO https://review.openstack.org/585003
[16:42:48] <openstackgerrit> Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_keystone master: Implement LDAP domains using last keystone host https://review.openstack.org/603440
[16:43:18] <hwoarang> we should clarify on the bug what is the OSA version that he is using
[16:43:20] <evrardjp> debug of https://github.com/openstack/openstack-ansible-plugins/blob/master/strategy/linear.py#L138-L147 would help I guess
[16:43:20] <logan-> that build used dc4abed00960c0123d45e9e67e666a5e9a1bb51f from plugins http://logs.openstack.org/e9/e957a7dc2fd07f86bc2ab60dd25e00504c705eca/post/limestone-ci-deploy/705562f/job-output.txt.gz#_2018-09-17_19_32_59_217416
[16:43:25] <hwoarang> because master and rocky should be ok
[16:43:37] <logan-> which is your fix: https://github.com/openstack/openstack-ansible-plugins/commits/stable/pike
[16:43:40] *** Quits: Darcidride (~Darcidrid@2a01:e35:8b4a:ca10::719e:e3e1) (Remote host closed the connection)
[16:43:49] <evrardjp> hwoarang: do we have the loop_var in testing?
[16:43:56] <hwoarang> prob not
[16:44:03] <odyssey4me> we do not, because there is only one cluster member
[16:44:14] <hwoarang> but in opnfv we use rocky with multinode and it's ok (so we are using the loop_var thing)
[16:44:15] <odyssey4me> we should add one to the plugins repo I guess
[16:44:16] *** Joins: Darcidride (~Darcidrid@2a01:e35:8b4a:ca10::719e:e3e1)
[16:44:22] <mnaser> jamesdenton: we can discuss after this issue :>
[16:44:33] <jamesdenton> yep - will sync up later
[16:44:42] <evrardjp> hwoarang: ok
[16:45:07] *** Quits: Darcidride (~Darcidrid@2a01:e35:8b4a:ca10::719e:e3e1) (Remote host closed the connection)
[16:45:15] <openstackgerrit> Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_keystone master: Implement LDAP domains using last keystone host https://review.openstack.org/603440
[16:45:36] <mnaser> so what do we do about this bug?
[16:45:40] *** Joins: dariko (~user@185.216.33.115)
[16:46:05] <openstackgerrit> Markos Chandras (hwoarang) proposed openstack/openstack-ansible-os_keystone master: SUSE: Add support for openSUSE Leap 15 https://review.openstack.org/603114
[16:46:15] <logan-> i don't know the fix yet, but it clearly exists in pike, not sure about ocata or queens, hwoarang says rocky is fixed and we can presume master is probably ok if rocky works
[16:46:33] <hwoarang> mnaser: well based on what logan said it's ay it's confirmed but I would ask the reporter to at least provide the OSA version so we know what we are looking for
[16:46:34] <odyssey4me> we either backout multiple fixes, and add a release note notifying people - or we add a test to expose it and fix it, and add a release note
[16:47:10] <logan-> https://github.com/openstack/openstack-ansible/commit/a8a809839484105d9cd27463defc19a8a617c64b#diff-db999e390dd84f2a8c2a48b19aa9533f could be reverted and implemented differently
[16:47:25] <logan-> rather than trying to fix plugins
[16:47:30] *** Quits: olivierb (~olivierbo@178.18.54.206) (Ping timeout: 252 seconds)
[16:47:33] *** Quits: dariko (~user@185.216.33.115) (Remote host closed the connection)
[16:47:51] <odyssey4me> or that
[16:47:53] <mnaser> incomplete = ask for information such as release?
[16:48:23] <evrardjp> I'd say so
[16:48:24] <logan-> i will append the bug with this log so we can mark it confirmed on pike at least
[16:48:31] <evrardjp> ok
[16:48:38] <mnaser> logan-: cool wfm
[16:49:10] <logan-> would be nice to know if queens is broken also