| *** markvoelker has quit IRC | 00:00 | |
| *** zhangjn has joined #puppet-openstack | 00:01 | |
| *** gfidente has joined #puppet-openstack | 00:01 | |
| *** gfidente has quit IRC | 00:01 | |
| *** gfidente has joined #puppet-openstack | 00:01 | |
| *** jerrygb has quit IRC | 00:03 | |
| *** ntpttr has joined #puppet-openstack | 00:08 | |
| *** zhangjn has quit IRC | 00:08 | |
| *** trown is now known as trown|outttypeww | 00:12 | |
| *** xarses has quit IRC | 00:21 | |
| *** ntpttr has quit IRC | 00:23 | |
| *** angdraug has quit IRC | 00:27 | |
| *** aimon has quit IRC | 00:29 | |
| *** aimon has joined #puppet-openstack | 00:29 | |
| *** gildub has quit IRC | 00:42 | |
| *** markvoelker has joined #puppet-openstack | 01:00 | |
| *** zhangjn has joined #puppet-openstack | 01:01 | |
| *** nlahouti has quit IRC | 01:03 | |
| *** aimon has quit IRC | 01:03 | |
| *** markvoelker has quit IRC | 01:05 | |
| *** gildub has joined #puppet-openstack | 01:29 | |
| *** aimon has joined #puppet-openstack | 01:32 | |
| *** ntpttr has joined #puppet-openstack | 01:36 | |
| *** jerrygb has joined #puppet-openstack | 01:38 | |
| *** ntpttr has left #puppet-openstack | 01:41 | |
| *** rahulait has joined #puppet-openstack | 01:54 | |
| *** zacdev has joined #puppet-openstack | 01:56 | |
| *** zacdev has quit IRC | 01:57 | |
| *** markvoelker has joined #puppet-openstack | 02:01 | |
| *** markvoelker has quit IRC | 02:06 | |
| *** sanjayu has joined #puppet-openstack | 02:13 | |
| *** gildub_ has joined #puppet-openstack | 02:23 | |
| *** ducttape_ has joined #puppet-openstack | 02:31 | |
| *** gildub_ has quit IRC | 02:32 | |
| *** nlahouti has joined #puppet-openstack | 02:36 | |
| *** nlahouti has quit IRC | 02:37 | |
| *** nlahouti has joined #puppet-openstack | 02:38 | |
| *** ducttape_ has quit IRC | 02:41 | |
| *** ducttape_ has joined #puppet-openstack | 02:42 | |
| *** gildub has quit IRC | 02:45 | |
| *** rahulait has quit IRC | 02:49 | |
| *** cwolferh has quit IRC | 02:56 | |
| *** rahulait has joined #puppet-openstack | 02:59 | |
| *** markvoelker has joined #puppet-openstack | 03:02 | |
| *** markvoelker has quit IRC | 03:07 | |
| *** ducttape_ has quit IRC | 03:08 | |
| *** nlahouti has quit IRC | 03:17 | |
| *** gildub has joined #puppet-openstack | 03:51 | |
| *** nlahouti has joined #puppet-openstack | 03:51 | |
| *** rahulait has left #puppet-openstack | 03:55 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Add composite namevar for tenant, user, user_role. https://review.openstack.org/226919 | 03:56 |
|---|---|---|
| *** sanjayu has quit IRC | 04:35 | |
| *** markvoelker has joined #puppet-openstack | 05:03 | |
| *** sanjayu has joined #puppet-openstack | 05:05 | |
| *** markvoelker has quit IRC | 05:07 | |
| *** guessi has joined #puppet-openstack | 05:19 | |
| *** nlahouti has quit IRC | 05:22 | |
| *** mwoodson has quit IRC | 05:22 | |
| *** guessi has left #puppet-openstack | 05:23 | |
| *** mwoodson has joined #puppet-openstack | 05:25 | |
| *** guessi has joined #puppet-openstack | 05:35 | |
| *** guessi has left #puppet-openstack | 05:35 | |
| *** nlahouti has joined #puppet-openstack | 05:37 | |
| *** jerrygb has quit IRC | 06:14 | |
| *** khushbu_ has joined #puppet-openstack | 06:15 | |
| *** sanjayu has quit IRC | 06:29 | |
| *** timfreund_ is now known as timfreund | 06:34 | |
| *** nlahouti has quit IRC | 06:34 | |
| *** nlahouti has joined #puppet-openstack | 06:36 | |
| *** jtomasek has quit IRC | 06:49 | |
| *** markvoelker has joined #puppet-openstack | 06:50 | |
| *** khushbu_ has quit IRC | 06:50 | |
| *** markvoelker has quit IRC | 06:55 | |
| *** rahulait has joined #puppet-openstack | 07:04 | |
| *** fedexo has joined #puppet-openstack | 07:11 | |
| *** jerrygb has joined #puppet-openstack | 07:15 | |
| *** jerrygb has quit IRC | 07:19 | |
| *** fedexo has quit IRC | 07:40 | |
| *** dgurtner has joined #puppet-openstack | 07:45 | |
| *** dgurtner has quit IRC | 07:45 | |
| *** dgurtner has joined #puppet-openstack | 07:45 | |
| *** sleinen-AS559 has joined #puppet-openstack | 07:46 | |
| *** xarses has joined #puppet-openstack | 07:50 | |
| *** zigo has quit IRC | 07:53 | |
| *** zigo has joined #puppet-openstack | 07:55 | |
| *** xarses has quit IRC | 07:57 | |
| *** xarses has joined #puppet-openstack | 07:58 | |
| *** paramite has joined #puppet-openstack | 08:02 | |
| *** nlahouti has quit IRC | 08:05 | |
| *** dgurtner has quit IRC | 08:07 | |
| *** rahulait has quit IRC | 08:11 | |
| *** jpena has joined #puppet-openstack | 08:40 | |
| *** paramite is now known as paramite|afk | 08:50 | |
| *** markvoelker has joined #puppet-openstack | 08:52 | |
| *** paramite|afk is now known as paramite | 08:55 | |
| *** markvoelker has quit IRC | 08:56 | |
| *** jistr has joined #puppet-openstack | 09:00 | |
| *** dgurtner has joined #puppet-openstack | 09:02 | |
| *** dgurtner has quit IRC | 09:07 | |
| *** jerrygb has joined #puppet-openstack | 09:16 | |
| *** jerrygb has quit IRC | 09:21 | |
| *** karimb has joined #puppet-openstack | 09:22 | |
| *** jtomasek has joined #puppet-openstack | 09:22 | |
| *** dgurtner has joined #puppet-openstack | 09:36 | |
| *** derekh has joined #puppet-openstack | 09:45 | |
| *** jtomasek has quit IRC | 09:52 | |
| *** jtomasek has joined #puppet-openstack | 09:52 | |
| *** markvoelker has joined #puppet-openstack | 09:53 | |
| *** markvoelker has quit IRC | 09:58 | |
| *** IvanBerezovskiy has joined #puppet-openstack | 10:00 | |
| *** samuelBartel has joined #puppet-openstack | 10:03 | |
| openstackgerrit | Mark Chappell proposed openstack/puppet-tripleo: Add hiera calls to allow management of ports and protocols https://review.openstack.org/240208 | 10:06 |
| *** dgurtner has quit IRC | 10:07 | |
| *** zhangjn has quit IRC | 10:12 | |
| *** zhangjn has joined #puppet-openstack | 10:13 | |
| *** zhangjn has quit IRC | 10:17 | |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-nova: [WIP] https://review.openstack.org/233011 | 10:31 |
| *** xarses has quit IRC | 10:33 | |
| *** zhangjn has joined #puppet-openstack | 10:38 | |
| *** zhangjn has quit IRC | 10:38 | |
| *** zhangjn has joined #puppet-openstack | 10:40 | |
| *** samuelBartel has quit IRC | 11:07 | |
| *** markvoelker has joined #puppet-openstack | 11:09 | |
| *** markvoelker has quit IRC | 11:13 | |
| *** khushbu_ has joined #puppet-openstack | 11:19 | |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-nova: [WIP] https://review.openstack.org/233011 | 11:19 |
| *** khushbu_ has quit IRC | 11:21 | |
| *** khushbu_ has joined #puppet-openstack | 11:32 | |
| *** khushbu_ has quit IRC | 11:37 | |
| *** skolekonov has joined #puppet-openstack | 11:38 | |
| *** khushbu_ has joined #puppet-openstack | 11:41 | |
| *** khushbu_ has quit IRC | 11:43 | |
| *** khushbu_ has joined #puppet-openstack | 11:49 | |
| *** khushbu_ has quit IRC | 11:50 | |
| *** khushbu has joined #puppet-openstack | 11:54 | |
| *** khushbu has quit IRC | 11:54 | |
| *** khushbu has joined #puppet-openstack | 11:58 | |
| *** khushbu has quit IRC | 11:59 | |
| *** jerrygb has joined #puppet-openstack | 12:03 | |
| *** jerrygb has quit IRC | 12:03 | |
| *** xarses has joined #puppet-openstack | 12:03 | |
| *** markvoelker has joined #puppet-openstack | 12:09 | |
| *** dgurtner has joined #puppet-openstack | 12:09 | |
| *** dgurtner has quit IRC | 12:09 | |
| *** dgurtner has joined #puppet-openstack | 12:09 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Add composite namevar for tenant, user, user_role. https://review.openstack.org/226919 | 12:12 |
| *** markvoelker has quit IRC | 12:14 | |
| *** chem has quit IRC | 12:22 | |
| *** chem has joined #puppet-openstack | 12:26 | |
| *** chem` has joined #puppet-openstack | 12:29 | |
| *** zhangjn has quit IRC | 12:31 | |
| *** chem has quit IRC | 12:33 | |
| *** EmilienM has quit IRC | 12:33 | |
| *** zhangjn has joined #puppet-openstack | 12:34 | |
| *** EmilienM has joined #puppet-openstack | 12:36 | |
| *** chem` has quit IRC | 12:37 | |
| *** trown|outttypeww is now known as trown | 12:41 | |
| *** morazi has quit IRC | 12:42 | |
| *** ducttape_ has joined #puppet-openstack | 12:43 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Client and dashboard class implementation https://review.openstack.org/218333 | 12:43 |
| *** xarses has quit IRC | 12:46 | |
| *** xarses has joined #puppet-openstack | 12:47 | |
| *** khushbu_ has joined #puppet-openstack | 12:47 | |
| *** khushbu_ has quit IRC | 12:48 | |
| *** jayg|g0n3 is now known as jayg | 12:48 | |
| *** dgurtner has quit IRC | 12:48 | |
| *** dgurtner has joined #puppet-openstack | 12:53 | |
| *** khushbu_ has joined #puppet-openstack | 12:53 | |
| *** khushbu_ has quit IRC | 12:55 | |
| *** dprince has joined #puppet-openstack | 12:56 | |
| *** jerrygb has joined #puppet-openstack | 13:01 | |
| *** khushbu_ has joined #puppet-openstack | 13:05 | |
| *** khushbu_ has quit IRC | 13:07 | |
| *** sleinen-AS559 has quit IRC | 13:13 | |
| EmilienM | good morning | 13:16 |
| *** asilenkov has quit IRC | 13:16 | |
| *** ducttape_ has quit IRC | 13:17 | |
| *** samuelBartel has joined #puppet-openstack | 13:17 | |
| *** asilenkov has joined #puppet-openstack | 13:19 | |
| *** degorenko has joined #puppet-openstack | 13:20 | |
| *** abramley_ has joined #puppet-openstack | 13:22 | |
| *** LimorStotland has quit IRC | 13:22 | |
| *** abramley has quit IRC | 13:22 | |
| *** abramley_ is now known as abramley | 13:22 | |
| *** gildub has quit IRC | 13:22 | |
| vinsh | heyooo | 13:22 |
| mkarpin1 | Hey! | 13:22 |
| *** LimorStotland has joined #puppet-openstack | 13:22 | |
| *** gildub has joined #puppet-openstack | 13:22 | |
| *** mkarpin1 has quit IRC | 13:23 | |
| *** mkarpin has joined #puppet-openstack | 13:23 | |
| *** richm has joined #puppet-openstack | 13:29 | |
| *** jpena is now known as jpena|lunch | 13:39 | |
| *** sleinen-AS559 has joined #puppet-openstack | 13:41 | |
| *** LimorStotland has left #puppet-openstack | 13:43 | |
| *** sleinen-AS559 has quit IRC | 13:45 | |
| *** delatte has quit IRC | 13:48 | |
| *** khushbu_ has joined #puppet-openstack | 13:55 | |
| *** khushbu_ has quit IRC | 13:55 | |
| *** dgurtner has quit IRC | 13:55 | |
| *** sleinen-AS559 has joined #puppet-openstack | 13:59 | |
| EmilienM | mwhahaha: everyone is trying to use $::os_service_default | 14:02 |
| EmilienM | I sent an email to stop it | 14:02 |
| EmilienM | we should only use puppet-cinder for now | 14:03 |
| EmilienM | mwhahaha: do you know which bug spredzy was talking about? | 14:04 |
| *** markvoelker has joined #puppet-openstack | 14:10 | |
| *** morazi has joined #puppet-openstack | 14:15 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-cinder: Support of PyMySQL driver for MySQL backend https://review.openstack.org/241214 | 14:15 |
| *** markvoelker has quit IRC | 14:15 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-cinder: Support of PyMySQL driver for MySQL backend https://review.openstack.org/241214 | 14:16 |
| degorenko | EmilienM, +1 :) | 14:16 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-horizon: Do not run collectstatic or compress in Debian https://review.openstack.org/241218 | 14:22 |
| EmilienM | degorenko: I replied on https://review.openstack.org/238623 | 14:22 |
| *** vinsh_ has joined #puppet-openstack | 14:22 | |
| EmilienM | I'm not sure we have an alternative | 14:22 |
| degorenko | EmilienM, erm, it is for one project. Patch dependency? | 14:23 |
| EmilienM | degorenko: yes but the first one will fail CI and will never merge | 14:23 |
| degorenko | EmilienM, oh, crap | 14:23 |
| EmilienM | the second one will pass, ok but what about the first? :) | 14:23 |
| degorenko | EmilienM, you're right, sorry | 14:23 |
| degorenko | my bad | 14:23 |
| EmilienM | I'm not sure it's a big deal | 14:23 |
| degorenko | yeah, i agree | 14:24 |
| *** nlahouti has joined #puppet-openstack | 14:24 | |
| *** vinsh has quit IRC | 14:25 | |
| *** nlahouti1 has joined #puppet-openstack | 14:27 | |
| *** nlahouti has quit IRC | 14:28 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-horizon: Do not run collectstatic or compress in Debian https://review.openstack.org/241218 | 14:32 |
| zigo | EmilienM: Could you give me comments on my patch just above? | 14:32 |
| zigo | That's my first puppet patch, so it's probably very bad... :P | 14:32 |
| mwhahaha | it's not so bad :D | 14:34 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-horizon: Do not run collectstatic or compress in Debian https://review.openstack.org/241218 | 14:34 |
| zigo | I'm gessing I should have written "Debian" and not "debian", as facter returns "Ubuntu" (with upper case U), so here comes iteration 3 of the patch... | 14:35 |
| degorenko | zigo, what is package_type? | 14:35 |
| degorenko | i mean, where patch for this? | 14:35 |
| zigo | A variable set if you install this package: http://anonscm.debian.org/cgit/openstack/puppet-openstack-debian-fact.git/tree/ | 14:36 |
| zigo | :) | 14:36 |
| *** delattec has joined #puppet-openstack | 14:40 | |
| EmilienM | oh nice | 14:41 |
| degorenko | zigo, yea, but also we are discussed about adding patch to openstacklib | 14:41 |
| degorenko | which will be override operating system if we need | 14:41 |
| EmilienM | mwhahaha: any thoughts about os_service_default? | 14:41 |
| degorenko | will* | 14:41 |
| degorenko | and you don't need any additional if's in params | 14:42 |
| mwhahaha | EmilienM: i think it's fine provided we properly check the variables being used have a sane default or if we have previously set something within puppet that we do not convert those (re: log dirs, log levels, etc) | 14:42 |
| *** nlahouti1 has quit IRC | 14:42 | |
| mwhahaha | EmilienM: I think the logging problem was that we converted it to os_service_default which the code interprets as no logging, so we shouldn't have converted those. | 14:43 |
| EmilienM | mwhahaha: I don't see where is the bug, since we still have log files in our jobs | 14:43 |
| mwhahaha | EmilienM: in my patches for the cinder backends I did not convert the items where we had previously specified something that was different than the default | 14:43 |
| mwhahaha | yea i haven't seen it either i'll take some time today to poke at it | 14:44 |
| *** jpena|lunch is now known as jpena | 14:45 | |
| EmilienM | mwhahaha: do you think we could go ahead with all other openstack modules? | 14:46 |
| EmilienM | mwhahaha: do you have an example or link about the logging problem? | 14:47 |
| mwhahaha | yes we just need to be diligent to make sure the settings don't incur any regressions | 14:47 |
| mwhahaha | i scrolled through my history and spredzy had mentioned two things | 14:50 |
| mwhahaha | spredzy: EmilienM: gchamoul mwhahaha the issue are those two lines https://github.com/openstack/puppet-cinder/blob/master/manifests/logging.pp#L96-L97 | 14:50 |
| mwhahaha | spredzy: Default in oslo.log is set to None, so by default it doesn't log anything in any log file, while it uses to be /var/log/cinder.log, so it breaks backward compat | 14:50 |
| mwhahaha | so we just need to revert those two parameters and when we convert things we need to make sure that what we're converting is currently set to undef or the actual default value | 14:51 |
| mwhahaha | if it's not the actual default value we'll want to come up with some sort of migration strategy or we just leave it | 14:52 |
| *** dfisher has quit IRC | 14:55 | |
| *** dfisher has joined #puppet-openstack | 14:55 | |
| skolekonov | EmilienM, hi, are you going to hold today's meeting? | 15:03 |
| EmilienM | skolekonov: why not? | 15:03 |
| skolekonov | I see nobody at #openstack-meeting-4 | 15:03 |
| skolekonov | Is it later? | 15:04 |
| EmilienM | I do | 15:04 |
| EmilienM | it's in one hour | 15:04 |
| EmilienM | I think? | 15:04 |
| skolekonov | Hm, I thought it's 15-00 UTC now | 15:05 |
| IvanBerezovskiy | me2 | 15:05 |
| *** vinsh has joined #puppet-openstack | 15:07 | |
| *** bogdando has quit IRC | 15:09 | |
| *** vinsh_ has quit IRC | 15:11 | |
| skolekonov | hobana.freenode.net :Tuesday November 3 2015 -- 15:10:48 +00:00 | 15:11 |
| *** bogdando has joined #puppet-openstack | 15:12 | |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-nova: [WIP] https://review.openstack.org/233011 | 15:13 |
| EmilienM | I think you're right, damn | 15:14 |
| EmilienM | clayton, crinkle, mfisch, degorenko: around? | 15:15 |
| mwhahaha | silly clocks | 15:15 |
| EmilienM | skolekonov, mwhahaha : let's do it | 15:16 |
| *** degorenko has quit IRC | 15:17 | |
| skolekonov | EmilienM, please look at this patch https://review.openstack.org/#/c/235356/, are you ok with it? | 15:25 |
| *** markvoelker has joined #puppet-openstack | 15:26 | |
| *** vinsh_ has joined #puppet-openstack | 15:26 | |
| IvanBerezovskiy | EmilienM: why did you miss 2 last topics? :-( | 15:27 |
| EmilienM | skolekonov: I'm reviewing it | 15:27 |
| skolekonov | ok, thanks | 15:28 |
| EmilienM | IvanBerezovskiy: ? | 15:28 |
| EmilienM | IvanBerezovskiy: the agenda was on https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-20151103 | 15:28 |
| EmilienM | I only see 2 topics | 15:28 |
| EmilienM | for open discussion, it's up to people to come up and talk | 15:28 |
| IvanBerezovskiy | I've just waited my turn to start discussion about pymysql | 15:29 |
| *** vinsh has quit IRC | 15:29 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Add basic acceptance test https://review.openstack.org/233591 | 15:29 |
| IvanBerezovskiy | https://review.openstack.org/#/c/241214/ | 15:29 |
| EmilienM | IvanBerezovskiy: don't worry, we can talk here if you want | 15:30 |
| EmilienM | it's in my review list for today | 15:30 |
| *** markvoelker has quit IRC | 15:31 | |
| IvanBerezovskiy | EmilienM: thx. I just wanna know you thoughts about it. If you are ok, I'll do this for all modules | 15:31 |
| EmilienM | IvanBerezovskiy: ok, looking now | 15:31 |
| *** ducttape_ has joined #puppet-openstack | 15:32 | |
| openstackgerrit | Nate Potter proposed openstack/puppet-neutron: Allow customizations of db sync command line https://review.openstack.org/241012 | 15:34 |
| EmilienM | skolekonov: I reviewed https://review.openstack.org/#/c/235356/ - the patch is almost good for me, please read comments and tell me if that's fine for you | 15:36 |
| *** sleinen-AS559 has quit IRC | 15:36 | |
| *** sleinen-AS559 has joined #puppet-openstack | 15:37 | |
| *** jistr has quit IRC | 15:37 | |
| skolekonov | EmilienM, if we leave 'password' value without default value, we will break backward compatibility, won't we? Because somebody can still want to use only deprecated nova_admin_password and related parameters | 15:38 |
| EmilienM | IvanBerezovskiy: why no package on centos? is it MySQL-python-1.2.3-11.el7.x86_64 ? | 15:38 |
| EmilienM | ah | 15:39 |
| EmilienM | I see | 15:39 |
| EmilienM | nice catch | 15:39 |
| EmilienM | skolekonov: just the test case then | 15:39 |
| EmilienM | IvanBerezovskiy: +2, lgtm | 15:40 |
| IvanBerezovskiy | EmilienM: no, it's not. latest version of pymysql is 0.6.6 and as I didn't find this package for centos | 15:40 |
| IvanBerezovskiy | thx again | 15:41 |
| skolekonov | EmilienM, I modified the existing test, now it's called 'should configure neutron server with deprecated parameters'. Should I add an additional one? | 15:41 |
| EmilienM | IvanBerezovskiy: oh wait | 15:41 |
| EmilienM | IvanBerezovskiy: can you patch acceptance? :) | 15:41 |
| skolekonov | EmilienM, and added new tests for new parameters. Diff looks a bit confusing I guess | 15:41 |
| IvanBerezovskiy | EmilienM: hm, I guess I can, but I don't exactly know where to patch it :) | 15:42 |
| EmilienM | IvanBerezovskiy: in spec/acceptance/ | 15:43 |
| IvanBerezovskiy | looks like here https://github.com/openstack/puppet-cinder/blob/master/spec/acceptance/basic_cinder_spec.rb#L100 | 15:43 |
| EmilienM | IvanBerezovskiy: yes, make sure we're using the new string | 15:43 |
| IvanBerezovskiy | ok, will do | 15:43 |
| *** sergmelikyan has joined #puppet-openstack | 15:46 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-cinder: Support of PyMySQL driver for MySQL backend https://review.openstack.org/241214 | 15:46 |
| IvanBerezovskiy | let's see how it will work on centos | 15:46 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service https://review.openstack.org/237662 | 15:46 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Client class https://review.openstack.org/237706 | 15:47 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Notifier service https://review.openstack.org/237708 | 15:47 |
| *** rahulait has joined #puppet-openstack | 15:49 | |
| *** dh64 has quit IRC | 15:53 | |
| openstackgerrit | Nate Potter proposed openstack/puppet-glance: Allow customization of db sync command line https://review.openstack.org/241003 | 15:53 |
| *** ntpttr has joined #puppet-openstack | 15:56 | |
| mfisch | meeting? | 16:01 |
| mfisch | oh damnit time changes! | 16:01 |
| vinsh_ | oh yeah! | 16:01 |
| *** chem has joined #puppet-openstack | 16:01 | |
| mfisch | sorry EmilienM | 16:01 |
| mfisch | damn you benjamin franklin! | 16:01 |
| mfisch | and your terrible ideas | 16:01 |
| *** jistr has joined #puppet-openstack | 16:01 | |
| vinsh_ | I'm sure he is still eating his morning crepes | 16:01 |
| vinsh_ | le crepes | 16:02 |
| EmilienM | lol | 16:02 |
| EmilienM | we did the meeting | 16:02 |
| mfisch | note to PTL, remind stupid americans about time changes!!! | 16:02 |
| EmilienM | I missed it too | 16:02 |
| EmilienM | I'm stupid :) | 16:02 |
| EmilienM | hey mfisch, can you look https://review.openstack.org/#/q/topic:puppet/dropfixtures+status:open,n,z also please? | 16:02 |
| *** angdraug has joined #puppet-openstack | 16:04 | |
| mfisch | oui mon ami | 16:05 |
| EmilienM | mfisch: petit? :) | 16:05 |
| mfisch | petit chu | 16:05 |
| *** zhangjn has quit IRC | 16:09 | |
| *** IvanBerezovskiy has quit IRC | 16:13 | |
| *** skolekonov has quit IRC | 16:17 | |
| *** sleinen-AS559 has quit IRC | 16:18 | |
| *** fedexo has joined #puppet-openstack | 16:18 | |
| *** dprince has quit IRC | 16:20 | |
| _ody | yaaaawwn... | 16:21 |
| _ody | bummer slept through the meeting. | 16:22 |
| *** IvanBerezovskiy has joined #puppet-openstack | 16:22 | |
| openstackgerrit | Nate Potter proposed openstack/puppet-ceilometer: Allow customization of db sync command line https://review.openstack.org/241276 | 16:23 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: DNM - CI test https://review.openstack.org/241277 | 16:24 |
| social | mfisch: i think the isdue isnt people but daylight saving. makes no sense | 16:24 |
| mfisch | EmilienM: this patch says to rely on git to pull stuff with r10k? | 16:25 |
| *** paramite has quit IRC | 16:25 | |
| EmilienM | mfisch: yes | 16:27 |
| EmilienM | we merged it in master a few time ago | 16:27 |
| EmilienM | mfisch: but we need to backport it | 16:27 |
| EmilienM | the stable branch CI is currently broken | 16:27 |
| jistr | does someone have bandwidth to give a second core review on this? https://review.openstack.org/#/c/239680/ | 16:31 |
| jistr | it should fix undefined ordering on the resources in heat::keystone::domain class and make it defined | 16:32 |
| EmilienM | jistr: do you think we can merge https://review.openstack.org/237612 ? ooo job is red | 16:32 |
| jistr | the red tripleo job shouldn't be a big issue as we have a patch ready to make it work https://review.openstack.org/#/c/239707/ | 16:33 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Remove occurrence of @@default_domain_id. https://review.openstack.org/241284 | 16:33 |
| *** skolekonov has joined #puppet-openstack | 16:33 | |
| jistr | EmilienM: ^^ that was for the patch i posted, not for the one you posted | 16:33 |
| mfisch | EmilienM: dumb questions: where's the file that r10k uses to pull from? | 16:33 |
| jistr | EmilienM: i'll look at the other one | 16:33 |
| EmilienM | mfisch: the Puppetfile in puppet-openstack-integration | 16:34 |
| EmilienM | mfisch: those patches are backports, master already has this code BTW | 16:34 |
| EmilienM | mwhahaha: I +A https://review.openstack.org/236004 so we can make progress on this | 16:34 |
| mwhahaha | ok | 16:35 |
| mwhahaha | i'll take a look at the comments left on the backend review today | 16:35 |
| *** skolekonov is now known as _skolekonov | 16:37 | |
| jistr | EmilienM: the tripleo jobs should be mostly stable right now. I see from the CI log that the error on that heat patch is Error: Could not find resource 'Anchor[heat::db::begin]' for relationship on 'Class[Heat::Db::Mysql]' | 16:37 |
| jistr | doesn't seem related to the patch itself though... i'll trigger a recheck and we'll see | 16:38 |
| EmilienM | jistr: mhhk | 16:38 |
| jistr | a lot of green lately http://tripleo.org/cistatus.html | 16:39 |
| mfisch | EmilienM: ah ok | 16:41 |
| jistr | EmilienM: hmm that puppet-heat patch says Parent(s): 3b9cdd01bcccee0f24edfcbf1f50fa81d0cd8352Merge "Prepare 6.1.0 release." | 16:42 |
| openstackgerrit | Nate Potter proposed openstack/puppet-sahara: Allow customization of db sync command line https://review.openstack.org/241290 | 16:42 |
| jistr | that might be a reason why it failed too... i think we had a pin of puppet-heat because of a different breakage recently | 16:43 |
| jistr | maybe rebasing that change to the latest master could make it pass | 16:43 |
| *** nlahouti has joined #puppet-openstack | 16:44 | |
| jistr | can this be done with the "rebase change" button in gerrit? | 16:44 |
| * jistr has never used it yet | 16:44 | |
| mfisch | EmilienM: why is the horizon one of these reverting? | 16:44 |
| EmilienM | mfisch: have you looked at the commit msg? | 16:45 |
| *** markvoelker has joined #puppet-openstack | 16:45 | |
| mfisch | not yet just saw it on the main list | 16:45 |
| mfisch | getting there | 16:45 |
| EmilienM | mfisch: let me look | 16:46 |
| *** chem has quit IRC | 16:46 | |
| EmilienM | I think denis did the patch | 16:46 |
| *** nlahouti has quit IRC | 16:46 | |
| mfisch | ah I see its a redo of a CP | 16:46 |
| EmilienM | mfisch: so the initial patch was wrong, denis did a mistake in the backprt | 16:46 |
| EmilienM | mfisch: when he backported my patch, he did an error in rebase | 16:47 |
| EmilienM | I want to revert the patch and re-do the backport | 16:47 |
| EmilienM | so it's cleaner | 16:47 |
| *** chem has joined #puppet-openstack | 16:47 | |
| openstackgerrit | Merged openstack/puppet-cinder: add kombu_reconnect_delay parameter to cinder https://review.openstack.org/239907 | 16:47 |
| openstackgerrit | Nate Potter proposed openstack/puppet-cinder: Allow customization of db sync command line https://review.openstack.org/240963 | 16:48 |
| _skolekonov | EmilienM, I've answered to your comment in https://review.openstack.org/#/c/235356, please see if it's what you mean | 16:48 |
| openstackgerrit | Nate Potter proposed openstack/puppet-nova: Allow customization of db sync command line https://review.openstack.org/241009 | 16:49 |
| EmilienM | _skolekonov: I'm dumb | 16:50 |
| EmilienM | _skolekonov: +a | 16:50 |
| openstackgerrit | Nate Potter proposed openstack/puppet-ceilometer: Allow customization of db sync command line https://review.openstack.org/241276 | 16:50 |
| _skolekonov | EmilienM, :) thank you | 16:51 |
| EmilienM | jistr: I would monitor https://review.openstack.org/#/c/235356/ and see if tripleo survives with it | 16:51 |
| EmilienM | _skolekonov: please take care of updating puppet-openstack-integration. We don't like warnings | 16:51 |
| openstackgerrit | Merged openstack/puppet-designate: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238627 | 16:51 |
| _skolekonov | EmilienM, I will | 16:51 |
| *** cwolferh has joined #puppet-openstack | 16:52 | |
| openstackgerrit | Merged openstack/puppet-cinder: Move default values to $::os_service_default https://review.openstack.org/236004 | 16:53 |
| IvanBerezovskiy | EmilienM: do you know folks who are working on packaging for cbs.centos.org ? I wanna ask them to build python-pymysql. for now I've found it only in fedora http://pkgs.fedoraproject.org/cgit/python-PyMySQL.git/tree/ | 16:53 |
| EmilienM | IvanBerezovskiy: go on #rdo and ask | 16:54 |
| EmilienM | ping number80 or apevec | 16:54 |
| IvanBerezovskiy | EmilienM: thx! | 16:54 |
| jistr | EmilienM: well... :D i'm hoping for it | 16:56 |
| openstackgerrit | Merged openstack/puppet-glance: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238616 | 16:58 |
| *** nlahouti has joined #puppet-openstack | 16:58 | |
| *** nlahouti has quit IRC | 16:59 | |
| *** nlahouti has joined #puppet-openstack | 17:00 | |
| openstackgerrit | Merged openstack/puppet-neutron: Deprecate nova_admin_ options https://review.openstack.org/235356 | 17:02 |
| EmilienM | jistr: yeah.. me too | 17:03 |
| myatsenko | EmilienM: hi Emilien! can you review it again https://review.openstack.org/#/c/233339/ and set +1 for Workflow to merge it. | 17:03 |
| EmilienM | myatsenko: so its only for nova network in puppet-nova? | 17:04 |
| myatsenko | EmilienM: yes, its needed for nova , for this one: https://review.openstack.org/#/c/233011/ | 17:05 |
| EmilienM | myatsenko: why not patching puppet-nova then? | 17:05 |
| EmilienM | the Gem is not used/useful for puppet-openstack-integration AFIK | 17:05 |
| *** samuelBartel has quit IRC | 17:06 | |
| openstackgerrit | Nate Potter proposed openstack/puppet-heat: Allow customization of db sync command line https://review.openstack.org/241299 | 17:08 |
| *** ntpttr has quit IRC | 17:08 | |
| chem | richm: EmilienM should be good to be merged, but lacks another core https://review.openstack.org/#/c/219289/ | 17:13 |
| myatsenko | EmilienM: yeh, to pass tests I updated Gemfile in openstack/puppet-nova patch , but I was told that correct way - > add this new requerement to openstack/puppet-openstack-integration project. Thus this patch was created https://review.openstack.org/#/c/233339/, but if correct way to solve this problem - add changes only to Gemfile of puppet-nova project, I will abondone patch for "openstack/puppet-openstack-integration" | 17:13 |
| karimb | hello buddies, shouldnt the neutron::agents::ml2::ovs class take care of creating br-tun bridge ? | 17:14 |
| karimb | provided i m setting enable_tunneling => true | 17:14 |
| clayton | I put the time for the meeting on my calendar as UTC, but I had a doctor's appointment at the same time | 17:14 |
| EmilienM | br-tun is created by neutron AFIK | 17:14 |
| EmilienM | clayton: np, hope all is fine | 17:14 |
| *** ducttape_ has quit IRC | 17:15 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Add composite namevar for tenant, user, user_role. https://review.openstack.org/226919 | 17:15 |
| *** rahulait has quit IRC | 17:15 | |
| *** ducttape_ has joined #puppet-openstack | 17:15 | |
| *** fedexo has quit IRC | 17:18 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: gemfile: Added netaddr package https://review.openstack.org/233339 | 17:19 |
| karimb | EmilienM what do you mean by neutron, neutron class ? | 17:20 |
| EmilienM | karimb: no, neutron ovs agent already take care of this | 17:20 |
| EmilienM | not puppet | 17:20 |
| EmilienM | neutron | 17:20 |
| karimb | ah ok | 17:20 |
| karimb | not happening here | 17:21 |
| _ody | I had this issue when I was setting things up from scrath a couple months ago using a new composite layer...sorry really can't remember | 17:23 |
| _ody | The two major problems I had was trying to use the master branch on kilo just doesn't work so I rolled back to 6.x and iptables. | 17:23 |
| _ody | OpenStack doesn't add the iptables rules for gre or vxlan and neither do the puppet modules. | 17:24 |
| _ody | Not a problem in our production deployment since we use midonet. | 17:24 |
| *** jtomasek has quit IRC | 17:25 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Add composite namevar for tenant, user, user_role. https://review.openstack.org/226919 | 17:25 |
| *** karimb has quit IRC | 17:26 | |
| *** vinsh has joined #puppet-openstack | 17:27 | |
| *** vinsh_ has quit IRC | 17:27 | |
| tdb | I'm having an issue with the puppet-ceph module. Mon resources keep reapplying. It looks like the problem this patch is trying to fix: https://review.openstack.org/#/c/151969/ . But that's months old, so makes me wonder if I'm missing something? | 17:29 |
| *** aimon has quit IRC | 17:30 | |
| mwhahaha | since that patch was never merged, you're probably not missing something | 17:31 |
| tdb | so either other people are ignoring the fact the resources reapply on every Puppet run, or I'm doing something differently? | 17:32 |
| mwhahaha | or people aren't doing it via puppet or that module | 17:33 |
| * mwhahaha shrugs | 17:33 | |
| tdb | of course; it won't affect people not using the module :) | 17:33 |
| mwhahaha | was going to take a look at fixing up that patch, seems the puppet-ceph module is a bit neglected | 17:34 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-trove: Fix rabbit_userid parameter https://review.openstack.org/241308 | 17:35 |
| EmilienM | jpena: ^ | 17:35 |
| tdb | mwhahaha: I'm just building a new Ceph cluster, and since all our other stuff is Puppet driven I thought it'd make sense to use the module rather than make a new one internally. | 17:36 |
| mwhahaha | no you're right in trying to use the puppet-ceph module rather then writing your own | 17:37 |
| mwhahaha | it seems it needs some love | 17:37 |
| tdb | but not if the module isn't getting a lot of love :) | 17:37 |
| tdb | I suppose I'm just raising it here to see if the module is still alive, or if everyone has moved on to something else I haven't seen | 17:38 |
| mwhahaha | well it seems that no one bothered to pay attention to the ci failures or merge conflicts | 17:38 |
| mwhahaha | I know we want to move to the puppet-ceph module so it'll get more attention in the upcoming future | 17:39 |
| *** ntpttr has joined #puppet-openstack | 17:39 | |
| *** dgurtner has joined #puppet-openstack | 17:40 | |
| *** dgurtner has quit IRC | 17:40 | |
| *** dgurtner has joined #puppet-openstack | 17:40 | |
| tdb | I'm happy to give it a try and report issues or give patches where I can | 17:41 |
| tdb | currently the patch I linked above is the only problem I'm having | 17:41 |
| openstackgerrit | Bradley Klein proposed openstack/puppet-monasca: Change default monasca api port to 8072 to avoid murano conflict https://review.openstack.org/241313 | 17:42 |
| mwhahaha | yea i'm taking a look at rebasing that | 17:42 |
| mwhahaha | to see if it can be salvaged | 17:42 |
| tdb | if you do I'll give it a try | 17:44 |
| EmilienM | jayg: I remember you mentionned to use Vagrant to build an openstack cloud with our modules, have you seen our outcome of a summit session? we want to help newcomers about that | 17:46 |
| *** dprince has joined #puppet-openstack | 17:51 | |
| *** jpena is now known as jpena|away | 17:52 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-ceph: fix mon creation to be more idempotent https://review.openstack.org/151969 | 17:54 |
| mwhahaha | tdb: try -^ and let me know if that doesn't work | 17:55 |
| tdb | k, will do | 17:56 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Remove occurrence of @@default_domain_id. https://review.openstack.org/241284 | 17:57 |
| *** bklei has joined #puppet-openstack | 17:59 | |
| *** derekh has quit IRC | 18:00 | |
| openstackgerrit | Merged openstack/puppet-ironic: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238596 | 18:02 |
| openstackgerrit | Merged openstack/puppet-tuskar: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238140 | 18:02 |
| *** aimon has joined #puppet-openstack | 18:03 | |
| openstackgerrit | Merged openstack/puppet-neutron: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238615 | 18:04 |
| *** trown is now known as trown|lunch | 18:09 | |
| *** IvanBerezovskiy has left #puppet-openstack | 18:11 | |
| *** rahulait has joined #puppet-openstack | 18:16 | |
| openstackgerrit | Merged openstack/puppet-glance: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238619 | 18:16 |
| EmilienM | clayton, mfisch: can you look https://review.openstack.org/#/c/241308/ ? it's fixing a bug in puppet-trove, I'm trying to bring back our integration testing alive | 18:18 |
| openstackgerrit | Merged openstack/puppet-horizon: Revert "Try to use zuul-cloner to prepare fixtures" https://review.openstack.org/240714 | 18:19 |
| tdb | mwhahaha: looks like you missed a bit of the original patch | 18:20 |
| tdb | but even with that, it still does these each time: | 18:20 |
| tdb | Notice: /Stage[main]/Main/Node[death]/Ceph::Mon[death]/File[/tmp/ceph-mon-keyring-death]/ensure: defined content as '{md5}b1b65a0cb1950831065b0a70bc9ed7ca' | 18:20 |
| tdb | Notice: /Stage[main]/Main/Node[death]/Ceph::Mon[death]/Exec[rm-keyring-death]/returns: executed successfully | 18:20 |
| tdb | but the original patch didn't address that either | 18:21 |
| mwhahaha | k i'll take a look at that as well, i'm trying to get the CI working as it appears to be broken | 18:21 |
| EmilienM | so integration testing is now failing because of second puppet run that is not idempotent on centos: http://logs.openstack.org/08/241308/1/check/gate-puppet-openstack-integration-scenario001-dsvm-centos7/f0dc430/console.html#_2015-11-03_17_57_05_804 | 18:21 |
| openstackgerrit | Merged openstack/puppet-tempest: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238154 | 18:22 |
| EmilienM | Hunner, _ody: ^ | 18:23 |
| EmilienM | what is managing /var/lib/puppet/state/* files? | 18:23 |
| mwhahaha | tdb: most of the other items from that review were addressed in other patches, what's left is the stuff that isn't already in the code base i think | 18:23 |
| EmilienM | it sounds like the SElinux context is wrong | 18:23 |
| openstackgerrit | Merged openstack/puppet-keystone: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238605 | 18:24 |
| tdb | mwhahaha: it's the bit at line 156 of mon.pp in patch set 5 that you were missing, and isn't already there | 18:25 |
| mwhahaha | ah got lost in the move | 18:26 |
| tdb | mwhahaha: let me know how it goes. there's not much point me looking at it if you're doing it, but if you give up I'll take a stab. | 18:26 |
| mwhahaha | i'll get that unless added back in | 18:26 |
| EmilienM | oh that's weird, puppet is installed in our integration jobs and it should not | 18:28 |
| *** angdraug has quit IRC | 18:30 | |
| jayg | EmilienM: just got back from lunch, that is good news about the making things easier for newcomers! | 18:31 |
| EmilienM | jayg: I was wondering if you're interested to contribute to it | 18:32 |
| openstackgerrit | Merged openstack/puppet-openstacklib: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238610 | 18:33 |
| EmilienM | crinkle: should not we stop installing puppet with spec/acceptance_helper and rely on Gems ? | 18:33 |
| EmilienM | http://logs.openstack.org/08/241308/1/check/gate-puppet-trove-puppet-beaker-rspec-dsvm-centos7/88abff6/console.html#_2015-11-03_17_48_48_756 | 18:34 |
| dmsimard | dgurtner: so I haven't had time to figure out why those centos tests are failing - I'll guess no one has had any either? :p | 18:34 |
| jayg | EmilienM: sure, if I have anything of use, I would be happy to contribute | 18:34 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-ceph: fix mon creation to be more idempotent https://review.openstack.org/151969 | 18:34 |
| dmsimard | mwhahaha: oh, please let me know if you figure it out :) | 18:36 |
| mwhahaha | :D | 18:36 |
| mwhahaha | my money is on no since beaker comes up with different errors locally than what the CI does | 18:36 |
| * mwhahaha flips tables | 18:36 | |
| *** bnemec has quit IRC | 18:37 | |
| *** dprince has quit IRC | 18:38 | |
| _ody | EmilienM: Best to my knowledge only the puppet agent should be managing /var/lib/puppet/state/* files. | 18:38 |
| *** jistr has quit IRC | 18:39 | |
| EmilienM | _ody: we don't run agent | 18:39 |
| EmilienM | but I think I found the bug | 18:40 |
| EmilienM | it's the version of puppet that we use | 18:40 |
| _ody | EmilienM: Sorry I use the words "puppet agent" and "puppet apply" interchangeably. | 18:40 |
| EmilienM | mhh | 18:41 |
| *** bnemec has joined #puppet-openstack | 18:41 | |
| EmilienM | that's weird we did not have this pb before | 18:42 |
| EmilienM | _ody: regarding the logs, can you confirm that's why second puppet run is not idempotent? | 18:42 |
| EmilienM | I'm not sure | 18:43 |
| EmilienM | _ody: I can't find why the second puppet run is not idempotent | 18:44 |
| _ody | looking | 18:46 |
| mwhahaha | dmsimard: are the ceph packages used centos7 compatible? | 18:52 |
| openstackgerrit | Bradley Klein proposed openstack/puppet-monasca: Change default monasca api port to 8070 to avoid murano conflict https://review.openstack.org/241313 | 18:53 |
| dmsimard | mwhahaha: I'm not familiar with the CentOS7 installation of Ceph :( | 18:53 |
| mwhahaha | i think that's why the ci is broken | 18:54 |
| mwhahaha | the systemd stuff looks broken | 18:54 |
| dmsimard | mwhahaha: you're unable to reproduce locally ? | 18:55 |
| mwhahaha | no i got it | 18:56 |
| mwhahaha | BEAKER_destroy=no BEAKER_set=centos-70-x64 bundle exec rake beaker | 18:57 |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-nova: [WIP] https://review.openstack.org/233011 | 18:59 |
| *** nlahouti has quit IRC | 19:00 | |
| openstackgerrit | Merged openstack/puppet-sahara: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238599 | 19:01 |
| *** sergmelikyan has quit IRC | 19:04 | |
| openstackgerrit | Merged openstack/puppet-heat: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238609 | 19:04 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: CI test - do not merge https://review.openstack.org/241334 | 19:08 |
| *** angdraug has joined #puppet-openstack | 19:08 | |
| _ody | EmilienM: I don't have beaker setup right now but I dug through our internal ticketing for PE customers and found the selinux contexts flip around if you don't have selinux bindings installed for ruby but according to the puppet package dependencies in epel, they should be being installed. | 19:09 |
| *** trown|lunch is now known as trown | 19:09 | |
| _ody | wait...we use gems don't we in testing | 19:09 |
| EmilienM | I thought we did | 19:11 |
| EmilienM | but no | 19:11 |
| EmilienM | so for beaker jobs, we're purging puppet packaging but our acceptance helper is installing it again: http://logs.openstack.org/08/241308/1/check/gate-puppet-trove-puppet-beaker-rspec-dsvm-centos7/88abff6/console.html#_2015-11-03_17_48_48_756 cc crinkle | 19:12 |
| EmilienM | and for integration jobs, we even don't run the macro that purge puppet package: https://review.openstack.org/241329 | 19:12 |
| EmilienM | crinkle: should we patch our beaker config to stop installing puppet ? | 19:13 |
| *** gfidente is now known as gfidente|afk | 19:13 | |
| crinkle | EmilienM: no? it should be installing puppet | 19:13 |
| crinkle | why is that a problem? | 19:13 |
| *** sergmelikyan has joined #puppet-openstack | 19:14 | |
| _ody | Yeah. If uninstall it, it should be fine. | 19:14 |
| EmilienM | crinkle: I've been investigating a bug for ~3 hours and trying to figure out why our second puppet run (integration testing on centos7) is failing | 19:14 |
| EmilienM | I mean, not failing, but not idempotent | 19:14 |
| EmilienM | I managed to reproduced it and it seems it's something in the version of Puppet installed by packaging | 19:14 |
| EmilienM | http://logs.openstack.org/08/241308/1/check/gate-puppet-openstack-integration-scenario001-dsvm-centos7/f0dc430/logs/rpm-qa.txt.gz | 19:15 |
| EmilienM | puppet-3.6.2-3.el7.noarch | 19:15 |
| EmilienM | and http://logs.openstack.org/08/241308/1/check/gate-puppet-trove-puppet-beaker-rspec-dsvm-centos7/88abff6/logs/rpm-qa.txt.gz | 19:15 |
| EmilienM | puppet-3.8.3-1.el7.noarch | 19:15 |
| EmilienM | the bug is not in puppet-3.8.3-1.el7.noarch because beaker jobs are green | 19:16 |
| EmilienM | but integration is red, which is using 3.6 | 19:16 |
| crinkle | the integration tests should install puppet, since the beaker jobs are installing puppet | 19:16 |
| EmilienM | crinkle: so you can -1 my patch | 19:16 |
| EmilienM | or no | 19:16 |
| EmilienM | you can still +1 | 19:16 |
| EmilienM | but we need to handle the puppet package in our integration testing | 19:16 |
| crinkle | yes | 19:17 |
| EmilienM | and make sure we do it the same way as beaker | 19:17 |
| EmilienM | crinkle: ok? | 19:17 |
| EmilienM | great, I'm doing it | 19:17 |
| crinkle | yes | 19:17 |
| EmilienM | thx crinkle | 19:17 |
| EmilienM | we still have a bug in puppet 3.6 though | 19:17 |
| EmilienM | I'll put it on my todo and trying to report it upstream | 19:17 |
| *** nlahouti has joined #puppet-openstack | 19:23 | |
| _ody | EmilienM: The difference between integration and beaker looks to be puppetlabs-release-7-11 vs. puppetlabs-release-7-10. | 19:24 |
| dgurtner | dmsimard: correct ;) | 19:24 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo https://review.openstack.org/241340 | 19:24 |
| EmilienM | _ody, crinkle ^ | 19:25 |
| dgurtner | dmsimard: i think the issue is that 1) there's not enough resources (of some kind) 2) there's a timing issue 3) there's already something on that port | 19:25 |
| dgurtner | dmsimard: but it seems that apache fails to start and then the request to test fails | 19:26 |
| dgurtner | dmsimard: maybe the easiest is to just remove rgw from the test set and instead concentrate on the base module functionality to work | 19:26 |
| mwhahaha | dgurtner: what are you referring to? | 19:27 |
| EmilienM | _ody: stupid copy paste, thx | 19:29 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo https://review.openstack.org/241340 | 19:29 |
| dgurtner | mwhahaha: http://logs.openstack.org/62/230362/1/check/gate-puppet-ceph-puppet-beaker-rspec-dsvm-centos7/bfb76f1/console.html.gz | 19:31 |
| mwhahaha | oh i've traced it down to radosgw won't start | 19:31 |
| vinsh | gah... learning how to run spec tests now that they are all r10k.. not as simple as before | 19:31 |
| vinsh | keep getting some script dir or puppet dir or something wrong | 19:31 |
| dgurtner | mwhahaha: right... but why? | 19:31 |
| mwhahaha | no idea trying to figure it out :D | 19:32 |
| dgurtner | mwhahaha: because it does start _sometimes_ | 19:32 |
| mwhahaha | it's a config with that test on centos7 | 19:32 |
| mwhahaha | specifically the 'should install one monitor/osd with a rgw and keystone integration' | 19:32 |
| mwhahaha | from ceph_rgw_spec.rb | 19:32 |
| dgurtner | mwhahaha: also clearly it works on the external CI: http://paste.ubuntu.com/12637643/ | 19:32 |
| mwhahaha | centos7 vs ubuntu packaging? | 19:32 |
| mwhahaha | systemd of fail? | 19:33 |
| mwhahaha | :D | 19:33 |
| dgurtner | mwhahaha: the exact same code has 100% success on the external CI system | 19:33 |
| mwhahaha | same packages? | 19:34 |
| mwhahaha | no it's not | 19:35 |
| dgurtner | which package differs? apache? | 19:35 |
| mwhahaha | so it's failing with 0.94.5 | 19:35 |
| mwhahaha | [00;00m[00;00m Notice: /Stage[main]/Main/Package[ceph-radosgw]/ensure: ensure changed '0.94.3-0.el7.centos' to 'purged' | 19:35 |
| mwhahaha | ceph-radosgw-0.94.5-0.el7.centos.x86_64 | 19:35 |
| dgurtner | theoretically should/could be the same... I was lazy to search through successful/failing tests | 19:36 |
| dgurtner | mwhahaha: here: http://logs.openstack.org/62/230362/1/check/gate-puppet-ceph-puppet-beaker-rspec-dsvm-centos7/8537272/console.html.gz | 19:37 |
| dgurtner | failing with 0.94.3-0.el7.centos | 19:38 |
| *** jerrygb has quit IRC | 19:39 | |
| *** _ody has quit IRC | 19:40 | |
| dgurtner | mwhahaha: are you using the rgw code anywhere? or do you see any added value in testing it? | 19:41 |
| mwhahaha | fuel uses rgw but we're not currently using puppet-ceph as we have our own | 19:41 |
| mwhahaha | the problem is we want to migrate to puppet-ceph so we'll need it | 19:41 |
| mwhahaha | the errors the ci is currently throwing is the radosgw not running | 19:42 |
| mwhahaha | so i'm not sure the historical ci issues match what is currently happening | 19:42 |
| dgurtner | mwhahaha: I can trigger a rerun with current code (you can as well, just "recheck") | 19:43 |
| mwhahaha | i know, i'm able to reproduce the radosgw not starting in a local vm and i'm looking into it now | 19:43 |
| dgurtner | mwhahaha: one other issue is that rgw can urn with apache and civetweb | 19:46 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo https://review.openstack.org/241340 | 19:46 |
| mwhahaha | yea i think we run it with apache | 19:46 |
| dgurtner | mwhahaha: and we dont provide civetweb config yet, so maybe the rgw module is outdated anyways? | 19:46 |
| mwhahaha | oh what the, i downgraded all the 0.94.5 packages to .3 and then updated them back to .5 and now it's working | 19:55 |
| dgurtner | mwhahaha: it's working _sometimes_ as mentioned before :P | 19:55 |
| openstackgerrit | Nate Potter proposed openstack/puppet-designate: Allow customization of db sync command line https://review.openstack.org/241351 | 19:55 |
| mwhahaha | (╯°□°)╯︵ ɥdǝɔ | 19:55 |
| dgurtner | mwhahaha: there's a lot of pain involved with this particular puppet module :P | 19:56 |
| *** jerrygb has joined #puppet-openstack | 19:57 | |
| mwhahaha | i wonder if there's an action in the packaging that has changed | 19:59 |
| openstackgerrit | Javier Peña proposed openstack/puppet-vswitch: Set NAME as well as DEVICE in ifcfg-* https://review.openstack.org/241354 | 20:02 |
| openstackgerrit | Rohit Pagedar proposed openstack/puppet-nova: Making instance_name_template configurable. https://review.openstack.org/234972 | 20:02 |
| openstackgerrit | Javier Peña proposed openstack/puppet-vswitch: Set NAME as well as DEVICE in ifcfg-* https://review.openstack.org/241354 | 20:03 |
| *** jtomasek has joined #puppet-openstack | 20:11 | |
| openstackgerrit | Rohit Pagedar proposed openstack/puppet-nova: Making instance_name_template configurable. https://review.openstack.org/234972 | 20:19 |
| *** paramite has joined #puppet-openstack | 20:20 | |
| vinsh | keystone module changes have broken puppet-swift spec tests. | 20:22 |
| vinsh | I'll put up a review to fix this | 20:23 |
| mwhahaha | dgurtner: i think it's the systemd script, manually running it seems to work but trying to start it via systemd seems like it's not working correctly | 20:23 |
| *** sergmelikyan has quit IRC | 20:24 | |
| vinsh | Author: Gilles Dubreuil <gilles@redhat.com> | 20:27 |
| vinsh | Date: Tue Sep 29 17:19:49 2015 +1000 | 20:27 |
| vinsh | broke it. :) | 20:27 |
| *** nlahouti has quit IRC | 20:28 | |
| *** nlahouti has joined #puppet-openstack | 20:29 | |
| mwhahaha | dgurtner: i think it's because the initscript attempts to start radosgw using a su command to 'apache' which is disabled on centos7 | 20:30 |
| *** nlahouti has quit IRC | 20:31 | |
| mwhahaha | dgurtner: https://github.com/ceph/ceph/pull/6271/ | 20:33 |
| openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Remove tenant parameter from keystone_user https://review.openstack.org/241360 | 20:37 |
| openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Remove tenant parameter from keystone_user https://review.openstack.org/241360 | 20:37 |
| *** cwolferh has quit IRC | 20:38 | |
| dgurtner | mwhahaha: so the answer is to wait until chuanhong, dachary and sage get this merged? | 20:39 |
| mwhahaha | probably | 20:40 |
| mwhahaha | seems to be a systemd thing since centos7 is the problematic test. we could switch beaker centos7 to non-voting for now maybe? | 20:40 |
| dgurtner | it would be nice to make individual test scenarios optional | 20:43 |
| dgurtner | I'm somewhat torn between making centos7 non-voting or disabling the rgw patch | 20:44 |
| dgurtner | s/patch/test/ | 20:44 |
| mwhahaha | it'd probably be best to comment out that scenario for now so we're still testing some things or switch it to pending maybe? | 20:47 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo https://review.openstack.org/241340 | 20:48 |
| *** jpena|away has quit IRC | 20:56 | |
| *** paramite has quit IRC | 20:59 | |
| *** _ody has joined #puppet-openstack | 21:00 | |
| *** cwolferh has joined #puppet-openstack | 21:02 | |
| EmilienM | _ody: what is the diff between https://yum.puppetlabs.com/puppetlabs-release-pc1-el-7.noarch.rpm and https://yum.puppetlabs.com/puppetlabs-release-pc1-el-7.noarch.rpm ? | 21:03 |
| *** dgurtner has quit IRC | 21:05 | |
| EmilienM | I wonder what is Puppet Collections | 21:05 |
| *** dgurtner has joined #puppet-openstack | 21:06 | |
| crinkle | it's how puppet-agent is distributed for puppet 4 | 21:09 |
| crinkle | so puppet, facter, hiera, etc are bundled together | 21:09 |
| EmilienM | crinkle: got it, thx | 21:10 |
| *** jayg is now known as jayg|g0n3 | 21:10 | |
| EmilienM | crinkle: if you have little time, do you see something wrong in https://jenkins04.openstack.org/job/gate-puppet-openstack-integration-scenario001-dsvm-centos7/90/consoleFull ? | 21:10 |
| EmilienM | the job is still failing even with puppet 3.8 | 21:11 |
| _ody | EmilienM: 1. you posted the same link twice. 2. What crinkle said. | 21:11 |
| EmilienM | _ody: oops :) | 21:11 |
| crinkle | EmilienM: idk it's super hard to find anything with the debug output on | 21:12 |
| *** dgurtner has quit IRC | 21:12 | |
| _ody | Puppet Collections makes distro packaging hard but everything else easier. | 21:12 |
| *** dgurtner_ has joined #puppet-openstack | 21:12 | |
| EmilienM | _ody: I thought the job would not fail but it does again. I don't see anything wrong in the puppet run | 21:12 |
| _ody | EmilienM: Well it was a bug that needed to be fixed anyway... | 21:12 |
| EmilienM | crinkle: could it be at 21:04:49.545 ? | 21:12 |
| EmilienM | _ody: yeah | 21:12 |
| EmilienM | crinkle: it looks like the 3 seluser things make the puppet run not idempotent | 21:13 |
| *** cwolferh has quit IRC | 21:13 | |
| crinkle | EmilienM: if something changed it would be a Notice: not a Debug: | 21:14 |
| EmilienM | I did not know that, ool | 21:14 |
| EmilienM | cool* | 21:14 |
| EmilienM | crinkle: maybe 2015-11-03 21:04:57.616 Notice: /Stage[main]/Nova/Package[python-greenlet]/ensure: created | 21:15 |
| EmilienM | in puppet-nova | 21:16 |
| crinkle | EmilienM: yep | 21:16 |
| crinkle | so it's a packaging issue | 21:16 |
| EmilienM | I would like to cry | 21:16 |
| *** jerrygb has quit IRC | 21:16 | |
| EmilienM | it sounds like it's installed in the first puppet run though | 21:17 |
| _ody | ha! Good eye crinkle, I didn't eve think about that. | 21:17 |
| crinkle | sometimes apt behaves weird with puppet | 21:18 |
| EmilienM | yeah, I did not know I had to look at "Notice" for second puppet run | 21:18 |
| crinkle | is python-greenlet a virtual package maybe? | 21:18 |
| EmilienM | crinkle: it's yum | 21:18 |
| crinkle | oh | 21:18 |
| crinkle | then idk | 21:18 |
| _ody | Yeah. If this was apt I'd say python-greenlet was a virtual package. | 21:18 |
| EmilienM | I'm investigating | 21:18 |
| openstackgerrit | David Gurtner proposed openstack/puppet-ceph: Update .gitreview for new namespace https://review.openstack.org/236539 | 21:19 |
| EmilienM | oh I think I got it | 21:20 |
| EmilienM | http://logs.openstack.org/40/241340/4/check/gate-puppet-openstack-integration-scenario001-dsvm-centos7/9ba9d77/logs/syslog.txt.gz | 21:20 |
| EmilienM | Nov 03 20:54:36 template-devstack-centos7-1446560094.novalocal yum[3578]: Installed: python2-greenlet-0.4.9-1.el7.x86_64 | 21:20 |
| EmilienM | it's python2-greenlet | 21:20 |
| dgurtner_ | mwhahaha: lets see if this helps... https://review.openstack.org/#/c/236539/2/spec/acceptance/ceph_rgw_spec.rb | 21:20 |
| EmilienM | crinkle, _ody : so puppet think python-greenlet is not installed while it is, with python2-greenlet name | 21:20 |
| EmilienM | maube that's it | 21:21 |
| *** delattec has quit IRC | 21:21 | |
| EmilienM | _ody: it looks like a bug in Package resource with Yum provider | 21:22 |
| EmilienM | I'm trying to reproduce | 21:22 |
| dgurtner_ | mwhahaha: I worry we still suffer from https://review.openstack.org/#/c/230362/ or duplicate https://review.openstack.org/#/c/233066/ | 21:23 |
| *** cwolferh has joined #puppet-openstack | 21:24 | |
| mwhahaha | i did't run into those errors | 21:24 |
| mwhahaha | so who knows | 21:24 |
| dgurtner_ | mwhahaha: also additionally painful that I need to squeeze this into the stackforge->openstack ns change, as I can't submit it otherwise | 21:27 |
| mwhahaha | not true but we need that done anyway :D | 21:27 |
| openstackgerrit | Merged openstack/puppet-trove: Fix rabbit_userid parameter https://review.openstack.org/241308 | 21:27 |
| *** nlahouti has joined #puppet-openstack | 21:28 | |
| EmilienM | _ody: http://projects.puppetlabs.com/issues/5831 | 21:29 |
| EmilienM | that's the bug we're hitting I think | 21:29 |
| dgurtner_ | mwhahaha: it's just ugly | 21:31 |
| mwhahaha | i know | 21:31 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: do not manage python-greenlet anymore https://review.openstack.org/241373 | 21:33 |
| nibalizer | anyone feeling up to reviewing a report processor? https://review.openstack.org/#/c/240459/3/modules/openstack_project/lib/puppet/reports/puppetdb_file.rb hunner spun that up at the infra hack day | 21:34 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo https://review.openstack.org/241340 | 21:34 |
| EmilienM | nibalizer: wow, that's a nice feature | 21:36 |
| EmilienM | nibalizer: why not pushing this kind of code in an upstream module/lib/repo? | 21:36 |
| Hunner | EmilienM: It's actually kind of specialized, I think | 21:38 |
| openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider https://review.openstack.org/203220 | 21:38 |
| Hunner | EmilienM: The puppetdb module tries to support masterless but assumes nodes can contact the puppetdb service | 21:38 |
| Hunner | EmilienM: It would be complicated to try and give a generic solution where the nodes can't talk to it though | 21:38 |
| vinsh | EmilienM: Need to merge: https://review.openstack.org/#/c/241360/ swift unit tests broken without it. | 21:38 |
| EmilienM | oh I missed that one? | 21:40 |
| EmilienM | I thought we fixed all modules | 21:40 |
| EmilienM | mfisch: ^ | 21:40 |
| openstackgerrit | David Gurtner proposed openstack/puppet-ceph: Update .gitreview for new namespace https://review.openstack.org/236539 | 21:42 |
| dgurtner_ | mwhahaha: ok, it's betternow except you can't purge what you never added, so ^ | 21:42 |
| *** nlahouti has quit IRC | 21:49 | |
| *** cwolferh has quit IRC | 21:49 | |
| *** cwolferh has joined #puppet-openstack | 21:54 | |
| *** nlahouti has joined #puppet-openstack | 21:58 | |
| *** nlahouti has quit IRC | 21:58 | |
| dgurtner_ | mwhahaha: look here :) https://review.openstack.org/#/c/236539/ | 21:59 |
| _ody | EmilienM: Looks like it. TBH I am still pretty new to EL and didn't even think it had virtual packages. I run into this all the time on debian, where it is also not fixed. | 21:59 |
| mwhahaha | hey it's green, woo | 21:59 |
| *** nlahouti has joined #puppet-openstack | 22:01 | |
| *** ntpttr has quit IRC | 22:03 | |
| EmilienM | ok, integration jobs should be back with https://review.openstack.org/#/c/241340/ and its dependencies | 22:04 |
| *** dtalton has joined #puppet-openstack | 22:05 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-cinder: Update cinder backends to use os_service_default https://review.openstack.org/239800 | 22:10 |
| EmilienM | crinkle: if you don't mind to review https://review.openstack.org/#/c/241340/ | 22:16 |
| EmilienM | and the dependency in puppet-nova | 22:17 |
| *** jerrygb has joined #puppet-openstack | 22:17 | |
| *** xarses has quit IRC | 22:18 | |
| *** jerrygb has quit IRC | 22:22 | |
| nibalizer | EmilienM: so it might eventually end up in a module/lib/repo but for now incubating in system-config makes sense | 22:22 |
| nibalizer | since it might need unforseen changes | 22:23 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo https://review.openstack.org/241340 | 22:27 |
| openstackgerrit | Merged openstack/puppet-ceilometer: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238613 | 22:29 |
| *** jerrygb has joined #puppet-openstack | 22:32 | |
| *** dgurtner_ has quit IRC | 22:37 | |
| *** dgurtner has joined #puppet-openstack | 22:40 | |
| *** dgurtner has quit IRC | 22:40 | |
| *** dgurtner has joined #puppet-openstack | 22:40 | |
| *** morazi has quit IRC | 22:41 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: loadbalancer: add Aodh API support https://review.openstack.org/241395 | 22:43 |
| *** rahulait has quit IRC | 22:45 | |
| openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider https://review.openstack.org/203220 | 22:46 |
| *** _ducttape_ has joined #puppet-openstack | 22:47 | |
| *** ducttape_ has quit IRC | 22:47 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo https://review.openstack.org/241340 | 22:48 |
| patrickeast | hi, i'm trying to get a all-in-one liberty setup going with the puppet modules and am seeing errors like: http://paste.openstack.org/show/477914/ | 22:49 |
| patrickeast | anyone know what might be causing that? | 22:49 |
| *** dgurtner has quit IRC | 22:51 | |
| patrickeast | for reference this is what my manifest looks like for the neutron part (where that error came from) http://paste.openstack.org/show/477915/ | 22:51 |
| *** dgurtner has joined #puppet-openstack | 22:52 | |
| *** dgurtner has quit IRC | 22:52 | |
| *** dgurtner has joined #puppet-openstack | 22:52 | |
| *** _ducttape_ has quit IRC | 22:52 | |
| *** ducttape_ has joined #puppet-openstack | 22:53 | |
| *** bklei has quit IRC | 22:53 | |
| *** ducttape_ has quit IRC | 22:58 | |
| *** rahulait has joined #puppet-openstack | 22:58 | |
| openstackgerrit | Merged openstack/puppet-swift: Remove tenant parameter from keystone_user https://review.openstack.org/241360 | 23:06 |
| *** nlahouti has quit IRC | 23:08 | |
| *** cwolferh has quit IRC | 23:10 | |
| *** nlahouti has joined #puppet-openstack | 23:11 | |
| *** cwolferh has joined #puppet-openstack | 23:23 | |
| *** gildub has quit IRC | 23:35 | |
| *** angdraug has quit IRC | 23:35 | |
| *** jerrygb has quit IRC | 23:43 | |
| *** nlahouti1 has joined #puppet-openstack | 23:43 | |
| *** nlahouti has quit IRC | 23:43 | |
| openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider https://review.openstack.org/203220 | 23:51 |
| *** guessi has joined #puppet-openstack | 23:52 | |
| patrickeast | hmm so if i'm reading this right you need to have keystone v3 available for this to work https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone_user/openstack.rb#L89-L126 | 23:54 |
| *** angdraug has joined #puppet-openstack | 23:55 | |
| patrickeast | which is maybe a problem since it appears following this as an example sets me up for v2 endpoints https://wiki.openstack.org/wiki/Puppet/Deploy#Keystone | 23:56 |
| *** guessi has left #puppet-openstack | 23:57 | |
| patrickeast | the first time applying the manifest things were ok, the service user/tenant are there and things are working, but subsequent apply's fail | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!