Wednesday, 2016-10-12

*** dfflanders has joined #openstack-ansible00:02
*** williamcaban has quit IRC00:04
*** williamcaban has joined #openstack-ansible00:05
*** rmelero has quit IRC00:07
*** ikalnytskyi_ has joined #openstack-ansible00:08
*** mubix has quit IRC00:08
*** chris_hultin|AWA has quit IRC00:08
*** mubix_ has joined #openstack-ansible00:09
*** ikalnytskyi has quit IRC00:09
*** ikalnytskyi_ is now known as ikalnytskyi00:09
*** aludwar has quit IRC00:09
*** chris_hultin|AWA has joined #openstack-ansible00:10
*** batkins61 has quit IRC00:10
*** chris_hultin|AWA is now known as chris_hultin00:10
*** aludwar has joined #openstack-ansible00:11
*** gouthamr has joined #openstack-ansible00:16
*** chris_hultin has quit IRC00:19
*** adrian_otto has joined #openstack-ansible00:21
*** sdake_ is now known as sdake00:22
*** gouthamr has quit IRC00:23
*** gouthamr has joined #openstack-ansible00:23
*** aetaric_ has quit IRC00:23
*** toan has quit IRC00:24
*** b3rn-n00dl3s has quit IRC00:24
*** b3rn-n00dl3s has joined #openstack-ansible00:25
*** chris_hultin|AWA has joined #openstack-ansible00:28
*** chris_hultin|AWA is now known as chris_hultin00:28
*** markvoelker has quit IRC00:28
*** toan has joined #openstack-ansible00:29
*** aetaric has joined #openstack-ansible00:29
*** adrian_otto has quit IRC00:29
*** adrian_otto has joined #openstack-ansible00:31
*** adrian_otto has quit IRC00:33
*** adrian_otto has joined #openstack-ansible00:36
*** markvoelker has joined #openstack-ansible00:36
*** mubix_ has quit IRC00:38
*** mubix_ has joined #openstack-ansible00:40
*** mgagne has quit IRC00:40
*** cmart has quit IRC00:42
*** adrian_otto has quit IRC00:42
*** mgagne has joined #openstack-ansible00:45
*** sdake has quit IRC00:46
*** jamielennox is now known as jamielennox|away00:56
*** aetaric has quit IRC00:59
*** sdake has joined #openstack-ansible01:01
*** jamielennox|away is now known as jamielennox01:02
*** thorst has joined #openstack-ansible01:04
*** aetaric has joined #openstack-ansible01:05
openstackgerritJimmy McCrory proposed openstack/openstack-ansible-os_nova: Use run_once to create authorized keys file script  https://review.openstack.org/38522101:05
openstackgerritJimmy McCrory proposed openstack/openstack-ansible-os_nova: Use run_once to create authorized keys file script  https://review.openstack.org/38522101:06
*** dfflanders has quit IRC01:07
*** chris_hultin has quit IRC01:09
*** chris_hultin|AWA has joined #openstack-ansible01:17
*** javeriak has joined #openstack-ansible01:17
*** chris_hultin|AWA is now known as chris_hultin01:17
openstackgerritMerged openstack/openstack-ansible: Fix value for openstack_host_manage_hosts_file  https://review.openstack.org/38508901:18
*** jcrst has quit IRC01:20
*** thorst has quit IRC01:22
*** javeriak has quit IRC01:27
*** Mudpuppy has joined #openstack-ansible01:28
*** dfflanders has joined #openstack-ansible01:37
*** jcrst has joined #openstack-ansible01:38
openstackgerritMerged openstack/openstack-ansible: Move inventory generation code into lib  https://review.openstack.org/38315901:45
*** woodard has joined #openstack-ansible01:47
*** woodard has quit IRC01:50
*** woodard has joined #openstack-ansible01:51
*** phalmos_ has joined #openstack-ansible01:55
*** phalmos has quit IRC01:55
*** thorst has joined #openstack-ansible01:58
*** thorst has quit IRC02:03
*** dfflanders has quit IRC02:09
*** jamesdenton has joined #openstack-ansible02:11
*** woodard has quit IRC02:14
*** rmelero has joined #openstack-ansible02:26
*** javeriak has joined #openstack-ansible02:28
*** rmelero has quit IRC02:36
*** rmelero has joined #openstack-ansible02:37
*** jcrst has quit IRC02:41
*** Mudpuppy has quit IRC02:41
*** kjw3 has quit IRC02:41
*** Mudpuppy has joined #openstack-ansible02:42
*** Mudpuppy has quit IRC02:46
*** dragonmaster has joined #openstack-ansible02:47
*** shan5464_ has quit IRC02:47
*** palendae has quit IRC02:49
*** palendae has joined #openstack-ansible02:50
*** palendae is now known as Guest3433902:50
*** shananigans has joined #openstack-ansible02:52
*** Guest34339 is now known as palendae02:52
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible: Update the neutron MTU settings on upgrade  https://review.openstack.org/38463502:56
*** sdake has quit IRC02:57
openstackgerritMerged openstack/openstack-ansible-pip_install: Ensure that package installs can fall back  https://review.openstack.org/38496603:00
openstackgerritMerged openstack/openstack-ansible-pip_install: Ensure that package installs can fall back  https://review.openstack.org/38496903:00
matttcloudnull: tried that again, not sure it worked03:08
cloudnullmattt: the script?03:08
cloudnullfor mtu changes?03:08
cloudnullis it the script or it it not executing ?03:09
matttcloudnull: i didn't catch a glimpse of the script before it got cleaned up, but my vxlan network didn't get an MTU of 145003:09
*** jamesdenton has quit IRC03:09
cloudnullwhat got set?03:09
cloudnullnothing?03:09
matttjust the public network03:10
matttlet me re-run it and disable script from getting deleted03:10
matttbrb03:10
*** gouthamr has quit IRC03:10
matttoh, it doesn't get cleaned up03:11
*** aetaric has quit IRC03:12
cloudnullinteresting. i see the same on a quite test.03:13
cloudnullpoking at it now03:13
*** palendae has quit IRC03:14
*** odyssey4me has quit IRC03:14
matttcloudnull: ok, so here's what i get03:14
matttcloudnull: https://gist.github.com/mattt416/c1566302d19912846bb652714c330492 ... the vxlan network has physical_network=NULL in the db03:14
*** odyssey4me has joined #openstack-ansible03:15
matttcloudnull: network_type=vxlan, physical_network=NULL03:15
*** aetaric has joined #openstack-ansible03:15
*** palendae has joined #openstack-ansible03:16
*** palendae is now known as Guest3506903:16
cloudnullso for a physical_network=NULL we should just assume 1500 ?03:18
cloudnullor 1450 ?03:18
*** sdake has joined #openstack-ansible03:18
matttcloudnull: refresh that gist, i dumped the mysql table in there so you can see03:19
*** jamielennox is now known as jamielennox|away03:20
cloudnullok so if its NULL I think we should just assume its a tenant type network and use 145003:21
*** Guest35069 has quit IRC03:21
cloudnullI have an AIO up that looks like so http://cdn.pasteraw.com/jjyea2e5ow6b3n9cugl6reg5w86pakp03:21
cloudnullan both the ha net and PRIVAATE are based on tenant nets03:22
cloudnullso that makes sense.03:22
*** palendae_ has joined #openstack-ansible03:22
*** palendae_ is now known as Guest5535503:23
matttcloudnull: yeah, i can't really suggest which way to go here, i'm horrible w/ networking and neutron03:23
matttbut i went on the assumption that vxlan == 1450, flat/vlan == 150003:23
*** williamcaban has quit IRC03:24
mattthow do network_type and physical_network fields differ?03:24
*** Guest55355 is now known as palendae03:24
matttah, the physical_network is the name of the physical network03:25
matttwhy not use network_type instead ?03:25
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible: Update the neutron MTU settings on upgrade  https://review.openstack.org/38463503:26
cloudnullgive that a spin03:27
cloudnulli used physical_network because while we name them to be the same thing as the type a deploy may have changed that up03:27
irtermitenice03:27
cloudnulland they may have different MTUs set for those types03:27
matttyeah it'd seem to make more sense to use network_type, since that's more accurate as to what it actually is03:28
matttlet me re-test that03:28
cloudnullexcept in the case where my type is vlan and I set the mtu to 9000 for only 1 of the vlan nets i have03:29
cloudnullI guess we could pull the type and use that as a fall back when the name is unset03:30
matttyeah tbh you'd know more about this than i03:30
matttso perhaps use whatever method you think is best03:31
mattt(and confirmed, that worked)03:31
matttthere is one last change i'd make03:31
mattti'll suggest it in the review itself03:31
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible: Update the neutron MTU settings on upgrade  https://review.openstack.org/38463503:34
cloudnullmattt: ^ updated to use type when going into fall back03:35
cloudnullso it'll use the name first and if nothing is found it'll use the type03:35
matttcloudnull: cool, i have to jet but i'll try to decipher that template a bit more tomorrow :)03:36
matttbut seems to be working in my very limited AIO instance03:36
*** jamielennox|away is now known as jamielennox03:38
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible: Update the neutron MTU settings on upgrade  https://review.openstack.org/38463503:49
cloudnullmattt: ^  implemented your sugestion on looking for mtu=003:49
cloudnullok im off.03:50
cloudnullcheers all03:50
*** rmelero has quit IRC03:51
*** haasn has quit IRC04:00
*** haasn has joined #openstack-ansible04:03
*** Jeffrey4l_ has quit IRC04:05
*** palendae has quit IRC04:11
*** palendae has joined #openstack-ansible04:15
*** palendae is now known as Guest8003004:15
*** chris_hultin has quit IRC04:24
*** dragonmaster has quit IRC04:26
*** kjw3 has joined #openstack-ansible04:26
*** Jeffrey4l has joined #openstack-ansible04:26
*** dragonmaster has joined #openstack-ansible04:27
*** chris_hultin|AWA has joined #openstack-ansible04:28
*** chris_hultin|AWA is now known as chris_hultin04:28
*** rmelero has joined #openstack-ansible04:30
*** vishwanathj has quit IRC04:32
*** Guest80030 is now known as palendae04:40
*** agireud has quit IRC04:42
*** rmelero has quit IRC04:43
*** dragonmaster has quit IRC04:43
*** kjw3 has quit IRC04:43
*** dragonmaster has joined #openstack-ansible04:44
*** dragonmaster has quit IRC04:45
*** agireud has joined #openstack-ansible04:51
*** rmelero has joined #openstack-ansible04:54
*** poopcat has quit IRC04:55
openstackgerritTuan Luong-Anh proposed openstack/openstack-ansible: Remove xrange for run both Python 2 and Python 3  https://review.openstack.org/38527104:55
*** agireud has quit IRC04:56
*** dragonmaster has joined #openstack-ansible04:58
*** dragonmaster has quit IRC04:58
*** dragonmaster has joined #openstack-ansible05:01
openstackgerritPraveen N proposed openstack/openstack-ansible: Add network config example for test and prod  https://review.openstack.org/38527205:02
*** Jeffrey4l has quit IRC05:09
*** chhavi has joined #openstack-ansible05:11
*** Jeffrey4l has joined #openstack-ansible05:11
*** Jeffrey4l has quit IRC05:12
*** Jeffrey4l has joined #openstack-ansible05:13
*** Jeffrey4l has quit IRC05:13
*** Jeffrey4l_ has joined #openstack-ansible05:13
*** aetaric has quit IRC05:16
*** dragonmaster has quit IRC05:16
*** rmelero has quit IRC05:16
*** admin0 has joined #openstack-ansible05:18
*** rmelero has joined #openstack-ansible05:18
*** aetaric has joined #openstack-ansible05:25
*** Jack_Iv has joined #openstack-ansible05:25
*** chris_hultin has quit IRC05:27
*** chris_hultin|AWA has joined #openstack-ansible05:30
*** chris_hultin|AWA is now known as chris_hultin05:30
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Fix value for openstack_host_manage_hosts_file  https://review.openstack.org/38527705:31
*** janki has joined #openstack-ansible05:32
*** pjm6_ has joined #openstack-ansible05:35
*** kylek3h_ has joined #openstack-ansible05:36
*** kylek3h has quit IRC05:38
*** pjm6 has quit IRC05:39
openstackgerritTuan Luong-Anh proposed openstack/openstack-ansible-plugins: Remove xrange for run both Python 2 and Python 3  https://review.openstack.org/38528005:39
*** Jeffrey4l_ has quit IRC05:41
*** admin0 has quit IRC05:47
*** Jeffrey4l_ has joined #openstack-ansible05:52
*** Jeffrey4l_ has quit IRC05:57
*** Jeffrey4l_ has joined #openstack-ansible05:58
*** Jeffrey4l_ has quit IRC06:06
*** rmelero has quit IRC06:15
*** Jeffrey4l_ has joined #openstack-ansible06:19
*** rmelero has joined #openstack-ansible06:20
ionimorning06:20
*** rmelero has quit IRC06:20
*** Jeffrey4l_ has quit IRC06:23
*** Jeffrey4l_ has joined #openstack-ansible06:23
*** cristicalin has joined #openstack-ansible06:38
*** messy has quit IRC06:39
*** admin0 has joined #openstack-ansible06:43
*** messy has joined #openstack-ansible06:50
*** haad1 has joined #openstack-ansible06:53
*** chhavi has quit IRC07:00
*** chris_hultin has quit IRC07:10
*** cristicalin has quit IRC07:11
*** chris_hultin|AWA has joined #openstack-ansible07:12
*** chris_hultin|AWA is now known as chris_hultin07:12
*** asettle has joined #openstack-ansible07:13
*** haad1 has quit IRC07:13
*** asettle has quit IRC07:13
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova: Fix ironic compute_driver  https://review.openstack.org/38530807:16
*** dragonmaster has joined #openstack-ansible07:19
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests: Fix glance_api_servers variable  https://review.openstack.org/38531207:20
*** sdake has quit IRC07:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer: Changed the home-page link  https://review.openstack.org/38222007:23
*** rgogunskiy has joined #openstack-ansible07:26
*** haad1 has joined #openstack-ansible07:26
*** fops_ has joined #openstack-ansible07:27
*** williamcaban has joined #openstack-ansible07:28
*** fops has quit IRC07:30
*** javeriak has quit IRC07:38
*** jcrst has joined #openstack-ansible07:38
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [docs] Add network config example for test and prod  https://review.openstack.org/38527207:40
*** drifterza has joined #openstack-ansible07:42
*** williamcaban has quit IRC07:45
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Update all SHAs for Ocata 2016-10-08  https://review.openstack.org/38032507:45
*** fops_ has quit IRC07:46
*** jcrst has quit IRC07:46
drifterzaehlo all07:46
andymccrmorning drifterza!07:48
drifterzahows things andymccr07:49
openstackgerritAndy McCrae proposed openstack/openstack-ansible-os_ironic: [WIP] Enable virtualBMC testing  https://review.openstack.org/38104907:49
andymccrpretty good - yourself?07:50
drifterzaagg okay, drawing diagrams. The bane of life.07:50
andymccrhahahahahaha07:51
andymccrpeople love pictures07:51
drifterzapeople only understand pictures dude.07:51
evrardjpgood morning07:52
andymccrmornin' evrardjp!07:55
evrardjpfun spec in neutron coming out https://review.openstack.org/#/c/360620/6/specs/ocata/bgp-evpn-advertisment.rst07:56
evrardjpandymccr: did you pass your coa?07:56
andymccrevrardjp: mine is tomorrow07:57
andymccrso not yet :)07:57
evrardjpk07:57
*** ssbarnea has joined #openstack-ansible07:59
ioniguys, i have questions regarding haproxy and the new ssl thingy08:06
ionihow can one make that use a valid ssl and also be valid on an ip?08:06
ionissls are valid on domains08:06
ioniunless i can configure it in openstack_user_config to use domain instead of ip08:07
ionias a feedback from M->N, i noticed that old configs are not purged08:08
ioni-rw-r--r-- 1 root root 1.4K Oct 11 22:48 ceilometer-agent-notification.conf08:09
ioni-rw-r--r-- 1 root root 1.2K Jul 20 15:51 ceilometer-api.conf08:09
ioni-rw-r--r-- 1 root root 1.3K Oct 11 22:48 ceilometer-polling.conf08:09
ioniceilometer-api.conf should have been purged, because the api now is in wsgi apache08:09
ionisame with haproxy, there were nova console configuration duplicate, old and new08:09
ionito be sure that i'm not running old services, should i destroy the containers and recreate them?08:10
evrardjpandymccr: odyssey4me do you have prio reviews for newton right now?08:11
evrardjpioni you can use a name in the external lb part of your openstack_user_config08:11
ionievrardjp, ok08:13
evrardjpioni I'm not sure about your configuration duplicates, everything should be reconfigured for the latest version, what do you mean?08:13
evrardjpioni pay attention, changing this cause a lot of issues08:13
evrardjpbecause you will change your openstack endpoints08:13
evrardjpso depending on what you need, you may think twice08:13
ionievrardjp, about the external lb part?08:13
evrardjpyes08:14
ionii really liked the old way without having everything through a global ssl08:14
ionii'll look to see if i can disable it08:14
evrardjpoh I understand better what you mean now08:14
evrardjpYou'd prefer going back to http by default instead of https08:15
ioniyes08:15
openstackgerritMerged openstack/openstack-ansible: Use UCA for keepalived by default  https://review.openstack.org/38514808:15
evrardjpwell at that time I was fighting to have https myself, so I guess we have different needs :p08:15
ionihttps even if i configure it with a valid ssl, it won't be valid because isn't configured on an ip08:15
ioninot on a domain08:15
odyssey4meandymccr FYI https://review.openstack.org/385334 - adding a skip for all os_ repositories for CentOS7 where they don't have it in newton yet08:16
evrardjpI understand it's hard to move when you've defined your endpoints and work patterns already08:16
andymccrodyssey4me: nice!08:17
odyssey4meevrardjp yes, priority reviews relating to the newton release are https://review.openstack.org/#/q/starredby:jesse-pretorius+project:%255Eopenstack/openstack-ansible.*+status:open08:17
ionievrardjp, about the old configs08:17
ionilets take as example my ceilomenter api container08:17
evrardjpok odyssey4me08:18
ionihttps://paste.xinu.at/E8S7/08:18
ioniyou see there 13.2.0/bin/ceilometer-api has been started?08:18
ionibecause there is still a /etc/init/ceilometer-api.conf08:18
evrardjpyes, I see08:19
ionithe old config was not purged08:19
evrardjplet me have a look it's been a while I didn't check this08:19
ionisame thing happen with haproxy configs08:19
ioni-rw-r--r-- 1 root root  390 Jun 28 12:11 ironic_api08:20
ioni-rw-r--r-- 1 root root 1.2K Oct 11 20:31 keystone_admin08:20
ioni-rw-r--r-- 1 root root  589 Jun 28 12:11 keystone_internal08:20
ioni-rw-r--r-- 1 root root 1.1K Oct 11 20:31 keystone_service08:20
ionithe keystone internal should have been purged08:20
evrardjplet's take the two cases independently08:21
ioniright08:21
evrardjpfirst the haproxy because it's easier for me :D08:21
evrardjpyou're listing conf.d right?08:21
ioniyes08:21
evrardjpand everything in your conf.d appears in your haproxy.cfg now08:21
evrardjpso the upgrade should have removed what's not deployed in conf.d08:22
evrardjpanymore*08:22
odyssey4methat'll need a bug which we can reference08:22
evrardjpso you're right there could be a bug there, due to the way we are generating our haproxy config08:22
evrardjpthat deserves a bug :D08:22
odyssey4meI think that ceilometer's API service now *has* to run behind a web server, so the clean up of old conf files and init files needs to happen in upgrades too08:23
odyssey4methat should be another bug08:23
evrardjpodyssey4me: oh ok I wasn't aware of this08:23
evrardjpioni: could you file the bugs please?08:23
evrardjpplease mention it's for upgrades :p08:23
odyssey4meevrardjp I think - I vaguely remember a last minute change for ceilometer or something in telemetry.08:23
odyssey4meioni yes, please add 'M>N Upgrade:' as a prefix to the subjet line of your bug08:24
evrardjpwell it makes sense to remove init file as ioni said08:24
ionievrardjp, about haproxy.cfg, i cannot say, i didn't saved it08:24
evrardjpioni: what do you mean with your haproxy.cfg?08:25
ioni<evrardjp> and everything in your conf.d appears in your haproxy.cfg now08:25
ioni<evrardjp> so the upgrade should have removed what's not deployed in conf.d08:25
evrardjpIt would be important for the haproxy bug to dump your configuration08:25
ionii cannot be sure about that one08:25
ionii only saved conf.d as conf.d.old08:25
ionii was thinking that it takes the configuration from there, like it was in the past08:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_aodh: Update paste, policy and rootwrap configurations 2016-10-12  https://review.openstack.org/38533908:26
ioninow i see that whatever is in that folder is now in haproxy.cfg08:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer: Update paste, policy and rootwrap configurations 2016-10-12  https://review.openstack.org/38534008:26
evrardjpioni: haproxy role now uses conf.d to generate one big haproxy.cfg due to the way systemd works by default08:26
evrardjpand haproxy*08:26
ioniright08:26
ionii have to take in consideration to edit haproxy.cfg when debugging and not conf.d :)08:27
evrardjpso if there is a conf.d file that shouldn't be there, it will still be there for generating the haproxy.cfg08:27
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_heat: Update paste, policy and rootwrap configurations 2016-10-12  https://review.openstack.org/38534108:27
evrardjpioni the problem with haproxy is double08:27
evrardjpor could be double08:27
ioniok, i'll submit both bugs08:28
evrardjpfor haproxy: I'd like to know which init of haproxy you have, and what happens if you completely flush the conf.d folder and restart haproxy manually08:28
evrardjpwell not completely flush, keep one for fun08:28
ionii have ubuntu 14 sa it's upstrat init08:29
ioniupstart08:29
evrardjpoh there is even a better solution to know this08:29
evrardjpcould you simply do a ps aux | grep haproxy in the bug?08:29
evrardjpand drop the haproxy.cfg08:30
evrardjpthat would be good enough for me08:30
ioniis just using the global haxproy.cfg08:30
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_magnum: Update paste, policy and rootwrap configurations 2016-10-12  https://review.openstack.org/38534308:30
evrardjpok08:30
evrardjpso the init upgrade is fine, it's just remnants in conf.d08:30
ioniyes08:31
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Update all SHAs for Ocata 2016-10-12  https://review.openstack.org/38032508:31
*** berendt has joined #openstack-ansible08:32
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible: Set default keepalived cidr if none is provided  https://review.openstack.org/38506908:32
ionihttps://bugs.launchpad.net/openstack-ansible/+bug/163260408:33
openstackLaunchpad bug 1632604 in openstack-ansible "During upgrare M->N old ceilometer startap configurations are not purged" [Undecided,New]08:33
ionifirst one08:33
ionihttps://bugs.launchpad.net/openstack-ansible/+bug/163260908:36
openstackLaunchpad bug 1632609 in openstack-ansible "During upgrare M->N old haproxy configurations from conf.d are not purged" [Undecided,New]08:36
ionisecond one08:36
odyssey4meevrardjp I wonder whether https://review.openstack.org/385069 should include a release note with something like a 'known issue' describing that if the right var is not provided, keepalived will default to that CIDR?08:38
ioniabout ceilomenter and ssl(rabbitmq) is there a way to resolve this problem? https://paste.xinu.at/32K2N/08:38
ioni:SSL routines:SSL3_WRITE_PENDING:bad write retry08:38
ioniapart from disable ssl communication with rabbitmq08:39
evrardjpodyssey4me: we include in the docs that it should be defined08:39
evrardjpmust be08:39
odyssey4meevrardjp yeah, I just figured that maybe we can highlight it in the release notes that if you get that CIDR then it shows that you haven't set that var08:39
evrardjpI'd be enclined to rewrite all these vars into group vars, and avoid the vars_files08:40
evrardjpso release note as known issue for newton and fix it in ocata?08:41
odyssey4meandymccr backport review please: https://review.openstack.org/38527708:41
odyssey4meevrardjp known issue for newton, yes - and fix in ocata if there's time08:42
*** williamcaban has joined #openstack-ansible08:42
odyssey4meie add the release note into that patch, we'll backport it... then go from there08:42
evrardjpthanks captain obvious !08:42
evrardjp:p08:42
odyssey4me:p08:42
odyssey4meclearly someone's feeling better08:43
evrardjpwell I can almost open both eyes at the same time for 5 minutes now.08:44
*** williamcaban has quit IRC08:46
odyssey4meandymccr evrardjp there are some backports here which need review please https://review.openstack.org/#/q/branch:stable/newton+project:%255Eopenstack/openstack-ansible.*+status:open08:47
odyssey4meevrardjp if I have a dict list, and for each item in the list I want to validate whether a file exists, then action a follow-on task based on the result - how do I do that?08:52
odyssey4methe problem here is that each tempest plugin we install might also need the packages in test-requirements.txt installed - but not every repository will necessarily have test-requirements.txt08:53
odyssey4meso we have to only try to install those if the file exists08:53
evrardjpdoes that work with content?08:53
evrardjpI'd say what you are probably looking for is (omit)08:53
odyssey4meso you're saying use a lookup with omit?08:54
evrardjpthat's what I've thought at first sight, but here I'd say more simply stat register with the dict items, and then another task to do you action based on the registered result08:55
*** oneswig has joined #openstack-ansible08:55
odyssey4mecan you capture the results of a stat for each file in its own register?08:56
odyssey4meit the stat task is acting on a list I mean08:56
*** evilt0ne has quit IRC08:58
odyssey4meevrardjp heh, apparently you can: http://www.it-recipes.com/articles/blog/20608:59
odyssey4meit seems that when you use register with a loop, it keeps the results in a 'results' list which you can iterate over08:59
evrardjpyes I know :)09:00
evrardjpI think you have examples already in osa09:00
odyssey4mewell, I didn't :p09:00
evrardjphttps://github.com/openstack/openstack-ansible/blob/stable/mitaka/scripts/upgrade-utilities/playbooks/rfc1034_1035-cleanup.yml#L92-L9709:01
evrardjpjust an example09:02
*** fops has joined #openstack-ansible09:05
*** fops has joined #openstack-ansible09:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Cater for situation where tempest plugins have no test-requirements.txt  https://review.openstack.org/38535509:06
odyssey4meevrardjp could you please take a look through https://review.openstack.org/385272 ?09:09
evrardjpyup just a sec09:10
openstackgerritMerged openstack/openstack-ansible-os_nova: Ensure that novnc/spice consoles use the public endpoint protocol  https://review.openstack.org/38501109:11
ioninice09:13
ioni^09:13
ionithat one i really need09:13
ionimy novnc didn't work and couldn't figure it out yet why09:13
openstackgerritJean-Philippe Evrard proposed openstack/openstack-ansible: Set default keepalived cidr if none is provided  https://review.openstack.org/38506909:14
*** chhavi has joined #openstack-ansible09:16
*** chris_hultin has quit IRC09:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Update all SHAs for Ocata 2016-10-12  https://review.openstack.org/38032509:27
*** chris_hultin|AWA has joined #openstack-ansible09:28
*** fops has quit IRC09:28
ionii noticed that now the number of endpoint after upgrade is greater09:28
*** chris_hultin|AWA is now known as chris_hultin09:28
ionicompared with a newly deployed N09:28
ionifor example, the public url is twice, once with http and once with https09:28
*** fops has joined #openstack-ansible09:28
ionion a newly deployed N the public url is only on https09:28
odyssey4meioni ah, so that's another bug where we need to clean things up09:29
ionii'm rocking !09:29
ionifinding bugs all over the place09:29
ionimaybe it's intentionally09:29
ionifor back compatibility?09:30
*** haad1 has quit IRC09:31
*** asettle has joined #openstack-ansible09:36
odyssey4meevrardjp do you mind if I tweak https://review.openstack.org/#/c/385069/4/releasenotes/notes/keepalived-default-cidr-workaround-8f2b5a0b074898e1.yaml slightly?09:39
odyssey4meioni it's not intentional09:39
ioniodyssey4me, ok09:39
ioniodyssey4me, can i pm you a bit?09:39
odyssey4meioni if need be - discussions are generally more beneficial for everyone in the channel09:40
*** haad1 has joined #openstack-ansible09:40
ionii want to resolve that public ssl thingy and somebody mentioned that it is risky because endpoints are changing(which is ofcourse logically)09:40
ionii used as haproxy public  an IP09:40
ioninow any ssl i'm going to configure, it's not going to be valid09:41
ionidid somebody tried to change the endpoints using ansible?09:41
ionishould i try to do it manually from db and then re-run the playbooks?09:41
*** pjm6_ is now known as pjm609:46
pjm6good morning all09:46
*** pjm6 has quit IRC09:47
openstackgerritMerged openstack/openstack-ansible: Fix value for openstack_host_manage_hosts_file  https://review.openstack.org/38527709:48
*** palendae has quit IRC09:49
*** oneswig has quit IRC09:51
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Update role SHAs and package pins for 14.0.0 2016-10-12  https://review.openstack.org/38537009:52
*** palendae has joined #openstack-ansible09:52
*** palendae is now known as Guest9292709:53
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Update role SHAs for 14.0.0 2016-10-12  https://review.openstack.org/38537009:53
*** pjm6 has joined #openstack-ansible09:55
*** chhavi has quit IRC09:57
ioniodyssey4me, done. https://bugs.launchpad.net/openstack-ansible/+bug/163264509:58
openstackLaunchpad bug 1632645 in openstack-ansible "During upgrare M->N old endpoints are not cleaned" [Undecided,New]09:58
odyssey4methanks ioni09:58
openstackgerritMerged openstack/openstack-ansible-os_nova: Fix ironic compute_driver  https://review.openstack.org/38530810:00
*** admin0 has quit IRC10:02
*** chhavi has joined #openstack-ansible10:05
openstackgerritMerged openstack/openstack-ansible-tests: Fix glance_api_servers variable  https://review.openstack.org/38531210:06
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Set default keepalived cidr if none is provided  https://review.openstack.org/38506910:10
*** asettle has quit IRC10:17
*** haad1 has quit IRC10:18
*** Mudpuppy has joined #openstack-ansible10:22
*** haad1 has joined #openstack-ansible10:23
openstackgerritPraveen N proposed openstack/openstack-ansible: [docs] Add network config example for test and prod  https://review.openstack.org/38527210:23
*** Praveen has joined #openstack-ansible10:28
*** asettle has joined #openstack-ansible10:29
PraveenHi10:31
PraveenHello Jesse, please provide suggestions for bug 163132110:33
openstackbug 1631321 in openstack-ansible "Docs: Test deployment shouldn't talk about bonds" [High,In progress] https://launchpad.net/bugs/1631321 - Assigned to Praveen N (praveenn)10:33
openstackgerritRavi Gummadi proposed openstack/openstack-ansible-repo_build: Add cython to repo_pip_packages  https://review.openstack.org/38538810:34
*** janki has quit IRC10:36
*** Praveen__ has joined #openstack-ansible10:39
*** Praveen__ has quit IRC10:40
*** Praveen__ has joined #openstack-ansible10:42
*** Mudpuppy has quit IRC10:45
*** Praveen__ has quit IRC10:46
odyssey4meHi Praveen - thanks for joining us here! I have no specific feedback at the moment - perhaps evrardjp will. It's ideal if you're able to be in the channel to field any questions or discuss options available.10:52
Praveensure, thanks a lot10:53
PraveenI've modified openstack_interface.cfg.test.example, please review & let me know for any changes10:54
*** chhavi has quit IRC10:57
*** ssbarnea has quit IRC10:57
odyssey4mePraveen yeah, it's not my area of expertise - I'm hoping that evrardjp, cloudnull or jamesdenton will pick up the next round of reviews some time soon10:58
Praveenok :)10:59
odyssey4meandymccr heh, cute - check the way that the rally results show up in the gate: http://logs.openstack.org/60/385360/1/check/gate-rally-dsvm-neutron-rally/0a0614d/11:05
odyssey4meI've seen something similar for devstack11:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Cater for situation where tempest plugins have no test-requirements.txt  https://review.openstack.org/38535511:12
*** spotz is now known as spotz_zzz11:12
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Install test requirements for plugins  https://review.openstack.org/38494211:15
andymccrodyssey4me: fancy fancy.11:16
odyssey4meandymccr yeah, I like it :)11:17
odyssey4mewe should do that11:17
odyssey4mewe'll need a different index for the role tests and for the integrated build though11:17
odyssey4mebut that's certainly helpful to help orientate contributors11:17
*** RaviGummadi has joined #openstack-ansible11:18
odyssey4melemme work up a bug quickly11:18
*** haad1 has quit IRC11:21
odyssey4medone https://bugs.launchpad.net/openstack-ansible/+bug/163267011:21
openstackLaunchpad bug 1632670 in openstack-ansible "Add html index to role & integrated repo results" [Wishlist,New]11:21
odyssey4meoh, and just for fun - it seems like we're not producing any logs for the containers11:23
*** evilt0ne has joined #openstack-ansible11:27
*** evilt0ne has quit IRC11:27
*** evilt0ne has joined #openstack-ansible11:27
*** askb has quit IRC11:30
openstackgerritDeepak proposed openstack/openstack-ansible: [Docs] Fix the listing of ops-remove-computehost.rst  https://review.openstack.org/38540611:36
drifterzawow11:40
drifterzaalmost the whole day on 1 diagram11:40
drifterzakill me11:41
ionievrardjp, i'm starting to like haproxy ssl now :D11:42
ioniworks like a charm :)11:42
openstackgerritAndy McCrae proposed openstack/openstack-ansible-os_ironic: [WIP] Enable virtualBMC testing  https://review.openstack.org/38104911:43
odyssey4medrifterza diagrams are hard, but very valuable when they're done right11:43
logan-hey evrardjp, you might be interested in this https://github.com/Logan2211/ansible-unbound/commit/a1f526df780d81685d635cabb3ab6fff2d81d804 -- i updated the tox config in my roles to run linter using latest stable ansible, and multiple envs for functional tests using different ansible versions.11:44
logan-I noticed the latest ansible-lint gives this error against all of our "Check init system" tasks https://travis-ci.org/Logan2211/ansible-unbound/jobs/166919724#L27311:44
logan-they will probably all need a "changed_when: false" at some point11:45
*** retreved has joined #openstack-ansible11:45
odyssey4melogan- oh neat11:45
odyssey4meI think when we release newton we should bump our ansible & ansible-lint requirements, then deal with the consequences11:46
*** thorst has joined #openstack-ansible11:47
*** haad1 has joined #openstack-ansible11:47
openstackgerritAndy McCrae proposed openstack/openstack-ansible-os_ironic: [WIP] Enable virtualBMC testing  https://review.openstack.org/38104911:48
odyssey4melogan- any chance that you could tag a release for https://github.com/logan2211/ansible-bird ?11:49
logan-err yes I must have missed that one11:49
logan-done11:50
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Update role SHAs for 14.0.0 2016-10-12  https://review.openstack.org/38537011:51
odyssey4methanks logan- :) it looks like your fixes are all good11:51
odyssey4memaybe I should mark the SHA bump as 'closes-bug' ?11:52
*** RaviGummadi has quit IRC11:53
*** johnmilton has joined #openstack-ansible11:53
*** johnmilton has joined #openstack-ansible11:54
odyssey4meyeah, lemme do that11:54
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Update role SHAs for 14.0.0 2016-10-12  https://review.openstack.org/38537011:54
openstackgerritAndy McCrae proposed openstack/openstack-ansible-plugins: Add get_networks command to neutron library  https://review.openstack.org/38542311:56
*** aetaric has quit IRC11:57
openstackgerritDeepak proposed openstack/openstack-ansible: [Docs] Removed extra grave accent(`)  https://review.openstack.org/38542511:58
*** aetaric has joined #openstack-ansible11:58
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Update role SHAs for 14.0.0 2016-10-12  https://review.openstack.org/38537012:00
openstackgerritAndy McCrae proposed openstack/openstack-ansible-plugins: Add get_networks command to neutron library  https://review.openstack.org/38542312:04
*** chris_hultin has quit IRC12:04
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [Docs] Removed extra grave accent(`)  https://review.openstack.org/38542512:05
*** chris_hultin|AWA has joined #openstack-ansible12:08
*** chris_hultin|AWA is now known as chris_hultin12:09
openstackgerritAndy McCrae proposed openstack/openstack-ansible-os_ironic: [WIP] Enable virtualBMC testing  https://review.openstack.org/38104912:13
*** chhavi has joined #openstack-ansible12:14
openstackgerritMerged openstack/openstack-ansible: [Docs] Removed extra grave accent(`)  https://review.openstack.org/38542512:14
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [Docs] Removed extra grave accent(`)  https://review.openstack.org/38543412:14
openstackgerritAndy McCrae proposed openstack/openstack-ansible-plugins: Add get_networks command to neutron library  https://review.openstack.org/38542312:16
odyssey4meandymccr evrardjp I'd appreciate a review of https://review.openstack.org/385355 - it fixes a bug which is blocking some role patches for master.12:17
openstackgerritAndy McCrae proposed openstack/openstack-ansible-os_ironic: [WIP] Enable virtualBMC testing  https://review.openstack.org/38104912:19
openstackgerritMerged openstack/openstack-ansible: [Docs] Removed extra grave accent(`)  https://review.openstack.org/38543412:21
mhaydenmornin'12:21
*** sdake has joined #openstack-ansible12:21
*** sdake has quit IRC12:22
*** sdake has joined #openstack-ansible12:23
*** woodard has joined #openstack-ansible12:24
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Cater for situation where tempest plugins have no test-requirements.txt  https://review.openstack.org/38535512:25
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Install test requirements for plugins  https://review.openstack.org/38494212:26
odyssey4meo/ mhayden how goes it?12:27
openstackgerritDonovan Francesco proposed openstack/openstack-ansible-os_monasca: Refactoring Monasca role to align more with OSA.  https://review.openstack.org/37565512:30
*** williamcaban has joined #openstack-ansible12:30
openstackgerritDonovan Francesco proposed openstack/openstack-ansible-os_monasca: Refactoring Monasca role to align more with OSA.  https://review.openstack.org/37565512:32
*** dragonmaster has quit IRC12:40
*** jheroux has joined #openstack-ansible12:46
*** chhavi has quit IRC12:47
openstackgerritDonovan Francesco proposed openstack/openstack-ansible-os_monasca: Refactoring Monasca role to align more with OSA.  https://review.openstack.org/37565512:48
*** chhavi has joined #openstack-ansible12:59
*** admin0 has joined #openstack-ansible12:59
ionican somebody see if they can replicate it? https://bugs.launchpad.net/openstack-ansible/+bug/163270213:04
openstackLaunchpad bug 1632702 in openstack-ansible "During upgrare M->N the public endpoint for keystone doesn't have https" [Undecided,New]13:04
*** klamath has joined #openstack-ansible13:05
*** klamath has quit IRC13:05
*** klamath has joined #openstack-ansible13:06
*** cathrichardson has joined #openstack-ansible13:07
*** cathrichardson has quit IRC13:07
*** cathrichardson has joined #openstack-ansible13:07
openstackgerritPraveen N proposed openstack/openstack-ansible: [docs] Add network config example for test and prod  https://review.openstack.org/38527213:07
*** haad1 has quit IRC13:08
*** haad1 has joined #openstack-ansible13:09
*** gouthamr has joined #openstack-ansible13:09
openstackgerritDonovan Francesco proposed openstack/openstack-ansible-os_monasca: Refactoring Monasca role to align more with OSA.  https://review.openstack.org/37565513:15
*** vishwanathj has joined #openstack-ansible13:15
*** jamesdenton has joined #openstack-ansible13:19
*** admin0 has quit IRC13:25
drifterzaodyssey4me, you around dude ?13:26
odyssey4medrifterza yep13:26
drifterzaneed advise on a role13:27
*** woodard has quit IRC13:27
odyssey4mesure13:27
drifterzamonasca is increasingly tricky to get pinned up, would you agree that having another role called monasca_backend for things like (zookeeper,kafka,storm,influxdb) and then have the os_monasca role only contain the monasca relevent things ?13:28
openstackgerritAndy McCrae proposed openstack/openstack-ansible-plugins: Add get_networks command to neutron library  https://review.openstack.org/38542313:28
drifterzaor should I try and pin everything in one role13:28
*** haad1 has quit IRC13:29
*** phschwartz has quit IRC13:30
openstackgerritMerged openstack/openstack-ansible-os_tempest: Install test requirements for plugins  https://review.openstack.org/38494213:32
odyssey4medrifterza it'd be better to have a role for each of the dependent services (zookeeper, etc), then have *only* monasca deployed in the os_monasca role13:33
odyssey4meideally use existing galaxy roles for zookeeper and such13:33
odyssey4meif you could figure out each of those dep roles, then you can do a patch to extract them and replace them with the dep role from galaxy - one per patch13:35
odyssey4methat'll make the work much easier to test, review and follow13:35
*** phschwartz has joined #openstack-ansible13:35
drifterzahmmm13:36
drifterzaI am close to having everything in the one role13:37
drifterzajust wondering how sustainable it would be13:37
odyssey4medrifterza if you really, must then go ahead - but ultimately it's more ideal if we can take advantage of the broader community for those dep services13:38
odyssey4methere are people who know much better than we how to install and configure them, and then those communities also keep them maintained as new versions come out13:38
odyssey4mewe ideally just want to maintain the stuff in our domain of expertise - ie OpenStack13:39
odyssey4methis cuts the workload for us and makes it much easier to sustain13:39
drifterzaokay13:39
openstackgerritDonovan Francesco proposed openstack/openstack-ansible-os_monasca: Refactoring Monasca role to align more with OSA.  https://review.openstack.org/37565513:39
drifterzaill work out a way to use galaxy in this PR13:39
*** dragonmaster has joined #openstack-ansible13:40
odyssey4meif you add the specific role into ansible-role-requirements.yml, then it'll get downloaded13:40
odyssey4methen all you need is to add it as a dependency in the meta/main.yml file and to set the right vars which the role wants13:41
odyssey4metry to find roles that are clearly well tested using travis, etc - and ideally already support ubuntu & centos13:41
*** Oku_OS is now known as Oku_OS-away13:41
*** Oku_OS-away is now known as Oku_OS13:42
drifterzathanks for the advise dude13:43
drifterzathink thats a better way to handle this beast13:43
*** batkins61 has joined #openstack-ansible13:53
*** dragonmaster_ has joined #openstack-ansible13:54
batkins61I'm still getting the following frequently, although 'glaera_root_password' is clearly defined in user_secrets.yml13:54
batkins61fatal: [stlrx2540m2-18_galera_container-1669b62a] => One or more undefined variables: 'galera_root_password' is undefined13:54
batkins61Any ideas on what's causing that?13:54
*** pcaruana has joined #openstack-ansible13:54
*** dragonmaster has quit IRC13:54
*** dragonmaster_ is now known as dragonmaster13:54
*** Praveen has quit IRC13:58
*** rmelero has joined #openstack-ansible13:58
openstackgerritMerged openstack/openstack-ansible: [DOCS] Update manual upgrade guide  https://review.openstack.org/38515013:59
*** phalmos_ has quit IRC14:00
*** agrebennikov has joined #openstack-ansible14:00
openstackgerritDonovan Francesco proposed openstack/openstack-ansible-os_monasca: Refactoring Monasca role to align more with OSA.  https://review.openstack.org/37565514:01
*** spotz_zzz is now known as spotz14:03
odyssey4mebatkins61 I would guess that you're not using the 'openstack-ansible' command, but are instead using 'ansible-playbook'14:04
batkins61Hmm, let me confirm that.14:05
*** rmelero has quit IRC14:06
openstackgerritMerged openstack/openstack-ansible-ceph_client: Add newline to ceph.conf created from ceph_conf_file  https://review.openstack.org/38511814:07
*** woodard has joined #openstack-ansible14:08
*** woodard has quit IRC14:08
batkins61Yep, looks like that was it according to command history.  I'll try again with openstack-ansible.14:08
*** woodard has joined #openstack-ansible14:09
*** haad1 has joined #openstack-ansible14:09
openstackgerritMerged openstack/openstack-ansible-os_tempest: Cater for situation where tempest plugins have no test-requirements.txt  https://review.openstack.org/38535514:10
openstackgerritMerged openstack/openstack-ansible-repo_build: Add cython to repo_pip_packages  https://review.openstack.org/38538814:10
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [WIP] Provide example configurations  https://review.openstack.org/38549614:15
*** chris_hultin has quit IRC14:15
odyssey4meautomagically evrardjp andymccr Can we get some reviews for https://review.openstack.org/385370 please14:16
*** michaelgugino has joined #openstack-ansible14:17
*** pester has quit IRC14:17
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build: Add cython to repo_pip_packages  https://review.openstack.org/38549814:18
drifterzaodyssey4me, could you point me to why this is happening ?14:18
drifterzahttp://logs.openstack.org/55/375655/72/check/gate-openstack-ansible-os_monasca-ansible-func-ubuntu-trusty/c0e4351/console.html#_2016-10-12_14_15_24_43955114:18
*** chris_hultin|AWA has joined #openstack-ansible14:19
*** chris_hultin|AWA is now known as chris_hultin14:19
odyssey4medrifterza looking14:19
*** Mudpuppy has joined #openstack-ansible14:19
drifterzaThanks14:19
drifterzaI'm trying to use the dict group mapping lookup like in nova, but cant see why this is happening14:19
spotzodyssey4me: question on that review, we're pulling from private repos vs openstack ones?14:20
openstackgerritMajor Hayden proposed openstack/openstack-ansible-security: Security: Add tasks for RHEL-07-010010  https://review.openstack.org/38394314:21
openstackgerritMajor Hayden proposed openstack/openstack-ansible-security: Security: Add tasks for RHEL-07-010010  https://review.openstack.org/38394314:23
*** Mudpuppy has quit IRC14:23
*** Mudpuppy has joined #openstack-ansible14:24
odyssey4medrifterza yeah, I've been getting the same weird error in https://review.openstack.org/378930 and couldn't figure it out14:25
odyssey4meI'll comment on a few things that may help though14:25
drifterzaokay so its not something i did wrong, cos I took the example from the nova plays and to me it should work.14:26
*** pcaruana has quit IRC14:27
odyssey4medrifterza added some comments - those adjustments *may* help14:28
drifterzacool testing14:28
odyssey4mespotz huh?14:29
spotzodyssey4me: ttps://github.com/willshersystems/ansible-sshd andhttps://github.com/logan221114:29
*** fxpester has joined #openstack-ansible14:30
odyssey4mespotz yeah, we use github as a source for repositories which are outside of OpenStack14:30
openstackgerritDonovan Francesco proposed openstack/openstack-ansible-os_monasca: Refactoring Monasca role to align more with OSA.  https://review.openstack.org/37565514:31
andymccrdrifterza: it looks like its failing because the group doesnt exist14:31
spotzodyssey4me: Ok wasn't sure if we were trying to call all the parts from openstack contralled repos14:31
andymccrsome of those worked14:31
drifterzaandymccr, ?14:31
andymccrdrifterza: check the line above the one you linked - its successful, basically the "zookeeper" one is failing because the group is called "monasca_zookeeper"14:32
andymccrso when it evaluates groups[item.value.group] that doesnt exist so it bombs.14:32
drifterzaah14:32
andymccrall your groups are called "monasca_x" at least in the inventory for testing14:33
drifterzalets try that14:33
*** rmelero has joined #openstack-ansible14:33
andymccrodyssey4me:  same for you and glance14:33
andymccradd glance_api and glance_registry to the inventory for testing14:33
andymccryou only have the "glance_all" group14:33
odyssey4meandymccr ah14:33
openstackgerritDonovan Francesco proposed openstack/openstack-ansible-os_monasca: Refactoring Monasca role to align more with OSA.  https://review.openstack.org/37565514:34
andymccrso its failing14:34
odyssey4me<---- dumbass14:34
drifterzasame <---->14:34
odyssey4meandymccr could you please comment in the review so that I can circle back to it at some point14:34
odyssey4methanks for looking into it :)14:34
*** BjoernT has joined #openstack-ansible14:34
andymccrnp! all good and fixed up14:35
drifterzalets see if it goes now14:40
*** chhavi has quit IRC14:40
*** hj-hpe has joined #openstack-ansible14:40
openstackgerritAndy McCrae proposed openstack/openstack-ansible-os_glance: Use dictionary for service group mappings  https://review.openstack.org/37893014:42
andymccrany reviews on https://review.openstack.org/#/c/385423/ would be appreciated :D14:44
*** weezS has joined #openstack-ansible14:49
*** rmelero has quit IRC14:52
*** dragonmaster has quit IRC14:54
drifterzathanks andymccr14:55
drifterzathat fixed that issue14:55
andymccrnp!14:56
openstackgerritCharles Farquhar proposed openstack/openstack-ansible-os_ironic: Use correct keystone_authtoken config keys  https://review.openstack.org/38550814:56
drifterzaandymccr you have any experience with using program_override in the upstart/systemd files14:57
drifterzatrying to use the dict but for certain groups I need to run a binary thats not the same as the name of the group14:57
drifterzamonasca-api = binary = gunicorn --options14:58
odyssey4meandymccr I've added stevelle and hughsaunders as they're usually best (along with jmccrory and automagically) at doing python reviews14:58
odyssey4mecloudnull is good too - will add him now14:58
odyssey4medrifterza we have no precedence for that anywhere, but we discussed an option th eother day14:59
odyssey4mebasically have another item in the dict - maybe 'service_bin'15:00
drifterzayup15:00
odyssey4methen adjust the templates to use it if it's defined15:00
*** phalmos has joined #openstack-ansible15:00
openstackgerritMerged openstack/openstack-ansible: Set default keepalived cidr if none is provided  https://review.openstack.org/38506915:01
odyssey4meso instead of 'program_override' in the template, use the dict path15:01
drifterzahmm15:01
odyssey4mewe can apply that updated pattern to all the other roles if that works out nicely15:01
openstackgerritMajor Hayden proposed openstack/openstack-ansible-tests: Move ANSIBLE_CLI_PARAMETERS after playbook path  https://review.openstack.org/38551115:01
*** rgogunskiy has quit IRC15:01
drifterzaitem.value.service_bin15:01
drifterzafor example15:01
*** rmelero has joined #openstack-ansible15:01
drifterzain the template now15:02
odyssey4mealso, I would guess that program_config_options in the template should be replaced by something in the dict like service_init_options15:02
odyssey4meyes, something like that15:02
drifterzathis is just for upstart/systemd15:03
drifterzanot specific app overrides.15:03
odyssey4mewell, maybe service_init_bin then?15:03
drifterzayeah lets use that15:03
odyssey4meif upstart and systemd need different values, then have it be something like service_upstart_bin / service_upstart_options15:04
drifterzaill test it out15:05
drifterzathanks15:05
*** sdake has quit IRC15:05
*** sdake has joined #openstack-ansible15:06
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Set default keepalived cidr if none is provided  https://review.openstack.org/38551415:06
*** batkins61 has quit IRC15:06
*** batkins61 has joined #openstack-ansible15:07
*** cmart has joined #openstack-ansible15:14
andymccrdrifterza: i haven't but I think you just set the program_override var to be the exec name that you want. I'd imagine you would want to change it to be "item.value.program_override" and then specify it in the monasca_services dict15:18
andymccri think it has to be the full path (which would be the only caveat)15:19
*** TxGirlGeek has joined #openstack-ansible15:19
*** fops has quit IRC15:19
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [WIP][docs] Provide example configurations  https://review.openstack.org/38549615:19
drifterzayeah thats okay cause the gunicorn is in the venv path15:19
drifterzabut Ill give it a test15:19
*** jimbaker has joined #openstack-ansible15:20
andymccrthen probably just do like "{{monasca_bin}}/gunicorn" as your program_override15:20
openstackgerritCharles Farquhar proposed openstack/openstack-ansible-os_ironic: Use correct keystone_authtoken config keys  https://review.openstack.org/38552315:20
drifterzayup im thinking of adding something like this to the dict: service_init_bin: "{{ monasca_bin}}/gunicorn" , service_init_options: "--listen etc.. etc.."15:21
drifterzathen I have to mod the upstart/systemd templates to look for these15:21
drifterzaor is this over kill ?15:22
andymccrthat sounds ok to me - i mean the program_override bit is already there so its just about changing it to pull from the dict - which makes sense to me, i think it's there as a relic of how we used to set these up15:23
*** timrc_ is now known as timrc15:24
drifterzais it worth changing it though to have bin and options to bin15:24
*** jamesdenton has quit IRC15:24
evrardjphey I'm back15:26
andymccrdrifterza: i think-so. the way we're doing it with dicts now (here and in a few others) suggests to me that all the values should be in that dict15:26
andymccrhow we used to do it is individually call the "init_common.yml" as many times as there are services with those variables set as "vars:" on the include15:26
drifterzaso +1 for the service_init_bin/service_init_options ?15:27
drifterzaor just use program_override15:27
evrardjpodyssey4me: you know I don't mind you tweaking the bits15:27
evrardjpas long as it stays honorable!15:27
andymccrdrifterza: would init options be like the "program_config_options" ?15:28
drifterzayes15:28
andymccrand then system_init_bin like program_override15:28
andymccrsure yeah that makes sense to me15:28
drifterzalike flags to the bin15:28
evrardjpI still hate this patch, but that's another story15:28
evrardjpOk I can get back to my todo list...15:29
andymccrdrifterza: yeah all good i think, the only reason program_override exists is because in most cases the service name is actually the binary name too - so its simple where that pattern matches you dont need to specify it15:30
evrardjplogan-: I think I'll shamelessly adapt my CI to be closer to yours15:30
drifterzaandymccr, 100%15:32
drifterzaits sometimes useful to add options even though the bin is the same name15:32
evrardjpdrifterza: I still have tests with trove on my todolist, is it still current or it's too late now?15:33
*** kylek3h_ is now known as kylek3h15:33
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [docs] Provide example configurations  https://review.openstack.org/38549615:33
drifterzathere is some work for trove still, getting trove to upload images to glance, and use tempest to test functionality15:33
drifterzaso functional tests needs to be worked on evrardjp15:34
odyssey4meevrardjp andymccr asettle I'd love some reviews of this docs change which provides the example configs we've all been waiting for. :p https://review.openstack.org/38549615:34
odyssey4mespotz ^15:35
asettleodyssey4me: if you insist15:35
spotzodyssey4me: Got it15:35
drifterzaevrardjp, I guess the answer you looking for is, yes it is still current15:35
drifterza:D15:36
evrardjpok I will have a look at everyone's demands15:36
logan-evrardjp: sounds good :) personally I think it will make maintaining my roles easier by running linters and tests using latest stable ansible to fix things upstream far ahead of OSA's ansible bumps.15:36
evrardjpI'll give answers demanding on my supply15:36
evrardjplogan-: I'm myself using ansible 1.9 and ansible 2. as requirements for my two types of tests15:37
evrardjpso I always upgrade to latest ansible15:37
evrardjpit's generally fine15:37
evrardjpbut then I don't do periodic jobs, and that's terrible15:37
*** Guest92927 is now known as palendae15:38
odyssey4meevrardjp in https://review.openstack.org/385496 I took a guess at how to make haproxy/keepalived work across multiple nodes... please correct my horrible guess work :)15:39
odyssey4methe idea is to provide a minimal and simple set of examples to help people understand how it works15:40
*** fguillot has quit IRC15:40
evrardjpodyssey4me: is that the proper link?15:42
*** rmelero has quit IRC15:42
*** admin0 has joined #openstack-ansible15:42
evrardjplol @1.2.3.4/3215:42
evrardjp:D15:42
odyssey4meevrardjp the one thing I'm not clear on is how keepalived actually sets the vip for the multinode environment15:43
*** rmelero has joined #openstack-ansible15:43
evrardjpwhat do you mean?15:44
evrardjpit's using vrrp15:44
asettleodyssey4me: all yours15:44
evrardjplet me review this :D15:44
odyssey4mewell, at some point it actually needs to have an IP - in the docs I set one, but in the config I don't know where to assign it so that it binds it15:44
spotzheheh I'll wait for asettle's changes to get in:)15:45
*** david-lyle has joined #openstack-ansible15:46
asettleTwo eyes are always better than one :D that'd be great if you can review after spotz :)15:46
evrardjpodyssey4me: I thought at some point to do a {{ lookup('dns',external_lb_vip_address) }}15:46
evrardjpbut it's not a terribly good idea15:46
odyssey4melemme change based on asettle's feedback quickly15:47
spotzasettle Yeah I figured I'd give him a chance to make yours just in case they end up changing things15:47
evrardjpthen you have to guess the cidr for the nic, which is fine based on facts, but it could be cleaner15:47
asettlespotz: excellente :) thanks odyssey4me15:48
*** rmelero has quit IRC15:48
asettleI'll be off shortly, I was in early as. But i'll review again after spotz :) (we'll go round and round and round )15:48
spotzheheh15:48
*** david-lyle has quit IRC15:52
*** asettle has quit IRC15:53
openstackgerritNolan Brubaker proposed openstack/openstack-ansible: Check for configured groups in the environment  https://review.openstack.org/38392015:53
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [docs] Provide example configurations  https://review.openstack.org/38549615:55
odyssey4me^ still working on it15:57
*** adrian_otto has joined #openstack-ansible15:57
drifterzaodyssey4me, so I update ansible-role-requirements, the -name: zookeeper , can the name be anything I want ?16:01
*** jamesdenton has joined #openstack-ansible16:01
evrardjpthat's the rolename16:01
odyssey4medrifterza the name should be the name used by the role in ansible galaxy16:01
evrardjpnot really16:02
drifterzaah so author.name16:02
*** cmart has quit IRC16:02
evrardjpname is the name on your host16:02
evrardjpit could be the same16:02
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [docs] Provide example configurations  https://review.openstack.org/38549616:02
evrardjpit's not forced to be the same16:02
drifterzaso at the moment I did, -name: grafana16:02
drifterzabut on galaxy its : tkuhlman.monasca-ui16:02
evrardjpif you just do - then the item will be the galaxy name16:02
evrardjpyou can use that in src16:03
evrardjpiirc16:03
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [docs] Provide example configurations  https://review.openstack.org/38549616:03
odyssey4mespotz ^ OK, ready to go16:03
evrardjpdrifterza: http://docs.ansible.com/ansible/galaxy.html#installing-multiple-roles-from-multiple-files16:03
spotzodyssey4me: Okie:)16:04
evrardjpsrc is the src, if you don't give full url, it's assumed to be a galaxy role and will look there16:04
evrardjpelse it will pack a gz from the url you give based on the scm16:04
*** thorst is now known as thorst_afk16:04
drifterzaaaah16:05
evrardjpand will copy it into your roles_folder/<name>16:05
drifterzaso if I put the github url16:05
drifterzain src16:05
drifterzaand use scm:git16:05
drifterzait wont use galaxy ?16:05
*** weezS_ has joined #openstack-ansible16:05
evrardjpit won't use galaxy website and api16:05
evrardjpit will still use ansible-galaxy cli :p16:05
*** david-lyle has joined #openstack-ansible16:06
*** weezS has quit IRC16:06
*** weezS_ is now known as weezS16:06
drifterzaso its just better to use -src16:06
drifterzaokay16:06
odyssey4memhayden is https://review.openstack.org/385511 what's screwing up the use of extra vars?16:08
odyssey4meI've been trying to figure that one out for two days16:08
openstackgerritPraveen N proposed openstack/openstack-ansible: [docs] Add network config example for test and prod  https://review.openstack.org/38527216:09
drifterzawould this work : http://paste.openstack.org/show/585492/16:11
drifterzafor my meta/main.yml16:11
*** chhavi has joined #openstack-ansible16:12
drifterzaand this for my ansible-role-requirements.yml : http://paste.openstack.org/show/585493/16:13
drifterzaevrardjp, ?16:13
evrardjpsorry16:14
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_aodh: Update paste, policy and rootwrap configurations 2016-10-12  https://review.openstack.org/38554216:14
evrardjpyup16:14
evrardjpit could work16:14
evrardjptheoretically16:15
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer: Update paste, policy and rootwrap configurations 2016-10-12  https://review.openstack.org/38554316:15
evrardjphowever in the gates we are almost never using that16:15
evrardjpbecause we now use git by default :p16:15
drifterzaso now in my defaults/main.yml for my role, I just need to cater for those galaxy roles required variable16:15
evrardjpand we bypass the ansible-galaxy cli completely16:15
drifterzarofl16:15
evrardjpbut theoretically it's the good usage16:15
spotzodyssey4me: one change16:15
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_heat: Update paste, policy and rootwrap configurations 2016-10-12  https://review.openstack.org/38554416:16
drifterzablah, so how would I use external roles to this monasca role16:16
evrardjpanyway now galaxy is open source16:16
*** rmelero has joined #openstack-ansible16:16
odyssey4meyes, we bypass ansible-galaxy's CLI because the API is flaky for gate usage16:16
evrardjpwe can have our own16:16
drifterzasigh16:16
logan-drifterza some revisions http://cdn.pasteraw.com/jdyen2bfxhsucdiy6o4vylidf9v2uos but also you don't have a git source listed for monasca-ui as you do the other ones16:16
drifterzalogan-, would that work in the gate ?16:16
evrardjpdrifterza: what I'd do if I were you, is to find the real url on every galaxy role16:16
logan-the role will be downloaded by osa bootstrap or ansible-galaxy using which ever name you place in ansible-role-requirements. then you can call the dependency using that name, not the full galaxy name16:16
evrardjpand put that into the requirements16:17
evrardjpand define the name you want, this way it's more explicit etc16:17
drifterzaah bonus16:17
drifterzathats what I wanted16:17
evrardjpyup that's the cleaner way to work in all worlds16:18
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_magnum: Update paste, policy and rootwrap configurations 2016-10-12  https://review.openstack.org/38554616:18
*** cmart has joined #openstack-ansible16:18
evrardjpit works with our clone process and it works with galaxy cli16:18
evrardjpbut now you understand the whole story :p16:18
drifterzalol16:19
drifterzathanks guys16:19
drifterzaill give it a shot16:19
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Update all SHAs for Ocata 2016-10-12  https://review.openstack.org/38032516:19
openstackgerritDonovan Francesco proposed openstack/openstack-ansible-os_monasca: Refactoring Monasca role to align more with OSA.  https://review.openstack.org/37565516:20
evrardjpthe con of doing this, is we introduce a non-fully scoped dependency, and we need the deployers to ansible-galaxy (or clone, or whatever) the roles before starting it, where if you put a full dependency in your meta, it can be auto-fetched16:20
evrardjpbut we do this way everywhere so it's fine :D16:21
odyssey4meevrardjp but only fetched if the role is actually in ansible galaxy16:21
*** hughmFLEXin has quit IRC16:21
drifterzaI like that we do openstack, and other dependancies we rely on tested galaxy roles16:21
evrardjpyes that's what I meant16:22
drifterzamakes my life easier16:22
evrardjpdrifterza: it makes everyone's life easier16:22
evrardjp:D16:22
drifterzacos monasca and its 100 dependancies for backend was making me bleak16:22
evrardjpby hoping overhead(dependency management) < overhead(whole role maintenance)16:23
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [docs] Provide example configurations  https://review.openstack.org/38549616:23
odyssey4mespotz ^ better?16:23
*** chris_hultin has quit IRC16:24
spotzodyssey4me: Works for me:)16:25
odyssey4meorsm, thanks spotz :)16:25
spotz:)16:25
openstackgerritDonovan Francesco proposed openstack/openstack-ansible-os_monasca: Refactoring Monasca role to align more with OSA.  https://review.openstack.org/37565516:26
*** fops has joined #openstack-ansible16:27
*** chris_hultin has joined #openstack-ansible16:27
odyssey4meevrardjp can you do a quick review of https://review.openstack.org/385511 please so that I can move on to better things :p16:28
evrardjpit's cool when I have reviews queues but I can't finish one of them because a more urgent review takes over :p16:29
evrardjpok doing it right now :p16:29
odyssey4mehaha - you're welcome to simply queue it... but it is a really simple one16:30
evrardjpdone16:31
evrardjpI'm not surprised by this bug16:31
odyssey4meit really makes me want to table flip16:31
odyssey4mebut I think I shall go and buy some scotch instead :)16:32
evrardjpdoes that makes me jade, a soothsayer or just predictable?16:32
evrardjpodyssey4me: well that's a proper way to fix the world16:32
*** drifterza has quit IRC16:35
ionii like the proposed documentation with example configurations in production16:37
mhaydenodyssey4me: yessir16:37
ioninow i need to see if i understood the initial documentation :)16:37
mhaydenodyssey4me: i didn't know you were fighting the problem, too16:38
odyssey4meioni glad you like it :) it'd be ideal if you could validate that it's right... it was all done in theory for me - I haven't tested it16:38
odyssey4memhayden I'm just glad that it's solved16:38
ioniodyssey4me, i for one did instead of vlan, vxlan, because i do not have access to the switch management16:38
ionithe network separation16:39
mhaydenodyssey4me: yeah, i was really banging my head against a desk in the security role16:39
odyssey4meioni well, it is entirely plausible that the network setup can be done without vlans even16:39
ioniodyssey4me, indeed16:39
ioniodyssey4me, but i did want to separate the networks16:39
odyssey4meioni perhaps you could push up another network configuration example for a test environment *without* VLAN's ?16:40
*** haad1 has quit IRC16:41
mhayden635555555555555555555555555555555555555516:42
odyssey4mecores - we could do with some eyes on https://review.openstack.org/#/q/starredby:jesse-pretorius+project:%255Eopenstack/openstack-ansible.*+status:open please16:42
mhaydenoops16:42
odyssey4memhayden cat walk on your keyboard?16:42
odyssey4meor was that you bashing your head again?16:43
mhaydenhah, laid something across the laptop16:43
ioniodyssey4me, i don't have without vlan or without vlanx16:46
ionii'll invent something if you want16:46
odyssey4megoing afk for a bit16:47
*** haad1 has joined #openstack-ansible16:50
*** retreved has quit IRC16:51
openstackgerritMerged openstack/openstack-ansible-tests: Move ANSIBLE_CLI_PARAMETERS after playbook path  https://review.openstack.org/38551116:54
*** appprod0 has joined #openstack-ansible16:54
openstackgerritMajor Hayden proposed openstack/openstack-ansible-security: Security: Add tasks for RHEL-07-010010  https://review.openstack.org/38394317:02
openstackgerritMajor Hayden proposed openstack/openstack-ansible-security: Security: Remove quotes from extra vars  https://review.openstack.org/38556317:02
mhaydenodyssey4me: now that the parameter move has merged, this *should* get the rhel7 security gate back in line -> https://review.openstack.org/#/c/385563/17:03
*** javeriak has joined #openstack-ansible17:06
*** thorst_afk is now known as thorst17:14
*** michaelgugino has quit IRC17:14
evrardjpI'm now headached, I'm off for today... Enough stacking for a day! :p17:14
evrardjpsee you tomorrow17:15
*** haad1 has quit IRC17:17
odyssey4memhayden does this mean it worked? http://logs.openstack.org/63/385563/1/check/gate-openstack-ansible-security-ansible-func_rhel7-centos-7-nv/dc050b9/console.html#_2016-10-12_17_07_14_41876217:21
*** NachoDuck has quit IRC17:22
openstackgerritMajor Hayden proposed openstack/openstack-ansible-security: Security: Add tasks for RHEL-07-010010  https://review.openstack.org/38394317:23
*** NachoDuck has joined #openstack-ansible17:24
*** Jack_Iv has quit IRC17:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift: Use centralised Ansible test scripts  https://review.openstack.org/38231417:26
*** Jack_Iv has joined #openstack-ansible17:27
mhaydenodyssey4me: yeah, it worked... this now passes, too https://review.openstack.org/38556317:28
*** michaelgugino has joined #openstack-ansible17:28
mhaydenis 'stdout_lines' getting deprecated?17:29
odyssey4memhayden not as far as I know17:30
odyssey4mewhy?17:30
mhaydenah, it's the 'when' getting parsed after the 'with_items'17:30
mhaydeni just need a |default([]) in there17:30
*** hughmFLEXin has joined #openstack-ansible17:32
odyssey4mestevelle I may be wrong, but it looks like the the api paste changes are causing a kerfuffel: https://review.openstack.org/38554317:32
*** chhavi has quit IRC17:32
openstackgerritLogan V proposed openstack/openstack-ansible: Configure Calico specific BIRD settings in OSA  https://review.openstack.org/38557317:33
stevelleodyssey4me: haven't noticed that paste change and I don't know that middlware off the top of my head17:33
openstackgerritMajor Hayden proposed openstack/openstack-ansible-security: Security: Add tasks for RHEL-07-010010  https://review.openstack.org/38394317:35
odyssey4mestevelle that's a change in master after the newton release - not urgent17:35
openstackgerritMerged openstack/openstack-ansible-plugins: Add get_networks command to neutron library  https://review.openstack.org/38542317:36
stevelleodyssey4me: as a note, we should get gnocchi bumped to 3.0 in stable/newton but I haven't had time to look to see if that was done yet17:36
stevelleI'll queue the ceilo change to look at it closer17:36
*** weezS has quit IRC17:37
*** retreved has joined #openstack-ansible17:41
*** retreved has quit IRC17:41
*** retreved has joined #openstack-ansible17:42
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [docs] Provide example configurations  https://review.openstack.org/38549617:45
odyssey4mestevelle yep, we're already using stable/3.0 for gnocchi17:46
odyssey4mestevelle FYI it appears that gnocchi doesn't use the global requirements management (ie no upper constraints) - this means that any of its requirements are uncapped for every tag we release17:46
odyssey4methis is a bit of an issue for repeatable deployments17:47
odyssey4meit'd be ideal if you could try and get gnocchi to apply the global requirements process17:47
*** evilt0ne has quit IRC17:49
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [docs] Provide example configurations  https://review.openstack.org/38549617:50
stevelleodyssey4me: I'm pretty sure there would be resistance to that.17:50
openstackgerritMerged openstack/openstack-ansible-repo_build: Add cython to repo_pip_packages  https://review.openstack.org/38549817:50
*** evilt0ne has joined #openstack-ansible17:50
*** evilt0ne has quit IRC17:50
*** evilt0ne has joined #openstack-ansible17:50
stevellethey want it to be more an independent library17:50
stevelleodyssey4me: your concern is upper constraints?17:51
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_keystone: Use centralised Ansible test scripts  https://review.openstack.org/38230617:53
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift: Use centralised Ansible test scripts  https://review.openstack.org/38231417:54
*** weezS has joined #openstack-ansible18:00
ioniodyssey4me, are you sure it's gnocchi 3.0?18:03
ionignocchi --version18:03
ionignocchi 2.6.018:03
odyssey4meioni odd, https://github.com/openstack/openstack-ansible/blob/stable/newton/playbooks/defaults/repo_packages/gnocchi.yml#L3618:07
ioniodyssey4me, and i did destroyed that container and the database earlier today18:08
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Update all SHAs for Ocata 2016-10-12  https://review.openstack.org/38032518:08
ionilets see in their git18:09
ionimaybe they didn't bumped the version18:09
*** evilt0ne has quit IRC18:13
*** asettle has joined #openstack-ansible18:14
*** seventy has joined #openstack-ansible18:14
*** evilt0ne has joined #openstack-ansible18:15
*** evilt0ne has quit IRC18:15
*** evilt0ne has joined #openstack-ansible18:15
*** hughmFLEXin has quit IRC18:16
ioniodyssey4me, well, i'm an idiot :D18:16
ionignocchiclient from pip is 2.6.018:16
ioni3.0.118:17
ioni /openstack/venvs/gnocchi-14.0.0/bin/gnocchi-metricd --version18:17
ioni3.0.118:17
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [DOCS] Update manual upgrade guide  https://review.openstack.org/38558218:17
odyssey4mehaha, you got it :)18:18
odyssey4meafk for a bit - need a break before I dig into some bugs18:18
ionialright18:19
*** asettle has quit IRC18:20
*** fguillot has joined #openstack-ansible18:20
*** aetaric has quit IRC18:23
openstackgerritMajor Hayden proposed openstack/openstack-ansible-security: Security: Add tasks for RHEL-07-010020  https://review.openstack.org/38558318:23
*** aetaric has joined #openstack-ansible18:23
openstackgerritMerged openstack/openstack-ansible-plugins: Remove xrange for run both Python 2 and Python 3  https://review.openstack.org/38528018:24
openstackgerritMerged openstack/openstack-ansible: Set default keepalived cidr if none is provided  https://review.openstack.org/38551418:26
openstackgerritMerged openstack/openstack-ansible: Update role SHAs for 14.0.0 2016-10-12  https://review.openstack.org/38537018:26
openstackgerritMerged openstack/openstack-ansible: Add documentation for known-issue with ceph client.  https://review.openstack.org/38510718:26
*** e-vad has left #openstack-ansible18:26
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Execute tempest using updated method  https://review.openstack.org/38221118:28
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Execute tempest using updated method  https://review.openstack.org/38221118:28
*** asettle has joined #openstack-ansible18:32
michaelguginosomeone please assign importance of https://bugs.launchpad.net/openstack-ansible/+bug/1632812 as low hanging fruit18:34
openstackLaunchpad bug 1632812 in openstack-ansible "Remove extra_volume_types from cinder.conf" [Undecided,New]18:34
odyssey4memichaelgugino can you add the output of what it is now and what it should look like, with a sample config to generate it?18:36
odyssey4methat'll make it really easy for someone to pick up18:36
michaelguginok18:36
*** retreved has quit IRC18:39
openstackgerritMerged openstack/openstack-ansible-security: Security: Remove quotes from extra vars  https://review.openstack.org/38556318:40
*** retreved has joined #openstack-ansible18:40
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift: Use centralised Ansible test scripts  https://review.openstack.org/38231418:40
openstackgerritMajor Hayden proposed openstack/openstack-ansible-security: Security: Add tasks for RHEL-07-010260  https://review.openstack.org/38558718:40
mhaydenodyssey4me: thanks for the nudge on those quotes ;)18:41
odyssey4memhayden np :)18:41
odyssey4meaint nobody got time fo' dat18:42
mhaydenhaha18:42
odyssey4meI'd appreciate a little review love for https://review.openstack.org/#/q/project:%255Eopenstack/openstack-ansible.*+status:open+topic:bp/central-test-repository18:42
michaelguginoodyssey4me: bug updated18:43
odyssey4megreat, thanks michaelgugino18:44
odyssey4meI pexct that someone will pick it up fairly quickly and that evrardjp will likely mark it low-hanging-fruit18:44
odyssey4metypically it needs to go through the triage meeting, but I'll leave it at his discretion18:45
odyssey4meI'm going to dig into https://bugs.launchpad.net/openstack-ansible/+bugs?search=Search&field.assignee=jesse-pretorius this evening to see what I can figure out.18:45
michaelguginoTypically I would just fix it myself, but it's simple enough for a new-comer, but more complex than a spelling error, and might actually entice someone.18:46
odyssey4mesure, appreciate that18:48
odyssey4meI think we're starting to get better at doing this sort of thing.18:48
odyssey4meit's easy to get side-tracked into fixing little niggles like this - but if you do a bug report instead then it can be picked up by someone else, or perhaps you later18:49
odyssey4methat keeps the focus on what you're doing instead of being side tracked18:49
*** asettle has quit IRC18:55
odyssey4mecloudnull FYI https://bugs.launchpad.net/openstack-ansible/+bug/163281818:56
openstackLaunchpad bug 1632818 in openstack-ansible "Newton: No container logs available in integrated gate jobs" [Undecided,New]18:56
*** chris_hultin has quit IRC18:58
*** chris_hultin has joined #openstack-ansible19:00
openstackgerritNish Patwa(nishpatwa_) proposed openstack/openstack-ansible-ops: Added cinder metric collection, telegraf conntrack plugin.  https://review.openstack.org/37913819:01
*** williamcaban has quit IRC19:01
*** spotz is now known as spotz_zzz19:05
*** jmckind has joined #openstack-ansible19:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Execute tempest using updated method  https://review.openstack.org/38221119:19
*** hughmFLEXin has joined #openstack-ansible19:22
*** cmart has quit IRC19:26
cloudnullodyssey4me: looking into that bug now19:27
*** hybridpollo has joined #openstack-ansible19:28
*** chris_hultin is now known as chris_hultin|AWA19:29
odyssey4mecloudnull mgariepy I suspect that https://github.com/openstack/openstack-ansible-repo_server/commit/95ac11e15cc4daf7cdc6e31e55e0fa3df2ac35ca is causing the master builds to fail. It's causing 503's for apt package requests.19:30
odyssey4meyou can see it consistently in the logs for https://review.openstack.org/38032519:31
odyssey4meunfortunately we have no container logs, so we can't see the cache log19:31
odyssey4meno issues for newton, which is why I suspect that patch19:34
cloudnullinteresting that would cause caching issues.19:35
cloudnullthat merged 16 days ago19:35
odyssey4meyeah, we've still got fixed role sha's in master19:35
cloudnulloh.19:35
cloudnullthat makes sense.19:36
odyssey4meI've been trying to move us forward for ages with all sorts of nonsense19:36
*** AnarchyAo has joined #openstack-ansible19:36
*** chris_hultin|AWA is now known as chris_hultin19:36
cloudnullim looking into the log bits now19:38
cloudnullmore in a min19:38
cloudnullhopfully19:38
cloudnulli also have a node spinning up with your PR to update19:40
cloudnullso i'll dig into that too19:40
*** rmelero has quit IRC19:40
odyssey4mejust a thought too: https://bugs.launchpad.net/openstack-ansible/+bug/163282619:40
openstackLaunchpad bug 1632826 in openstack-ansible "apt/yum cache causes problems when rebuilding" [Undecided,New]19:40
cloudnullwhat else do we need to tackle for the newton release ?19:40
*** rmelero has joined #openstack-ansible19:41
odyssey4methese are the open reviews at this point: https://review.openstack.org/#/q/starredby:jesse-pretorius+project:%255Eopenstack/openstack-ansible.*+status:open19:41
cloudnulldo we have open issues ?19:41
odyssey4meand the bug hit list is in the channel topc19:41
cloudnullok19:41
cloudnullthe bug list has quite a few untriaged bugs.19:43
cloudnullis there a hitlist of known issues that we need to resolve?19:43
odyssey4mecloudnull yeah, as they come in qe're itemising the greenfield related ones and dealing with them19:45
odyssey4methere's no shortlist really19:45
odyssey4meso ignore anything upgrade related, unless you think it has something to do with a fundamental issue under it19:46
odyssey4mefeel free to pick up on trying to triage these if you like? https://bugs.launchpad.net/openstack-ansible/+bugs?search=Search&field.assignee=jesse-pretorius19:46
odyssey4methe first one was assigned to you19:46
cloudnullok i'll try and work through some of those too19:48
*** Jack_Iv has quit IRC19:49
*** johnmilton has quit IRC19:50
*** javeriak has quit IRC19:52
*** elarson has quit IRC19:53
*** spotz_zzz is now known as spotz19:54
*** appprod0 has quit IRC20:00
*** gouthamr has quit IRC20:14
*** TxGirlGeek has quit IRC20:16
*** Mudpuppy has quit IRC20:16
*** TxGirlGeek has joined #openstack-ansible20:16
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Execute tempest using updated method  https://review.openstack.org/38221120:20
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Execute tempest using updated method  https://review.openstack.org/38221120:21
*** askb has joined #openstack-ansible20:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Execute tempest using updated method  https://review.openstack.org/38221120:22
*** jmckind has quit IRC20:32
*** cmart has joined #openstack-ansible20:32
*** javeriak has joined #openstack-ansible20:32
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Update all SHAs for Ocata 2016-10-12  https://review.openstack.org/38032520:32
*** haad1 has joined #openstack-ansible20:33
*** agireud has joined #openstack-ansible20:35
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Update all SHAs for Ocata 2016-10-12  https://review.openstack.org/38032520:36
*** appprod0 has joined #openstack-ansible20:36
*** cmart has quit IRC20:37
odyssey4mealright, that should work as it leaves a SHA in place for the repo_server role, but at least gets us to master for everything else20:37
*** gouthamr has joined #openstack-ansible20:39
*** fxpester has quit IRC20:40
*** fxpester has joined #openstack-ansible20:40
mrdaMorning OSA20:41
evrardjpmorning to you20:45
evrardjpodyssey4me: what's the issue with the repo_server?20:45
odyssey4meevrardjp https://bugs.launchpad.net/openstack-ansible/+bug/163284720:46
openstackLaunchpad bug 1632847 in openstack-ansible "repo_server package cache causing failures" [Undecided,New]20:46
evrardjpwell that sounds bad20:47
odyssey4meyeah20:47
*** pester has joined #openstack-ansible20:47
evrardjpand it's not osic20:49
cloudnullwhat?20:49
*** seventy has quit IRC20:49
odyssey4meevrardjp yes, I've confirmed by downloading the package myself and confirming with infra that the issue is not provider-side20:50
irtermitelow blow evrardjp... low blow   :(    ;)20:50
evrardjpok20:50
*** fxpester has quit IRC20:50
odyssey4meapparently osic has a fresh sore :p20:50
evrardjpnot a low blow, just checking it's not the source :)20:50
cloudnullhahaha.20:50
* irtermite sees that cloudnull has an "osic" pouncer too20:50
cloudnullyea. blog post coming soon.20:50
irtermitehah20:51
evrardjpyes I am afraid of saying something with osic in it now20:51
irtermiteodyssey4me: it's turning green now... it'll be ready for picture day soon20:51
irtermiteevrardjp: pings me too20:51
evrardjpit's my pleasure :D20:52
*** rmelero has quit IRC20:52
odyssey4meto be fair, the osic clusters are those we get the best results from20:52
cloudnullodyssey4me: so i have a deployment going now,20:52
irtermitebelieve it or not... i also have a cloudnull pouncer! yup20:52
cloudnulland its installing gnocci20:52
cloudnullso i'm not seeing the package cache issue.20:52
odyssey4mecloudnull orly20:52
cloudnullTASK [os_tempest : Check if tempest flavor 202 exists] *************************20:52
odyssey4mecloudnull are you deploying with the master branch for all role?20:52
odyssey4mecloudnull ie are you deploying with https://review.openstack.org/#/c/380325/1920:53
*** cmart has joined #openstack-ansible20:53
cloudnullhttps://review.openstack.org/#/c/380325/20:53
cloudnullI did a checkout of that.20:53
odyssey4mevery specifically PS 1920:53
cloudnullcurrently on e2c73f54a1e9b28139b5cde8272c200038e33af720:54
odyssey4mewhat's that SHA for?20:54
*** weezS has quit IRC20:54
cloudnullgit fetch https://git.openstack.org/openstack/openstack-ansible refs/changes/25/380325/19 && git checkout FETCH_HEAD20:54
cloudnullthat's the SHA from your checkout20:54
odyssey4methe repo_server role's SHA?20:55
cloudnull101d613b55c1ee2ea51358812ca41e5dc06727d620:55
evrardjpcloudnull: git review -d 380325,19 too long to type? :p20:55
*** jcrst has joined #openstack-ansible20:55
odyssey4mehrmf20:56
odyssey4methen obviously something else is wrong20:56
odyssey4mewithout container logs we're stuck trying to figure out what20:56
cloudnullnow my sources are the instance defaults .20:56
cloudnullhttp://nova.clouds.archive.ubuntu.com/ubuntu/20:56
cloudnullso maybe its related to the mirror after all?20:57
evrardjpcould we just disable this patch if we know it's buggy?20:57
odyssey4meevrardjp we don't, at this time20:57
cloudnullI can do another kick and lock it down to the osic mirror20:57
odyssey4mecloudnull well, maybe20:57
evrardjpwell that's easy just have a patchset and recheck :p20:57
odyssey4mecloudnull can you confirm that the cache is configured?20:57
odyssey4megood plan cloudnull20:58
cloudnullcat /etc/apt/apt.conf.d/00apt-cacher-proxy20:58
cloudnullAcquire::http { Proxy "http://172.29.236.100:3142"; };20:58
cloudnullits configured20:58
cloudnullAIO test passed -- [ Run Time = 4671 seconds || 77 minutes ]20:58
cloudnullso maybe something mirror + cache related.20:59
evrardjpodyssey4me: did you see that in a regular way?20:59
evrardjp0or is that intermittent?20:59
odyssey4meevrardjp on that review, every time20:59
cloudnulland this instance is running on the same flavor and network that the OSIC gate jobs run on21:00
cloudnullso i dont think its net related.21:00
*** pester has quit IRC21:00
*** pester has joined #openstack-ansible21:01
odyssey4mecloudnull the parent for https://github.com/openstack/openstack-ansible/commit/d98eb49edc4934ac3a18d49748d2b569e9db7520 has container logs, but that patch does not21:02
*** hughmFLEXin has quit IRC21:03
*** javeriak has quit IRC21:04
*** thorst has quit IRC21:05
*** woodard has quit IRC21:05
evrardjpodyssey4me: Oct 12 18:28:58 localhost haproxy[16492]: Server repo_cache-back/aio1_repo_container-b00b4ae2 is DOWN, reason: Layer4 connection problem, info: "Connection refused", check duration: 0ms. 0 active and 0 backup servers left. 0 sessions active, 0 requeued, 0 remaining in queue.21:07
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Execute tempest using updated method  https://review.openstack.org/38221121:07
evrardjpon your patchset 1921:08
evrardjpit doesn't sound good21:08
cloudnullok found the issue with logs21:08
cloudnullPR inbound.21:08
*** haad2 has joined #openstack-ansible21:08
evrardjpcool you make us languish?21:09
*** Mudpuppy has joined #openstack-ansible21:11
*** thorst has joined #openstack-ansible21:11
*** jlockwood has joined #openstack-ansible21:12
*** Mr_Smurf_ has joined #openstack-ansible21:13
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible: Fix container log bind mount regression  https://review.openstack.org/38561921:13
cloudnullodyssey4me evrardjp ^21:13
*** jlockwood has quit IRC21:14
*** retreved has quit IRC21:14
evrardjpwell it makes perfect sense21:14
*** gus_ has joined #openstack-ansible21:14
* cloudnull off to look at the mirror issues.21:15
cloudnullevrardjp: do you think that the cache is flapping ?21:15
*** chris__hultin has joined #openstack-ansible21:15
evrardjpI don't know I don't have enough facts to have a theory21:15
evrardjpbut I see issues from haproxy side of view, which doesn't mean much because I don't have timing info21:16
*** thorst has quit IRC21:16
evrardjpit could have been attempts when the apt-mirror is not yet ready21:16
*** odyssey4me_ has joined #openstack-ansible21:17
*** mgagne_ has joined #openstack-ansible21:17
*** aetaric_ has joined #openstack-ansible21:17
*** Guest7820 has joined #openstack-ansible21:17
evrardjpodyssey4me: I'm definitely lagging behind bugs now :p21:17
*** irtermit- has joined #openstack-ansible21:18
evrardjpcloudnull: you mind to edit odyssey4me's comment this way we can continue with our lives?21:18
*** odyssey4me has quit IRC21:18
*** odyssey4me_ is now known as odyssey4me21:18
evrardjpis it "lives" or "lifes" ?21:18
evrardjpodyssey4me: you're flapping21:19
odyssey4me:p21:19
odyssey4meback now21:19
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible: Fix container log bind mount regression  https://review.openstack.org/38561921:19
odyssey4melost connection21:19
cloudnulldone;21:19
*** haad1 has quit IRC21:19
*** chris_hultin has quit IRC21:19
*** aetaric has quit IRC21:19
*** jheroux has quit IRC21:19
*** palendae has quit IRC21:19
*** haasn has quit IRC21:19
*** mgagne has quit IRC21:19
*** LanceHaig has quit IRC21:19
*** goldenfri has quit IRC21:19
*** Mr_Smurf has quit IRC21:19
*** irtermite has quit IRC21:19
*** gus has quit IRC21:19
cloudnullnetsplit all the things21:19
*** chris__hultin is now known as chris_hultin21:19
evrardjpI will blame this to be the cause of our mirroring issues.21:19
evrardjpthanks cloudnull21:20
*** haasn` has joined #openstack-ansible21:20
cloudnullnp21:20
*** jcrst has quit IRC21:20
odyssey4meif anyone has the time or inclination, here's a few patches to get reviewed: https://review.openstack.org/#/q/project:%255Eopenstack/openstack-ansible.*+status:open+topic:bp/central-test-repository21:20
*** jcrst has joined #openstack-ansible21:20
odyssey4meand a few more, related more specifically to the newton release: https://review.openstack.org/#/q/starredby:jesse-pretorius+project:%255Eopenstack/openstack-ansible.*+status:open21:21
evrardjpI like limechat aggregated view of channels. When there is a netsplit I see everybody leaving the openstack- channels like this one, neutron, ... I don't see anything in my log anymore with the amount of ppl netsplitting :p21:21
evrardjpanyway21:22
*** xar- has quit IRC21:23
*** jmccrory has quit IRC21:23
*** adreznec has quit IRC21:23
*** kamtamtun has quit IRC21:23
*** dalees has quit IRC21:23
*** raddaoui has quit IRC21:23
*** xar- has joined #openstack-ansible21:23
*** javeriak has joined #openstack-ansible21:24
*** jimbaker has quit IRC21:24
*** phschwartz has quit IRC21:24
*** cathrichardson has quit IRC21:24
*** phschwartz has joined #openstack-ansible21:25
evrardjpodyssey4me: about this? https://bugs.launchpad.net/openstack-ansible/+bug/1632645 I think it's time to bring idempotency into our keystone module in the future21:26
openstackLaunchpad bug 1632645 in openstack-ansible "During upgrare M->N old endpoints are not cleaned" [Undecided,New]21:26
evrardjpin the meantime, ops ?21:26
evrardjpor upgrade procedure?21:26
odyssey4menot sure21:27
odyssey4menot thinking too hard about upgrades just yet21:27
odyssey4mebut yes, idempotency would be great21:27
odyssey4meI think it's time for me to stop computering.21:28
*** jmccrory has joined #openstack-ansible21:28
*** adreznec has joined #openstack-ansible21:28
*** kamtamtun has joined #openstack-ansible21:28
*** dalees has joined #openstack-ansible21:28
*** raddaoui has joined #openstack-ansible21:28
*** haasn` has quit IRC21:29
*** haasn has joined #openstack-ansible21:31
*** michaelgugino has quit IRC21:31
*** Jeffrey4l__ has joined #openstack-ansible21:34
*** Jeffrey4l_ has quit IRC21:35
openstackgerritMerged openstack/openstack-ansible: Configure Calico specific BIRD settings in OSA  https://review.openstack.org/38557321:38
*** rmelero has joined #openstack-ansible21:40
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible: Change the common proxy cache manage tasks to be stateful  https://review.openstack.org/38562521:42
*** haad2 has quit IRC21:43
odyssey4me:) cloudnull that's pretty much what I was thinking21:44
*** thorst has joined #openstack-ansible21:45
cloudnullgreat minds :)21:45
cloudnullwhats the status on making xenial voting?21:48
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Execute tempest using updated method  https://review.openstack.org/38221121:48
odyssey4mecloudnull well, so far xenial fails more than trusty - it times out more21:49
odyssey4megenerally speaking we'd need to make it voting with trusty on newton, but drop trusty for ocata21:49
*** thorst has quit IRC21:49
odyssey4mebut making trusty and xenial voting for the integrated gate won't work for us right now - we very rarely get both passing at once21:50
evrardjpseems clean and reduces the amount of bugs in our list. Good addition cloudnull21:50
cloudnulloff to create more bugs21:50
odyssey4meso right now we leave the status quo - for all the roles we have both trusty and xenial voting21:50
logan-xenial timing out is not surprising. i was seeing consistent 1:10 - 1:15 CI builds locally on mitaka+trusty. newton+xenial is building around 1:45 - 1:5021:50
cloudnullok.21:50
cloudnullon my AIO tests i have only been using xenial.21:51
evrardjplogan-: do you have stats on n + trusty?21:51
cloudnullso maybe there is something making it slower in the gate.21:51
cloudnull's/./?/'21:51
odyssey4mefor newton we either need to switch xenial to voting and trusty to non voting for the integrated gate - or we need to cut the time down so we can make them both voting21:51
logan-evrardjp: yes... https://bugs.launchpad.net/openstack-ansible/+bug/1631690 :)21:52
openstackLaunchpad bug 1631690 in openstack-ansible "Trusty AIO fails to start DHCP on test network" [Undecided,New] - Assigned to Jesse Pretorius (jesse-pretorius)21:52
odyssey4mewell newton has a massive bunch more code to skip, so it's not surprising that it's slower21:52
logan-have not had a successful test with newton+trusty yet21:52
evrardjpodyssey4me: I think if have to choose I'd choose xenial for O as the only voting21:52
*** evilt0ne has quit IRC21:52
odyssey4meevrardjp for O, yes - xenial should be the only thing - we need to purge all trusty code21:53
*** jperry has joined #openstack-ansible21:53
*** evilt0ne has joined #openstack-ansible21:53
odyssey4mewe just haven't decided yet when to hit that switch21:53
evrardjpthat's simple: start by removing all the variables files for ubuntu 14, it's gonna break!21:53
*** chris_hultin is now known as chris_hultin|AWA21:53
odyssey4memy vote would be for after milestone 1 when the upgrade work is done21:54
cloudnulllogan-: is that related to the agents container bug noted earlier ?21:54
odyssey4mecloudnull yep, both https://bugs.launchpad.net/openstack-ansible/+bugs?search=Search&field.assignee=jesse-pretorius are related21:54
logan-yep cloudnull it seems related to the stuff we've been seeing in the agents container21:54
cloudnullok.21:54
cloudnullthat's next on my hitlist21:54
odyssey4mecloudnull logan- can add your pub ssh key to some broken nodes if you like - I haven't had a chance to peruse21:54
odyssey4mecloudnull note that https://review.openstack.org/380325 now worked, so there's something definitely wrong in that repo_server patch21:55
cloudnullhttp://cdn.pasteraw.com/o7m0t0dtxsvj3p34uantrsq8cvturvx21:55
*** thorst has joined #openstack-ansible21:55
odyssey4menote that it still ran on osic121:55
cloudnullodyssey4me: seems odd that it would work and then not.21:56
odyssey4meevrardjp cloudnull mhayden can we get https://review.openstack.org/380325 merged asap so that we can get back to testing the roles as they change21:56
odyssey4mecloudnull odd, certainly, but at this stage it's pretty essential that we get back to testing everything properly21:57
mhaydenodyssey4me: looking21:57
odyssey4mewe can take a bit of time to deal with the repo_server issue - and https://review.openstack.org/385625 might just be a perfectly good thing to cover it21:57
*** jperry has quit IRC21:57
odyssey4melogan- did you see my note in https://review.openstack.org/385573 ?21:58
logan-yup! i do need to BP it. will get that done this evening and include the tag bump21:59
logan-thanks21:59
odyssey4mesure thing21:59
odyssey4meit hits none of the current code paths, so it doesn't concern me in terms of risking the release stability21:59
*** thorst has quit IRC22:00
odyssey4metwo birds with one stone :p22:02
logan-yeah. my opinion too. i will be testing it heavily over the next months as we prep newton so if there are issues I will iterate22:02
odyssey4melogan- will you be executing upgrade tests?22:02
odyssey4meie can we count on you to help improve the upgrade process22:03
logan-yes22:03
*** elarson has joined #openstack-ansible22:04
logan-except upgrade is not the top priority so it may be towards christmas until I get to that part unfortunately22:04
odyssey4mewell, we're all volunteers here so we'll take what we can get :)22:05
evrardjpodyssey4me: for 325 we could use master in the ARR22:05
evrardjpfor the repo-server22:05
evrardjpwhen the rest is merged22:05
*** fguillot has quit IRC22:05
odyssey4meevrardjp maybe - that can be a follow on patch22:05
evrardjpwe shouldn't forget this22:05
*** elarson has quit IRC22:05
evrardjpyes sure22:06
odyssey4meI've been trying to get that merged for over a week now22:06
odyssey4meI've been to rally and back22:06
*** elarson has joined #openstack-ansible22:06
odyssey4meaint nothing gonna stop that damned patch from merging now :p22:06
odyssey4mewe had two issues ultimately - one was we broke the repo build, which I got fixed22:06
odyssey4methe next was that the venvs wouldn't build because rally doesn't care about requirements management in terms of the broad openstack process22:07
odyssey4meto prevent that we need to solve https://bugs.launchpad.net/openstack-ansible/+bug/163199222:07
openstackLaunchpad bug 1631992 in openstack-ansible "Allow venvs to be built with disparate python requirements" [High,Confirmed]22:07
odyssey4mebut I eventually got their requirements updated, and fixed a problem with their update\22:07
odyssey4methat merged this afternoon22:08
odyssey4methen the silly cache issues22:08
*** johnmilton has joined #openstack-ansible22:08
odyssey4mecan you see me pulling my hair out yet? ;)22:08
evrardjpyour hair? I thought it was everything else :p22:08
evrardjpbut yes22:08
evrardjpwe can reduce the high importance soon also on the disparate requirements :p22:09
evrardjpeverything is gonna work out :D22:09
odyssey4methe disparate requirements can go down to medium now22:09
odyssey4methe urgency is over22:10
odyssey4merally is now compliant22:10
odyssey4me(mostly)22:10
*** jamesdenton has quit IRC22:12
*** hughmFLEXin has joined #openstack-ansible22:12
evrardjplet's discuss that later when/if other stuff gets more important than this22:12
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Execute tempest using updated method  https://review.openstack.org/38221122:20
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Execute tempest using updated method  https://review.openstack.org/38221122:21
odyssey4me3.14^6 patch sets later22:21
*** jcrst has quit IRC22:23
*** jcrst has joined #openstack-ansible22:25
*** smatzek has joined #openstack-ansible22:27
*** TxGirlGeek has quit IRC22:28
*** AnarchyAo has quit IRC22:35
*** phalmos has quit IRC22:36
*** hughmFLEXin has quit IRC22:37
*** evilt0ne has quit IRC22:42
*** admin0 has quit IRC22:43
*** evilt0ne has joined #openstack-ansible22:44
openstackgerritNish Patwa(nishpatwa_) proposed openstack/openstack-ansible-ops: Added cinder metric collection, telegraf conntrack plugin.  https://review.openstack.org/37913822:47
*** smatzek has quit IRC22:49
*** jimbaker has joined #openstack-ansible22:49
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer: Make ceilometer sample interval configurable  https://review.openstack.org/38205722:52
openstackgerritNish Patwa(nishpatwa_) proposed openstack/openstack-ansible-ops: Adding Kapacitor scripts [WIP]  https://review.openstack.org/38517422:53
*** Guest7820 is now known as palendae22:54
openstackgerritMerged openstack/openstack-ansible: Update all SHAs for Ocata 2016-10-12  https://review.openstack.org/38032522:57
*** jimbaker has quit IRC23:00
cloudnullodyssey4me: logan-: this seems like an overlayfs bug -- "tail: unrecognized file system type 0x794c7630 for ‘/var/log/neutron/neutron-ns-metadata-proxy-fd9a6190-29cc-4d75-b7e5-b1eadb36dda3.log’. please report this to bug-coreutils@gnu.org. reverting to polling"23:03
cloudnullI've not pin pointed it yet.23:03
cloudnullbut working on it23:03
odyssey4memhayden if you're around, a review of https://review.openstack.org/385625 would be great23:03
odyssey4meor evrardjp - or anyone else other than cloudnull for that matter23:03
cjloaderodyssey4me: sorry about that, I originally did that cause I was still learning how to contribute23:05
cjloaderodyssey4me: all good though23:06
*** gus_ is now known as gus23:06
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Switch the repo_server role to master  https://review.openstack.org/38568423:07
odyssey4mecjloader no worries - just doing some clean-up23:08
openstackgerritBjoern Teipel proposed openstack/openstack-ansible-os_cinder: Add remote procedure call tunables to cinder  https://review.openstack.org/36074923:09
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Execute tempest using updated method  https://review.openstack.org/38221123:12
cloudnullodyssey4me logan-: interestingly the module is in the base container23:14
*** spotz is now known as spotz_zzz23:14
cloudnull## /var/lib/lxc/ubuntu-trusty-amd64/rootfs/lib/x86_64-linux-gnu/libdbus-1.so.323:14
*** jimbaker has joined #openstack-ansible23:15
cloudnullwhich is not the case in an instance NOT using overlayfs23:15
odyssey4mecan you load it manually?23:15
odyssey4mehmm, in the instance not using overlayfs does it get it from elsewhere perhaps?23:16
*** cathrichardson has joined #openstack-ansible23:17
cloudnulli thinks its just something that comes in with the net/dns utils.23:17
cloudnullconfirming now23:17
odyssey4mebasically I'm thinking that it tries to load it, doesn't find it, then hunts one of the paths it knows to look into and finds it elsewhere23:18
*** BjoernT has quit IRC23:19
odyssey4mecloudnull can you please put your findings in https://bugs.launchpad.net/openstack-ansible/+bug/1612412 or https://bugs.launchpad.net/openstack-ansible/+bug/1631690 (whichever is applicable) before you head to bed so that I can pick it up in the morning, or try to23:20
openstackLaunchpad bug 1612412 in openstack-ansible "Xenial: neutron agents container unable to resolve names" [Low,Confirmed] - Assigned to Jesse Pretorius (jesse-pretorius)23:20
openstackLaunchpad bug 1631690 in openstack-ansible "Trusty AIO fails to start DHCP on test network" [Undecided,New] - Assigned to Jesse Pretorius (jesse-pretorius)23:20
cloudnullits in the underlay which is then in the container on the delta23:21
cloudnulland i believe we see this in trusty and not xenial because we install so much shit in trusty23:21
cloudnullthat we dont in xenial23:21
cloudnullbasically.23:21
cloudnulli think the only fix for something like this is to disable overlayfs on trusty AIOs23:22
cloudnullor we can go find the busted package providing the lib and remove it23:22
cloudnulland hope that doesn't break something else.23:22
odyssey4mewe can very easily just disable overlayfs for trusty at this point23:22
odyssey4meit's served its purpose23:22
odyssey4mewe just need to restore https://review.openstack.org/351776 and make it not a test thing23:22
odyssey4meshall I revise that quickly?23:23
cloudnulllet me see if i can find the package and remove just that23:23
* cloudnull hunting23:23
odyssey4meok, otherwise feel free to restore that patch, link the two bugs and be happy23:23
odyssey4mewell, lemme revise the patch quickly23:24
odyssey4methen you can opt to use it - or not23:24
cloudnullkk23:24
odyssey4mewhat's the best way to detect trusty instead of the kernel version?23:25
logan-i use ansible_distribution_release usually23:26
odyssey4meyeah, if that's 'trusty' - that'll do23:27
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-lxc_hosts: Remove dbus packages installed in trusty defaults  https://review.openstack.org/38568723:29
cloudnull^ logan- odyssey4me: makes my quick test happier.23:31
cloudnulltrying a full stack now23:31
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Prevent overlayfs being used for Ubuntu Trusty or kernel < 3.18  https://review.openstack.org/35177623:32
odyssey4mecloudnull for some reason that's showing in gerrit as 'cannot merge'23:33
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-lxc_hosts: Remove dbus packages installed in trusty defaults  https://review.openstack.org/38568723:34
cloudnullrebased23:34
*** rmelero has quit IRC23:34
cloudnullwe likely could prune that list down further23:34
odyssey4mecloudnull maybe good to have an upgrade release note indicating the removed packages, just in case.23:35
cloudnulldue to all of the independent gating we should be good to go.23:35
cloudnulli can add that23:35
cloudnull** removing packages from the base container ** I take that back, we should not prune that list. it's too far into the release to do that at this point.23:36
cloudnullbut these packages are not used anywhere.23:36
cloudnullso i think it's safe.23:36
odyssey4mecloudnull it may be worth paring https://review.openstack.org/351776 down to only not doing overlayfs based on the kernel version again23:37
*** rmelero has joined #openstack-ansible23:37
cloudnullyea i'd key off the kernel version.23:38
cloudnullbut in this  case logan-'s install is on kernel 4.423:38
cloudnullso that wouldn't really fix the issue.23:38
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Prevent overlayfs being used for Ubuntu Trusty or kernel < 3.18  https://review.openstack.org/35177623:38
cloudnulli think removing those packages is the key23:38
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Prevent overlayfs being used when kernel < 3.18  https://review.openstack.org/35177623:39
*** markvoelker has quit IRC23:39
odyssey4mecool - so we have a double-fence here23:40
odyssey4meI'd rather have the surgical fix, but I'd also rather prevent an unwanted situation23:40
*** klamath has quit IRC23:40
*** cathrichardson has quit IRC23:42
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Prevent overlayfs being used for an AIO when kernel < 3.18  https://review.openstack.org/35177623:43
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Change the common proxy cache manage tasks to be stateful  https://review.openstack.org/38562523:43
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Switch the repo_server role to master  https://review.openstack.org/38568423:44
*** evilt0ne has quit IRC23:45
*** evilt0ne has joined #openstack-ansible23:47
*** evilt0ne has quit IRC23:47
*** evilt0ne has joined #openstack-ansible23:47
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Prevent overlayfs being used for an AIO when kernel < 3.18  https://review.openstack.org/35177623:49
odyssey4meok, that's definitely me out for the night23:52
odyssey4mecheers all, chat again tomorrow23:52
cloudnullttyl23:53

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