Wednesday, 2016-03-09

« Tuesday, 2016-03-08 Index Thursday, 2016-03-10 »
openstackgerritMerged openstack/puppet-glance: Add glance multi/single store declaration examples  https://review.openstack.org/29003300:00
_odyEmilienM: I've been following it and just noticed it is now +1 by CI.00:02
_odyI'll definitely find the time tomorrow.00:02
EmilienMyeah, we fixed all selinux things00:02
EmilienM_ody: awesome!00:02
EmilienM_ody: I'm looking forward to see the progress on puppet4 testing too00:03
_odyIf our cloud doesn't break again tomorrow I'll make more progress.00:03
openstackgerritMerged openstack/puppet-glance: Remove POSIX users, groups, and file modes.  https://review.openstack.org/21159300:03
openstackgerritMerged openstack/puppet-ceilometer: Add api_paste type/provider for Ceilometer  https://review.openstack.org/28535800:03
EmilienM_ody: good luck!00:03
_odySo we have a handful of changes with failing tests that keep coming back as failed...but I noticed something...00:04
_odyThey are all changes that are running old tests.00:04
_odyNew tests pass fine.00:04
*** chem has quit IRC00:04
_odySeems like a rebase might be the only thing that kicks them into current test suite definitions.00:05
_odyhttps://review.openstack.org/#/c/281499 for example doesn't list all the correct tests.00:05
_odyrebase will cancel everyone's reviews.00:05
EmilienMdmsimard: https://bugs.launchpad.net/puppet-neutron/+bug/1554808 -- I'll work on it ASAP00:06
openstackLaunchpad bug 1554808 in puppet-neutron "on single node deployment, we need to make sure puppet-neutron starts before agents" [Critical,New] - Assigned to Emilien Macchi (emilienm)00:06
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: fix filter_project param values  https://review.openstack.org/28149900:06
EmilienM_ody: I rebased it00:07
EmilienMit does not cancel +2s00:07
EmilienM_ody: the proof my patch on puppet-nova works: https://review.openstack.org/#/c/289562/00:07
_odyYou're right.  I so thought it did.00:07
* _ody goes rebasing things00:08
_odyEmilienM: Can't seem to get this to rebase since it already has a parent of master. https://review.openstack.org/#/c/197572/00:13
_odyI suspect an abandon and re-enable would do it.00:13
_odyThink I'll just mail the list.00:17
openstackgerritCody Herriges proposed openstack/puppet-nova: Add support for the libvirt/hw_disk_discard option  https://review.openstack.org/27905500:17
*** saneax is now known as saneax_AFK00:20
openstackgerritCody Herriges proposed openstack/puppet-cinder: module to set quotas per volume type  https://review.openstack.org/19757200:29
openstackgerritCody Herriges proposed openstack/puppet-cinder: module to set quotas per volume type  https://review.openstack.org/19757200:29
_odyCool that did it.00:29
openstackgerritMerged openstack/puppet-keystone: admin role: make sure openrc is created after  https://review.openstack.org/28964000:34
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: make sure neutron-server starts before collocated agents  https://review.openstack.org/29020500:36
*** bapalm has quit IRC00:37
*** dfisher has quit IRC00:41
*** bapalm has joined #puppet-openstack00:54
*** andreww has quit IRC01:02
*** trown|outtypewww is now known as trown01:06
*** priteau has joined #puppet-openstack01:10
openstackgerritLiao Penghui proposed openstack/puppet-nova: Fix inconsistent type/provider for paste configs.  https://review.openstack.org/26669201:11
*** johnmilton has joined #puppet-openstack01:14
*** priteau has quit IRC01:15
openstackgerritLiao Penghui proposed openstack/puppet-nova: Fix inconsistent type/provider for paste configs.  https://review.openstack.org/26669201:18
*** johnmilton has quit IRC01:19
*** ducttape_ has joined #puppet-openstack01:24
*** johnmilton has joined #puppet-openstack01:31
openstackgerritLiao Penghui proposed openstack/puppet-nova: Fix inconsistent type/provider for paste configs.  https://review.openstack.org/26669201:34
*** andreww has joined #puppet-openstack01:43
*** ducttape_ has quit IRC01:52
*** ducttape_ has joined #puppet-openstack01:57
*** toMeloos has joined #puppet-openstack02:02
*** baker has joined #puppet-openstack02:03
*** baker_ has joined #puppet-openstack02:09
*** johnmilton has quit IRC02:11
*** baker has quit IRC02:13
*** ducttape_ has quit IRC02:15
*** toMeloos has quit IRC02:34
*** andreww has quit IRC02:42
*** xarses has joined #puppet-openstack02:44
*** ilbot3 has quit IRC02:47
*** ilbot3 has joined #puppet-openstack02:49
*** richm has quit IRC02:54
*** ducttape_ has joined #puppet-openstack03:25
*** markvoelker has joined #puppet-openstack03:30
*** fedexo has joined #puppet-openstack03:40
*** baker_ has quit IRC03:46
*** ducttape_ has quit IRC03:55
*** ducttape_ has joined #puppet-openstack04:10
*** ducttape_ has quit IRC04:40
*** xarses has quit IRC04:45
*** xarses has joined #puppet-openstack04:46
*** xarses_ has joined #puppet-openstack05:03
*** xarses has quit IRC05:06
*** saneax_AFK is now known as saneax05:10
*** darkxploit has quit IRC05:24
*** darkxploit has joined #puppet-openstack05:39
openstackgerritMerged openstack/puppet-openstack_extras: auth_file: add tag to File resource  https://review.openstack.org/28963605:42
*** rcernin has joined #puppet-openstack06:09
*** bnemec has quit IRC06:11
*** jtomasek has joined #puppet-openstack06:19
*** fedexo has quit IRC06:41
*** saneax is now known as saneax_AFK07:12
*** magicboiz has joined #puppet-openstack07:18
*** magicboiz has left #puppet-openstack07:18
*** saneax_AFK is now known as saneax07:23
*** gfidente has joined #puppet-openstack07:58
*** d0ugal has joined #puppet-openstack08:09
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Allow setting of public_addr from ::profile::mon and ::mon  https://review.openstack.org/28035108:16
*** dgurtner has joined #puppet-openstack08:17
*** foxhunt_away is now known as foxhunt7208:25
*** beddari has quit IRC08:34
*** mkarpin has joined #puppet-openstack08:39
*** hjensas has quit IRC08:46
*** pma has joined #puppet-openstack08:46
*** openstackgerrit has quit IRC08:47
*** openstackgerrit has joined #puppet-openstack08:48
*** toMeloos has joined #puppet-openstack08:51
openstackgerritJavier Peña proposed openstack/puppet-tempest: Add parameter to configure enabled Sahara plugins  https://review.openstack.org/29004908:53
*** dgurtner has quit IRC08:57
*** dgurtner has joined #puppet-openstack08:57
*** dgurtner has quit IRC08:57
*** dgurtner has joined #puppet-openstack08:57
*** jaosorior has joined #puppet-openstack09:06
openstackgerritOleksiy Molchanov proposed openstack/puppet-ceph: Add possibility to preserve previous apache deployment  https://review.openstack.org/26063909:11
openstackgerritOleksiy Molchanov proposed openstack/puppet-ceph: Add possibility to preserve previous apache deployment  https://review.openstack.org/26063909:12
*** bapalm has quit IRC09:14
*** _degorenko|afk is now known as degorenko09:15
*** jistr has joined #puppet-openstack09:19
*** priteau has joined #puppet-openstack09:21
*** karimb has joined #puppet-openstack09:22
*** bapalm has joined #puppet-openstack09:27
*** paramite has joined #puppet-openstack09:36
*** derekh has joined #puppet-openstack09:46
openstackgerritJavier Peña proposed openstack/puppet-openstack-integration: Change Sahara plugins configured to vanilla, cdh  https://review.openstack.org/29036609:54
*** paramite is now known as paramite|afk09:56
openstackgerritJavier Peña proposed openstack/puppet-openstack-integration: Change Sahara plugins configured to default  https://review.openstack.org/29036610:05
*** johnmilton has joined #puppet-openstack10:20
*** hjensas has joined #puppet-openstack10:21
*** johnmilton has quit IRC10:26
*** paramite|afk is now known as paramite10:30
karimbis gate-tripleo-ci-f22-ha broken ?10:32
*** johnmilton has joined #puppet-openstack10:38
*** johnmilton has quit IRC10:40
*** dmnord has joined #puppet-openstack10:46
*** dmnord has quit IRC10:47
*** IvanBerezovskiy has left #puppet-openstack10:47
*** iberezovskiy has joined #puppet-openstack10:48
*** dmnord has joined #puppet-openstack10:52
*** dmnord has quit IRC10:55
*** karimb has quit IRC10:56
*** karimb has joined #puppet-openstack10:57
*** yottatsa has joined #puppet-openstack11:15
*** _skolekonov is now known as skolekonov11:24
*** toMeloos has quit IRC11:38
*** priteau has quit IRC11:39
openstackgerritDenis Egorenko proposed openstack/puppet-cinder: Add support of Block Device backend for Cinder  https://review.openstack.org/28483411:42
*** johnmilton has joined #puppet-openstack11:44
*** delatte has quit IRC11:45
openstackgerritJerzy Mikolajczak proposed openstack/puppet-neutron: Deprecate API parameters from metadata agent  https://review.openstack.org/28227611:53
openstackgerritJerzy Mikolajczak proposed openstack/puppet-neutron: Deprecate API parameters from metadata agent  https://review.openstack.org/28227612:00
*** jaosorior has quit IRC12:02
*** jaosorior has joined #puppet-openstack12:03
*** trown is now known as trown|commute12:03
openstackgerritMykyta Karpin proposed openstack/puppet-glance: spec: Add Unit Tests for glance paste_ini types/providers  https://review.openstack.org/29043712:06
*** ChanServ sets mode: +o EmilienM12:16
*** emccormick has quit IRC12:23
*** emccormick has joined #puppet-openstack12:25
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate class for configuring logging  https://review.openstack.org/28533312:25
*** tdb_ has joined #puppet-openstack12:26
*** Mosibi_ has joined #puppet-openstack12:26
*** rlaurika_ has joined #puppet-openstack12:26
*** tdb has quit IRC12:27
*** rlaurika has quit IRC12:27
*** tristanC has quit IRC12:27
*** Mosibi has quit IRC12:27
*** jaosorior has quit IRC12:27
*** bapalm has quit IRC12:27
*** zeus has quit IRC12:27
*** tristanC has joined #puppet-openstack12:27
*** jaosorior has joined #puppet-openstack12:27
*** bapalm has joined #puppet-openstack12:27
*** zeus has joined #puppet-openstack12:30
*** zeus is now known as Guest864612:30
*** cdelatte has joined #puppet-openstack12:32
EmilienMhello12:34
*** foxhunt72 is now known as foxhunt_away12:34
openstackgerritAlexey Deryugin proposed openstack/puppet-glance: Switch Glance to use Keystone auth_plugin  https://review.openstack.org/28784712:37
degorenkohey :)12:38
*** beddari has joined #puppet-openstack12:40
*** foxhunt_away is now known as foxhunt7212:41
mjblackEmilienM: I have a question, what is setting $::os_service_default?12:41
EmilienMmjblack: https://wiki.openstack.org/wiki/Puppet/Coding_style#Config_file_defaults_and_parameters12:42
mjblackthank you12:42
mjblackok... little confused, it's just an uninitialized variable?12:44
EmilienMskolekonov, degorenko: I found something weird in neutron12:49
EmilienMwhen I update RDO repo to trunk (yesterday), neutron_network provider is broken12:50
EmilienMhttp://logs.openstack.org/97/289997/3/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/8d0d8b3/console.html#_2016-03-09_00_19_48_64412:50
EmilienMmaybe something changed in client?12:50
EmilienMmjblack: yes12:50
mjblackah ok, just making sure :D12:50
EmilienMmjblack: $::os_service_default means we don't set the value so the service will use default value set by the OpenStack project12:50
mjblackso when it's set it will still have the config line uncommented but the value missing?12:51
EmilienMmjblack: only if config file provided by packaging contains it12:52
EmilienMotherwise value is ignored12:52
EmilienMskolekonov, degorenko: it sounds like admin_state_up="True" can't be found12:52
degorenkoEmilienM, really weird, what is patch number where you faced this issue?12:52
EmilienMdegorenko: and it's happening during second puppet run12:53
EmilienMit sounds like random12:54
degorenkomay be some dependency ?12:55
*** dgurtner has quit IRC12:59
*** dgurtner has joined #puppet-openstack12:59
*** dgurtner has quit IRC12:59
*** dgurtner has joined #puppet-openstack12:59
EmilienMjpena, degorenko: https://github.com/openstack/tempest/commit/d750c2d284ddcadc37ccea0cfa625ae987b7e98213:01
EmilienMthis patch breaks our scenario00313:01
jpenaEmilienM: yep, that's why I proposed https://review.openstack.org/29036613:01
*** skolekonov has left #puppet-openstack13:02
EmilienMand thats why I approved this patch13:02
EmilienMgood13:02
EmilienMjpena: sorry, missed that patch this morning :)13:02
*** skolekonov has joined #puppet-openstack13:02
jpenaEmilienM: no problem ;)13:02
degorenkoEmilienM, from neutron logs i don't see errors for your error13:02
EmilienMdegorenko: let me try something13:02
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: [inspector] create a separate class for configuring logging  https://review.openstack.org/28533313:03
*** morazi has joined #puppet-openstack13:04
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repo: promote RDO to March 9th packaging  https://review.openstack.org/28999713:05
*** jayg|g0n3 is now known as jayg13:05
EmilienMok I want to see if this bug happens again13:05
*** markvoelker_ has joined #puppet-openstack13:08
*** markvoelker has quit IRC13:11
*** ducttape_ has joined #puppet-openstack13:12
*** dprince has joined #puppet-openstack13:14
*** jaosorior has quit IRC13:15
*** jaosorior has joined #puppet-openstack13:15
*** trown|commute is now known as trown13:18
degorenkoEmilienM, we've reproduced this issue13:20
degorenkowith latest neutron client13:20
EmilienMdamn13:20
*** markvoelker_ has quit IRC13:20
degorenkowe are trying to understand problem13:21
*** richm has joined #puppet-openstack13:22
degorenkoEmilienM, we found it13:23
skolekonovThe provider checks 'if results =~ /Created a new network:', but new client doesn't show it anymore13:24
EmilienMok13:24
EmilienMnice catch guys13:24
skolekonov:q13:24
skolekonovSorry :)13:24
EmilienMwe need to send a patch in puppet-neutron that is backward compatible with old clients13:24
openstackgerritAlexey Deryugin proposed openstack/puppet-glance: Switch Glance to use Keystone auth_plugin  https://review.openstack.org/28784713:24
EmilienMbecause otherwise CI will never pass13:24
*** ducttape_ has quit IRC13:27
degorenkoEmilienM, i guess, we can remove this check https://github.com/openstack/puppet-neutron/blob/master/lib/puppet/provider/neutron_network/neutron.rb#L10513:28
EmilienMdegorenko: we need to find something that check if network is created13:28
EmilienMwhat is the new message or new pattern?13:28
EmilienMnothing?13:29
degorenkoEmilienM, why? We will have errors if it will not be created13:29
degorenkoit is just forced quit13:29
EmilienMwell, I'm ok to trying that13:29
skolekonovYeah, currently client returns only values13:29
degorenkoEmilienM, we can use 'rescue' for https://github.com/openstack/puppet-neutron/blob/master/lib/puppet/provider/neutron_network/neutron.rb#L106 if you want to check that, but in my opinion we don't need this check  anymore - we will have an error from shell in logs anyway13:31
degorenkoand btw, we don't have such checks for other providers in other modules13:31
degorenkoskolekonov, thoughts? ^13:31
skolekonovAgree13:32
openstackgerritMerged openstack/puppet-openstack-integration: Change Sahara plugins configured to default  https://review.openstack.org/29036613:32
*** dgurtner has quit IRC13:32
EmilienMdegorenko: ok go ahead13:33
degorenkoskolekonov, i guess :)13:33
skolekonovok13:33
openstackgerritAlexey Morlang proposed openstack/puppet-designate: spec: Add Unit-Test for designate rootwrap_config and paste_ini types/providers.  https://review.openstack.org/29048913:34
EmilienMyou can submit the patch, I'll rebase my puppet-neutron patch on top of it and I'll depends-on this one in the promotion patch13:34
skolekonovI guess we also need to check other resources13:35
degorenkoEmilienM, omg, we have such checks for router, subnet too13:35
EmilienMdegorenko: yeah13:35
EmilienMwe need to fix them all13:35
*** toMeloos has joined #puppet-openstack13:36
*** v1k0d3n has quit IRC13:38
*** v1k0d3n has joined #puppet-openstack13:39
*** markvoelker has joined #puppet-openstack13:39
*** jpena is now known as jpena|lunch13:39
openstackgerritEmilien Macchi proposed openstack/puppet-heat: add more heat.conf settings to heat, engine classes  https://review.openstack.org/29049413:40
*** asilenkov_ has quit IRC13:41
*** yottatsa has quit IRC13:41
*** mkarpin has quit IRC13:41
*** dmburmistrov has quit IRC13:41
openstackgerritRadoslaw Smigielski proposed openstack/puppet-nova: Add missing block_device_allocate_* params in nova.conf  https://review.openstack.org/29049613:41
*** trown is now known as trown|brb13:50
*** trown|brb is now known as trown13:52
openstackgerritAlexey Morlang proposed openstack/puppet-designate: spec: Add Unit-Test for designate rootwrap_config and paste_ini types/providers.  https://review.openstack.org/29048913:57
*** markvoelker has quit IRC13:58
openstackgerritEmilien Macchi proposed openstack/puppet-heat: add more heat.conf settings to heat, engine classes  https://review.openstack.org/29049414:08
*** ducttape_ has joined #puppet-openstack14:12
*** yottatsa has joined #puppet-openstack14:12
*** saneax is now known as saneax_AFK14:13
*** igajsin has joined #puppet-openstack14:14
openstackgerritSergey Kolekonov proposed openstack/puppet-neutron: Remove broken checks from providers  https://review.openstack.org/29052114:15
skolekonovEmilienM, let's try my patch https://review.openstack.org/#/c/290521/ for providers14:16
EmilienMok I'll rebase mine14:16
EmilienMand test it with new neutron14:16
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: make sure neutron-server starts before collocated agents  https://review.openstack.org/29020514:17
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repo: promote RDO to March 9th packaging  https://review.openstack.org/28999714:17
*** mkarpin has joined #puppet-openstack14:18
*** asilenkov has joined #puppet-openstack14:19
openstackgerritMarcus Furlong proposed openstack/puppet-nova: cpu_allocation_ratio / ram_allocation_ratio options  https://review.openstack.org/29053414:33
*** ducttape_ has quit IRC14:34
EmilienMdegorenko: any chance to review https://review.openstack.org/#/c/290494/ ?14:36
degorenkoEmilienM, hmmmm, i don't know14:36
degorenkojoking14:36
* degorenko looking14:36
EmilienMit's not friday! jokes are not allowed \o/14:37
degorenkookay :(14:37
EmilienMahah14:37
EmilienMit was a joke :P14:37
degorenkoi know :P14:37
degorenkoEmilienM, you know, heat scenario still broken https://review.openstack.org/#/c/286717/14:38
degorenkos/scenario/beaker14:38
EmilienMdegorenko: on ubuntu ?14:41
degorenkoyep14:42
degorenko:(14:42
*** ducttape_ has joined #puppet-openstack14:42
EmilienMiirc they did not update heat14:42
EmilienMwe need to check with james14:42
degorenko#openstack-dev?14:43
dmsimardEmilienM: did that fix for neutron go in ?14:44
EmilienMdmsimard: not yet14:44
EmilienMdmsimard: today if CI is green14:44
EmilienMdmsimard: see https://review.openstack.org/#/c/289997/ and it's dependencies14:45
EmilienMhttps://review.openstack.org/#/c/290205/ and https://review.openstack.org/#/c/290521/14:45
dmsimardapevec has a point on your agent review14:46
dmsimardmaybe wait until the connection to neutron server is up instead or something14:46
*** jpena|lunch is now known as jpena14:47
*** markvoelker has joined #puppet-openstack14:48
degorenkoEmilienM, they said that clients already updated, hm, do we need bump our repo?14:48
*** dgurtner has joined #puppet-openstack14:48
*** dgurtner has quit IRC14:48
*** dgurtner has joined #puppet-openstack14:48
EmilienMright apevec is right14:49
dmsimardEmilienM: maybe something like http://git.openstack.org/cgit/openstack/puppet-neutron/tree/manifests/plugins/ovs/opendaylight.pp#n5214:49
dmsimard(the exec retry/sleep)14:49
dmsimardI think there was something for waiting for a port to become available too ? I forget.. I don't do advanced puppet much anymore14:50
EmilienMno execs please14:50
EmilienMit's a bug in neutron14:50
EmilienMhttps://bugs.launchpad.net/tripleo/+bug/155484614:50
openstackLaunchpad bug 1554846 in tripleo "stable/liberty CI: jobs failing with Could not start Service[neutron-ovs-agent-service]: Execution of '/usr/bin/systemctl start neutron-openvswitch-agent' returned 1: Job for neutron-openvswitch-agent.service failed because a timeout was exceeded." [Critical,In progress] - Assigned to James Slagle (james-slagle)14:50
EmilienMand likely a packaging change https://review.gerrithub.io/#/c/265669/14:50
openstackgerritJavier Peña proposed openstack/puppet-tempest: Add parameter to configure enabled Sahara plugins  https://review.openstack.org/29004914:52
dmsimardEmilienM: puppetdb checks for availability through a provider like so https://github.com/puppetlabs/puppetlabs-puppetdb/blob/master/manifests/server/validate_db.pp#L17 and https://github.com/puppetlabs/puppetlabs-puppetdb/blob/master/lib/puppet/provider/puppetdb_conn_validator/puppet_https.rb14:52
EmilienMwe don't have puppetdb14:52
EmilienMand don't want to rely on it14:52
EmilienMa lot of people don't use it14:53
EmilienMdmsimard: it's a bug in neutron14:53
EmilienMlike apevec said14:53
dmsimardEmilienM: I know, I am showing you a way to "wait and retry" until a service is available14:53
dmsimardand I know about the bug as well14:53
*** bnemec has joined #puppet-openstack14:53
EmilienMI'm not sure but https://review.gerrithub.io/#/c/265671/ might be the fix14:54
*** mkarpin has left #puppet-openstack14:54
dmsimardEmilienM: more or less, tripleo has the same issue where agents can/could start before neutron-server14:55
dmsimardit will probably fix it, but it makes sense to add a proper require14:56
EmilienMI can try to abandon my patch and see how it works with recent package14:57
*** xarses_ has quit IRC15:00
*** dfisher has joined #puppet-openstack15:01
*** saneax_AFK is now known as saneax15:08
*** gchamoul has quit IRC15:10
*** dgurtner has quit IRC15:12
openstackgerritRadoslaw Smigielski proposed openstack/puppet-nova: Add missing block_device_allocate_* params in nova.conf  https://review.openstack.org/29049615:18
*** baker has joined #puppet-openstack15:26
EmilienMskolekonov, degorenko: it pass well https://review.openstack.org/#/c/289997/15:28
openstackgerritMerged openstack/puppet-tripleo: Make OpenStack service ports configurable in HAProxy  https://review.openstack.org/28719915:28
EmilienMdegorenko: can you review https://review.openstack.org/#/c/290521/ ?15:28
degorenkoyeah15:30
openstackgerritRadoslaw Smigielski proposed openstack/puppet-openstacklib: Fix type in os_service_default fact  https://review.openstack.org/29058715:31
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Bump RDO repo to March 9th  https://review.openstack.org/29058815:31
EmilienMdmsimard: trying a promotion without my hack15:31
*** xarses_ has joined #puppet-openstack15:33
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Make OpenStack service ports configurable in HAProxy  https://review.openstack.org/28796115:50
openstackgerritMerged openstack/puppet-neutron: Remove broken checks from providers  https://review.openstack.org/29052116:07
*** foxhunt72 is now known as foxhunt_away16:07
*** paramite has quit IRC16:08
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch Keystone/Glance/Ironic/Nova to SSL  https://review.openstack.org/28689916:39
*** trown has quit IRC16:41
EmilienMdegorenko: can you review https://review.openstack.org/#/c/290588/ please?16:43
EmilienMdmsimard: ^ the promotion will work without my hack, that I abandonned btw16:44
*** trown has joined #puppet-openstack16:46
degorenkoEmilienM, done16:46
EmilienMdegorenko: thx16:46
EmilienMdegorenko: and thanks skolekonov for the help16:46
skolekonovnp16:47
*** EmilienM sets mode: +o _ody16:47
*** EmilienM sets mode: +o clayton16:47
zigoEmilienM: How come there's a setup.py in puppet-keystone?16:51
EmilienMzigo: we use reno16:51
EmilienMhttp://docs.openstack.org/releasenotes/puppet-keystone/16:51
zigoEmilienM: And? So it needs setup.py ?16:51
EmilienMyes16:52
zigo:/16:52
zigoNot convenient at all for me, but I'll addapt.16:52
*** trown is now known as trown|brb16:52
zigoEmilienM: So, no python code there at all, right?16:52
EmilienMzigo: https://github.com/openstack/puppet-keystone/tree/master/releasenotes/source16:52
zigoReno is a real nightmare for me... :(16:53
*** trown has joined #puppet-openstack16:53
*** jaosorior has quit IRC16:55
zigoEmilienM: The result is that puppet-keystone generates /usr/lib/python2.7/dist-packages/puppet-keystone-8.0.0b1.egg-info !!!16:56
openstackgerritMerged openstack/puppet-heat: add more heat.conf settings to heat, engine classes  https://review.openstack.org/29049416:58
zigoEmilienM: Will you use reno on all puppet modules, or just keystone?16:59
EmilienMfor now, we use it in puppet-keystone and see how it wors16:59
EmilienMworks*17:00
EmilienMif we like it and find it useful, we'll use it everywhere17:00
zigoI'd prefer if you didn't.17:00
zigo(but I would understand if you did...)17:00
EmilienMwhat do you suggest?17:02
zigoThat Doug Hellman stops writing such a problematic software...17:03
zigo:)17:03
EmilienMgo talk to him17:03
zigoI'll try to, in Austin.17:03
*** gchamoul has joined #puppet-openstack17:04
EmilienMthe main motivation of using reno in puppet modules was to involve everyone is release notes17:04
*** trown|brb has quit IRC17:05
*** jistr has quit IRC17:06
*** rcernin has quit IRC17:09
*** dhollinger has joined #puppet-openstack17:12
*** saneax is now known as saneax_AFK17:16
*** skolekonov is now known as _skolekonov17:16
*** karimb has quit IRC17:16
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch OpenStack APIs to IPv6  https://review.openstack.org/28752117:22
*** hjensas has quit IRC17:24
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch OpenStack APIs to IPv6  https://review.openstack.org/28752117:24
degorenkoEmilienM, mwhahaha can you guys take a look on this again please? :) https://review.openstack.org/28483417:25
EmilienMno17:25
EmilienMdegorenko: +A17:27
degorenkothanks o/17:28
*** gchamoul has quit IRC17:29
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch Keystone/Glance/Ironic/Nova to SSL  https://review.openstack.org/28689917:30
EmilienMdegorenko: do you mind approving https://review.openstack.org/#/c/290588/ please?17:30
degorenkoEmilienM, oh, we voted with dprince in same time17:31
degorenko+A17:31
EmilienMcool we promoted to OpenStack master17:32
degorenkonow we need master for trusty :D17:32
EmilienMthat's another story17:32
degorenkosadly - true17:34
*** gchamoul has joined #puppet-openstack17:35
zigoEmilienM: All of puppet-openstack Mitaka b1 tag uploaded to Debian Experimental. Thanks a lot for this tag, this is really super useful.17:35
EmilienMcool17:35
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch OpenStack APIs to IPv6  https://review.openstack.org/28752117:39
*** degorenko is now known as _degorenko|afk17:42
*** igajsin has quit IRC17:56
EmilienMmwhahaha: in case you have some time this week, I have this patch https://review.openstack.org/#/c/213315/ (a bit big)17:57
mwhahahak17:58
*** yottatsa has quit IRC18:05
*** derekh is now known as derekh_afk18:06
openstackgerritMerged openstack/puppet-openstack-integration: Bump RDO repo to March 9th  https://review.openstack.org/29058818:09
*** hjensas has joined #puppet-openstack18:10
openstackgerritRadoslaw Smigielski proposed openstack/puppet-nova: Add missing block_device_allocate_* params in nova.conf  https://review.openstack.org/29049618:11
*** jpena is now known as jpena|zzz18:14
*** jaosorior has joined #puppet-openstack18:21
*** iberezovskiy has quit IRC18:24
*** yolanda has quit IRC18:28
*** toMeloos has quit IRC18:28
*** yolanda has joined #puppet-openstack18:29
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Implement WSGI support for Nova API  https://review.openstack.org/21331518:41
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Implement WSGI support for Nova API  https://review.openstack.org/21331518:48
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Deploy Nova API in WSGI with Apache  https://review.openstack.org/28956218:53
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch Keystone/Glance/Ironic/Nova to SSL  https://review.openstack.org/28689918:53
*** markvoelker has quit IRC19:01
*** ducttape_ has quit IRC19:02
*** markvoelker has joined #puppet-openstack19:03
*** trown is now known as trown|lunch19:05
*** ducttape_ has joined #puppet-openstack19:07
*** markvoelker has quit IRC19:17
*** markvoelker has joined #puppet-openstack19:18
*** saneax_AFK is now known as saneax19:21
*** brandon has joined #puppet-openstack19:25
*** gfidente is now known as gfidente|afk19:41
*** gfidente|afk has quit IRC19:43
*** jaosorior has quit IRC20:02
*** jaosorior has joined #puppet-openstack20:03
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Implement WSGI support for Nova API  https://review.openstack.org/21331520:03
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch Keystone/Glance/Ironic/Nova to SSL  https://review.openstack.org/28689920:17
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch Keystone/Glance/Ironic/Nova to SSL  https://review.openstack.org/28689920:25
*** trown|lunch is now known as trown20:27
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: IPv6: duak-stack support for Keystone  https://review.openstack.org/28634420:38
*** saneax is now known as saneax_AFK20:50
openstackgerritNate Potter proposed openstack/puppet-openstack-integration: Update glance::api stores parameters  https://review.openstack.org/28747720:58
EmilienMmwhahaha: your last comment on puppet-nova / wsgi is actually wrong I think21:04
mwhahahawhich one?21:04
EmilienMwhen you ask to move the conditional21:04
EmilienMabout enable_metadata21:04
EmilienMthe conditional is only when we run service_name == httpd21:04
mwhahahaRight but you were only checking it when it was a string21:05
mwhahahayou need the service_name ==httpd but you were only doing that logic if $enabled_apis was not an array21:05
mwhahahawhich was wrong21:05
EmilienMmhh so why the jobs are failing now21:05
mwhahahabecause the jobs don't use an array :D21:05
mwhahahayour logic is bad21:05
mwhahahahttp://www.quickmeme.com/img/b1/b1029b7e44ca8cbe79ddb7e085be2ff2a8334e17f9987eaa093da2d6d99cb4f0.jpg21:06
mwhahaha</troll>21:06
EmilienMI feel so  bad now21:06
dmsimardlol21:06
EmilienMmwhahaha: have you seen my last PS? https://review.openstack.org/#/c/213315/31/manifests/api.pp21:13
mwhahahanope not yet, let me go look21:13
EmilienMthat's why I'm confused21:14
mwhahahaERROR: InvocationError: '/bin/bash tools/pretty_tox.sh --concurrency=2 smoke dashboard TelemetryAlarming api.baremetal'21:15
EmilienMcan I press the "noooo" button?21:16
EmilienMso it seems like it's realted to my patch21:17
EmilienMhttp://logs.openstack.org/15/213315/31/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/a3196a5/console.html#_2016-03-09_20_54_41_23421:17
EmilienMmetadata is not enabled in nova-api21:17
EmilienMI must have missed something21:17
*** markvoelker has quit IRC21:20
mwhahahayou shouldn't need to dupe the !empty check because any2array should cover both cases21:21
EmilienMright21:21
*** xarses_ is now known as xarses21:22
mwhahahaline 270, should that be $enabled_api_string?21:24
*** trown is now known as trown|outtypewww21:25
EmilienMsigh21:26
EmilienMI think yes21:26
EmilienMI'm missing unit tests to catch this case21:27
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Implement WSGI support for Nova API  https://review.openstack.org/21331521:28
*** derekh_afk is now known as derekh21:29
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Implement WSGI support for Nova API  https://review.openstack.org/21331521:47
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Deploy Nova API in WSGI with Apache  https://review.openstack.org/28956221:48
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch Keystone/Glance/Ironic/Nova to SSL  https://review.openstack.org/28689921:48
*** ducttape_ has quit IRC21:59
*** ducttape_ has joined #puppet-openstack22:01
*** jayg is now known as jayg|g0n322:03
*** jayg|g0n3 is now known as jayg22:04
*** jayg is now known as jayg|g0n322:05
*** derekh has quit IRC22:19
*** angdraug has joined #puppet-openstack22:28
ntpttrEmilienM: when you have a moment would you mind looking this over for a possible +A? Adds manifests/api.pp to puppet-magnum.22:30
ntpttrhttps://review.openstack.org/#/c/254392/22:30
EmilienMntpttr: done22:30
ntpttrEmilienM: thanks! I appreciate how responsive/quick you are22:32
openstackgerritEmilien Macchi proposed openstack/puppet-modulesync-configs: add puppet-octavia to managed modules  https://review.openstack.org/29088622:34
*** johnmilton has quit IRC22:37
*** jtomasek has quit IRC22:39
EmilienMntpttr: cool anytime22:43
*** rahulait has joined #puppet-openstack22:43
openstackgerritEmilien Macchi proposed openstack/puppet-octavia: puppet-octavia: Initial commit  https://review.openstack.org/29089222:45
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Puppetfile: add puppet-oslo & puppet-octavia  https://review.openstack.org/29089322:46
EmilienMmwhahaha: does fuel have plans to use octavia?22:46
mwhahahaprobably not built into the product22:47
mwhahahaprobably via a plugin at some point22:47
EmilienMok, trying to bootstrap the module22:47
EmilienMmwhahaha: https://review.openstack.org/#/c/213315/ is ready again for review22:47
mwhahahaoh look when you fix the logic it works ;)22:47
EmilienMLOL he's trolling me again22:47
openstackgerritMerged openstack/puppet-magnum: Initial commit of api.pp  https://review.openstack.org/25439222:54
*** reynir has quit IRC22:57
*** reynir has joined #puppet-openstack23:01
openstackgerritBen Nemec proposed openstack/puppet-tripleo: Allow enabling authentication on haproxy.stats  https://review.openstack.org/29089623:01
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: switch Keystone/Glance/Ironic/Nova to SSL  https://review.openstack.org/28689923:01
« Tuesday, 2016-03-08 Index Thursday, 2016-03-10 »

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