Thursday, 2018-07-05

*** cjloader has joined #openstack-ansible00:14
*** cjloader has quit IRC00:17
*** dxiri has quit IRC00:22
openstackgerritLeopard Ma proposed openstack/openstack-ansible-os_horizon master: Add source code repository notes link to README  https://review.openstack.org/57942100:35
openstackgerritLeopard Ma proposed openstack/openstack-ansible-os_heat master: Add source code repository notes link to README  https://review.openstack.org/57941900:39
openstackgerritLeopard Ma proposed openstack/openstack-ansible-os_gnocchi master: Add source code repository notes link to README  https://review.openstack.org/57941700:44
openstackgerritLeopard Ma proposed openstack/openstack-ansible-os_glance master: Add source code repository notes link to README  https://review.openstack.org/57941600:47
*** tux_ has joined #openstack-ansible00:50
*** chyka has joined #openstack-ansible00:55
*** chyka has quit IRC01:00
cloudnullI like ponies01:17
guilhermespcloudnull really like them, that's cute01:27
guilhermespbe happy: https://i.imgur.com/ottKmhq.jpg01:33
*** dxiri has joined #openstack-ansible01:43
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-os_tempest master: fix tox python3 overrides  https://review.openstack.org/58006902:26
*** dxiri has quit IRC02:40
*** udesale has joined #openstack-ansible03:44
*** dxiri has joined #openstack-ansible03:52
*** tux_ has quit IRC03:55
*** dxiri has quit IRC03:59
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-os_sahara master: fix tox python3 overrides  https://review.openstack.org/58006404:12
*** Bhujay has joined #openstack-ansible04:21
*** vnogin has joined #openstack-ansible04:32
*** vnogin has quit IRC04:36
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-os_neutron master: fix tox python3 overrides  https://review.openstack.org/58005804:41
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-repo_server master: fix tox python3 overrides  https://review.openstack.org/58007904:44
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-rabbitmq_server master: fix tox python3 overrides  https://review.openstack.org/58007604:48
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-plugins master: fix tox python3 overrides  https://review.openstack.org/58007504:53
*** jbadiapa has quit IRC05:24
*** yolanda has joined #openstack-ansible05:30
*** dxiri has joined #openstack-ansible05:33
*** ianychoi has joined #openstack-ansible05:34
*** nicolasbock has joined #openstack-ansible05:49
*** dxiri has quit IRC06:06
*** armaan has joined #openstack-ansible06:13
*** armaan has quit IRC06:15
*** armaan has joined #openstack-ansible06:16
*** armaan has quit IRC06:21
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-os_neutron master: fix tox python3 overrides  https://review.openstack.org/58005806:22
*** cshen has joined #openstack-ansible06:24
*** dxiri has joined #openstack-ansible06:27
*** dxiri has quit IRC06:31
*** cjloader has joined #openstack-ansible06:32
*** lvdombrkr has joined #openstack-ansible06:32
*** cjloader has quit IRC06:36
openstackgerritVu Cong Tuan proposed openstack/ansible-config_template master: Replace unicode with six.text_type  https://review.openstack.org/58029906:41
hwoaranggood morning06:42
hwoarangi have a question about the dynamic inventory06:42
hwoarangmy understsanding is that group/hosts are defined in inventory/env.d/* . How can an operator disable/remove some of these groups?06:42
*** maybebuggy has joined #openstack-ansible06:42
hwoarangi am specifically looking to remove the repo_all group when deploying with distribution packages06:42
openstackgerritLeopard Ma proposed openstack/openstack-ansible-memcached_server master: Add source code repository notes link to README  https://review.openstack.org/57971506:43
*** jbadiapa has joined #openstack-ansible06:43
openstackgerritVu Cong Tuan proposed openstack/openstack-ansible-plugins master: Replace unicode with six.text_type  https://review.openstack.org/58030106:49
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-os_neutron master: fix tox python3 overrides  https://review.openstack.org/58005806:59
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-repo_build master: fix tox python3 overrides  https://review.openstack.org/58007707:04
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-pip_install master: fix tox python3 overrides  https://review.openstack.org/58007407:08
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-os_rally master: fix tox python3 overrides  https://review.openstack.org/58006007:10
*** gkadam has joined #openstack-ansible07:11
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-os_ironic master: fix tox python3 overrides  https://review.openstack.org/58002407:12
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-os_glance master: fix tox python3 overrides  https://review.openstack.org/58002007:15
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-os_designate master: fix tox python3 overrides  https://review.openstack.org/58001807:18
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-os_barbican master: fix tox python3 overrides  https://review.openstack.org/58001607:21
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-galera_server master: fix tox python3 overrides  https://review.openstack.org/58001207:23
*** vnogin has joined #openstack-ansible07:23
*** fghaas has joined #openstack-ansible07:25
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-lxc_container_create master: fix tox python3 overrides  https://review.openstack.org/58001307:26
*** epalper has joined #openstack-ansible07:26
*** vnogin has quit IRC07:27
*** fghaas has quit IRC07:29
*** Bhujay has quit IRC07:31
*** tosky has joined #openstack-ansible07:32
*** cjloader has joined #openstack-ansible07:32
*** cjloader has quit IRC07:36
*** DanyC has joined #openstack-ansible07:43
*** DanyC has quit IRC07:44
*** DanyC has joined #openstack-ansible07:44
*** fghaas has joined #openstack-ansible07:47
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible master: Add support for using distribution packages for OpenStack services  https://review.openstack.org/57977007:53
*** suggestable has joined #openstack-ansible08:01
evrardjpgppd morning08:06
evrardjphwoarang: removal is hard08:07
evrardjphwoarang: making sure the group is empty is easier08:07
*** vnogin has joined #openstack-ansible08:07
evrardjptwo ways to do that08:07
evrardjp1) do not use the global "infra_hosts" groups, that are spawning entries in subgroups08:08
evrardjp2) use an affinity of 0 into a container type08:08
hwoaranghmm08:11
hwoarangdo you have an example about the 2) option?08:12
evrardjphttps://docs.openstack.org/openstack-ansible/latest/reference/inventory/configure-inventory.html#deploying-0-or-more-than-one-of-component-type-per-host08:13
evrardjphwoarang: ^08:13
hwoarangthank you. let me try that...08:14
*** Bhujay has joined #openstack-ansible08:15
*** fghaas has quit IRC08:17
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible master: Add support for using distribution packages for OpenStack services  https://review.openstack.org/57977008:31
*** fghaas has joined #openstack-ansible08:38
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-os_neutron master: fix tox python3 overrides  https://review.openstack.org/58005808:45
*** fghaas has quit IRC08:58
*** fghaas has joined #openstack-ansible09:15
odyssey4mehwoarang: perhaps better to separate out the repo group into its own conf.d file, then only apply it to the scenarios which use it09:28
hwoarangpossibly... i am still experimenting :/09:30
*** cjloader has joined #openstack-ansible09:32
odyssey4meideally I'd prefer to move away from using the super-group infra_hosts anyway, and instead ensure the use of specific conf.d files for components we want to include09:34
odyssey4melater after M3 that'll be something I work on to allow us to use the integrated repo for role tests09:34
*** cjloader has quit IRC09:37
openstackgerritTuan Do Anh proposed openstack/ansible-hardening master: fix tox python3 overrides  https://review.openstack.org/57977809:38
hwoarangodyssey4me: right now i am looking at the simplest and less disruptive way to decouple AIO from repo_container so i can have basekit tests for distro packages09:40
hwoarangi haven't put much thought to make it pretty :/09:41
odyssey4mein my mind, that is the simplest and least disruptive way ;) but yeah, it'll be interesting to see what you come up with09:43
hwoarangyeah prob because for me the dynamic inventory is complex ;p09:44
*** armaan has joined #openstack-ansible09:44
hwoarangbut i think what you say makes sense09:45
openstackgerritRobert Donovan proposed openstack/openstack-ansible-os_horizon master: Add support for the default_dns_nameservers variable  https://review.openstack.org/57422709:46
*** spsurya_ has quit IRC09:49
*** fghaas has quit IRC09:50
openstackgerritNguyen Hung Phuong proposed openstack/openstack-ansible-os_neutron master: fix tox python3 overrides  https://review.openstack.org/58005809:51
*** fghaas has joined #openstack-ansible09:54
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible master: Allow inventories with no "properties"  https://review.openstack.org/58033510:05
*** armaan has quit IRC10:06
*** armaan has joined #openstack-ansible10:07
*** pingfrog has joined #openstack-ansible10:10
hwoarangor if one could decouple repo from infra-hosts i can build on top of it...10:10
*** armaan has quit IRC10:11
*** jbadiapa has quit IRC10:15
openstackgerritRobert Donovan proposed openstack/openstack-ansible-os_nova master: Adds support for the cpu_model customisation in nova.conf  https://review.openstack.org/58033710:16
*** Bhujay has quit IRC10:19
evrardjpwell I am building a static inventory, so it would all become simpler.10:26
evrardjpwe could then for gate testing use ini_file for patching the file. or config template.10:26
evrardjpno need to care about the dynamic inventory anymore.10:26
evrardjpI could spend cycles on the separation, but there is no point to it in the future, so for now, I'd say go for the past with the least resistence10:27
*** cjloader has joined #openstack-ansible10:32
*** cjloader has quit IRC10:36
evrardjphwoarang: for your case there is another solution10:50
evrardjpas repo is already kinda conditional10:50
evrardjpyou could have an extra condition when building repo_masters group10:50
evrardjpthat depends on the global flag install_packages state10:51
evrardjpwhen install_packages (the global boolean flip) is set to true, you can skip that task, and therefore no hosts will be added to the group, and no repo will be built10:51
evrardjpwe only then need to document that people should not BOTH set install_packages to True and a subpart like nova_install_packages to False.10:52
evrardjpthat would apply to all users which makes it quite elegant at the end10:53
*** Bhujay has joined #openstack-ansible10:55
*** radeks has joined #openstack-ansible11:01
*** radeks has quit IRC11:05
*** radeks has joined #openstack-ansible11:06
*** MB_Adriaan has joined #openstack-ansible11:06
*** suggestable has quit IRC11:07
*** suggestable has joined #openstack-ansible11:08
*** cshen has quit IRC11:08
*** cshen has joined #openstack-ansible11:21
*** armaan has joined #openstack-ansible11:22
*** pingfrog has quit IRC11:25
*** chyka has joined #openstack-ansible11:36
hwoarangevrardjp: i would need time to parse and implement that . this area of aio is not very clear to me11:37
hwoaranganyway lets make basekit pass first and we will see11:38
*** hachi has joined #openstack-ansible11:38
*** radeks has quit IRC11:39
*** chyka has quit IRC11:40
evrardjpthat's the interesting thing: this would not become an AIO thing or a testing thing. With what I proposed a few lines above, it works for all the deployers whatever their env is11:41
*** radeks has joined #openstack-ansible11:48
hwoarangok11:48
mgariepymorning everyone11:54
*** mardim has quit IRC11:57
evrardjpmorning mgariepy11:57
*** lvdombrkr89 has joined #openstack-ansible11:58
*** mardim has joined #openstack-ansible11:59
*** lvdombrkr has quit IRC11:59
hwoarangmorning12:00
mgariepyhow are you guys doing ?12:00
hwoarangbuuuuuusy12:09
hwoarangevrardjp: "[13:50:49] <evrardjp> you could have an extra condition when building repo_masters group"12:09
hwoarangwhere does that happen?12:09
*** admin0 has quit IRC12:12
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible master: Be consistent with haproxy group name  https://review.openstack.org/58035912:13
evrardjphwoarang: https://github.com/openstack/openstack-ansible/blob/master/playbooks/repo-build.yml#L4012:13
evrardjpjust add a when: :)12:13
hwoarangevrardjp: i skip this playbook entirely12:14
hwoarangit's not enough12:14
evrardjpyeah12:14
evrardjpbut how do you skip it?12:14
hwoarangwhen: install_method == source in setup-infrastructure12:14
hwoarangor wherever it's called12:14
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible master: Ensure container name doesn't need to be defined  https://review.openstack.org/58036312:19
evrardjpoh ok12:19
evrardjptrue12:19
evrardjpand that works?12:19
evrardjpwhy do you need the group to be not defined/empty ?12:20
*** jbadiapa has joined #openstack-ansible12:20
hwoarangbecause some things depend on repo container be there. like haproxy configuration.12:21
odyssey4meevrardjp: because the repo container will be built, although that shouldn't be a problem given that the repo server container is used for a package cacher12:21
hwoarangi mean, it's ok to leave repo container build, populate stuff etc. but nothing will use it so here is 20 minutes of wasted time12:21
odyssey4mehwoarang: but if you skip the repo build, does that not work?12:22
hwoarangnope becuase repo_all group is present12:22
andymccrif anybody is about: https://review.openstack.org/#/c/580140/ :D12:22
hwoarangand contains repo_container12:22
hwoarangand haproxy checks the length of that group and tries to do stuff12:22
hwoarangand i hit undefined variables12:22
andymccralso (less urgent from my PoV): https://review.openstack.org/#/c/580128/12:22
hwoaranga bit of a mess12:22
hwoarang;p12:22
suggestableHey guys, question for you. We would prefer to go entirely on metal. If that means having to split out haproxy onto dedicated metal, that's potentially fine, too. Alternatively, what should we look for in a dedicated hardware load balancer?12:23
*** hachi has quit IRC12:24
*** hachi has joined #openstack-ansible12:25
odyssey4mecores - could we get reviews for https://review.openstack.org/#/q/I6aa297ee6e1ca3a76f016a6a0c3d4a735433a9b2 and https://review.openstack.org/#/q/I5a4d4096457d84fcf3a9e21a6873dd61f2a456fc please12:26
mgariepyodyssey4me, done.12:28
evrardjphwoarang: I am cleaning up stuff, maybe it's in one of my fixes12:28
odyssey4mecores, also could we get eyes on https://review.openstack.org/#/q/topic:move-mq-setup-into-roles+(status:open+OR+status:merged) please12:29
odyssey4methanks mgariepy12:29
evrardjpdone12:29
odyssey4methanks evrardjp12:29
hwoarangwe shall see :)12:30
*** radeks has quit IRC12:37
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible master: Do not run tasks for containers when on metal  https://review.openstack.org/58036512:38
openstackgerritMerged openstack/openstack-ansible-pip_install stable/pike: Pin get-pip.py to 3.2  https://review.openstack.org/58011712:43
openstackgerritMerged openstack/openstack-ansible-pip_install stable/ocata: Pin get-pip.py to 3.2  https://review.openstack.org/58011812:43
openstackgerritMerged openstack/openstack-ansible-pip_install stable/queens: Pin get-pip.py to 3.2  https://review.openstack.org/58011612:43
*** armaan has quit IRC12:46
*** armaan has joined #openstack-ansible12:47
*** radeks has joined #openstack-ansible12:47
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible master: Add support for using distribution packages for OpenStack services  https://review.openstack.org/57977012:49
*** mardim_ has joined #openstack-ansible12:50
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible master: [WIP] Use a static inventory skeleton  https://review.openstack.org/58036812:50
*** mardim has quit IRC12:55
openstackgerritMatt Riedemann proposed openstack/openstack-ansible-os_nova master: Run "nova-status upgrade check" during install and upgrade  https://review.openstack.org/57512512:56
suggestableAnyone? evrardjp odyssey4me13:11
hwoarangandymccr: evrardjp re https://review.openstack.org/#/c/580128/ could you tell how does docstring affect the actual code? or tell me what to google for :)13:13
evrardjpsuggestable: you really have something against lxc apparently : p13:13
evrardjpanyway if you have two nodes for the haproxy it should be fine13:13
evrardjpyou can use these nodes for network nodes if you like13:13
andymccrhwoarang: i pasted teh error in there - im sure we could dig into the ansible internals to find out where it is functionally using that13:13
andymccr*pasted13:13
hwoarangah it's an ansible thing13:14
hwoarangnot a python13:14
suggestableevrardjp: So do you, it seems, from your recent code proposals. Static inventory sounds great to me!13:14
*** cmart has joined #openstack-ansible13:14
hwoarangok then13:14
hwoarangthank you andymccr13:14
evrardjpsuggestable: don't get me wrong: it's possible to do it with existing code, probably submitting a few patches.13:14
evrardjpsuggestable: you can for example add extra ips on your management nodes, and make sure the services listen to the right IPs13:15
evrardjpwe have a bug open for that if you're ready to tackle it13:15
suggestableevrardjp: What sort of hardware would need to be in the nodes to support the environment I'm deploying? We're running 20Gbps LACP bonds on the controller hosts and 10Gbps links on the compute hosts.13:15
andymccrhwoarang: the exception happens here: https://github.com/ansible/ansible/blob/devel/lib/ansible/config/manager.py#L31013:15
andymccrbut i didn't dig too much further than that13:15
hwoarangok thank you i was just curious13:15
andymccrno problem, definitely worth asking those questions13:15
openstackgerritkourosh vivan proposed openstack/openstack-ansible stable/pike: Install gnocchi before ceilometer  https://review.openstack.org/58037513:16
openstackgerritkourosh vivan proposed openstack/openstack-ansible stable/queens: Install gnocchi before ceilometer  https://review.openstack.org/58037613:16
*** armaan has quit IRC13:16
*** armaan has joined #openstack-ansible13:17
evrardjpsuggestable: I cannot be the architect of your environement, I don't know your workloads or anything ...13:17
evrardjponly you can now that.13:17
evrardjpknow*13:17
evrardjpwhat I can tell, is that osa has no limitations whatsoever in regards to what you want to use13:17
suggestableevrardjp: OK. How would you suggest I go about getting haproxy to use a specific IP address (or two of them, if necessary) so we can use the three controller hosts as LB hosts?13:19
*** maybebuggy has quit IRC13:19
*** throwsb1 has joined #openstack-ansible13:19
suggestableSimply add the extra IPs to the management interface and set those IPs in the openstack_user_config.yml?13:20
suggestable(This being a purely on-metal deployment)13:21
*** armaan has quit IRC13:21
evrardjpsuggestable: well, again, I don't know your deployment. If it's small, yes, I'd patch OSA to make sure everything runs on metal with haproxy on the same node on metal too.13:22
*** kouroshvivan has joined #openstack-ansible13:23
suggestableOur current Icehouse cluster is about 30 nova nodes, running about 300 instances across them. If that's small, then yes, it's a small deployment.13:23
evrardjpsuggestable: please also see: https://bugs.launchpad.net/openstack-ansible/+bug/177891413:24
openstackLaunchpad bug 1778914 in openstack-ansible "galera server binds to 0.0.0.0 address" [Low,Confirmed]13:24
*** tux___ has joined #openstack-ansible13:24
evrardjpsuggestable: basically nobody has cared about your use case, because everybody prefer using containers, so isolation is properly done.13:24
*** armaan has joined #openstack-ansible13:24
evrardjpif you're willing to step in and help for your use case, we can help you.13:24
evrardjpI still don't understand why you don't want lxc ocntainers, as you'd be alone in that story, so alone also for the upgrades, which will probably be harder than with containers.13:25
evrardjpbut I respect your opinion13:26
ioniis anyone running network nodes on compute nodes?13:26
evrardjpI just want to pinpoint what the reference architecture is13:26
evrardjpand there was reasons for it13:26
evrardjpsuggestable: you might also want to read: https://docs.openstack.org/openstack-ansible/latest/reference/architecture/index.html13:26
evrardjpioni: I guess ppl using DVR : p13:27
kouroshvivanHello, we have a patch who brake gnocchi installation in master (see: https://bugs.launchpad.net/openstack-ansible/+bug/1778195) fix need some +2, my question is : If I want to backport in stable/pike and stable/queens, i need to cherry-pick the first commit then the fix ?13:29
openstackLaunchpad bug 1778195 in openstack-ansible "[ceilometer on gnocchi] install failed" [Undecided,In progress] - Assigned to kourosh vivan (kourosh-vivan)13:29
ionievrardjp for DVR is openvswitch required or works with linuxbridge as well?13:30
openstackgerritMohammed Naser proposed openstack/openstack-ansible master: Disable keepalived ping tests by default  https://review.openstack.org/58037913:30
kouroshvivandvr use ovs13:30
evrardjpioni: dvr is an ovs thing. Distributed routers works with lxb too, it's just a different thing.13:31
evrardjpioni: dvr is... meh.13:31
evrardjpif you can do it differently using lxb, do it.13:31
*** ansmith has joined #openstack-ansible13:32
ionii'll dig into the docs to see how to enable it with openstack-ansible13:32
openstackgerritMohammed Naser proposed openstack/openstack-ansible master: Disable keepalived ping tests by default  https://review.openstack.org/58037913:33
evrardjpioni: you could also do linux bridge on some of your compute nodes?13:34
evrardjpioni: dvr is hard to debug.13:35
ionievrardjp i'm currently all in linux bridge13:35
evrardjpioni: ok so you can keep it that way : )13:35
evrardjpioni: what's the issue?13:35
ionijust currious to see if there are issues with having network nodes on compute nodes13:35
evrardjpioni: not per se13:35
ionii just want to have routers mainly on them13:36
evrardjpioni: but I am not sure it's a good idea -- bad neighbours on one compute node would have an impact on all your cloud networking13:36
ionidhcp and bgp agents can still be on lxc13:36
evrardjpthat sounds like a bad design to me.13:36
evrardjpioni: I guess it all depends on your nic setup and everything13:37
kouroshvivanI have network node = compute node, no issues13:37
evrardjpthere is no technical issue for running compute and network nodes at the same place.13:37
evrardjpfor OSA at least13:37
evrardjpafter that, what you do, is architecting for your solution : )13:37
evrardjpkouroshvivan: that's fine for me to do a combined backport if you prefer. But you can also backport separately.13:38
evrardjpMake sure your patches have the information on the cherry pick.13:38
evrardjpif you do a combined backport it also helps on explaining WHY you are doing it.13:39
evrardjpelse you can maybe file a bug13:39
evrardjpand reference to it13:39
evrardjpkouroshvivan: it seems you already have your bug open,so you can just reference to it :p13:39
kouroshvivanevardjp: maybe my question is stupid but can i propose a change directly to stable branch ?13:42
evrardjpI am not sure to understand what you imply13:42
evrardjpif a fix need backporting, go from upper branch (master) and then propose the cherry picks on lower branches.13:43
openstackgerritMohammed Naser proposed openstack/openstack-ansible master: Disable keepalived ping tests by default  https://review.openstack.org/58037913:43
evrardjpif the patch doesn't apply to master, skip that branch, but apply the same reasoning.13:43
toskyand never merge in 'branch-1' if it's not merged in 'branch' is not merg13:43
toskyor stable maintainers will complain :)13:44
kouroshvivanso only cherry pick from master can go to stable branch ?13:44
evrardjpkouroshvivan: we accept re-implementations if it is well explained why it needed re-implementing.13:44
evrardjp(impossible to cherry pick without bringing features for example)13:45
evrardjpI guess it all depends on the patch : )13:45
evrardjpbut the idea is there13:45
evrardjpwe respect standard openstack behaviours.13:45
kouroshvivanok nicest way is combined backport, how to do it ?13:45
mnaserodyssey4me (and other cores): https://review.openstack.org/#/c/579941 and https://review.openstack.org/#/c/579944 are ready and passing and should clean things up in the role and unbreak centos gate13:46
evrardjpon your environement, you apply both patches into the same commir13:46
evrardjpcommit13:46
*** cshen has quit IRC13:46
evrardjpmaking sure the cherry-picked from ... message is there.13:46
odyssey4memnaser: I'll trade you :) https://review.openstack.org/#/q/topic:move-mq-setup-into-roles+(status:open+OR+status:merged)13:47
evrardjpmnaser: isn't cinder moving to privsep?13:47
*** tux___ has quit IRC13:47
mnaserevrardjp: all projects will eventually move to that but until they do we need to make sure we have the right rootwrap config files13:48
mnaserif we don't, then tests dont pass :<13:48
mnaserodyssey4me: let me see13:48
evrardjpmnaser: FYI, we are carrying those files because 1) we modify/modified them 2) Jesse's patches to ship them inside packages weren't merged13:48
evrardjpmnaser: if they are unmodified vs upstream, I am fine to not carry those : )13:48
odyssey4meevrardjp: we don't modify the rootwrap filters at all - we used to13:49
evrardjpodyssey4me: "modified"13:49
evrardjp:p13:49
odyssey4mealso, almost every single project now has them built into the venv :)13:49
evrardjpodyssey4me: that's point 213:49
evrardjpppl should read my sentences until the end : p13:50
evrardjpor maybe I should just be clearer, I have to think about it.13:50
mnaserodyssey4me: i like the idea of syncing that file13:51
mnaseri really don't want to add more work but is there any reason why we didnt go the role path?13:51
odyssey4me"go to the role path" ?13:52
mnaseras in, create a role called.. "openstack_mq" or "oslo_messaging"13:52
mnaserand do an include_role rather than include a list of tasks13:52
odyssey4memnaser: because it's not worth the extra weight - with this we can use the proposal bot to keep them in sync13:53
odyssey4meif that task list grows, which is might, then it might be worth splitting out13:53
evrardjpmnaser: if there is no variables defaults, no need for a role13:53
mnaseryeah i'm totally okay with that too.  plus include_role is slowwwww13:53
evrardjpmnaser: and buggy.13:53
*** cshen has joined #openstack-ansible13:53
mnaseryeah cloudnull apparently found out that `private` didnt really mean what it is13:54
evrardjpmnaser: well it never really worked13:54
evrardjpas far as I am aware13:54
mnaserit was just a prank13:54
evrardjpmnaser: what did cloudnull discover though?13:54
evrardjpI am not aware of this13:54
evrardjpmnaser: I reviewed https://review.openstack.org/#/c/579944 could you explain the WHY of the change to use synchronize and delegation?13:55
odyssey4meevrardjp: looks to me like this is the only tweak being done to a rootwrap filter, which may not even be relevant any more: https://github.com/openstack/openstack-ansible/blob/master/scripts/sources-branch-updater-lib.sh#L172-L17613:55
evrardjpodyssey4me: thanks for checking.13:55
mnaseri dont know the details but they found some ansible bug related use of prifvate13:55
evrardjpodyssey4me: we should clean up that script then13:55
evrardjpodyssey4me: yeah that's what I remember with the path : p13:55
odyssey4meevrardjp mnaser basically the private implementation was merged into docs, but not into code - and is still not in code13:56
evrardjpodyssey4me: but I think there was some with the exec dir though?13:56
evrardjpodyssey4me: haha13:56
evrardjpok13:56
odyssey4meinclude_role is currently "private: yes" as it stands, whereas "roles: foo" if "private: no" as it stands13:56
evrardjpI don't understand that sentence.13:57
odyssey4meie if you use include_role, any facts set in the role do not leave the role13:57
evrardjpmmm13:57
odyssey4meif you use the older roles mechanism, the facts set in the role are available after the role execution in the same play13:57
evrardjpnot so sure about that : p13:57
evrardjpbut ok13:57
evrardjpif you say so13:57
evrardjpI will try just to be sure13:57
odyssey4mewell, that's what the bug said13:57
evrardjpok13:58
mnaserevrardjp: copy with remote_src does not work because i cant "copy all files inside folder" and i dont want to add a list of files there13:58
odyssey4meI haven't validated13:58
mnasercause if i add a list13:58
mnaserthen we might run into the same silly issue13:58
guilhermespthat's why I spent a lot of time adapting the tests becasue the fake feature of include_role13:58
mnaserif a project decides to add new file13:58
odyssey4mesynchronize is better because it will also remove files ;)13:58
mnaseryup so effectively cleaning things up13:58
mnaserso the state will be clean13:58
guilhermespunfortunatly the os_monasca will not use it as the other roles13:59
guilhermespmnaser: can you show me the ansible bug? I think it was the one that I commented13:59
evrardjpmnaser: I am not against the change (while not really FOR it) -- I am against the fact it's not documented we've changed minds.13:59
mnaserguilhermesp: i dont know the details much, i just saw talk here in the channel13:59
mnaserevrardjp: can you tell me what you need to get a +214:00
mnaseri want to fix centos.14:00
evrardjpmnaser: we should say it's because we want to ensure folder contents, not only copying the files.14:00
mnaserOK. so change the commit message?!14:00
evrardjp(including deletion if need be)14:00
evrardjpmnaser: I can change it if you like.14:00
evrardjpyou prefer that way?14:00
mnaserno, ill change it, ask for odyssey4me to +2 again and wait for the whole check/gate dance14:00
mnaserevrardjp: if you want to edit it now14:01
mnaseri'd appreciate it14:01
mnaserand then i'll try to get more reviews and chase it through the gate.14:01
evrardjptrust me 6 months from now when ansible synchronize and delegation will fail, we'll check at this and all the others.14:01
evrardjphahaha14:01
guilhermespmnaser: https://github.com/ansible/ansible/issues/2189014:02
guilhermespFYI14:02
mnaserpretty stable api, dont think it's going to be breaking seriously anytime soon on such a common usecase14:02
evrardjpmnaser: ok14:02
evrardjpI am editing14:02
odyssey4meevrardjp: by then I'm hoping we won't need to bother with all that, because source builds will just use a symlink from /etc/<service> to /openstack/venv/<service>-<tag>/etc/<service>14:02
odyssey4meor something to that effect14:02
odyssey4meinstead of copying things around, we just use symlinks to the venv contents - this'll enable roll-back for minor upgrades or other changes14:03
odyssey4mebut hey, I'm getting ahead of myself14:03
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-os_cinder master: Use rootwrap configuration files from the virtualenv  https://review.openstack.org/57994414:04
mnaserodyssey4me: ^14:05
kouroshvivancan I put two #bug in Closes-Bug ? If yes what is the separator ?14:05
evrardjpkouroshvivan: yes14:05
evrardjptwo lines14:05
mnaserkouroshvivan: you put two Closes-Bug lines14:05
kouroshvivanthanks14:05
mnaserCloses-Bug: #1 <enter> Closes-Bug: #214:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Enable CentOS 7 distro jobs  https://review.openstack.org/57994514:05
*** vnogin has quit IRC14:06
*** vnogin has joined #openstack-ansible14:06
mnaserthanks odyssey4me14:06
evrardjpmnaser: if you still believe the ansible api and implementation is stable accross versions, I will give you a euro next time we meet.14:06
mnaserand evrardjp14:06
evrardjp(or a beverage of your choice)14:06
evrardjpmnaser: the problem is that synchronize and delegation was poorly tested.14:06
odyssey4mehahaha14:06
odyssey4meit's getting better14:06
mnaserbtw14:06
evrardjpodyssey4me: yeah due to massive testing, starting by gundalow and co14:07
mnaseryeah i agree i think its a progress14:07
mnaseri dont have time to look into this right now14:07
mnaserbut bionic + rabbitmq seems to fail14:07
evrardjpI just can't give 100% certainty that in 6 months it's still gonna work like it is now : p14:07
evrardjpanyway.14:07
mnaserso if someone can dig into that it will help with some of the other rabbitmq project efforts14:07
evrardjpI am done fighting.14:07
evrardjpmnaser: link?14:08
evrardjpalso I thought the copy module was recursive14:09
odyssey4mehttps://media.giphy.com/media/3xC2k2iGO40Io/giphy.gif14:09
evrardjpI will double check.14:09
mnaserhttp://logs.openstack.org/54/575954/5/check/openstack-ansible-functional-ubuntu-bionic/f194719/logs/ara-report/14:09
mnaserany bionic job14:09
openstackgerritMerged openstack/openstack-ansible-tests stable/pike: Pin get-pip.py to 3.2  https://review.openstack.org/58013814:09
*** Bhujay has quit IRC14:09
evrardjpodyssey4me: I don't get it , the pythons have hurt your head too much : p14:10
*** tux_ has joined #openstack-ansible14:10
evrardjpodyssey4me: what's the reference?14:10
evrardjpformy culture14:10
odyssey4meevrardjp: the ppl stare at him because he doesn't look like a newt... so he says 'I got better.'14:10
odyssey4meit's like ansible :p14:10
evrardjpmnaser: thanks.14:10
openstackgerritkourosh vivan proposed openstack/openstack-ansible-os_ceilometer stable/pike: Adds ceilometer-upgrade task  https://review.openstack.org/58039014:11
evrardjpodyssey4me: hahaha14:11
openstackgerritkourosh vivan proposed openstack/openstack-ansible-os_ceilometer stable/queens: Adds ceilometer-upgrade task  https://review.openstack.org/58039114:12
cloudnullmornings14:16
*** cjloader has joined #openstack-ansible14:20
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible master: Be consistent with memcached group name  https://review.openstack.org/58039414:23
openstackgerritMerged openstack/openstack-ansible-tests stable/ocata: Pin get-pip.py to 3.2  https://review.openstack.org/58013914:24
openstackgerritMerged openstack/openstack-ansible-tests stable/queens: Pin get-pip.py to 3.2  https://review.openstack.org/58013714:24
evrardjpmorning cloudnull14:24
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Pin get-pip.py to 3.2  https://review.openstack.org/58039814:29
*** armaan has quit IRC14:29
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/pike: Pin get-pip.py to 3.2  https://review.openstack.org/58039914:32
jrossercloudnull: with the elk stuff do the container journals somehow end up on the host?14:41
openstackgerritMerged openstack/openstack-ansible-os_cinder master: Copy rootwrap.d config files in source installs only  https://review.openstack.org/57994114:45
openstackgerritMerged openstack/openstack-ansible stable/pike: Add example host confd file for barbican  https://review.openstack.org/57928114:46
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/ocata: Pin get-pip.py to 3.2  https://review.openstack.org/58040214:46
*** ansmith has quit IRC14:48
odyssey4memnaser evrardjp hwoarang oh, by the way - it seems that xenial is the officially tested platform for openstack in general for ubuntu for rocky, and it's most likely that they'll only start removing xenial in stein: http://lists.openstack.org/pipermail/openstack-dev/2018-June/131194.html14:52
odyssey4meso we can add bionic support during rocky and we should be fine14:53
evrardjpodyssey4me: that's what we are doing, and what we decided? : )14:55
odyssey4meevrardjp: yep, but it's nice to know that we aren't an outlier this time14:55
evrardjphaha14:55
evrardjpyeah14:55
evrardjpI'd need your help figuring something out odyssey4me14:55
odyssey4meevrardjp sure, I have a meeting for an hour in 5 ins, then can help out14:56
odyssey4me*mins14:56
evrardjpok14:56
cloudnulljrosser they should end up on the host however that may need tweaking.15:06
jrossercloudnull: hmm yes i think i'm missing journal logs on the host from in the containers15:08
*** armaan has joined #openstack-ansible15:08
hwoarangodyssey4me: but but will there be packages?15:09
jrossercloudnull: i recall you looking at journal forwarding, but what do you think the best approach is15:10
* hwoarang afk15:10
cloudnullwe probably just need to see what's missing in the base setup.15:11
cloudnullI suspect I simply overlooked a step15:12
jrosseryes, i wasnt sure if it was expected that they all went to the host, or if journalbeat needs to land everywhere15:12
cloudnullI think in an older deployment (starting with newton) it might make some sense to install journalbeat everywhere.15:16
cloudnullI'd have to go look but  I believe the container journals are on the host in queens +15:16
*** chyka has joined #openstack-ansible15:17
*** fghaas has quit IRC15:20
openstackgerritMerged openstack/openstack-ansible-os_glance master: Move MQ vhost/user creation into role  https://review.openstack.org/56851715:21
*** fghaas has joined #openstack-ansible15:21
jrossercloudnull: if i use lsof on the journalbeat process i see it only has journals open for the host machine-id15:23
cloudnulldo you see the containers machine-id's in /var/log/journal  ?15:24
jrosseryes i've got a whole bunch of dirs in there15:24
jrosseralthough they dont appear to line up with /etc/machine-id from inside the containers15:26
openstackgerritjacky06 proposed openstack/openstack-ansible-ops master: Pin get-pip.py to 3.2  https://review.openstack.org/58040715:26
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-os_keystone master: Ensure __pycache__ folder is deleted  https://review.openstack.org/58040815:30
*** gyee has joined #openstack-ansible15:31
openstackgerritjacky06 proposed openstack/openstack-ansible-repo_build master: Pin get-pip.py to 3.2  https://review.openstack.org/58040915:31
*** epalper has quit IRC15:35
*** radeks has quit IRC15:35
*** srf_ has joined #openstack-ansible15:36
srf_hello everyone, can I ask, when I install packstack I have some problem in "Availability Zone", the error say :"There are no Availability Zones."15:36
srf_whether this person has experienced like this and how to overcome it, thx15:36
srf_the error i found in packstack15:37
*** kouroshvivan has quit IRC15:38
evrardjpsrf_: well , we don't use packstack to deploy so I doubt we can be very helpful15:47
evrardjpmaybe someone in #openstack or #openstack-dev can help15:47
evrardjpI can't at least.15:47
*** dxiri has joined #openstack-ansible15:51
*** fghaas has quit IRC15:51
srf_evrardjp : okay thx you, but ya, i ask too in #openstack and #openstack-dev, no problem :)15:52
*** radeks has joined #openstack-ansible15:55
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-ops master: Collect journals from containers as well as the host  https://review.openstack.org/58043315:57
jrossercloudnull: ^^ becasue of https://github.com/mheese/journalbeat/blob/master/beater/journalbeat.go#L59-L8315:58
cloudnulljrosser nice!15:59
jrosserslightly eccentric behaviour in the config file there16:00
openstackgerritjacky06 proposed openstack/openstack-ansible master: Pin get-pip.py to 3.2  https://review.openstack.org/58043616:00
*** cjloader_ has joined #openstack-ansible16:02
*** cjloader has quit IRC16:02
*** udesale has quit IRC16:04
*** cshen has quit IRC16:05
*** fghaas has joined #openstack-ansible16:06
*** cjloader_ has quit IRC16:08
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible master: Update all SHAs for milestone 3  https://review.openstack.org/57400616:08
*** srf_ has quit IRC16:08
*** yolanda_ has joined #openstack-ansible16:13
evrardjpcloudnull: I am happy if you submit patches for https://review.openstack.org/#/c/574006/7/playbooks/defaults/repo_packages/openstack_services.yml but you're breaking the things at the same time16:14
evrardjpI don't know how you bump things16:14
*** lvdombrkr89 has quit IRC16:14
evrardjpbut it doesn't respect the comments # FROZEN16:15
cloudnullwhat?16:15
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible master: Update all SHAs for milestone 3  https://review.openstack.org/57400616:15
evrardjpcloudnull: I've fixed it ^16:15
evrardjpyou can check the diff16:15
cloudnulloh, what is #FROZEN ?16:16
*** cjloader has joined #openstack-ansible16:16
cloudnullI just ran the script16:16
evrardjpnetworking-bgpvpn and networking-odl16:16
*** suggestable has quit IRC16:16
evrardjpwhich script?16:16
*** yolanda has quit IRC16:17
evrardjpI haven't merged my ugly spaghetti with what's inside openstack/openstack-ansible (and in fact, we should remove that code)16:17
cloudnullhttps://github.com/openstack/openstack-ansible/blob/master/scripts/sources-branch-updater.sh16:18
evrardjpwe have to refactor releases tooling -- as we now release differently.16:18
cloudnull ah.16:18
evrardjpand there are these cases that are not handled in that script.16:18
cloudnullok.16:18
evrardjpwe could also fix things upstream, but I haven't got the chance. I hope mbuil would eventually do it16:20
evrardjpbut I need to remember him16:20
*** radeks has quit IRC16:22
cloudnullok16:23
evrardjpso cloudnull who will you support in the World cup now?16:24
evrardjpstopped watching it?16:24
cloudnullfrance and belgium!16:24
evrardjpfriday!16:24
cloudnullindeed!16:25
evrardjplet's hope for the miracle for the 6PM UTC match, if you see what I mean ;)16:26
*** cjloader_ has joined #openstack-ansible16:27
*** cjloader has quit IRC16:27
*** radeks has joined #openstack-ansible16:30
guilhermespnoooo brazil on friday evrardjp cloudnull you two against me hahaha16:39
evrardjphaha guilhermesp :)16:40
openstackgerritMerged openstack/openstack-ansible master: Install gnocchi before ceilometer  https://review.openstack.org/57737916:45
*** armaan_ has joined #openstack-ansible16:45
*** armaan has quit IRC16:45
jrossercloudnull: whilst that patch opens all the journal files something is still wrong16:46
jrosserthe host journal dir names are not matching all of my container machine-id's16:47
jrosserand the majority of them are stale in some way16:47
*** armaan_ has quit IRC16:50
*** cjloader_ has quit IRC16:51
*** cjloader has joined #openstack-ansible16:53
*** cjloader has quit IRC16:53
*** yolanda__ has joined #openstack-ansible16:53
*** cjloader has joined #openstack-ansible16:53
*** yolanda_ has quit IRC16:56
*** suggestable has joined #openstack-ansible16:59
*** suggestable has quit IRC17:07
openstackgerritMerged openstack/openstack-ansible-ops master: Collect journals from containers as well as the host  https://review.openstack.org/58043317:09
*** yolanda_ has joined #openstack-ansible17:12
*** yolanda__ has quit IRC17:15
*** ansmith has joined #openstack-ansible17:28
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-os_tempest master: tasks: tempest_install: Add conditional for PIP package installation  https://review.openstack.org/58045517:28
openstackgerritGerman Eichberger proposed openstack/openstack-ansible-os_octavia master: Fix quota for Octavia  https://review.openstack.org/57136817:40
*** cshen has joined #openstack-ansible17:58
*** mattinator has joined #openstack-ansible18:01
*** mattinator has quit IRC18:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Sync task files to roles  https://review.openstack.org/57952718:06
odyssey4mehwoarang could you take a peek at https://review.openstack.org/579527 to validate whether it will work and do the right thing?18:07
*** poopcat has joined #openstack-ansible18:09
*** yolanda__ has joined #openstack-ansible18:13
*** mattinator has joined #openstack-ansible18:15
*** yolanda_ has quit IRC18:15
*** vnogin has quit IRC18:16
openstackgerritMerged openstack/openstack-ansible-tests master: Move MQ vhost/user creation into role (glance)  https://review.openstack.org/56852218:21
*** armaan has joined #openstack-ansible18:32
*** yolanda_ has joined #openstack-ansible18:35
*** DanyC has quit IRC18:38
*** yolanda__ has quit IRC18:39
*** chris__ has joined #openstack-ansible18:39
*** dxiri has quit IRC18:39
*** yolanda__ has joined #openstack-ansible18:39
chris__Hey Guys, does anyone know how to modify the console_token_ttl expiration time in openstack-ansible queens?18:40
*** yolanda_ has quit IRC18:42
*** armaan has quit IRC18:44
*** armaan has joined #openstack-ansible18:45
*** vnogin has joined #openstack-ansible18:47
*** armaan has quit IRC18:49
*** vnogin has quit IRC18:52
*** cshen has quit IRC19:00
*** tomtom002 has joined #openstack-ansible19:05
*** dxiri has joined #openstack-ansible19:07
*** tomtom001 has quit IRC19:08
*** cmart has quit IRC19:21
*** Chealion has quit IRC19:21
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible master: On container destroy remove container journal  https://review.openstack.org/58047419:23
cloudnulljrosser  ^19:24
*** Chealion has joined #openstack-ansible19:24
openstackgerritMerged openstack/openstack-ansible master: Move MQ vhost/user creation into role (glance)  https://review.openstack.org/56853119:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Ensure remote_user and become are consistently implemented  https://review.openstack.org/58015619:27
*** radeks has quit IRC19:27
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cloudkitty master: Execute service setup against a delegated host using Ansible built-in modules  https://review.openstack.org/57997719:29
*** gkadam has quit IRC19:32
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_congress master: Execute service setup against a delegated host using Ansible built-in modules  https://review.openstack.org/57998119:34
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-ops master: Use openstack_version to determine Journalbeat hosts  https://review.openstack.org/58047619:34
jrossercloudnull: nice, will try that. one back for you ^19:34
*** cshen has joined #openstack-ansible19:35
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_barbican master: Execute service setup against a delegated host using Ansible built-in modules  https://review.openstack.org/57994719:36
cloudnulljrosser giving that a spin now19:39
jrossersame, mines just running now19:40
openstackgerritGuilherme  Steinmuller Pimentel proposed openstack/openstack-ansible-os_monasca master: [WIP] Fix monasca  https://review.openstack.org/57872119:40
*** cshen has quit IRC19:40
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_horizon master: Execute service setup against a delegated host using Ansible built-in modules  https://review.openstack.org/57999119:41
cloudnulljrosser it works! however I think we'll need to create a pre-playbook to dynamically create the group19:46
cloudnullI had to run `openstack-ansible /tmp/test.yml -e openstack_release=17`19:46
cloudnullto make it detect the openstack_release variable19:46
*** cmart has joined #openstack-ansible19:47
jrosserah did you see what i put in the readme?19:51
*** poopcat has quit IRC19:51
jrosserif you can think of a better way than that then cool19:51
*** poopcat has joined #openstack-ansible19:52
*** poopcat has quit IRC19:54
cloudnullhttp://paste.openstack.org/show/725142/20:00
cloudnullThat picks up the embedded variable when coupled along with osa20:01
cloudnullthen i changed teh hosts: section in the playbook to use all_journalbeat_deployments20:01
jrossercloudnull: i'll give that a go20:03
cloudnullhttp://paste.openstack.org/show/725143/20:04
cloudnullthat was the whole playbook20:04
jrosserhow does it get openstack_release?20:05
*** vnogin has joined #openstack-ansible20:07
cloudnullThe group vars are exposed to the tasks of a playbook.20:08
jrossercloudnull: the version is in here /opt/openstack-ansible/inventory/group_vars/all/all.yml and i dont think that is included?20:08
cloudnullIt was when running with `openstack-ansible`20:08
cloudnullWith the embedded ansible it would not be.20:09
cloudnullSo the option would still be required.20:09
cloudnullIn that case.20:09
jrosserah hmm right because for me the embedded is the way to get 2.520:10
cloudnull++ I'm running with Rocky at the moment.20:11
cloudnullSo i think we still need the blerb in the readme to ensure all edge cases are accounted for.20:12
*** poopcat has joined #openstack-ansible20:12
openstackgerritGerman Eichberger proposed openstack/openstack-ansible-os_octavia master: Fix quota for Octavia  https://review.openstack.org/57136820:12
odyssey4mecloudnull: https://media.giphy.com/media/DwIdasRkFKsMg/giphy.gif20:12
cloudnull? Lol20:13
odyssey4me:) running rocky, not really something that jrosser can afford to do :p20:13
odyssey4meright, time for me to !computer - cheers all20:14
cloudnulltake care odyssey4me20:25
*** poopcat1 has joined #openstack-ansible20:40
*** throwsb1 has quit IRC20:41
*** poopcat has quit IRC20:42
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-ops master: Use openstack_version to determine Journalbeat hosts  https://review.openstack.org/58047620:48
jrossercloudnull: ^ your change works for me20:49
jrosseri do notice that TASK [Enable and restart journalbeat] is quick on my containers and exceedingly slow on the hosts20:50
openstackgerritMerged openstack/openstack-ansible-os_cinder master: Use rootwrap configuration files from the virtualenv  https://review.openstack.org/57994421:06
*** albertcard2 has joined #openstack-ansible21:10
*** poopcat1 has quit IRC21:13
*** albertcard2 is now known as poopcat21:16
*** poopcat is now known as Guest7277621:17
*** cjloader_ has joined #openstack-ansible21:18
*** cjloader has quit IRC21:18
*** cjloader_ has quit IRC21:18
*** cjloader has joined #openstack-ansible21:18
*** Guest72776 has quit IRC21:19
*** poopcat1 has joined #openstack-ansible21:19
mnaseryay21:20
mnaserthat merged21:20
mnaseri think this will finally make centos metal GREEN21:21
*** armaan has joined #openstack-ansible21:23
jrossercloudnull: this journal collection stuff is all missing tags21:25
jrosseri wonder if taking the first part of {foo}.slice as the tag is enough21:26
*** tux_ has quit IRC21:28
cloudnulljrosser ? odd that container would be quick and slow on the host ?21:28
jrosseryes i don't understand that21:29
cloudnullare you seeing missing field tags or just tags in general  ?21:29
jrosseralthough we've done little with ansible 2.5 and i'm expecting the usual surprises21:29
jrosserfor things that are collected from the journal they're not tagged in the same way as the log files are21:30
jrosserso no "tags       openstack, nova, beats_input_codec_plain_applied"21:31
cloudnullah yes. thats largely coming from the logstash groks21:31
jrosserso it's sort of there21:31
jrosserin the systemd slice name21:32
cloudnullI wonder if we could extend those groks to do that  ?21:32
jrosseri was wondering if that was an easy way to do the same21:32
jrosserbut it does highlight some missing slice names21:32
*** cjloader has quit IRC21:33
*** cshen has joined #openstack-ansible21:36
*** cshen has quit IRC21:41
cloudnullhum21:41
cloudnulllooking at the journalbeat data and the config file IDK if we can apply that same logic there.21:42
*** ivve has quit IRC21:55
*** vnogin has quit IRC21:55
*** vnogin has joined #openstack-ansible22:04
*** chris__ has quit IRC22:04
*** cmart has quit IRC22:09
*** albertcard2 has joined #openstack-ansible22:10
*** poopcat1 has quit IRC22:13
*** nicolasbock has quit IRC22:17
*** fghaas has quit IRC22:23
*** hachi has quit IRC22:30
cloudnulljrosser looks like we need to change the service units to have `--use-journal`22:35
cloudnullhttp://paste.openstack.org/show/725160/22:36
*** armaan has quit IRC22:36
cloudnullwhich requires `apt install pkg-config` and `/openstack/venvs/nova-17.0.5/bin/pip install  systemd-python`22:36
*** armaan has joined #openstack-ansible22:37
cloudnullI think I can make this all part of the repo server, its something that I will need to look into regardless, especially if the services are not actually loogging to the journal22:37
*** armaan has quit IRC22:41
openstackgerritMerged openstack/openstack-ansible stable/queens: Pin get-pip.py to 3.2  https://review.openstack.org/58039822:47
*** chyka has quit IRC22:57
*** tosky has quit IRC23:01
*** ianychoi_ has joined #openstack-ansible23:01
*** ianychoi has quit IRC23:04
*** yolanda_ has joined #openstack-ansible23:07
*** yolanda__ has quit IRC23:08
*** yolanda__ has joined #openstack-ansible23:11
*** yolanda_ has quit IRC23:13
*** vnogin has quit IRC23:15
*** cshen has joined #openstack-ansible23:37
*** yolanda_ has joined #openstack-ansible23:40
*** poopcat1 has joined #openstack-ansible23:40
*** cshen has quit IRC23:42
*** yolanda__ has quit IRC23:42
*** albertcard2 has quit IRC23:43

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