Tuesday, 2018-01-23

*** gokhan_ has joined #openstack-ansible00:10
*** mancdaz has quit IRC00:12
*** robcresswell has quit IRC00:12
*** mancdaz has joined #openstack-ansible00:14
*** chyka_ has joined #openstack-ansible00:14
*** chyka has quit IRC00:18
*** chyka_ has quit IRC00:40
*** armaan has quit IRC00:49
*** armaan has joined #openstack-ansible00:50
*** jafeha_ has joined #openstack-ansible00:52
*** jafeha has quit IRC00:56
*** woodard has joined #openstack-ansible00:57
*** lbragstad has quit IRC00:57
*** openstacking_123 has joined #openstack-ansible01:00
*** openstacking_123 has quit IRC01:02
*** cjloader has quit IRC01:26
*** chyka has joined #openstack-ansible01:48
*** chyka has quit IRC02:06
*** yolanda has joined #openstack-ansible02:09
*** robcresswell has joined #openstack-ansible02:09
*** lbragstad has joined #openstack-ansible02:14
*** germs has joined #openstack-ansible02:41
*** germs1 has joined #openstack-ansible02:51
*** germs has quit IRC02:52
*** germs has joined #openstack-ansible02:57
*** germs1 has quit IRC02:58
*** ivveh has quit IRC03:07
*** ivveh has joined #openstack-ansible03:08
*** smatzek has joined #openstack-ansible03:11
*** dave-mcc_ has quit IRC03:16
*** germs has quit IRC03:20
*** germs has joined #openstack-ansible03:22
*** mgariepy has quit IRC03:24
*** germs has quit IRC03:26
*** mgariepy has joined #openstack-ansible03:27
*** rstarmer has joined #openstack-ansible03:30
*** smatzek has quit IRC03:34
*** ykarel|afk has joined #openstack-ansible04:19
*** yolanda has quit IRC04:32
*** cjloader has joined #openstack-ansible04:34
*** cjloader has quit IRC04:36
*** cjloader has joined #openstack-ansible04:36
*** rstarmer has quit IRC04:37
*** bhujay has joined #openstack-ansible04:41
*** ivve has joined #openstack-ansible04:51
*** cjloader has quit IRC04:53
*** cjloader_ has joined #openstack-ansible04:55
*** cjloader_ has quit IRC04:56
*** poopcat1 has quit IRC05:03
*** cjloader has joined #openstack-ansible05:11
*** cxc99 has quit IRC05:14
*** cxc99 has joined #openstack-ansible05:17
*** cjloader has quit IRC05:20
*** cjloader has joined #openstack-ansible05:21
*** pcaruana has joined #openstack-ansible05:24
*** hybridpollo has quit IRC05:30
*** smatzek has joined #openstack-ansible05:35
*** pcaruana has quit IRC05:36
*** smatzek has quit IRC05:39
*** andries has quit IRC06:00
*** andries has joined #openstack-ansible06:01
Taseerodyssey4me: your suggestion for Sphinx requirement resulted in this => http://logs.openstack.org/91/522491/42/check/requirements-check/8edadcd/job-output.txt.gz#_2018-01-22_11_54_13_99059006:06
*** ykarel|afk is now known as ykarel06:27
*** cjloader has quit IRC06:34
*** arbrandes1 has joined #openstack-ansible07:01
*** arbrandes has quit IRC07:04
*** ThomasS has joined #openstack-ansible07:32
*** pcaruana has joined #openstack-ansible07:33
*** armaan has quit IRC07:51
*** armaan has joined #openstack-ansible07:51
*** jafeha_ has quit IRC07:54
*** mbuil has joined #openstack-ansible07:56
*** pcaruana has quit IRC07:58
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-os_nova master: Reinitialize the virtualenv  https://review.openstack.org/53584408:08
*** pcaruana has joined #openstack-ansible08:13
odyssey4meTaseer looks like, according to the same line, you've specified a package called 'sphin' instead of 'sphinx'08:19
*** sxc731 has joined #openstack-ansible08:35
jafeha__good morning everyone08:37
*** gillesMo has joined #openstack-ansible08:40
*** gillesMo has quit IRC08:41
*** jafeha__ is now known as jafeha08:44
evrardjpgood morning jafeha08:44
evrardjpgood morning odyssey4me  too08:44
*** gillesMo has joined #openstack-ansible08:45
evrardjpand others if any :)08:47
*** sxc731 has quit IRC08:47
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible master: Build everything on metal  https://review.openstack.org/50422408:47
openstackgerritTaseer Ahmed proposed openstack/openstack-ansible-os_congress master: Introduce os_congress role in gerrit  https://review.openstack.org/52249108:48
*** jbadiapa has joined #openstack-ansible08:53
openstackgerritMerged openstack/openstack-ansible-os_ironic master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53642808:56
*** shardy has joined #openstack-ansible08:56
openstackgerritMerged openstack/openstack-ansible-haproxy_server master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53641208:57
openstackgerritMerged openstack/openstack-ansible-rabbitmq_server master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53644608:59
openstackgerritMerged openstack/openstack-ansible-rsyslog_client master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53644908:59
openstackgerritMerged openstack/openstack-ansible-pip_install master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53644408:59
openstackgerritsun cheng proposed openstack/openstack-ansible-os_sahara master: Update url links in files of openstack-ansible-os_sahara  https://review.openstack.org/53671909:00
openstackgerritMerged openstack/openstack-ansible-os_nova master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53643309:01
openstackgerritMerged openstack/openstack-ansible-galera_client master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53641009:01
openstackgerritMerged openstack/openstack-ansible-plugins master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53644509:02
openstackgerritMerged openstack/openstack-ansible-os_sahara master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53643609:04
openstackgerritMerged openstack/ansible-hardening master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53645109:10
openstackgerritMerged openstack/openstack-ansible-repo_build master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53644709:10
openstackgerritMerged openstack/openstack-ansible-os_glance master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53642409:11
*** ivveh_ has joined #openstack-ansible09:24
*** rpittau has joined #openstack-ansible09:25
*** ivveh has quit IRC09:25
openstackgerritMerged openstack/openstack-ansible-galera_server master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53641109:27
*** ivveh has joined #openstack-ansible09:43
*** ivveh_ has quit IRC09:43
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible master: Cleanup the gate-check-commit dstat part  https://review.openstack.org/53673809:45
evrardjpodyssey4me: did your thing ^09:48
*** markvoelker has quit IRC09:59
openstackgerritMerged openstack/openstack-ansible stable/ocata: Update all SHAs for 15.1.15  https://review.openstack.org/53587610:00
*** taseer1 has joined #openstack-ansible10:08
*** taseer2 has joined #openstack-ansible10:09
*** Taseer has quit IRC10:09
*** taseer1 has quit IRC10:13
*** chas has joined #openstack-ansible10:18
evrardjpodyssey4me: are you there?10:22
odyssey4meyes10:22
evrardjpgreat I'd like to discuss an approach10:23
evrardjpso this is the issue: http://logs.openstack.org/periodic/git.openstack.org/openstack/openstack-ansible/master/openstack-ansible-upgrade-aio-ubuntu-xenial/0f7042d/job-output.txt.gz#_2018-01-23_08_28_34_36455610:23
evrardjpit probably appeared after your patch here: https://review.openstack.org/#/c/521750/210:24
evrardjpwhich has a dependency chain and moving parts.10:24
evrardjpthe problem, as you can see in the link url, is happening during upgrades from P to Q.10:24
evrardjpbecause P has: https://github.com/openstack/openstack-ansible-repo_server/blob/stable/pike/tasks/repo_post_install.yml#L105-L12210:25
evrardjp(and others)10:25
evrardjpwould you rather backport the Q changes (which is for me a little big series of patches), or reintroduce changes in Q to handle the upgrade case?10:26
evrardjpI'd do the latter, but I am not fully aware of the context.10:26
evrardjpif that's something we need to do in P anyway, let's do it10:27
odyssey4meoh bother10:28
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible master: Cleanup the gate-check-commit dstat part  https://review.openstack.org/53673810:28
evrardjpodyssey4me: :D10:28
odyssey4meso we could revert just that one patch and be fine, or we could implement some sort of migration tooling10:29
odyssey4methe migration would need to check for the link, then if it exists copy the contents somewhere temporary, then remove the link and move the contents to a newly created folder10:29
evrardjpyeah the revert would work, the migration would work, and the cleanup of P would be fine too. These are sorted with the amount of work and risks :D10:29
odyssey4meporting that back to p will then give you the same problem for o->p10:30
evrardjpodyssey4me: yes10:30
odyssey4meit'd be turtles all the way down10:30
evrardjpthat's why I'd like to not do it :D10:30
evrardjpI think the best approach is the tooling in the repo_server role10:30
evrardjpodyssey4me: I think for that process I'd just do a shell script. It seems less tedious than a series of ansible tasks.10:32
odyssey4meit's hardly worth the extra machinery, given that it's AIO specific10:32
evrardjpwhat do you mean?10:32
evrardjpyou mean just reverting?10:32
odyssey4meIt would only be a symlink if the repo container is on the deploy host.10:33
evrardjpI am still not sure what you mean....10:34
evrardjpI understand that part, but I don't understand what you'd prefer to do.10:35
evrardjpI know it's not a big deal, but it's still something we have to fix10:35
odyssey4mewell, one option is to revert and add a TODO to be rid of it in Rocky... and add a release note telling people that the functionality is deprecated... it's highly unlikely that the functionality is being used anywhere outside the gates but we should probably notify people10:35
evrardjpok, so no migration tooling10:36
odyssey4meanother option is to add tooling to migrate from the previous state to the new state - which is nice, but adds machinery that may break10:36
evrardjpI thought we could use something like memcached: https://review.openstack.org/#/c/526022/1/tasks/memcached_config.yml10:36
evrardjpbut that won't keep existing cache, like you'd suggest.10:37
odyssey4meoption 2 is nicer IF someone is using it, option 1 is easier and simpler from a code standpoint10:37
evrardjpto be 100% certain we are talking about the same thing, you mean the feature that a deploy node can be used as git cache, right?10:39
evrardjpbecause we kinda broke that promise when doing the first patch, didn't we?10:39
odyssey4methe idea is that if the deploy node has the git contents already, it gets pushed to the repo container when the repo container gets setup10:39
odyssey4methis still happens today10:40
evrardjpok10:40
odyssey4mewe previously also allowed a symlink if the deploy host and repo container host were the same - so the contents were not pushed up but instead just symlinked10:40
evrardjpso it's just that the bind mount and symlink thing changed10:40
odyssey4methis was faster for gating, and was done for the purpose of making gating faster10:40
evrardjpmmm I see.10:41
*** jbadiapa has quit IRC10:41
odyssey4meit's possible that we could implement something in the zuul v3 world to make gating faster using the git cache, but we're not doing that right now for some reasons... primarily that it would take time to figure out that I don't have and that doing it would require a fair bit of test implementation restructuring10:42
evrardjpLet skip that part, our gating works.10:42
evrardjp:p10:42
odyssey4meand right now our gate times are kinda ok - as we've discussed before, instead of using hacks like this I think we need more fundamental changes to how things are done10:43
evrardjpagreed.10:43
evrardjpbut right now, I'd say let's deal with what we have10:43
evrardjpmy concern is that for people that will do a P-Q-R upgrade path10:44
odyssey4meyep, obviously10:44
evrardjpif we do a revert, and get rid of rocky, they would still have that folder present10:44
openstackgerritMerged openstack/openstack-ansible-rsyslog_server master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53645010:44
odyssey4meso it comes down to the question of whether you want the migration done automagically, or whether we inform the deployers that the symlink functionality is going away and let them figure it out on their own10:45
odyssey4meI'm OK with either approach and will be happy to put the patch together, but I'd like you to choose which approach you prefer.10:45
evrardjpI think it's a simple risk10:46
evrardjphaha I was kinda the same, ok for the patch, let's discuss the approach :D10:46
openstackgerritMerged openstack/openstack-ansible-os_searchlight master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53643710:47
evrardjpI think getting rid of the symlink is fine10:47
evrardjpI mean10:47
evrardjpdoing a revert + release note is fine10:47
evrardjptelling it's going away, because they can still use the folder in itself10:47
evrardjpit's not like it was a big change, and on top of that, with developer mode changes or changes we plan with the way we do repo, I think it would make sense to move forward and inform our deployers10:48
openstackgerritMerged openstack/openstack-ansible-os_watcher master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53644210:49
evrardjpwe need to improve our relationship with deployers, but doing everything automatically isn't necessary a good thing10:49
*** shardy has quit IRC10:49
openstackgerritMerged openstack/openstack-ansible-os_zaqar master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53644310:50
openstackgerritMerged openstack/openstack-ansible-os_rally master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53643510:50
evrardjpodyssey4me: I can issue the patch if you're busy.10:50
*** shardy has joined #openstack-ansible10:51
evrardjpok let me get that rolling10:52
odyssey4meok10:52
odyssey4megreat, let me know when you have it ready for review10:52
openstackgerritMerged openstack/openstack-ansible-repo_server master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53644810:53
openstackgerritMerged openstack/openstack-ansible-os_tempest master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53644010:53
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-repo_server master: Revert "Remove sylinked git cache functionality"  https://review.openstack.org/53676310:59
evrardjpodyssey4me: ^10:59
evrardjpnot sure if that's really clear10:59
odyssey4memind if I edit it?10:59
evrardjpnot at all11:00
evrardjpenglish things.11:00
evrardjphwoarang: odyssey4me if you could vote on https://review.openstack.org/#/c/536738/2 that would be great. I kinda broke the upgrades from O to P while trying to fix them.11:02
*** jcos has joined #openstack-ansible11:04
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_server master: Revert "Remove sylinked git cache functionality"  https://review.openstack.org/53676311:08
evrardjphwoarang: we seem to have issues in role tests for suse, a problem always appearing on keystone: http://logs.openstack.org/15/535815/1/gate/openstack-ansible-functional-opensuse-423/fc43563/job-output.txt.gz#_2018-01-23_09_46_40_53076811:08
evrardjpdo you think the db changes could have an impact?11:09
evrardjpodyssey4me: that's exactly what I meant!11:10
evrardjp:D11:10
odyssey4meGlad we agree then. :)11:11
evrardjpahah I can't consider you my official translator, eumel would feel bad :p11:12
openstackgerritMerged openstack/openstack-ansible-os_swift master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53643811:12
*** jbadiapa has joined #openstack-ansible11:13
*** SmearedBeard has joined #openstack-ansible11:16
*** Smeared_Beard has quit IRC11:18
evrardjphwoarang: thinking about this patch: https://review.openstack.org/#/c/535254/ but I might be wrong.11:18
evrardjpI am probably wrong11:18
evrardjpI will try in more details11:19
*** gillesMo has quit IRC11:19
hwoarangevrardjp: forget about it for now i am lining up more patches11:29
evrardjpyeah but it's blocking gates11:29
hwoarangoh11:29
hwoarangwait11:29
hwoarangagain11:30
evrardjpI have started a build locally, see if that's linked11:30
hwoarangit's not related to this patch i think. just some locking is going terribly wrong on galera11:30
hwoarangit was there on friday too, then it was ok in weekend and monday and now broke again11:30
evrardjpI am worried of a version change impacting suse, that would give a keystone failure11:30
evrardjpoh ok11:31
hwoarangthe package hasn't updated in 10 days I am afraid11:31
evrardjpok11:31
evrardjpI haven't digged deeper, I just looked at what changed recently :p11:31
evrardjpI am building right now, I will tell you more if that's the same issue.11:31
hwoarangthere are a lot of warnings and errors in teh galera log file on that job. some of them lead to a bug upstream that was fixed in 10.2.12 which is the version currently used by suse11:32
evrardjpgood to know.11:32
evrardjplet's keep in touch if we find out more about what can fix this11:32
hwoarangok11:33
evrardjpI'll go to lunch in the time it's building :D11:33
evrardjpin the meantime, if you want to help with a quick thing: https://review.openstack.org/#/c/536738/ :D11:33
hwoarangenjoy your lunch11:33
evrardjpty!11:34
armaanHello folks, I was wondering whether it is possible that even after running the "repo-install.yml" playbooks, some repo container can still have the older versions?11:38
*** jbadiapa has quit IRC11:38
armaanIn other words, how can i make sure that the new package is synced to all the repo containers.11:41
*** jwitko has quit IRC11:45
openstackgerritMerged openstack/openstack-ansible-os_heat master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53642611:48
evrardjparmaan: it should be done automatically11:52
openstackgerritMerged openstack/openstack-ansible-os_tacker master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53643911:52
evrardjpit takes a few minutes to sync between repo containers, but the master repo should have a process handling that sync11:53
evrardjpcheck if lsyncd is running11:53
evrardjpon your master11:54
openstackgerritMerged openstack/openstack-ansible-apt_package_pinning master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53640811:57
*** jcosmao has joined #openstack-ansible11:58
openstackgerritMerged openstack/openstack-ansible-os_aodh master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53641911:58
openstackgerritMerged openstack/openstack-ansible-lxc_container_create master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53641311:59
*** markvoelker has joined #openstack-ansible12:00
openstackgerritMerged openstack/openstack-ansible-ceph_client master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53640912:00
openstackgerritMerged openstack/openstack-ansible-os_cloudkitty master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53642212:00
openstackgerritMerged openstack/openstack-ansible-os_cinder master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53642112:00
armaanevrardjp: roger12:00
openstackgerritMerged openstack/openstack-ansible-os_keystone master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53642912:00
*** jcos has quit IRC12:00
openstackgerritMerged openstack/openstack-ansible-os_gnocchi master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53642512:00
openstackgerritMerged openstack/openstack-ansible-memcached_server master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53641512:01
openstackgerritMerged openstack/openstack-ansible-os_horizon master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53642712:01
openstackgerritMerged openstack/openstack-ansible-os_designate master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53642312:01
openstackgerritMerged openstack/openstack-ansible-os_octavia master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53643412:01
openstackgerritMerged openstack/openstack-ansible-openstack_hosts master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53641612:01
openstackgerritMerged openstack/openstack-ansible-openstack_openrc master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53641712:01
openstackgerritMerged openstack/openstack-ansible-os_monasca master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53643112:02
openstackgerritMerged openstack/openstack-ansible-lxc_hosts master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53641412:05
armaanevrardjp: http://paste.openstack.org/show/650938/12:06
openstackgerritMerged openstack/openstack-ansible-os_neutron master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53643212:06
armaanevrardjp: it appears that only links and pkg-cache directory got updated12:07
*** taseer2 is now known as Taseer12:08
openstackgerritMerged openstack/openstack-ansible-os_almanach master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53641812:09
evrardjpmmm pools should be updated too, nothing in the logs?12:10
evrardjplsyncd logs12:10
evrardjpnot sure if lsyncd has logs...12:10
evrardjpbut we should have logs somewhere12:10
armaanevrardjp: i also tried this, openstack-ansible repo-build.yml -e repo_build_git_reclone=yes -e repo_build_wheel_rebuild=yes -e repo_build_venv_rebuild=yes12:10
evrardjpI don't think that would be necessary12:11
evrardjpdo you have a log of your run repo build run?12:11
evrardjpmaybe restart lsyncd on the master node12:12
armaanevrardjp: lsyncd log http://paste.openstack.org/show/650940/12:12
armaanok let me restart12:12
openstackgerritMerged openstack/openstack-ansible-os_barbican master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53642012:12
evrardjphwoarang: http://paste.openstack.org/show/650942/12:15
armaanevrardjp: http://paste.openstack.org/show/650943/12:15
evrardjparmaan: and you are checked out in 16.0.6 I guess?12:16
evrardjpor you are checked out in 16.0.5 ?12:16
evrardjpok let's recap, I was maybe distracted12:17
evrardjpthe issue is not that the files aren't synced, is that they aren't created in the first place12:18
*** tnogisto2 has joined #openstack-ansible12:18
*** tnogisto has joined #openstack-ansible12:18
armaanevrardjp: I was running stable/pike(16.0.5) few days back and the i moved back few commits because magnum is broken in stable/pike.12:18
*** tnogisto has quit IRC12:18
evrardjpso you're running some kind of 16.0.512:18
armaanevrardjp: so, i changed the commit in here https://github.com/openstack/openstack-ansible/blob/stable/pike/playbooks/defaults/repo_packages/openstack_services.yml#L19112:18
evrardjp(you can check in your group vars all )12:18
*** smatzek has joined #openstack-ansible12:19
evrardjpok12:19
evrardjpthen re-did the repo build12:19
armaanevrardjp: yes, after changing the commit to c58b727bec3561d8d283497d784a3437185924dd, i did a repo build12:20
evrardjpodyssey4me: in the fact we setup for the need of a repo rebuild, do we care about a change in the git repos?12:20
evrardjparmaan: all you did seems correct.12:20
evrardjpit should have worked12:20
armaanevrardjp: but i still had the code from latest commits12:21
armaanevrardjp: ok, good to hear that12:21
evrardjpyou mean the new SHA of magnum wasn't respected, and it's like it did nothing?12:21
armaanevrardjp: Yes.12:21
evrardjpyou can probably remove your venv of 16.0.5 on the repo nodes and try again, but I'd rather know why it didn't do the proper thing in the first place12:21
armaanevrardjp: roger that.12:22
evrardjpif you have your ansible run log that would be great12:22
armaanevrardjp: something like this "root@control02-repo-container-ef17687b:/var/www/repo/venvs# rm -rf 16.0.5/"12:22
evrardjp(of the first one, when you changed sha)12:22
evrardjparmaan: you can do it with ansible :)12:23
evrardjpbut I'd like to have another person's opinion12:23
armaanevrardjp: unfortunately, i ran a lot of things after the first run, so log won't be available12:23
armaanevrardjp: running "rm -rf" with ansible in a production environment is a bit scary :p12:24
armaanevrardjp: btw it appears that venv was changed. http://paste.openstack.org/show/650943/12:26
evrardjparmaan: technically the rm shouldn't be needed. I am concerned because if you did openstack-ansible repo-build.yml -e repo_build_git_reclone=yes -e repo_build_wheel_rebuild=yes -e repo_build_venv_rebuild=yes everything should be bypassed, and all the reclones/rebuild should be done12:29
evrardjpso what did you do right now? Removed the venvs, and rebuild?12:30
evrardjp(or moved the venvs and rebuild)12:30
armaanevrardjp: i only ran "openstack-ansible repo-build.yml -e repo_build_git_reclone=yes -e repo_build_wheel_rebuild=yes -e repo_build_venv_rebuild=yes"12:31
armaandidn't remove the venvs12:31
armaanrunning repo-install.yml again12:32
*** markvoelker has quit IRC12:33
odyssey4meevrardjp yes, a git sha change will cause the constraints file to change and trigger a rebuild12:36
evrardjpyeah so there is something wring on armaan 's case12:37
evrardjpweird*12:37
*** cmart has quit IRC12:37
evrardjparmaan: please paste the log of your repo-build part12:37
odyssey4mearmaan in your os-releases folder for the release you're on, there'll be a bunch of *.txt files12:38
armaanevrardjp odyssey4me http://paste.openstack.org/show/650945/12:39
odyssey4mecheck the requirements_constraints.txt and verify that magnum has the sha you intend it to have?12:39
armaanodyssey4me: git+file:///var/www/repo/openstackgit/magnum@c58b727bec3561d8d283497d784a3437185924dd#egg=magnum&gitname=magnum&projectgroup=magnum_all12:40
armaanodyssey4me: it does.12:40
armaanI want to switch back to this commit c58b727bec3561d8d283497d784a3437185924dd12:40
odyssey4meok, and how have you adjusted the git sha? did you modify the integrated repo code or did you override using user_variables?12:41
armaanodyssey4me: I changed this line https://github.com/openstack/openstack-ansible/blob/stable/pike/playbooks/defaults/repo_packages/openstack_services.yml#L19112:42
armaanmagnum_git_install_branch: c58b727bec3561d8d283497d784a3437185924dd12:42
odyssey4meok, for future reference you can just override that in /etc/openstack_deploy/user_variables.yml rather than modify the git code. Anyway, what's the full file name for magnum*.whl in the os-releases folder?12:43
evrardjparmaan: that's a lot of skipping12:43
odyssey4methe skipping is right if nothing's changed12:43
evrardjpyes but the sha was changed12:44
evrardjpor maybe I missed the point?12:44
odyssey4meyes, but it seems like the output given is after a previous run12:44
odyssey4meie the sha was changed, and the repo build was run after that which changed things - the output given is a subsequent run12:44
evrardjpok12:45
evrardjpI thought this was the thing JUST after12:45
odyssey4medoesn't look like it12:45
evrardjplet's follow your lead odyssey4me12:45
evrardjp(fixing cinder in the meantime, so i am missing parts of the convo)12:46
armaanodyssey4me: http://paste.openstack.org/show/650947/12:46
armaanevrardjp: Yes, the output is from a subsequent run.12:47
odyssey4meah, there's the problem12:47
*** woodard has quit IRC12:47
odyssey4methe venv build is not version specific - it relies on the os-releases folder having a single version available... which it does not in this case12:47
odyssey4methat's odd12:47
*** woodard has joined #openstack-ansible12:48
odyssey4mehttps://github.com/openstack/openstack-ansible-repo_build/blob/master/tasks/repo_build_wheels.yml#L65-L77 should be cleaning them up when the wheel build happens, so something fishy is going on here12:49
armaanodyssey4me: should i remove these http://paste.openstack.org/show/650954/ and rebuild the repo?12:51
odyssey4mehmm, hang on a minute - armaan what branch is this?12:51
odyssey4mepike?12:51
armaanyes, pike12:51
evrardjpno 2.4 change :p12:51
odyssey4meok, this is a bug introduced by a change in ansible 2.3 - same issue as https://review.openstack.org/#/c/529043/12:52
odyssey4meI'll push up a patch now - armaan if you could validate that it works that'd be marvellous12:52
armaanodyssey4me: awesome! Sure i will do that.12:53
evrardjpahah good catch odyssey4me12:53
evrardjpI expected an ansible issue, not that one though :p12:53
evrardjpthanks odyssey4me again! and thanks armaan for the source of the bug :)12:54
armaanevrardjp: :)12:54
evrardjparmaan: if you want an easier and faster debugging session, it's maybe worth creating a bug directly with the first log output, then the re-run output, and telling what you're trying to achieve/what you expected. Because if we have everything under our eyes it goes faster :)12:55
evrardjpbut anyway, it was good!12:55
evrardjpthanks odyssey4me and armaan again!12:55
armaanevrardjp: Good advice. I will keep that in mind.12:56
evrardjpyeah, I totally have the same issue: when I am in a problem, I am heading directly towards it for the solution, but sometimes rephrasing it for other people helps me. Just sharing my experience :D12:57
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build master: Ensure that all release files are cleaned up when rebuilding  https://review.openstack.org/53680212:57
odyssey4mearmaan that should fix it - you'll need to tell the build to rebuild, ie: openstack-ansible repo-build.yml -e repo_build_wheel_rebuild=yes -e repo_build_venv_rebuild=yes12:58
odyssey4me(no need to reclone the git sources)12:58
armaanroger!12:59
evrardjparmaan: if you could leave a review on the patch above to confirm it fixes it for you, that would be great too12:59
odyssey4me++12:59
*** cmart has joined #openstack-ansible13:02
*** askb has quit IRC13:05
*** yolanda has joined #openstack-ansible13:05
*** askb has joined #openstack-ansible13:06
evrardjpwow tempest ImportError exception isn't reported as ImportError when using the plugin load... That's confusing13:07
evrardjpwell no that could be alright, but it's the same error message if you can't find the plugin and if you can't load the plugin basically13:08
evrardjpthat's the confusing part :D13:09
*** vnogin has joined #openstack-ansible13:10
odyssey4meconfusing error messages FTW13:11
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-os_cinder master: Fix upgrade test  https://review.openstack.org/53579113:14
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-os_cinder master: Fix upgrade test  https://review.openstack.org/53579113:15
evrardjpodyssey4me: should fix it ^13:15
*** bhujay has quit IRC13:16
odyssey4melooks good - let's see what the gate says13:16
*** jbadiapa has joined #openstack-ansible13:16
evrardjphwoarang: the fun start when you see something like this: DBAPIError exception wrapped from (pymysql.err.InternalError)13:17
*** cmart has quit IRC13:21
armaanevrardjp odyssey4me: That patch fixed my issue. thanks a million folks :)13:21
*** vnogin has quit IRC13:22
evrardjparmaan: you're welcome13:22
*** ykarel is now known as ykarel|afk13:23
openstackgerritJakob Hasselmann proposed openstack/openstack-ansible-os_neutron master: Add var and docs for designate domain port config  https://review.openstack.org/53680913:24
*** electrofelix has joined #openstack-ansible13:25
*** SerenaFeng has joined #openstack-ansible13:30
*** markvoelker has joined #openstack-ansible13:31
*** ykarel|afk has quit IRC13:32
*** SerenaFeng has quit IRC13:32
*** ivveh has quit IRC13:33
evrardjpjafeha: thanks! :D13:34
evrardjphwoarang: are you there?13:34
evrardjpI see the difference with my suse build13:34
*** SerenaFeng has joined #openstack-ansible13:34
*** cmart has joined #openstack-ansible13:35
evrardjpI don't know how deep you digged into it yet, so I'd rather avoid re-debugging what you already found.13:35
evrardjpbut it looks like a violation of a table constraint that prevent the keystone db migrate to run13:35
evrardjp  CONSTRAINT `CONSTRAINT_1` CHECK (`allow_application_credential_creation` in (0,1))13:35
evrardjpdoes that ring you a bell?13:36
*** RandomTech has joined #openstack-ansible13:37
*** esberglu_ has quit IRC13:39
evrardjpandymccr: are you there by any chance?13:42
andymccro/13:42
evrardjpI will just ask you to review two things real quick then13:43
jafehaevrardjp, np.13:43
andymccrsure13:43
evrardjphttps://review.openstack.org/#/c/536738/ https://review.openstack.org/#/c/536763/13:43
evrardjpbreaking upgrade gates13:43
evrardjpperiodics*13:43
*** cjloader has joined #openstack-ansible13:43
andymccrevrardjp: on it13:43
evrardjpthanks!13:43
*** vnogin has joined #openstack-ansible13:45
odyssey4meandymccr while you're reviewing, https://review.openstack.org/#/c/536802/ could do with more eyes :)13:45
andymccrthat is crazy13:46
andymccrlgtm - but wtf module change :P13:46
odyssey4meyup, fun times13:46
andymccrevrardjp: why do we need to || true on run_dstat? surely run_dstat just doesnt do anything if gate_exit_run_dstat is not true?13:47
odyssey4mewe should probably do a bit of an audit of everywhere we use the find module to see if we don't have more hidden bugs13:47
evrardjpandymccr: I think I wrote it in the commit message. it's for upgrades.13:47
evrardjpWe are loading the gate_check_commit, then we do checkout to previous branch, then we load the lib13:47
evrardjpthe lib may or may not have run_dstat13:48
evrardjpso we run || true13:48
andymccrah so if it doesnt exist13:48
evrardjpyeah13:48
evrardjpalso, it's simpler than doing if upgrade, set dstat to false, etc.13:48
evrardjplooked cleaner imo13:48
andymccryeah i mean im usually against || true in general, but im trying to think of a better way and not really coming up with much13:49
evrardjpyeah and that's conditional of the upgrade ofc :p13:49
evrardjpthe other way would be to load in a different order, but then you're hitting other issues.13:49
evrardjpwhich is why we did in this order13:49
evrardjp|| true is evil indeed.13:50
andymccri guess you could check if it exists using type?13:50
andymccrwe in urgent mode?13:50
evrardjpnor urgent urgent, but it's breaking O to P upgrades, which is not fun13:51
andymccryeh13:51
evrardjpP to Q could maybe wait13:51
odyssey4meevrardjp is being modest, upgrades have been broken for months and he's been slowly working through fixes to get them back to working13:51
odyssey4meI would say we're in urgent mode.13:51
andymccryeh taht sounds about right13:51
andymccrcool lets just merge it13:51
mgariepymorning everyone13:52
evrardjpmgariepy: good morning13:52
mgariepyI know it's in a bit but anyone knows he goes to openstack-summit in vancouver ?13:52
evrardjpI don't know, but I will try to submit a talk there to help my case :D13:53
RandomTechif i wanted to configure an extra network for the contains to get ips for an external connection(floating ips) would i just put "all_containers" in group binds?13:54
mgariepyhehe ok :) that's a good idea.13:54
mgariepyI'll ask by boss if he wants to pay for me ;) haha13:54
evrardjpit's the same country!13:55
mgariepyyeah only 5000 km west13:56
evrardjp:p13:57
*** ykarel|afk has joined #openstack-ansible13:57
*** aludwar has joined #openstack-ansible13:57
*** esberglu has joined #openstack-ansible13:58
*** cjloader has quit IRC14:00
hwoarangevrardjp: i am here now14:00
evrardjpgreat14:01
hwoarangit rings a bell...14:01
evrardjpso, could you remind me what was the issue?14:01
evrardjpdarn14:01
*** armaan_ has joined #openstack-ansible14:01
evrardjpshould we speak to keystone team to help us finding out what's the root cause of the issue?14:02
hwoarangevrardjp:  is this similar to  https://bugs.launchpad.net/barbican/+bug/1734329 ?14:02
openstackLaunchpad bug 1734329 in Barbican "CHECK constraint bug in db migration on Mariadb 10.2" [Undecided,New]14:02
evrardjpsounds like it14:02
hwoarangthat's a keystone regression then14:02
hwoarangi know a keystone dev at suse14:02
evrardjpyup14:02
hwoarangdo you have a public link for that?14:02
evrardjpnope14:03
evrardjpwell I had a paste14:03
evrardjpjust a second14:03
*** markvoelker has quit IRC14:04
* odyssey4me pushes up reviews everywhere to add || true to make everything work. I know andymccr really loves that.14:05
*** armaan has quit IRC14:05
andymccrahah14:05
andymccrwell stuff will work14:05
evrardjphwoarang: http://paste.openstack.org/show/650942/14:05
andymccrjust wrap the upgrade task in a || true14:05
andymccr:D14:05
andymccrgreen gates are green14:05
*** SerenaFeng has quit IRC14:05
odyssey4mesuccess!14:05
evrardjpwoot, where?!14:05
evrardjp:p14:05
hwoarangevrardjp: thanks I will forward taht14:06
evrardjpyeah, it's probably 10.2 issue though14:06
*** dave-mccowan has joined #openstack-ansible14:10
*** dave-mcc_ has joined #openstack-ansible14:12
evrardjpis it me or launchpad is in a bad shapre rn?14:13
evrardjpshape*14:13
hwoarangit's lp14:14
hwoarangbroken for me14:14
evrardjpoh I got an answer14:14
*** dave-mccowan has quit IRC14:15
openstackgerritMerged openstack/openstack-ansible-repo_server master: Revert "Remove sylinked git cache functionality"  https://review.openstack.org/53676314:16
*** indistylo has joined #openstack-ansible14:16
*** ivveh has joined #openstack-ansible14:17
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-openstack_hosts master: templates: hostfile-setup: Improve regexp for host entries  https://review.openstack.org/53684214:18
*** smatzek has quit IRC14:20
*** cmart has quit IRC14:28
evrardjpdear cloudnull, DimGR, andymccr, d34dh0r53, hughsaunders, b3rnard0, palendae, odyssey4me, serverascode, rromans, erikmwilson, mancdaz, _shaps_, BjoernT, claco, echiu, dstanek, jwagner, ayoung, prometheanfire, evrardjp, arbrandes, mhayden, scarlisle, luckyinva, ntt, javeriak, spotz, vdo, jmccrory, alextricity25, jasondotstar, admin0, michaelgugino, ametts, v1k0d3n, severion, bgmccollum, darrenc,14:33
evrardjpJRobinson__, asettle, colinmcnamara, thorst, adreznec, eil397, qwang,nishpatwa_, cathrichardson, drifterza, hwoarang, cshen, ullbeking:14:34
evrardjpthe osa bug triage will start in 1h30’. Please have a look at the bug list before starting: https://etherpad.openstack.org/p/osa-bugtriage14:34
evrardjpThank you14:34
*** jwitko has joined #openstack-ansible14:34
*** markvoelker has joined #openstack-ansible14:41
*** bhujay has joined #openstack-ansible14:43
*** cmart has joined #openstack-ansible14:44
openstackgerritMerged openstack/openstack-ansible-repo_build master: Ensure that all release files are cleaned up when rebuilding  https://review.openstack.org/53680214:47
evrardjpodyssey4me: let me backport this ^14:48
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible-repo_build stable/pike: Ensure that all release files are cleaned up when rebuilding  https://review.openstack.org/53685014:48
*** bhujay has quit IRC14:49
*** ykarel|afk has quit IRC14:50
*** bhujay has joined #openstack-ansible14:51
*** ykarel|afk has joined #openstack-ansible14:53
*** cjloader has joined #openstack-ansible14:54
*** dave-mcc_ has quit IRC14:55
*** yolanda has quit IRC14:56
*** gillesMo has joined #openstack-ansible15:02
mgariepyHow do I write my override to have this output ? https://github.com/openstack/openstack-ansible-plugins/blob/master/action/config_template.py#L130-L13415:03
mgariepyI'm trying to add 2 pci_alias in the nova.conf but i cannot seems to do that correclty.15:04
mgariepyi might also be mislead in thinking that is the way it works... :)15:05
*** dave-mccowan has joined #openstack-ansible15:05
*** smatzek has joined #openstack-ansible15:07
evrardjphttps://docs.openstack.org/openstack-ansible/latest/admin/advanced-config.html#original-template-file-test-ini-j215:07
evrardjpmgariepy: ^15:08
evrardjpshould have linked to https://docs.openstack.org/openstack-ansible/latest/admin/advanced-config.html#setting-overrides-in-configuration-files15:08
mgariepycan I add 2 lines with the same key ?15:09
hwoarangevrardjp: what do you think of https://review.openstack.org/#/c/536358/ ?15:09
*** shardy is now known as shardy_afk15:10
evrardjphwoarang: voted15:13
hwoarangah thanks!15:15
*** indistylo has quit IRC15:19
*** esberglu_ has joined #openstack-ansible15:23
*** esbergl__ has joined #openstack-ansible15:24
*** armaan has joined #openstack-ansible15:26
*** esberglu has quit IRC15:26
*** esberglu_ has quit IRC15:28
*** armaan_ has quit IRC15:29
openstackgerritMerged openstack/openstack-ansible master: Cleanup the gate-check-commit dstat part  https://review.openstack.org/53673815:29
*** esberglu has joined #openstack-ansible15:31
openstackgerritMerged openstack/openstack-ansible-openstack_hosts master: tests: Add missing pip_install role dependency  https://review.openstack.org/53635815:33
*** esbergl__ has quit IRC15:35
openstackgerritTaseer Ahmed proposed openstack/openstack-ansible-os_congress master: Introduce os_congress role in gerrit  https://review.openstack.org/52249115:35
*** pbandark has joined #openstack-ansible15:35
evrardjphwoarang: I am surprised you got that issue though, but whatever.15:37
evrardjpit doesn't hurt15:37
evrardjpand it makes sense15:37
evrardjpyou are probably hitting a precedence issue15:37
evrardjpone of the many15:37
evrardjp:p15:37
evrardjphttps://github.com/ansible/ansible/issues/3478215:38
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible stable/pike: Cleanup the gate-check-commit dstat part  https://review.openstack.org/53686815:41
hwoarangevrardjp: i think it makes sense because in zuul all the roles are linked in but in vagrant you have to be explicit on what ou want15:45
hwoarangevrardjp: how can I reproduce the galera thing? just deploy os_keystone with vagrant? because I wouldn't see it locally :/15:45
hwoarangi got a patch from colleen15:46
*** shardy_afk is now known as shardy15:46
evrardjphwoarang: yes I did that with my vagrant15:55
evrardjphwoarang: http://paste.openstack.org/show/650980/ <- a crop of my vagrant file15:56
evrardjptake what you need :D15:56
evrardjpyou're gonna get that issue.15:56
*** ykarel|afk is now known as ykarel15:58
evrardjpbug triage time!15:59
evrardjpBug triage cloudnull, DimGR, andymccr, d34dh0r53, hughsaunders, b3rnard0, palendae, odyssey4me, serverascode, rromans, erikmwilson, mancdaz, _shaps_, BjoernT, claco, echiu, dstanek, jwagner, ayoung, prometheanfire, evrardjp, arbrandes, mhayden, scarlisle, luckyinva, ntt, javeriak, spotz, vdo, jmccrory, alextricity25, jasondotstar, admin0, michaelgugino, ametts, v1k0d3n, severion, bgmccollum,15:59
evrardjpdarrenc, JRobinson__, asettle, colinmcnamara, thorst, adreznec, eil397, qwang,nishpatwa_, cathrichardson, drifterza, hwoarang, cshen15:59
evrardjpHere is our bug list for today https://etherpad.openstack.org/p/osa-bugtriage15:59
evrardjp#startmeeting openstack_ansible_meeting15:59
openstackMeeting started Tue Jan 23 15:59:52 2018 UTC and is due to finish in 60 minutes.  The chair is evrardjp. Information about MeetBot at http://wiki.debian.org/MeetBot.15:59
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:59
*** openstack changes topic to " (Meeting topic: openstack_ansible_meeting)"15:59
openstackThe meeting name has been set to 'openstack_ansible_meeting'15:59
evrardjp#topic rollcall16:00
*** openstack changes topic to "rollcall (Meeting topic: openstack_ansible_meeting)"16:00
spotz\o/16:00
evrardjphello spotz :)16:00
spotzhola16:00
evrardjpgiving a couple of minutes before starting the thing...16:01
evrardjp#topic Focus of the week16:02
*** openstack changes topic to "Focus of the week (Meeting topic: openstack_ansible_meeting)"16:02
hwoarango/16:02
odyssey4meo/16:02
d34dh0r53o/16:03
evrardjpThe FOTW (focus of the week, not fall of the wall) for this week should be "get your stuff in".16:03
evrardjpI think we have stabilized things the last two weeks, and I am confident the last bugs to iron out are easy to fix, so I think we should continue to innovate now!16:03
*** openstackgerrit has quit IRC16:03
*** rstarmer has joined #openstack-ansible16:04
evrardjpnext week is community meeting we should plan the 4 next focus of the week there.16:04
evrardjpok let's move on to...16:04
evrardjpthe real deal16:04
evrardjp#topic bug triage16:04
*** openstack changes topic to "bug triage (Meeting topic: openstack_ansible_meeting)"16:04
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/174468116:04
openstackLaunchpad bug 1744681 in openstack-ansible "Appendix B: Example production environment configuration in OpenStack-Ansible: clarification of infra load-balancers" [Undecided,Invalid]16:04
evrardjpI started to triage this one a little earlier, so I'd like to know if my triage was correct.16:05
evrardjpIf nobody complaints about this one, I move on.16:05
evrardjpit could be a doc bug, but I think it's more a support request.16:06
hwoarangyeah i agree16:06
evrardjpok16:06
evrardjplet's move on then16:06
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/174445816:06
openstackLaunchpad bug 1744458 in openstack-ansible "Failed to build cradox on CentOS7 " [Undecided,Incomplete]16:06
evrardjpmgariepy: mhayden are you there?16:06
mhaydenevrardjp: yeah, in a video meeting :/16:07
evrardjp:(16:07
evrardjpok if nobody can confirm, I'll leave it as new, instead of incomplete.16:08
evrardjpthe bug reporter gave all the information to confirm, we just don't have enough resources to check right now.16:08
evrardjpok16:08
evrardjpnext16:08
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/174401416:08
openstackLaunchpad bug 1744014 in openstack-ansible "Backups in OpenStack-Ansible" [Undecided,New]16:08
evrardjpThat seems valid and problematic indeed. It's not because we install xtrabackup that we do the backups.16:10
evrardjpI think that we are giving false expectations there, so I'd mark it as confirmed and high16:10
evrardjpor medium16:10
mgariepysorry i dont have time to validate16:11
evrardjpmgariepy: no worries.16:11
evrardjpok let's move on if ppl agree with the confirmed / medium for the docs issue.16:12
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/174380516:12
openstackLaunchpad bug 1743805 in openstack-ansible "neutron-db-manage fails on hostname with underscore" [Undecided,New]16:12
evrardjpI think we should mark it as incomplete, we don't know which branch...16:12
evrardjpagreed?16:13
evrardjpit might look like master, but I haven't seen that issue in master.16:13
hwoarangok16:14
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/174303216:14
openstackLaunchpad bug 1743032 in openstack-ansible "Galera cluster maintenance in OpenStack-Ansible" [Undecided,New] - Assigned to Kevin Carter (kevin-carter)16:14
evrardjpcloudnull: do you have an update on this one?16:15
evrardjpI'll leave it as is then.16:15
evrardjpnext16:16
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/174108516:16
openstackLaunchpad bug 1741085 in openstack-ansible "Playbook ceph-install.yml won't boostrap cluster" [Undecided,New]16:16
*** SerenaFeng has joined #openstack-ansible16:16
evrardjpandymccr logan- should we go ahead and remove the serial part there?16:16
evrardjpFor me the bug is completely invalid, as I see deploys with ceph working.16:17
evrardjpbut I understand why we'd want to remove that barrier if that's not useful16:17
odyssey4mebetter to variable it than remove it16:18
odyssey4methat's what we discussed last week IIRC16:18
odyssey4melogan- was also going to validate the issue16:19
odyssey4meI think16:19
evrardjpyeah logan- said he will try to validate it16:19
evrardjpyeah I like the idea to variable it.16:19
evrardjpI answered on the bug, saying we don't have issue in the deploys in our gates.16:20
odyssey4methat would be a good workaround for now16:20
odyssey4meour gate only has one mon, so it's not a good test of the issue16:20
evrardjpoh I see.16:20
*** rstarmer has quit IRC16:21
evrardjpok I will leave it as is then.16:22
evrardjpnext16:22
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/173947216:22
openstackLaunchpad bug 1739472 in openstack-ansible "mariadb-client excluded in ocata centos7 deploy" [Undecided,New]16:22
evrardjpanyone working on centos available to check that?16:23
hwoarangi could have a look at it since i am messing with galera already16:23
hwoarang*sigh*16:24
evrardjpthat would be nice16:24
evrardjpsorry for that :(16:24
hwoarangplease assign it to me16:24
evrardjpI think a vagrant machine should do the trick, see if there are issues there.16:24
evrardjpnext16:25
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/173886016:25
openstackLaunchpad bug 1738860 in openstack-ansible "nginx-common dependency prevents repo container build" [Undecided,Incomplete]16:25
evrardjpIncomplete, ok for you?16:25
hwoarangyep16:26
evrardjpok next16:27
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/173852916:27
openstackLaunchpad bug 1738529 in openstack-ansible "trove install fails" [Undecided,New]16:27
hwoarangwell16:28
hwoarangseems valid i suppose16:28
*** yifei has quit IRC16:28
*** yifei has joined #openstack-ansible16:29
evrardjpshould we do something like https://github.com/openstack/openstack-ansible-os_trove/blob/b6b933df6f915fb91b7e28a177cffb2b5ebcc1e0/tasks/trove_service_network.yml#L57 at L72?16:29
bhujayivve: I checked it and it works well  when os:nova:server is defined within the autoscalling group itself , here  is the template , please note two methods that i have tried http://paste.openstack.org/show/65098416:29
evrardjpyeah it seems valid.16:29
evrardjphwoarang: confirmed medium?16:30
bhujayivveh:^16:30
*** openstackgerrit has joined #openstack-ansible16:30
openstackgerritAndy McCrae proposed openstack/openstack-ansible master: Install mon servers in parallel.  https://review.openstack.org/53689116:30
evrardjpthat wasn't for hwoarang alone, sorry for that.16:30
hwoarangi think so16:30
evrardjpok16:30
evrardjpnext16:30
evrardjpthanks andymccr ;)16:30
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/173845116:30
openstackLaunchpad bug 1738451 in openstack-ansible "openstack_host_kernel_modules missing nf_conntrack_ipv6" [Undecided,New]16:30
andymccrevrardjp: np, i added logan- as a reviewer and -w'd it until he reviews it :)16:30
evrardjpthat looks valid and high.16:30
evrardjpandymccr: smart.16:31
hwoarangοκ16:31
hwoarang*ok16:31
evrardjphaha16:31
evrardjpkeyboard tricks!16:31
*** rstarmer has joined #openstack-ansible16:31
hwoarang:)16:32
evrardjpso anyone agrees on confirmed and high for the conntrack?16:32
evrardjpoh you said ok.16:32
evrardjpfor that :p16:32
hwoarangyeah fine by me16:32
hwoarangsounds reasonable16:32
andymccryeah seems sensible16:32
evrardjpnext then16:32
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/173782716:32
openstackLaunchpad bug 1737827 in openstack-ansible "(ceph-client): setting 'nova_ceph_client' results in deployment where volumes can't be attached to VMs" [Undecided,New]16:32
evrardjpthat's andymccr 's domain of expertise now, even if he'd say it's not :p16:33
evrardjp:D16:33
andymccrhm16:34
openstackgerritBjoern Teipel proposed openstack/openstack-ansible-ops master: Add insecure flag for post upgrade CLI commands  https://review.openstack.org/53597316:34
andymccrthat seems to me to be a var that exists to name space vars16:34
andymccrand then you use group_vars to glue them16:34
andymccrits kinda like saying if you change the default value of lots of settings it breaks things16:35
evrardjpyeah that could, but I like the idea of simplifying things with only one set :p16:35
evrardjpyeah16:35
evrardjpI think it's a big bug.16:35
ivvebhujay: cool, ill have a looksie16:35
*** bhujay has quit IRC16:35
evrardjpConfirmed and high?16:35
andymccri dont see how its high really16:35
andymccri mean if you change your keystone_endpoint vars to be something random they will fail too16:35
evrardjpyeah you're right.16:36
andymccrim not really sure what the fix even is? unless we take all the ceph bits out of the individual roles and make them a playbook16:36
andymccrwhich probably makes sense16:36
evrardjpyes I think it would make more sense...16:36
andymccrbut i think tahts a topic for the PTG :D16:36
andymccri will put a comment in that bug16:37
evrardjpyou can add it then :D16:37
evrardjp( andymccr : https://etherpad.openstack.org/p/osa-rocky-ptg )16:38
evrardjpok next16:38
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/172926316:38
openstackLaunchpad bug 1729263 in openstack-ansible "Swift (master) transient tempest failures under centos " [Undecided,New]16:38
evrardjplet's leave it as is16:38
evrardjpmove on16:38
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/172901416:38
openstackLaunchpad bug 1729014 in openstack-ansible "os_monasca: Pike: roles are pulling down master branch" [Undecided,New]16:38
evrardjpagreed with your comment there hwoarang16:39
evrardjpI don't know what we should do with things that are unmaintained16:39
evrardjpprobably mark it as wishlist?16:40
hwoarangyeah16:40
evrardjpok done16:40
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/172693616:40
openstackLaunchpad bug 1726936 in openstack-ansible "Configure rados_connect_timeout with reasonable timeout" [Undecided,Fix released]16:40
evrardjpjust marked it fix released, so let's move on16:40
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/172155416:40
openstackLaunchpad bug 1721554 in openstack-ansible "os_ceilometer fails without swift installed" [Medium,New]16:40
evrardjpdarn, ceilometer again.16:40
odyssey4meI think that monasca issue is already resolved16:41
evrardjpodyssey4me: oh interesting16:41
evrardjpwell it's still a whishlist if the role is unmaintained.16:41
*** pcaruana has quit IRC16:41
odyssey4mehttps://github.com/openstack/openstack-ansible-os_monasca/commit/f58399356fcd3766316c726f6fd397482a191e0516:41
evrardjpnice :)16:41
evrardjpfor ceilometer, I suggest we leave it as is for now.16:42
odyssey4memarking fix released16:42
evrardjpodyssey4me: thanks :D16:42
odyssey4methere is a team working on the monasca stuff around once per cycle to update it16:43
evrardjpoh interesting who is that?16:43
odyssey4methe same team also tends to work on sahara and some other odds and ends16:43
evrardjpMaybe we should tell them to be maintainer of the role, they would get an easier life.16:43
odyssey4mesee https://review.openstack.org/#/q/project:openstack/openstack-ansible-os_monasca for recent updates16:44
*** ivve has quit IRC16:44
odyssey4methey kinda do maintain it, just not actively all the time - only when they prep for their upgrades do they start testing and fixing16:44
odyssey4meit's a university I think16:45
evrardjpmmm ok. I think it's a conversation for outside the bug triage16:45
evrardjpIf I recall correctly, from Brasil16:45
evrardjpanyway... not helping the triage. Let's move on.16:45
odyssey4mehttp://stackalytics.com/?module=openstackansible-group&company=universidade%20federal%20de%20campina%20grande16:45
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/171951716:45
openstackLaunchpad bug 1719517 in openstack-ansible "Ceilometer policy.json includes python unicode encoding" [Undecided,New]16:45
evrardjpodyssey4me: nice :)16:46
evrardjpI will contact them.16:46
odyssey4meah, that old chestnut - I thought there was a patch up for that already?16:46
evrardjpnope.16:46
evrardjpit was, and then it broke, and then things again, or am I mistaken?16:47
odyssey4mehttps://review.openstack.org/50736816:47
odyssey4meah yes, as we don't have tests working we need to manually confirm that it works16:47
odyssey4meor we just merge it?16:48
evrardjphaha16:48
evrardjpblind approach.16:48
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer stable/ocata: Initial OSA zuul v3 role jobs  https://review.openstack.org/51664016:48
evrardjpI rechecked.16:49
odyssey4merebased the adding of tests on top of the patch - maybe it works or helps16:49
evrardjpok16:49
evrardjpyeah16:49
evrardjplet's see.16:49
*** chas has quit IRC16:49
evrardjpwait.16:49
*** chas has joined #openstack-ansible16:49
evrardjpI thought you'd do the opposite, but fine for me.16:50
evrardjpwe'll see.16:50
evrardjpnext one16:50
evrardjp#link https://bugs.launchpad.net/openstack-ansible/+bug/169887116:50
odyssey4memeh, same-same16:50
openstackLaunchpad bug 1698871 in openstack-ansible "[master] [os_gnocchi]Gnocchi role tests failing on installing pip packages" [Undecided,New]16:50
evrardjpfun stuff.16:50
evrardjpalways the same kind of trio of winners: telemetry, extra roles, and ceph.16:50
odyssey4methi s may be old and already fixed?16:51
evrardjpInvalid then16:51
odyssey4melooks like it's working? https://review.openstack.org/#/q/project:openstack/openstack-ansible-os_gnocchi16:51
evrardjpsomeone can reopn if needed.16:51
evrardjpyeah I kinda fixed master16:51
odyssey4methat said, maybe it's an integrated build issue - but it seems like it's old if alextricity reported it16:52
evrardjpgokhan_: is helping a lot on this one16:52
evrardjpyeah.16:52
evrardjpI'd say it's safe to mark it as invalid for now.16:52
odyssey4mesure16:52
odyssey4meor need more info to reproduce16:53
evrardjpyup incomplete is better. This way it expires.16:53
evrardjp:D16:53
evrardjpok so.16:53
evrardjpwe are done for today.16:53
evrardjpwe caught up everything.16:53
*** chas has quit IRC16:54
evrardjpthere is one bonus that's not on the triage because I just filed it, but let's skip that for now.16:54
evrardjpso thanks everyone!16:54
evrardjpnext week is community meeting16:54
hwoarangyay16:54
evrardjpwe'll discuss PTG, focus of the weeks, liaisons can talk about what they want, and we'll also have the floor open for discussions on specs or anything (works with whiskey questions for Dublin)16:55
evrardjphave a good day everyone!16:55
hwoarangyou too16:55
evrardjp#endmeeting16:55
*** openstack changes topic to "Launchpad: https://launchpad.net/openstack-ansible || Weekly Meetings: https://wiki.openstack.org/wiki/Meetings/openstack-ansible || Review Dashboard: https://goo.gl/tTmdgs"16:55
openstackMeeting ended Tue Jan 23 16:55:28 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:55
openstackMinutes:        http://eavesdrop.openstack.org/meetings/openstack_ansible_meeting/2018/openstack_ansible_meeting.2018-01-23-15.59.html16:55
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/openstack_ansible_meeting/2018/openstack_ansible_meeting.2018-01-23-15.59.txt16:55
openstackLog:            http://eavesdrop.openstack.org/meetings/openstack_ansible_meeting/2018/openstack_ansible_meeting.2018-01-23-15.59.log.html16:55
odyssey4methanks evrardjp16:55
hwoarangandymccr: do you think you can update http://blog.andy.mc.it/osa-dev-env/ to match the latest paths? it seems ~/git/openstack/ is now ~/src/git.openstack.org/16:56
hwoarangand i am surprised that there is no official documentation for this feature :)16:56
*** jbadiapa has quit IRC16:56
hwoarangor if it is it's well hidden :)16:56
*** ivve has joined #openstack-ansible16:56
*** chyka has joined #openstack-ansible17:07
*** rstarmer has quit IRC17:11
*** tacco_ is now known as tacco17:12
*** tacco has quit IRC17:12
*** tacco has joined #openstack-ansible17:12
ivveis ceilometer service disabled by default?17:15
*** yolanda has joined #openstack-ansible17:17
*** cjloader has quit IRC17:20
*** cjloader has joined #openstack-ansible17:21
*** openstacking_123 has joined #openstack-ansible17:27
ivvemeaning the api17:28
*** smatzek_ has joined #openstack-ansible17:34
*** gillesMo has quit IRC17:34
*** vnogin has quit IRC17:36
*** idlemind has joined #openstack-ansible17:39
*** yolanda has quit IRC17:47
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-lxc_hosts master: Properly match the LXC index item to download  https://review.openstack.org/53691517:48
odyssey4mehwoarang evrardjp not sure if you're still around, but ^ is a bug we uncovered today and would appreciate some eyes on17:50
*** hamza21 has joined #openstack-ansible17:51
odyssey4mehwoarang it was kinda rushed in, but yeah - a doc page for it would be nice :) is that you volunteering?17:51
odyssey4meivve ceilometer no longer has an api17:51
odyssey4megnocchi, aodh, panko all have API's IIRC - ceilometer now only collects data and pushes it to the other services17:52
*** chas has joined #openstack-ansible17:53
ivveaha17:54
ivvemetering in the api access was confusing17:54
ivve(im trying out some metering alarms17:54
ivve)17:54
ivveits was empty17:55
cloudnullafternoons!17:57
*** chas has quit IRC17:57
ivvehey:)17:58
*** SerenaFeng has quit IRC18:04
openstackgerritTaseer Ahmed proposed openstack/openstack-ansible-os_congress master: Introduce os_congress role in gerrit  https://review.openstack.org/52249118:06
*** rstarmer has joined #openstack-ansible18:11
*** vnogin has joined #openstack-ansible18:11
*** vnogin has quit IRC18:16
*** rstarmer has quit IRC18:19
*** mbuil has quit IRC18:19
*** shardy has quit IRC18:22
*** rstarmer has joined #openstack-ansible18:26
*** smatzek_ has quit IRC18:27
*** yolanda has joined #openstack-ansible18:29
*** indistylo has joined #openstack-ansible18:36
odyssey4mecloudnull if you have a moment, I'd appreciate some eyes for https://review.openstack.org/53691518:37
cloudnullodyssey4me: ++ will look right now18:43
*** indistylo has quit IRC18:44
odyssey4methx18:46
cloudnullodyssey4me: nice find!18:47
odyssey4meja, was quite an obscure bug18:47
odyssey4mewell, pair of bugs18:48
*** indistylo has joined #openstack-ansible18:49
*** smatzek has quit IRC18:54
mwynneHi guys. Has anyone seen this error before? I'm seeing this in my neutron openvswitch log: http://paste.openstack.org/show/651007/18:54
mwynneCan anyone help me out here? I've had this deployment running for quite some time, and after rebooting it I'm getting that error.18:55
*** smatzek has joined #openstack-ansible18:56
*** smatzek_ has joined #openstack-ansible18:58
*** cjloader has quit IRC19:00
*** cmurphy has joined #openstack-ansible19:00
*** smatzek has quit IRC19:00
*** cjloader has joined #openstack-ansible19:00
mwynneSamYaple: Any ideas?19:01
SamYaplemwynne: seems kind of like you lost the openvswitch db19:01
cmurphyhwoarang: hey are you still around? I'm wondering if you were able to start verifying https://review.openstack.org/#/c/536869/ ?19:01
mwynneSamYaple: haha, of course.19:02
mwynneSamYaple: Any way of verifying this?19:02
mwynneOr resolving this19:02
SamYaplemwynne: stop openvswitch. purge the database and containers. restart the box (required). then run the playbooks again19:04
SamYapleid try that if youre stuck19:05
mwynneSamYaple: "dpif_netlink|WARN|Generic Netlink family 'ovs_datapath' does not exist. The Open vSwitch kernel module is probably not loaded."19:05
mwynnelsmod | grep openvswitch returns nothing19:05
mwynnemaybe I just need to load the module???19:05
SamYaplemaybe. i dont really recall how kolla loads them19:06
mgariepylogan-, are you there?19:06
mwynneSamYaple: Not kolla... OSA19:06
SamYapleoops* openstack-ansible*19:06
SamYapleyea sorry19:06
SamYaplechannel confusion :)19:06
mgariepylogan-, are you running ceph luminous with bluestore ?19:07
SamYaplemwynne: lets wind back on my advice for a minute. give me a second to check something19:07
mwynneSamYaple: Yeha, I'm not going to run the playbooks again unless I HAVE to.19:08
mwynneSamYaple: Might have made some progress. Give me a sec here.19:08
SamYapleok let me know19:09
SamYapleah nvm. this is the issue i was thinking of, http://www.opencloudblog.com/?p=24019:09
*** smatzek_ has quit IRC19:09
SamYaplebut i dont think its related in this case19:09
mwynneSamYaple: For some reason the openvswitch kernel module wasn't loaded in the host...19:09
logan-o/ mgariepy, yes19:09
*** poopcat has joined #openstack-ansible19:10
mgariepyany issue ?19:10
SamYaplemwynne: hmmm make sure its in /etc/modules19:10
SamYaplethough the service should load it19:10
SamYaplei think OSA preloads modules it needs like that19:10
odyssey4meIIRc the modules should be loaded in the play run, but also added to /etc/modules to ensure they load again after a reboot19:11
odyssey4methe load should happen on the host if OVS is in a container though19:11
logan-mgariepy: beware the bluestore osd cache size. for ssd it is defaulted to 3GB. in reality I am rss usage around 5-6GB per osd19:11
mwynneSamYaple: It's not in /etc/modules19:12
* odyssey4me summons andymccr for the bluestore discussion. :)19:12
logan-mgariepy: although I think the latest ceph minor version bump fixed a memory leak in bluestore. we just rolled the new minor version with our meltdown patches last week, let me see where some osds are at now19:12
mwynneSamYaple: Can I just add it to the end of the list19:12
*** acormier has joined #openstack-ansible19:12
SamYaplemwynne: yes, that wont hurt anything19:12
*** poopcat1 has joined #openstack-ansible19:14
*** kstev has joined #openstack-ansible19:14
*** vnogin has joined #openstack-ansible19:16
*** poopcat has quit IRC19:17
logan-mgariepy: so one of the changes we've made is to drop the bluestore ssd cache size to 1GB. all of my ssd osds are currently running in the 1.7-1.9GB rss range19:18
logan-so as long as you budget memory accordingly you will be fine. my suggestion is budget 1.8-2x memory usage per osd from whatever you set the bluestore cache to19:19
logan-stability and performance are good, have not had any issues with luminous/bluestore aside from that. the mgr web status dashboard is a little bit junky still. i still always use https://github.com/Crapworks/ceph-dash instead19:21
mgariepygreat thanks for the hints logan-19:22
mwynneSamYaple: Thanks. I'll see if I can get something going now.19:23
logan-oh another thing to watch out for if you're upgrading from ocata mgariepy, check if you use ram/cpu ratio settings in host aggregates instead of nova.conf19:23
*** smatzek has joined #openstack-ansible19:23
logan-if you use host aggregates, read the thread on openstack-operators from last week19:24
*** vnogin has quit IRC19:24
mgariepyyeah I saw that19:24
logan-the most painful part of ocata for us by far has been nova-scheduler causing oom's on computes due to that19:25
mgariepydo you overcommit ram ?19:26
logan-no19:26
logan-we undercommit ram using host aggregates19:26
mgariepyok i'll take a deeper look into that then.19:26
logan-so after upgrading to ocata, that aggregate ram filter is not used in scheduling anymore19:26
odyssey4meouch19:27
logan-only the nova.conf settings are used by placement, so if you have ratios set in host aggregates then you no longer undercommit!19:27
logan-heh19:27
*** vnogin has joined #openstack-ansible19:28
*** RandomTech has quit IRC19:28
*** smatzek has quit IRC19:29
odyssey4melogan- so how is one supposed to do settings for host aggregates now?19:32
odyssey4meare you using extra groups for ansible and applying different overrides per group?19:33
logan-yes, we have to roll out nova.conf overrides to set the ratios on each compute19:33
logan-no cpu/ram/disk ratios can be set in host aggregates now19:34
*** mhayden_ has joined #openstack-ansible19:34
*** mhayden has quit IRC19:34
odyssey4meI've not seen the operators list post - has there been any feedback about how to do it differently, perhaps with one of the other scheduler filters or whatever?19:35
odyssey4meOr is this the way it is forever?19:35
*** phalmos has joined #openstack-ansible19:35
logan-some ideas are kicking around in there for changes to allow you to submit API calls to the placement service to set the ratios on the resource records for each compute19:36
logan-right now that is not possible because nova-compute has a periodic task that updates the placement service record with ratios sourced from nova.conf19:36
logan-so currently the only way to implement ratios is in nova.conf on each compute host19:37
*** smatzek has joined #openstack-ansible19:37
logan-there may be a switch added to disable that periodic ratio update task so operators can write their own ratios into placement and not have them overwritten19:38
odyssey4mewell, at least it's implemented in one place - consistency is useful in complex systems19:40
odyssey4meeven if the consistency is a pain19:40
*** smatzek has quit IRC19:42
logan-yeah it all makes capacity management more kludgy imo. moving compute hosts between aggregates with different resource alloc ratios/placement policies is much easier than having to maintain these records for each host. oh well19:42
*** smatzek has joined #openstack-ansible19:43
logan-at least most of the other aggregate filters still work, like the flavor metadata and max instances per host19:44
*** germs has joined #openstack-ansible19:45
*** smatzek has quit IRC19:47
*** smatzek has joined #openstack-ansible19:49
*** mhayden_ has quit IRC19:49
*** mhayden has joined #openstack-ansible19:50
*** chas has joined #openstack-ansible19:50
*** phalmos_ has joined #openstack-ansible19:50
*** ykarel is now known as ykarel|afk19:50
*** phalmos has quit IRC19:51
*** indistylo has quit IRC19:52
*** thanmad has joined #openstack-ansible19:52
*** smatzek has quit IRC19:53
*** chas has quit IRC19:54
*** smatzek has joined #openstack-ansible19:55
mwynneSamYaple: Another quick question for you. Does OSA deploy lxc containers for the l3 and metadata agents on a compute node for DVR?19:55
mwynneOr is it done on the host?19:55
*** vnogin has quit IRC19:58
*** smatzek has quit IRC19:59
SamYaplemwynne: sorry, im not the guy for that question, i dont work with OSA (or Kolla for that matter) much anymore. but my understanding would is on the computes, its not in a container19:59
odyssey4memwynne it'll be on the host as far as I know20:00
*** smatzek has joined #openstack-ansible20:01
thanmadAnsible-Hardening question - getting fatal on auditd.conf (ubuntu16.04-user:ubuntu) should this be running as a diff user?20:01
*** electrofelix has quit IRC20:01
odyssey4methanmad not sure about what's going on there, but the role should be run as root20:01
odyssey4meas far as I know at least - not sure if mhayden is around20:02
thanmadcheers, will try to adjust20:02
odyssey4meit may be a good idea to actually have it error out right at the start if it's running as a user which won't work, or alternatively perhaps it should make use of become appropriately20:02
odyssey4meif that's something you want, then I'd suggest registering a bug asking for it20:03
*** ykarel|afk has quit IRC20:03
thanmadsounds good.  first time trying from remote host, gotta figure out how to get in as root :)20:04
*** rstarmer has quit IRC20:05
mhaydenthanmad: i'm here (but horribly under the weather)20:05
mhaydenwhat did you find with auditd?20:06
*** smatzek has quit IRC20:06
*** ykarel|afk has joined #openstack-ansible20:06
*** smatzek has joined #openstack-ansible20:13
mwynnethanks SamYaple odyssey4me20:14
*** chason has quit IRC20:16
*** smatzek has quit IRC20:17
*** smatzek has joined #openstack-ansible20:19
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-galera_server master: tasks: galera_post_install: Configure the service before securing the DB  https://review.openstack.org/53695420:22
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-galera_server master: SUSE: Switch to upstream MariaDB repository  https://review.openstack.org/53695520:22
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: DNM - Test galera  https://review.openstack.org/49724120:23
*** smatzek has quit IRC20:23
*** smatzek has joined #openstack-ansible20:24
*** Jeffrey4l has quit IRC20:25
*** Jeffrey4l has joined #openstack-ansible20:25
*** armaan has quit IRC20:27
*** chason has joined #openstack-ansible20:28
*** armaan has joined #openstack-ansible20:28
*** smatzek has quit IRC20:29
*** hw_wutianwei has quit IRC20:31
*** hw_wutianwei has joined #openstack-ansible20:32
*** smatzek has joined #openstack-ansible20:32
*** germs has quit IRC20:35
*** hamza21 has quit IRC20:48
*** ykarel|afk has quit IRC20:59
thanmadmhayden: thanks got it sorted.   needed to modify the playbook for become: true.  rookie mistake21:02
thanmadthx all21:03
*** pramodrj07 has joined #openstack-ansible21:03
mhaydenoh okay21:03
mhaydengood stuff21:03
*** rstarmer has joined #openstack-ansible21:05
*** smatzek_ has joined #openstack-ansible21:15
*** smatzek_ has quit IRC21:15
*** germs has joined #openstack-ansible21:17
*** germs has quit IRC21:17
*** smatzek has quit IRC21:23
*** chas has joined #openstack-ansible21:35
openstackgerritMerged openstack/openstack-ansible stable/pike: Cleanup the gate-check-commit dstat part  https://review.openstack.org/53686821:36
odyssey4memhayden got a minute to peek at https://review.openstack.org/536915 ?21:36
odyssey4methat needs to be pulled back to pike once it merges21:36
*** chas has quit IRC21:39
*** dave-mccowan has quit IRC21:43
mhaydenodyssey4me: lookin'21:44
* mhayden is behind on reviews today21:44
odyssey4metyvm sir21:44
odyssey4meyou have plenty of reviews under your belt in stackalytics this cycle, so slacking off for a day is not a worry ;)21:44
mhaydenbut my queens cloud is humming along21:44
mhaydenoh nice catch on that one21:45
openstackgerritMajor Hayden proposed openstack/openstack-ansible-os_magnum master: Fix linting error with systemctl commands  https://review.openstack.org/53698421:52
openstackgerritMajor Hayden proposed openstack/openstack-ansible-os_magnum master: Fix linting error with systemctl commands  https://review.openstack.org/53698421:53
openstackgerritMajor Hayden proposed openstack/openstack-ansible-os_magnum master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53643021:53
*** armaan has quit IRC21:54
*** armaan has joined #openstack-ansible21:54
*** armaan has quit IRC21:56
openstackgerritMajor Hayden proposed openstack/openstack-ansible-os_trove master: Updated from OpenStack Ansible Tests  https://review.openstack.org/53644121:57
openstackgerritMajor Hayden proposed openstack/openstack-ansible-os_trove master: Use systemctl module for daemon reload  https://review.openstack.org/53698521:57
openstackgerritMerged openstack/openstack-ansible-os_ceilometer stable/ocata: Remove python unicode encoding from policy.json file  https://review.openstack.org/50736821:57
*** armaan has joined #openstack-ansible21:57
openstackgerritMerged openstack/openstack-ansible-ops master: Remove BOOTSTRAP_ANSIBLE_FOLDER variable  https://review.openstack.org/53653722:00
* mhayden is catchin' up22:00
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build master: Correct location of venv build logs  https://review.openstack.org/53698722:01
openstackgerritMajor Hayden proposed openstack/openstack-ansible-os_magnum master: Updated from global requirements  https://review.openstack.org/53506122:01
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer stable/ocata: Initial OSA zuul v3 role jobs  https://review.openstack.org/51664022:02
odyssey4methat gets us a working gate for os_ceilometer on ocata22:02
*** dave-mccowan has joined #openstack-ansible22:03
mhaydenyay22:05
mhaydenyou might say the lack of that was putting a ceil-ing on our efforts22:05
* mhayden chuckles22:05
mhaydenodyssey4me: can i trade you for a gander at https://review.openstack.org/534327 ?22:07
odyssey4menot so sure about https://review.openstack.org/527592 mhayden - any idea why those conditions are removed?22:08
mhaydenthe disabling of the quota part? looks like it's disabled all the time now22:08
odyssey4meno, the rfresh of the image22:09
*** jwitko_ has joined #openstack-ansible22:09
odyssey4meit looks like every run will remove and add it back22:10
mhaydenhmm, there's a check for machinectl_mount.rc != 0 at the end22:10
mhaydenoh hold on, i was looking at the old systemd prep22:11
odyssey4meboth old and new do it - towards the end after the setup22:11
mhaydenoh yeah, it does look like the image cache is refreshed each time22:12
odyssey4menot sure if that's intentional or not22:13
odyssey4meif so, it's not explained in the commit msg or anything22:13
openstackgerritMerged openstack/openstack-ansible-lxc_hosts master: Properly match the LXC index item to download  https://review.openstack.org/53691522:13
odyssey4mewell, as far as I could see - it's late, so maybe I'm being dumb22:13
*** jwitko has quit IRC22:13
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-lxc_hosts stable/pike: Properly match the LXC index item to download  https://review.openstack.org/53699122:13
* mhayden looks around to find where cache_refresh_del/add are set22:13
odyssey4meif you could add your vote to that there backport I'd be much obliged :) cc cloudnull if you're still around22:14
openstackgerritMerged openstack/openstack-ansible-os_neutron master: Adds Neutron OVS DVR Documentation  https://review.openstack.org/53279822:15
mhaydenoh, i think i see what's up22:15
mhaydenit seems like the conditional should be there22:16
* mhayden adds a comment22:16
*** jwitko_ has quit IRC22:17
* mhayden transforms into dad taxi22:17
mhaydentalk to y'all tomorrow22:17
*** mhayden has quit IRC22:17
odyssey4mecheerio!22:18
openstackgerritTaseer Ahmed proposed openstack/openstack-ansible-os_congress master: Introduce os_congress role in gerrit  https://review.openstack.org/52249122:22
*** acormier has quit IRC22:34
openstackgerritMerged openstack/openstack-ansible-repo_build stable/pike: Ensure that all release files are cleaned up when rebuilding  https://review.openstack.org/53685022:34
*** acormier has joined #openstack-ansible22:34
*** acormier has quit IRC22:35
*** acormier has joined #openstack-ansible22:35
*** dave-mccowan has quit IRC22:37
*** acormier has quit IRC22:37
*** acormier has joined #openstack-ansible22:37
*** esberglu has quit IRC22:38
*** jwitko has joined #openstack-ansible22:40
openstackgerritMerged openstack/ansible-hardening master: Use import/include_tasks  https://review.openstack.org/53432722:43
*** cmurphy has left #openstack-ansible22:44
*** cjloader has quit IRC23:05
*** acormier has quit IRC23:12
*** acormier has joined #openstack-ansible23:13
*** masber has quit IRC23:14
*** acormier has quit IRC23:18
*** kstev has quit IRC23:20
*** chas has joined #openstack-ansible23:23
*** acormier has joined #openstack-ansible23:26
*** chas has quit IRC23:27
*** acormier has quit IRC23:30
openstackgerritMerged openstack/openstack-ansible master: Updated from global requirements  https://review.openstack.org/53503623:36
*** oanson has quit IRC23:42
*** openstacking_123 has quit IRC23:42
*** oanson has joined #openstack-ansible23:43
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build master: [WIP] Make the git clone and wheel build optional  https://review.openstack.org/53701523:50
*** dave-mccowan has joined #openstack-ansible23:51
*** smatzek has joined #openstack-ansible23:53
*** smatzek has quit IRC23:55

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