*** markvoelker has joined #puppet-openstack | 00:19 | |
*** xarses has quit IRC | 00:21 | |
*** xarses has joined #puppet-openstack | 00:50 | |
*** ducttape_ has joined #puppet-openstack | 01:06 | |
*** ducttape_ has quit IRC | 01:32 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: unit tests: make Keystone_endpoint match service by name/type https://review.openstack.org/309236 | 01:55 |
---|---|---|
openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: Remove tenant parameter from keystone_user https://review.openstack.org/309189 | 01:56 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: swift-ring-builder 2.2.2+ output parse fix https://review.openstack.org/309141 | 01:56 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Deprecate service_provider https://review.openstack.org/309154 | 01:56 |
mtreinish | EmilienM: http://health.openstack.org/runs/key/project/openstack/puppet-keystone/recent/rss | 02:03 |
mgagne | just a reminder that Puppet facts (with 3.7 at least) are strings although they look to be returning boolean value :-/ | 02:07 |
mgagne | http://paste.openstack.org/show/495072/ =( | 02:07 |
mgagne | str2bool them :D | 02:07 |
*** richm has quit IRC | 02:32 | |
*** markvoelker has quit IRC | 02:35 | |
*** fedexo has joined #puppet-openstack | 02:52 | |
*** haplo37 has quit IRC | 03:30 | |
*** markvoelker has joined #puppet-openstack | 03:36 | |
*** markvoelker has quit IRC | 03:41 | |
*** mkarpin1 has quit IRC | 03:58 | |
*** mkarpin has joined #puppet-openstack | 03:58 | |
*** dtantsur|afk has quit IRC | 03:59 | |
*** dtantsur has joined #puppet-openstack | 04:03 | |
*** aarefiev has quit IRC | 04:16 | |
*** aarefiev has joined #puppet-openstack | 04:16 | |
openstackgerrit | Babu Shanmugam proposed openstack/puppet-ovn: Implement basic beakers tests https://review.openstack.org/307920 | 04:28 |
*** markvoelker has joined #puppet-openstack | 04:37 | |
*** markvoelker has quit IRC | 04:42 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Create keystone endpoints in keystone profile https://review.openstack.org/304125 | 04:51 |
*** saneax_AFK is now known as saneax | 04:56 | |
*** rcernin has joined #puppet-openstack | 05:03 | |
*** jaosorior has joined #puppet-openstack | 05:12 | |
*** dtantsur has quit IRC | 05:18 | |
*** jaosorior has quit IRC | 05:27 | |
*** dtantsur has joined #puppet-openstack | 05:36 | |
*** chandankumar has joined #puppet-openstack | 05:44 | |
*** jmikolajczak has joined #puppet-openstack | 05:53 | |
*** jtomasek has joined #puppet-openstack | 05:57 | |
*** dgurtner has joined #puppet-openstack | 06:01 | |
*** dgurtner has quit IRC | 06:01 | |
*** dgurtner has joined #puppet-openstack | 06:01 | |
*** dtantsur has quit IRC | 06:04 | |
*** tlbr has quit IRC | 06:05 | |
*** tlbr has joined #puppet-openstack | 06:05 | |
*** chandankumar has quit IRC | 06:07 | |
openstackgerrit | Sergii Golovatiuk proposed openstack/puppet-openstack-integration: Remove sticky version from concat https://review.openstack.org/307354 | 06:07 |
*** chandankumar has joined #puppet-openstack | 06:08 | |
*** dtantsur has joined #puppet-openstack | 06:08 | |
*** magicboiz has joined #puppet-openstack | 06:24 | |
*** magicboiz has left #puppet-openstack | 06:24 | |
*** v1k0d3n has quit IRC | 06:27 | |
*** v1k0d3n has joined #puppet-openstack | 06:27 | |
*** chandankumar has quit IRC | 06:29 | |
*** tesseract has joined #puppet-openstack | 06:43 | |
*** tesseract is now known as Guest83016 | 06:43 | |
*** Guest83016 has quit IRC | 06:48 | |
*** tesseract- has joined #puppet-openstack | 07:02 | |
*** dgurtner has quit IRC | 07:02 | |
*** foxhunt_away is now known as foxhunt72 | 07:02 | |
*** karimb has joined #puppet-openstack | 07:05 | |
*** fedexo has quit IRC | 07:15 | |
*** amoralej|off is now known as amoralej | 07: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-openstack | 07:32 | |
*** furlongm_ has joined #puppet-openstack | 07:32 | |
*** furlongm has quit IRC | 07:33 | |
*** dgurtner has joined #puppet-openstack | 07:46 | |
*** dgurtner has quit IRC | 07:46 | |
*** dgurtner has joined #puppet-openstack | 07:46 | |
*** shardy has joined #puppet-openstack | 07:55 | |
*** chandankumar has joined #puppet-openstack | 08:02 | |
*** gfidente has joined #puppet-openstack | 08:27 | |
*** gfidente has quit IRC | 08:27 | |
*** gfidente has joined #puppet-openstack | 08:27 | |
*** imcsk8 is now known as imcsk8|zZz | 08:27 | |
*** jistr has joined #puppet-openstack | 08:28 | |
*** jpena|off is now known as jpena | 08:36 | |
openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-heat: Drop all current deprecated parameters for heat https://review.openstack.org/308511 | 08:39 |
*** chem has quit IRC | 08:41 | |
*** chem has joined #puppet-openstack | 08:42 | |
*** Vinsh has quit IRC | 08:43 | |
*** derekh has joined #puppet-openstack | 08:46 | |
mattymo | anyone want to review? https://review.openstack.org/#/c/309103/ | 08:47 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add missing services ports to service_ports map https://review.openstack.org/305929 | 08:48 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Fix Sahara SSL default port https://review.openstack.org/309306 | 08:48 |
*** mgould has joined #puppet-openstack | 08:50 | |
*** chandankumar has quit IRC | 09:02 | |
*** chandankumar has joined #puppet-openstack | 09:05 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add missing services ports to service_ports map https://review.openstack.org/305929 | 09:18 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Fix Sahara SSL default port https://review.openstack.org/309306 | 09:18 |
openstackgerrit | Matthew Mosesohn proposed openstack/puppet-nova: Add nova_service type for purging nova services from offline hosts https://review.openstack.org/309103 | 09:18 |
*** darkxploit has quit IRC | 09:20 | |
*** dgurtner has quit IRC | 09:21 | |
*** dgurtner has joined #puppet-openstack | 09:22 | |
*** dgurtner has quit IRC | 09:22 | |
*** dgurtner has joined #puppet-openstack | 09:22 | |
*** openstackstatus has quit IRC | 09:24 | |
*** openstackstatus has joined #puppet-openstack | 09:26 | |
*** ChanServ sets mode: +v openstackstatus | 09:26 | |
*** furlongm_ has quit IRC | 09:28 | |
*** furlongm has joined #puppet-openstack | 09:29 | |
*** darkxploit has joined #puppet-openstack | 09:33 | |
openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-heat: Drop all current deprecated parameters for heat https://review.openstack.org/308511 | 09:34 |
*** chandankumar has quit IRC | 09:35 | |
*** jaosorior is now known as jaosorior_lunch | 09:35 | |
*** karimb has quit IRC | 09:47 | |
*** shardy has quit IRC | 09:54 | |
*** toMeloos has joined #puppet-openstack | 10:05 | |
*** karimb has joined #puppet-openstack | 10:15 | |
*** jaosorior_lunch is now known as jaosorior | 10:19 | |
*** beddari has quit IRC | 10:34 | |
*** beddari has joined #puppet-openstack | 10:34 | |
*** _degorenko|afk is now known as degorenko_ | 10:43 | |
*** chandankumar has joined #puppet-openstack | 10:45 | |
*** iurygregory has quit IRC | 10:46 | |
*** cdelatte has quit IRC | 10: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 IRC | 10:54 | |
openstackgerrit | Shaik Apsar proposed openstack/puppet-trove: Add amqp_durable_queues option in trove-guestagent.conf.erb. https://review.openstack.org/309355 | 10:56 |
*** jaosorior has joined #puppet-openstack | 10:56 | |
openstackgerrit | Shaik Apsar proposed openstack/puppet-trove: Add amqp_durable_queues option in trove-guestagent.conf.erb. https://review.openstack.org/309355 | 10:59 |
*** johnmilton has joined #puppet-openstack | 11:00 | |
openstackgerrit | Shaik Apsar proposed openstack/puppet-trove: Add amqp_durable_queues option in trove-guestagent.conf.erb. https://review.openstack.org/309355 | 11:02 |
*** toMeloos has quit IRC | 11:05 | |
*** ansmith has joined #puppet-openstack | 11:11 | |
*** cdelatte has joined #puppet-openstack | 11:15 | |
*** ansmith has quit IRC | 11:18 | |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker. https://review.openstack.org/309069 | 11:33 |
*** markvoelker has joined #puppet-openstack | 11:42 | |
openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-heat: Drop all current deprecated parameters for heat https://review.openstack.org/308511 | 11:45 |
*** markvoelker has quit IRC | 11:47 | |
*** paramite has joined #puppet-openstack | 11:47 | |
EmilienM | hello folks | 11:55 |
degorenko_ | hey o/ | 11:55 |
EmilienM | mtreinish: yeah!!!! | 11:55 |
*** dgurtner has quit IRC | 11:56 | |
mattymo | hey EmilienM | 11:58 |
mattymo | can you check out https://review.openstack.org/#/c/309103/ when you get a chance? | 11:58 |
*** paramite is now known as paramite|afk | 11:58 | |
EmilienM | mattymo: reviewed | 12:03 |
openstackgerrit | Merged openstack/puppet-tempest: Update spec tests for Tempest https://review.openstack.org/307853 | 12:05 |
*** dgurtner has joined #puppet-openstack | 12:06 | |
*** dgurtner has quit IRC | 12:06 | |
*** dgurtner has joined #puppet-openstack | 12:06 | |
*** ducttape_ has joined #puppet-openstack | 12:11 | |
*** paramite|afk is now known as paramite | 12:15 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: unit tests: make Keystone_endpoint match service by name/type https://review.openstack.org/309189 | 12:17 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: swift-ring-builder 2.2.2+ output parse fix https://review.openstack.org/309141 | 12:17 |
EmilienM | degorenko_, mgagne: ok I sorted the pupept-swift failures | 12:18 |
openstackgerrit | Merged openstack/puppet-openstack-integration: run_tests: add more debug when Puppet fails https://review.openstack.org/308991 | 12:18 |
EmilienM | we missed 3 cherry-picks, and 2 of them needed to be squashed | 12:18 |
openstackgerrit | Merged openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI https://review.openstack.org/307947 | 12:20 |
EmilienM | mtreinish: this http://health.openstack.org/runs/key/build_name/periodic-puppet-openstack-integration-4-scenario001-tempest-ubuntu-trusty/recent/rss | 12:21 |
EmilienM | does not work | 12:21 |
EmilienM | I'm trying to get RSS by build_name | 12:21 |
*** karimb has quit IRC | 12:23 | |
openstackgerrit | Merged openstack/puppet-keystone: Add Unit Tests for keystone_config type https://review.openstack.org/306644 | 12:24 |
*** morazi has joined #puppet-openstack | 12:25 | |
EmilienM | btw, 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-puppet | 12:25 |
*** karimb has joined #puppet-openstack | 12:28 | |
*** markvoelker has joined #puppet-openstack | 12:28 | |
*** chandankumar has quit IRC | 12:29 | |
*** ansmith has joined #puppet-openstack | 12:29 | |
*** dgurtner has quit IRC | 12:32 | |
*** ducttape_ has quit IRC | 12:39 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: CI test - never merge https://review.openstack.org/309387 | 12:40 |
*** jpena is now known as jpena|lunch | 12:41 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: CI test - never merge https://review.openstack.org/309387 | 12:41 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-horizon: CI test - never merge https://review.openstack.org/309388 | 12:42 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-horizon: CI test - never merge https://review.openstack.org/309388 | 12:42 |
*** trown|outtypewww is now known as trown | 12:47 | |
*** richm1 has joined #puppet-openstack | 12:51 | |
EmilienM | degorenko_: can you review https://review.openstack.org/#/c/309156/ again please? | 12:51 |
EmilienM | mjblack: hi, puppet-ovn / beaker is ready: https://review.openstack.org/#/c/307920/ | 12:53 |
EmilienM | mjblack: please update your patch for debian accordingly, thanks | 12:54 |
*** dgurtner has joined #puppet-openstack | 12:55 | |
*** dgurtner has quit IRC | 12:55 | |
*** dgurtner has joined #puppet-openstack | 12:55 | |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker. https://review.openstack.org/309069 | 12:55 |
*** jaosorior has quit IRC | 12:58 | |
*** karimb has quit IRC | 13:03 | |
*** mgould has quit IRC | 13:07 | |
degorenko_ | EmilienM, we can try do nothing in default: https://docs.puppet.com/guides/style_guide.html#defaults-for-case-statements-and-selectors | 13:12 |
EmilienM | degorenko_: ok Sir | 13:12 |
EmilienM | I'll do that | 13:12 |
degorenko_ | EmilienM, just try to test it locally | 13:12 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Stop overriding service provider https://review.openstack.org/309156 | 13:13 |
EmilienM | degorenko_: let's try ^ | 13:13 |
degorenko_ | EmilienM, wait - this one will not work | 13:14 |
degorenko_ | we can just have epmty {} | 13:14 |
EmilienM | degorenko_: this one will work | 13:14 |
degorenko_ | EmilienM, it will fail on fedora | 13:15 |
degorenko_ | EmilienM, also i think you can set it to undef | 13:16 |
degorenko_ | now we don't have value for default case for $messagebus_service_name | 13:17 |
*** chandankumar has joined #puppet-openstack | 13:17 | |
degorenko_ | so i think $messagebus_service_name = undef for default will be ok | 13:17 |
EmilienM | degorenko_: ok | 13:20 |
*** foxhunt72 is now known as foxhunt_away | 13:20 | |
EmilienM | I'm on it | 13:21 |
*** foxhunt_away is now known as foxhunt72 | 13:22 | |
degorenko_ | thanks | 13:24 |
EmilienM | degorenko_: wait, it will break fedora isn't? | 13:30 |
EmilienM | oh | 13:30 |
EmilienM | nevermind | 13:30 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Stop overriding service provider https://review.openstack.org/309156 | 13:31 |
EmilienM | degorenko_: done ^ | 13:31 |
openstackgerrit | James Slagle proposed openstack/puppet-tripleo: Add tripleo::selinux https://review.openstack.org/309408 | 13:32 |
degorenko_ | EmilienM, looking | 13: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-openstack | 13:37 | |
*** MatUm has quit IRC | 13:37 | |
*** amoralej is now known as amoralej|lunch | 13:40 | |
EmilienM | mjblack: excellent work on keystone/eventlet, thanks a lot | 13:40 |
*** degorenko has joined #puppet-openstack | 13:41 | |
EmilienM | degorenko: I'll let you whether or not approve his work | 13:42 |
EmilienM | https://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 this | 13:42 |
mwhahaha | i'll take a look in a bit | 13:43 |
degorenko_ | mjblack, i posted one comment :) | 13:44 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: keystone: avoid race condition with eventlet on Ubuntu. https://review.openstack.org/309413 | 13:44 |
EmilienM | I'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 |
EmilienM | degorenko_: you're right | 13:46 |
EmilienM | I dropped +2 | 13:46 |
*** mgould has joined #puppet-openstack | 13:47 | |
*** degorenko has quit IRC | 13:50 | |
*** degorenko has joined #puppet-openstack | 13:51 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka https://review.openstack.org/305886 | 13:53 |
*** jpena|lunch is now known as jpena | 13:53 | |
openstackgerrit | Merged openstack/puppet-tripleo: Add steps to database profiles https://review.openstack.org/307863 | 13:55 |
EmilienM | mtreinish: http://health.openstack.org/runs/key/build_name/periodic-nova-py34-with-oslo-master/recent/rss | 13:57 |
EmilienM | it works fine, when some failures | 13:57 |
EmilienM | where 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/rss | 13:57 |
*** chandankumar has quit IRC | 13:57 | |
EmilienM | mtreinish: it causes an issue with RSS client, where no feed is available, maybe we could add an empty feed if no failure | 14:00 |
EmilienM | I'll look into it | 14:00 |
*** degorenko_ sets mode: +o degorenko | 14:01 | |
*** degorenko_ has quit IRC | 14:02 | |
*** degorenko_ has joined #puppet-openstack | 14:03 | |
*** degorenko_ has quit IRC | 14:03 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Add aodh and gnocchi to schema profiles https://review.openstack.org/309424 | 14:05 |
*** degorenko_ has joined #puppet-openstack | 14:06 | |
*** degorenko_ has quit IRC | 14:07 | |
mjblack | EmilienM: I'll have the puppet-ovn change updated soon | 14:09 |
EmilienM | ++ | 14:09 |
*** paramite has quit IRC | 14:12 | |
*** xarses has quit IRC | 14:14 | |
*** ducttape_ has joined #puppet-openstack | 14:16 | |
*** dilyin has joined #puppet-openstack | 14:22 | |
*** haplo37 has joined #puppet-openstack | 14:22 | |
openstackgerrit | Matthew J Black proposed openstack/puppet-ovn: Added params for debian packages https://review.openstack.org/307943 | 14:33 |
degorenko | EmilienM, https://review.openstack.org/#/c/309413/ - passed ci | 14:34 |
EmilienM | degorenko: yeah, it's cool | 14:34 |
EmilienM | degorenko: we'll wait mjblack to update https://review.openstack.org/#/c/308012/ | 14:34 |
EmilienM | and we're good I think | 14:34 |
degorenko | yeah | 14:34 |
EmilienM | oh and https://review.openstack.org/#/c/308833/ | 14:35 |
mjblack | EmilienM: 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 |
EmilienM | yes | 14:37 |
EmilienM | I don't know how, but some pople might use a specific service_name | 14:37 |
mjblack | sure, I just wanted to be clear | 14:38 |
openstackgerrit | Matthew J Black proposed openstack/puppet-keystone: Prevent keystone eventlet from starting https://review.openstack.org/308012 | 14:39 |
mjblack | EmilienM,degorenko: ^ thats it | 14:39 |
openstackgerrit | Matthew J Black proposed openstack/puppet-keystone: Prevent keystone eventlet from starting https://review.openstack.org/308012 | 14:40 |
mjblack | sorry that is it | 14:40 |
mjblack | figured not to commit an extra a new line :D | 14:40 |
EmilienM | mjblack: also openstacklib please | 14:41 |
*** xarses has joined #puppet-openstack | 14:41 | |
openstackgerrit | Matthew J Black proposed openstack/puppet-openstacklib: Added policy-rc.d class. https://review.openstack.org/308833 | 14:41 |
mjblack | EmilienM: yeah was in process, ^ | 14:42 |
EmilienM | cool | 14:42 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Prevent keystone eventlet from starting https://review.openstack.org/308012 | 14:43 |
EmilienM | rebased ^ | 14:43 |
EmilienM | and https://review.openstack.org/#/c/309413/ rechecked | 14:43 |
EmilienM | let's see how all is going | 14:43 |
mjblack | EmilienM: with the puppet-ovn change, with rspec, I'm still seeing that issue where its not using the correct osfamily when it runs | 14:45 |
EmilienM | maybe mwhahaha can help, I'm not hyper familiar with it yet | 14:45 |
mjblack | http://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_292 | 14:46 |
*** amoralej|lunch is now known as amoralej | 14:50 | |
openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Call ::sync in addition to ::mysql https://review.openstack.org/309450 | 14:51 |
*** v1k0d3n has quit IRC | 14:51 | |
*** v1k0d3n has joined #puppet-openstack | 14:52 | |
openstackgerrit | Merged openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot https://review.openstack.org/307949 | 14:52 |
*** dhollinger has joined #puppet-openstack | 14:54 | |
EmilienM | dfisher: work to have tarball on each new tag: https://review.openstack.org/#/c/309453/1 | 14:55 |
EmilienM | dfisher: if it works, we'll generate tarballs for previous tags | 14:56 |
*** dgurtner has quit IRC | 14:57 | |
*** dilyin has quit IRC | 14:59 | |
*** saneax is now known as saneax_AFK | 15: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 | |
EmilienM | dfisher: 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.0 | 15:03 |
dfisher | outstanding. | 15:03 |
EmilienM | mwhahaha: can you approve https://review.openstack.org/#/c/309189/3 please ? | 15:04 |
mjblack | EmilienM: figured it out, has to do with supported platforms in the openstack spec helper | 15:06 |
EmilienM | cool | 15:07 |
* mjblack feels pretty stupid | 15:09 | |
*** tesseract- has quit IRC | 15:09 | |
openstackgerrit | Matthew J Black proposed openstack/puppet-ovn: Added params for debian packages https://review.openstack.org/307943 | 15:11 |
*** bnemec is now known as beekneemech | 15:13 | |
mwhahaha | do we not have fuel-ci on puppet-swift? | 15:13 |
mwhahaha | oh kilo | 15:14 |
EmilienM | yea | 15:14 |
*** haplo37 has quit IRC | 15:14 | |
*** haplo37 has joined #puppet-openstack | 15:15 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-murano: Fix issue with importing packages with enabled Glare https://review.openstack.org/309139 | 15:23 |
*** dhollinger has quit IRC | 15:23 | |
*** openstack has joined #puppet-openstack | 15:27 | |
*** imcsk8|zZz is now known as imcsk8 | 15:28 | |
openstackgerrit | Merged openstack/puppet-swift: unit tests: make Keystone_endpoint match service by name/type https://review.openstack.org/309189 | 15:30 |
openstackgerrit | Merged openstack/puppet-swift: swift-ring-builder 2.2.2+ output parse fix https://review.openstack.org/309141 | 15:30 |
*** johnmilton has quit IRC | 15:31 | |
EmilienM | mgagne: ^ | 15:32 |
*** foxhunt72 is now known as foxhunt_away | 15:33 | |
*** jmikolajczak has quit IRC | 15:38 | |
*** dhollinger has joined #puppet-openstack | 15:39 | |
*** ducttape_ has quit IRC | 15:41 | |
*** ducttape_ has joined #puppet-openstack | 15:41 | |
EmilienM | dfisher: 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 hours | 15:45 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-trove: Use oslo module for messaging (rabbit) configuration https://review.openstack.org/307907 | 15:45 |
dfisher | nice! | 15:45 |
openstackgerrit | Merged openstack/puppet-ec2api: Changing default params for db/* and init.pp https://review.openstack.org/308863 | 15:47 |
*** johnmilton has joined #puppet-openstack | 15:52 | |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-trove: Use oslo module for logging and database configuration https://review.openstack.org/308207 | 15:54 |
*** MatUm has joined #puppet-openstack | 15:59 | |
openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Use oslo module to configure oslo related parameters https://review.openstack.org/309483 | 16:02 |
openstackgerrit | Merged openstack/puppet-ovn: Implement basic beakers tests https://review.openstack.org/307920 | 16:03 |
*** jistr has quit IRC | 16:04 | |
EmilienM | mwhahaha, degorenko: how come we merge some patches without tests ? https://review.openstack.org/#/c/308863/ | 16:06 |
mwhahaha | i 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 modules | 16:07 |
mwhahaha | since ec2api is still pretty much a WIP | 16:08 |
EmilienM | ok | 16:08 |
EmilienM | I don't want to end up with a module we consider WIP without any tests | 16:08 |
EmilienM | the minimum is beaker + unit tests | 16:08 |
degorenko | mflobo, ^ add please tests for init.pp :) | 16:08 |
degorenko | i missed that too :( | 16:08 |
openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Move databases creation and sync in role manifests https://review.openstack.org/309450 | 16:10 |
openstackgerrit | Sergii Rizvan proposed openstack/puppet-keystone: Remove user/role prefetch to support multi-domain. https://review.openstack.org/308365 | 16:10 |
openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Move databases creation and sync in role manifests https://review.openstack.org/309450 | 16:12 |
openstackgerrit | Denis Egorenko proposed openstack/puppet-murano: Fix issue with importing packages with enabled Glare https://review.openstack.org/309139 | 16:13 |
openstackgerrit | Sergii Rizvan proposed openstack/puppet-keystone: Remove user/role prefetch to support multi-domain. https://review.openstack.org/308365 | 16:15 |
degorenko | EmilienM, 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 |
EmilienM | degorenko: sure, once CI is green I'll review it | 16:16 |
degorenko | EmilienM, thanks | 16:17 |
*** MatUm has quit IRC | 16:17 | |
*** dhollinger has quit IRC | 16:19 | |
*** rcernin has quit IRC | 16:19 | |
*** gchamoul has quit IRC | 16:23 | |
openstackgerrit | Merged openstack/puppet-openstacklib: Added policy-rc.d class. https://review.openstack.org/308833 | 16:29 |
*** dhollinger has joined #puppet-openstack | 16:33 | |
*** tlbr has quit IRC | 16:35 | |
*** tlbr has joined #puppet-openstack | 16:36 | |
EmilienM | can someone approve https://review.openstack.org/#/c/309154/ please? | 16:41 |
degorenko | i wish, but i already putted +2 | 16:42 |
mjblack | EmilienM: you're gonna make me rebase arent you... :P | 16:42 |
EmilienM | mjblack: no, don't worry | 16:42 |
*** gchamoul has joined #puppet-openstack | 16:43 | |
*** derekh has quit IRC | 16:43 | |
mjblack | EmilienM,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 |
openstackgerrit | Merged openstack/puppet-nova: Stop overriding service provider https://review.openstack.org/309156 | 16:44 |
*** chandankumar has joined #puppet-openstack | 16:44 | |
*** johnmilton has quit IRC | 16:45 | |
*** johnmilton has joined #puppet-openstack | 16:45 | |
EmilienM | mjblack: where are used the text files? | 16:45 |
EmilienM | I don't see them in the workspace | 16:46 |
EmilienM | maybe you'll have to patch https://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/copy_puppet_logs.sh | 16:46 |
degorenko | EmilienM, a new file is not collected by jenkins :) | 16:46 |
mjblack | EmilienM: the new puppet_run.log? | 16:47 |
EmilienM | I thought using --test --trace was enough | 16:47 |
mjblack | EmilienM: well | 16:47 |
mjblack | the system in that change doesnt have to be for just debug | 16:47 |
mjblack | it can be for any output from puppet we dont care about | 16:48 |
mjblack | even with test and trace ;) | 16:48 |
mjblack | but 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 too | 16:48 |
openstackgerrit | Denis Egorenko proposed openstack/puppet-murano: Fix issue with importing packages with enabled Glare https://review.openstack.org/309139 | 16:52 |
*** haplo37 has quit IRC | 16:58 | |
openstackgerrit | Matthew J Black proposed openstack/puppet-openstack-integration: Filter debug messages. https://review.openstack.org/304797 | 17:00 |
degorenko | i'm off for today. EmilienM mwhahaha and others - see you in Austin, guys ;) | 17:02 |
EmilienM | degorenko: have a safe flight man | 17:02 |
EmilienM | degorenko: see you next week!!!! | 17:02 |
mwhahaha | later | 17:03 |
degorenko | EmilienM, thanks :) | 17:03 |
*** degorenko is now known as _degorenko|afk | 17:03 | |
*** mdavidson has quit IRC | 17:07 | |
*** bmjen has quit IRC | 17:13 | |
*** bodepd_ has quit IRC | 17:13 | |
*** finchd has quit IRC | 17:13 | |
*** jpena is now known as jpena|off | 17:17 | |
*** bmjen has joined #puppet-openstack | 17:19 | |
*** dtantsur is now known as dtantsur|afk | 17:19 | |
*** finchd has joined #puppet-openstack | 17:19 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: CI test on stable/liberty https://review.openstack.org/309525 | 17:20 |
*** bodepd has joined #puppet-openstack | 17:22 | |
*** haplo37 has joined #puppet-openstack | 17:28 | |
*** mgould has quit IRC | 17:29 | |
*** ansmith has quit IRC | 17:31 | |
*** gfidente has quit IRC | 17:33 | |
openstackgerrit | Sergii Rizvan proposed openstack/puppet-keystone: Remove user/role prefetch to support multi-domain. https://review.openstack.org/308365 | 17:37 |
*** brad[] has quit IRC | 17:38 | |
*** morazi has quit IRC | 17:39 | |
*** brad[] has joined #puppet-openstack | 17:40 | |
*** morazi has joined #puppet-openstack | 17:41 | |
*** ansmith has joined #puppet-openstack | 17:46 | |
*** ducttape_ has quit IRC | 17:49 | |
*** francois1 has quit IRC | 17:54 | |
*** francois has joined #puppet-openstack | 17:55 | |
openstackgerrit | Merged openstack/puppet-ovn: Added params for debian packages https://review.openstack.org/307943 | 17:58 |
*** ducttape_ has joined #puppet-openstack | 18:03 | |
EmilienM | mwhahaha: err, a bit fast ^ | 18:04 |
EmilienM | I would like to see beaker tests, we already test on centos | 18:05 |
EmilienM | mjblack: can you send a patch to test on ubuntu also? so we actually test this patch | 18:05 |
openstackgerrit | Merged openstack/puppet-keystone: Deprecate service_provider https://review.openstack.org/309154 | 18:06 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-ovn: acceptance: test OVN on Ubuntu https://review.openstack.org/309542 | 18:10 |
EmilienM | mjblack: this is a WIP, feel free to take over ^ | 18:10 |
*** amoralej is now known as amoralej|off | 18:12 | |
*** dgurtner has joined #puppet-openstack | 18:12 | |
*** trown is now known as trown|lunch | 18:15 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Prevent keystone eventlet from starting https://review.openstack.org/308012 | 18:15 |
EmilienM | mjblack: rebased ^ | 18:15 |
*** morazi has quit IRC | 18:15 | |
*** darkxploit has quit IRC | 18:18 | |
*** chandankumar has quit IRC | 18:28 | |
*** morazi has joined #puppet-openstack | 18:29 | |
*** darkxploit has joined #puppet-openstack | 18:30 | |
*** ansmith has quit IRC | 18:32 | |
*** darkxploit has quit IRC | 18:33 | |
*** darkxploit has joined #puppet-openstack | 18:34 | |
*** darkxploit has quit IRC | 18:37 | |
*** ayoung has quit IRC | 18:38 | |
*** darkxploit has joined #puppet-openstack | 18:38 | |
mjblack | EmilienM | 18:39 |
mjblack | oops | 18:39 |
mjblack | you mean beaker test for ubuntu? | 18:39 |
EmilienM | yes | 18:40 |
EmilienM | see my patch. | 18:40 |
mjblack | EmilienM: ok, I'll work on it | 18:40 |
*** dgurtner has quit IRC | 18:41 | |
*** darkxploit has quit IRC | 18:41 | |
EmilienM | mjblack: thanks | 18:42 |
*** chandankumar has joined #puppet-openstack | 18:43 | |
*** ansmith has joined #puppet-openstack | 18:45 | |
*** kgalanov has quit IRC | 18:54 | |
*** kgalanov has joined #puppet-openstack | 18:57 | |
*** darkxploit has joined #puppet-openstack | 18:59 | |
*** paramite has joined #puppet-openstack | 19:11 | |
*** chandankumar has quit IRC | 19:23 | |
*** trown|lunch is now known as trown | 19:28 | |
*** chandankumar has joined #puppet-openstack | 19:32 | |
EmilienM | _ody: I'm having a problem to build tarball for puppet-keystone | 19:35 |
EmilienM | something with the metadata.json | 19:35 |
EmilienM | can you try and tell me if it works for you? | 19:36 |
crinkle | EmilienM: http://git.openstack.org/cgit/openstack/puppet-keystone/tree/metadata.json#n38 is wrong | 19:41 |
EmilienM | thanks | 19:41 |
*** twm2016 has joined #puppet-openstack | 19:42 | |
EmilienM | ok, /me fixing stable/liberty CI and then metadata | 19:42 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Remove existed checkout before use https://review.openstack.org/309571 | 19:45 |
EmilienM | crinkle, mwhahaha: outstanding review to fix stable/liberty CI jobs, thanks in advance ^ | 19:45 |
mwhahaha | k | 19:46 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: metadata.json: fix oslo module name https://review.openstack.org/309572 | 19:47 |
crinkle | EmilienM: what problem is this fixing? why is CI running run_tests.sh twice? | 19:48 |
EmilienM | http://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_913 | 19:48 |
*** johnmilton has quit IRC | 19:48 | |
EmilienM | infra images already provide the rpm | 19:49 |
EmilienM | we fixed it in mitaka but never backported it | 19:49 |
*** chandankumar has quit IRC | 19:49 | |
crinkle | oh right | 19:49 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-heat: metadata.json: fix oslo module name https://review.openstack.org/309575 | 19:50 |
EmilienM | core reviewers: please review the last 3 patches, thanks a lot | 19:51 |
EmilienM | I checked http://codesearch.openstack.org/?q=openstack%2Fpuppet-oslo&i=nope&files=&repos= | 19:51 |
EmilienM | and only puppet-heat & puppet-keystone are affected | 19:51 |
EmilienM | mjblack: 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_395 | 19:58 |
EmilienM | the second puppet run is not idempotent | 19:58 |
EmilienM | it's probably northd & controller who fail to start | 19:58 |
*** paramite has quit IRC | 19:59 | |
EmilienM | looking at process: http://logs.openstack.org/42/309542/1/check/gate-puppet-ovn-puppet-beaker-rspec-ubuntu-trusty/1f6dd45/logs/ps.txt.gz | 19:59 |
EmilienM | it seems like ovn is running | 20:00 |
EmilienM | so maybe the upstart script is not checking correctly | 20:00 |
EmilienM | I'm going to patch infra to get /var/log/openvswitch/* logs | 20:00 |
*** ducttape_ has quit IRC | 20:09 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Deprecations and updates for Newton https://review.openstack.org/308573 | 20:12 |
mjblack | EmilienM: heh, can you help push my infra change through too? :D | 20:12 |
openstackgerrit | Nate Potter proposed openstack/puppet-magnum: Add magnum-conductor https://review.openstack.org/285847 | 20:12 |
EmilienM | I'm not sure we need a dedicated file for puppet logs to be honnest | 20:13 |
mjblack | EmilienM: if you want I can take it out, I figured that we may wanted the unfiltered logs | 20:13 |
mjblack | just in case something important got filtered out | 20:13 |
mjblack | EmilienM: if we're sure that we wont want the unfiltered log, I'll amend the change I have to remove the tee | 20:16 |
mjblack | or the very least the writing to log | 20:16 |
EmilienM | dfisher: ok so we found a bug in a script I wrote for the tarball | 20:21 |
EmilienM | the tarballs with tags won't be created today | 20:21 |
dfisher | ok | 20:21 |
EmilienM | I'm working on the fix now, but before it lands and get applied, we will wait | 20:21 |
EmilienM | sorry | 20:21 |
dfisher | no worries. | 20:21 |
mjblack | EmilienM: oh god...I put a bug in with cinder about the wsgi stuff...and they added the line that pbr puts in now | 20:22 |
EmilienM | mjblack: ? | 20:22 |
mjblack | https://review.openstack.org/#/c/309375/ | 20:22 |
mjblack | EmilienM: ^ | 20:22 |
mjblack | looks like CI is failing for centos | 20:28 |
mjblack | http://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_508 | 20:29 |
EmilienM | mjblack: which patch? | 20:30 |
mwhahaha | if 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_675 | 20:31 |
EmilienM | weird, let's do recheck | 20:32 |
mwhahaha | id id | 20:32 |
mwhahaha | i did | 20:32 |
mjblack | mwhahaha: the reason it marked the build as a fail is rabbit service failed to start | 20:33 |
mjblack | or rather looking at the syslog it starts but cant contact rabbit | 20:33 |
openstackgerrit | Sergii Rizvan proposed openstack/puppet-keystone: Remove user/role prefetch to support multi-domain. https://review.openstack.org/308365 | 20:33 |
mjblack | its not collecting the rabbit logs so dont know why it failed to start | 20:34 |
mwhahaha | yea but it should have error well before the 60min timeout | 20:36 |
mwhahaha | something weird went on with that one | 20:36 |
mwhahaha | well see if it happens again | 20:36 |
mjblack | mwhahaha: I forgot to sacrifice a chicken to zuul | 20:36 |
EmilienM | dfisher: so during summit I'll poke infra folks but if they aren't too busy. Otherwise it will wait post-summit | 20:36 |
*** cdelatte has quit IRC | 20:36 | |
mjblack | mwhahaha: http://imgur.com/MXSI1J7 | 20:38 |
EmilienM | mwhahaha: can you +2 https://review.openstack.org/#/c/309413/ please? I'll approve once the dependency has merged | 20:40 |
EmilienM | so it's once race condition in less | 20:40 |
*** haplo37 has quit IRC | 20:41 | |
mwhahaha | k | 20:41 |
*** ansmith has quit IRC | 20:41 | |
EmilienM | mwhahaha: and https://review.openstack.org/#/c/309571/ (please) | 20:41 |
EmilienM | I think I'll stop here people will hate me for asking reviews | 20:41 |
mwhahaha | ha i had that one up waiting for ci | 20:41 |
*** haplo37 has joined #puppet-openstack | 20:42 | |
*** ducttape_ has joined #puppet-openstack | 20:50 | |
*** dhollinger has quit IRC | 20:51 | |
openstackgerrit | Merged openstack/puppet-heat: metadata.json: fix oslo module name https://review.openstack.org/309575 | 20:54 |
*** morazi has quit IRC | 20:56 | |
*** trown is now known as trown|outtypewww | 21:05 | |
mwhahaha | so it passed this time, so we'll see if it merges | 21:12 |
openstackgerrit | Merged openstack/puppet-keystone: metadata.json: fix oslo module name https://review.openstack.org/309572 | 21:14 |
openstackgerrit | Merged openstack/puppet-openstack-integration: Remove existed checkout before use https://review.openstack.org/309571 | 21:14 |
openstackgerrit | Nate Potter proposed openstack/puppet-magnum: Update tests with rspec facts https://review.openstack.org/309596 | 21:19 |
mwhahaha | mjblack, EmilienM: looks like the ovn controller/northd are already running on ubuntu which is why beaker pukes | 21:19 |
EmilienM | its not idempotent | 21:20 |
EmilienM | so I guess it's a problem in the upstart script | 21:20 |
mwhahaha | yea | 21:20 |
EmilienM | I'm out | 21:21 |
EmilienM | see you later | 21:21 |
mwhahaha | later | 21:21 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: test - do not merge https://review.openstack.org/309599 | 21:24 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: test - do not merge https://review.openstack.org/309599 | 21:25 |
*** haplo37 has quit IRC | 21:30 | |
*** haplo37 has joined #puppet-openstack | 21:32 | |
openstackgerrit | Nate Potter proposed openstack/puppet-magnum: Add magnum-conductor https://review.openstack.org/285847 | 21:46 |
*** ducttape_ has quit IRC | 21:57 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-ovn: Ubuntu acceptance tests and status workarounds https://review.openstack.org/309542 | 22:13 |
*** ducttape_ has joined #puppet-openstack | 22:13 | |
*** dfisher has quit IRC | 22:25 | |
*** dgurtner has joined #puppet-openstack | 22:29 | |
openstackgerrit | Merged openstack/puppet-tripleo: Add missing services ports to service_ports map https://review.openstack.org/305929 | 22:34 |
openstackgerrit | Merged openstack/puppet-tripleo: Fix Sahara SSL default port https://review.openstack.org/309306 | 22:35 |
openstackgerrit | Alex Schultz proposed openstack/puppet-ovn: Ubuntu acceptance tests and status workarounds https://review.openstack.org/309542 | 22:35 |
*** xarses has quit IRC | 22:36 | |
*** markvoelker has quit IRC | 22:39 | |
*** imcsk8 has quit IRC | 22:39 | |
openstackgerrit | Merged openstack/puppet-keystone: Prevent keystone eventlet from starting https://review.openstack.org/308012 | 22:44 |
*** ducttape_ has quit IRC | 22:45 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-ovn: Ubuntu acceptance tests and status workarounds https://review.openstack.org/309542 | 22:47 |
*** dgurtner has quit IRC | 22:51 | |
*** twm2016 has quit IRC | 22:53 | |
*** agrebennikov has quit IRC | 23:01 | |
*** agrebennikov has joined #puppet-openstack | 23:05 | |
*** xarses has joined #puppet-openstack | 23:06 | |
openstackgerrit | Matthew J Black proposed openstack/puppet-openstacklib: Added policy-rc.d class. https://review.openstack.org/309616 | 23:09 |
*** markvoelker has joined #puppet-openstack | 23:09 | |
openstackgerrit | Matthew J Black proposed openstack/puppet-openstacklib: Added policy-rc.d class. https://review.openstack.org/309617 | 23:09 |
mjblack | mwhahaha: ^ those are cherry-picks to mitaka and liberty branch | 23:11 |
*** haplo37 has quit IRC | 23:14 | |
*** haplo37 has joined #puppet-openstack | 23:16 | |
mwhahaha | K | 23:31 |
*** ducttape_ has joined #puppet-openstack | 23:50 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!