Monday, 2018-11-05

*** ravs has quit IRC00:02
*** cshen has joined #openstack-ansible00:17
*** cshen has quit IRC00:22
*** ravs has joined #openstack-ansible00:22
ravsHello everyone, Currently testing OSA Rocky, I'm having an issue while running the "setup-openstack" playbook during the task "[os_ceilometer : Copy ceilometer configuration files]", the same is returning this: 'fatal:... "neutron_oslomsg_notify_host_group' is undefined" pretty much the same issue that "reineruhry" reported on last Monday...  Does anyone here how to fix this? Thanks in advance!00:36
*** mbi has quit IRC00:36
*** DanyC has joined #openstack-ansible00:42
*** nurdie has joined #openstack-ansible01:16
*** nurdie has quit IRC01:20
*** nurdie has joined #openstack-ansible02:04
*** nurdie has quit IRC02:15
*** cshen has joined #openstack-ansible02:18
*** cshen has quit IRC02:23
*** ravs has quit IRC02:35
*** nurdie has joined #openstack-ansible02:36
*** nurdie_ has joined #openstack-ansible02:41
*** nurdie_ has quit IRC02:42
*** nurdie has quit IRC02:45
*** nurdie has joined #openstack-ansible02:50
*** nurdie_ has joined #openstack-ansible02:51
*** nurdie has quit IRC02:54
*** nurdie_ has quit IRC03:07
*** nurdie has joined #openstack-ansible03:08
*** nurdie has quit IRC03:12
*** nurdie has joined #openstack-ansible03:33
*** nurdie has quit IRC03:37
*** nurdie has joined #openstack-ansible03:53
*** mino_ has joined #openstack-ansible03:58
*** dave-mccowan has quit IRC04:00
*** nurdie has quit IRC04:00
*** mrda has joined #openstack-ansible04:07
*** udesale has joined #openstack-ansible04:31
*** nurdie has joined #openstack-ansible04:36
*** nurdie has quit IRC04:41
openstackgerritJimmy McCrory proposed openstack/openstack-ansible-galera_server master: Remove static option from include_tasks  https://review.openstack.org/61547104:49
*** pcaruana has joined #openstack-ansible05:23
*** pcaruana has quit IRC05:32
*** ivve has joined #openstack-ansible05:33
*** cshen has joined #openstack-ansible05:51
*** cshen has quit IRC05:58
*** faizy98 has joined #openstack-ansible06:05
*** cshen has joined #openstack-ansible06:18
*** fghaas has joined #openstack-ansible06:22
*** cshen has quit IRC06:23
*** fghaas has quit IRC06:27
*** chkumar|off is now known as chandankumar06:41
openstackgerritAndriy Shevchenko proposed openstack/ansible-role-systemd_service master: fix tox python3 overrides  https://review.openstack.org/60680806:42
openstackgerritAndriy Shevchenko proposed openstack/ansible-role-systemd_networkd master: fix tox python3 overrides  https://review.openstack.org/60676306:45
openstackgerritAndriy Shevchenko proposed openstack/ansible-role-systemd_mount master: fix tox python3 overrides  https://review.openstack.org/60676406:46
openstackgerritAndriy Shevchenko proposed openstack/ansible-config_template master: fix tox python3 overrides  https://review.openstack.org/60676606:47
*** jbadiapa has joined #openstack-ansible07:11
*** evrardjp_ is now known as evrardjp07:15
evrardjpgood morning everyone07:16
*** DanyC has quit IRC07:18
*** DanyC has joined #openstack-ansible07:19
*** cshen has joined #openstack-ansible07:22
evrardjpwow I was disconnected from gerrit this morning07:29
evrardjpIt's been quite a good week-off :D07:30
jrosserMorning evrardjp07:30
evrardjpmorning jrosser :)07:31
evrardjpI didn't dare opening my emails yet07:31
jrosserhttps://review.openstack.org/#/c/615443/ and https://review.openstack.org/#/c/615310/ would be a good way to free up some brokenness on a Monday morning. :)07:33
evrardjpthanks jrosser07:34
evrardjpwill have a look right away07:34
evrardjpwhat did I miss last week?07:34
jrosserI awarded opensuse a prize07:35
jrosserIt took the honor of first job to succeed with mitogen07:35
evrardjpthanks jmccrory :D07:36
evrardjpjrosser: oh nice07:36
evrardjpbare metal or with containers?07:36
evrardjphow much did you shave at the end?07:36
jrosserMetal, the container connection still needs some work07:37
jrosserDown to 40mins07:37
evrardjpfor the 40 minutes: in gates, or in your machine ?07:39
evrardjpif 40 minutes in your machine that might not be a huge win07:39
jrossergate, on a vexxhost box07:39
jrosserLooks like about 30% improvement in runtime, and I believe there is still more to be had.07:40
jrosserWe should be able to “stack” the strategy plugins so we get a combination of the mitogen work and the osa optimisation got skipped tasks, for example.07:41
evrardjpyeah.07:43
evrardjpWe can optionally be using an env var to override if we want to be conservative07:43
openstackgerritAndriy Shevchenko proposed openstack/openstack-ansible-galera_server master: fix tox python3 overrides  https://review.openstack.org/61549007:43
*** DanyC has quit IRC07:48
openstackgerritAndriy Shevchenko proposed openstack/openstack-ansible-galera_server master: fix tox python3 overrides  https://review.openstack.org/61549008:06
*** pcaruana has joined #openstack-ansible08:06
*** fghaas has joined #openstack-ansible08:16
openstackgerritMerged openstack/openstack-ansible-os_cinder master: Add libxml2 to cinder_distro_packages lists  https://review.openstack.org/61544308:32
*** gkadam_ has joined #openstack-ansible08:44
evrardjpfor those who aren't aware of it yet: https://review.openstack.org/#/c/613856/09:18
*** DanyC has joined #openstack-ansible09:24
*** DanyC has quit IRC09:31
*** pcaruana has quit IRC09:31
*** pcaruana has joined #openstack-ansible09:32
*** DanyC has joined #openstack-ansible09:34
*** DanyC has quit IRC09:34
*** DanyC has joined #openstack-ansible09:35
*** DanyC has quit IRC09:39
fghaasodyssey4me: is there anything I can do about the bionic failures that appear to be blocking https://review.openstack.org/#/c/614194/ ?09:39
jrosserfghaas: you could try a recheck - this merged this morning https://review.openstack.org/615443 and the failure is cinder related09:43
fghaasjrosser: sure!09:43
jrosserif you want something to look at i was surprised that there were no cinder logs at all from that run http://logs.openstack.org/94/614194/4/check/openstack-ansible-deploy-aio_lxc-ubuntu-bionic/7536f4c/logs/openstack/aio1_cinder_api_container-d0fc163e/09:43
fghaasby the way I added you to the reviewers list on that rgw/Swift API bug because I figured you might be able to cry wolf if I did anything stupid, jrosser.09:44
fghaasI particular, I hope the reno is complete in describe the implications of the change.09:45
jrosser:) well, the best i can do is to try ity locally, i'm just trying to pick up loose ends from my Q->R upgrade and get refstack passing again09:45
fghaassure, no rush at all09:46
fghaass/describe/describing/09:46
dcdamienjrosser: can you please explain where can I find policy that mention about requirement to specifically mark cherrypicks in commit messages?09:51
jrosserdcdamien: there are some notes here https://review.openstack.org/Documentation/intro-user.html is you search for "cherry-pick"09:55
openstackgerritMerged openstack/openstack-ansible-galera_server master: Remove static option from include_tasks  https://review.openstack.org/61547109:58
jrosserdcdamien: here is a (totally random) example of a cherry picked commit with the message that gerrit inserted automatically when the "cherry pick" button in the UI was used https://review.openstack.org/#/c/601870/09:59
dcdamienjrosser: I'm trying to find a _rule_ that describe it but I can't find any. The fact that gerrit inserts does not mean that it is a requirement, right? I've done backports you've reviewed -1 manually and git doesn't added anything.10:05
*** electrofelix has joined #openstack-ansible10:06
*** nurdie has joined #openstack-ansible10:12
dcdamienjrosser: IMO this kind of reviews are unnecessary because they are just slowing down velocity of the project and willingness of the community. Change that is compliant with policy = good change. No rule = no complaint. Bad rule / no rule = policy update.10:14
*** nurdie has quit IRC10:17
*** shardy has joined #openstack-ansible10:21
*** mino_ has quit IRC10:35
*** fghaas has left #openstack-ansible10:43
*** fghaas has joined #openstack-ansible10:43
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build stable/rocky: Ensure pip/setuptools/wheel are up to date in the wheel build venv  https://review.openstack.org/61552510:49
odyssey4mejrosser fghaas yeah, the reason there's no log at all is because the systemd unit doesn't start - it fails the initial load10:56
odyssey4mewe should ideally work out improvements to the systemd unit files we use so that they fork, and we can switch them to forking rather than simple - then we'd get logs10:57
odyssey4meor figure out another way of achieving the same10:57
evrardjphey10:58
evrardjpdcdamien: isn't the cherry pick in the policy though?10:59
evrardjpdcdamien: I don't think the intent was negative in this case: It helps track why and how this was done. In this case, we have only a "related bug" , and it's easier to track when this was all started by having the cherry pick id.11:00
evrardjpdcdamien: the other idea is also to help track what needs manual cherry picking11:00
odyssey4medcdamien if it's not present and it was done through gerrit, then it was likely done before it merged in the source branch11:00
noonedeadpunkmorning folks11:00
odyssey4medcdamien jrosser also, the sdocumenting providing guidelines for backports is in https://docs.openstack.org/openstack-ansible/latest/contributor/contribute.html#backporting11:01
evrardjpin any case, your change is positive dcdamien :)11:01
noonedeadpunkodyssey4me: I've got one small question regarding new python_venv_build role. Do you have some time?11:02
*** cshen has quit IRC11:02
evrardjpmorning noonedeadpunk odyssey4me and others :)11:02
evrardjpdcdamien: thanks for your change ! :)11:02
odyssey4memorning all11:02
odyssey4menoonedeadpunk sure, still having my second cup of coffee while catching up to the weekend's activities11:03
odyssey4meapologies all for breaking the cinder role :/11:03
noonedeadpunkYestarday I was working on masakari role (in which you took part) https://review.openstack.org/#/c/584629/, and wrote another bunch of patches. Everything works ok, except one thing. I need libsystemd to be installed on venv_build_host, as it's required for wheels build11:05
evrardjp(**)it happens odyssey4me :)11:05
dcdamienevrardjp: Cool, this document indeed explains it. Thanks, this is what I'm looking for.11:06
evrardjpdcdamien: but we shouldn't reach a point where you feel bad about those reviews, that is NEVER an intent here.11:07
noonedeadpunkI really  tried but couldn't understand the correct way to specify condition, that installation should be run on masakari-monitor hosts and on the venv_build_host only. I've got smth like this https://review.openstack.org/#/c/584629/34/vars/main.yml, but it is still failing on suse. Adding venv_build_host == 'inventory_hostname' fixes suse, but it shouldmn't work, due to https://github.com/openstack/ansible-role-python_venv_11:07
noonedeadpunkbuild/blob/master/tasks/python_venv_wheel_build.yml#L19-L2011:07
noonedeadpunk*https://github.com/openstack/ansible-role-python_venv_build/blob/master/tasks/python_venv_wheel_build.yml#L19-L2011:07
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: RHEL: Move xlib-devel to cinder_devel_distro_packages  https://review.openstack.org/61552611:08
evrardjpdcdamien: if you feel we are picking nits, tell us and we'll improve. :)11:08
dcdamienevrardjp: okay :)11:08
* dcdamien is adding references to original commit SHAs11:08
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: RHEL: Move zlib-devel to cinder_devel_distro_packages  https://review.openstack.org/61552611:09
evrardjpdcdamien: I have a tendency to assign freely -1 myselfs, but I try to always explain why I am doing so. So don't feel bad if you see a -1, it's generally fixable very easily :)11:09
noonedeadpunkodyssey4me: so in other words the question is - what is the right condition to get packages filtered and installed for venv_build_host?11:10
jrosserdcdamien: apologies for not having that link to hand :/ we really value contributions and want to make openstack-ansible a welcoming and accessible place to collaborate........11:10
odyssey4menoonedeadpunk looking11:11
dcdamienjrosser: okay, I just wanted to say that clear (and easy to find) rules are really important. Especially for those who are not old stager :)11:13
odyssey4menoonedeadpunk ok, I've suggested some changes to one of the var sets - but the same applies to all distro var lists11:17
noonedeadpunkodyssey4me: ok, got it, will investigate now. thanks!11:18
dcdamienI'll be gratefull for reviewing that https://review.openstack.org/#/c/614556/11:20
noonedeadpunkodyssey4me: and the package installation task on the hosts is the separate one (not part of the python_venv_build role)?11:21
jrosseri think we broke tempest somewhere - there is a whole bunch of these popped up http://logs.openstack.org/36/614336/2/check/openstack-ansible-functional-distro_install-opensuse-150/2e75ad3/job-output.txt.gz#_2018-11-05_09_23_38_90657111:21
odyssey4menoonedeadpunk the python_venv_build role has two distro install vars - one for wheel builds and one for installs11:21
odyssey4meit actually has 3, but one should not be overwridden under normal circumstances: https://github.com/openstack/ansible-role-python_venv_build/blob/master/defaults/main.yml#L28-L3911:22
noonedeadpunkI just see, that during mentioned clenup all distro_installs were moved away11:22
odyssey4menoonedeadpunk that doesn't prevent another distro install task in the role - but using one should not be necessary in many cases11:24
odyssey4meand ideally we want to ensure we're only trying to install them once11:25
odyssey4mewhen building wheels we will install the build_base and build package list: https://github.com/openstack/ansible-role-python_venv_build/blob/master/tasks/python_venv_wheel_build.yml#L2411:26
odyssey4mewhen running through the install, we might install those - but only if the wheel build was done: https://github.com/openstack/ansible-role-python_venv_build/blob/master/tasks/python_venv_install.yml#L2311:26
noonedeadpunkodyssey4me: thanks for so detailed explanation. I think now I get the point11:28
*** francois has quit IRC11:28
*** cshen has joined #openstack-ansible11:30
*** cshen has quit IRC11:35
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_masakari master: Basic implementation of masakari-monitors  https://review.openstack.org/58462911:37
*** DanyC has joined #openstack-ansible11:39
guilhermespmorning ppl! Could I have someone else eyes here? https://review.openstack.org/#/c/615272/11:40
noonedeadpunkcores, can you please take a look at https://review.openstack.org/#/c/613977/ ?11:41
*** DanyC has quit IRC11:44
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-ceph_client master: SUSE: Fix ceph packages for Leap 15  https://review.openstack.org/61553511:49
odyssey4meguilhermesp a few small tweaks and that's good to go11:50
arxcruzodyssey4me: mnaser hey, regarding https://review.openstack.org/#/c/612968/ I believe it's ready to merge as initial support to python-tempestconf11:51
arxcruzI want you guys to take a look before I send a new patch setting the value to false, so the regular tempest.conf will be used11:51
arxcruzand also I have some questions11:51
arxcruzon tempest.conf template there are some settings hardcoded, and I wondering if that's really necessary, for example, do we need to use internalURL instead of publicURL in the endpoint types?11:52
odyssey4mearxcruz with the config_template engine those are easy to override, which is why they're hard-coded as such - see https://docs.openstack.org/openstack-ansible/latest/reference/configuration/using-overrides.html#overriding-openstack-configuration-defaults for how that's used11:53
odyssey4mearxcruz we use the internal/admin endpoints by default because in OSA those are non SSL by default, whereas in OSA the public endpoint is SSL using a self-signed cert... using this we've mmanaged to catch all sorts of bad client config and service config, and in the early days client SSL didn't work so good11:54
odyssey4mearxcruz all that said, if we can do something more sensible which makes it easier to consume for other folks, we can do that11:55
guilhermespthanks odyssey4me !11:55
arxcruzodyssey4me: okay, so, I check python-tempestconf tempest.conf output, and it's pretty different from the tempest.conf template in osa, but tests pass anyway11:56
arxcruzI would merge python-tempestconf patch right now, then I'll work on the override configs11:56
arxcruzof course I will keep the tempestconf option as false for now11:56
arxcruzand if you guys agree, add one more job that uses tempestconf11:56
odyssey4mearxcruz sure - ideally that should be done in the same patch so that we know it works - but if further changes are needed for that, then perhaps we rather get this in first - I'll go through review in a bit11:57
arxcruzodyssey4me: well, i wanted to split it, because it would be a big patch, and i'm not a big fan of big patches, it's hard later to identify a problem11:58
arxcruzI want incrementally add new functionalities11:58
odyssey4mearxcruz agreed11:58
odyssey4mearxcruz best to have a commit msg which explains this is partial and not yet working then, and that further work will come in a follow on patch11:59
arxcruzodyssey4me: sure, once you say it's good to go, i'll update the commit message and set the variable to use tempestconf to false11:59
odyssey4menoonedeadpunk do ceilometer services need to start in a particular order if they're on a shared host?12:03
*** dave-mccowan has joined #openstack-ansible12:04
openstackgerritGuilherme  Steinmuller Pimentel proposed openstack/openstack-ansible-os_placement master: Initial commit to os_placement role  https://review.openstack.org/61527212:05
noonedeadpunkodyssey4me: I don't thinks that thay have such dependencies. At least I never faced with such12:08
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-ceph_client master: SUSE: Fix ceph packages for Leap 15  https://review.openstack.org/61553512:09
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-ceph_client master: tests: Add glance_api group to test package installation  https://review.openstack.org/61554112:09
noonedeadpunkas polling agents are writing to rabbit, and notification agents are reading from it and sending data to gnocchi12:10
jrossernoonedeadpunk: do you know about the ceilometer/gnocchi stuff? My Rocky install is making tons of log errors related to those12:11
*** francois has joined #openstack-ansible12:11
noonedeadpunkjrosser: I'm facing a bug on rocky, when ceilometer-notification-agent stucks on volume creation (when volume created from image)12:13
jrossernoonedeadpunk: one thing i see a lot of is this http://paste.openstack.org/show/734136/12:14
noonedeadpunkI've partially fixed it by removing index [0] here https://github.com/openstack/ceilometer/blob/stable/rocky/ceilometer/data/meters.d/meters.yaml#L13112:14
*** udesale has quit IRC12:14
noonedeadpunkjrosser: no,. mine is different http://paste.openstack.org/show/733979/12:15
noonedeadpunkjrosser: probably you need to run ceilometer-upgrade?12:16
noonedeadpunkI think in rocky it should be run without any flags12:17
odyssey4mearxcruz reviewed - not much to change as far as I can see on my first pass12:18
jrossernoonedeadpunk: also i have 3 gnocchi nodes - only the first one has anything in /var/lib/gnocchi-tmp, like gnocchi-config and gnocchi-storage-config12:18
jrosserthe other two are really unhappy because those config files are missing, it's very unclear whey they come from though12:18
arxcruzodyssey4me: working on that12:19
jrossernoonedeadpunk: is  ceilometer-upgrade something that the osa ceilometer role should be doing?12:20
odyssey4mearxcruz added two more suggestions12:20
noonedeadpunkjrosser: I've deployed 2 gnocchi nodes with OSA and have no problems with them. Also I've added coordination between them via zookeeper12:20
arxcruzodyssey4me: i didn't get the pip list..12:21
noonedeadpunkAnd I don't have mentioned path on neither of them12:21
jrossernoonedeadpunk: it possibly depends which storage type it is using12:21
noonedeadpunkjrosser: probably. Mine is ceph with indexer on mysql12:22
*** ravirjn has joined #openstack-ansible12:22
odyssey4mearxcruz in the integrated build, for now at least, there's a lookup which finds all the vars named *_pip_packages and compiles them into a list of wheels to build - so any role that has a pip package should ideally have <feature>_pip_packages, like https://github.com/openstack/openstack-ansible-os_nova/blob/master/defaults/main.yml#L571-L59612:23
odyssey4meI'm working to be rid of it, but it's heavily embedded in how OSA works right now - so it's a slow moving set of changes.12:23
arxcruzodyssey4me: so add tempest_pip_packages in main, and get rid of the manual installation ?12:24
*** cshen has joined #openstack-ansible12:25
odyssey4mearxcruz nope, add a new var called something like tempest_conf_pip_packages in defaults/main - then use it in the python_venv_build task12:25
arxcruzodyssey4me: gotcha12:25
odyssey4mearxcruz with that you should also be able to get rid of the --isolated flag12:25
odyssey4mecores - we have some ansible updates to approve: https://review.openstack.org/#/q/topic:osa-update-ansible+status:open12:29
chandankumararxcruz: odyssey4me https://review.openstack.org/612968 python-tempestconf initial support needs a release notes na?12:30
odyssey4mechandankumar arxcruz yep, a small release note annoucing the new feature would be good too12:31
ravirjnI have an existing openstack ansible queens setup without cinder. Now I have to deploy cinder also, I modified openstack_user_config.yml and running playbook as openstack-ansible setup-hosts.yml --limit storage_hosts. At the ens of the output I am getting "skipped: no hosts matched" under ceate container. What am I missing? http://paste.openstack.org/show/734137/12:31
openstackgerritMerged openstack/openstack-ansible-lxc_container_create stable/rocky: Stop creating duplicate lines in lxc config files  https://review.openstack.org/61544712:32
*** jroll has quit IRC12:32
odyssey4meravirjn --limit lxc_hosts,cinder_all12:33
odyssey4methere are storage hosts, and containers on the infra hosts too12:33
chandankumararxcruz: odyssey4me I have one query about tempestconf patch: since tempestconf is going to discover and create resources when admin credentails are used, on non-admin just discovry and populating in tempest.conf12:34
*** jroll has joined #openstack-ansible12:34
chandankumararxcruz: odyssey4me but in os_tempest role https://review.openstack.org/#/c/612968/10/tasks/main.yml@37 we create resources can this be disabled when tempestconf will be used?12:34
odyssey4mechandankumar that's a good point, and might be necessary - I don't know exactly what the tools does, but if those resources will clash then the resource enablement should be disabled if tempestconf is enabled12:35
arxcruzchandankumar: where tempestconf create resources? right now is just reading the values12:36
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_masakari master: Basic implementation of masakari-monitors  https://review.openstack.org/58462912:36
*** ansmith has quit IRC12:37
chandankumararxcruz: for example tasks related to image https://github.com/openstack/openstack-ansible-os_tempest/blob/master/tasks/tempest_resources.yml#L34 -> these are already supported by tempestconf tool12:37
arxcruzchandankumar: okay, so, again, one thing at time12:38
arxcruzincremental work12:38
chandankumararxcruz: this is just a discussion12:38
arxcruzchandankumar: ok12:38
odyssey4mearxcruz perhaps it's a good time to update the commit msg with the intended follow on work ;)12:39
arxcruzodyssey4me: yup, on it12:39
chandankumararxcruz: tempestconf needs external network (but in this case, tempest_resources can be used)12:39
chandankumarwhat I am saying is that there are few stuff we need to use when tempestconf is used from tempest_resouces and few things needs to be disabled12:40
chandankumarI am not sure what is the best way to handle it12:40
openstackgerritArx Cruz proposed openstack/openstack-ansible-os_tempest master: Enable python-tempestconf support  https://review.openstack.org/61296812:42
arxcruzodyssey4me: ^12:42
arxcruzodyssey4me: I keep the use tempestconf to true just to get the results, then if everything is okay, i'll set it to false in a new patch and then you can review12:43
arxcruzsounds good ?12:43
odyssey4mearxcruz a couple more tweaks - then we verify that it passes the tests12:48
arxcruzodyssey4me: gotcha12:48
odyssey4mearxcruz oh, and the reno would be good to add now12:49
odyssey4mesometimes they need a bit of tweaking too12:49
arxcruzoh, yeah, forgot about that12:49
odyssey4mehttps://docs.openstack.org/openstack-ansible/latest/contributor/code-rules.html#release-notes12:49
arxcruzodyssey4me: btw, shall I add all environments to environment or just OS_PASSWORD?12:52
odyssey4mearxcruz the only one I'm worried about is the password - the rest are up to you12:53
odyssey4meI don't feel strongly either way12:53
arxcruzodyssey4me: i'll keep only os_password for now12:53
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_ceilometer master: Usage of systemd services is mapped to the host group  https://review.openstack.org/61397712:57
evrardjpglad to see openstack module changes merging in ansible :)13:06
openstackgerritArx Cruz proposed openstack/openstack-ansible-os_tempest master: Enable python-tempestconf support  https://review.openstack.org/61296813:06
arxcruzodyssey4me: done ^13:06
odyssey4meevrardjp yeah, there are so many bugs and PR's open that it's very noisy as a maintainer... I try to do a few a day to keep reducing the backlog, but I don't think I'm really keeping up :/13:16
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-ceph_client master: tests: Add nova_compute group to test package installation  https://review.openstack.org/61554113:24
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-ceph_client master: SUSE: Fix ceph packages for Leap 15  https://review.openstack.org/61553513:24
odyssey4mehwoarang fyi I'm still working through the suse 42 aio_metal build failures for rocky - I expect these fixes may also help centos pass13:34
hwoarangyeah look like they might be related13:34
odyssey4mehttps://review.openstack.org/615525 is one that can be reviewed so long13:34
openstackgerritArx Cruz proposed openstack/openstack-ansible-os_tempest master: Enable python-tempestconf support  https://review.openstack.org/61296813:40
jrosserodyssey4me: looks like that swift service service fix is only part of the trouble there http://logs.openstack.org/10/615310/1/gate/openstack-ansible-functional-centos-7/3328646/logs/openstack/swift-storage1/swift-object-replicator.service.journal.log.txt.gz13:42
odyssey4mejrosser :/13:43
* odyssey4me summons andymccr :p13:43
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_masakari master: Basic implementation of masakari-monitors  https://review.openstack.org/58462913:44
*** nurdie has joined #openstack-ansible13:45
mgariepymorning everyone13:47
*** nurdie has quit IRC13:49
odyssey4mehwoarang well, that's interesting - I don't get http://logs.openstack.org/77/614777/1/check/openstack-ansible-deploy-aio_metal-opensuse-423/1a94cb3/logs/ara-report/result/2272ec6b-aa44-4824-b6d4-4c82f9f0488a/ with https://review.openstack.org/615525 and https://review.openstack.org/615258 applied together, along with https://review.openstack.org/61521313:50
odyssey4meif we can get https://review.openstack.org/615525 reviewed and merged, then I can update https://review.openstack.org/615258 with a role sha bump and we can see if the issue is resolved in the gate13:51
*** strattao has joined #openstack-ansible13:52
*** ram5391 has joined #openstack-ansible13:53
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-ceph_client master: tests: Skip role if ceph_mons list is empty  https://review.openstack.org/61554113:53
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-ceph_client master: SUSE: Fix ceph packages for Leap 15  https://review.openstack.org/61553513:53
hwoarangi see13:54
hwoarangok then lets do that13:54
hwoarangwho knows what's happening there13:54
*** ansmith has joined #openstack-ansible13:57
*** hamzaachi has joined #openstack-ansible13:59
odyssey4mehwoarang I got a similar obscure error for the utility container, and it turned out to be due to a differing version of setuptools - so maybe this is the same issue.14:01
openstackgerritMichael Vollman proposed openstack/openstack-ansible-tests master: Adding manila role tests and vars  https://review.openstack.org/61205514:08
noonedeadpunkodyssey4me: I've patched https://review.openstack.org/#/c/613977/3 according to your comment. Can you please recheck/revote?14:08
openstackgerritMichael Vollman proposed openstack/openstack-ansible-os_manila master: Converting os_cinder role to os_manila role  https://review.openstack.org/61193014:09
*** ram5391 has quit IRC14:09
*** thuydang has joined #openstack-ansible14:10
openstackgerritMerged openstack/openstack-ansible-repo_build stable/rocky: Ensure pip/setuptools/wheel are up to date in the wheel build venv  https://review.openstack.org/61552514:13
*** thuydang has left #openstack-ansible14:17
openstackgerritMerged openstack/openstack-ansible-tests master: Update ansible to 2.7.1  https://review.openstack.org/61545014:18
*** hamzaachi has quit IRC14:20
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/rocky: Ensure pip/setuptools/wheel are up to date in the utility venv  https://review.openstack.org/61525814:23
arxcruzodyssey4me: i didn't get your comment...14:27
odyssey4mearxcruz https://review.openstack.org/#/c/612968/11/tasks/tempestconf.yml@21 ?14:28
arxcruzohhh14:31
arxcruzodyssey4me: what would be the args?14:31
arxcruzgot it, the isolated14:32
odyssey4mearxcruz the args should be tempest_pip_install_args, as stated in the comment14:33
odyssey4meThey're defaulted in the role.14:33
openstackgerritArx Cruz proposed openstack/openstack-ansible-os_tempest master: Enable python-tempestconf support  https://review.openstack.org/61296814:35
arxcruzodyssey4me: done!14:35
odyssey4mearxcruz one more minor change, but no need to do it until the tests pass.14:37
openstackgerritArx Cruz proposed openstack/openstack-ansible-os_tempest master: Enable python-tempestconf support  https://review.openstack.org/61296814:39
arxcruzodyssey4me: done already, since tests are in the beginning14:39
odyssey4mearxcruz alright, can you mark -w so that it doesn't get merged by mistake in the last test verification?14:41
arxcruzdone14:42
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_masakari master: Basic implementation of masakari-monitors  https://review.openstack.org/58462914:47
vollmanodyssey4me: I have made the changes you requested.  Can you take another look at https://review.openstack.org/#/c/614210/14:47
*** evrardjp has quit IRC14:49
*** evrardjp has joined #openstack-ansible14:50
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_neutron master: Remove dragonflow ML2 driver  https://review.openstack.org/61557614:51
*** strattao has quit IRC14:54
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Remove Dragonflow ML2 driver  https://review.openstack.org/61557714:54
odyssey4methanks evrardjp for notifying us - this lets us purge the bits now, rather than find out later14:55
*** aedc has quit IRC14:58
*** strattao has joined #openstack-ansible14:59
openstackgerritMerged openstack/openstack-ansible-os_placement master: Initial commit to os_placement role  https://review.openstack.org/61527215:00
*** munimeha1 has joined #openstack-ansible15:02
openstackgerritMerged openstack/ansible-role-systemd_service master: Allow services to provide after_targets directly  https://review.openstack.org/61518515:03
*** d34dh0r53 has quit IRC15:22
*** cloudnull has quit IRC15:22
*** eglute has quit IRC15:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: RHEL: Move zlib-devel to cinder_devel_distro_packages  https://review.openstack.org/61552615:22
*** d34dh0r53 has joined #openstack-ansible15:23
*** eglute has joined #openstack-ansible15:23
openstackgerritMichael Vollman proposed openstack/openstack-ansible-os_manila master: Iniital commit to os_manila role  https://review.openstack.org/61192915:24
*** cloudnull has joined #openstack-ansible15:24
cloudnullo/ mornings all '15:24
jrossero/15:25
cloudnullhows it jrosser ?15:33
jrossernot too bad at all15:33
jrosserfixing stuff -> good :)15:34
odyssey4mevollman looking15:37
noonedeadpunkfolks, can you please remind me, whether I should add "lxc_hosts" to "--limit" for containers-deploy.yml?15:39
noonedeadpunkIf I'm re-creating some container15:40
openstackgerritAntony Messerli proposed openstack/openstack-ansible master: Adds option to enable ara during upgrades  https://review.openstack.org/61277515:40
odyssey4menoonedeadpunk yes, you should always have the lxc_hosts group along with the target container group15:41
noonedeadpunkodyssey4me: thanks, for some reason I thought, that this playbook changed this situation somehow.15:41
odyssey4mevollman a few minor changes - otherwise LGTM15:42
noonedeadpunkbtw, thanks for help with masakari - role now passes tests)15:43
jrosserthe swift systemd services look even more of a mess than i'd thought https://github.com/openstack/openstack-ansible-os_swift/blob/master/defaults/main.yml15:46
jrosserall of the replicator services are there twice15:46
jrosserwho understands how this is all supposed to be?15:46
odyssey4mejrosser andymccr did, cloudnull might15:46
jrosserthat may/may not explain this failure to connect to port 6000 if there are two competing services15:47
jrosserbut i cant reproduce it here15:47
vollmanThanks odyssey4me  Do you have time to give https://review.openstack.org/#/c/614210/ another look?15:47
*** mwe1 has joined #openstack-ansible15:48
odyssey4mevollman wowzers, that's a lot of hard-coded references15:48
jrossercloudnull: can you take a quick look at this https://github.com/openstack/openstack-ansible-os_swift/blob/master/defaults/main.yml - we seem to end up with swift-<XXX>-replicator-server.service and swift-<XXX>-replicator.service which look duplicate to me15:51
openstackgerritMerged openstack/openstack-ansible master: Pin argparse for idempotent pip installs  https://review.openstack.org/61176715:52
openstackgerritMerged openstack/openstack-ansible master: Restart cinder-scheduler instead of reloading  https://review.openstack.org/61369415:52
vollmanodyssey4me sure is.  just wanted to get the tests working again then tackle the hard coding15:53
openstackgerritMerged openstack/openstack-ansible master: Update ansible to 2.7.1  https://review.openstack.org/61522515:54
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_masakari master: Basic implementation of masakari-monitors  https://review.openstack.org/58462915:57
*** vollman has quit IRC15:59
*** vollman has joined #openstack-ansible16:04
*** pcaruana has quit IRC16:06
*** cshen has quit IRC16:11
*** tomtom001 has joined #openstack-ansible16:14
tomtom001Hello, I've gotten queens installed on 16.04, is the stable/queens branch ready for 18.04 just yet, or will that only be in Rocky?16:14
*** vollman has quit IRC16:16
noonedeadpunktomtom001: bionic is compatible with OSA only since Rocky16:19
noonedeadpunktomtom001: So if you want to upgrade from xenial to bionic, you'd probably should upgrade to rocky first16:20
*** vollman has joined #openstack-ansible16:20
noonedeadpunkat least I've folowwed this way for the upgrade16:22
tomtom001noonedeadpunk thanks!! appreciate the help.16:23
noonedeadpunktomtom001 these notes may help you with Xenial-> bionic upgrade, when you will be on rocky https://etherpad.openstack.org/p/osa-rocky-bionic-upgrade16:25
openstackgerritMerged openstack/openstack-ansible-os_nova master: Fix issues with nova distro install on Ubuntu  https://review.openstack.org/61437216:26
cloudnulljrosser I can have a look at the swift services later today . should you not get it sorted.16:28
jrossercloudnull: i can patch it to what looks right - but i don't really understand it16:29
jrosserin particular the ternary on the config file, one of them must be wrong16:29
openstackgerritMichael Vollman proposed openstack/openstack-ansible-ops master: Fix ansible path for elk_metrics_6x tests  https://review.openstack.org/61421016:34
openstackgerritMichael Vollman proposed openstack/openstack-ansible-os_manila master: Initial commit to os_manila role  https://review.openstack.org/61192916:42
openstackgerritAndy Smith proposed openstack/openstack-ansible master: Add qdrouterd role for oslo.messaging backend deployment  https://review.openstack.org/57026116:47
*** openstackgerrit has quit IRC16:48
tomtom001noonedeadpunk I'd like to use qemu 2.5 so that I can get native disk encryption support in queens.  What would be the best way to get qemu from 2.11 to 2.5 for installation?  Any ideas?16:49
noonedeadpunktomtom001: unfortunatelly I have no ideas... Probably you'd ask someone else around16:55
*** gkadam_ has quit IRC16:57
cloudnulltomtom001 on ubuntu ?16:59
cloudnulltomtom001 looks like 2.5 is in Xenial https://launchpad.net/~ubuntu-security-proposed/+archive/ubuntu/ppa/+build/1492719617:01
*** gyee has joined #openstack-ansible17:02
tomtom00116.0417:02
noonedeadpunkyep, it's xenial17:02
tomtom001But when I try to apt-get upgrade it's not one of the packages?17:03
noonedeadpunkI think, that packages are cached on the repo container17:04
cloudnulltomtom001 what version is installed17:05
*** openstackgerrit has joined #openstack-ansible17:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_neutron master: Remove dragonflow ML2 driver  https://review.openstack.org/61557617:05
tomtom0012.1117:05
openstackgerritGuilherme  Steinmuller Pimentel proposed openstack/openstack-ansible-os_nova master: Add support to kvm nested virt  https://review.openstack.org/61560817:06
*** ivve has quit IRC17:12
tomtom001cloudnull: 2.1117:13
openstackgerritAndy Smith proposed openstack/openstack-ansible master: Add qdrouterd role for oslo.messaging backend deployment  https://review.openstack.org/57026117:16
odyssey4metomtom001 cloudnull qemu 2.11 comes from UCA queens: http://ubuntu-cloud.archive.canonical.com/ubuntu/dists/xenial-updates/queens/main/binary-amd64/Packages17:29
odyssey4meif ubuntu xenial repositories accept the update to 2.5, then that will supercede UCA I think - but given it's only in proposed right now, it'll be some time before it becomes part of updates17:30
odyssey4meI'm not seeing qemu-kvm in http://archive.ubuntu.com/ubuntu/dists/xenial-proposed/main/binary-amd64/Packages.gz - and http://archive.ubuntu.com/ubuntu/dists/xenial-updates/main/binary-amd64/Packages.gz contains 1:2.5 - so the way to get it for xenial is either to use a PPA, or to upgrade to bionic.17:34
odyssey4mehmm, acually I read it wrong - UCA has Version: 1:2.11 ... isn't Version: 1:2.11 a higher version that 1:2.5 ?17:35
*** evrardjp has quit IRC17:39
odyssey4metomtom001 cloudnull I might be wrong, but it seems from the dates in https://download.qemu.org/ that 2.11 is a later version than 2.5 - so the UCA/queens version is later for xenial than the native 2.5 version in xenial17:39
*** evrardjp has joined #openstack-ansible17:40
*** nurdie has joined #openstack-ansible17:41
noonedeadpunkodyssey4me: Once you have time, can you please dig into masakari role? https://review.openstack.org/#/c/584629/39 Probably, time to get this patch merged is near...17:43
noonedeadpunkI just don't know what's wrong with suse15, as I can't find it's image in free acceess17:44
odyssey4menoonedeadpunk suse15 is not a required test to pass right now - hwoarang is still working through all sorts of things for it17:46
tomtom001odyssey4me cloudnull I get it, it's not 2.1 it's 2.11 so it's actually a later version is this right?  I read it wrong too.17:46
odyssey4metomtom001 as I understand it, the versioning is major.minor and the minor version is incremented by 1 for each release ... so 1,2,3,4,5,6,7,8,9,10,11 -> 11 is thus later than 517:47
tomtom001 ok so 2.11 is later than 2.50 or 2.5?17:48
tomtom001or no 2.50 would be later17:48
*** dave-mccowan has quit IRC17:49
noonedeadpunkodyssey4me: it's failing on libvirt-python symlinking on command "rpm -ql libvirt-python | egrep '^.*python2.*/(site|dist)-packages/.*'". I'm even not sure whether suse have rpm or egrep on board....17:50
*** cshen has joined #openstack-ansible17:50
noonedeadpunktomtom001 2.11 is newer than 2.5 for sure17:51
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_masakari master: Add missing docs/releasenotes tests  https://review.openstack.org/61562217:51
*** aredinger has joined #openstack-ansible17:54
*** cshen has quit IRC17:55
openstackgerritMerged openstack/openstack-ansible master: Integrate Blazar with OpenStack Ansible  https://review.openstack.org/54995617:58
*** hamzaachi has joined #openstack-ansible17:58
*** aredinger has quit IRC18:04
*** weezS has joined #openstack-ansible18:08
*** DanyC has joined #openstack-ansible18:10
*** aedc has joined #openstack-ansible18:11
odyssey4menoonedeadpunk looks good - just a few tweaks and then a rebase on top of https://review.openstack.org/615622 to ensure all the docs tests run too18:11
*** weezS has joined #openstack-ansible18:12
noonedeadpunkodyssey4me: got it. Just need the last to pass checks and get merged.18:13
noonedeadpunkodyssey4me: when I set "when: 'masakari_monitor' in group_names" here https://review.openstack.org/#/c/584629/39/tasks/masakari_install.yml ansible-lint marks it as error for some reason....18:21
noonedeadpunksorry, my fault, never mind18:23
*** hamzaachi has quit IRC18:23
odyssey4menoonedeadpunk it'll need to be when: "'masakari_monitor' in group_names"18:25
odyssey4meOK folks, I'm out for the night.18:26
kaiokmohave a great one odyssey4me18:27
*** mmercer has joined #openstack-ansible18:27
openstackgerritAntony Messerli proposed openstack/openstack-ansible master: Adds option to enable ARA plugin install  https://review.openstack.org/61277518:29
openstackgerritAntony Messerli proposed openstack/openstack-ansible master: Adds option to enable ARA plugin install  https://review.openstack.org/61277518:30
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_masakari master: Basic implementation of masakari-monitors  https://review.openstack.org/58462918:34
openstackgerritMerged openstack/openstack-ansible-os_manila master: Initial commit to os_manila role  https://review.openstack.org/61192918:34
*** hamzaachi has joined #openstack-ansible18:39
*** electrofelix has quit IRC18:43
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_masakari master: Basic implementation of masakari-monitors  https://review.openstack.org/58462918:45
*** aedc has quit IRC18:50
*** aedc has joined #openstack-ansible18:51
*** ram5391 has joined #openstack-ansible18:51
*** ivve has joined #openstack-ansible18:57
*** DanyC has quit IRC19:04
*** weezS has quit IRC19:04
*** DanyC has joined #openstack-ansible19:05
*** weezS has joined #openstack-ansible19:16
*** hamzaachi has quit IRC19:22
*** weezS has quit IRC19:23
*** cshen has joined #openstack-ansible19:23
*** weezS has joined #openstack-ansible19:26
openstackgerritGuilherme  Steinmuller Pimentel proposed openstack/openstack-ansible-os_nova master: Add support to kvm nested virt  https://review.openstack.org/61560819:26
*** nurdie has quit IRC19:29
*** DanyC has quit IRC19:36
ram5391having issues related to IOCTL when running setup-openstack. Output is here: http://paste.openstack.org/show/734164/19:38
ram5391all playbooks execute successfully up to that point. Help is much appreciated, not a super descriptive error, and I cant glean anything from attaching to the neutron container19:38
ram5391this is all on stable/rocky fyi19:39
*** aedc has quit IRC19:40
*** shardy has quit IRC19:52
*** shardy has joined #openstack-ansible19:52
*** DanyC has joined #openstack-ansible19:56
*** nurdie has joined #openstack-ansible20:09
openstackgerritMichael Vollman proposed openstack/openstack-ansible-tests master: Adding manila role tests and vars  https://review.openstack.org/61205520:18
openstackgerritMichael Vollman proposed openstack/openstack-ansible-os_manila master: Basic working os_manila role  https://review.openstack.org/61193020:19
*** DanyC has quit IRC20:34
openstackgerritAntony Messerli proposed openstack/openstack-ansible stable/rocky: Restart cinder-scheduler instead of reloading  https://review.openstack.org/61564520:35
*** aedc has joined #openstack-ansible20:35
openstackgerritAntony Messerli proposed openstack/openstack-ansible stable/queens: Restart cinder-scheduler instead of reloading  https://review.openstack.org/61564720:36
*** strattao has quit IRC20:45
cloudnullram5391 ive seen this error a few times20:49
cloudnullin general its been an issue with the provider_networks section within the openstack_user_config.yml file20:49
cloudnulljrosser were you able to take a crack at swift?20:50
jrosserno, other than i did a local run of the swift role test here on bionic, which annoyingly passed20:50
*** aedc has quit IRC20:50
ram5391cloudnull yup, I am following your guid here: https://cloudnull.io/2017/11/minimal-openstack-ansible-deployment/20:51
jrossercloudnull: best value/time is to take a look at the defaults file and decide if there are too many entries for the "replicator" services20:51
ram5391gonna comb through the whitespace again and make triple sure I'm not missing anything. You think its something like that though vs connectivity?20:52
jrossercloudnull: i have a patch here which fixes some of it https://review.openstack.org/#/c/615310/20:52
cloudnullram5391 yes I would suspect a whitespace issue or some missing field20:53
cloudnulljrosser sweet. ill work from that pr and see if there;s more to fix / cleanup20:54
jrossercloudnull: from what i can see there are duplicate replicator services20:54
cloudnullthats all bad...20:55
jrosseri got this far http://logs.openstack.org/10/615310/1/gate/openstack-ansible-functional-centos-7/3328646/logs/openstack/swift-storage1/ and was all a bit puzzled, then looked back at the defaults file20:55
*** nurdie has quit IRC21:00
*** nurdie has joined #openstack-ansible21:00
*** noonedeadpunk[m] has joined #openstack-ansible21:06
noonedeadpunk[m]Folks, can you point me please, what wrong I have written in release note here https://review.openstack.org/#/c/584629/ that the test fails?21:08
*** nurdie has quit IRC21:13
openstackgerritMichael Vollman proposed openstack/openstack-ansible-os_manila master: Basic working os_manila role  https://review.openstack.org/61193021:14
jrossernoonedeadpunk[m]: looks like it is here http://logs.openstack.org/29/584629/41/check/build-openstack-releasenotes/0db0f51/job-output.txt.gz#_2018-11-05_19_13_23_17342221:17
jrosser"Bullet list ends without a blank line; unexpected unindent."21:17
noonedeadpunk[m]Yeah, I saw it, but don't quite understand how to fix it:)21:18
*** cshen has quit IRC21:18
jrosseri would guess that you need a blank like following the list in your release note, to end the bullet list21:18
*** cshen has joined #openstack-ansible21:19
noonedeadpunk[m]Oh, I get it. It's came not from mine commit, but for some reason this test wasn't runned when the check was added in linked commit.21:20
noonedeadpunk[m]But it's good opportunity to get it fixed)21:22
*** weezS has quit IRC21:23
noonedeadpunk[m]Thanks jrosser for your time)21:24
jrosserno problem :)21:25
*** noonedeadpunk[m] has quit IRC21:33
*** kaiokmo has quit IRC21:35
*** dhellmann has quit IRC21:44
*** ivve has quit IRC21:51
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_masakari master: Basic implementation of masakari-monitors  https://review.openstack.org/58462921:51
openstackgerritMerged openstack/openstack-ansible-ceph_client master: tests: Skip role if ceph_mons list is empty  https://review.openstack.org/61554121:52
*** ansmith has quit IRC21:54
*** dhellmann_ has joined #openstack-ansible22:18
*** dhellmann_ is now known as dhellmann22:20
*** nurdie has joined #openstack-ansible22:39
*** munimeha1 has quit IRC22:42
*** fghaas has quit IRC22:42
*** cshen has quit IRC22:42
*** nurdie has quit IRC22:44
*** lbragstad has quit IRC23:09
*** lbragstad has joined #openstack-ansible23:10

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