Friday, 2016-04-08

*** ayoung has joined #puppet-openstack00:08
*** xarses has joined #puppet-openstack00:11
*** ducttape_ has joined #puppet-openstack00:15
*** Matum has quit IRC00:23
*** ducttape_ has quit IRC00:34
*** moravec has quit IRC00:42
*** moravec has joined #puppet-openstack00:42
*** cdelatte has quit IRC00:53
*** moravec has quit IRC00:55
*** ducttape_ has joined #puppet-openstack01:05
*** baker has joined #puppet-openstack01:14
*** ducttape_ has quit IRC01:22
*** moravec has joined #puppet-openstack01:44
*** ilbot3 has quit IRC01:46
*** ilbot3 has joined #puppet-openstack01:48
*** baker has quit IRC01:58
*** moravec has quit IRC02:03
*** jtomasek has quit IRC02:15
*** openstackstatus has quit IRC02:19
*** rstarmer has joined #puppet-openstack02:25
*** baker has joined #puppet-openstack02:26
*** baker_ has joined #puppet-openstack02:28
*** rstarmer has quit IRC02:31
*** baker has quit IRC02:31
openstackgerritXiaohua Yuan proposed openstack/puppet-trove: Add related parameters to profiler section  https://review.openstack.org/30318702:33
*** richm has quit IRC02:33
*** richm1 has joined #puppet-openstack02:33
*** xiaohua has joined #puppet-openstack02:35
*** ducttape_ has joined #puppet-openstack02:36
*** richm1 has quit IRC02:38
*** ducttape_ has quit IRC02:41
*** baker has joined #puppet-openstack02:42
*** baker_ has quit IRC02:46
*** baker has quit IRC02:58
*** beddari has quit IRC03:09
*** beddari has joined #puppet-openstack03:20
*** baker has joined #puppet-openstack03:29
*** chandankumar has joined #puppet-openstack03:36
*** baker has quit IRC03:50
*** baker has joined #puppet-openstack03:52
*** baker has quit IRC04:10
*** ducttape_ has joined #puppet-openstack04:37
*** xiaohua has quit IRC04:39
*** xiaohua has joined #puppet-openstack04:40
*** ducttape_ has quit IRC04:41
*** xiaohua has quit IRC04:49
*** rcernin has joined #puppet-openstack04:54
openstackgerritMark Kirkwood proposed openstack/puppet-swift: Allow a replication network to be specified  https://review.openstack.org/30248304:57
*** fedexo has joined #puppet-openstack04:57
*** imcsk8 has quit IRC05:07
*** imcsk8 has joined #puppet-openstack05:07
*** markvoelker has quit IRC05:30
*** xiaohua has joined #puppet-openstack05:47
*** xiaohua has quit IRC05:53
*** saneax_AFK is now known as saneax05:55
openstackgerritMichael Chapman proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/29343605:56
*** xiaohua has joined #puppet-openstack06:08
*** fedexo has quit IRC06:14
*** jmikolajczak has joined #puppet-openstack06:21
*** markvoelker has joined #puppet-openstack06:30
*** astupnikov has joined #puppet-openstack06:35
*** markvoelker has quit IRC06:36
*** ducttape_ has joined #puppet-openstack06:38
*** amoralej has joined #puppet-openstack06:41
*** ducttape_ has quit IRC06:42
*** foxhunt_away is now known as foxhunt7206:44
*** moravec has joined #puppet-openstack06:47
*** xiaohua has quit IRC06:47
*** xiaohua has joined #puppet-openstack06:48
*** moravec has quit IRC06:51
*** xiaohua has quit IRC06:56
*** jtomasek_ has joined #puppet-openstack07:03
*** jaosorior has joined #puppet-openstack07:04
*** jtomasek_ has quit IRC07:04
*** dtantsur|afk is now known as dtantsur07:12
*** gfidente has joined #puppet-openstack07:23
*** gfidente has quit IRC07:23
*** gfidente has joined #puppet-openstack07:23
*** rmart04 has joined #puppet-openstack07:23
*** yottatsa has joined #puppet-openstack07:24
*** yottatsa has quit IRC07:27
*** jpena|off is now known as jpena07:28
*** dgurtner has joined #puppet-openstack07:30
*** jtomasek has joined #puppet-openstack07:38
*** jtomasek has quit IRC07:38
*** jtomasek has joined #puppet-openstack07:38
*** jaosorior has quit IRC07:47
*** jaosorior has joined #puppet-openstack07:48
*** xiaohua has joined #puppet-openstack07:55
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer endpoints  https://review.openstack.org/30257408:01
*** xiaohua has quit IRC08:01
*** igajsin1 has joined #puppet-openstack08:01
*** chem has joined #puppet-openstack08:02
*** chem has quit IRC08:08
*** xiaohua has joined #puppet-openstack08:12
openstackgerritJerzy Mikolajczak proposed openstack/puppet-neutron: Remove deprecated options from agents  https://review.openstack.org/30110608:23
*** xiaohua has quit IRC08:25
*** xiaohua has joined #puppet-openstack08:26
*** jistr has joined #puppet-openstack08:28
*** derekh has joined #puppet-openstack08:28
*** dtantsur is now known as dtantsur|brb08:30
*** markvoelker has joined #puppet-openstack08:31
*** openstackgerrit has quit IRC08:33
*** openstackgerrit has joined #puppet-openstack08:33
*** paramite has joined #puppet-openstack08:34
*** xiaohua has quit IRC08:35
*** markvoelker has quit IRC08:36
*** mgould has joined #puppet-openstack08:37
*** ducttape_ has joined #puppet-openstack08:38
*** yottatsa has joined #puppet-openstack08:42
*** ducttape_ has quit IRC08:43
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer endpoints  https://review.openstack.org/30257408:48
*** yottatsa has quit IRC08:54
*** yottatsa has joined #puppet-openstack08:58
*** saneax is now known as saneax_AFK09:00
*** xiaohua has joined #puppet-openstack09:03
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer endpoints  https://review.openstack.org/30257409:03
*** moravec has joined #puppet-openstack09:14
*** moravec has quit IRC09:18
*** dtantsur|brb is now known as dtantsur09:21
*** xiaohua has quit IRC09:36
*** xiaohua has joined #puppet-openstack09:37
*** mkarpin has quit IRC09:40
*** yottatsa has quit IRC09:43
*** xiaohua has quit IRC09:44
*** gfidente has quit IRC09:45
*** gfidente has joined #puppet-openstack09:47
*** gfidente has quit IRC09:47
*** gfidente has joined #puppet-openstack09:47
*** yottatsa has joined #puppet-openstack09:48
*** igajsin1 has left #puppet-openstack09:53
*** mkarpin has joined #puppet-openstack09:55
*** fwdit has joined #puppet-openstack09:56
tore_hi, I'm having troubles with neutron::agents::ml2::linuxbridge failing, because it attempts to do stuff to /etc/neutron/plugins/linuxbridge/linuxbridge_conf.ini which doesn't exists10:00
tore_I think it should have been using /etc/neutron/plugins/ml2/linuxbridge_agent.ini instead10:00
*** iberezovskiy_ is now known as iberezovskiy10:01
tore_I see in puppet-neutron/lib/puppet/provider/neutron_plugin_linuxbridge/ini_setting.rb that there's a file_path def where this is set - can the broken default be overridden in hiera or puppet manifests somehow?10:02
yottatsaplease review/merge https://review.openstack.org/#/c/302699 Pass correct security groups driver for ML2 plugin10:04
*** leanderthal|afk is now known as leanderthal10:09
*** beddari has quit IRC10:12
*** _degorenko|afk is now known as degorenko10:16
*** beddari has joined #puppet-openstack10:19
*** markvoelker has joined #puppet-openstack10:32
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Remove individual service certificates  https://review.openstack.org/30149310:35
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer endpoints  https://review.openstack.org/30257410:35
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add support for internal/admin endpoint TLS in HAProxy  https://review.openstack.org/30334210:35
*** markvoelker has quit IRC10:39
*** ducttape_ has joined #puppet-openstack10:39
*** ducttape_ has quit IRC10:43
*** xiaohua has joined #puppet-openstack10:43
*** skolekonov has joined #puppet-openstack10:46
openstackgerritDmitry Burmistrov proposed openstack/puppet-oslo: Refactor array values processing  https://review.openstack.org/30227110:47
*** mgould has quit IRC10:50
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer endpoints  https://review.openstack.org/30257410:51
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add support for internal/admin endpoint TLS in HAProxy  https://review.openstack.org/30334210:51
*** xiaohua has quit IRC10:51
*** mgould has joined #puppet-openstack11:03
*** saneax_AFK is now known as saneax11:17
*** saneax is now known as saneax_AFK11:18
*** saneax_AFK is now known as saneax11:18
*** saneax is now known as saneax_AFK11:30
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add support for internal/admin endpoint TLS in HAProxy  https://review.openstack.org/30334211:34
*** chem has joined #puppet-openstack11:36
*** moravec has joined #puppet-openstack11:41
*** moravec has quit IRC11:45
*** dprince has joined #puppet-openstack11:47
*** cdelatte has joined #puppet-openstack11:48
*** xiaohua has joined #puppet-openstack11:49
*** ayoung has quit IRC11:54
*** johnmilton has joined #puppet-openstack11:54
*** moravec has joined #puppet-openstack11:54
*** xiaohua has quit IRC11:56
*** derekh has quit IRC11:58
*** xiaohua has joined #puppet-openstack12:02
*** yottatsa has quit IRC12:03
*** yottatsa has joined #puppet-openstack12:04
openstackgerritMatthew J Black proposed openstack/puppet-neutron: WIP lbaas v2 service plugin support.  https://review.openstack.org/30179712:08
openstackgerritXiaohua Yuan proposed openstack/puppet-trove: Add related parameters to profiler section  https://review.openstack.org/30318712:15
*** jayg|g0n3 is now known as jayg12:20
EmilienMtore_: what version of OpenStack do you install?12:21
*** markvoelker has joined #puppet-openstack12:21
*** ducttape_ has joined #puppet-openstack12:25
EmilienMdgurtner, xarses: fyi: https://review.openstack.org/#/c/303086/12:26
KvisleEmilienM: it's Liberty12:26
Kvisle(tore_ is a coworker of mine)12:26
Kvislebut he just headed out12:26
EmilienMKvisle: do you use master or puppet-neutron or stable/liberty?12:26
Kvislestable/liberty12:27
EmilienMI'm aware we had some issues some weeks ago with linuxbridge12:27
EmilienMwe fixed it in stable/mitaka and master12:27
Kvislewaiting for mitaka packages for el712:27
EmilienMwe're even testing it now in our integration testing12:27
*** xiaohua has quit IRC12:29
*** trown|outtypewww is now known as trown12:29
*** fwdit has quit IRC12:35
openstackgerritDmitry Burmistrov proposed openstack/puppet-keystone: Set oslo options in keystone module through puppet-oslo  https://review.openstack.org/30273812:37
*** ducttape_ has quit IRC12:38
*** jpena is now known as jpena|lunch12:41
*** derekh has joined #puppet-openstack12:43
dmburmistrovEmilienM, hello!, can we finish oslo commits today? :) I updated commit message to be more descriptive - https://review.openstack.org/#/c/302271/12:57
*** amoralej is now known as amoralej|lunch12:57
EmilienMdmburmistrov: sure12:58
openstackgerritMatthew J Black proposed openstack/puppet-horizon: Added ensure_resource for python-memcache package.  https://review.openstack.org/29867912:58
dmburmistrovthanks!12:59
*** Matum has joined #puppet-openstack13:02
*** dgurtner has quit IRC13:20
*** ducttape_ has joined #puppet-openstack13:23
mjblackEmilienM: the beaker tests, what release of openstack are they using?13:26
*** richm has joined #puppet-openstack13:26
mjblackthe neutron change I made is failing because its not able to find the package on ubuntu13:27
EmilienMmjblack: mitaka on stable/mitaka and master13:27
EmilienMmjblack: it's in your logs btw13:27
mjblackhmmm I'm at a loss of why its not finding the packages13:27
EmilienMmjblack: in logs/dpkg-l.txt.gz13:28
mjblackthat'll only tell me what is installed13:28
mjblackthe apt repo doesnt contain the neutron-lbaasv2-agent13:29
EmilienMmjblack: did you talk with ubuntu folks?13:30
EmilienMmjblack: /join #openstack-dev && ping coreycb jamespage13:30
mjblackEmilienM: the thing is I'm using the mitaka apt and I'm able to successfully search and find it13:32
mjblackeven after apt-get upgrade13:32
HunnerEmilienM: Whoops, I was thinking that mongodb.conf.2.6.erb would be used for mongo 2 and mongodb.conf.erb for 313:34
*** dgurtner has joined #puppet-openstack13:34
EmilienMHunner: something is weird, 2.6 should support bindIp13:34
EmilienMHunner: ie: the PR should be OK13:34
EmilienMbut it's not, I'm trying to reproduce13:34
mjblackEmilienM: how hard would it be to update the CI jobs to capture the /etc/apt directory in the logging?13:35
*** amoralej|lunch is now known as amoralej13:36
EmilienMHunner: don't merge the revert now, I'm trying to reproduce the bug and maybe fix it13:40
EmilienMmjblack: show me logs...13:40
Hunnerk13:40
*** jpena|lunch is now known as jpena13:42
EmilienMmfisch: I'm trying to make the puppet schedule (summit) according to your talks13:43
EmilienMit's very challenging13:43
*** dgurtner has quit IRC13:46
*** dgurtner has joined #puppet-openstack13:48
*** paramite is now known as paramite|afk13:51
*** skolekonov_ has joined #puppet-openstack13:54
*** skolekonov has quit IRC13:55
*** ayoung has joined #puppet-openstack13:57
*** saneax_AFK is now known as saneax14:02
jaosorioro/14:03
*** shardy has joined #puppet-openstack14:03
EmilienMjaosorior, Hunner: I think I found an interesting bug in puppetlabs-mongodb14:03
EmilienMhttps://github.com/puppetlabs/puppetlabs-mongodb/blob/master/manifests/server/config.pp#L6514:03
EmilienMmongodb::server::version does not exist, it's mongodb::globals::version14:03
*** skolekonov has joined #puppet-openstack14:03
EmilienMwhich means, version parameter is never addressed14:04
EmilienMand we always deploy the new config file; 2.6 file is never deployed14:04
*** ayoung has quit IRC14:04
*** myatsenko has quit IRC14:04
*** psanchez has joined #puppet-openstack14:04
*** xarses has quit IRC14:04
EmilienMexcept if we set mongo::server::config::version with Hiera14:05
EmilienMbut that's odd14:05
*** skolekonov_ has quit IRC14:06
jaosoriorEmilienM: it also seems to me that 2.6 is the one to be used for the newer mongodb installs https://github.com/puppetlabs/puppetlabs-mongodb/blob/master/manifests/server/config.pp#L10414:06
*** ToMiles has joined #puppet-openstack14:07
jaosoriorit will use mongodb/mongodb.conf.2.6.erb for 2.6.0 and up, as I read that14:07
*** ToMiles has quit IRC14:07
EmilienMjaosorior: no14:07
EmilienMjaosorior: elsif $version and (versioncmp($version, '2.6.0') >= 0)14:07
EmilienM$version is undef by default14:07
EmilienMsee in mongodb::globals14:08
EmilienMwhich means we don't go in this conditional14:08
EmilienMwe go in else for newer versions14:08
jaosoriorEmilienM: oh damn, I see14:08
EmilienMand for tripleo too, we need to 1) set mongo::globals::version 2) fix $version usage14:08
EmilienMwe have 2 patches one on tripleo and one in puppetlabs mongodb14:09
EmilienMI'm working on it14:09
EmilienMour puppet openstack CI did not catch it because we are pin to latest tag14:09
*** skolekonov_ has joined #puppet-openstack14:14
*** paramite|afk is now known as paramite14:15
*** skolekonov has quit IRC14:16
*** skolekonov_ has quit IRC14:19
iberezovskiyEmilienM, hey. Is it possible to recheck only tripleo jobs? I think I saw your comment like 'recheck tripleo'14:19
EmilienMiberezovskiy: no it's not14:19
EmilienMI should stop adding words after recheck, it's confusing14:19
iberezovskiyk, got it :)14:19
*** jaosorior has quit IRC14:20
iberezovskiyIs tripleo ready for oslo module usage?14:20
EmilienMiberezovskiy: yes14:20
EmilienMiberezovskiy: and fuel?14:20
iberezovskiycool. fuel was update yesterday14:20
iberezovskiyupdated*14:21
mwhahahai put a request in to get fuel-ci on puppet-oslo14:21
iberezovskiygreat14:22
openstackgerritMerged openstack/puppet-oslo: Refactor array values processing  https://review.openstack.org/30227114:24
*** pma has quit IRC14:28
EmilienMmwhahaha, iberezovskiy: before merging any patch for using puppet-oslo, I want to see both fuel CI & tripleo green14:29
iberezovskiysure14:29
iberezovskiyit's rule number one I guess :)14:29
EmilienMyeah!14:29
*** dgurtner has quit IRC14:29
*** ducttape_ has quit IRC14:31
openstackgerritIvan Berezovskiy proposed openstack/puppet-nova: Configure oslo related sections using puppet-oslo module  https://review.openstack.org/30283914:33
*** baker has joined #puppet-openstack14:34
*** trown is now known as trown|relocating14:37
*** karimb has joined #puppet-openstack14:38
*** haplo37 has joined #puppet-openstack14:39
*** karimb has quit IRC14:40
*** karimb has joined #puppet-openstack14:41
*** ducttape_ has joined #puppet-openstack14:44
*** xarses has joined #puppet-openstack14:48
*** foxhunt72 is now known as foxhunt_away14:57
openstackgerritDmitry Burmistrov proposed openstack/puppet-keystone: Set oslo options in keystone module through puppet-oslo  https://review.openstack.org/30273815:01
*** mdorman has joined #puppet-openstack15:06
mjblackEmilienM: here are the logs http://logs.openstack.org/97/301797/6/check/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/16c694a/logs/15:07
EmilienMmjblack: well, I would like to see specific failure15:07
*** baker has quit IRC15:07
EmilienMlike http://logs.openstack.org/97/301797/6/check/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/16c694a/console.html#_2016-04-08_12_24_03_96215:09
mjblackEmilienM: line #29903 in http://logs.openstack.org/97/301797/6/check/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/16c694a/console.html15:09
EmilienMmjblack: you have links on the left...15:09
EmilienMmjblack: what is your patch?15:09
mjblackEmilienM: https://review.openstack.org/#/c/301797/15:10
EmilienMmjblack: syntax is wrong15:10
EmilienMand lint is telling you15:10
EmilienMhttps://review.openstack.org/#/c/301797/6/manifests/agents/lbaas.pp15:10
EmilienML11115:11
mjblackEmilienM: yeah it was dinging me on double quotes instead of single quotes15:12
openstackgerritMatthew J Black proposed openstack/puppet-neutron: WIP lbaas v2 service plugin support.  https://review.openstack.org/30179715:13
mjblackalready had the change for that queued, just wanted to figure out the apt issue15:13
EmilienMmjblack: there is no apt issue15:14
EmilienMmjblack: apt was trying to install a package with a wrong name15:14
EmilienMwith { and such15:14
*** imcsk8 is now known as imcsk8|afk15:15
*** ayoung has joined #puppet-openstack15:16
*** alee has joined #puppet-openstack15:16
EmilienMdegorenko: ping about https://etherpad.openstack.org/p/newton-design-puppet15:19
EmilienMdegorenko: do we really need a session about puppet-oslo?15:19
degorenkoEmilienM, well, work is already started, we can remove it now, i guess15:20
EmilienMyeah15:20
*** ducttape_ has quit IRC15:21
*** baker has joined #puppet-openstack15:23
*** mkarpin has left #puppet-openstack15:24
*** leanderthal is now known as leanderthal|afk15:24
*** jayg is now known as jayg|g0n315:25
*** trown|relocating is now known as trown15:31
*** jmikolajczak has quit IRC15:32
*** rmart04 has quit IRC15:34
*** saneax is now known as saneax_AFK15:37
openstackgerritMatthew J Black proposed openstack/puppet-horizon: Added ensure_resource for python-memcache package.  https://review.openstack.org/29867915:40
*** jayg|g0n3 is now known as jayg15:40
*** ducttape_ has joined #puppet-openstack15:42
mwhahahamgould: do you mind if i propose tests for that patch for you?15:43
mwhahahai don't want to step on your toes or anything :)15:43
EmilienMmwhahaha: when are fuel sessions at summit?15:44
EmilienMmwhahaha: how does it overlap with puppet?15:44
mwhahahalet me dig up the schedule we're working on15:44
EmilienMmfisch or clayton : can you attend puppet sessions on Wednesday? I'm trying to find a slot to talk about docker15:45
mwhahahahttps://etherpad.openstack.org/p/fuel-newton-summit-planning15:45
claytonlikely, let me see when my talks are15:45
mgouldmwhahaha, please do!15:45
EmilienMomg it overlaps a lot15:45
mwhahahawell not all of it is puppet related15:45
claytonI have a talk wed before lunch, but I should be able to make it work other than that15:46
mwhahahawe've got the fuel/puppet integration thing on thursday @9:5015:46
EmilienMclayton: ok15:46
claytonto be honest, I really need to start looking at the schedule soon, I don't know what I'm doing when yet15:46
mwhahahamgould: ok i'll get something together here in a minute15:46
mgouldmwhahaha, thanks!15:46
openstackgerritMerged openstack/puppet-openstack-integration: Implement a script to generate Puppetfile  https://review.openstack.org/30069615:47
*** paramite has quit IRC15:47
openstackgerritAlex Schultz proposed openstack/puppet-oslo: Add oslo.middleware defines  https://review.openstack.org/27442215:48
*** saneax_AFK is now known as saneax15:49
mjblackEmilienM: the reference for stdlib for ensure_packages has it as a hash15:50
*** dtantsur is now known as dtantsur|afk15:50
openstackgerritMerged openstack/puppet-openstack-integration: scenario001/centos7: deploy Ceph on IPv6  https://review.openstack.org/30308615:51
mjblackbut that explains why I missed it15:51
*** fedexo has joined #puppet-openstack15:51
mjblack:D15:51
*** ducttape_ has quit IRC15:53
*** bnemec has quit IRC15:56
*** astupnikov has quit IRC16:00
*** aimon has joined #puppet-openstack16:04
*** bnemec has joined #puppet-openstack16:04
openstackgerritMiles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/30239016:08
openstackgerritAlex Schultz proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/30237616:09
mwhahahamgould: -^ tests16:09
mgouldmwhahaha, thanks!16:10
mgouldcontain_exec looks like just the thing16:13
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: bump RDO to latest Mitaka  https://review.openstack.org/30352216:15
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: bump RDO to latest Mitaka  https://review.openstack.org/30352216:16
*** imcsk8|afk is now known as imcsk816:19
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: neutron: use ipv6 network for tunneling when needed  https://review.openstack.org/30352716:24
openstackgerritMiles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/30237616:25
*** _ducttape_ has joined #puppet-openstack16:26
mgouldmwhahaha, fixed a couple of typos ^^^16:26
openstackgerritDmitry Burmistrov proposed openstack/puppet-heat: Set oslo options in heat module through puppet-oslo  https://review.openstack.org/30352816:26
EmilienMmwhahaha: jpena found something in puppet-glance16:27
EmilienMwith the patch we merged yesterday https://github.com/openstack/puppet-glance/commit/d0957fb622a0f5f4f6e038aa22d87ee61a2f1ad2#diff-b74717103ca8cd0432d6d26461aaca64L10516:27
*** karimb has quit IRC16:27
EmilienMit seems that list = request('image', 'list', '--long') worked fine16:28
EmilienMand  list = request('image', 'list') is broken (for packstack)16:28
EmilienMbut I noticed that list is not working anymore good16:28
EmilienMwe talked about it yesterday16:28
jpenaEmilienM mwhahaha I think I know why16:31
jpenait's in https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack.rb#L7416:31
jpenawe do response = openstack(service, action, '--quiet', '--format', 'csv', properties)16:31
jpenaso service=='image', action=='list', and properties =='--long'16:32
jpenaif we remove --long, properties == ''16:32
jpenaand that's what is causing openstackclient to complain16:32
EmilienMoh right16:32
openstackgerritMiles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/30239016:33
*** derekh has quit IRC16:33
EmilienMjpena: nice catch, I confirm the bug16:34
dmsimardjpena to the rescue \o/16:34
EmilienMas usual ...16:34
dmsimardRIGHT IN THE MIDDLE OF A RELEASE16:34
jpenaso, should this be fixed in puppet-openstacklib or puppet-glance? Can properties be considered an optional argument?16:35
dmsimardjpena: at first glance (see what I did there) it looks like https://jenkins02.openstack.org/job/gate-packstack-integration-scenario001-tempest-dsvm-centos7/50/console is passing16:36
dmsimardbut OPM stable/mitaka could not carry that commit (yet) checking16:36
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: swift: use ipv6 binding on centos7 nodes  https://review.openstack.org/30353016:36
EmilienMjpena: mwhahaha did that recently https://github.com/openstack/puppet-glance/commit/d0957fb622a0f5f4f6e038aa22d87ee61a2f1ad216:37
EmilienMjpena: in puppet-glance I think16:39
jpenaEmilienM: ok, so as a quick fix I'm adding back --long to the call16:39
*** imcsk8 is now known as imcsk8|afk16:40
dmsimardHow did integration jobs and beaker jobs pass though ?16:40
dmsimardIf that was very broken ?16:40
EmilienMjpena: we have functional tests that will break if your patch is not valid anyway, so go ahead16:40
EmilienMdmsimard: so on IRC yesterday I said that I noticed the openstack image list operation was failing in catalog, but not critical, puppet catalog didn't fail16:41
dmsimardthe image list is called, amongst other things, when creating an image (i.e tempest pre-provisioning)16:42
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: bump RDO to latest Mitaka  https://review.openstack.org/30352216:42
dmsimardso it should have failed16:42
EmilienMdmsimard: http://logs.openstack.org/26/301826/2/gate/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/3b88dcc/console.html#_2016-04-07_18_48_46_64316:42
EmilienMyes, it should have failed16:42
EmilienMjpena: is it what you see? ^16:43
openstackgerritJavier Peña proposed openstack/puppet-glance: Fix image list in glance_image provider  https://review.openstack.org/30353316:44
jpenaEmilienM: exactly that16:44
EmilienMjpena: thanks16:45
dmsimardjpena: you probably need to update spec tests too16:46
jpenadmsimard: mmm, didn't think of that16:46
jpenaI'll check it16:46
EmilienMwhy does it need backport? patch from mwhahaha merged yesterday or so16:46
dmsimardjpena: i.e, https://github.com/openstack/puppet-glance/commit/d0957fb622a0f5f4f6e038aa22d87ee61a2f1ad2#diff-b74717103ca8cd0432d6d26461aaca64R9616:46
openstackgerritMatthew J Black proposed openstack/puppet-neutron: WIP lbaas v2 service plugin support.  https://review.openstack.org/30179716:46
dmsimardjpena: wait, no, wrong link16:47
jpenaEmilienM: https://review.openstack.org/#/c/301706/16:47
*** rmart04 has joined #puppet-openstack16:47
EmilienMjpena: oh right :-)16:47
*** rmart04 has quit IRC16:48
jpenaI need to fix the commit msg, btw16:48
mwhahahaso it was the space16:48
mwhahahawe can fix it but we should also create a bug to openstack client, it shouldn't puke on trailing spaces16:48
*** _ducttape_ has quit IRC16:49
dmsimardmwhahaha: well technically '' is not a trailing space16:52
mwhahahaso it's null?16:52
dmsimardmwhahaha: http://paste.openstack.org/show/493514/16:53
dmsimard:P16:53
mwhahahawhat teh16:54
dmsimardit can be a literal character or a string delimiter16:54
*** rcernin has quit IRC16:55
mwhahahaby default that last param is [] i think, we really should fix the providers because we shouldn't have to specify --long16:57
mwhahahajpena: it's going to fail on the tests, do you want me to update them for you?16:57
jpenamwahaha: I'm fixing them, thx16:58
mwhahahak16:58
jpenaoops, I missed one 'ha' :o)17:02
openstackgerritJavier Peña proposed openstack/puppet-glance: Fix image list in glance_image provider  https://review.openstack.org/30353317:03
mjblackEmilienM: with regards to your comment about the neutron_lbaas_service_config, it's suppose to configure the service_provider on neutron server17:03
mjblackneither ubuntu or redhat are doing --config-file /etc/neutron/services_lbass.conf in the service scripts17:04
EmilienMmjblack: ok, I'm just asking because martin this the original work17:04
mjblacksure np17:04
mjblacklbaas v2 has two components, the services (neutron-server) and agent17:04
mjblackI'm actually thinking we can consolidate neutron_*_services_config into one type/provider17:05
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Implement Cinder API WSGI support  https://review.openstack.org/28796417:11
*** brandon has joined #puppet-openstack17:12
*** xarses_ has joined #puppet-openstack17:12
*** xarses has quit IRC17:13
EmilienMmjblack: http://logs.openstack.org/91/288091/7/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/2eef0ca/logs/apache/cinder_wsgi_error.txt.gz17:13
*** brandon has left #puppet-openstack17:14
EmilienMIOError: [Errno 13] Permission denied: '/var/log/cinder/osapi_volume.log'17:14
EmilienMwe need to change something17:14
* mgould -> home; good night!17:15
*** mgould has quit IRC17:15
*** Matum has quit IRC17:17
*** jpena is now known as jpena|off17:19
*** shardy has quit IRC17:20
*** aimon has quit IRC17:21
*** aimon has joined #puppet-openstack17:22
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Implement Cinder API WSGI support  https://review.openstack.org/28796417:22
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: deploy cinder api in WSGI with Apache  https://review.openstack.org/28809117:23
EmilienMConfigFileValueError: Value for option local_ip is not valid: ::1 is not an IPv4 address17:23
EmilienMI'm trying to deploy neutron with ipv6 tunnels17:24
*** gfidente has quit IRC17:24
*** jistr has quit IRC17:25
*** baker has quit IRC17:26
*** fedexo has quit IRC17:26
*** xarses_ is now known as xarses17:33
*** degorenko is now known as _degorenko|afk17:34
*** baker has joined #puppet-openstack17:35
*** fawadkhaliq has joined #puppet-openstack17:39
*** baker has quit IRC17:40
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: swift: use ipv6 binding on centos7 nodes  https://review.openstack.org/30353017:47
*** ducttape_ has joined #puppet-openstack17:47
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: deploy & test Zaqar  https://review.openstack.org/28151517:49
EmilienMmjblack: lbaas1 works fine: https://review.openstack.org/#/c/27229717:51
EmilienMmjblack: so the plan is to have lbaasv1 tests until we can have v2 with your work17:52
EmilienMoh wait, tests didn't run17:53
*** fawadkhaliq has quit IRC18:03
*** fawadkhaliq has joined #puppet-openstack18:03
mjblackEmilienM: getting my hopes up to destroy them :D18:08
*** baker has joined #puppet-openstack18:10
openstackgerritMatthew J Black proposed openstack/puppet-neutron: WIP lbaas v2 service plugin support.  https://review.openstack.org/30179718:14
*** ayoung has quit IRC18:14
EmilienMmwhahaha: https://review.openstack.org/#/c/303533/ fails18:16
EmilienMhttp://logs.openstack.org/33/303533/2/check/gate-puppet-glance-puppet-beaker-rspec-centos-7/fbae8f2/console.html#_2016-04-08_17_24_19_75318:16
EmilienM(Keystone_endpoint[RegionOne/Image Service::image] => Service[glance-api] => Keystone_endpoint[RegionOne/Image Service::image])18:16
mwhahahawell that's odd18:17
EmilienMI'll dig after my lunch, but feel free to look :-P18:17
*** ducttape_ has quit IRC18:18
mwhahahaoh yea don't we test for a cycle in glance?18:18
mwhahahawe should remove that test18:18
*** foxhunt_away is now known as foxhunt7218:22
mjblackEmilienM: will that integration test ever go green if you're making it test the cinder wsgi change?18:26
openstackgerritMatthew J Black proposed openstack/puppet-neutron: WIP lbaas v2 service plugin support.  https://review.openstack.org/30179718:30
openstackgerritAde Lee proposed openstack/puppet-barbican: Added keystone config to barbican api manifest  https://review.openstack.org/26346118:48
openstackgerritAde Lee proposed openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module  https://review.openstack.org/26346218:48
*** dprince has quit IRC18:56
* EmilienM back19:02
EmilienMmjblack: it should yes19:02
*** ayoung has joined #puppet-openstack19:09
EmilienMmwhahaha: re puppet-glance, why should we drop the test?19:15
*** imcsk8|afk is now known as imcsk819:17
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: run_tests: list tempest plugins at the end of the job  https://review.openstack.org/30358719:18
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: deploy & test neutron lbaasv1 service  https://review.openstack.org/27229719:19
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: run_tests: list tempest plugins at the end of the job  https://review.openstack.org/30358719:21
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: deploy & test neutron lbaasv1 service  https://review.openstack.org/27229719:21
EmilienMmwhahaha: do we drop them?19:22
EmilienMI've looked at the code and I don't see why we have this now19:22
mwhahahayea19:23
mwhahahaand they just make it harder to figure out failures19:23
EmilienMI did recheck, to see if it's consistent19:23
EmilienMok19:23
mwhahahacause we see those and assume it's that,w hen that's actually a test :D19:23
*** foxhunt72 is now known as foxhunt_away19:25
openstackgerritEmilien Macchi proposed openstack/puppet-glance: acceptance: drop dependency cycles tests  https://review.openstack.org/30359319:26
openstackgerritEmilien Macchi proposed openstack/puppet-glance: Fix image list in glance_image provider  https://review.openstack.org/30353319:27
EmilienMmwhahaha: I confirm jpena's patch works fine19:32
EmilienMwe don't have the error in osclient with glance19:32
EmilienMyou can double check in http://logs.openstack.org/33/303533/2/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/5ef916f/console.html19:32
*** karimb has joined #puppet-openstack19:34
EmilienMdmsimard: can you review again https://review.openstack.org/#/c/303522/ please?19:38
mwhahahasure19:39
mwhahahawe should fix the provider not to require the extra options19:39
mwhahahaor at least merge it with the options we are hard coding in the openstacklib provider19:40
openstackgerritAde Lee proposed openstack/puppet-barbican: Added keystone config to barbican api manifest  https://review.openstack.org/26346119:41
openstackgerritAde Lee proposed openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module  https://review.openstack.org/26346219:41
EmilienMmwhahaha: in the same patch?19:43
mwhahahawell it would fix that19:44
mwhahahabut no we can do it later19:44
mwhahahajust an observation19:44
mwhahahai could see this issue coming up again next time someone adds an new openstack client thing with no additional options19:45
EmilienMmwhahaha: oh you mean fix it in openstack.rb provider (openstacklib) ?19:45
mwhahahahttps://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack.rb#L6419:45
mwhahahayea19:45
EmilienM+119:45
mwhahahaproperties is techincally a required param19:46
mwhahahahey look, fuel-ci on puppet-oslo https://review.openstack.org/#/c/274422/19:47
mwhahaha:)19:47
*** amoralej has left #puppet-openstack19:48
EmilienMmwhahaha: excellent new19:50
EmilienMI'll review it shortly19:50
EmilienMmwhahaha: I want to see it in puppet-nova :P19:50
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: deploy & test Zaqar  https://review.openstack.org/28151519:52
dmsimardEmilienM: review what again, https://review.openstack.org/#/c/303522/ there hasn't been a new patchset19:52
EmilienMdmsimard: I'm asking you to -1/+1 just19:54
EmilienMdmsimard: it's on master19:54
EmilienMI thought it was good after our convo on #rdo19:54
dmsimardright19:54
*** bodepd has quit IRC19:57
*** zeus has quit IRC19:58
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: swift: use ipv6 binding on centos7 nodes  https://review.openstack.org/30353019:58
*** bookwar has quit IRC19:58
*** zeus has joined #puppet-openstack20:03
*** zeus is now known as Guest8001920:03
openstackgerritDmitry Ilyin proposed openstack/puppet-ec2api: Initial commit for puppet-ec2api  https://review.openstack.org/27610320:04
*** johnmilton has quit IRC20:06
*** bodepd has joined #puppet-openstack20:06
*** bookwar has joined #puppet-openstack20:07
aleeEmilienM, _degorenko|afk -- finally ! https://review.openstack.org/#/c/26346120:09
*** fawadkhaliq has quit IRC20:11
*** fawadkhaliq has joined #puppet-openstack20:11
mwhahahai wonder if we can make an openstacklib::wsgi::apache class to replace all the different ones we have out there and allow them to just pass params to the openstacklib version20:12
mwhahahaseems like we might have a lot of duplication in those20:12
*** karimb has quit IRC20:17
*** ducttape_ has joined #puppet-openstack20:18
*** ducttape_ has quit IRC20:23
*** dhollinger has joined #puppet-openstack20:28
beaglesEmilienM, I was running into some errors in the swift puppet modules when doing an "undercloud install" on trunk. It was erroring out on https://github.com/openstack/puppet-swift/blob/master/manifests/storage/account.pp#L6220:30
beaglescomplaining that it couldn't find the resource ...20:31
EmilienMbeagles: can we take that on #tripleo maybe?20:31
beaglesyup :020:31
*** jayg is now known as jayg|g0n320:32
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/29655720:35
EmilienMany core around please look https://review.openstack.org/#/c/303593/20:37
EmilienMand the most important: https://review.openstack.org/#/c/303533/20:37
dmsimardEmilienM: cool for https://review.openstack.org/#/c/303533/ so can we backport it once it merges ?21:00
EmilienMdmsimard: of course21:00
dmsimardok21:00
EmilienMdmsimard: submit it and I"ll +221:00
openstackgerritDavid Moreau Simard proposed openstack/puppet-glance: Fix image list in glance_image provider  https://review.openstack.org/30361321:01
EmilienMdmsimard: you missed cherry-pick -x21:01
dmsimardI cherry-picked from gerrit :/21:01
EmilienMyou used Gerrit UI21:01
dmsimardI'll fix21:01
EmilienMand the patch is not merged in master21:01
dmsimardoh, right21:01
openstackgerritDavid Moreau Simard proposed openstack/puppet-glance: Fix image list in glance_image provider  https://review.openstack.org/30361321:04
EmilienMdmsimard: I'm afraid it might fail, maybe we also need to backport https://review.openstack.org/#/c/303593/21:04
EmilienMbut let's see how it goes21:05
dmsimardok21:05
*** trown is now known as trown|outtypewww21:05
*** cdelatte has quit IRC21:07
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Remove user/role prefetch to support multi-domain.  https://review.openstack.org/29930121:13
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: WIP: multi-domain testing with ldap backend.  https://review.openstack.org/29637021:14
EmilienMalee: reviewed21:18
aleeEmilienM, thanks21:19
EmilienMalee: -1 on both, please look21:19
aleeEmilienM, what do you mean by "please add tests in the file" in https://review.openstack.org/#/c/263461/23/spec/acceptance/basic_barbican_spec.rb ?21:21
openstackgerritMerged openstack/puppet-glance: acceptance: drop dependency cycles tests  https://review.openstack.org/30359321:21
EmilienMalee: the keystone thing21:21
openstackgerritMerged openstack/puppet-glance: Fix image list in glance_image provider  https://review.openstack.org/30353321:21
EmilienMdo not create a new acceptance manifest21:21
*** karimb has joined #puppet-openstack21:23
*** haplo37 has quit IRC21:24
aleeEmilienM, ah, so you just want me to concatenate the contents of the new keystone_barbican_spec.rb file to the end of basic_barbcian_spec.rb ?21:24
aleeEmilienM, I'm trying to test both scenarios -- one with keystone auth and one without21:25
EmilienMalee: yeah, use one scenario21:26
EmilienMit adds complexity21:26
EmilienMfor a first iteration, not sure we want it21:26
aleeEmilienM, so you want me to just test with keystine auth (and not without)?  basically then I can remove the basic_barbican_spec.rb ..21:28
EmilienMalee: please move it to basic21:28
EmilienMlater, in wsgi, we'll drop basic and use with_apache21:28
EmilienMsee second review21:28
*** mdorman has quit IRC21:30
aleeEmilienM, seems a bit of manipulation to fix up basic to include keystone -- and then drop it entirely in the next patch ..21:30
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: deploy & test Zaqar  https://review.openstack.org/28151521:31
EmilienMalee: ok, you can keep it like this21:32
EmilienMalee: please just drop https://review.openstack.org/#/c/263462/20/spec/acceptance/basic_barbican_spec.rb21:32
EmilienMalee: and update https://review.openstack.org/#/c/263462/20/spec/acceptance/keystone_barbican_spec.rb to deploy apache21:33
EmilienMalee: I want to use apache by default at maximum21:33
aleeEmilienM, ok21:33
*** twm2016 has joined #puppet-openstack21:34
twm2016tristanC: Hey, I just assigned myself to a bug you reported!21:34
EmilienMalee: still -1 on https://review.openstack.org/#/c/263461/21:35
twm2016https://bugs.launchpad.net/puppet-gnocchi/+bug/154373021:35
openstackLaunchpad bug 1543730 in puppet-gnocchi "puppet-gnocchi doesn't manage the pipeline in api-paste.ini" [Medium,Confirmed] - Assigned to Trevor McCasland (twm2016)21:35
aleeEmilienM, ok -- actually by only having one test scenario, I dont need any of that cleanup code.21:37
EmilienMalee: I just sent new comments.21:38
aleeEmilienM, yeah - I was referring to your comments in https://review.openstack.org/#/c/263461/23/spec/spec_helper_acceptance.rb21:39
aleethe cleanup code there is to make sure things don't get messed up from run to run21:39
EmilienMalee: look https://github.com/openstack/puppet-openstack_spec_helper/blob/master/lib/puppet-openstack_spec_helper/beaker_spec_helper.rb21:39
EmilienMwe have a lib for that now21:39
aleeEmilienM, so -- the key bit I needed is this -- https://review.openstack.org/#/c/263461/23/spec/spec_helper_acceptance.rb  lines 59-6621:41
*** ducttape_ has joined #puppet-openstack21:42
aleeEmilienM, I dont need that anymore as there will be only one test, but may need it later if we add another more complicated test --21:42
aleelike maybe deploying with dogtag/ipa as the backend21:42
aleeEmilienM, so in that case, given that we now have a lib. what would that file look like?21:43
EmilienMalee: that's why I'm asking to have only one beaker file21:43
EmilienMit makes things simpler21:43
EmilienMplease use basic file for now, and switch to with_apache on the wsgi patch21:44
EmilienMwe can iterate later21:44
aleeEmilienM, ok21:44
*** twm2016 has quit IRC21:48
pabelangerEmilienM: do you mind linking me that review again for puppet421:55
*** ducttape_ has quit IRC21:59
*** karimb has quit IRC22:02
EmilienMpabelanger: https://review.openstack.org/29655722:06
*** dhollinger has quit IRC22:10
EmilienMmjblack: in case I was not clear for your lbaasv2 patch: we want to test lbaasv2 in acceptance, so please add the class & parameters, like you would deploy it22:12
EmilienMmjblack: you can pick the haproxy backend probably22:12
pabelangerEmilienM: thanks22:14
openstackgerritAde Lee proposed openstack/puppet-barbican: Added keystone config to barbican api manifest  https://review.openstack.org/26346122:15
openstackgerritAde Lee proposed openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module  https://review.openstack.org/26346222:15
aleeEmilienM, ^^ should have your requested changes22:15
*** markvoelker has quit IRC22:16
EmilienMalee: cool; afk now, will look later tonight22:18
aleeEmilienM, thanks!22:18
aleemwhahaha, _degorenko|afk ya'll too please.  thanks!22:19
mwhahahasure22:19
*** karimb has joined #puppet-openstack22:20
*** fawadkhaliq has quit IRC22:25
*** fawadkhaliq has joined #puppet-openstack22:25
*** karimb has quit IRC22:27
*** karimb has joined #puppet-openstack22:27
*** baker has quit IRC22:29
*** Guest80019 is now known as zeus22:34
*** zeus has joined #puppet-openstack22:34
pabelangerEmilienM: okay, next step is to build facter manually and start debugging it with gdb22:38
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/29655722:51
*** karimb has quit IRC22:53
*** karimb has joined #puppet-openstack22:54
*** dgurtner has joined #puppet-openstack22:58
*** karimb has quit IRC23:01
*** richm has quit IRC23:10
*** markvoelker has joined #puppet-openstack23:16
*** markvoelker has quit IRC23:21
*** xarses has quit IRC23:39
*** fawadkhaliq has quit IRC23:48
*** fawadkhaliq has joined #puppet-openstack23:48
*** markvoelker has joined #puppet-openstack23:52
*** guessi has quit IRC23:56
*** guessi has joined #puppet-openstack23:56

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