*** ayoung has joined #puppet-openstack | 00:08 | |
*** xarses has joined #puppet-openstack | 00:11 | |
*** ducttape_ has joined #puppet-openstack | 00:15 | |
*** Matum has quit IRC | 00:23 | |
*** ducttape_ has quit IRC | 00:34 | |
*** moravec has quit IRC | 00:42 | |
*** moravec has joined #puppet-openstack | 00:42 | |
*** cdelatte has quit IRC | 00:53 | |
*** moravec has quit IRC | 00:55 | |
*** ducttape_ has joined #puppet-openstack | 01:05 | |
*** baker has joined #puppet-openstack | 01:14 | |
*** ducttape_ has quit IRC | 01:22 | |
*** moravec has joined #puppet-openstack | 01:44 | |
*** ilbot3 has quit IRC | 01:46 | |
*** ilbot3 has joined #puppet-openstack | 01:48 | |
*** baker has quit IRC | 01:58 | |
*** moravec has quit IRC | 02:03 | |
*** jtomasek has quit IRC | 02:15 | |
*** openstackstatus has quit IRC | 02:19 | |
*** rstarmer has joined #puppet-openstack | 02:25 | |
*** baker has joined #puppet-openstack | 02:26 | |
*** baker_ has joined #puppet-openstack | 02:28 | |
*** rstarmer has quit IRC | 02:31 | |
*** baker has quit IRC | 02:31 | |
openstackgerrit | Xiaohua Yuan proposed openstack/puppet-trove: Add related parameters to profiler section https://review.openstack.org/303187 | 02:33 |
---|---|---|
*** richm has quit IRC | 02:33 | |
*** richm1 has joined #puppet-openstack | 02:33 | |
*** xiaohua has joined #puppet-openstack | 02:35 | |
*** ducttape_ has joined #puppet-openstack | 02:36 | |
*** richm1 has quit IRC | 02:38 | |
*** ducttape_ has quit IRC | 02:41 | |
*** baker has joined #puppet-openstack | 02:42 | |
*** baker_ has quit IRC | 02:46 | |
*** baker has quit IRC | 02:58 | |
*** beddari has quit IRC | 03:09 | |
*** beddari has joined #puppet-openstack | 03:20 | |
*** baker has joined #puppet-openstack | 03:29 | |
*** chandankumar has joined #puppet-openstack | 03:36 | |
*** baker has quit IRC | 03:50 | |
*** baker has joined #puppet-openstack | 03:52 | |
*** baker has quit IRC | 04:10 | |
*** ducttape_ has joined #puppet-openstack | 04:37 | |
*** xiaohua has quit IRC | 04:39 | |
*** xiaohua has joined #puppet-openstack | 04:40 | |
*** ducttape_ has quit IRC | 04:41 | |
*** xiaohua has quit IRC | 04:49 | |
*** rcernin has joined #puppet-openstack | 04:54 | |
openstackgerrit | Mark Kirkwood proposed openstack/puppet-swift: Allow a replication network to be specified https://review.openstack.org/302483 | 04:57 |
*** fedexo has joined #puppet-openstack | 04:57 | |
*** imcsk8 has quit IRC | 05:07 | |
*** imcsk8 has joined #puppet-openstack | 05:07 | |
*** markvoelker has quit IRC | 05:30 | |
*** xiaohua has joined #puppet-openstack | 05:47 | |
*** xiaohua has quit IRC | 05:53 | |
*** saneax_AFK is now known as saneax | 05:55 | |
openstackgerrit | Michael Chapman proposed openstack/puppet-tripleo: Add neutron profiles https://review.openstack.org/293436 | 05:56 |
*** xiaohua has joined #puppet-openstack | 06:08 | |
*** fedexo has quit IRC | 06:14 | |
*** jmikolajczak has joined #puppet-openstack | 06:21 | |
*** markvoelker has joined #puppet-openstack | 06:30 | |
*** astupnikov has joined #puppet-openstack | 06:35 | |
*** markvoelker has quit IRC | 06:36 | |
*** ducttape_ has joined #puppet-openstack | 06:38 | |
*** amoralej has joined #puppet-openstack | 06:41 | |
*** ducttape_ has quit IRC | 06:42 | |
*** foxhunt_away is now known as foxhunt72 | 06:44 | |
*** moravec has joined #puppet-openstack | 06:47 | |
*** xiaohua has quit IRC | 06:47 | |
*** xiaohua has joined #puppet-openstack | 06:48 | |
*** moravec has quit IRC | 06:51 | |
*** xiaohua has quit IRC | 06:56 | |
*** jtomasek_ has joined #puppet-openstack | 07:03 | |
*** jaosorior has joined #puppet-openstack | 07:04 | |
*** jtomasek_ has quit IRC | 07:04 | |
*** dtantsur|afk is now known as dtantsur | 07:12 | |
*** gfidente has joined #puppet-openstack | 07:23 | |
*** gfidente has quit IRC | 07:23 | |
*** gfidente has joined #puppet-openstack | 07:23 | |
*** rmart04 has joined #puppet-openstack | 07:23 | |
*** yottatsa has joined #puppet-openstack | 07:24 | |
*** yottatsa has quit IRC | 07:27 | |
*** jpena|off is now known as jpena | 07:28 | |
*** dgurtner has joined #puppet-openstack | 07:30 | |
*** jtomasek has joined #puppet-openstack | 07:38 | |
*** jtomasek has quit IRC | 07:38 | |
*** jtomasek has joined #puppet-openstack | 07:38 | |
*** jaosorior has quit IRC | 07:47 | |
*** jaosorior has joined #puppet-openstack | 07:48 | |
*** xiaohua has joined #puppet-openstack | 07:55 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer endpoints https://review.openstack.org/302574 | 08:01 |
*** xiaohua has quit IRC | 08:01 | |
*** igajsin1 has joined #puppet-openstack | 08:01 | |
*** chem has joined #puppet-openstack | 08:02 | |
*** chem has quit IRC | 08:08 | |
*** xiaohua has joined #puppet-openstack | 08:12 | |
openstackgerrit | Jerzy Mikolajczak proposed openstack/puppet-neutron: Remove deprecated options from agents https://review.openstack.org/301106 | 08:23 |
*** xiaohua has quit IRC | 08:25 | |
*** xiaohua has joined #puppet-openstack | 08:26 | |
*** jistr has joined #puppet-openstack | 08:28 | |
*** derekh has joined #puppet-openstack | 08:28 | |
*** dtantsur is now known as dtantsur|brb | 08:30 | |
*** markvoelker has joined #puppet-openstack | 08:31 | |
*** openstackgerrit has quit IRC | 08:33 | |
*** openstackgerrit has joined #puppet-openstack | 08:33 | |
*** paramite has joined #puppet-openstack | 08:34 | |
*** xiaohua has quit IRC | 08:35 | |
*** markvoelker has quit IRC | 08:36 | |
*** mgould has joined #puppet-openstack | 08:37 | |
*** ducttape_ has joined #puppet-openstack | 08:38 | |
*** yottatsa has joined #puppet-openstack | 08:42 | |
*** ducttape_ has quit IRC | 08:43 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer endpoints https://review.openstack.org/302574 | 08:48 |
*** yottatsa has quit IRC | 08:54 | |
*** yottatsa has joined #puppet-openstack | 08:58 | |
*** saneax is now known as saneax_AFK | 09:00 | |
*** xiaohua has joined #puppet-openstack | 09:03 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer endpoints https://review.openstack.org/302574 | 09:03 |
*** moravec has joined #puppet-openstack | 09:14 | |
*** moravec has quit IRC | 09:18 | |
*** dtantsur|brb is now known as dtantsur | 09:21 | |
*** xiaohua has quit IRC | 09:36 | |
*** xiaohua has joined #puppet-openstack | 09:37 | |
*** mkarpin has quit IRC | 09:40 | |
*** yottatsa has quit IRC | 09:43 | |
*** xiaohua has quit IRC | 09:44 | |
*** gfidente has quit IRC | 09:45 | |
*** gfidente has joined #puppet-openstack | 09:47 | |
*** gfidente has quit IRC | 09:47 | |
*** gfidente has joined #puppet-openstack | 09:47 | |
*** yottatsa has joined #puppet-openstack | 09:48 | |
*** igajsin1 has left #puppet-openstack | 09:53 | |
*** mkarpin has joined #puppet-openstack | 09:55 | |
*** fwdit has joined #puppet-openstack | 09: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 exists | 10:00 |
tore_ | I think it should have been using /etc/neutron/plugins/ml2/linuxbridge_agent.ini instead | 10:00 |
*** iberezovskiy_ is now known as iberezovskiy | 10: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 |
yottatsa | please review/merge https://review.openstack.org/#/c/302699 Pass correct security groups driver for ML2 plugin | 10:04 |
*** leanderthal|afk is now known as leanderthal | 10:09 | |
*** beddari has quit IRC | 10:12 | |
*** _degorenko|afk is now known as degorenko | 10:16 | |
*** beddari has joined #puppet-openstack | 10:19 | |
*** markvoelker has joined #puppet-openstack | 10:32 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Remove individual service certificates https://review.openstack.org/301493 | 10:35 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer endpoints https://review.openstack.org/302574 | 10:35 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add support for internal/admin endpoint TLS in HAProxy https://review.openstack.org/303342 | 10:35 |
*** markvoelker has quit IRC | 10:39 | |
*** ducttape_ has joined #puppet-openstack | 10:39 | |
*** ducttape_ has quit IRC | 10:43 | |
*** xiaohua has joined #puppet-openstack | 10:43 | |
*** skolekonov has joined #puppet-openstack | 10:46 | |
openstackgerrit | Dmitry Burmistrov proposed openstack/puppet-oslo: Refactor array values processing https://review.openstack.org/302271 | 10:47 |
*** mgould has quit IRC | 10:50 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer endpoints https://review.openstack.org/302574 | 10:51 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add support for internal/admin endpoint TLS in HAProxy https://review.openstack.org/303342 | 10:51 |
*** xiaohua has quit IRC | 10:51 | |
*** mgould has joined #puppet-openstack | 11:03 | |
*** saneax_AFK is now known as saneax | 11:17 | |
*** saneax is now known as saneax_AFK | 11:18 | |
*** saneax_AFK is now known as saneax | 11:18 | |
*** saneax is now known as saneax_AFK | 11:30 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add support for internal/admin endpoint TLS in HAProxy https://review.openstack.org/303342 | 11:34 |
*** chem has joined #puppet-openstack | 11:36 | |
*** moravec has joined #puppet-openstack | 11:41 | |
*** moravec has quit IRC | 11:45 | |
*** dprince has joined #puppet-openstack | 11:47 | |
*** cdelatte has joined #puppet-openstack | 11:48 | |
*** xiaohua has joined #puppet-openstack | 11:49 | |
*** ayoung has quit IRC | 11:54 | |
*** johnmilton has joined #puppet-openstack | 11:54 | |
*** moravec has joined #puppet-openstack | 11:54 | |
*** xiaohua has quit IRC | 11:56 | |
*** derekh has quit IRC | 11:58 | |
*** xiaohua has joined #puppet-openstack | 12:02 | |
*** yottatsa has quit IRC | 12:03 | |
*** yottatsa has joined #puppet-openstack | 12:04 | |
openstackgerrit | Matthew J Black proposed openstack/puppet-neutron: WIP lbaas v2 service plugin support. https://review.openstack.org/301797 | 12:08 |
openstackgerrit | Xiaohua Yuan proposed openstack/puppet-trove: Add related parameters to profiler section https://review.openstack.org/303187 | 12:15 |
*** jayg|g0n3 is now known as jayg | 12:20 | |
EmilienM | tore_: what version of OpenStack do you install? | 12:21 |
*** markvoelker has joined #puppet-openstack | 12:21 | |
*** ducttape_ has joined #puppet-openstack | 12:25 | |
EmilienM | dgurtner, xarses: fyi: https://review.openstack.org/#/c/303086/ | 12:26 |
Kvisle | EmilienM: it's Liberty | 12:26 |
Kvisle | (tore_ is a coworker of mine) | 12:26 |
Kvisle | but he just headed out | 12:26 |
EmilienM | Kvisle: do you use master or puppet-neutron or stable/liberty? | 12:26 |
Kvisle | stable/liberty | 12:27 |
EmilienM | I'm aware we had some issues some weeks ago with linuxbridge | 12:27 |
EmilienM | we fixed it in stable/mitaka and master | 12:27 |
Kvisle | waiting for mitaka packages for el7 | 12:27 |
EmilienM | we're even testing it now in our integration testing | 12:27 |
*** xiaohua has quit IRC | 12:29 | |
*** trown|outtypewww is now known as trown | 12:29 | |
*** fwdit has quit IRC | 12:35 | |
openstackgerrit | Dmitry Burmistrov proposed openstack/puppet-keystone: Set oslo options in keystone module through puppet-oslo https://review.openstack.org/302738 | 12:37 |
*** ducttape_ has quit IRC | 12:38 | |
*** jpena is now known as jpena|lunch | 12:41 | |
*** derekh has joined #puppet-openstack | 12:43 | |
dmburmistrov | EmilienM, 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|lunch | 12:57 | |
EmilienM | dmburmistrov: sure | 12:58 |
openstackgerrit | Matthew J Black proposed openstack/puppet-horizon: Added ensure_resource for python-memcache package. https://review.openstack.org/298679 | 12:58 |
dmburmistrov | thanks! | 12:59 |
*** Matum has joined #puppet-openstack | 13:02 | |
*** dgurtner has quit IRC | 13:20 | |
*** ducttape_ has joined #puppet-openstack | 13:23 | |
mjblack | EmilienM: the beaker tests, what release of openstack are they using? | 13:26 |
*** richm has joined #puppet-openstack | 13:26 | |
mjblack | the neutron change I made is failing because its not able to find the package on ubuntu | 13:27 |
EmilienM | mjblack: mitaka on stable/mitaka and master | 13:27 |
EmilienM | mjblack: it's in your logs btw | 13:27 |
mjblack | hmmm I'm at a loss of why its not finding the packages | 13:27 |
EmilienM | mjblack: in logs/dpkg-l.txt.gz | 13:28 |
mjblack | that'll only tell me what is installed | 13:28 |
mjblack | the apt repo doesnt contain the neutron-lbaasv2-agent | 13:29 |
EmilienM | mjblack: did you talk with ubuntu folks? | 13:30 |
EmilienM | mjblack: /join #openstack-dev && ping coreycb jamespage | 13:30 |
mjblack | EmilienM: the thing is I'm using the mitaka apt and I'm able to successfully search and find it | 13:32 |
mjblack | even after apt-get upgrade | 13:32 |
Hunner | EmilienM: Whoops, I was thinking that mongodb.conf.2.6.erb would be used for mongo 2 and mongodb.conf.erb for 3 | 13:34 |
*** dgurtner has joined #puppet-openstack | 13:34 | |
EmilienM | Hunner: something is weird, 2.6 should support bindIp | 13:34 |
EmilienM | Hunner: ie: the PR should be OK | 13:34 |
EmilienM | but it's not, I'm trying to reproduce | 13:34 |
mjblack | EmilienM: 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 amoralej | 13:36 | |
EmilienM | Hunner: don't merge the revert now, I'm trying to reproduce the bug and maybe fix it | 13:40 |
EmilienM | mjblack: show me logs... | 13:40 |
Hunner | k | 13:40 |
*** jpena|lunch is now known as jpena | 13:42 | |
EmilienM | mfisch: I'm trying to make the puppet schedule (summit) according to your talks | 13:43 |
EmilienM | it's very challenging | 13:43 |
*** dgurtner has quit IRC | 13:46 | |
*** dgurtner has joined #puppet-openstack | 13:48 | |
*** paramite is now known as paramite|afk | 13:51 | |
*** skolekonov_ has joined #puppet-openstack | 13:54 | |
*** skolekonov has quit IRC | 13:55 | |
*** ayoung has joined #puppet-openstack | 13:57 | |
*** saneax_AFK is now known as saneax | 14:02 | |
jaosorior | o/ | 14:03 |
*** shardy has joined #puppet-openstack | 14:03 | |
EmilienM | jaosorior, Hunner: I think I found an interesting bug in puppetlabs-mongodb | 14:03 |
EmilienM | https://github.com/puppetlabs/puppetlabs-mongodb/blob/master/manifests/server/config.pp#L65 | 14:03 |
EmilienM | mongodb::server::version does not exist, it's mongodb::globals::version | 14:03 |
*** skolekonov has joined #puppet-openstack | 14:03 | |
EmilienM | which means, version parameter is never addressed | 14:04 |
EmilienM | and we always deploy the new config file; 2.6 file is never deployed | 14:04 |
*** ayoung has quit IRC | 14:04 | |
*** myatsenko has quit IRC | 14:04 | |
*** psanchez has joined #puppet-openstack | 14:04 | |
*** xarses has quit IRC | 14:04 | |
EmilienM | except if we set mongo::server::config::version with Hiera | 14:05 |
EmilienM | but that's odd | 14:05 |
*** skolekonov_ has quit IRC | 14:06 | |
jaosorior | EmilienM: 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#L104 | 14:06 |
*** ToMiles has joined #puppet-openstack | 14:07 | |
jaosorior | it will use mongodb/mongodb.conf.2.6.erb for 2.6.0 and up, as I read that | 14:07 |
*** ToMiles has quit IRC | 14:07 | |
EmilienM | jaosorior: no | 14:07 |
EmilienM | jaosorior: elsif $version and (versioncmp($version, '2.6.0') >= 0) | 14:07 |
EmilienM | $version is undef by default | 14:07 |
EmilienM | see in mongodb::globals | 14:08 |
EmilienM | which means we don't go in this conditional | 14:08 |
EmilienM | we go in else for newer versions | 14:08 |
jaosorior | EmilienM: oh damn, I see | 14:08 |
EmilienM | and for tripleo too, we need to 1) set mongo::globals::version 2) fix $version usage | 14:08 |
EmilienM | we have 2 patches one on tripleo and one in puppetlabs mongodb | 14:09 |
EmilienM | I'm working on it | 14:09 |
EmilienM | our puppet openstack CI did not catch it because we are pin to latest tag | 14:09 |
*** skolekonov_ has joined #puppet-openstack | 14:14 | |
*** paramite|afk is now known as paramite | 14:15 | |
*** skolekonov has quit IRC | 14:16 | |
*** skolekonov_ has quit IRC | 14:19 | |
iberezovskiy | EmilienM, hey. Is it possible to recheck only tripleo jobs? I think I saw your comment like 'recheck tripleo' | 14:19 |
EmilienM | iberezovskiy: no it's not | 14:19 |
EmilienM | I should stop adding words after recheck, it's confusing | 14:19 |
iberezovskiy | k, got it :) | 14:19 |
*** jaosorior has quit IRC | 14:20 | |
iberezovskiy | Is tripleo ready for oslo module usage? | 14:20 |
EmilienM | iberezovskiy: yes | 14:20 |
EmilienM | iberezovskiy: and fuel? | 14:20 |
iberezovskiy | cool. fuel was update yesterday | 14:20 |
iberezovskiy | updated* | 14:21 |
mwhahaha | i put a request in to get fuel-ci on puppet-oslo | 14:21 |
iberezovskiy | great | 14:22 |
openstackgerrit | Merged openstack/puppet-oslo: Refactor array values processing https://review.openstack.org/302271 | 14:24 |
*** pma has quit IRC | 14:28 | |
EmilienM | mwhahaha, iberezovskiy: before merging any patch for using puppet-oslo, I want to see both fuel CI & tripleo green | 14:29 |
iberezovskiy | sure | 14:29 |
iberezovskiy | it's rule number one I guess :) | 14:29 |
EmilienM | yeah! | 14:29 |
*** dgurtner has quit IRC | 14:29 | |
*** ducttape_ has quit IRC | 14:31 | |
openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-nova: Configure oslo related sections using puppet-oslo module https://review.openstack.org/302839 | 14:33 |
*** baker has joined #puppet-openstack | 14:34 | |
*** trown is now known as trown|relocating | 14:37 | |
*** karimb has joined #puppet-openstack | 14:38 | |
*** haplo37 has joined #puppet-openstack | 14:39 | |
*** karimb has quit IRC | 14:40 | |
*** karimb has joined #puppet-openstack | 14:41 | |
*** ducttape_ has joined #puppet-openstack | 14:44 | |
*** xarses has joined #puppet-openstack | 14:48 | |
*** foxhunt72 is now known as foxhunt_away | 14:57 | |
openstackgerrit | Dmitry Burmistrov proposed openstack/puppet-keystone: Set oslo options in keystone module through puppet-oslo https://review.openstack.org/302738 | 15:01 |
*** mdorman has joined #puppet-openstack | 15:06 | |
mjblack | EmilienM: 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 |
EmilienM | mjblack: well, I would like to see specific failure | 15:07 |
*** baker has quit IRC | 15:07 | |
EmilienM | like 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_962 | 15:09 |
mjblack | EmilienM: line #29903 in http://logs.openstack.org/97/301797/6/check/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/16c694a/console.html | 15:09 |
EmilienM | mjblack: you have links on the left... | 15:09 |
EmilienM | mjblack: what is your patch? | 15:09 |
mjblack | EmilienM: https://review.openstack.org/#/c/301797/ | 15:10 |
EmilienM | mjblack: syntax is wrong | 15:10 |
EmilienM | and lint is telling you | 15:10 |
EmilienM | https://review.openstack.org/#/c/301797/6/manifests/agents/lbaas.pp | 15:10 |
EmilienM | L111 | 15:11 |
mjblack | EmilienM: yeah it was dinging me on double quotes instead of single quotes | 15:12 |
openstackgerrit | Matthew J Black proposed openstack/puppet-neutron: WIP lbaas v2 service plugin support. https://review.openstack.org/301797 | 15:13 |
mjblack | already had the change for that queued, just wanted to figure out the apt issue | 15:13 |
EmilienM | mjblack: there is no apt issue | 15:14 |
EmilienM | mjblack: apt was trying to install a package with a wrong name | 15:14 |
EmilienM | with { and such | 15:14 |
*** imcsk8 is now known as imcsk8|afk | 15:15 | |
*** ayoung has joined #puppet-openstack | 15:16 | |
*** alee has joined #puppet-openstack | 15:16 | |
EmilienM | degorenko: ping about https://etherpad.openstack.org/p/newton-design-puppet | 15:19 |
EmilienM | degorenko: do we really need a session about puppet-oslo? | 15:19 |
degorenko | EmilienM, well, work is already started, we can remove it now, i guess | 15:20 |
EmilienM | yeah | 15:20 |
*** ducttape_ has quit IRC | 15:21 | |
*** baker has joined #puppet-openstack | 15:23 | |
*** mkarpin has left #puppet-openstack | 15:24 | |
*** leanderthal is now known as leanderthal|afk | 15:24 | |
*** jayg is now known as jayg|g0n3 | 15:25 | |
*** trown|relocating is now known as trown | 15:31 | |
*** jmikolajczak has quit IRC | 15:32 | |
*** rmart04 has quit IRC | 15:34 | |
*** saneax is now known as saneax_AFK | 15:37 | |
openstackgerrit | Matthew J Black proposed openstack/puppet-horizon: Added ensure_resource for python-memcache package. https://review.openstack.org/298679 | 15:40 |
*** jayg|g0n3 is now known as jayg | 15:40 | |
*** ducttape_ has joined #puppet-openstack | 15:42 | |
mwhahaha | mgould: do you mind if i propose tests for that patch for you? | 15:43 |
mwhahaha | i don't want to step on your toes or anything :) | 15:43 |
EmilienM | mwhahaha: when are fuel sessions at summit? | 15:44 |
EmilienM | mwhahaha: how does it overlap with puppet? | 15:44 |
mwhahaha | let me dig up the schedule we're working on | 15:44 |
EmilienM | mfisch or clayton : can you attend puppet sessions on Wednesday? I'm trying to find a slot to talk about docker | 15:45 |
mwhahaha | https://etherpad.openstack.org/p/fuel-newton-summit-planning | 15:45 |
clayton | likely, let me see when my talks are | 15:45 |
mgould | mwhahaha, please do! | 15:45 |
EmilienM | omg it overlaps a lot | 15:45 |
mwhahaha | well not all of it is puppet related | 15:45 |
clayton | I have a talk wed before lunch, but I should be able to make it work other than that | 15:46 |
mwhahaha | we've got the fuel/puppet integration thing on thursday @9:50 | 15:46 |
EmilienM | clayton: ok | 15:46 |
clayton | to be honest, I really need to start looking at the schedule soon, I don't know what I'm doing when yet | 15:46 |
mwhahaha | mgould: ok i'll get something together here in a minute | 15:46 |
mgould | mwhahaha, thanks! | 15:46 |
openstackgerrit | Merged openstack/puppet-openstack-integration: Implement a script to generate Puppetfile https://review.openstack.org/300696 | 15:47 |
*** paramite has quit IRC | 15:47 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-oslo: Add oslo.middleware defines https://review.openstack.org/274422 | 15:48 |
*** saneax_AFK is now known as saneax | 15:49 | |
mjblack | EmilienM: the reference for stdlib for ensure_packages has it as a hash | 15:50 |
*** dtantsur is now known as dtantsur|afk | 15:50 | |
openstackgerrit | Merged openstack/puppet-openstack-integration: scenario001/centos7: deploy Ceph on IPv6 https://review.openstack.org/303086 | 15:51 |
mjblack | but that explains why I missed it | 15:51 |
*** fedexo has joined #puppet-openstack | 15:51 | |
mjblack | :D | 15:51 |
*** ducttape_ has quit IRC | 15:53 | |
*** bnemec has quit IRC | 15:56 | |
*** astupnikov has quit IRC | 16:00 | |
*** aimon has joined #puppet-openstack | 16:04 | |
*** bnemec has joined #puppet-openstack | 16:04 | |
openstackgerrit | Miles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot https://review.openstack.org/302390 | 16:08 |
openstackgerrit | Alex Schultz proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI https://review.openstack.org/302376 | 16:09 |
mwhahaha | mgould: -^ tests | 16:09 |
mgould | mwhahaha, thanks! | 16:10 |
mgould | contain_exec looks like just the thing | 16:13 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: bump RDO to latest Mitaka https://review.openstack.org/303522 | 16:15 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: bump RDO to latest Mitaka https://review.openstack.org/303522 | 16:16 |
*** imcsk8|afk is now known as imcsk8 | 16:19 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: neutron: use ipv6 network for tunneling when needed https://review.openstack.org/303527 | 16:24 |
openstackgerrit | Miles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI https://review.openstack.org/302376 | 16:25 |
*** _ducttape_ has joined #puppet-openstack | 16:26 | |
mgould | mwhahaha, fixed a couple of typos ^^^ | 16:26 |
openstackgerrit | Dmitry Burmistrov proposed openstack/puppet-heat: Set oslo options in heat module through puppet-oslo https://review.openstack.org/303528 | 16:26 |
EmilienM | mwhahaha: jpena found something in puppet-glance | 16:27 |
EmilienM | with the patch we merged yesterday https://github.com/openstack/puppet-glance/commit/d0957fb622a0f5f4f6e038aa22d87ee61a2f1ad2#diff-b74717103ca8cd0432d6d26461aaca64L105 | 16:27 |
*** karimb has quit IRC | 16:27 | |
EmilienM | it seems that list = request('image', 'list', '--long') worked fine | 16:28 |
EmilienM | and list = request('image', 'list') is broken (for packstack) | 16:28 |
EmilienM | but I noticed that list is not working anymore good | 16:28 |
EmilienM | we talked about it yesterday | 16:28 |
jpena | EmilienM mwhahaha I think I know why | 16:31 |
jpena | it's in https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack.rb#L74 | 16:31 |
jpena | we do response = openstack(service, action, '--quiet', '--format', 'csv', properties) | 16:31 |
jpena | so service=='image', action=='list', and properties =='--long' | 16:32 |
jpena | if we remove --long, properties == '' | 16:32 |
jpena | and that's what is causing openstackclient to complain | 16:32 |
EmilienM | oh right | 16:32 |
openstackgerrit | Miles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot https://review.openstack.org/302390 | 16:33 |
*** derekh has quit IRC | 16:33 | |
EmilienM | jpena: nice catch, I confirm the bug | 16:34 |
dmsimard | jpena to the rescue \o/ | 16:34 |
EmilienM | as usual ... | 16:34 |
dmsimard | RIGHT IN THE MIDDLE OF A RELEASE | 16:34 |
jpena | so, should this be fixed in puppet-openstacklib or puppet-glance? Can properties be considered an optional argument? | 16:35 |
dmsimard | jpena: 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 passing | 16:36 |
dmsimard | but OPM stable/mitaka could not carry that commit (yet) checking | 16:36 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: swift: use ipv6 binding on centos7 nodes https://review.openstack.org/303530 | 16:36 |
EmilienM | jpena: mwhahaha did that recently https://github.com/openstack/puppet-glance/commit/d0957fb622a0f5f4f6e038aa22d87ee61a2f1ad2 | 16:37 |
EmilienM | jpena: in puppet-glance I think | 16:39 |
jpena | EmilienM: ok, so as a quick fix I'm adding back --long to the call | 16:39 |
*** imcsk8 is now known as imcsk8|afk | 16:40 | |
dmsimard | How did integration jobs and beaker jobs pass though ? | 16:40 |
dmsimard | If that was very broken ? | 16:40 |
EmilienM | jpena: we have functional tests that will break if your patch is not valid anyway, so go ahead | 16:40 |
EmilienM | dmsimard: 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 fail | 16:41 |
dmsimard | the image list is called, amongst other things, when creating an image (i.e tempest pre-provisioning) | 16:42 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: bump RDO to latest Mitaka https://review.openstack.org/303522 | 16:42 |
dmsimard | so it should have failed | 16:42 |
EmilienM | dmsimard: 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_643 | 16:42 |
EmilienM | yes, it should have failed | 16:42 |
EmilienM | jpena: is it what you see? ^ | 16:43 |
openstackgerrit | Javier Peña proposed openstack/puppet-glance: Fix image list in glance_image provider https://review.openstack.org/303533 | 16:44 |
jpena | EmilienM: exactly that | 16:44 |
EmilienM | jpena: thanks | 16:45 |
dmsimard | jpena: you probably need to update spec tests too | 16:46 |
jpena | dmsimard: mmm, didn't think of that | 16:46 |
jpena | I'll check it | 16:46 |
EmilienM | why does it need backport? patch from mwhahaha merged yesterday or so | 16:46 |
dmsimard | jpena: i.e, https://github.com/openstack/puppet-glance/commit/d0957fb622a0f5f4f6e038aa22d87ee61a2f1ad2#diff-b74717103ca8cd0432d6d26461aaca64R96 | 16:46 |
openstackgerrit | Matthew J Black proposed openstack/puppet-neutron: WIP lbaas v2 service plugin support. https://review.openstack.org/301797 | 16:46 |
dmsimard | jpena: wait, no, wrong link | 16:47 |
jpena | EmilienM: https://review.openstack.org/#/c/301706/ | 16:47 |
*** rmart04 has joined #puppet-openstack | 16:47 | |
EmilienM | jpena: oh right :-) | 16:47 |
*** rmart04 has quit IRC | 16:48 | |
jpena | I need to fix the commit msg, btw | 16:48 |
mwhahaha | so it was the space | 16:48 |
mwhahaha | we can fix it but we should also create a bug to openstack client, it shouldn't puke on trailing spaces | 16:48 |
*** _ducttape_ has quit IRC | 16:49 | |
dmsimard | mwhahaha: well technically '' is not a trailing space | 16:52 |
mwhahaha | so it's null? | 16:52 |
dmsimard | mwhahaha: http://paste.openstack.org/show/493514/ | 16:53 |
dmsimard | :P | 16:53 |
mwhahaha | what teh | 16:54 |
dmsimard | it can be a literal character or a string delimiter | 16:54 |
*** rcernin has quit IRC | 16:55 | |
mwhahaha | by default that last param is [] i think, we really should fix the providers because we shouldn't have to specify --long | 16:57 |
mwhahaha | jpena: it's going to fail on the tests, do you want me to update them for you? | 16:57 |
jpena | mwahaha: I'm fixing them, thx | 16:58 |
mwhahaha | k | 16:58 |
jpena | oops, I missed one 'ha' :o) | 17:02 |
openstackgerrit | Javier Peña proposed openstack/puppet-glance: Fix image list in glance_image provider https://review.openstack.org/303533 | 17:03 |
mjblack | EmilienM: with regards to your comment about the neutron_lbaas_service_config, it's suppose to configure the service_provider on neutron server | 17:03 |
mjblack | neither ubuntu or redhat are doing --config-file /etc/neutron/services_lbass.conf in the service scripts | 17:04 |
EmilienM | mjblack: ok, I'm just asking because martin this the original work | 17:04 |
mjblack | sure np | 17:04 |
mjblack | lbaas v2 has two components, the services (neutron-server) and agent | 17:04 |
mjblack | I'm actually thinking we can consolidate neutron_*_services_config into one type/provider | 17:05 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-cinder: Implement Cinder API WSGI support https://review.openstack.org/287964 | 17:11 |
*** brandon has joined #puppet-openstack | 17:12 | |
*** xarses_ has joined #puppet-openstack | 17:12 | |
*** xarses has quit IRC | 17:13 | |
EmilienM | mjblack: http://logs.openstack.org/91/288091/7/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/2eef0ca/logs/apache/cinder_wsgi_error.txt.gz | 17:13 |
*** brandon has left #puppet-openstack | 17:14 | |
EmilienM | IOError: [Errno 13] Permission denied: '/var/log/cinder/osapi_volume.log' | 17:14 |
EmilienM | we need to change something | 17:14 |
* mgould -> home; good night! | 17:15 | |
*** mgould has quit IRC | 17:15 | |
*** Matum has quit IRC | 17:17 | |
*** jpena is now known as jpena|off | 17:19 | |
*** shardy has quit IRC | 17:20 | |
*** aimon has quit IRC | 17:21 | |
*** aimon has joined #puppet-openstack | 17:22 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-cinder: Implement Cinder API WSGI support https://review.openstack.org/287964 | 17:22 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: deploy cinder api in WSGI with Apache https://review.openstack.org/288091 | 17:23 |
EmilienM | ConfigFileValueError: Value for option local_ip is not valid: ::1 is not an IPv4 address | 17:23 |
EmilienM | I'm trying to deploy neutron with ipv6 tunnels | 17:24 |
*** gfidente has quit IRC | 17:24 | |
*** jistr has quit IRC | 17:25 | |
*** baker has quit IRC | 17:26 | |
*** fedexo has quit IRC | 17:26 | |
*** xarses_ is now known as xarses | 17:33 | |
*** degorenko is now known as _degorenko|afk | 17:34 | |
*** baker has joined #puppet-openstack | 17:35 | |
*** fawadkhaliq has joined #puppet-openstack | 17:39 | |
*** baker has quit IRC | 17:40 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: swift: use ipv6 binding on centos7 nodes https://review.openstack.org/303530 | 17:47 |
*** ducttape_ has joined #puppet-openstack | 17:47 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002: deploy & test Zaqar https://review.openstack.org/281515 | 17:49 |
EmilienM | mjblack: lbaas1 works fine: https://review.openstack.org/#/c/272297 | 17:51 |
EmilienM | mjblack: so the plan is to have lbaasv1 tests until we can have v2 with your work | 17:52 |
EmilienM | oh wait, tests didn't run | 17:53 |
*** fawadkhaliq has quit IRC | 18:03 | |
*** fawadkhaliq has joined #puppet-openstack | 18:03 | |
mjblack | EmilienM: getting my hopes up to destroy them :D | 18:08 |
*** baker has joined #puppet-openstack | 18:10 | |
openstackgerrit | Matthew J Black proposed openstack/puppet-neutron: WIP lbaas v2 service plugin support. https://review.openstack.org/301797 | 18:14 |
*** ayoung has quit IRC | 18:14 | |
EmilienM | mwhahaha: https://review.openstack.org/#/c/303533/ fails | 18:16 |
EmilienM | http://logs.openstack.org/33/303533/2/check/gate-puppet-glance-puppet-beaker-rspec-centos-7/fbae8f2/console.html#_2016-04-08_17_24_19_753 | 18:16 |
EmilienM | (Keystone_endpoint[RegionOne/Image Service::image] => Service[glance-api] => Keystone_endpoint[RegionOne/Image Service::image]) | 18:16 |
mwhahaha | well that's odd | 18:17 |
EmilienM | I'll dig after my lunch, but feel free to look :-P | 18:17 |
*** ducttape_ has quit IRC | 18:18 | |
mwhahaha | oh yea don't we test for a cycle in glance? | 18:18 |
mwhahaha | we should remove that test | 18:18 |
*** foxhunt_away is now known as foxhunt72 | 18:22 | |
mjblack | EmilienM: will that integration test ever go green if you're making it test the cinder wsgi change? | 18:26 |
openstackgerrit | Matthew J Black proposed openstack/puppet-neutron: WIP lbaas v2 service plugin support. https://review.openstack.org/301797 | 18:30 |
openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added keystone config to barbican api manifest https://review.openstack.org/263461 | 18:48 |
openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module https://review.openstack.org/263462 | 18:48 |
*** dprince has quit IRC | 18:56 | |
* EmilienM back | 19:02 | |
EmilienM | mjblack: it should yes | 19:02 |
*** ayoung has joined #puppet-openstack | 19:09 | |
EmilienM | mwhahaha: re puppet-glance, why should we drop the test? | 19:15 |
*** imcsk8|afk is now known as imcsk8 | 19:17 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: list tempest plugins at the end of the job https://review.openstack.org/303587 | 19:18 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: deploy & test neutron lbaasv1 service https://review.openstack.org/272297 | 19:19 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: list tempest plugins at the end of the job https://review.openstack.org/303587 | 19:21 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: deploy & test neutron lbaasv1 service https://review.openstack.org/272297 | 19:21 |
EmilienM | mwhahaha: do we drop them? | 19:22 |
EmilienM | I've looked at the code and I don't see why we have this now | 19:22 |
mwhahaha | yea | 19:23 |
mwhahaha | and they just make it harder to figure out failures | 19:23 |
EmilienM | I did recheck, to see if it's consistent | 19:23 |
EmilienM | ok | 19:23 |
mwhahaha | cause we see those and assume it's that,w hen that's actually a test :D | 19:23 |
*** foxhunt72 is now known as foxhunt_away | 19:25 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: acceptance: drop dependency cycles tests https://review.openstack.org/303593 | 19:26 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: Fix image list in glance_image provider https://review.openstack.org/303533 | 19:27 |
EmilienM | mwhahaha: I confirm jpena's patch works fine | 19:32 |
EmilienM | we don't have the error in osclient with glance | 19:32 |
EmilienM | you can double check in http://logs.openstack.org/33/303533/2/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/5ef916f/console.html | 19:32 |
*** karimb has joined #puppet-openstack | 19:34 | |
EmilienM | dmsimard: can you review again https://review.openstack.org/#/c/303522/ please? | 19:38 |
mwhahaha | sure | 19:39 |
mwhahaha | we should fix the provider not to require the extra options | 19:39 |
mwhahaha | or at least merge it with the options we are hard coding in the openstacklib provider | 19:40 |
openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added keystone config to barbican api manifest https://review.openstack.org/263461 | 19:41 |
openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module https://review.openstack.org/263462 | 19:41 |
EmilienM | mwhahaha: in the same patch? | 19:43 |
mwhahaha | well it would fix that | 19:44 |
mwhahaha | but no we can do it later | 19:44 |
mwhahaha | just an observation | 19:44 |
mwhahaha | i could see this issue coming up again next time someone adds an new openstack client thing with no additional options | 19:45 |
EmilienM | mwhahaha: oh you mean fix it in openstack.rb provider (openstacklib) ? | 19:45 |
mwhahaha | https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack.rb#L64 | 19:45 |
mwhahaha | yea | 19:45 |
EmilienM | +1 | 19:45 |
mwhahaha | properties is techincally a required param | 19:46 |
mwhahaha | hey look, fuel-ci on puppet-oslo https://review.openstack.org/#/c/274422/ | 19:47 |
mwhahaha | :) | 19:47 |
*** amoralej has left #puppet-openstack | 19:48 | |
EmilienM | mwhahaha: excellent new | 19:50 |
EmilienM | I'll review it shortly | 19:50 |
EmilienM | mwhahaha: I want to see it in puppet-nova :P | 19:50 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002: deploy & test Zaqar https://review.openstack.org/281515 | 19:52 |
dmsimard | EmilienM: review what again, https://review.openstack.org/#/c/303522/ there hasn't been a new patchset | 19:52 |
EmilienM | dmsimard: I'm asking you to -1/+1 just | 19:54 |
EmilienM | dmsimard: it's on master | 19:54 |
EmilienM | I thought it was good after our convo on #rdo | 19:54 |
dmsimard | right | 19:54 |
*** bodepd has quit IRC | 19:57 | |
*** zeus has quit IRC | 19:58 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: swift: use ipv6 binding on centos7 nodes https://review.openstack.org/303530 | 19:58 |
*** bookwar has quit IRC | 19:58 | |
*** zeus has joined #puppet-openstack | 20:03 | |
*** zeus is now known as Guest80019 | 20:03 | |
openstackgerrit | Dmitry Ilyin proposed openstack/puppet-ec2api: Initial commit for puppet-ec2api https://review.openstack.org/276103 | 20:04 |
*** johnmilton has quit IRC | 20:06 | |
*** bodepd has joined #puppet-openstack | 20:06 | |
*** bookwar has joined #puppet-openstack | 20:07 | |
alee | EmilienM, _degorenko|afk -- finally ! https://review.openstack.org/#/c/263461 | 20:09 |
*** fawadkhaliq has quit IRC | 20:11 | |
*** fawadkhaliq has joined #puppet-openstack | 20:11 | |
mwhahaha | i 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 version | 20:12 |
mwhahaha | seems like we might have a lot of duplication in those | 20:12 |
*** karimb has quit IRC | 20:17 | |
*** ducttape_ has joined #puppet-openstack | 20:18 | |
*** ducttape_ has quit IRC | 20:23 | |
*** dhollinger has joined #puppet-openstack | 20:28 | |
beagles | EmilienM, 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#L62 | 20:30 |
beagles | complaining that it couldn't find the resource ... | 20:31 |
EmilienM | beagles: can we take that on #tripleo maybe? | 20:31 |
beagles | yup :0 | 20:31 |
*** jayg is now known as jayg|g0n3 | 20:32 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 20:35 |
EmilienM | any core around please look https://review.openstack.org/#/c/303593/ | 20:37 |
EmilienM | and the most important: https://review.openstack.org/#/c/303533/ | 20:37 |
dmsimard | EmilienM: cool for https://review.openstack.org/#/c/303533/ so can we backport it once it merges ? | 21:00 |
EmilienM | dmsimard: of course | 21:00 |
dmsimard | ok | 21:00 |
EmilienM | dmsimard: submit it and I"ll +2 | 21:00 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-glance: Fix image list in glance_image provider https://review.openstack.org/303613 | 21:01 |
EmilienM | dmsimard: you missed cherry-pick -x | 21:01 |
dmsimard | I cherry-picked from gerrit :/ | 21:01 |
EmilienM | you used Gerrit UI | 21:01 |
dmsimard | I'll fix | 21:01 |
EmilienM | and the patch is not merged in master | 21:01 |
dmsimard | oh, right | 21:01 |
openstackgerrit | David Moreau Simard proposed openstack/puppet-glance: Fix image list in glance_image provider https://review.openstack.org/303613 | 21:04 |
EmilienM | dmsimard: I'm afraid it might fail, maybe we also need to backport https://review.openstack.org/#/c/303593/ | 21:04 |
EmilienM | but let's see how it goes | 21:05 |
dmsimard | ok | 21:05 |
*** trown is now known as trown|outtypewww | 21:05 | |
*** cdelatte has quit IRC | 21:07 | |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Remove user/role prefetch to support multi-domain. https://review.openstack.org/299301 | 21:13 |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: WIP: multi-domain testing with ldap backend. https://review.openstack.org/296370 | 21:14 |
EmilienM | alee: reviewed | 21:18 |
alee | EmilienM, thanks | 21:19 |
EmilienM | alee: -1 on both, please look | 21:19 |
alee | EmilienM, 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 |
openstackgerrit | Merged openstack/puppet-glance: acceptance: drop dependency cycles tests https://review.openstack.org/303593 | 21:21 |
EmilienM | alee: the keystone thing | 21:21 |
openstackgerrit | Merged openstack/puppet-glance: Fix image list in glance_image provider https://review.openstack.org/303533 | 21:21 |
EmilienM | do not create a new acceptance manifest | 21:21 |
*** karimb has joined #puppet-openstack | 21:23 | |
*** haplo37 has quit IRC | 21:24 | |
alee | EmilienM, 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 |
alee | EmilienM, I'm trying to test both scenarios -- one with keystone auth and one without | 21:25 |
EmilienM | alee: yeah, use one scenario | 21:26 |
EmilienM | it adds complexity | 21:26 |
EmilienM | for a first iteration, not sure we want it | 21:26 |
alee | EmilienM, 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 |
EmilienM | alee: please move it to basic | 21:28 |
EmilienM | later, in wsgi, we'll drop basic and use with_apache | 21:28 |
EmilienM | see second review | 21:28 |
*** mdorman has quit IRC | 21:30 | |
alee | EmilienM, seems a bit of manipulation to fix up basic to include keystone -- and then drop it entirely in the next patch .. | 21:30 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002: deploy & test Zaqar https://review.openstack.org/281515 | 21:31 |
EmilienM | alee: ok, you can keep it like this | 21:32 |
EmilienM | alee: please just drop https://review.openstack.org/#/c/263462/20/spec/acceptance/basic_barbican_spec.rb | 21:32 |
EmilienM | alee: and update https://review.openstack.org/#/c/263462/20/spec/acceptance/keystone_barbican_spec.rb to deploy apache | 21:33 |
EmilienM | alee: I want to use apache by default at maximum | 21:33 |
alee | EmilienM, ok | 21:33 |
*** twm2016 has joined #puppet-openstack | 21:34 | |
twm2016 | tristanC: Hey, I just assigned myself to a bug you reported! | 21:34 |
EmilienM | alee: still -1 on https://review.openstack.org/#/c/263461/ | 21:35 |
twm2016 | https://bugs.launchpad.net/puppet-gnocchi/+bug/1543730 | 21:35 |
openstack | Launchpad 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 |
alee | EmilienM, ok -- actually by only having one test scenario, I dont need any of that cleanup code. | 21:37 |
EmilienM | alee: I just sent new comments. | 21:38 |
alee | EmilienM, yeah - I was referring to your comments in https://review.openstack.org/#/c/263461/23/spec/spec_helper_acceptance.rb | 21:39 |
alee | the cleanup code there is to make sure things don't get messed up from run to run | 21:39 |
EmilienM | alee: look https://github.com/openstack/puppet-openstack_spec_helper/blob/master/lib/puppet-openstack_spec_helper/beaker_spec_helper.rb | 21:39 |
EmilienM | we have a lib for that now | 21:39 |
alee | EmilienM, so -- the key bit I needed is this -- https://review.openstack.org/#/c/263461/23/spec/spec_helper_acceptance.rb lines 59-66 | 21:41 |
*** ducttape_ has joined #puppet-openstack | 21:42 | |
alee | EmilienM, 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 |
alee | like maybe deploying with dogtag/ipa as the backend | 21:42 |
alee | EmilienM, so in that case, given that we now have a lib. what would that file look like? | 21:43 |
EmilienM | alee: that's why I'm asking to have only one beaker file | 21:43 |
EmilienM | it makes things simpler | 21:43 |
EmilienM | please use basic file for now, and switch to with_apache on the wsgi patch | 21:44 |
EmilienM | we can iterate later | 21:44 |
alee | EmilienM, ok | 21:44 |
*** twm2016 has quit IRC | 21:48 | |
pabelanger | EmilienM: do you mind linking me that review again for puppet4 | 21:55 |
*** ducttape_ has quit IRC | 21:59 | |
*** karimb has quit IRC | 22:02 | |
EmilienM | pabelanger: https://review.openstack.org/296557 | 22:06 |
*** dhollinger has quit IRC | 22:10 | |
EmilienM | mjblack: 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 it | 22:12 |
EmilienM | mjblack: you can pick the haproxy backend probably | 22:12 |
pabelanger | EmilienM: thanks | 22:14 |
openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added keystone config to barbican api manifest https://review.openstack.org/263461 | 22:15 |
openstackgerrit | Ade Lee proposed openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module https://review.openstack.org/263462 | 22:15 |
alee | EmilienM, ^^ should have your requested changes | 22:15 |
*** markvoelker has quit IRC | 22:16 | |
EmilienM | alee: cool; afk now, will look later tonight | 22:18 |
alee | EmilienM, thanks! | 22:18 |
alee | mwhahaha, _degorenko|afk ya'll too please. thanks! | 22:19 |
mwhahaha | sure | 22:19 |
*** karimb has joined #puppet-openstack | 22:20 | |
*** fawadkhaliq has quit IRC | 22:25 | |
*** fawadkhaliq has joined #puppet-openstack | 22:25 | |
*** karimb has quit IRC | 22:27 | |
*** karimb has joined #puppet-openstack | 22:27 | |
*** baker has quit IRC | 22:29 | |
*** Guest80019 is now known as zeus | 22:34 | |
*** zeus has joined #puppet-openstack | 22:34 | |
pabelanger | EmilienM: okay, next step is to build facter manually and start debugging it with gdb | 22:38 |
openstackgerrit | Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4 https://review.openstack.org/296557 | 22:51 |
*** karimb has quit IRC | 22:53 | |
*** karimb has joined #puppet-openstack | 22:54 | |
*** dgurtner has joined #puppet-openstack | 22:58 | |
*** karimb has quit IRC | 23:01 | |
*** richm has quit IRC | 23:10 | |
*** markvoelker has joined #puppet-openstack | 23:16 | |
*** markvoelker has quit IRC | 23:21 | |
*** xarses has quit IRC | 23:39 | |
*** fawadkhaliq has quit IRC | 23:48 | |
*** fawadkhaliq has joined #puppet-openstack | 23:48 | |
*** markvoelker has joined #puppet-openstack | 23:52 | |
*** guessi has quit IRC | 23:56 | |
*** guessi has joined #puppet-openstack | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!