Friday, 2016-04-22

*** markvoelker has joined #puppet-openstack00:19
*** xarses has quit IRC00:21
*** xarses has joined #puppet-openstack00:50
*** ducttape_ has joined #puppet-openstack01:06
*** ducttape_ has quit IRC01:32
openstackgerritEmilien Macchi proposed openstack/puppet-swift: unit tests: make Keystone_endpoint match service by name/type  https://review.openstack.org/30923601:55
openstackgerritEmilien Macchi proposed openstack/puppet-swift: Remove tenant parameter from keystone_user  https://review.openstack.org/30918901:56
openstackgerritEmilien Macchi proposed openstack/puppet-swift: swift-ring-builder 2.2.2+ output parse fix  https://review.openstack.org/30914101:56
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: Deprecate service_provider  https://review.openstack.org/30915401:56
mtreinishEmilienM: http://health.openstack.org/runs/key/project/openstack/puppet-keystone/recent/rss02:03
mgagnejust a reminder that Puppet facts (with 3.7 at least) are strings although they look to be returning boolean value :-/02:07
mgagnehttp://paste.openstack.org/show/495072/ =(02:07
mgagnestr2bool them :D02:07
*** richm has quit IRC02:32
*** markvoelker has quit IRC02:35
*** fedexo has joined #puppet-openstack02:52
*** haplo37 has quit IRC03:30
*** markvoelker has joined #puppet-openstack03:36
*** markvoelker has quit IRC03:41
*** mkarpin1 has quit IRC03:58
*** mkarpin has joined #puppet-openstack03:58
*** dtantsur|afk has quit IRC03:59
*** dtantsur has joined #puppet-openstack04:03
*** aarefiev has quit IRC04:16
*** aarefiev has joined #puppet-openstack04:16
openstackgerritBabu Shanmugam proposed openstack/puppet-ovn: Implement basic beakers tests  https://review.openstack.org/30792004:28
*** markvoelker has joined #puppet-openstack04:37
*** markvoelker has quit IRC04:42
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Create keystone endpoints in keystone profile  https://review.openstack.org/30412504:51
*** saneax_AFK is now known as saneax04:56
*** rcernin has joined #puppet-openstack05:03
*** jaosorior has joined #puppet-openstack05:12
*** dtantsur has quit IRC05:18
*** jaosorior has quit IRC05:27
*** dtantsur has joined #puppet-openstack05:36
*** chandankumar has joined #puppet-openstack05:44
*** jmikolajczak has joined #puppet-openstack05:53
*** jtomasek has joined #puppet-openstack05:57
*** dgurtner has joined #puppet-openstack06:01
*** dgurtner has quit IRC06:01
*** dgurtner has joined #puppet-openstack06:01
*** dtantsur has quit IRC06:04
*** tlbr has quit IRC06:05
*** tlbr has joined #puppet-openstack06:05
*** chandankumar has quit IRC06:07
openstackgerritSergii Golovatiuk proposed openstack/puppet-openstack-integration: Remove sticky version from concat  https://review.openstack.org/30735406:07
*** chandankumar has joined #puppet-openstack06:08
*** dtantsur has joined #puppet-openstack06:08
*** magicboiz has joined #puppet-openstack06:24
*** magicboiz has left #puppet-openstack06:24
*** v1k0d3n has quit IRC06:27
*** v1k0d3n has joined #puppet-openstack06:27
*** chandankumar has quit IRC06:29
*** tesseract has joined #puppet-openstack06:43
*** tesseract is now known as Guest8301606:43
*** Guest83016 has quit IRC06:48
*** tesseract- has joined #puppet-openstack07:02
*** dgurtner has quit IRC07:02
*** foxhunt_away is now known as foxhunt7207:02
*** karimb has joined #puppet-openstack07:05
*** fedexo has quit IRC07:15
*** amoralej|off is now known as amoralej07:22
-openstackstatus- NOTICE: Logs are failing to be uploaded causing jobs to be marked as UNSTABLE. We are working on repairing the log filesystem and will update when ready. Please do not recheck before then.07:31
*** ChanServ changes topic to "Logs are failing to be uploaded causing jobs to be marked as UNSTABLE. We are working on repairing the log filesystem and will update when ready. Please do not recheck before then."07:31
*** jaosorior has joined #puppet-openstack07:32
*** furlongm_ has joined #puppet-openstack07:32
*** furlongm has quit IRC07:33
*** dgurtner has joined #puppet-openstack07:46
*** dgurtner has quit IRC07:46
*** dgurtner has joined #puppet-openstack07:46
*** shardy has joined #puppet-openstack07:55
*** chandankumar has joined #puppet-openstack08:02
*** gfidente has joined #puppet-openstack08:27
*** gfidente has quit IRC08:27
*** gfidente has joined #puppet-openstack08:27
*** imcsk8 is now known as imcsk8|zZz08:27
*** jistr has joined #puppet-openstack08:28
*** jpena|off is now known as jpena08:36
openstackgerritIvan Berezovskiy proposed openstack/puppet-heat: Drop all current deprecated parameters for heat  https://review.openstack.org/30851108:39
*** chem has quit IRC08:41
*** chem has joined #puppet-openstack08:42
*** Vinsh has quit IRC08:43
*** derekh has joined #puppet-openstack08:46
mattymoanyone want to review? https://review.openstack.org/#/c/309103/08:47
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add missing services ports to service_ports map  https://review.openstack.org/30592908:48
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Fix Sahara SSL default port  https://review.openstack.org/30930608:48
*** mgould has joined #puppet-openstack08:50
*** chandankumar has quit IRC09:02
*** chandankumar has joined #puppet-openstack09:05
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add missing services ports to service_ports map  https://review.openstack.org/30592909:18
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Fix Sahara SSL default port  https://review.openstack.org/30930609:18
openstackgerritMatthew Mosesohn proposed openstack/puppet-nova: Add nova_service type for purging nova services from offline hosts  https://review.openstack.org/30910309:18
*** darkxploit has quit IRC09:20
*** dgurtner has quit IRC09:21
*** dgurtner has joined #puppet-openstack09:22
*** dgurtner has quit IRC09:22
*** dgurtner has joined #puppet-openstack09:22
*** openstackstatus has quit IRC09:24
*** openstackstatus has joined #puppet-openstack09:26
*** ChanServ sets mode: +v openstackstatus09:26
*** furlongm_ has quit IRC09:28
*** furlongm has joined #puppet-openstack09:29
*** darkxploit has joined #puppet-openstack09:33
openstackgerritIvan Berezovskiy proposed openstack/puppet-heat: Drop all current deprecated parameters for heat  https://review.openstack.org/30851109:34
*** chandankumar has quit IRC09:35
*** jaosorior is now known as jaosorior_lunch09:35
*** karimb has quit IRC09:47
*** shardy has quit IRC09:54
*** toMeloos has joined #puppet-openstack10:05
*** karimb has joined #puppet-openstack10:15
*** jaosorior_lunch is now known as jaosorior10:19
*** beddari has quit IRC10:34
*** beddari has joined #puppet-openstack10:34
*** _degorenko|afk is now known as degorenko_10:43
*** chandankumar has joined #puppet-openstack10:45
*** iurygregory has quit IRC10:46
*** cdelatte has quit IRC10:53
-openstackstatus- NOTICE: Log server has been repaired and jobs are stable again. If necessary please recheck changes that have 'UNSTABLE' results.10:54
*** jaosorior has quit IRC10:54
openstackgerritShaik Apsar proposed openstack/puppet-trove: Add amqp_durable_queues option in trove-guestagent.conf.erb.  https://review.openstack.org/30935510:56
*** jaosorior has joined #puppet-openstack10:56
openstackgerritShaik Apsar proposed openstack/puppet-trove: Add amqp_durable_queues option in trove-guestagent.conf.erb.  https://review.openstack.org/30935510:59
*** johnmilton has joined #puppet-openstack11:00
openstackgerritShaik Apsar proposed openstack/puppet-trove: Add amqp_durable_queues option in trove-guestagent.conf.erb.  https://review.openstack.org/30935511:02
*** toMeloos has quit IRC11:05
*** ansmith has joined #puppet-openstack11:11
*** cdelatte has joined #puppet-openstack11:15
*** ansmith has quit IRC11:18
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/30906911:33
*** markvoelker has joined #puppet-openstack11:42
openstackgerritIvan Berezovskiy proposed openstack/puppet-heat: Drop all current deprecated parameters for heat  https://review.openstack.org/30851111:45
*** markvoelker has quit IRC11:47
*** paramite has joined #puppet-openstack11:47
EmilienMhello folks11:55
degorenko_hey o/11:55
EmilienMmtreinish: yeah!!!!11:55
*** dgurtner has quit IRC11:56
mattymohey EmilienM11:58
mattymocan you check out https://review.openstack.org/#/c/309103/ when you get a chance?11:58
*** paramite is now known as paramite|afk11:58
EmilienMmattymo: reviewed12:03
openstackgerritMerged openstack/puppet-tempest: Update spec tests for Tempest  https://review.openstack.org/30785312:05
*** dgurtner has joined #puppet-openstack12:06
*** dgurtner has quit IRC12:06
*** dgurtner has joined #puppet-openstack12:06
*** ducttape_ has joined #puppet-openstack12:11
*** paramite|afk is now known as paramite12:15
openstackgerritEmilien Macchi proposed openstack/puppet-swift: unit tests: make Keystone_endpoint match service by name/type  https://review.openstack.org/30918912:17
openstackgerritEmilien Macchi proposed openstack/puppet-swift: swift-ring-builder 2.2.2+ output parse fix  https://review.openstack.org/30914112:17
EmilienMdegorenko_, mgagne: ok I sorted the pupept-swift failures12:18
openstackgerritMerged openstack/puppet-openstack-integration: run_tests: add more debug when Puppet fails  https://review.openstack.org/30899112:18
EmilienMwe missed 3 cherry-picks, and 2 of them needed to be squashed12:18
openstackgerritMerged openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/30794712:20
EmilienMmtreinish: this http://health.openstack.org/runs/key/build_name/periodic-puppet-openstack-integration-4-scenario001-tempest-ubuntu-trusty/recent/rss12:21
EmilienMdoes not work12:21
EmilienMI'm trying to get RSS by build_name12:21
*** karimb has quit IRC12:23
openstackgerritMerged openstack/puppet-keystone: Add Unit Tests for keystone_config type  https://review.openstack.org/30664412:24
*** morazi has joined #puppet-openstack12:25
EmilienMbtw, if someone wants to look the periodic jobs: http://status.openstack.org/openstack-health/#/?groupKey=build_name&resolutionKey=hour&end=2016-04-22T12:19:32.912Z&searchProject=periodic-puppet12:25
*** karimb has joined #puppet-openstack12:28
*** markvoelker has joined #puppet-openstack12:28
*** chandankumar has quit IRC12:29
*** ansmith has joined #puppet-openstack12:29
*** dgurtner has quit IRC12:32
*** ducttape_ has quit IRC12:39
openstackgerritEmilien Macchi proposed openstack/puppet-swift: CI test - never merge  https://review.openstack.org/30938712:40
*** jpena is now known as jpena|lunch12:41
openstackgerritEmilien Macchi proposed openstack/puppet-swift: CI test - never merge  https://review.openstack.org/30938712:41
openstackgerritEmilien Macchi proposed openstack/puppet-horizon: CI test - never merge  https://review.openstack.org/30938812:42
openstackgerritEmilien Macchi proposed openstack/puppet-horizon: CI test - never merge  https://review.openstack.org/30938812:42
*** trown|outtypewww is now known as trown12:47
*** richm1 has joined #puppet-openstack12:51
EmilienMdegorenko_: can you review https://review.openstack.org/#/c/309156/ again please?12:51
EmilienMmjblack: hi, puppet-ovn / beaker is ready: https://review.openstack.org/#/c/307920/12:53
EmilienMmjblack: please update your patch for debian accordingly, thanks12:54
*** dgurtner has joined #puppet-openstack12:55
*** dgurtner has quit IRC12:55
*** dgurtner has joined #puppet-openstack12:55
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/30906912:55
*** jaosorior has quit IRC12:58
*** karimb has quit IRC13:03
*** mgould has quit IRC13:07
degorenko_EmilienM, we can try do nothing in default: https://docs.puppet.com/guides/style_guide.html#defaults-for-case-statements-and-selectors13:12
EmilienMdegorenko_: ok Sir13:12
EmilienMI'll do that13:12
degorenko_EmilienM, just try to test it locally13:12
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Stop overriding service provider  https://review.openstack.org/30915613:13
EmilienMdegorenko_: let's try ^13:13
degorenko_EmilienM, wait - this one will not work13:14
degorenko_we can just have epmty {}13:14
EmilienMdegorenko_: this one will work13:14
degorenko_EmilienM, it will fail on fedora13:15
degorenko_EmilienM, also i think you can set it to undef13:16
degorenko_now we don't have value for default case for $messagebus_service_name13:17
*** chandankumar has joined #puppet-openstack13:17
degorenko_so i think $messagebus_service_name = undef for default will be ok13:17
EmilienMdegorenko_: ok13:20
*** foxhunt72 is now known as foxhunt_away13:20
EmilienMI'm on it13:21
*** foxhunt_away is now known as foxhunt7213:22
degorenko_thanks13:24
EmilienMdegorenko_: wait, it will break fedora isn't?13:30
EmilienMoh13:30
EmilienMnevermind13:30
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Stop overriding service provider  https://review.openstack.org/30915613:31
EmilienMdegorenko_: done ^13:31
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Add tripleo::selinux  https://review.openstack.org/30940813:32
degorenko_EmilienM, looking13:33
degorenko_crinkle, hey :) can you please take a look on this one: https://review.openstack.org/#/c/309103/ when you will have time :)13:36
*** dfisher has joined #puppet-openstack13:37
*** MatUm has quit IRC13:37
*** amoralej is now known as amoralej|lunch13:40
EmilienMmjblack: excellent work on keystone/eventlet, thanks a lot13:40
*** degorenko has joined #puppet-openstack13:41
EmilienMdegorenko: I'll let you whether or not approve his work13:42
EmilienMhttps://review.openstack.org/#/c/308012/ and https://review.openstack.org/#/c/308833/13:42
degorenko_EmilienM, i'm looking :) i also want to see mwhahaha feedback on this13:42
mwhahahai'll take a look in a bit13:43
degorenko_mjblack, i posted one comment :)13:44
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: keystone: avoid race condition with eventlet on Ubuntu.  https://review.openstack.org/30941313:44
EmilienMI'm testing it with our jobs ^13:44
degorenko_mjblack, also you have one minor problem in release note for adding policyrcd.pp patch :)13:45
EmilienMdegorenko_: you're right13:46
EmilienMI dropped +213:46
*** mgould has joined #puppet-openstack13:47
*** degorenko has quit IRC13:50
*** degorenko has joined #puppet-openstack13:51
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/30588613:53
*** jpena|lunch is now known as jpena13:53
openstackgerritMerged openstack/puppet-tripleo: Add steps to database profiles  https://review.openstack.org/30786313:55
EmilienMmtreinish: http://health.openstack.org/runs/key/build_name/periodic-nova-py34-with-oslo-master/recent/rss13:57
EmilienMit works fine, when some failures13:57
EmilienMwhere there is no failure the rss feed is not available, see http://health.openstack.org/runs/key/build_name/periodic-puppet-openstack-integration-4-scenario001-tempest-ubuntu-trusty/recent/rss13:57
*** chandankumar has quit IRC13:57
EmilienMmtreinish: it causes an issue with RSS client, where no feed is available, maybe we could add an empty feed if no failure14:00
EmilienMI'll look into it14:00
*** degorenko_ sets mode: +o degorenko14:01
*** degorenko_ has quit IRC14:02
*** degorenko_ has joined #puppet-openstack14:03
*** degorenko_ has quit IRC14:03
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Add aodh and gnocchi to schema profiles  https://review.openstack.org/30942414:05
*** degorenko_ has joined #puppet-openstack14:06
*** degorenko_ has quit IRC14:07
mjblackEmilienM: I'll have the puppet-ovn change updated soon14:09
EmilienM++14:09
*** paramite has quit IRC14:12
*** xarses has quit IRC14:14
*** ducttape_ has joined #puppet-openstack14:16
*** dilyin has joined #puppet-openstack14:22
*** haplo37 has joined #puppet-openstack14:22
openstackgerritMatthew J Black proposed openstack/puppet-ovn: Added params for debian packages  https://review.openstack.org/30794314:33
degorenkoEmilienM, https://review.openstack.org/#/c/309413/ - passed ci14:34
EmilienMdegorenko: yeah, it's cool14:34
EmilienMdegorenko: we'll wait mjblack to update https://review.openstack.org/#/c/308012/14:34
EmilienMand we're good I think14:34
degorenkoyeah14:34
EmilienMoh and https://review.openstack.org/#/c/308833/14:35
mjblackEmilienM: just to be clear, we want to move that conditional for policy-rc.d outside of the conditional on method of running keystone?14:36
EmilienMyes14:37
EmilienMI don't know how, but some pople might use a specific service_name14:37
mjblacksure, I just wanted to be clear14:38
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Prevent keystone eventlet from starting  https://review.openstack.org/30801214:39
mjblackEmilienM,degorenko: ^ thats it14:39
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Prevent keystone eventlet from starting  https://review.openstack.org/30801214:40
mjblacksorry that is it14:40
mjblackfigured not to commit an extra a new line :D14:40
EmilienMmjblack: also openstacklib please14:41
*** xarses has joined #puppet-openstack14:41
openstackgerritMatthew J Black proposed openstack/puppet-openstacklib: Added policy-rc.d class.  https://review.openstack.org/30883314:41
mjblackEmilienM: yeah was in process, ^14:42
EmilienMcool14:42
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: Prevent keystone eventlet from starting  https://review.openstack.org/30801214:43
EmilienMrebased ^14:43
EmilienMand https://review.openstack.org/#/c/309413/ rechecked14:43
EmilienMlet's see how all is going14:43
mjblackEmilienM: with the puppet-ovn change, with rspec, I'm still seeing that issue where its not using the correct osfamily when it runs14:45
EmilienMmaybe mwhahaha can help, I'm not hyper familiar with it yet14:45
mjblackhttp://logs.openstack.org/43/307943/6/check/gate-puppet-ovn-puppet-unit-3.3-centos-7/331f407/console.html#_2016-04-22_14_42_07_29214:46
*** amoralej|lunch is now known as amoralej14:50
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Call ::sync in addition to ::mysql  https://review.openstack.org/30945014:51
*** v1k0d3n has quit IRC14:51
*** v1k0d3n has joined #puppet-openstack14:52
openstackgerritMerged openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/30794914:52
*** dhollinger has joined #puppet-openstack14:54
EmilienMdfisher: work to have tarball on each new tag: https://review.openstack.org/#/c/309453/114:55
EmilienMdfisher: if it works, we'll generate tarballs for previous tags14:56
*** dgurtner has quit IRC14:57
*** dilyin has quit IRC14:59
*** saneax is now known as saneax_AFK15:01
-openstackstatus- NOTICE: Log server was repaired as of 10:50 UTC and jobs have been stable since. If necessary, please recheck changes that have 'UNSTABLE' results.15:02
EmilienMdfisher: once it lands, infra will have to run http://paste.openstack.org/show/495164/ and we'll have tarballs for: master stable/mitaka stable/liberty 7.0.0 and 8.0.015:03
dfisheroutstanding.15:03
EmilienMmwhahaha: can you approve https://review.openstack.org/#/c/309189/3 please ?15:04
mjblackEmilienM: figured it out, has to do with supported platforms in the openstack spec helper15:06
EmilienMcool15:07
* mjblack feels pretty stupid 15:09
*** tesseract- has quit IRC15:09
openstackgerritMatthew J Black proposed openstack/puppet-ovn: Added params for debian packages  https://review.openstack.org/30794315:11
*** bnemec is now known as beekneemech15:13
mwhahahado we not have fuel-ci on puppet-swift?15:13
mwhahahaoh kilo15:14
EmilienMyea15:14
*** haplo37 has quit IRC15:14
*** haplo37 has joined #puppet-openstack15:15
openstackgerritDenis Egorenko proposed openstack/puppet-murano: Fix issue with importing packages with enabled Glare  https://review.openstack.org/30913915:23
*** dhollinger has quit IRC15:23
*** openstack has joined #puppet-openstack15:27
*** imcsk8|zZz is now known as imcsk815:28
openstackgerritMerged openstack/puppet-swift: unit tests: make Keystone_endpoint match service by name/type  https://review.openstack.org/30918915:30
openstackgerritMerged openstack/puppet-swift: swift-ring-builder 2.2.2+ output parse fix  https://review.openstack.org/30914115:30
*** johnmilton has quit IRC15:31
EmilienMmgagne: ^15:32
*** foxhunt72 is now known as foxhunt_away15:33
*** jmikolajczak has quit IRC15:38
*** dhollinger has joined #puppet-openstack15:39
*** ducttape_ has quit IRC15:41
*** ducttape_ has joined #puppet-openstack15:41
EmilienMdfisher: infra merged my patch, and going to run the script to create tarballs for tags. I'll ping you when they are here, should be in ~1 or 2 hours15:45
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/30790715:45
dfishernice!15:45
openstackgerritMerged openstack/puppet-ec2api: Changing default params for db/* and init.pp  https://review.openstack.org/30886315:47
*** johnmilton has joined #puppet-openstack15:52
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Use oslo module for logging and database configuration  https://review.openstack.org/30820715:54
*** MatUm has joined #puppet-openstack15:59
openstackgerritSergey Kolekonov proposed openstack/puppet-neutron: Use oslo module to configure oslo related parameters  https://review.openstack.org/30948316:02
openstackgerritMerged openstack/puppet-ovn: Implement basic beakers tests  https://review.openstack.org/30792016:03
*** jistr has quit IRC16:04
EmilienMmwhahaha, degorenko: how come we merge some patches without tests ? https://review.openstack.org/#/c/308863/16:06
mwhahahai missed it for the init, the rest were doc changes. pretty sure we need to redo init anyway to make it match more like the rest of our modules16:07
mwhahahasince ec2api is still pretty much a WIP16:08
EmilienMok16:08
EmilienMI don't want to end up with a module we consider WIP without any tests16:08
EmilienMthe minimum is beaker + unit tests16:08
degorenkomflobo, ^ add please tests for init.pp :)16:08
degorenkoi missed that too :(16:08
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Move databases creation and sync in role manifests  https://review.openstack.org/30945016:10
openstackgerritSergii Rizvan proposed openstack/puppet-keystone: Remove user/role prefetch to support multi-domain.  https://review.openstack.org/30836516:10
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Move databases creation and sync in role manifests  https://review.openstack.org/30945016:12
openstackgerritDenis Egorenko proposed openstack/puppet-murano: Fix issue with importing packages with enabled Glare  https://review.openstack.org/30913916:13
openstackgerritSergii Rizvan proposed openstack/puppet-keystone: Remove user/role prefetch to support multi-domain.  https://review.openstack.org/30836516:15
degorenkoEmilienM, mwhahaha guys, we have one backport to liberty ^^^ which can not be applied without conflicts and adaption. Can you please take a look on this?16:16
EmilienMdegorenko: sure, once CI is green I'll review it16:16
degorenkoEmilienM, thanks16:17
*** MatUm has quit IRC16:17
*** dhollinger has quit IRC16:19
*** rcernin has quit IRC16:19
*** gchamoul has quit IRC16:23
openstackgerritMerged openstack/puppet-openstacklib: Added policy-rc.d class.  https://review.openstack.org/30883316:29
*** dhollinger has joined #puppet-openstack16:33
*** tlbr has quit IRC16:35
*** tlbr has joined #puppet-openstack16:36
EmilienMcan someone approve https://review.openstack.org/#/c/309154/ please?16:41
degorenkoi wish, but i already putted +216:42
mjblackEmilienM: you're gonna make me rebase arent you... :P16:42
EmilienMmjblack: no, don't worry16:42
*** gchamoul has joined #puppet-openstack16:43
*** derekh has quit IRC16:43
mjblackEmilienM,degorenko: btw from the meeting where we wanted to filter log messages that we dont care about... https://review.openstack.org/#/c/304797/16:43
openstackgerritMerged openstack/puppet-nova: Stop overriding service provider  https://review.openstack.org/30915616:44
*** chandankumar has joined #puppet-openstack16:44
*** johnmilton has quit IRC16:45
*** johnmilton has joined #puppet-openstack16:45
EmilienMmjblack: where are used the text files?16:45
EmilienMI don't see them in the workspace16:46
EmilienMmaybe you'll have to patch https://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/copy_puppet_logs.sh16:46
degorenkoEmilienM, a new file is not collected by jenkins :)16:46
mjblackEmilienM: the new puppet_run.log?16:47
EmilienMI thought using --test --trace was enough16:47
mjblackEmilienM: well16:47
mjblackthe system in that change doesnt have to be for just debug16:47
mjblackit can be for any output from puppet we dont care about16:48
mjblackeven with test and trace ;)16:48
mjblackbut I'll put in a change for https://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/copy_puppet_logs.sh to copy the log too16:48
openstackgerritDenis Egorenko proposed openstack/puppet-murano: Fix issue with importing packages with enabled Glare  https://review.openstack.org/30913916:52
*** haplo37 has quit IRC16:58
openstackgerritMatthew J Black proposed openstack/puppet-openstack-integration: Filter debug messages.  https://review.openstack.org/30479717:00
degorenkoi'm off for today. EmilienM mwhahaha and others - see you in Austin, guys ;)17:02
EmilienMdegorenko: have a safe flight man17:02
EmilienMdegorenko: see you next week!!!!17:02
mwhahahalater17:03
degorenkoEmilienM, thanks :)17:03
*** degorenko is now known as _degorenko|afk17:03
*** mdavidson has quit IRC17:07
*** bmjen has quit IRC17:13
*** bodepd_ has quit IRC17:13
*** finchd has quit IRC17:13
*** jpena is now known as jpena|off17:17
*** bmjen has joined #puppet-openstack17:19
*** dtantsur is now known as dtantsur|afk17:19
*** finchd has joined #puppet-openstack17:19
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: CI test on stable/liberty  https://review.openstack.org/30952517:20
*** bodepd has joined #puppet-openstack17:22
*** haplo37 has joined #puppet-openstack17:28
*** mgould has quit IRC17:29
*** ansmith has quit IRC17:31
*** gfidente has quit IRC17:33
openstackgerritSergii Rizvan proposed openstack/puppet-keystone: Remove user/role prefetch to support multi-domain.  https://review.openstack.org/30836517:37
*** brad[] has quit IRC17:38
*** morazi has quit IRC17:39
*** brad[] has joined #puppet-openstack17:40
*** morazi has joined #puppet-openstack17:41
*** ansmith has joined #puppet-openstack17:46
*** ducttape_ has quit IRC17:49
*** francois1 has quit IRC17:54
*** francois has joined #puppet-openstack17:55
openstackgerritMerged openstack/puppet-ovn: Added params for debian packages  https://review.openstack.org/30794317:58
*** ducttape_ has joined #puppet-openstack18:03
EmilienMmwhahaha: err, a bit fast ^18:04
EmilienMI would like to see beaker tests, we already test on centos18:05
EmilienMmjblack: can you send a patch to test on ubuntu also? so we actually test this patch18:05
openstackgerritMerged openstack/puppet-keystone: Deprecate service_provider  https://review.openstack.org/30915418:06
openstackgerritEmilien Macchi proposed openstack/puppet-ovn: acceptance: test OVN on Ubuntu  https://review.openstack.org/30954218:10
EmilienMmjblack: this is a WIP, feel free to take over ^18:10
*** amoralej is now known as amoralej|off18:12
*** dgurtner has joined #puppet-openstack18:12
*** trown is now known as trown|lunch18:15
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: Prevent keystone eventlet from starting  https://review.openstack.org/30801218:15
EmilienMmjblack: rebased ^18:15
*** morazi has quit IRC18:15
*** darkxploit has quit IRC18:18
*** chandankumar has quit IRC18:28
*** morazi has joined #puppet-openstack18:29
*** darkxploit has joined #puppet-openstack18:30
*** ansmith has quit IRC18:32
*** darkxploit has quit IRC18:33
*** darkxploit has joined #puppet-openstack18:34
*** darkxploit has quit IRC18:37
*** ayoung has quit IRC18:38
*** darkxploit has joined #puppet-openstack18:38
mjblackEmilienM18:39
mjblackoops18:39
mjblackyou mean beaker test for ubuntu?18:39
EmilienMyes18:40
EmilienMsee my patch.18:40
mjblackEmilienM: ok, I'll work on it18:40
*** dgurtner has quit IRC18:41
*** darkxploit has quit IRC18:41
EmilienMmjblack: thanks18:42
*** chandankumar has joined #puppet-openstack18:43
*** ansmith has joined #puppet-openstack18:45
*** kgalanov has quit IRC18:54
*** kgalanov has joined #puppet-openstack18:57
*** darkxploit has joined #puppet-openstack18:59
*** paramite has joined #puppet-openstack19:11
*** chandankumar has quit IRC19:23
*** trown|lunch is now known as trown19:28
*** chandankumar has joined #puppet-openstack19:32
EmilienM_ody: I'm having a problem to build tarball for puppet-keystone19:35
EmilienMsomething with the metadata.json19:35
EmilienMcan you try and tell me if it works for you?19:36
crinkleEmilienM: http://git.openstack.org/cgit/openstack/puppet-keystone/tree/metadata.json#n38 is wrong19:41
EmilienMthanks19:41
*** twm2016 has joined #puppet-openstack19:42
EmilienMok, /me fixing stable/liberty CI and then metadata19:42
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Remove existed checkout before use  https://review.openstack.org/30957119:45
EmilienMcrinkle, mwhahaha: outstanding review to fix stable/liberty CI jobs, thanks in advance ^19:45
mwhahahak19:46
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: metadata.json: fix oslo module name  https://review.openstack.org/30957219:47
crinkleEmilienM: what problem is this fixing? why is CI running run_tests.sh twice?19:48
EmilienMhttp://logs.openstack.org/25/309525/1/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/b43f9a2/console.html#_2016-04-22_17_24_58_91319:48
*** johnmilton has quit IRC19:48
EmilienMinfra images already provide the rpm19:49
EmilienMwe fixed it in mitaka but never backported it19:49
*** chandankumar has quit IRC19:49
crinkleoh right19:49
openstackgerritEmilien Macchi proposed openstack/puppet-heat: metadata.json: fix oslo module name  https://review.openstack.org/30957519:50
EmilienMcore reviewers: please review the last 3 patches, thanks a lot19:51
EmilienMI checked http://codesearch.openstack.org/?q=openstack%2Fpuppet-oslo&i=nope&files=&repos=19:51
EmilienMand only puppet-heat & puppet-keystone are affected19:51
EmilienMmjblack: it sounds like we're close for ovn: http://logs.openstack.org/42/309542/1/check/gate-puppet-ovn-puppet-beaker-rspec-ubuntu-trusty/1f6dd45/console.html#_2016-04-22_18_28_02_39519:58
EmilienMthe second puppet run is not idempotent19:58
EmilienMit's probably northd & controller who fail to start19:58
*** paramite has quit IRC19:59
EmilienMlooking at process: http://logs.openstack.org/42/309542/1/check/gate-puppet-ovn-puppet-beaker-rspec-ubuntu-trusty/1f6dd45/logs/ps.txt.gz19:59
EmilienMit seems like ovn is running20:00
EmilienMso maybe the upstart script is not checking correctly20:00
EmilienMI'm going to patch infra to get /var/log/openvswitch/* logs20:00
*** ducttape_ has quit IRC20:09
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Deprecations and updates for Newton  https://review.openstack.org/30857320:12
mjblackEmilienM: heh, can you help push my infra change through too? :D20:12
openstackgerritNate Potter proposed openstack/puppet-magnum: Add magnum-conductor  https://review.openstack.org/28584720:12
EmilienMI'm not sure we need a dedicated file for puppet logs to be honnest20:13
mjblackEmilienM: if you want I can take it out, I figured that we may wanted the unfiltered logs20:13
mjblackjust in case something important got filtered out20:13
mjblackEmilienM: if we're sure that we wont want the unfiltered log, I'll amend the change I have to remove the tee20:16
mjblackor the very least the writing to log20:16
EmilienMdfisher: ok so we found a bug in a script I wrote for the tarball20:21
EmilienMthe tarballs with tags won't be created today20:21
dfisherok20:21
EmilienMI'm working on the fix now, but before it lands and get applied, we will wait20:21
EmilienMsorry20:21
dfisherno worries.20:21
mjblackEmilienM: oh god...I put a bug in with cinder about the wsgi stuff...and they added the line that pbr puts in now20:22
EmilienMmjblack: ?20:22
mjblackhttps://review.openstack.org/#/c/309375/20:22
mjblackEmilienM: ^20:22
mjblacklooks like CI is failing for centos20:28
mjblackhttp://logs.openstack.org/12/308012/21/gate/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/459c490/console.html#_2016-04-22_19_32_23_50820:29
EmilienMmjblack: which patch?20:30
mwhahahaif it errored why did it timeout, http://logs.openstack.org/12/308012/21/gate/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/459c490/console.html#_2016-04-22_20_20_27_67520:31
EmilienMweird, let's do recheck20:32
mwhahahaid id20:32
mwhahahai did20:32
mjblackmwhahaha: the reason it marked the build as a fail is rabbit service failed to start20:33
mjblackor rather looking at the syslog it starts but cant contact rabbit20:33
openstackgerritSergii Rizvan proposed openstack/puppet-keystone: Remove user/role prefetch to support multi-domain.  https://review.openstack.org/30836520:33
mjblackits not collecting the rabbit logs so dont know why it failed to start20:34
mwhahahayea but it should have error well before the 60min timeout20:36
mwhahahasomething weird went on with that one20:36
mwhahahawell see if it happens again20:36
mjblackmwhahaha: I forgot to sacrifice a chicken to zuul20:36
EmilienMdfisher: so during summit I'll poke infra folks but if they aren't too busy. Otherwise it will wait post-summit20:36
*** cdelatte has quit IRC20:36
mjblackmwhahaha: http://imgur.com/MXSI1J720:38
EmilienMmwhahaha: can you +2 https://review.openstack.org/#/c/309413/ please? I'll approve once the dependency has merged20:40
EmilienMso it's once race condition in less20:40
*** haplo37 has quit IRC20:41
mwhahahak20:41
*** ansmith has quit IRC20:41
EmilienMmwhahaha: and https://review.openstack.org/#/c/309571/ (please)20:41
EmilienMI think I'll stop here people will hate me for asking reviews20:41
mwhahahaha i had that one up waiting for ci20:41
*** haplo37 has joined #puppet-openstack20:42
*** ducttape_ has joined #puppet-openstack20:50
*** dhollinger has quit IRC20:51
openstackgerritMerged openstack/puppet-heat: metadata.json: fix oslo module name  https://review.openstack.org/30957520:54
*** morazi has quit IRC20:56
*** trown is now known as trown|outtypewww21:05
mwhahahaso it passed this time, so we'll see if it merges21:12
openstackgerritMerged openstack/puppet-keystone: metadata.json: fix oslo module name  https://review.openstack.org/30957221:14
openstackgerritMerged openstack/puppet-openstack-integration: Remove existed checkout before use  https://review.openstack.org/30957121:14
openstackgerritNate Potter proposed openstack/puppet-magnum: Update tests with rspec facts  https://review.openstack.org/30959621:19
mwhahahamjblack, EmilienM: looks like the ovn controller/northd are already running on ubuntu which is why beaker pukes21:19
EmilienMits not idempotent21:20
EmilienMso I guess it's a problem in the upstart script21:20
mwhahahayea21:20
EmilienMI'm out21:21
EmilienMsee you later21:21
mwhahahalater21:21
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: test - do not merge  https://review.openstack.org/30959921:24
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: test - do not merge  https://review.openstack.org/30959921:25
*** haplo37 has quit IRC21:30
*** haplo37 has joined #puppet-openstack21:32
openstackgerritNate Potter proposed openstack/puppet-magnum: Add magnum-conductor  https://review.openstack.org/28584721:46
*** ducttape_ has quit IRC21:57
openstackgerritAlex Schultz proposed openstack/puppet-ovn: Ubuntu acceptance tests and status workarounds  https://review.openstack.org/30954222:13
*** ducttape_ has joined #puppet-openstack22:13
*** dfisher has quit IRC22:25
*** dgurtner has joined #puppet-openstack22:29
openstackgerritMerged openstack/puppet-tripleo: Add missing services ports to service_ports map  https://review.openstack.org/30592922:34
openstackgerritMerged openstack/puppet-tripleo: Fix Sahara SSL default port  https://review.openstack.org/30930622:35
openstackgerritAlex Schultz proposed openstack/puppet-ovn: Ubuntu acceptance tests and status workarounds  https://review.openstack.org/30954222:35
*** xarses has quit IRC22:36
*** markvoelker has quit IRC22:39
*** imcsk8 has quit IRC22:39
openstackgerritMerged openstack/puppet-keystone: Prevent keystone eventlet from starting  https://review.openstack.org/30801222:44
*** ducttape_ has quit IRC22:45
openstackgerritAlex Schultz proposed openstack/puppet-ovn: Ubuntu acceptance tests and status workarounds  https://review.openstack.org/30954222:47
*** dgurtner has quit IRC22:51
*** twm2016 has quit IRC22:53
*** agrebennikov has quit IRC23:01
*** agrebennikov has joined #puppet-openstack23:05
*** xarses has joined #puppet-openstack23:06
openstackgerritMatthew J Black proposed openstack/puppet-openstacklib: Added policy-rc.d class.  https://review.openstack.org/30961623:09
*** markvoelker has joined #puppet-openstack23:09
openstackgerritMatthew J Black proposed openstack/puppet-openstacklib: Added policy-rc.d class.  https://review.openstack.org/30961723:09
mjblackmwhahaha: ^ those are cherry-picks to mitaka and liberty branch23:11
*** haplo37 has quit IRC23:14
*** haplo37 has joined #puppet-openstack23:16
mwhahahaK23:31
*** ducttape_ has joined #puppet-openstack23:50

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