| *** yottatsa has joined #puppet-openstack | 00:06 | |
| *** xarses has quit IRC | 00:09 | |
| *** yottatsa has quit IRC | 00:13 | |
| *** baker has joined #puppet-openstack | 00:18 | |
| *** vpati011_ has joined #puppet-openstack | 00:23 | |
| vpati011_ | Hi Denis, I have addressed your comments for https://review.openstack.org/#/c/247239 | 00:23 |
|---|---|---|
| *** nlahouti has quit IRC | 00:24 | |
| *** nlahouti has joined #puppet-openstack | 00:24 | |
| *** vpati011_ has quit IRC | 00:24 | |
| *** nlahouti1 has joined #puppet-openstack | 00:38 | |
| *** xarses has joined #puppet-openstack | 00:39 | |
| *** nlahouti has quit IRC | 00:40 | |
| *** nlahouti1 has quit IRC | 00:44 | |
| *** saneax is now known as saneax_AFK | 00:44 | |
| *** nlahouti has joined #puppet-openstack | 00:46 | |
| *** zhangjn has joined #puppet-openstack | 00:59 | |
| *** baker has quit IRC | 01:00 | |
| *** RichardRaseley has quit IRC | 01:00 | |
| *** baker has joined #puppet-openstack | 01:04 | |
| *** zhangjn_ has joined #puppet-openstack | 01:05 | |
| *** nlahouti has quit IRC | 01:07 | |
| *** zhangjn has quit IRC | 01:08 | |
| EmilienM | dmsimard: yes | 01:14 |
| EmilienM | dmsimard: I think https://review.openstack.org/#/c/253416 fix it | 01:15 |
| EmilienM | or not https://review.openstack.org/#/c/256060/ | 01:15 |
| *** cwolferh has quit IRC | 01:19 | |
| *** rahulait has joined #puppet-openstack | 01:24 | |
| *** rahulait has quit IRC | 01:25 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-cookiecutter: Puppet cookiecutter with $::os_service_default https://review.openstack.org/256176 | 01:59 |
| *** morazi has joined #puppet-openstack | 01:59 | |
| *** 5EXAAFXI5 has quit IRC | 02:01 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-cookiecutter: Puppet cookiecutter with $::os_service_default https://review.openstack.org/256176 | 02:03 |
| iurygregory | EmilienM, ^ | 02:05 |
| EmilienM | iurygregory: thhanks! | 02:05 |
| EmilienM | I'll look tomorrow | 02:05 |
| iurygregory | nice =D | 02:05 |
| iurygregory | wow you are wake up O.o | 02:05 |
| EmilienM | iurygregory: how was webinar? | 02:05 |
| EmilienM | iurygregory: 9pm | 02:05 |
| iurygregory | very nice =) | 02:06 |
| iurygregory | here is 11pm =P | 02:06 |
| EmilienM | iurygregory: see you tomorrow | 02:07 |
| iurygregory | bye o/ | 02:07 |
| *** baker has quit IRC | 02:34 | |
| *** zhangjn_ has quit IRC | 02:51 | |
| *** zhangjn has joined #puppet-openstack | 03:00 | |
| *** baker has joined #puppet-openstack | 03:24 | |
| *** baker has quit IRC | 03:47 | |
| *** morazi has quit IRC | 03:56 | |
| *** zhangjn has quit IRC | 03:56 | |
| *** baker has joined #puppet-openstack | 04:01 | |
| *** richm has quit IRC | 04:08 | |
| *** zhangjn has joined #puppet-openstack | 04:19 | |
| *** ducttape_ has joined #puppet-openstack | 04:36 | |
| *** ducttape_ has quit IRC | 04:41 | |
| *** jaosorior has quit IRC | 04:51 | |
| *** jaosorior has joined #puppet-openstack | 04:51 | |
| *** saneax_AFK is now known as saneax | 04:56 | |
| *** zhangjn has quit IRC | 04:57 | |
| *** nlahouti has joined #puppet-openstack | 05:18 | |
| *** zhangjn has joined #puppet-openstack | 05:28 | |
| *** nlahouti has quit IRC | 05:34 | |
| *** nlahouti has joined #puppet-openstack | 05:45 | |
| *** nlahouti1 has joined #puppet-openstack | 05:48 | |
| openstackgerrit | Merged openstack/puppet-nova: Rewrite nova_network provider with using only nova client https://review.openstack.org/229548 | 05:49 |
| *** nlahouti has quit IRC | 05:51 | |
| *** simonmcc has quit IRC | 06:27 | |
| *** simonmcc has joined #puppet-openstack | 06:27 | |
| gildub | core member, please review https://review.openstack.org/245643 | 06:38 |
| *** saneax is now known as saneax_AFK | 06:47 | |
| openstackgerrit | Colleen Murphy proposed openstack/puppet-keystone: Add keystone::pipeline class to secure keystone https://review.openstack.org/256248 | 06:54 |
| *** gildub has quit IRC | 07:00 | |
| *** nlahouti1 has quit IRC | 07:00 | |
| *** nlahouti has joined #puppet-openstack | 07:05 | |
| *** nlahouti has quit IRC | 07:06 | |
| *** nlahouti has joined #puppet-openstack | 07:09 | |
| *** saneax_AFK is now known as saneax | 07:11 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add hiera calls to allow management of ports and protocols https://review.openstack.org/240208 | 07:13 |
| *** cwolferh has joined #puppet-openstack | 07:22 | |
| *** nlahouti has quit IRC | 07:49 | |
| *** mesamoo has joined #puppet-openstack | 07:57 | |
| *** jaosorior has quit IRC | 08:04 | |
| *** sleinen-AS5591 has quit IRC | 08:06 | |
| *** dgurtner has joined #puppet-openstack | 08:19 | |
| *** dgurtner has quit IRC | 08:19 | |
| *** dgurtner has joined #puppet-openstack | 08:19 | |
| *** jpena has joined #puppet-openstack | 08:23 | |
| *** jaosorior has joined #puppet-openstack | 08:31 | |
| *** toMeloos has joined #puppet-openstack | 08:43 | |
| *** rcernin has joined #puppet-openstack | 08:44 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add hiera calls to allow management of ports and protocols https://review.openstack.org/240208 | 08:44 |
| skolekonov | EmilienM, looking | 08:59 |
| *** rochaporto has joined #puppet-openstack | 09:01 | |
| *** foxhunt_away is now known as foxhunt72 | 09:37 | |
| *** _degorenko|afk is now known as degorenko | 09:39 | |
| *** igajsin has left #puppet-openstack | 09:45 | |
| *** jistr has joined #puppet-openstack | 09:55 | |
| *** zhangjn has quit IRC | 10:11 | |
| *** davideagle_ has joined #puppet-openstack | 10:17 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Move Neutron to $::os_service_default - part 2 https://review.openstack.org/253416 | 10:46 |
| openstackgerrit | Merged openstack/puppet-swift: swift-ring-builder 2.2.2+ output parse fix https://review.openstack.org/256054 | 10:47 |
| *** figless has joined #puppet-openstack | 10:49 | |
| *** figless has left #puppet-openstack | 10:54 | |
| *** admin0 has joined #puppet-openstack | 11:11 | |
| *** zhangjn has joined #puppet-openstack | 11:13 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Fix backward compatibility for some parameters https://review.openstack.org/256370 | 11:34 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-tempest: CI test - do not merge https://review.openstack.org/256060 | 11:45 |
| openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency https://review.openstack.org/251319 | 11:46 |
| openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Remove all 'validate_array' statements https://review.openstack.org/250763 | 11:46 |
| openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Adding MidoNet LoadBalancing options https://review.openstack.org/250777 | 11:47 |
| iurygregory | degorenko, i think i've correct https://review.openstack.org/#/c/250945/ =) | 12:11 |
| degorenko | iurygregory, lgtm, have you checked working of memcache and cache parameters without cache enabled true and memcached_servers? | 12:15 |
| iurygregory | i've tested based on conditions, according to the documentation for each parameter they only have effect if memecache_servers is set, so there is no problem to put them after check this =) | 12:17 |
| *** rcernin has quit IRC | 12:18 | |
| degorenko | ok, i will double review your change | 12:18 |
| degorenko | skolekonov, ^ you can also review it :) | 12:18 |
| iurygregory | Tks o/ | 12:18 |
| iurygregory | tks guys =) | 12:18 |
| *** degorenko is now known as _degorenko|afk | 12:24 | |
| *** admin0 has quit IRC | 12:28 | |
| openstackgerrit | Andrey Shestakov proposed openstack/puppet-neutron: Add 'distributed' parameter for neutron_router https://review.openstack.org/255913 | 12:32 |
| *** dprince has joined #puppet-openstack | 12:39 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Set defaultbranch in .gitreview to stable/liberty https://review.openstack.org/255919 | 12:45 |
| *** cdelatte has joined #puppet-openstack | 12:49 | |
| iurygregory | spredzy, _degorenko|afk can you review this patch https://review.openstack.org/#/c/239800/ ? (switch to os_service_default) the owner is Alex =) | 12:51 |
| *** jaosorior has quit IRC | 12:52 | |
| *** jaosorior has joined #puppet-openstack | 12:52 | |
| *** gfidente has joined #puppet-openstack | 12:54 | |
| *** gfidente has quit IRC | 12:54 | |
| *** gfidente has joined #puppet-openstack | 12:54 | |
| *** jaosorior has quit IRC | 12:56 | |
| EmilienM | good morning! | 12:56 |
| iurygregory | morning o/ | 12:57 |
| *** _degorenko|afk is now known as degorenko | 12:58 | |
| degorenko | morning :) | 12:58 |
| *** sleinen-AS559 has joined #puppet-openstack | 13:03 | |
| *** ducttape_ has joined #puppet-openstack | 13:05 | |
| *** sleinen-AS559 has quit IRC | 13:07 | |
| *** jistr is now known as jistr|biab | 13:10 | |
| EmilienM | nice work degorenko and skolekonov | 13:19 |
| skolekonov | np | 13:20 |
| *** baker has quit IRC | 13:26 | |
| *** ducttape_ has quit IRC | 13:27 | |
| *** paramite has joined #puppet-openstack | 13:37 | |
| iurygregory | EmilienM, https://review.openstack.org/#/c/256176/ o/ | 13:39 |
| openstackgerrit | Merged openstack/puppet-openstack-cookiecutter: Puppet cookiecutter with $::os_service_default https://review.openstack.org/256176 | 13:40 |
| iurygregory | wow O.o | 13:40 |
| iurygregory | the fastest merge i've seen | 13:41 |
| *** bklei has quit IRC | 13:43 | |
| *** bklei has joined #puppet-openstack | 13:43 | |
| *** derekh has joined #puppet-openstack | 13:45 | |
| mflobo | EmilienM, If could have some time to this governance patch https://review.openstack.org/#/c/252959/ :) | 13:46 |
| *** dprince has quit IRC | 13:46 | |
| mflobo | EmilienM, thanks :) | 13:48 |
| *** bklei has quit IRC | 13:48 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/puppet-swift: Require swift package installation before swift user. https://review.openstack.org/256415 | 13:48 |
| *** samuelBartel has joined #puppet-openstack | 13:49 | |
| openstackgerrit | Merged openstack/puppet-neutron: Fix backward compatibility for some parameters https://review.openstack.org/256370 | 13:49 |
| *** dprince has joined #puppet-openstack | 13:50 | |
| *** dprince_ has joined #puppet-openstack | 14:02 | |
| *** admin0 has joined #puppet-openstack | 14:04 | |
| *** richm has joined #puppet-openstack | 14:07 | |
| *** jistr|biab is now known as jistr | 14:09 | |
| *** darkxploit has quit IRC | 14:10 | |
| *** derekh has quit IRC | 14:11 | |
| *** paramite is now known as paramite|afk | 14:16 | |
| *** morazi has joined #puppet-openstack | 14:17 | |
| *** dprince_ has quit IRC | 14:21 | |
| *** dprince_ has joined #puppet-openstack | 14:21 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Move Neutron to $::os_service_default - part 2 https://review.openstack.org/253416 | 14:22 |
| openstackgerrit | Maty Grosz proposed openstack/puppet-vitrage: puppet-vitrage: Initial commit https://review.openstack.org/255431 | 14:23 |
| *** dprince_ has quit IRC | 14:26 | |
| openstackgerrit | Merged openstack/puppet-tempest: Switch id setters to openstack client https://review.openstack.org/242052 | 14:33 |
| iurygregory | skolekonov, have you looked my comments in patch 6? o/ | 14:36 |
| skolekonov | iurygregory, yep, thank you. I've rebased the patch and then noticed it. I think I'll update this parameter to os_service_default as well | 14:37 |
| iurygregory | thanks o/ | 14:37 |
| *** mgrosz has joined #puppet-openstack | 14:43 | |
| *** darkxploit has joined #puppet-openstack | 14:45 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Move Neutron to $::os_service_default - part 2 https://review.openstack.org/253416 | 14:48 |
| *** dfisher has joined #puppet-openstack | 14:48 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/puppet-trove: Use swift_url variable in the template instead of the hardcoded url. https://review.openstack.org/256444 | 14:49 |
| dmsimard | EmilienM: is tempest fixed now? :) | 14:50 |
| EmilienM | dmsimard: yes | 14:50 |
| dmsimard | EmilienM: k I'll try | 14:51 |
| openstackgerrit | Andrey Shestakov proposed openstack/puppet-neutron: Add 'distributed' parameter for neutron_router https://review.openstack.org/255913 | 14:54 |
| *** ferest has joined #puppet-openstack | 14:54 | |
| mgrosz | EmilienM: Hey, can you please proceed with reviewing https://review.openstack.org/#/c/255431? Thanks | 14:55 |
| mgrosz | I finally got Jenkins not to fail... | 14:55 |
| iurygregory | nice =) i'm reviewing it | 14:56 |
| *** ferest_ has joined #puppet-openstack | 14:56 | |
| *** xarses has quit IRC | 14:57 | |
| *** ferest has quit IRC | 15:00 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/puppet-openstacklib: Fix unit tests failing against Puppet 4.3.0 https://review.openstack.org/256452 | 15:00 |
| EmilienM | iurygregory: is it compliant with https://review.openstack.org/#/c/256176/ ? | 15:01 |
| iurygregory | EmilienM, yes | 15:04 |
| *** ferest_ has quit IRC | 15:06 | |
| iurygregory | EmilienM, wait, he can remove the picks in db.pp | 15:06 |
| *** mdavidson has quit IRC | 15:06 | |
| *** bklei has joined #puppet-openstack | 15:07 | |
| *** bklei has quit IRC | 15:07 | |
| iurygregory | mgrosz, just one more =) | 15:08 |
| *** bklei has joined #puppet-openstack | 15:08 | |
| mkarpin | @Core guys, please review https://review.openstack.org/#/c/249711/ heat os_service_default | 15:08 |
| mkarpin | tnx! | 15:08 |
| mgrosz | iurygregory: mmmmm.... yes I saw... | 15:09 |
| iurygregory | sorry =) | 15:09 |
| *** iurygregory is now known as iury|lunch | 15:11 | |
| openstackgerrit | Maty Grosz proposed openstack/puppet-vitrage: puppet-vitrage: Initial commit https://review.openstack.org/255431 | 15:12 |
| *** admin0 has quit IRC | 15:13 | |
| *** baker has joined #puppet-openstack | 15:15 | |
| mgrosz | $database_connection_real = pick($::vitrage::database_connection, $database_connection) | 15:16 |
| mgrosz | $database_idle_timeout_real = pick($::vitrage::database_idle_timeout, $database_idle_timeout) | 15:16 |
| mgrosz | $database_min_pool_size_real = pick($::vitrage::database_min_pool_size, $database_min_pool_size) | 15:16 |
| mgrosz | $database_max_pool_size_real = pick($::vitrage::database_max_pool_size, $database_max_pool_size) | 15:16 |
| mgrosz | $database_max_retries_real = pick($::vitrage::database_max_retries, $database_max_retries) | 15:16 |
| mgrosz | $database_retry_interval_real = pick($::vitrage::database_retry_interval, $database_retry_interval) | 15:16 |
| mgrosz | iurygregory, EmilienM: sent a fix. Please review.... thanks | 15:16 |
| EmilienM | mgrosz: I have notifications of each patch in puppet-openstack | 15:17 |
| EmilienM | but thank you | 15:17 |
| *** dhollinger has joined #puppet-openstack | 15:18 | |
| EmilienM | we have 11 core reviewers https://review.openstack.org/#/admin/groups/134,members | 15:18 |
| EmilienM | just don't ping me all the time :) | 15:18 |
| *** ducttape_ has joined #puppet-openstack | 15:19 | |
| EmilienM | we documented how to ask for review https://wiki.openstack.org/wiki/Puppet/Coding_style#Asking_for_review | 15:20 |
| *** _ducttape_ has joined #puppet-openstack | 15:24 | |
| *** bklei has quit IRC | 15:25 | |
| *** ducttape_ has quit IRC | 15:25 | |
| mgrosz | iurygregory: Now my push failed... | 15:27 |
| mgrosz | iurygregory : Now my push failed... | 15:27 |
| *** ferest has joined #puppet-openstack | 15:28 | |
| *** saneax is now known as saneax_AFK | 15:30 | |
| *** xarses has joined #puppet-openstack | 15:32 | |
| *** mgrosz has quit IRC | 15:33 | |
| *** admin0 has joined #puppet-openstack | 15:42 | |
| *** dprince has quit IRC | 15:43 | |
| *** ntpttr has joined #puppet-openstack | 15:43 | |
| *** dprince has joined #puppet-openstack | 15:43 | |
| *** paramite|afk is now known as paramite | 15:44 | |
| *** ducttape_ has joined #puppet-openstack | 15:47 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-keystone: comment: note how to unset version w. empty string https://review.openstack.org/255639 | 15:47 |
| *** _ducttape_ has quit IRC | 15:49 | |
| *** nlahouti has joined #puppet-openstack | 15:54 | |
| *** nlahouti1 has joined #puppet-openstack | 15:56 | |
| *** ferest has quit IRC | 15:57 | |
| *** _ducttape_ has joined #puppet-openstack | 15:58 | |
| *** nlahouti has quit IRC | 15:59 | |
| degorenko | iury|lunch, hey, can you create test commit to puppet-tempest based on your os_service_default switch for keystone? We already faced with problem on neutron, and now we must be sure that patch doesn't break anything | 16:01 |
| *** ducttape_ has quit IRC | 16:01 | |
| degorenko | mkarpin, can you also please do the same for heat ^^^ | 16:01 |
| *** nlahouti1 has quit IRC | 16:02 | |
| degorenko | skolekonov, you too please | 16:04 |
| skolekonov | sure | 16:04 |
| mkarpin | ok | 16:04 |
| *** dprince has quit IRC | 16:05 | |
| *** yottatsa has joined #puppet-openstack | 16:07 | |
| *** yottatsa has quit IRC | 16:08 | |
| *** dprince has joined #puppet-openstack | 16:08 | |
| *** yottatsa has joined #puppet-openstack | 16:13 | |
| openstackgerrit | Merged openstack/puppet-murano: Fix service and package tags https://review.openstack.org/255283 | 16:16 |
| *** mdavidson has joined #puppet-openstack | 16:19 | |
| *** iury|lunch is now known as iurygregory | 16:19 | |
| iurygregory | degorenko, i'll do. i just need to use de depends on right? | 16:20 |
| degorenko | iurygregory, yea :) fix some docs and use depends-on | 16:23 |
| *** _ducttape_ has quit IRC | 16:28 | |
| openstackgerrit | Merged openstack/puppet-murano: Run db sync only from main script https://review.openstack.org/254809 | 16:29 |
| *** admin0 has quit IRC | 16:29 | |
| *** yottatsa has quit IRC | 16:30 | |
| *** darkxploit has quit IRC | 16:30 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-cookiecutter: switch database_connection_real https://review.openstack.org/256570 | 16:31 |
| iurygregory | degorenko, ^ | 16:31 |
| *** foxhunt72 is now known as foxhunt_away | 16:32 | |
| *** yottatsa has joined #puppet-openstack | 16:32 | |
| degorenko | iurygregory, EmilienM we definitely need some simple tests for cookiecutter :D to avoid typo errors | 16:33 |
| iurygregory | agree XD | 16:33 |
| EmilienM | yes | 16:33 |
| EmilienM | we need a bash script | 16:33 |
| EmilienM | that create a fake module | 16:33 |
| EmilienM | puppet-fake! | 16:33 |
| EmilienM | and a job for that | 16:33 |
| degorenko | something like that, yeah | 16:33 |
| iurygregory | puppet-fake kkkkkkkkkkkk | 16:33 |
| EmilienM | I'm adding that do the sprint agenda | 16:34 |
| degorenko | cool | 16:34 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-tempest: CI test - do not merge https://review.openstack.org/256576 | 16:41 |
| iurygregory | degorenko, this should work? ^ | 16:41 |
| degorenko | iurygregory, just put link to keystone commit, that everyone can see - it works | 16:42 |
| iurygregory | i put Depends on and the change id =) | 16:43 |
| *** darkxploit has joined #puppet-openstack | 16:43 | |
| *** baker_ has joined #puppet-openstack | 16:46 | |
| *** baker has quit IRC | 16:46 | |
| degorenko | iurygregory, no, put in in review with os_service to comments | 16:47 |
| iurygregory | oh put the review link to tempest in my keystone patch right? | 16:48 |
| *** samuelBartel has quit IRC | 16:50 | |
| ntpttr | +1 for puppet fake, would definitely smooth out the new module process | 16:52 |
| openstackgerrit | Christopher Brown proposed openstack/puppet-cinder: acceptance: checkout stable/kilo puppet modules https://review.openstack.org/256583 | 16:56 |
| *** zhangjn has quit IRC | 16:59 | |
| *** nlahouti has joined #puppet-openstack | 17:09 | |
| *** nlahouti has quit IRC | 17:11 | |
| *** nlahouti has joined #puppet-openstack | 17:11 | |
| openstackgerrit | Vishal Patil proposed openstack/puppet-neutron: Fix dependencies for Nuage plugin configuration https://review.openstack.org/247239 | 17:18 |
| *** ducttape_ has joined #puppet-openstack | 17:18 | |
| *** yottatsa has quit IRC | 17:26 | |
| *** yottatsa has joined #puppet-openstack | 17:26 | |
| *** skolekonov is now known as _skolekonov | 17:31 | |
| *** yottatsa has quit IRC | 17:32 | |
| *** dgurtner has quit IRC | 17:33 | |
| *** yottatsa has joined #puppet-openstack | 17:34 | |
| *** rochaporto has quit IRC | 17:34 | |
| *** rochaporto has joined #puppet-openstack | 17:34 | |
| *** rochaporto has quit IRC | 17:35 | |
| openstackgerrit | Merged openstack/puppet-keystone: comment: note how to unset version w. empty string https://review.openstack.org/255639 | 17:35 |
| *** dgurtner has joined #puppet-openstack | 17:40 | |
| *** dgurtner has quit IRC | 17:40 | |
| *** dgurtner has joined #puppet-openstack | 17:40 | |
| *** davideagle_ has quit IRC | 17:48 | |
| iurygregory | degorenko, it pass \o/ | 17:50 |
| degorenko | +2 | 17:51 |
| *** aimon has quit IRC | 17:51 | |
| iurygregory | tks =) | 17:52 |
| EmilienM | iurygregory: +A | 17:53 |
| iurygregory | wow | 17:54 |
| iurygregory | =D | 17:54 |
| iurygregory | about the cookiecutter i think we should wait for mgrosz to send other patch ( https://review.openstack.org/#/c/256570/ ) | 17:55 |
| dmsimard | EmilienM: you generate the subunit output from integration tests, where is it used ? | 18:00 |
| dmsimard | don't see anything special on, i.e, jenkins01 | 18:01 |
| EmilienM | dmsimard: should be on http://status.openstack.org/openstack-health/#/ | 18:01 |
| EmilienM | mtreinish: I still don't see the jobs ^ | 18:01 |
| dmsimard | EmilienM: so that tool picks up the subunit test results from every job and puts them in fancy graphs ? | 18:01 |
| EmilienM | mtreinish: last time you told be something had to restart ? gearman client? | 18:02 |
| EmilienM | dmsimard: in logstash | 18:02 |
| EmilienM | wait I have a nice doc | 18:02 |
| mtreinish | EmilienM: yeah, the gearman client needs to be restarted to pick up the config changes | 18:03 |
| *** dgurtner has quit IRC | 18:04 | |
| mtreinish | EmilienM: I asked clarkb about it this morning he said that the client had a backlog, but he'd restart it when it was free | 18:04 |
| EmilienM | mtreinish: thanks | 18:04 |
| EmilienM | mtreinish: I can't find the doc again | 18:04 |
| EmilienM | the subunit thing you showed me | 18:04 |
| mtreinish | EmilienM: this? http://docs.openstack.org/infra/system-config/logstash.html | 18:05 |
| EmilienM | mtreinish: yeah I think | 18:06 |
| mtreinish | EmilienM: I also wrote a blog post on how the subunit2sql side works: http://blog.kortar.org/?p=212 It predates the dasboard by a month though | 18:06 |
| EmilienM | dmsimard: ^ | 18:06 |
| EmilienM | nice reading | 18:06 |
| *** cwolferh has quit IRC | 18:07 | |
| *** cwolferh has joined #puppet-openstack | 18:07 | |
| dmsimard | mtreinish: that's pretty cool and it'd be pretty fun to see a specific test decrease/increase in performance across all jobs resulting from a specific change | 18:13 |
| *** degorenko is now known as _degorenko|afk | 18:15 | |
| *** baker_ has quit IRC | 18:18 | |
| openstackgerrit | Merged openstack/puppet-keystone: Switch Keystone to $::os_service_default https://review.openstack.org/250945 | 18:18 |
| *** delattec has joined #puppet-openstack | 18:20 | |
| *** IvanBerezovskiy has left #puppet-openstack | 18:20 | |
| *** cdelatte has quit IRC | 18:23 | |
| *** yottatsa has quit IRC | 18:23 | |
| *** aimon has joined #puppet-openstack | 18:24 | |
| *** yottatsa has joined #puppet-openstack | 18:26 | |
| *** mgrosz has joined #puppet-openstack | 18:32 | |
| *** mgrosz has quit IRC | 18:33 | |
| *** baker has joined #puppet-openstack | 18:34 | |
| *** baker has quit IRC | 18:35 | |
| *** ducttape_ has quit IRC | 18:45 | |
| *** jistr has quit IRC | 18:45 | |
| *** davideagle has joined #puppet-openstack | 18:46 | |
| *** mgrosz has joined #puppet-openstack | 18:53 | |
| *** mgrosz has quit IRC | 18:54 | |
| *** davideagle has quit IRC | 19:00 | |
| *** davideagle has joined #puppet-openstack | 19:01 | |
| *** baker has joined #puppet-openstack | 19:02 | |
| *** yottatsa has quit IRC | 19:11 | |
| *** mgrosz has joined #puppet-openstack | 19:11 | |
| *** davideagle has quit IRC | 19:12 | |
| *** yottatsa has joined #puppet-openstack | 19:16 | |
| *** hogepodge has quit IRC | 19:18 | |
| *** mgrosz has quit IRC | 19:18 | |
| *** hogepodge has joined #puppet-openstack | 19:18 | |
| *** hogepodge has quit IRC | 19:19 | |
| *** hogepodge has joined #puppet-openstack | 19:19 | |
| *** hogepodge has quit IRC | 19:23 | |
| *** hogepodge has joined #puppet-openstack | 19:23 | |
| *** dprince has quit IRC | 19:24 | |
| *** iurygregory has quit IRC | 19:30 | |
| *** ericksonsantos has quit IRC | 19:30 | |
| *** iurygregory has joined #puppet-openstack | 19:31 | |
| *** ericksonsantos has joined #puppet-openstack | 19:32 | |
| *** colonwq has joined #puppet-openstack | 19:38 | |
| *** dhollinger has quit IRC | 19:47 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Federation support for mellon https://review.openstack.org/256642 | 19:49 |
| iurygregory | richm, ^ mellon =) | 19:50 |
| *** delatte has joined #puppet-openstack | 19:52 | |
| *** delattec has quit IRC | 19:55 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Update Puppetfile to latest releases / branches https://review.openstack.org/256644 | 19:56 |
| EmilienM | _ody: ^ | 19:57 |
| EmilienM | let me know if what do you think | 19:57 |
| EmilienM | we might want our modules testing the latest versions | 19:57 |
| *** imcsk8 is now known as imcsk8|afk | 19:59 | |
| *** ducttape_ has joined #puppet-openstack | 20:00 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Correct the describe for shibboleth in tests https://review.openstack.org/256645 | 20:00 |
| *** baker has quit IRC | 20:00 | |
| _ody | I generally agree but it probably means we need to also update some modulefile.json files. | 20:01 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: CI test - do not merge https://review.openstack.org/256646 | 20:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-barbican: CI test - do not merge https://review.openstack.org/256647 | 20:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: CI test - do not merge https://review.openstack.org/256648 | 20:02 |
| *** baker has joined #puppet-openstack | 20:02 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-cinder: CI test - do not merge https://review.openstack.org/256649 | 20:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-designate: CI test - do not merge https://review.openstack.org/256650 | 20:02 |
| EmilienM | _ody: right | 20:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: CI test - do not merge https://review.openstack.org/256651 | 20:02 |
| iurygregory | wow | 20:02 |
| EmilienM | _ody: I'll take care of it | 20:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-gnocchi: CI test - do not merge https://review.openstack.org/256652 | 20:02 |
| EmilienM | if we bump it | 20:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-heat: CI test - do not merge https://review.openstack.org/256653 | 20:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-horizon: CI test - do not merge https://review.openstack.org/256654 | 20:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ironic: CI test - do not merge https://review.openstack.org/256655 | 20:02 |
| *** paramite has quit IRC | 20:02 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: CI test - do not merge https://review.openstack.org/256656 | 20:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-manila: CI test - do not merge https://review.openstack.org/256657 | 20:03 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: CI test - do not merge https://review.openstack.org/256658 | 20:03 |
| dmsimard | *EmilienM updates a bunch of things and checks what breaks* | 20:03 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: CI test - do not merge https://review.openstack.org/256659 | 20:03 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstacklib: CI test - do not merge https://review.openstack.org/256660 | 20:03 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: CI test - do not merge https://review.openstack.org/256661 | 20:03 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: CI test - do not merge https://review.openstack.org/256662 | 20:03 |
| EmilienM | dmsimard: yeah sorry for spam | 20:03 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-sahara: CI test - do not merge https://review.openstack.org/256663 | 20:03 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: CI test - do not merge https://review.openstack.org/256664 | 20:03 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-trove: CI test - do not merge https://review.openstack.org/256665 | 20:03 |
| EmilienM | dmsimard: I want to test https://review.openstack.org/#/c/256644/ | 20:04 |
| EmilienM | and make sure our modules are compliant | 20:04 |
| *** yottatsa has quit IRC | 20:04 | |
| dmsimard | yes, god knows puppetlabs tend to make a lot of backwards incompatible changes sometimes :) | 20:05 |
| EmilienM | it happens we also do mistakes | 20:05 |
| Vinsh | NEVER | 20:05 |
| Vinsh | we code pure gold. all the time. | 20:05 |
| EmilienM | it's also good feedback for our Puppetlabs to use latest rleases | 20:05 |
| EmilienM | Vinsh: only yours. | 20:06 |
| _ody | EmilienM: I did a quick look and I wouldn't expect any API breakage but...modules aren't historically good at keeping with semantic version requirements. | 20:07 |
| _ody | I was going to take a closer look after lunch. | 20:08 |
| _ody | Ah. That is what you're doing with "CI test - do not merge" | 20:08 |
| _ody | Cool. | 20:08 |
| *** yottatsa has joined #puppet-openstack | 20:11 | |
| EmilienM | _ody: yeah, so zuul-cloner can test the modules with that changz | 20:11 |
| iurygregory | https://github.com/openstack/puppet-ironic/blob/master/manifests/init.pp#L218#L222 why we have two variables with same name? O.o | 20:11 |
| EmilienM | it's a canary test | 20:11 |
| *** dhollinger has joined #puppet-openstack | 20:11 | |
| EmilienM | iurygregory: nice one | 20:11 |
| EmilienM | iurygregory: can you send a patch? | 20:11 |
| iurygregory | i'm moving ironic to service_default | 20:12 |
| EmilienM | nice | 20:12 |
| iurygregory | i'll remove =) | 20:12 |
| EmilienM | iurygregory++ | 20:12 |
| iurygregory | :D | 20:12 |
| *** guimaluf has joined #puppet-openstack | 20:26 | |
| *** gfidente has quit IRC | 20:31 | |
| *** yottatsa has quit IRC | 20:40 | |
| *** yottatsa has joined #puppet-openstack | 20:43 | |
| *** hogepodge has quit IRC | 20:47 | |
| *** toMeloos has quit IRC | 20:49 | |
| _ody | Wow. I am surprised that actually parses. | 20:50 |
| iurygregory | lol | 20:52 |
| _ody | Don't tell ANYONE or next thing you know people will be exploiting "variable re-assignment" | 20:53 |
| iurygregory | ahushasuahs nice | 20:53 |
| iurygregory | you should push a feature to puppet labs to not let this happen i think | 20:54 |
| *** yottatsa has quit IRC | 20:55 | |
| *** rcernin has joined #puppet-openstack | 20:56 | |
| *** cwolferh has quit IRC | 21:07 | |
| iurygregory | bye people, have a nice weekend | 21:10 |
| iurygregory | o/ | 21:10 |
| *** hogepodge has joined #puppet-openstack | 21:12 | |
| *** ntpttr has left #puppet-openstack | 21:15 | |
| *** cwolferh has joined #puppet-openstack | 21:23 | |
| *** baker has quit IRC | 21:25 | |
| *** ducttape_ has quit IRC | 21:35 | |
| *** ducttape_ has joined #puppet-openstack | 21:36 | |
| *** baker has joined #puppet-openstack | 21:37 | |
| *** ducttape_ has quit IRC | 21:40 | |
| openstackgerrit | Keith Schincke proposed openstack/puppet-ceph: Add support for civetweb support https://review.openstack.org/254548 | 21:43 |
| *** nlahouti has quit IRC | 21:44 | |
| *** nlahouti has joined #puppet-openstack | 21:47 | |
| *** yottatsa has joined #puppet-openstack | 21:58 | |
| EmilienM | _ody: so my patch would break openstack_extras | 22:00 |
| EmilienM | http://logs.openstack.org/61/256661/1/check/gate-puppet-openstack_extras-puppet-unit-3.8-dsvm-centos7/1a6a640/console.html | 22:00 |
| EmilienM | maybe not related though | 22:01 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: CI test - do not merge https://review.openstack.org/256778 | 22:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_extras: CI test - do not merge https://review.openstack.org/256778 | 22:03 |
| _ody | Errors don't immediately make me think dependent module issue and the pick function in stdlib has changed in years. | 22:11 |
| _ody | (besides some typo and error message changes) | 22:11 |
| *** imcsk8|afk is now known as imcsk8 | 22:14 | |
| openstackgerrit | Cody Herriges proposed openstack/puppet-designate: Update designate::db class to match other module pattern https://review.openstack.org/227217 | 22:18 |
| *** yottatsa has quit IRC | 22:22 | |
| *** ducttape_ has joined #puppet-openstack | 22:26 | |
| *** openstackgerrit has quit IRC | 22:37 | |
| *** rcernin has quit IRC | 22:37 | |
| *** openstackgerrit has joined #puppet-openstack | 22:37 | |
| *** davideagle_ has joined #puppet-openstack | 22:39 | |
| *** davideagle_ has quit IRC | 22:43 | |
| *** baker has quit IRC | 23:01 | |
| *** saneax_AFK is now known as saneax | 23:02 | |
| colonwq | xarses, dmsimard I removed frontend_type from params.pp and set the default in rgw.pp to 'apache' . The rgw tests failing. Shouln't the default be picked up from rgw.pp? | 23:08 |
| colonwq | xarses, dmsimard: https://review.openstack.org/#/c/254548/ | 23:09 |
| xarses | colonwq: the rgw tests are broken, you won't be able to make them pass unless you re-write them to deploy the new front-end | 23:09 |
| xarses | the apache-fastcgi package from gitbuilder.ceph.com is no longer available | 23:10 |
| colonwq | xarses, ah thanks. I will start working on tests. I am climbing the learning curve. | 23:11 |
| xarses | colonwq: I'll propose moving the test shortly so that it can be run in a job that votes differently | 23:12 |
| xarses | I haven't completed it yet | 23:12 |
| *** ducttape_ has quit IRC | 23:12 | |
| colonwq | xarses, not a problem. I need to learn. there are always too many things to do. | 23:17 |
| *** dhollinger has quit IRC | 23:22 | |
| *** yottatsa has joined #puppet-openstack | 23:25 | |
| *** dfisher has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!