Sunday, 2018-07-15

*** Adri2000 has quit IRC00:05
*** cjloader has quit IRC00:09
*** cjloader has joined #openstack-ansible00:10
*** Adri2000 has joined #openstack-ansible00:10
*** poul has joined #openstack-ansible00:20
*** poul has quit IRC00:24
*** poul has joined #openstack-ansible00:37
*** poul has quit IRC00:42
*** poul has joined #openstack-ansible00:55
*** poul has quit IRC00:59
*** poul has joined #openstack-ansible01:13
*** poul has quit IRC01:17
*** yolanda_ has joined #openstack-ansible01:23
*** yolanda__ has quit IRC01:24
*** poul has joined #openstack-ansible01:30
logan-cloudnull i have 2 reproducible test fails for the connection plugin issues outstanding. https://review.openstack.org/#/c/582329/ shows the way it is broken right now, and https://review.openstack.org/#/c/582330/ is the way it is broken if I "fix" the way it is broken now.01:33
logan-if you have any ideas on how to get https://review.openstack.org/#/c/582330/ to work without breaking delegation please lmk01:34
*** poul has quit IRC01:34
cloudnulllogan- I've have a look at that.01:40
logan-thanks01:40
cloudnullI just got done doing work around the house, so time for dinner but then i'll sit down with it01:41
cloudnullessenatially we need to be able to detect ansible_host not being an IP and then attempt to further render it?01:41
cloudnulllookws like here http://logs.openstack.org/29/582329/3/check/openstack-ansible-functional-ubuntu-xenial/4f4b222/job-output.txt.gz#_2018-07-13_02_32_14_410220 hostname resolution is simply failing01:44
cloudnullshould we expect that hostname resolution be working for that test?01:45
logan-what happens in 582329 is containers don't get detected as containers because of the line removed in 582330 (container_name == ansible_host)01:46
logan-so it is trying to ssh to the container name instead of lxc-attach via physical host01:46
logan-then in 582330 that restriction is removed, except now when you delegate from a container to a different inventory host, it still detects the non delegated host as a container and tries to lxc-attach the task01:47
logan-its not too hard to sort out whats going on with verbose output. let me pastebin that while you are gone01:48
*** poul has joined #openstack-ansible01:48
*** poul has quit IRC01:52
*** poul has joined #openstack-ansible02:05
*** poul has quit IRC02:09
*** tux__ has joined #openstack-ansible02:20
*** poul has joined #openstack-ansible02:23
*** tux__ has quit IRC02:25
*** poul has quit IRC02:27
*** Bhujay has joined #openstack-ansible02:30
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Add meta groups to the elk readme and env.d file  https://review.openstack.org/58273802:36
*** poul has joined #openstack-ansible02:41
logan-here's how the tests look with 582329: http://paste.openstack.org/raw/725883/ (no container detected)02:44
*** poul has quit IRC02:45
logan-http://paste.openstack.org/raw/725885/ is with 582330 applied02:47
*** poul has joined #openstack-ansible02:58
openstackgerritLogan V proposed openstack/openstack-ansible-plugins master: tests: Do not use ansible_host to pass container IP  https://review.openstack.org/58232902:59
openstackgerritLogan V proposed openstack/openstack-ansible-plugins master: tests: Do not use ansible_host to pass container IP  https://review.openstack.org/58232902:59
openstackgerritLogan V proposed openstack/openstack-ansible-plugins master: [WIP] Allow container detection when host matches container_name  https://review.openstack.org/58233003:03
openstackgerritLogan V proposed openstack/openstack-ansible-plugins master: tests: Do not use ansible_host to pass container IP  https://review.openstack.org/58232903:04
openstackgerritLogan V proposed openstack/openstack-ansible-plugins master: [WIP] Allow container detection when host matches container_name  https://review.openstack.org/58233003:04
*** poul has quit IRC03:05
logan-582330 with no ansible_host defined on the container: http://paste.openstack.org/raw/725887/03:11
*** cjloader has quit IRC03:16
cloudnulland back03:35
cloudnulllooking now03:35
openstackgerritLogan V proposed openstack/openstack-ansible-plugins master: [WIP] Allow container detection when host matches container_name  https://review.openstack.org/58233003:46
logan-fixes it ^03:52
*** elbragstad is now known as lbragstad03:56
cloudnullwoot!03:59
cloudnulllogan- looks like that's still failing tests?04:03
cloudnullseems "ansible_host" can't be a group_var?04:03
logan-yeah if you set ansible_host: "{{ container_name }}" it breaks04:08
openstackgerritLogan V proposed openstack/openstack-ansible-tests master: Optional OpenStack openrc role execution  https://review.openstack.org/58273904:09
cloudnullis that breaking from an assumption in that role ?04:10
logan-or actually if ansible_host is not set04:10
cloudnullyea. i think it's doing a hostvar lookup and it's not being found.04:10
openstackgerritLogan V proposed openstack/openstack-ansible-plugins master: [WIP] Allow container detection when host matches container_name  https://review.openstack.org/58233004:11
cloudnullI suspect we'll need to set the hostvars accordingly in and just have one of the containers use ansible_host==container_name04:11
cloudnullinstead of trying to be dynamic04:12
cloudnullIDK though04:12
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible master: Change the osa wrapper from a heredoc to a file  https://review.openstack.org/58263304:13
openstackgerritMerged openstack/openstack-ansible-ops master: Add meta groups to the elk readme and env.d file  https://review.openstack.org/58273804:16
openstackgerritMerged openstack/openstack-ansible-ops master: Add dynamic retention policies to curator  https://review.openstack.org/58111804:18
openstackgerritLogan V proposed openstack/openstack-ansible-plugins master: [WIP] Allow container detection when host matches container_name  https://review.openstack.org/58233004:22
openstackgerritweizj proposed openstack/openstack-ansible-os_tempest master: Drop python-cinderclient usage  https://review.openstack.org/57812204:32
openstackgerritweizj proposed openstack/openstack-ansible-os_glance master: Drop python-cinderclient usage  https://review.openstack.org/57812004:33
*** poul has joined #openstack-ansible04:43
cloudnulllogan- is the use-case where ansible_host is undefined for containers?04:45
cloudnulltesting locally and dumping vars it looks like it's not which is why that openrc role seems to be failing ?04:45
*** mattinator has quit IRC04:46
cloudnullIE on https://review.openstack.org/#/c/582329 the host variable "ansible_host" seems to be undefined04:47
*** poul has quit IRC04:47
cloudnullwhich I think is because of https://review.openstack.org/#/c/582329/6/tests/group_vars/all_containers.yml04:48
logan-either way is fine with me, I am testing it both ways right now trying to find a working setup04:48
cloudnullI think if we uncomment ansible_host here https://review.openstack.org/#/c/582329/6/tests/group_vars/all_containers.yml I think it should work04:49
cloudnullhowever idk if that's actually testing the desired state?04:49
logan-worth a try, 1 sec05:05
openstackgerritLogan V proposed openstack/openstack-ansible-plugins master: tests: Do not use ansible_host to pass container IP  https://review.openstack.org/58232905:05
openstackgerritLogan V proposed openstack/openstack-ansible-plugins master: [WIP] Allow container detection when host matches container_name  https://review.openstack.org/58233005:05
*** poul has joined #openstack-ansible05:09
*** poul has quit IRC05:14
*** cshen has joined #openstack-ansible05:15
*** cshen has quit IRC05:20
*** poul has joined #openstack-ansible05:27
*** poul has quit IRC05:31
cloudnulllogan- mind if i make an edit ?05:34
*** poul has joined #openstack-ansible05:45
*** poul has quit IRC05:49
*** poul has joined #openstack-ansible06:02
*** vnogin has joined #openstack-ansible06:05
*** poul has quit IRC06:07
*** vnogin has quit IRC06:08
logan-go for it06:13
*** DanyC has joined #openstack-ansible06:15
*** poul has joined #openstack-ansible06:20
*** poul has quit IRC06:24
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-plugins master: Allow container detection when host matches container_name  https://review.openstack.org/58233006:24
cloudnullin my local test, with the other commit as a dependent, that works06:25
*** DanyC has quit IRC06:26
cloudnullwith setting the task var to task.delegate_to we were setting the physical host to "{{ item }}"06:26
*** DanyC has joined #openstack-ansible06:27
cloudnullin the vase where the delegation is used with a loop06:27
*** DanyC has quit IRC06:31
*** DanyC has joined #openstack-ansible06:31
*** poul has joined #openstack-ansible06:37
*** poul has quit IRC06:42
*** poul has joined #openstack-ansible06:55
*** poul has quit IRC06:59
*** cshen has joined #openstack-ansible07:05
*** cshen has quit IRC07:09
*** poul has joined #openstack-ansible07:13
*** poul has quit IRC07:17
*** cshen has joined #openstack-ansible07:19
*** poul has joined #openstack-ansible07:22
*** poul has quit IRC07:27
*** poul has joined #openstack-ansible07:39
*** DanyC has quit IRC07:40
*** DanyC has joined #openstack-ansible07:40
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-plugins master: tests: Do not use ansible_host to pass container IP  https://review.openstack.org/58232907:42
*** poul has quit IRC07:43
*** DanyC has quit IRC07:45
*** poul has joined #openstack-ansible07:57
*** DanyC has joined #openstack-ansible07:58
*** DanyC_ has joined #openstack-ansible07:59
*** poul has quit IRC08:01
*** DanyC has quit IRC08:03
*** DanyC has joined #openstack-ansible08:07
*** DanyC_ has quit IRC08:11
*** cshen has quit IRC08:24
*** cshen has joined #openstack-ansible08:25
*** cshen has quit IRC08:29
*** hachi has joined #openstack-ansible08:30
openstackgerritTaseer Ahmed proposed openstack/openstack-ansible-os_blazar master: This patch ensures that the functional tests for Blazar pass.  https://review.openstack.org/57138808:51
*** DanyC has quit IRC08:53
*** DanyC has joined #openstack-ansible08:54
*** cshen has joined #openstack-ansible08:55
*** DanyC has quit IRC08:56
*** DanyC has joined #openstack-ansible08:57
*** vnogin has joined #openstack-ansible08:59
*** cshen has quit IRC09:03
*** vnogin has quit IRC09:09
*** poul has joined #openstack-ansible09:12
*** vnogin has joined #openstack-ansible09:14
*** DanyC has quit IRC09:15
*** vnogin has quit IRC09:22
*** admin0 has joined #openstack-ansible09:32
*** armaan has joined #openstack-ansible10:11
*** vnogin has joined #openstack-ansible10:18
*** armaan has quit IRC10:23
*** armaan has joined #openstack-ansible10:24
*** Bhujay has quit IRC10:26
*** Bhujay has joined #openstack-ansible10:26
*** vnogin has quit IRC10:29
*** armaan has quit IRC10:46
*** armaan has joined #openstack-ansible10:47
*** cshen has joined #openstack-ansible10:59
*** cjloader has joined #openstack-ansible11:09
*** cshen has quit IRC11:28
*** hachi has quit IRC11:31
*** armaan has quit IRC11:32
*** armaan has joined #openstack-ansible11:32
*** cjloader has quit IRC11:35
*** cjloader has joined #openstack-ansible11:35
*** armaan has quit IRC11:36
*** cjloader has quit IRC11:40
*** radeks has joined #openstack-ansible11:47
*** radeks has quit IRC11:56
*** cjloader has joined #openstack-ansible12:12
*** cjloader has quit IRC12:17
*** cshen has joined #openstack-ansible12:18
*** tosky has joined #openstack-ansible12:22
*** poul_ has joined #openstack-ansible12:26
*** poul has quit IRC12:30
*** cshen has quit IRC12:31
*** cshen has joined #openstack-ansible12:35
openstackgerritcaoyuan proposed openstack/openstack-ansible-os_aodh master: fix tox python3 overrides  https://review.openstack.org/57431912:37
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_aodh master: fix tox python3 overrides  https://review.openstack.org/57431912:45
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_aodh master: fix tox python3 overrides  https://review.openstack.org/57431912:47
*** poul_ has quit IRC12:47
*** poul has joined #openstack-ansible12:48
*** cjloader has joined #openstack-ansible12:48
*** cshen has quit IRC12:50
*** cjloader has quit IRC12:53
*** cshen has joined #openstack-ansible12:53
*** cshen has quit IRC13:05
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_zun master: Remove the unnecessary space  https://review.openstack.org/58276613:11
*** cjloader has joined #openstack-ansible13:12
*** cjloader has quit IRC13:17
*** hachi has joined #openstack-ansible13:24
openstackgerritMerged openstack/openstack-ansible-openstack_hosts master: Fix usage of "|" for tests  https://review.openstack.org/58225813:26
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_gnocchi master: Remove the unnecessary space  https://review.openstack.org/58276813:36
openstackgerritzhulingjie proposed openstack/openstack-ansible-repo_server master: Remove the unnecessary space  https://review.openstack.org/58276913:37
openstackgerritzhulingjie proposed openstack/openstack-ansible-pip_install master: Remove the unnecessary space  https://review.openstack.org/58277013:37
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_tempest master: Remove the unnecessary space  https://review.openstack.org/58277113:38
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_monasca-agent master: Remove the unnecessary space  https://review.openstack.org/58277213:38
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_panko master: Remove the unnecessary space  https://review.openstack.org/58277313:38
openstackgerritzhulingjie proposed openstack/openstack-ansible-memcached_server master: Remove the unnecessary space  https://review.openstack.org/58277413:39
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_horizon master: Fix the UI Panel name of Octavia  https://review.openstack.org/58277513:44
openstackgerritMerged openstack/openstack-ansible-galera_server master: Fix usage of "|" for tests  https://review.openstack.org/58224313:50
openstackgerritzhulingjie proposed openstack/openstack-ansible-lxc_hosts master: Remove the unnecessary space  https://review.openstack.org/58277613:50
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_searchlight master: Remove the unnecessary space  https://review.openstack.org/58277713:51
*** cshen has joined #openstack-ansible13:51
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_congress master: Remove the unnecessary space  https://review.openstack.org/58277813:51
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_ceilometer master: Remove the unnecessary space  https://review.openstack.org/58277913:52
openstackgerritzhulingjie proposed openstack/openstack-ansible-ops master: Remove the unnecessary space  https://review.openstack.org/58278013:52
openstackgerritzhulingjie proposed openstack/openstack-ansible-rsyslog_server master: Remove the unnecessary space  https://review.openstack.org/58278113:52
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_cloudkitty master: Remove the unnecessary space  https://review.openstack.org/58278213:52
openstackgerritzhulingjie proposed openstack/openstack-ansible-haproxy_server master: Remove the unnecessary space  https://review.openstack.org/58278313:52
openstackgerritzhulingjie proposed openstack/openstack-ansible-tests master: Remove the unnecessary space  https://review.openstack.org/58278413:53
openstackgerritzhulingjie proposed openstack/openstack-ansible-lxc_container_create master: Remove the unnecessary space  https://review.openstack.org/58278513:53
*** cjloader has joined #openstack-ansible13:54
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_glance master: Remove the unnecessary space  https://review.openstack.org/58278613:55
openstackgerritzhulingjie proposed openstack/openstack-ansible-repo_build master: Remove the unnecessary space  https://review.openstack.org/58278713:57
*** cjloader has quit IRC13:58
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_nova master: Remove the unnecessary space  https://review.openstack.org/58278814:01
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_neutron master: Remove the unnecessary space  https://review.openstack.org/58278914:02
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_trove master: Remove the unnecessary space  https://review.openstack.org/58279014:06
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_keystone master: Remove the unnecessary space  https://review.openstack.org/58279114:06
*** vnogin has joined #openstack-ansible14:28
*** vnogin has quit IRC14:30
openstackgerritzhulingjie proposed openstack/openstack-ansible-os_horizon master: Update the horizon help link  https://review.openstack.org/58279314:30
*** poul_ has joined #openstack-ansible14:52
*** poul has quit IRC14:55
*** ansmith has quit IRC14:55
*** cshen has quit IRC14:59
*** admin0 has quit IRC15:03
*** armaan has joined #openstack-ansible15:11
*** armaan_ has joined #openstack-ansible15:15
*** armaan_ has quit IRC15:15
*** armaan_ has joined #openstack-ansible15:16
*** armaan_ has quit IRC15:17
*** armaan__ has joined #openstack-ansible15:17
*** vnogin has joined #openstack-ansible15:18
*** vnogin has quit IRC15:18
*** armaan has quit IRC15:18
*** vnogin has joined #openstack-ansible15:19
*** vnogin has quit IRC15:19
*** armaan__ has quit IRC15:27
*** armaan has joined #openstack-ansible15:28
*** poul has joined #openstack-ansible15:32
*** armaan has quit IRC15:32
*** poul_ has quit IRC15:35
*** armaan has joined #openstack-ansible15:44
*** admin0 has joined #openstack-ansible15:56
*** armaan has quit IRC16:11
*** armaan has joined #openstack-ansible16:12
*** admin0 has quit IRC16:18
*** aludwar has quit IRC16:20
*** vnogin has joined #openstack-ansible16:21
*** vnogin has quit IRC16:22
openstackgerritMerged openstack/openstack-ansible master: Fix usage of "|" for tests  https://review.openstack.org/58226116:29
*** armaan has quit IRC16:30
*** armaan has joined #openstack-ansible16:31
*** armaan has quit IRC16:40
*** aludwar has joined #openstack-ansible16:43
*** vnogin has joined #openstack-ansible16:43
*** cshen has joined #openstack-ansible16:54
*** vnogin has quit IRC16:59
*** cshen has quit IRC16:59
*** hachi has quit IRC17:01
*** vnogin has joined #openstack-ansible17:05
*** cjloader has joined #openstack-ansible17:12
*** cjloader has quit IRC17:17
*** armaan has joined #openstack-ansible17:17
*** Bhujay has quit IRC17:18
*** vnogin has quit IRC17:22
*** vnogin has joined #openstack-ansible17:28
*** vnogin has quit IRC17:32
*** admin0 has joined #openstack-ansible18:06
*** cjloader has joined #openstack-ansible18:08
*** aludwar has quit IRC18:16
*** cjloader has quit IRC18:17
*** ansmith has joined #openstack-ansible18:23
*** cjloader has joined #openstack-ansible18:30
*** cjloader has quit IRC18:35
*** hachi has joined #openstack-ansible18:37
*** ansmith has quit IRC18:39
*** v1a4 has joined #openstack-ansible18:40
*** nurdie has joined #openstack-ansible18:49
*** aludwar has joined #openstack-ansible18:51
*** tosky has quit IRC18:53
*** aludwar has quit IRC18:58
*** nurdie has quit IRC19:00
*** nurdie has joined #openstack-ansible19:01
*** nurdie has quit IRC19:05
*** cjloader has joined #openstack-ansible19:12
*** vnogin has joined #openstack-ansible19:13
*** cjloader has quit IRC19:17
*** vnogin has quit IRC19:19
*** coreywright has joined #openstack-ansible19:20
*** aludwar has joined #openstack-ansible19:25
*** cshen has joined #openstack-ansible19:42
*** cjloader has joined #openstack-ansible19:45
*** v1a4 has quit IRC20:12
*** vnogin has joined #openstack-ansible20:16
*** vnogin has quit IRC20:21
*** cjloader has quit IRC20:31
*** cjloader has joined #openstack-ansible20:32
openstackgerritCorey Wright proposed openstack/openstack-ansible-ops master: mnaio: Only resize machines00 LV when defaulting to systemd-nspawn  https://review.openstack.org/58280820:45
openstackgerritCorey Wright proposed openstack/openstack-ansible-ops master: mnaio: Only resize Swift & Cinder machines00 LV when using nspawn  https://review.openstack.org/58280820:49
*** ansmith has joined #openstack-ansible20:55
*** cjloader has quit IRC20:57
coreywrightantonym: i added you as a reviewer to https://review.openstack.org/582808 to allow you to correct me if i misunderstood / misinterpreted the intention of your original commit/change, ie https://review.openstack.org/58188320:57
*** cshen has quit IRC21:15
*** cshen has joined #openstack-ansible21:20
*** cshen has quit IRC21:25
guilhermespansible's include_role fix was merged today for 2.7 https://github.com/ansible/ansible/pull/4133021:33
*** pathcl_ is now known as pathcl21:48
*** tux_ has joined #openstack-ansible23:39
*** tux_ has quit IRC23:55

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!