Thursday, 2019-09-19

*** brokencycle has quit IRC00:19
*** gyee has quit IRC00:22
*** markvoelker has joined #openstack-ansible00:25
*** mpjetta has quit IRC00:27
*** markvoelker has quit IRC00:29
*** dave-mccowan has quit IRC00:47
*** macz has quit IRC00:52
*** dave-mccowan has joined #openstack-ansible00:52
*** markvoelker has joined #openstack-ansible01:00
cjloaderhttps://review.opendev.org/#/c/682446/ and https://review.opendev.org/#/c/680284/ passed gates (Ironic packages/Ironic Inspector) noonedeadpunk jrosser please look in the morning =)02:13
*** markvoelker has quit IRC02:47
*** BjoernT has joined #openstack-ansible02:51
*** nicolasbock has quit IRC02:52
*** BjoernT_ has joined #openstack-ansible02:54
*** BjoernT has quit IRC02:56
*** udesale has joined #openstack-ansible04:04
*** vakuznet has quit IRC04:16
*** pcaruana has joined #openstack-ansible04:42
*** gillesMo has quit IRC05:07
openstackgerritMerged openstack/openstack-ansible-os_ironic master: Remove UCS packages from os_ironic  https://review.opendev.org/68244605:08
cjloaderty d34dh0r53 for pushing it through05:09
cjloadernow need to focus on my inspector patch noonedeadpunk jrosser05:09
cjloaderits passed gates05:14
*** radeks has joined #openstack-ansible05:28
ChosSimbaOneGood morning. So finally got the lb working. Had to purge my containers as it added the bond interrface to the container, and f'ed conectivity to the host :-)05:38
ChosSimbaOneWould you guys recommend an dedicated neutron node on metal for queens? This is for a pilot deployment of openstack with max 10 compute nodes. Connectivity is 2x25Gbit, so I was thinking it could offload the rest of the service on the infra node by having a dedicated neutron server?05:40
*** BjoernT_ has quit IRC05:45
jrosserChosSimbaOne: queens? really? :)06:00
ChosSimbaOnejrosser: Yeah, well long story short, we would have integrated with ACI and our network consultant told us that the ACI module from cisco was only supported in Queens so that is why :-)06:01
jrosserthis is why i have avoided any kind of vendor magic in networking completely06:02
jrossercisco constantly try to get us doing ACI for stuff and we just say no every time06:03
ChosSimbaOneYeah, I am starting to regread that decision as well, but to late now :-S06:03
ChosSimbaOneI wish we had that kind of will power :-)06:03
jrosserwell anyway, i have dedicated network nodes, and did that back on queens too06:04
jrosserso it can be done and does work06:04
jrosseri would strongly suggest doing "whatever it takes" to make the any interface names you have to specify to neutron to be uniform across your compute and network nodes06:05
ChosSimbaOneOkay super. Well i guess it should just be connected to the br-mgmt and br-vxlan right? and the set the is_metal: true in an env.d file right?06:08
jrosserthe neutron agent is on metal anyway for queens afaik?06:13
jrosserimho env.d should be fiddled to put the neutron api server on the infra node and the l3 agent only on the network node06:14
ChosSimbaOneCool I will dig into that, and let you know how it goes :-)06:17
*** kopecmartin|off is now known as kopecmartin06:18
*** prometheanfire has quit IRC06:23
*** prometheanfire has joined #openstack-ansible06:24
*** markvoelker has joined #openstack-ansible06:27
*** markvoelker has quit IRC06:31
*** cshen has joined #openstack-ansible06:52
*** udesale has quit IRC06:57
*** udesale has joined #openstack-ansible06:58
*** arbrandes has quit IRC07:00
*** arbrandes has joined #openstack-ansible07:01
*** rpittau|afk is now known as rpittau07:05
cshenGood morning. Does anybody deep know about rpc_conn_pool_size used in neutron.conf and nova.conf? Is this parameter shared with all rpc workers, or each worker has its own conn pool?07:15
*** udesale has quit IRC07:18
*** udesale has joined #openstack-ansible07:19
*** tosky has joined #openstack-ansible07:23
*** luksky has joined #openstack-ansible07:37
*** ivve has joined #openstack-ansible07:45
*** kaiokmo has quit IRC07:57
*** zbr is now known as zbr|ruck07:57
*** kaiokmo has joined #openstack-ansible08:02
*** arxcruz|ruck is now known as arxcruz|rover08:20
*** markvoelker has joined #openstack-ansible08:28
*** markvoelker has quit IRC08:33
*** kaiokmo has quit IRC08:36
*** kaiokmo has joined #openstack-ansible08:37
CeeMacmorning08:38
jrosserCeeMac: morning..... do i remember right you had dedicated network nodes and moved the neutron API off them?08:40
CeeMachi jrosser08:47
CeeMaccorrect08:47
jrossercould you share what you did to env.d?08:47
CeeMaclet me poke in my notes08:47
CeeMacits been a while and i've been distracted with non-osa stuff (boo)08:47
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_barbican master: Add global override for service bind address  https://review.opendev.org/68306208:49
CeeMacChosSimbaOne: jrosser  http://paste.openstack.org/show/777611/08:56
CeeMacthats what I needed to do to move neutron-server from network nodes to infra nodes08:57
jrosserexcellent thanks, i'm going to do the same08:57
CeeMac:)08:58
ChosSimbaOneCeeMac: Thank you so much! This is a really great help :-)08:58
ChosSimbaOneWill try and deploy right away.08:59
CeeMacsure thing, good luck :)08:59
ChosSimbaOneSo is the neutron service setup on directly on the infra host normally or is everything in the container neutron_server?08:59
CeeMaccontainer for neutron_server on infra08:59
CeeMacneutron agents on bare metal on compute nodes for me09:00
CeeMaci'm on rocky though, not sure if that makes a difference09:00
CeeMacalso, if you run into issues with network and compute nodes having differnet physical interface names there is a workaround for that, although I don't know if that works in queens09:01
jrosseri think that was only added later09:01
CeeMacyeah, think it was backported to rocky09:01
jrosserbut you can work around it by adding neutron.conf overrides to group_vars/<things> rather than the documented setup in openstack_user_config->provider_networks09:02
jrosserbut that gets messy09:02
jrosserChosSimbaOne: with the standard setup if you make a dedicated network node the neutron API container and the L3 agent will run on the network node09:03
jrosseri want to move the API container back to the infra nodes09:03
ChosSimbaOneOkay, but if i have already run the osa playbooks where my network host was the infra node, do i need to do some cleanup or should purging the container be enough.09:04
ChosSimbaOneGod advice with the api. Will see if can get it to work :-)09:06
jrosserChosSimbaOne: there are two parts09:09
jrosserthere is an API part and the agent09:09
jrosserthe agent is making your routers and dhcp servers for project networks09:10
ChosSimbaOnejrosser: Okay. But without a dedicated network host it all runs in the neutron_server_container?09:17
ChosSimbaOneCeeMac: the opendalight is only relevant if I have the opendaylight SDN right?09:19
jrosserno, the l3 agent is on bare metal, that used to be containerised but at some point around P/Q that was changed09:19
jrosserand the neutron API runs in the container09:19
ChosSimbaOneokay, now I follow :) Thank you for the clarificaton.09:21
CeeMacChosSimbaOne: where were you seeing the opendaylight?09:27
CeeMacactually, i see it now09:27
CeeMacthat was in the original skel so i just left it there to keep things straight09:27
CeeMacyou could try leaving it out if you wanted09:28
ChosSimbaOneOKay, super :-)09:28
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_cinder master: Add global override for service bind address  https://review.opendev.org/68307409:28
noonedeadpunkmornings09:29
ChosSimbaOneCeeMac: Super, will try and see if my world explodes :-)09:30
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_blazar master: Add global override for service bind address  https://review.opendev.org/68307509:30
noonedeadpunkits high time upgrade jobs got reviewed:) https://review.opendev.org/#/c/671105/09:30
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_congress master: Add global override for service bind address  https://review.opendev.org/68307609:32
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_horizon master: Allow to configure horizon bind address  https://review.opendev.org/68171809:32
noonedeadpunkjrosser: ^ updated horizon patch to use openstack_service_bind_address09:33
jrossernoonedeadpunk: hi09:33
noonedeadpunkbut I'm not sure right now what's the idea - will veriable with container address work properly with lxc?09:33
jrossermy plan is to put openstack_service_bind_address in group_vars/all.yml09:34
jrosseropenstack_service_bind_address: "{{ management_address }}"09:34
noonedeadpunkyeah, ok, I guess this should work then09:34
noonedeadpunkI think we'll need to heavily update our docs during next cycle:)09:36
jrosseryes09:36
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_glance master: Add global override for service bind address  https://review.opendev.org/68308009:41
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_gnocchi master: Add global override for service bind address  https://review.opendev.org/68308209:43
*** jawad_axd has joined #openstack-ansible09:44
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_heat master: Add global override for service bind address  https://review.opendev.org/68308409:45
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Add global override for service bind address  https://review.opendev.org/68308609:47
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_keystone master: Add global override for service bind address  https://review.opendev.org/68308909:52
*** udesale has quit IRC10:01
*** cshen has quit IRC10:09
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_swift master: Revert "Update fallocate_reserve to work with py3"  https://review.opendev.org/68310210:13
*** cshen has joined #openstack-ansible10:25
*** markvoelker has joined #openstack-ansible10:29
*** cshen has quit IRC10:31
*** udesale has joined #openstack-ansible10:32
*** cshen has joined #openstack-ansible10:33
*** markvoelker has quit IRC10:34
*** trident has quit IRC10:35
*** udesale has quit IRC10:37
*** udesale has joined #openstack-ansible10:37
*** cshen has quit IRC10:37
*** cshen has joined #openstack-ansible10:38
*** trident has joined #openstack-ansible10:48
*** pcaruana has quit IRC10:54
*** rpittau is now known as rpittau|afk10:56
*** udesale has quit IRC11:01
*** nicolasbock has joined #openstack-ansible11:06
jrossernoonedeadpunk: we need this https://review.opendev.org/#/c/682830/11:10
jawad_axdHi! Can anyone suggest what is wrong with these settings http://paste.openstack.org/show/777691/ ? This is scenario when you have different network interfaces names on compute nodes and you need to define them for neutron_provider_networks.Would appreciate your help in this.11:10
noonedeadpunkjrosser: sure, I saw it (and even thought that voted on it)11:11
noonedeadpunkor decided to postpone reviews for later time:)11:12
noonedeadpunkbtw, jrosser, do you know how to get delegated host group_vars during delegate_to?11:12
jrosserjawad_axd: i think the issue might be that you are trying to set the same var in more than one place, and user_variables.yml is the highest priority and will win over everything else11:13
jrosserit is passed as a -e parameter to ansible-playbook which is highest precedence11:13
jrossernoonedeadpunk: group_vars specifically? i think the only thing is hostvars[<thing>] ?11:14
jawad_axdI need to define separate network_mappings variable for compute and network node. How can I do that?11:14
jrosserjawad_axd: which release?11:15
jawad_axdstein11:15
jawad_axdcentos711:15
noonedeadpunkyep, I'm trying to achieve step 8 from https://docs.openstack.org/murano/latest/install/from-source.html#install-the-api-service-and-engine11:16
noonedeadpunkby delegating to utility host, which obviously supposed top have python-muranoclient11:16
*** pcaruana has joined #openstack-ansible11:17
noonedeadpunkbut, I need utility_venv_bin, which is defined here https://opendev.org/openstack/openstack-ansible/src/branch/master/inventory/group_vars/utility_all.yml11:17
noonedeadpunkhostvars are empty for some reason:(11:18
jrosserjawad_axd: there is excellent docuemtnation by jamesdenton here https://docs.openstack.org/openstack-ansible/latest/user/prod/provnet_groups.html11:18
jrosserjawad_axd: you can do that in stein11:18
*** ansmith has quit IRC11:21
jrossernoonedeadpunk: i wonder if instead it is easier to do that from the murano_api[0] (or whatever it is called) host?11:22
noonedeadpunkyeah, probably it's easier:)11:22
jrosserthen you use the murano venv, and you have (maybe?) all those files that need uploading already there?11:22
noonedeadpunkbut utiity is probably more porper place for such operations...11:22
jrosserperhaps delegate_to: "{{ murano_service_bootstrap_host }}"11:22
jrosserwhere do you make the zip file?11:23
noonedeadpunkI was thinking about utility11:23
noonedeadpunkso, I need to clone repo, do archive, and run murano client11:24
jrosserdoes all the zip file content get installed with the service?11:24
noonedeadpunkbut yep, the easier thing to do this on murano host11:24
jrosserwhat actually is it btw in the zip?11:25
noonedeadpunknope, they don't install it, which is pretty ridiculous11:25
jrosserhah11:25
jawad_axdjrosser: It helped. Thanks11:26
noonedeadpunkso, it's some internal core packages without which murano can't init any environment11:26
noonedeadpunkthe most dissapointing thing, that they ship some of them http://paste.openstack.org/show/777696/11:27
noonedeadpunkbut not the essential one11:27
noonedeadpunkwhich is io.murano.Environment11:27
*** macz has joined #openstack-ansible11:28
noonedeadpunkso yep, I'll probably do this on murano_api[0]11:30
noonedeadpunkif you don't mind:)11:30
jawad_axdjrosser: You guys are doing great work and must be busy with your priority work.I just want to know if some came up with fix for stein/centos7 heat-engine service issue http://paste.openstack.org/show/777695/ http://paste.openstack.org/show/777694/11:32
*** macz has quit IRC11:32
jrossernoonedeadpunk: also we use the deployment host a lot for this11:39
jrosserdownload to the deploy host11:39
jrosserzip up  / whatever11:40
jrossercopy to target, delegate to murano[0] and use the cli11:40
jrosserthats very similar to how we do things like HATop and is also friendly to environments where the compute nodes don't have much external connectivity, but the deploy host might11:41
noonedeadpunkit will slow down things because of upload - no sense in copying it as this can be done remotely....11:41
jrosseryou need to think of people who want near offline installs too11:41
noonedeadpunkthe still will need to set murano_git_repo11:42
noonedeadpunkwe don't build venvs on the deployment host, are we?11:43
noonedeadpunkand these files are part of the default murano_git_repo11:47
openstackgerritRalf Haferkamp proposed openstack/ansible-role-uwsgi master: Fix distro install on openSUSE  https://review.opendev.org/68312011:59
*** goldyfruit_ has quit IRC12:11
openstackgerritRalf Haferkamp proposed openstack/ansible-role-uwsgi master: Fix distro install on openSUSE  https://review.opendev.org/68312012:13
*** markvoelker has joined #openstack-ansible12:13
openstackgerritRalf Haferkamp proposed openstack/openstack-ansible-os_placement master: Fix distro install on openSUSE  https://review.opendev.org/68312312:16
*** vakuznet has joined #openstack-ansible12:16
openstackgerritAndrew Gibb proposed openstack/openstack-ansible-os_trove master: Add global override for service bind address  https://review.opendev.org/68312912:25
*** mgariepy has quit IRC12:37
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add global override for service bind address  https://review.opendev.org/68313312:43
*** ansmith has joined #openstack-ansible12:44
openstackgerritAndrew Gibb proposed openstack/openstack-ansible-os_tacker master: Add global override for service bind address  https://review.opendev.org/68313412:44
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_manila master: Add global override for service bind address  https://review.opendev.org/68313612:46
*** markvoelker has quit IRC12:46
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_masakari master: Add global override for service bind address  https://review.opendev.org/68313812:48
openstackgerritAndrew Gibb proposed openstack/openstack-ansible-os_sahara master: Add global override for service bind address  https://review.opendev.org/68313912:49
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_mistral master: Add global override for service bind address  https://review.opendev.org/68314012:49
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Add global override for service bind address  https://review.opendev.org/68314112:50
openstackgerritAndrew Gibb proposed openstack/openstack-ansible-os_placement master: Add global override for service bind address  https://review.opendev.org/68314212:53
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_nova master: Add global override for service bind address  https://review.opendev.org/68314312:54
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_octavia master: Add global override for service bind address  https://review.opendev.org/68314412:55
openstackgerritAndrew Gibb proposed openstack/openstack-ansible-os_panko master: Add global override for service bind address  https://review.opendev.org/68314512:56
jrosserjawad_axd: how did you configure it to log to systemd? in stein the default is to log to files afaik13:07
*** SmearedBeard has joined #openstack-ansible13:08
jawad_axdI did set use_journal = True in /etc/heat/heat.conf.13:08
jawad_axdI have two stein setup both are using 19.0.2 tag. Both setups have same issue with heat engine service.13:12
jrosserthis will be becasue there is nowhere in stein that the systemd-python library is installed for heat, i expect13:13
jrosserin master / train we switched all the logging to systemd so these things would have been fixed on that branch13:14
jrosserjawad_axd: here is the patch in master https://github.com/openstack/openstack-ansible-os_heat/commit/88c29bb66547d791c0035f688d5097a7a7632fd713:14
jrossernoonedeadpunk: i am tracking the bind-to-mgmt stuff a bit here https://etherpad.openstack.org/p/osa-bind-to-mgmt13:16
jrosserthe roles with missing patches don't have an obvious place in defaults/main.yml that the bind address is set13:16
jrosserthey need a bit more investigation13:16
*** goldyfruit_ has joined #openstack-ansible13:16
noonedeadpunkok, will have a look at them13:18
jawad_axdjrosser: Thanks. I am looking into it. I am improving my python skills, would love to contribute in coming time.13:18
jrossernoonedeadpunk: i'm going to -W the existing patches for now and make a DNM to test them all in the integrated repo13:19
jrosserjawad_axd: contributions are always welcome :)13:19
*** macz has joined #openstack-ansible13:21
*** udesale has joined #openstack-ansible13:23
*** udesale has quit IRC13:24
*** udesale has joined #openstack-ansible13:25
*** macz has quit IRC13:26
*** pcaruana has quit IRC13:28
jawad_axdjrosser: That patch worked !!! Thumbs up man. Heat-engine service got started properly after patch.13:40
*** jawad_axd has quit IRC13:51
*** jawad_axd has joined #openstack-ansible13:53
*** pcaruana has joined #openstack-ansible13:54
*** jawad_axd has quit IRC13:57
*** BjoernT has joined #openstack-ansible13:59
*** BjoernT_ has joined #openstack-ansible14:02
*** macz has joined #openstack-ansible14:03
*** BjoernT has quit IRC14:04
*** macz has quit IRC14:07
*** ivve has quit IRC14:09
*** goldyfruit___ has joined #openstack-ansible14:15
*** goldyfruit_ has quit IRC14:18
*** macz has joined #openstack-ansible14:19
*** macz has quit IRC14:23
*** pcaruana has quit IRC14:33
*** SmearedBeard has quit IRC14:33
*** spatel has joined #openstack-ansible14:33
*** jawad_axd has joined #openstack-ansible14:40
*** jawad_axd has quit IRC14:44
*** macz has joined #openstack-ansible14:45
mnaserjrosser: did you use my tool to generate those patches by any chance :D14:48
jrosserctrl-r14:49
mnaseraww14:49
jrossertype fast :)14:49
mnasersorry i cant keep up14:49
jrosserno worries14:50
*** macz has quit IRC14:50
jrosserwe’re still churning stuff out to finish various bits14:50
jrosserbut that’s soon going to turn into needing to merge it all which might need a few more bodies14:51
*** gyee has joined #openstack-ansible14:56
*** tosky has quit IRC14:57
*** tosky has joined #openstack-ansible14:59
mnaserive personally been pretty backed up with a lot of stuff going on until ansiblefest15:00
*** gillesMo has joined #openstack-ansible15:00
mnaseri think i'll be able to breathe a bit more when it comes time for that15:00
*** gillesMo has quit IRC15:00
*** cshen has quit IRC15:11
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible master: Add support to deploy murano  https://review.opendev.org/67949615:14
*** gillesMo has joined #openstack-ansible15:20
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_murano master: Initial commit to os_murano role  https://review.opendev.org/67783015:32
*** kopecmartin is now known as kopecmartin|off15:35
*** jawad_axd has joined #openstack-ansible15:42
*** jawad_axd has quit IRC15:47
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_designate master: Add global override for service bind address  https://review.opendev.org/68319215:50
cjloadermorning15:50
*** mgoddard has quit IRC15:58
openstackgerritRalf Haferkamp proposed openstack/openstack-ansible-os_cinder master: Fix distro install on openSUSE  https://review.opendev.org/68319516:00
openstackgerritRalf Haferkamp proposed openstack/openstack-ansible-os_tempest master: Fix distro install on openSUSE  https://review.opendev.org/68319716:05
*** openstackgerrit has quit IRC16:06
*** macz has joined #openstack-ansible16:10
*** macz has quit IRC16:15
*** markvoelker has joined #openstack-ansible16:15
cjloaderso noonedeadpunk with your venv_build_vars comment, do I add each of the vars?16:20
noonedeadpunkyeah, it was kind of the idea. SO the thing is, that we want to use import in main.yml for tags to be working16:22
noonedeadpunkSo we can't use with_items there16:22
cjloaderthats why I had include16:22
cjloaderbut I understand now16:23
*** openstackgerrit has joined #openstack-ansible16:25
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_nova master: Only clone console repos on source based installs  https://review.opendev.org/66646316:25
*** macz has joined #openstack-ansible16:25
*** udesale has quit IRC16:27
*** udesale has joined #openstack-ansible16:28
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_nova master: [redhat] Update variables to be version agnostic  https://review.opendev.org/65825916:37
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_neutron master: [redhat] Update variables to be version agnostic  https://review.opendev.org/65826116:40
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_glance master: Turn the filesystem directory to be configurable  https://review.opendev.org/61392416:45
*** udesale has quit IRC16:52
*** mbuil has quit IRC16:59
*** mbuil has joined #openstack-ansible16:59
*** pcaruana has joined #openstack-ansible17:01
*** markvoelker has quit IRC17:01
*** markvoelker has joined #openstack-ansible17:04
openstackgerritCam J. Loader (cjloader) proposed openstack/openstack-ansible-os_ironic master: Adds configuration for the Ironic Inspector  https://review.opendev.org/68028417:05
cjloadernoonedeadpunk: like that?17:05
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_neutron master: Implement uWSGI for neutron-api  https://review.opendev.org/48615617:06
*** zbr|ruck is now known as zbr17:07
noonedeadpunkcjloader: I'd say you can do this with single update17:08
odyssey4menoonedeadpunk: apologies for jumping into your conversation, but I noticed you mention import vs include, and I've done some stuff around that recently17:08
noonedeadpunkodyssey4me: you're always welcome:)17:08
odyssey4meit turns out that include_tasks/include_role will work with tags, but only if the include_tasks/include_role task is tagged with the always tag17:09
cjloaderhi odyssey4me17:09
odyssey4meo/ cjloader noonedeadpunk :)17:09
noonedeadpunkyeah, you're right, always trigger it.17:09
noonedeadpunkBut I guess specific tags are valuable as well.17:09
odyssey4methere is some detail here: https://docs.ansible.com/ansible/latest/porting_guides/porting_guide_2.5.html#dynamic-includes-and-attribute-inheritance17:10
cjloaderand noonedeadpunk I updated all the things you asked me to17:10
odyssey4meit's not well described though17:10
*** newbie has joined #openstack-ansible17:10
odyssey4meyou can also apply tags to the included tasks using the 'apply' key word: https://docs.ansible.com/ansible/latest/modules/include_tasks_module.html17:11
noonedeadpunkodyssey4me: oh, you mean that we should just place tag inside every task of the included file?17:11
odyssey4mewell, it depends on what you want17:11
odyssey4meif you want all tasks inside an included task file to be tagged the same, then use the apply key word on the include task17:12
odyssey4mebut make sure to also tag the include task 'always' so that the tags are useful17:12
odyssey4meif you can point me at a review and explain what you want, I can comment in review how to achieve it :)17:13
cjloaderhttps://review.opendev.org/#/c/680284/17:13
noonedeadpunkyeah, i see. Thanks for sharing that17:13
cjloadertasks/main.yml17:13
noonedeadpunkodyssey4me: actually cjloader already made import intead:)17:13
*** pcaruana has quit IRC17:14
odyssey4meif there's no conditional, or no need for dynamic - then use import17:14
noonedeadpunkI think imports are cleaner anyway...17:14
cjloaderthere was a conditional17:14
cjloaderbut I removed/ used import17:14
newbiehey guys, I'm running linuxbridge agent and recently upgraded from Rocky to stein.  I'm finding that the linuxbridgeagent running on my neutron nodes (the ones running on copute nodes are fine) are dying and getting privsep errors.17:14
noonedeadpunkodyssey4me: I think conditionals might be aplied. There's restrictions for iterations and variables inside task path17:15
newbied getting privsep errors.17:15
newbie Specifically getting an error that says it could't drop privileges and one Value Error where max_bin_len was exceeded17:15
odyssey4meok, so what happens with a conditional on an import is that the conditional gets applied to every task in the task file or role17:15
odyssey4methat means slower processing17:15
odyssey4meif you want it faster, then use include - if you need to use tags inside the task file or role, then apply the always tag to the include task17:16
noonedeadpunkthan we need to move everything back to includes:(17:16
odyssey4mewell, do what works for now - but perhaps it's worth an experiment to validate... no-one needs slow processing ;)17:16
cjloaderi originally had include17:17
odyssey4meI only sat to figure it all out recently, and in our situation we *had* to use include because there was a loop for the set of tasks.17:17
noonedeadpunkYeah, than you don't have another choice17:17
cjloaderbut just switched to import because of the looping17:18
noonedeadpunkcjloader: in my opinion - let's leave things as is for now17:18
odyssey4mecjloader: import won't allow a loop17:18
cjloaderyea17:18
cjloaderthats why I had incclude17:18
odyssey4meyou'd have to loop inside the imported task set, rather than loop from the outside17:19
noonedeadpunkIf we will decide to speedup things - we'll do migrations across all roles17:19
*** newbie has quit IRC17:19
cjloadernoonedeadpunk: ok17:19
noonedeadpunkjust to be consisten for now in used things17:19
*** newbie has joined #openstack-ansible17:19
cjloaderthx odyssey4me for your advice17:20
noonedeadpunkbut yeah, thanks odyssey4me for sharing this17:20
odyssey4meI think I may just write up a blog post for this, because it's a bit obscure.17:20
noonedeadpunkyour input as always very valuable17:20
noonedeadpunkbtw, since you're here, what do you think about placement migration ? I guess it works now, but now sure how good things I'm doing for this https://review.opendev.org/#/c/664867/17:21
cjloadernoonedeadpunk: do my recent changes address your comments?17:21
noonedeadpunkactually it's your's patch:)17:21
noonedeadpunkcjloader: yess17:21
odyssey4menoonedeadpunk: oh yay, thanks for resurrecting it17:22
cjloaderas for OSA depends... I'm going to iterate later on that... i'm just trying to get my role into OSA noonedeadpunk17:22
noonedeadpunkodyssey4me: related change actually make use of it and tests upgrade path17:23
odyssey4menoonedeadpunk: whoah! https://review.opendev.org/#/c/671105/33 :)17:24
noonedeadpunkyeah, there was lots of rebases:)17:24
cjloaderodyssey4me: ty for that catch, was remnant from include17:24
*** pcaruana has joined #openstack-ansible17:25
openstackgerritCam J. Loader (cjloader) proposed openstack/openstack-ansible-os_ironic master: Adds configuration for the Ironic Inspector  https://review.opendev.org/68028417:25
odyssey4menoonedeadpunk: well, I can't really comment on the patch since I haven't really followed all the changes in the past 6-9 months - but it seems to work at least for the metal jobs, which is a great start!17:26
odyssey4mebetter than nothing, that's for sure17:26
noonedeadpunkfor lxc there's a problem with timeouts:(17:26
cjloaderodyssey4me: my patch is fairly hefty lol17:27
noonedeadpunkand for lxc kinda 30% of things might be dropped out...17:27
noonedeadpunkcjloader: the second one is optional17:29
noonedeadpunkbut qwuotes around dict keys are really missing17:30
cjloaderok17:34
*** jawad_axd has joined #openstack-ansible17:37
newbieok, so I was able to sort my linuxbridge agent issue.  I had to downgrade msgpack...not sure if this was a requirements issue or if did something worng at some point17:40
openstackgerritCam J. Loader (cjloader) proposed openstack/openstack-ansible-os_ironic master: Adds configuration for the Ironic Inspector  https://review.opendev.org/68028417:45
cjloadernoonedeadpunk: addressed17:45
*** pcaruana has quit IRC17:55
*** ansmith_ has joined #openstack-ansible17:58
*** ansmith has quit IRC18:00
openstackgerritMerged openstack/openstack-ansible-os_nova stable/stein: Restore nova-key tag  https://review.opendev.org/68125318:03
*** jawad_axd has quit IRC18:21
*** trident has quit IRC18:45
*** macz has quit IRC18:47
*** newbie has quit IRC18:51
*** radeks has quit IRC18:51
*** trident has joined #openstack-ansible18:55
*** trident has quit IRC19:01
*** cshen has joined #openstack-ansible19:05
*** trident has joined #openstack-ansible19:10
*** cshen has quit IRC19:19
*** ansmith_ has quit IRC19:25
jrossercan we get another vote on this? https://review.opendev.org/#/c/682830/19:26
jrosserand another here https://review.opendev.org/#/c/681610/19:30
*** cshen has joined #openstack-ansible19:30
jrosserand another here https://review.opendev.org/#/c/681724/19:31
*** ansmith_ has joined #openstack-ansible19:31
cjloadersimple patches jrosser19:32
jrosseri'm looking for +2 +W :)19:32
cjloadermake me core lol19:32
cjloaderjk19:32
*** cshen has quit IRC19:36
openstackgerritCam J. Loader (cjloader) proposed openstack/openstack-ansible-os_ironic master: Adds configuration for the Ironic Inspector  https://review.opendev.org/68028419:37
cjloaderjrosser: can you take another glance at my patch?19:41
*** macz has joined #openstack-ansible19:49
openstackgerritMerged openstack/openstack-ansible-os_octavia master: Create mgmt subnet with gateway if provided  https://review.opendev.org/68164219:50
*** pcaruana has joined #openstack-ansible19:51
*** pcaruana has quit IRC20:31
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: [WIP] Bind services to mgmt network addresses  https://review.opendev.org/67005120:34
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: [WIP] Bind services to mgmt network addresses  https://review.opendev.org/67005120:34
*** ansmith_ has quit IRC20:34
guilhermespjrosser: I guees we will need to update all roles? ^20:37
guilhermespall right I can see a burst of patches here https://review.opendev.org/#/q/topic:bind-to-mgmt+(status:open+OR+status:merged)20:37
jrosseryeah i did them all today, execpt those where there isnt a var in defaults20:38
jrosserheres where we are https://etherpad.openstack.org/p/osa-bind-to-mgmt20:38
guilhermespoh nice jrosser20:39
jrosserguilhermesp: if you are able to review those 3 patches i mentioned up there ^^ just earlier it would unblock some other stuff20:40
guilhermesp /me looks20:40
guilhermespups20:40
jrosserguilhermesp: so my plan was that we have openstack_service_bind_address for openstack-ansible-os_*20:40
guilhermespanyways, lookign20:40
jrosserand then we deal with the infrastructure things as needed in group_vars20:41
jrosserwhich lets you do the thing for galera however is needed eventually20:41
*** sshnaidm is now known as sshnaidm|off20:42
guilhermespall right reviews done jrosser20:42
jrosserthankyou20:42
guilhermespthanks for the work with mgmt stuff, I will come back to galera changes asap20:42
jrossercool - i think this is a really nice thing to get merged20:43
openstackgerritMerged openstack/openstack-ansible-os_octavia master: Octavia use barbican internalURL  https://review.opendev.org/67887120:45
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_placement master: Add global override for service bind address  https://review.opendev.org/68314220:45
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Move provider_networks into os_neutron  https://review.opendev.org/65812620:49
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Move provider_networks into os_neutron  https://review.opendev.org/65812620:49
openstackgerritCam J. Loader (cjloader) proposed openstack/openstack-ansible-os_ironic master: Adds configuration for the Ironic Inspector  https://review.opendev.org/68028421:16
openstackgerritMerged openstack/openstack-ansible-os_nova stable/stein: tasks: add conditionals for ironic  https://review.opendev.org/67387921:19
*** markvoelker has quit IRC21:20
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible master: Add support to deploy murano  https://review.opendev.org/67949621:24
*** ansmith_ has joined #openstack-ansible21:34
*** luksky has quit IRC21:41
*** markvoelker has joined #openstack-ansible21:48
*** BjoernT_ has quit IRC22:00
*** weifan has joined #openstack-ansible22:09
*** weifan has quit IRC22:09
*** macz has quit IRC22:25
*** markvoelker has quit IRC22:32
openstackgerritMerged openstack/openstack-ansible-os_neutron master: Mark calico jobs as non voting due to python3 bug  https://review.opendev.org/68283022:39
*** spatel has quit IRC22:41
openstackgerritCam J. Loader (cjloader) proposed openstack/openstack-ansible-os_ironic master: Adds configuration for the Ironic Inspector  https://review.opendev.org/68028422:42
openstackgerritMerged openstack/openstack-ansible-os_nova master: Only clone console repos on source based installs  https://review.opendev.org/66646322:46
openstackgerritMerged openstack/openstack-ansible-os_masakari master: Add uwsgi role to role requirements  https://review.opendev.org/68172422:48
openstackgerritMerged openstack/openstack-ansible-os_nova master: [redhat] Update variables to be version agnostic  https://review.opendev.org/65825922:49
*** macz has joined #openstack-ansible22:52
*** macz has quit IRC22:54
*** macz has joined #openstack-ansible22:54
*** goldyfruit___ has quit IRC23:00
*** tosky has quit IRC23:12
*** ondrejburian has quit IRC23:23
*** kaiokmo has quit IRC23:23
*** ondrejburian has joined #openstack-ansible23:24
*** macz has quit IRC23:54

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