| *** zhangjn has joined #puppet-openstack | 00:03 | |
| *** mdorman has quit IRC | 00:04 | |
| *** markvoelker has quit IRC | 00:06 | |
| *** richm has quit IRC | 00:07 | |
| openstackgerrit | Alan Pevec proposed openstack/puppet-openstack-integration: [DO NOT MERGE] test with pure RDO Liberty testing repo (stage CBS repos) https://review.openstack.org/241825 | 00:14 |
|---|---|---|
| *** dhollinger has quit IRC | 00:19 | |
| *** zhangjn has quit IRC | 00:25 | |
| *** karimb has quit IRC | 00:25 | |
| *** dfisher has quit IRC | 00:26 | |
| *** jerrygb has joined #puppet-openstack | 00:31 | |
| *** jerrygb has quit IRC | 00:32 | |
| *** chem has quit IRC | 00:34 | |
| karmix | EmilienM: Did puppet-neutron have some issues with CI related to ovs on red hat a few weeks ago? I can't figure out why Change 237530 by Lukas Bezdicka triggered a bunch of ovs errors in CI. | 00:40 |
| *** nihilifer has quit IRC | 00:44 | |
| *** markvoelker has joined #puppet-openstack | 00:47 | |
| *** zhangjn has joined #puppet-openstack | 01:00 | |
| *** jmccrory has quit IRC | 01:09 | |
| *** nlahouti has quit IRC | 01:13 | |
| *** jmccrory has joined #puppet-openstack | 01:13 | |
| *** jmccrory has quit IRC | 01:14 | |
| *** angdraug has quit IRC | 01:24 | |
| karmix | Yep. CI is broken on stable/kilo, for at least the puppet-neutron repo. It looks like it has been broken for a few weeks, and likely has something to do with changes in puppet-openstack-integration. | 01:40 |
| *** dgurtner has joined #puppet-openstack | 01:50 | |
| *** dgurtner has quit IRC | 01:50 | |
| *** dgurtner has joined #puppet-openstack | 01:50 | |
| *** dgurtner has quit IRC | 01:56 | |
| *** jmccrory has joined #puppet-openstack | 02:08 | |
| karmix | I was mistaken on the puppet-openstack-integration part. | 02:33 |
| EmilienM | karmix: it should not | 02:40 |
| EmilienM | this patch merged yesterday: https://review.openstack.org/#/c/238615/ | 02:42 |
| EmilienM | in stable/kilo | 02:42 |
| karmix | CI is broken on stable/kilo for puppet-neutron because puppet_spec_helper is cloning the master branch into spec/fixtures/modules/keystone. Master on keystone includes change 228744 (commit: 8461e9ac), which alters how the tenant/project is set in a way that is no longer compatible with the neutron-puppet unit tests in stable/kilo. | 02:43 |
| karmix | EmilienM: I'll look at that patch real quick. I think the solution here is to pin the repos to stable/kilo in .fixtures.yml. | 02:44 |
| *** ilbot3 has quit IRC | 02:44 | |
| EmilienM | karmix: you should rebase your patch | 02:44 |
| EmilienM | it's not using fixtures anymore | 02:44 |
| EmilienM | karmix: see https://review.openstack.org/#/c/238615/ | 02:45 |
| *** ilbot3 has joined #puppet-openstack | 02:47 | |
| karmix | EmilienM: Thanks, that explains why I was seeing puppet-openstack-integration clone the modules until I switched to the review branch. | 02:49 |
| karmix | EmilienM: I am still seeing one unit test fail on the tip of stable/kilo. Have you seen any issues with the "neutron::agents::ml2::ovs on Debian platforms configures neutron plugin ovs agent with ml2 plugin should contain Class[neutron::params]" test, lately? | 02:50 |
| openstackgerrit | Doug Knight proposed openstack/puppet-neutron: Drop nova_admin_tenant_id_setter https://review.openstack.org/237530 | 02:55 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install modules after installing puppet https://review.openstack.org/241853 | 03:10 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: DNM - CI test https://review.openstack.org/241774 | 03:14 |
| *** zhangjn_ has joined #puppet-openstack | 03:15 | |
| *** guessi has joined #puppet-openstack | 03:15 | |
| *** zhangjn has quit IRC | 03:17 | |
| *** guessi has left #puppet-openstack | 03:18 | |
| EmilienM | I broke integration CI, but I have a patch: https://review.openstack.org/#/c/241853 - please review it | 03:22 |
| openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: WIP multi port swift config https://review.openstack.org/241726 | 03:44 |
| *** nlahouti has joined #puppet-openstack | 03:49 | |
| *** markvoelker has quit IRC | 03:54 | |
| *** nlahouti1 has joined #puppet-openstack | 04:10 | |
| *** nlahouti has quit IRC | 04:11 | |
| *** nlahouti1 has quit IRC | 04:16 | |
| *** nlahouti has joined #puppet-openstack | 04:17 | |
| *** fedexo has joined #puppet-openstack | 04:20 | |
| *** morazi has quit IRC | 04:48 | |
| *** markvoelker has joined #puppet-openstack | 04:55 | |
| *** degorenko has joined #puppet-openstack | 04:56 | |
| *** markvoelker has quit IRC | 05:00 | |
| *** jamespage has quit IRC | 05:05 | |
| *** jamespage has joined #puppet-openstack | 05:06 | |
| openstackgerrit | shravya Gaddam proposed openstack/puppet-trove: log file for guestagent is modified https://review.openstack.org/241884 | 05:28 |
| *** zhangjn_ has quit IRC | 05:39 | |
| *** nihilifer has joined #puppet-openstack | 05:39 | |
| *** sanjayu has joined #puppet-openstack | 05:39 | |
| *** zhangjn has joined #puppet-openstack | 05:41 | |
| *** bogdando has quit IRC | 05:46 | |
| *** rahulait has joined #puppet-openstack | 06:15 | |
| *** nlahouti has quit IRC | 06:31 | |
| *** nlahouti has joined #puppet-openstack | 06:33 | |
| *** nlahouti has quit IRC | 06:37 | |
| *** nlahouti has joined #puppet-openstack | 06:37 | |
| *** jtomasek has joined #puppet-openstack | 06:50 | |
| *** markvoelker has joined #puppet-openstack | 06:56 | |
| *** nlahouti has quit IRC | 06:57 | |
| *** nlahouti has joined #puppet-openstack | 06:57 | |
| *** rahulait has quit IRC | 06:57 | |
| *** markvoelker has quit IRC | 07:01 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-horizon: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/241908 | 07:05 |
| *** fedexo has quit IRC | 07:05 | |
| *** dgurtner has joined #puppet-openstack | 07:05 | |
| *** dgurtner has quit IRC | 07:05 | |
| *** dgurtner has joined #puppet-openstack | 07:05 | |
| *** nlahouti has quit IRC | 07:06 | |
| *** nlahouti has joined #puppet-openstack | 07:07 | |
| *** zhangjn has quit IRC | 07:33 | |
| *** dgurtner has quit IRC | 07:34 | |
| *** nlahouti has quit IRC | 07:48 | |
| *** zigo has quit IRC | 07:53 | |
| *** zigo has joined #puppet-openstack | 07:56 | |
| *** zhangjn has joined #puppet-openstack | 08:00 | |
| *** dgurtner has joined #puppet-openstack | 08:01 | |
| *** dgurtner has quit IRC | 08:01 | |
| *** dgurtner has joined #puppet-openstack | 08:01 | |
| *** _skolekonov is now known as skolekonov | 08:10 | |
| *** dh64 has quit IRC | 08:11 | |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-nova: Update parameters for nova https://review.openstack.org/241614 | 08:11 |
| *** openstackgerrit has quit IRC | 08:16 | |
| *** openstackgerrit has joined #puppet-openstack | 08:17 | |
| *** dgurtner has quit IRC | 08:20 | |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-keystone: add policy driver option for keystone https://review.openstack.org/240946 | 08:29 |
| *** karimb has joined #puppet-openstack | 08:30 | |
| *** gildub has quit IRC | 08:31 | |
| *** karimb_ has joined #puppet-openstack | 08:32 | |
| *** IvanBerezovskiy has joined #puppet-openstack | 08:33 | |
| *** openstack has joined #puppet-openstack | 08:36 | |
| myatsenko | Guys please review this patch https://review.openstack.org/#/c/203675/ | 08:36 |
| *** jpena has joined #puppet-openstack | 08:38 | |
| *** rahulait has joined #puppet-openstack | 08:46 | |
| *** paramite has joined #puppet-openstack | 08:51 | |
| *** dgurtner has joined #puppet-openstack | 08:55 | |
| *** dgurtner has quit IRC | 08:55 | |
| *** dgurtner has joined #puppet-openstack | 08:55 | |
| *** ddmitriev has joined #puppet-openstack | 08:56 | |
| *** markvoelker has joined #puppet-openstack | 08:57 | |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-nova: [WIP] https://review.openstack.org/233011 | 09:00 |
| *** markvoelker has quit IRC | 09:01 | |
| *** rahulait has quit IRC | 09:04 | |
| *** gildub has joined #puppet-openstack | 09:06 | |
| *** derekh has joined #puppet-openstack | 09:09 | |
| *** gfidente has joined #puppet-openstack | 09:24 | |
| *** jistr has joined #puppet-openstack | 09:24 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-cinder: Support of PyMySQL driver for MySQL backend https://review.openstack.org/241214 | 09:32 |
| *** markvoelker has joined #puppet-openstack | 09:57 | |
| *** bogdando has joined #puppet-openstack | 10:00 | |
| *** bogdando has quit IRC | 10:01 | |
| *** markvoelker has quit IRC | 10:02 | |
| *** dgurtner has quit IRC | 10:08 | |
| *** bogdando has joined #puppet-openstack | 10:20 | |
| *** zhangjn has quit IRC | 10:20 | |
| *** zerda has joined #puppet-openstack | 10:22 | |
| *** zhangjn has joined #puppet-openstack | 10:23 | |
| openstackgerrit | Merged openstack/puppet-openstacklib: Fallback to default rcfile https://review.openstack.org/227762 | 10:25 |
| *** zhangjn_ has joined #puppet-openstack | 10:35 | |
| *** zhangjn has quit IRC | 10:36 | |
| *** gildub has quit IRC | 10:43 | |
| *** paramite is now known as paramite|afk | 10:50 | |
| *** paramite|afk is now known as paramite | 10:50 | |
| *** paramite is now known as paramite|afk | 10:50 | |
| *** chem has joined #puppet-openstack | 10:50 | |
| *** jerrygb has joined #puppet-openstack | 10:58 | |
| *** sergmelikyan has joined #puppet-openstack | 11:00 | |
| *** sergmelikyan has quit IRC | 11:13 | |
| *** markvoelker has joined #puppet-openstack | 11:13 | |
| *** markvoelker has quit IRC | 11:18 | |
| *** paramite|afk is now known as paramite | 11:48 | |
| *** khushbu has joined #puppet-openstack | 11:49 | |
| *** khushbu has quit IRC | 11:52 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider https://review.openstack.org/216821 | 11:54 |
| iurygregory | myatsenko, take a look at my comment in your patch =) | 11:59 |
| *** sanjayu has quit IRC | 12:06 | |
| *** khushbu_ has joined #puppet-openstack | 12:08 | |
| *** zhangjn_ has quit IRC | 12:08 | |
| *** zhangjn has joined #puppet-openstack | 12:08 | |
| *** jerrygb has quit IRC | 12:10 | |
| *** zhangjn has quit IRC | 12:17 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Add basic acceptance test https://review.openstack.org/233591 | 12:17 |
| *** zerda has quit IRC | 12:20 | |
| *** khushbu_ has quit IRC | 12:25 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider https://review.openstack.org/216821 | 12:30 |
| EmilienM | hello | 12:35 |
| chem | EmilienM: hé | 12:36 |
| EmilienM | degorenko: can you look https://review.openstack.org/#/c/241853 please ? | 12:36 |
| iurygregory | morning =D | 12:36 |
| degorenko | hey o/ | 12:37 |
| degorenko | EmilienM, ack, looking | 12:37 |
| EmilienM | degorenko: the current integration testing jobs are broken withot it | 12:37 |
| EmilienM | degorenko: tested with https://review.openstack.org/#/c/241774/ | 12:37 |
| *** zhangjn has joined #puppet-openstack | 12:39 | |
| *** zhangjn has quit IRC | 12:39 | |
| *** zhangjn has joined #puppet-openstack | 12:40 | |
| degorenko | EmilienM, now we are trying to run puppet module list when puppet doesn't installed yet? | 12:41 |
| EmilienM | yes | 12:41 |
| degorenko | lol :) | 12:41 |
| EmilienM | it's because of me | 12:41 |
| EmilienM | let me tell you why | 12:41 |
| EmilienM | yesterday we discovered our integration jobs were using a different version of puppet than beaker jobs so I added the macro that purge puppet in the job workflow and some bash code that install puppet | 12:42 |
| EmilienM | #2 in the wrong place :) | 12:42 |
| iurygregory | wow | 12:44 |
| degorenko | EmilienM, wow :) PTO time is dark time - you don't know what's happens during this period :D | 12:44 |
| EmilienM | the good thing is now our stable/kilo is green again and our integration jobs too | 12:44 |
| degorenko | yep :) i saw | 12:44 |
| EmilienM | degorenko: don't worry - next week I'm in your situation, I'm PTO all week | 12:44 |
| degorenko | bth, i've reupload fix for horizon | 12:44 |
| degorenko | EmilienM, nice :) | 12:44 |
| degorenko | btw* | 12:45 |
| degorenko | thanks for notification | 12:45 |
| degorenko | who will be also on PTO? | 12:45 |
| EmilienM | degorenko: I don't know | 12:45 |
| degorenko | ah, ok :) | 12:45 |
| degorenko | also, i've read weekly meeting on this week | 12:46 |
| degorenko | very short :D | 12:46 |
| *** sergmelikyan has joined #puppet-openstack | 12:48 | |
| degorenko | EmilienM, do we have some progress with os_service_default? | 12:48 |
| EmilienM | degorenko: puppet-cinder should be covered, look at the trello card | 12:49 |
| degorenko | EmilienM, yep, i read comments - may be something out of this comments was here in irc? | 12:49 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Support for services different only by type. https://review.openstack.org/238954 | 12:49 |
| EmilienM | degorenko: nope | 12:50 |
| degorenko | EmilienM, ok, thanks | 12:50 |
| EmilienM | degorenko: still in japan? | 12:50 |
| degorenko | EmilienM, no, already in Moscow :) last day in PTO. Because of jetlag :) | 12:51 |
| *** jistr has quit IRC | 12:52 | |
| degorenko | EmilienM, also, i saw your patches for switching to Liberty GA repos | 12:54 |
| iurygregory | jetlag sucks =/ I've slept for 14hours =P | 12:54 |
| degorenko | it seems that we don't have trove | 12:54 |
| EmilienM | degorenko: it's blocked | 12:54 |
| EmilienM | degorenko: right, RDO team is finishing the GA repo | 12:54 |
| degorenko | it is only one problem, i guess? | 12:55 |
| *** sergmelikyan has quit IRC | 12:59 | |
| *** dprince has joined #puppet-openstack | 13:02 | |
| EmilienM | degorenko: yes | 13:03 |
| EmilienM | we're working on it | 13:03 |
| EmilienM | I think today or tomorrow we can have the packages | 13:03 |
| degorenko | cool :) | 13:06 |
| *** paramite is now known as paramite|afk | 13:07 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: run_tests: install modules after installing puppet https://review.openstack.org/241853 | 13:07 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: [DO NOT MERGE] test with pure RDO Liberty testing repo (stage CBS repos) https://review.openstack.org/241825 | 13:08 |
| *** jistr has joined #puppet-openstack | 13:11 | |
| *** jerrygb has joined #puppet-openstack | 13:11 | |
| iurygregory | we can try recheck now EmilienM ? | 13:12 |
| *** markvoelker has joined #puppet-openstack | 13:14 | |
| *** trown|outttypeww is now known as trown | 13:14 | |
| degorenko | iurygregory, i can TRY :) | 13:14 |
| *** jerrygb has quit IRC | 13:16 | |
| *** jayg|g0n3 is now known as jayg | 13:16 | |
| *** markvoelker_ has joined #puppet-openstack | 13:18 | |
| *** markvoelker has quit IRC | 13:18 | |
| *** zhangjn has quit IRC | 13:18 | |
| *** xarses has joined #puppet-openstack | 13:22 | |
| *** paramite|afk is now known as paramite | 13:24 | |
| *** morazi has joined #puppet-openstack | 13:27 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-manila: [DO NOT MERGE] test with pure RDO Liberty testing repo (stage CBS repos) https://review.openstack.org/242045 | 13:27 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: [DO NOT MERGE] test with pure RDO Liberty testing repo (stage CBS repos) https://review.openstack.org/242046 | 13:28 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-gnocchi: [DO NOT MERGE] test with pure RDO Liberty testing repo (stage CBS repos) https://review.openstack.org/242048 | 13:29 |
| *** sergmelikyan has joined #puppet-openstack | 13:29 | |
| openstackgerrit | Lukas Bezdicka proposed openstack/puppet-tempest: Switch id setters to openstack client https://review.openstack.org/242052 | 13:34 |
| *** degorenko has quit IRC | 13:35 | |
| openstackgerrit | Lukas Bezdicka proposed openstack/puppet-tempest: Switch id setters to openstack client https://review.openstack.org/242052 | 13:35 |
| social | grr git | 13:36 |
| social | oh it's not git it's gerrit | 13:40 |
| openstackgerrit | Lukas Bezdicka proposed openstack/puppet-tempest: Switch id setters to openstack client https://review.openstack.org/242052 | 13:41 |
| *** jpena is now known as jpena|lunch | 13:49 | |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-nova: Floating IP range support in Nova network https://review.openstack.org/233011 | 13:49 |
| *** richm has joined #puppet-openstack | 13:55 | |
| *** jerrygb has joined #puppet-openstack | 14:04 | |
| *** bklei has joined #puppet-openstack | 14:09 | |
| openstackgerrit | Merged openstack/puppet-neutron: Drop nova_admin_tenant_id_setter https://review.openstack.org/237530 | 14:10 |
| EmilienM | paramite: I think you need to update https://review.openstack.org/#/c/239680/ | 14:20 |
| EmilienM | paramite: yesterday, we merged a change in puppet-keystone (cc chem) and it probably broke puppet-heat | 14:20 |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-cinder: Support of PyMySQL driver for MySQL backend https://review.openstack.org/241214 | 14:23 |
| paramite | EmilienM, perfect news :) ... ok, will check that out after z3 stuff | 14:24 |
| *** rgowrishankar has joined #puppet-openstack | 14:25 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-gnocchi: [DO NOT MERGE] test with pure RDO Liberty testing repo (stage CBS repos) https://review.openstack.org/242048 | 14:26 |
| EmilienM | paramite: thx. Also you can talk with chem to understand what we changed | 14:27 |
| paramite | EmilienM, definitely | 14:29 |
| openstackgerrit | Lukas Bezdicka proposed openstack/puppet-tempest: Switch id setters to openstack client https://review.openstack.org/242052 | 14:33 |
| *** dgurtner has joined #puppet-openstack | 14:36 | |
| zigo | EmilienM: Can you review that one? https://review.openstack.org/#/c/241615/ | 14:37 |
| zigo | Sorry to insist, but it's very important for me. | 14:37 |
| *** paramite is now known as paramite|afk | 14:38 | |
| *** paramite|afk is now known as paramite | 14:40 | |
| iurygregory | Hey EmilienM, puppet-keystone CI is broken? https://jenkins05.openstack.org/job/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/231/consoleFull | 14:41 |
| iurygregory | https://jenkins06.openstack.org/job/gate-puppet-keystone-puppet-unit-3.3-dsvm-centos7/96/consoleFull | 14:41 |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-nova: Floating IP range support in Nova network https://review.openstack.org/233011 | 14:41 |
| EmilienM | chem: have you seen that error before? | 14:42 |
| EmilienM | [1;31mError: /Stage[main]/Main/Keystone_user_role[adminv3::admin_domain@openstackv3::admin_domain]: Could not evaluate: undefined method `empty?' for nil:NilClassm | 14:42 |
| EmilienM | iurygregory: have you rebased your patch? | 14:42 |
| *** _matthias_ has quit IRC | 14:43 | |
| *** trash has quit IRC | 14:43 | |
| *** omartsyniuk has quit IRC | 14:43 | |
| *** jprs has quit IRC | 14:43 | |
| *** jesusaurus has quit IRC | 14:44 | |
| *** karmix has quit IRC | 14:44 | |
| *** dfisher has joined #puppet-openstack | 14:45 | |
| *** partner has quit IRC | 14:46 | |
| *** omartsyniuk has joined #puppet-openstack | 14:46 | |
| *** _matthias_ has joined #puppet-openstack | 14:46 | |
| *** karmix has joined #puppet-openstack | 14:47 | |
| *** partner has joined #puppet-openstack | 14:47 | |
| *** jprs has joined #puppet-openstack | 14:47 | |
| *** trash has joined #puppet-openstack | 14:48 | |
| *** iurygregory has quit IRC | 14:48 | |
| *** iurygregory has joined #puppet-openstack | 14:49 | |
| *** jpena|lunch is now known as jpena | 14:54 | |
| *** trown is now known as trown|outttypeww | 14:57 | |
| *** ducttape_ has joined #puppet-openstack | 15:01 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/puppet-modulesync-configs: Remove class_parameter_defaults puppet-lint check https://review.openstack.org/242105 | 15:08 |
| *** dhollinger has joined #puppet-openstack | 15:10 | |
| *** jesusaurus has joined #puppet-openstack | 15:12 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-cinder: Support of PyMySQL driver for MySQL backend https://review.openstack.org/241214 | 15:14 |
| *** ntpttr has joined #puppet-openstack | 15:14 | |
| paramite | chem, hey, you're there? | 15:14 |
| EmilienM | mgagne, mfisch: can we merge https://review.openstack.org/#/c/240998/ and https://review.openstack.org/#/c/238631 please ? | 15:15 |
| *** ntpttr has quit IRC | 15:19 | |
| *** dgurtner has quit IRC | 15:21 | |
| *** sergmelikyan has quit IRC | 15:24 | |
| openstackgerrit | Lukas Bezdicka proposed openstack/puppet-tempest: Switch id setters to openstack client https://review.openstack.org/242052 | 15:25 |
| *** sergmelikyan has joined #puppet-openstack | 15:25 | |
| *** sergmelikyan has quit IRC | 15:28 | |
| *** sergmelikyan has joined #puppet-openstack | 15:28 | |
| *** markvoelker has joined #puppet-openstack | 15:30 | |
| *** markvoelker_ has quit IRC | 15:31 | |
| *** ntpttr has joined #puppet-openstack | 15:33 | |
| *** paramite has quit IRC | 15:34 | |
| *** markvoelker_ has joined #puppet-openstack | 15:35 | |
| *** markvoelker has quit IRC | 15:37 | |
| *** dprince has quit IRC | 15:37 | |
| *** dgurtner has joined #puppet-openstack | 15:40 | |
| *** dgurtner has quit IRC | 15:40 | |
| *** dgurtner has joined #puppet-openstack | 15:40 | |
| *** mdorman has joined #puppet-openstack | 15:44 | |
| *** sergmelikyan has quit IRC | 15:47 | |
| *** paramite has joined #puppet-openstack | 15:50 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-openstack-integration: Switch from nova_admin_* options for Neutron to actual https://review.openstack.org/242126 | 15:52 |
| *** jistr is now known as jistr|afkmtg | 15:53 | |
| *** bnemec has quit IRC | 15:55 | |
| openstackgerrit | Lukas Bezdicka proposed openstack/puppet-tempest: Switch id setters to openstack client https://review.openstack.org/242052 | 15:57 |
| *** angdraug has joined #puppet-openstack | 15:57 | |
| *** vinsh has quit IRC | 16:00 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-keystone: Support of PyMySQL driver for MySQL backend https://review.openstack.org/242134 | 16:01 |
| *** paramite has quit IRC | 16:02 | |
| *** sergmelikyan has joined #puppet-openstack | 16:04 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-gnocchi: [DO NOT MERGE] test with pure RDO Liberty testing repo (stage CBS repos) https://review.openstack.org/242048 | 16:10 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-gnocchi: [DO NOT MERGE] test with pure RDO Liberty testing repo (stage CBS repos) https://review.openstack.org/242048 | 16:10 |
| jpena | does anyone know why puppet-neutron is duplicating auth configuration in neutron.conf and /etc/neutron/api-paste.ini? I've seen https://github.com/openstack/puppet-neutron/tree/master/manifests#L327-L337 and several other instances, and I'm pretty sure that api-paste.ini file is not even parsed in RDO | 16:11 |
| *** jerrygb has quit IRC | 16:13 | |
| *** jerrygb has joined #puppet-openstack | 16:15 | |
| EmilienM | jpena: mgagne filed a bug similar for cinder iirc | 16:15 |
| jpena | EmilienM: ok, I'll file another bug and prepare a patch in a few days | 16:17 |
| IvanBerezovskiy | EmilienM: hi, I've found that this commit https://review.openstack.org/#/c/227762/ broke keystone unit tests - e.g. http://logs.openstack.org/21/216821/29/check/gate-puppet-keystone-puppet-unit-3.3-dsvm-centos7/52e266c/console.html | 16:17 |
| IvanBerezovskiy | what should we do? Update keystone tests or revert change? | 16:17 |
| IvanBerezovskiy | I think I can research and fix the problem | 16:19 |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-keystone: add policy driver option for keystone https://review.openstack.org/240946 | 16:19 |
| EmilienM | IvanBerezovskiy: you can file a bug, and send a patch if you see the solution. | 16:19 |
| IvanBerezovskiy | EmilienM: ok, sure | 16:20 |
| EmilienM | chem: see https://review.openstack.org/#/c/227762/ | 16:22 |
| EmilienM | our whole might be broken because of ^ | 16:24 |
| IvanBerezovskiy | https://bugs.launchpad.net/puppet-keystone/+bug/1513533 | 16:24 |
| openstack | Launchpad bug 1513533 in puppet-keystone "unit tests are broken" [Undecided,New] - Assigned to Ivan Berezovskiy (iberezovskiy) | 16:24 |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-keystone: add policy driver option for keystone https://review.openstack.org/240946 | 16:24 |
| *** sanjayu has joined #puppet-openstack | 16:27 | |
| *** fedexo has joined #puppet-openstack | 16:30 | |
| EmilienM | IvanBerezovskiy: I think we need to patch puppet-keystone | 16:30 |
| IvanBerezovskiy | I'm working on it | 16:30 |
| EmilienM | cool | 16:31 |
| *** jerrygb has quit IRC | 16:31 | |
| *** jerrygb has joined #puppet-openstack | 16:31 | |
| chem | IvanBerezovskiy: I have a patch at the ready for the problem, is it ok or do you have it as well ? | 16:36 |
| IvanBerezovskiy | I didn't publish it yet | 16:37 |
| chem | IvanBerezovskiy: your choice, if you have it go ahead :) | 16:38 |
| *** bklei has quit IRC | 16:43 | |
| *** sergmelikyan has quit IRC | 16:43 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-keystone: Support of PyMySQL driver for MySQL backend https://review.openstack.org/242134 | 16:45 |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-keystone: Fix unit tests according to fallback for openrc file https://review.openstack.org/242153 | 16:45 |
| *** nlahouti has joined #puppet-openstack | 16:45 | |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-keystone: add policy driver option for keystone https://review.openstack.org/240946 | 16:47 |
| *** nlahouti has quit IRC | 16:48 | |
| *** severion has joined #puppet-openstack | 16:50 | |
| *** mjblack- has joined #puppet-openstack | 16:52 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-keystone: Support of PyMySQL driver for MySQL backend https://review.openstack.org/242134 | 16:52 |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-keystone: Fix unit tests according to fallback for openrc file https://review.openstack.org/242153 | 16:52 |
| *** Aegil_ has joined #puppet-openstack | 16:54 | |
| chem | EmilienM: could you +2 the Ivan's patch, degorenko is not there anymore, another core would be nice to get this stuff in master quickly. | 16:55 |
| EmilienM | sure | 16:55 |
| chem | EmilienM: thanks | 16:55 |
| EmilienM | if richm is around ^ | 16:56 |
| *** timrc_ has joined #puppet-openstack | 16:58 | |
| *** m3ntat has joined #puppet-openstack | 16:59 | |
| *** nlahouti has joined #puppet-openstack | 17:01 | |
| *** ntpttr1 has joined #puppet-openstack | 17:02 | |
| *** simonmcc_ has joined #puppet-openstack | 17:03 | |
| *** dprince has joined #puppet-openstack | 17:03 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Implement all-in-one.sh script https://review.openstack.org/242162 | 17:03 |
| EmilienM | jayg: ^ | 17:03 |
| EmilienM | that's wip | 17:03 |
| *** mjblack has quit IRC | 17:04 | |
| *** v1k0d3n has quit IRC | 17:04 | |
| *** simonmcc has quit IRC | 17:04 | |
| *** bastelfreak has quit IRC | 17:04 | |
| *** mentat has quit IRC | 17:04 | |
| *** timrc has quit IRC | 17:04 | |
| *** bkero has quit IRC | 17:04 | |
| *** Aegil has quit IRC | 17:04 | |
| *** mjblack- is now known as mjblack | 17:04 | |
| *** rahulait has joined #puppet-openstack | 17:06 | |
| openstackgerrit | Nate Potter proposed openstack/puppet-magnum: puppet-magnum: Initial commit https://review.openstack.org/242164 | 17:06 |
| *** bastelfreak has joined #puppet-openstack | 17:07 | |
| *** bastelfreak has quit IRC | 17:07 | |
| chem | EmilienM: richm I have a question about the keystone_endpoint patch: the bug is that the type is not given so it does all sort of weird things. Can I make this a requirement (breaking existing code) without a deprecation period, or should I add a deprecation warning (while it's really a bug, not a change of feature) ? | 17:07 |
| *** simonmcc_ is now known as simonmcc | 17:07 | |
| *** bastelfreak has joined #puppet-openstack | 17:07 | |
| *** bkero has joined #puppet-openstack | 17:08 | |
| chem | EmilienM: richm in both case all the keystone manifests have to be reworked | 17:08 |
| EmilienM | are we breaking the interface? | 17:08 |
| chem | EmilienM: yes definitively | 17:08 |
| chem | (https://review.openstack.org/#/c/239434/8/lib/puppet/type/keystone_endpoint.rb) | 17:09 |
| chem | EmilienM: ^ | 17:09 |
| richm | Do we have to do this now or can we postpone? | 17:09 |
| chem | hum ... it's a quite severe bug. and is required to have keystone triple-o integration | 17:09 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Implement all-in-one.sh script https://review.openstack.org/242162 | 17:09 |
| EmilienM | argh | 17:10 |
| EmilienM | we're breaking some stuffs in liberty | 17:10 |
| chem | EmilienM: richm: the bug is here https://review.openstack.org/#/c/234704 | 17:10 |
| *** bnemec has joined #puppet-openstack | 17:10 | |
| EmilienM | we need good communication: ML, doc, etc | 17:10 |
| *** karmix has quit IRC | 17:10 | |
| chem | (triple o stuff) | 17:10 |
| *** karmix has joined #puppet-openstack | 17:10 | |
| chem | EmilienM: the thing is that it was broken, but working in simple case: only one type per service name | 17:11 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Implement all-in-one.sh script https://review.openstack.org/242162 | 17:11 |
| chem | richm: ^ | 17:11 |
| richm | chem: so there is no way we can just wait an use openstackclient with the fix? We have to fix in puppet-keystone? | 17:11 |
| chem | EmilienM: richm in triple-o they need to have nova/compute and nova/computev3 | 17:11 |
| *** dgurtner has quit IRC | 17:12 | |
| richm | they can't have novav3/computev3? | 17:12 |
| chem | richm: dunno, I'm not sure I understand how it would work ^ | 17:12 |
| chem | richm: ^ I don't know :) | 17:12 |
| richm | This is different than the problem where you could not view endpoints with v2 that were created with v3? | 17:13 |
| chem | I think it's related as puppet completly messed up the enpoints/service database | 17:14 |
| *** bkero has quit IRC | 17:14 | |
| *** bkero has joined #puppet-openstack | 17:14 | |
| chem | not certain though | 17:14 |
| *** jistr|afkmtg is now known as jistr | 17:14 | |
| chem | richm: EmilienM I can quickly try to change the stuff to have a deprecation warning, instead of a failure. I don't know if it's doable (it's more complicated) and will let the bug open in some strange way I think, but I can investigate (one houre or so) | 17:16 |
| *** sanjayu has quit IRC | 17:16 | |
| *** aimon_ has quit IRC | 17:16 | |
| richm | If we are proposing this as a fix for the triple-o keystone problem, we should investigate that problem more to find the root cause of the problem | 17:17 |
| EmilienM | if the bug occurs somewhere else, we need to fix it asap | 17:17 |
| EmilienM | even if that's something not compatible | 17:17 |
| EmilienM | folks will understand that and make the change | 17:17 |
| EmilienM | for that we need good communication | 17:18 |
| *** ddmitriev has quit IRC | 17:19 | |
| chem | EmilienM: richm the bug is there for everyone :) But not a lot of people seems to do different than in the manifests, so fixing the manifest should render the stuff transparent for most people. I can start a discussion on the ML for the other people to let them know about the problem and the necessary fix. | 17:19 |
| EmilienM | chem++ | 17:20 |
| chem | EmilienM: ack, I'll do that. | 17:21 |
| *** dgurtner has joined #puppet-openstack | 17:22 | |
| *** dgurtner has quit IRC | 17:22 | |
| *** dgurtner has joined #puppet-openstack | 17:22 | |
| richm | ack | 17:24 |
| chem | richm: the root cause of the problem is that they uncovered this puppet bug by having a naming convention that few people seems to have. In fact it is impossible with puppet to create such service (same name different type), while it's completly legit (fixed there https://review.openstack.org/#/c/238954/), that's why for people only using puppet they cannot detect the bug. The problem appeared here because the installation was | 17:25 |
| chem | bootstrapped by some other tool and puppet apply after. | 17:25 |
| chem | anyway, I'll write the mail. | 17:25 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Keystone_endpoint match service by name/type. https://review.openstack.org/239434 | 17:28 |
| *** sergmelikyan has joined #puppet-openstack | 17:31 | |
| richm | chem: It sounds to me like they are trying to use a "new" feature that until now, no one has ever used and no one has ever tried to do it with puppet-keystone keystone_endpoint | 17:32 |
| richm | and in the process found a bug in openstackclient | 17:33 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Implement all-in-one.sh script https://review.openstack.org/242162 | 17:33 |
| *** dhollinger has quit IRC | 17:33 | |
| chem | richm: humm, but having a service named nova/compute and nova/computev3 is not a new feature ? | 17:33 |
| chem | richm: and the bug in not in openstackclient | 17:33 |
| chem | richm: openstackclient in this case does exactly what puppet ask it to do | 17:34 |
| EmilienM | IvanBerezovskiy: thx a lot for the bugfix, nice work | 17:34 |
| chem | richm: or maybe I'm missing something ? | 17:35 |
| *** karimb_ has quit IRC | 17:35 | |
| richm | chem: no, not a new feature exactly, but they are doing something different than what was done in the past with respect to service names types | 17:37 |
| richm | chem: the usual way was to have nova/compute and novav3/computev3 | 17:37 |
| richm | chem: but they are trying to do nova/compute and nova/computev3 | 17:38 |
| chem | richm: ah, oki. | 17:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Implement all-in-one.sh script https://review.openstack.org/242162 | 17:38 |
| chem | richm: but it's not forbiden by the API, the usual way was just a convention | 17:38 |
| chem | richm: which make puppet _appears_ to work | 17:39 |
| chem | richm: can we explain that you should use the convention and not use anything else when you use puppet while it's completly possible to do it when you use the cli directly ? | 17:40 |
| chem | richm: (it's really a question, no irony here) | 17:40 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Implement all-in-one.sh script https://review.openstack.org/242162 | 17:44 |
| dfisher | ^ no solaris. /sobs | 17:45 |
| richm | chem: If tripleo has a legitimate use case, that they can do in no other way, then we have to fix it. | 17:45 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Implement all-in-one.sh script https://review.openstack.org/242162 | 17:46 |
| richm | chem: Especially since it is not explicitly forbidden to do so by the keystone_endpoint api | 17:46 |
| EmilienM | the fact is they did not use puppet to manage keystone resources before | 17:46 |
| EmilienM | that's why we found the bug | 17:46 |
| *** aimon has joined #puppet-openstack | 17:47 | |
| *** derekh has quit IRC | 17:48 | |
| chem | Oki, I'll try and see if triple-o people could accept to change their naming convention to novav3/computev3 to check the time frame we have for fixing this bug. | 17:53 |
| chem | richm: EmilienM ^ | 17:53 |
| richm | chem: ack | 17:53 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: tempest: use $::id fact for permissions https://review.openstack.org/242179 | 17:54 |
| *** timrc_ is now known as timrc | 17:59 | |
| *** jistr has quit IRC | 18:00 | |
| *** cfloare has quit IRC | 18:01 | |
| *** cfloare has joined #puppet-openstack | 18:02 | |
| *** sergmelikyan has quit IRC | 18:05 | |
| *** sergmelikyan has joined #puppet-openstack | 18:05 | |
| *** jfluhmann has joined #puppet-openstack | 18:10 | |
| chem | richm: I've found that => https://bugzilla.redhat.com/show_bug.cgi?id=1236989 | 18:19 |
| openstack | bugzilla.redhat.com bug 1236989 in rhel-osp-director "RFE: change computev3 service name to novav3" [Unspecified,New] - Assigned to calfonso | 18:19 |
| chem | richm: I know how to fix this :) | 18:20 |
| chem | richm: so it seems they won't need to be convinced after all :) | 18:20 |
| *** iurygregory has left #puppet-openstack | 18:20 | |
| *** iurygregory has joined #puppet-openstack | 18:21 | |
| richm | chem: excellent | 18:22 |
| chem | richm: do you a link that shows the convention used in the official doc or something (if you don't have something don't bother) | 18:24 |
| richm | chem: no - it was just a convention, so nothing official afaik | 18:25 |
| *** jpena has quit IRC | 18:26 | |
| *** mdorman has quit IRC | 18:31 | |
| openstackgerrit | Merged openstack/puppet-keystone: Fix unit tests according to fallback for openrc file https://review.openstack.org/242153 | 18:35 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider https://review.openstack.org/216821 | 18:36 |
| *** mdorman has joined #puppet-openstack | 18:37 | |
| _ody | mwhahaha: Having to statically set '<SERVICE DEFAULT>' in rspec-puppet tests? Seems counter to the spirit of the $::os_service_default variable in the first place, yes? | 18:41 |
| _ody | I am not sure where the best place to put a "constant" is in rspec. | 18:41 |
| mwhahaha | not necessarily | 18:41 |
| mwhahaha | depends on where you are setting <SERVICE DEFAULT>, like are you trying to check if it's actually <SERVICE DEFAULT> and the fact is being picked up correctly (that's valid) | 18:42 |
| mwhahaha | or are we defining the default expected value is <SERVICE DEFAULT> because our tests are poorly structured (needs work) | 18:43 |
| _ody | Agreed. I was just thinking we'd use a variable set locally to rspec so that if the value of $::os_service_default was changed in the future an msync run would update all the module according. | 18:45 |
| _ody | It would still mean tests would have to be updated but the testing of the actual value would be seperate from the actual fact. | 18:46 |
| _ody | ...just not have to be updating in as many places | 18:46 |
| mwhahaha | it would be nice if we could use a constant for that in rspec, i mentioned yesterday that we should probably look at rspec-puppet-facts to managing our facts in the tests | 18:47 |
| mwhahaha | that way we could stop doing the debian/redhat facts all over the place | 18:47 |
| mwhahaha | which is really annoying to update | 18:48 |
| *** gfidente has quit IRC | 18:48 | |
| _ody | Oh yes. Indeed. | 18:48 |
| mwhahaha | because then we could just reference facts[:os_service_default] or something | 18:48 |
| *** mdorman has quit IRC | 18:49 | |
| _ody | rspec-puppet-facts is probably worth meeting agenda/mailing list discussion. I'd be happy to convert a couple modules. | 18:53 |
| * _ody dislikes duplicating facts in every test | 18:53 | |
| *** rahulait has quit IRC | 18:54 | |
| *** jerrygb has quit IRC | 18:57 | |
| *** mdorman has joined #puppet-openstack | 18:57 | |
| *** jerrygb has joined #puppet-openstack | 18:58 | |
| *** nlahouti has quit IRC | 18:59 | |
| *** nlahouti has joined #puppet-openstack | 18:59 | |
| *** degorenko has joined #puppet-openstack | 19:01 | |
| clayton | mwhahaha nod, I was thinking tyhe same thing, I don't like putting the string in the tests either | 19:01 |
| *** jerrygb has quit IRC | 19:03 | |
| openstackgerrit | Nate Potter proposed openstack/puppet-magnum: puppet-magnum: Initial commit https://review.openstack.org/242164 | 19:04 |
| *** ddmitriev has joined #puppet-openstack | 19:07 | |
| *** nlahouti has quit IRC | 19:08 | |
| *** fedexo has quit IRC | 19:24 | |
| iurygregory | CI for puppet-keystone still broken? http://logs.openstack.org/21/216821/30/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/0c26bfe/console.html#_2015-11-05_19_15_52_102 | 19:27 |
| chem | iurygregory: I look into it | 19:29 |
| iurygregory | thanks chem =) | 19:29 |
| chem | iurygregory: can you give me the link to your patch | 19:29 |
| chem | ? | 19:29 |
| iurygregory | yes, https://review.openstack.org/#/c/216821/ | 19:29 |
| chem | iurygregory: thanks | 19:30 |
| iurygregory | np ^^ | 19:30 |
| *** jfluhmann has quit IRC | 19:33 | |
| *** dhollinger has joined #puppet-openstack | 19:45 | |
| *** dgurtner has quit IRC | 19:47 | |
| mwhahaha | hey i'm trying to use the new keystone module with fuel and i'm getting errors about openstack COuld not determine suitable URL for the plugin from the keystone providers, anyone have any thoughts about what I might be missing/need to fix? | 19:50 |
| mwhahaha | the errors are https://ci.fuel-infra.org/job/master.fuel-library.pkgs.ubuntu.neutron_vlan_ha/2551/artifact/logs/2551/fail_error_deploy_neutron_vlan_ha-fuel-snapshot-2015-11-05_18-42-10.tar.xz.filtered.log | 19:50 |
| iurygregory | mwhahaha, i think chem is looking into it =) | 19:50 |
| *** sergmelikyan has quit IRC | 19:50 | |
| *** jfluhmann has joined #puppet-openstack | 19:50 | |
| mwhahaha | i think mine might be a v2 vs v3 thing | 19:51 |
| iurygregory | looks like the problem i'm having in https://review.openstack.org/#/c/216821/ | 19:51 |
| chem | iurygregory: this is not the same error... | 19:51 |
| iurygregory | oh ok =) | 19:51 |
| *** dgurtner has joined #puppet-openstack | 19:51 | |
| *** dgurtner has quit IRC | 19:51 | |
| *** dgurtner has joined #puppet-openstack | 19:51 | |
| chem | iurygregory: i think :) | 19:51 |
| mwhahaha | i think we had previously patched our implementation of keystone to do something slightly different so it might be a url parameter thing | 19:51 |
| mwhahaha | just was wondering if anyone had any thoughts, the googles returns something about needing to provide a v2 url instead of a v3 but i'm not sure which one is assumed by the keystone module | 19:52 |
| *** jerrygb has joined #puppet-openstack | 19:53 | |
| chem | iurygregory: for your error I think that the ci is not broken (I have no confirmation of it yet ... waiting for a recheck). but the error message is very unhelpful :) My guess is that the apache server is not properly restarted. I have to go now for ~1h, when I'm back I'll have a vm with your code in in running the beaker, so I will be able to fully debug it. See you in 1h30 :) | 19:54 |
| iurygregory | chem, ok ;) i'm leaving the office in about 1h but in about 3h:30min i'll be at home =) | 19:56 |
| iurygregory | you can ping me on irc ^^ | 19:56 |
| chem | richm: EmilienM : I have to update a lot of thing but basically I got this for review https://review.openstack.org/#/c/242198/ this should buy us some time for the fix in puppet-keystone to be implemented | 19:57 |
| EmilienM | looking | 19:57 |
| openstackgerrit | Merged openstack/puppet-cinder: Fix cinder default log_dir https://review.openstack.org/241691 | 19:57 |
| EmilienM | chem: the problem is upgrade I think | 19:57 |
| *** ntpttr1 has quit IRC | 19:57 | |
| EmilienM | chem: you should ask slague to look | 19:58 |
| chem | EmilienM I had the discussion with "bnemec" | 19:58 |
| EmilienM | ok | 19:58 |
| EmilienM | crinkle: what do you think about https://review.openstack.org/#/c/242162/ ? | 20:01 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Implement all-in-one.sh script https://review.openstack.org/242162 | 20:03 |
| *** bogdando has quit IRC | 20:08 | |
| crinkle | EmilienM: cool! i made a comment | 20:08 |
| *** jfluhmann has quit IRC | 20:09 | |
| openstackgerrit | Merged openstack/puppet-openstack_extras: spec: drop ::foo fake module https://review.openstack.org/240998 | 20:09 |
| EmilienM | crinkle: I don't think we need a Vagrantfile. People just need this kind of script, imho | 20:10 |
| crinkle | EmilienM: fine with me | 20:11 |
| openstackgerrit | Merged openstack/puppet-openstack_extras: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238631 | 20:11 |
| dgurtner | xarses: dachary dmsimard what's your thought on https://review.openstack.org/#/c/236539/ | 20:12 |
| dgurtner | xarses: dachary dmsimard personally I'm not entirely happy with disabling tests, but the RGW/CentOS issues are blocking everything else - and from my understanding it's not the module's fault. | 20:14 |
| dgurtner | mwhahaha: feel free to chime in ^ | 20:14 |
| *** degorenko has quit IRC | 20:15 | |
| *** ntpttr1 has joined #puppet-openstack | 20:15 | |
| *** ntpttr1 has quit IRC | 20:15 | |
| dmsimard | dgurtner: I was conflicted as well for that particular patchset. I think the cleanest action would be to split the test disable into another commit, file a bug and revert it when we figure the fix | 20:16 |
| dmsimard | And then rebase the other patches onto master once the test disable is merged | 20:17 |
| *** dgurtner has quit IRC | 20:19 | |
| *** dgurtner has joined #puppet-openstack | 20:20 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Implement all-in-one.sh script https://review.openstack.org/242162 | 20:20 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Implement all-in-one.sh script https://review.openstack.org/242162 | 20:21 |
| *** jfluhmann has joined #puppet-openstack | 20:26 | |
| dmsimard | dgurtner: thoughts ? | 20:30 |
| mwhahaha | probably would be a good idea to do the test one separately so it could be reverted at a later date. That being said I'd assume the changes required to make it work would probably require updating the tests anyway so i don't think relying on a revert is a must | 20:32 |
| dmsimard | I'm not exactly sure about that since the third party CI isn't broken | 20:35 |
| *** ntpttr1 has joined #puppet-openstack | 20:43 | |
| dgurtner | mwhahaha: it's impossible to do the test separately: to merge it needs to pass the gate - to be able to submit a new patchset the .gitreview change is required | 20:49 |
| *** aimon has quit IRC | 20:49 | |
| mwhahaha | no it's not | 20:49 |
| mwhahaha | you just need to have your gerrit setup correclty | 20:49 |
| mwhahaha | i've done it before would you like me to do it for you? | 20:49 |
| dgurtner | dmsimard: from my understanding we are now having one more issue on the RGW, before it just failed the swift check, but now it actually fails to start... | 20:49 |
| dgurtner | mwhahaha: I'd be happy to get pointers - my experience is you need .gitreview pointing to openstack/puppet-ceph, but if I change that it's an uncommited change in my repo and then gerrit refuses to submit the patch... | 20:50 |
| mwhahaha | git stash it before git review | 20:51 |
| mwhahaha | if you've already got a gerrit remote in your working dir it'll submit just fine | 20:51 |
| dgurtner | it didn't... maybe it's a difference if you commit on top of an existing patchset or create a new one? | 20:52 |
| dgurtner | and again, if I stash it, it refuses on the basis that the stackforge/puppet-ceph namespace is no longe known | 20:52 |
| mwhahaha | weird i managed to do it on another repo | 20:52 |
| mwhahaha | let me see | 20:52 |
| dgurtner | but please feel free to split the two patches and resubmit if you know how! having it stuffed together like that is just plain ugly | 20:53 |
| dmsimard | dgurtner: I'll give it a stab | 20:54 |
| dgurtner | dmsimard: given we don't even have civetweb support in the RGW I'm happy to remove the RGW test and concentrate on getting the core part of the module running | 20:54 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-ceph: Disable the RGW acceptance test https://review.openstack.org/242217 | 20:55 |
| mwhahaha | -^ | 20:55 |
| *** karimb has joined #puppet-openstack | 20:56 | |
| *** jmccrory has left #puppet-openstack | 20:57 | |
| dmsimard | he beat me to it :p | 20:57 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-horizon: Switch debian to openstack-dashboard https://review.openstack.org/242218 | 20:59 |
| dgurtner | mwhahaha++ | 21:00 |
| dgurtner | I still don't understand gerrit :/ | 21:00 |
| mwhahaha | http://goo.gl/YeZlt8 | 21:00 |
| openstackgerrit | David Moreau Simard proposed openstack/puppet-ceph: Disable the RGW acceptance test https://review.openstack.org/242217 | 21:02 |
| dmsimard | I added a bug reference ^ | 21:02 |
| *** dgurtner has quit IRC | 21:09 | |
| *** dgurtner has joined #puppet-openstack | 21:11 | |
| *** dgurtner has quit IRC | 21:11 | |
| *** dgurtner has joined #puppet-openstack | 21:11 | |
| EmilienM | mwhahaha: lol | 21:11 |
| EmilienM | I love dogs & computes | 21:12 |
| mwhahaha | :) | 21:12 |
| EmilienM | mwhahaha: http://goo.gl/DK6fWP | 21:12 |
| dgurtner | ¯\_(ツ)_/¯ | 21:16 |
| EmilienM | crinkle, mfisch: if you wanna have a look, https://review.openstack.org/#/c/242162/ is ready for review. I'll update the doc so people can start using it | 21:16 |
| *** rahulait has joined #puppet-openstack | 21:21 | |
| *** aimon has joined #puppet-openstack | 21:22 | |
| *** aimon has quit IRC | 21:22 | |
| *** mdorman has quit IRC | 21:25 | |
| *** jayg is now known as jayg|g0n3 | 21:25 | |
| *** mdorman has joined #puppet-openstack | 21:25 | |
| *** aimon has joined #puppet-openstack | 21:26 | |
| *** aimon has quit IRC | 21:33 | |
| chem | iurygregory: so it's definitively not the gate :) | 21:39 |
| chem | iurygregory: looking what is happening right now | 21:40 |
| *** dprince has quit IRC | 21:41 | |
| *** nlahouti has joined #puppet-openstack | 21:42 | |
| chem | iurygregory: http://fpaste.org/287445/67597751 | 21:43 |
| chem | iurygregory: that's what I though, apache is not restarting properly, but we don't see the full puppet log, which make this obvious: | 21:43 |
| chem | http://fpaste.org/287446/46759842 | 21:44 |
| chem | iurygregory: the full puppet log ^ | 21:44 |
| chem | iurygregory: but the undefined method `collect' for nil:NilClass is not helpful, I'll see if the error message can be improved here | 21:45 |
| *** mdorman has quit IRC | 21:50 | |
| *** dgurtner has quit IRC | 21:50 | |
| *** _morgan has quit IRC | 22:07 | |
| *** nlahouti has quit IRC | 22:08 | |
| *** nlahouti has joined #puppet-openstack | 22:08 | |
| *** bastelfreak has quit IRC | 22:09 | |
| chem | iurygregory: I've found the solution. Put it in the review :) | 22:09 |
| *** dhollinger has quit IRC | 22:10 | |
| *** dhollinger has joined #puppet-openstack | 22:10 | |
| EmilienM | chem: is https://review.openstack.org/#/c/238954/ ready? | 22:13 |
| *** dgurtner has joined #puppet-openstack | 22:13 | |
| *** dgurtner has joined #puppet-openstack | 22:13 | |
| chem | EmilienM: yeap | 22:14 |
| EmilienM | chem: lgtm | 22:14 |
| *** _morgan has joined #puppet-openstack | 22:15 | |
| *** _morgan has left #puppet-openstack | 22:15 | |
| chem | EmilienM: cool | 22:16 |
| openstackgerrit | David Moreau Simard proposed openstack/puppet-ceph: Update .gitreview for new namespace https://review.openstack.org/236539 | 22:17 |
| dmsimard | mwhahaha: that was quick | 22:18 |
| mwhahaha | :D | 22:18 |
| *** ntpttr1 has quit IRC | 22:19 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-gnocchi: Implement Gnocchi Statsd support https://review.openstack.org/240663 | 22:21 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-gnocchi: [DO NOT MERGE] test with pure RDO Liberty testing repo (stage CBS repos) https://review.openstack.org/242048 | 22:22 |
| *** bastelfreak has joined #puppet-openstack | 22:22 | |
| *** xarses has quit IRC | 22:29 | |
| *** cwolferh has quit IRC | 22:29 | |
| *** msimonin has joined #puppet-openstack | 22:35 | |
| *** dhollinger has quit IRC | 22:35 | |
| *** msimonin has left #puppet-openstack | 22:36 | |
| mwhahaha | i don't know what's up with the rubygems & openstack infra today | 22:36 |
| mwhahaha | all the puppet jobs have been super flakey :/ | 22:36 |
| dmsimard | rubygems is unstable | 22:40 |
| dmsimard | :) | 22:40 |
| *** karimb has quit IRC | 22:41 | |
| *** _ducttape_ has joined #puppet-openstack | 22:52 | |
| *** ducttape_ has quit IRC | 22:55 | |
| *** morazi has quit IRC | 22:56 | |
| openstackgerrit | Merged openstack/puppet-ceph: Disable the RGW acceptance test https://review.openstack.org/242217 | 23:06 |
| *** jerrygb has quit IRC | 23:08 | |
| *** dgurtner has quit IRC | 23:10 | |
| *** gildub has joined #puppet-openstack | 23:16 | |
| *** karimb has joined #puppet-openstack | 23:17 | |
| *** aimon has joined #puppet-openstack | 23:22 | |
| *** nibalizer has quit IRC | 23:24 | |
| *** jerrygb has joined #puppet-openstack | 23:33 | |
| *** karimb has quit IRC | 23:36 | |
| *** jerrygb_ has joined #puppet-openstack | 23:49 | |
| *** angdraug has quit IRC | 23:49 | |
| *** jerrygb has quit IRC | 23:50 | |
| *** dfisher has quit IRC | 23:52 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-heat: DNM - CI test https://review.openstack.org/242271 | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!