15:00:01 #startmeeting puppet-openstack 15:00:01 Meeting started Tue Dec 1 15:00:01 2015 UTC and is due to finish in 60 minutes. The chair is EmilienM. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:00:02 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:00:05 The meeting name has been set to 'puppet_openstack' 15:00:13 #link agenda: https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-20151201 15:00:14 o/ 15:00:15 o/ 15:00:15 hi 15:00:16 o/ 15:00:19 hi o/ 15:00:19 hi 15:00:33 #topic Review past action items 15:00:39 yottatsa follows-up aviator discussion on ML 15:00:51 yottatsa: I'm not sure I've seen that discussion 15:01:30 EmilienM: I'm in process of further aviator assessment, I will report right after it 15:01:37 good 15:01:46 we don't have a big agenda today 15:01:49 so I created a topic 15:01:54 #topic blueprint reviews 15:02:01 we have 2 blueprints 15:02:08 one for Mitaka from clayton : https://review.openstack.org/240515 15:02:21 #link blueprint: Enhance inifile option deprecations spec https://review.openstack.org/240515 15:02:57 people interested in this work, could have a look 15:03:20 I already +2'ed it, feel free to review 15:03:51 clayton: anything you wanted to discuss about that one? 15:04:02 hi 15:04:03 nope, but I'd be glad to answer questions 15:04:17 now, or later in #puppet-openstack if anyone would like 15:04:22 cool 15:04:31 #link Domain scoped object should have a domain name https://review.openstack.org/#/c/219127/ 15:04:45 this one is modifying a kilo blueprint 15:04:53 I'm not sure why we're doing that actually 15:05:02 gilles is offline, maybe chem knows? 15:05:24 no, It was a bit before I entered the project 15:05:44 I think this should go in liberty 15:06:03 I'm not sure we need to amend the blueprints after a release 15:06:09 morever, the code is already merged afik 15:06:28 EmilienM: that's correct. 15:06:32 +1 15:06:43 I'll comment on gerrit 15:06:58 ack 15:07:11 the agenda is done, I'll open the next topic 15:07:20 #topic Open Discussion, Bug and Review triage (submit modules to triage here) 15:07:50 there are a lot of patches about os_service_default, good to see the progress on it 15:08:00 Hi yeah 15:08:08 just wanted to ask about review 15:08:13 clayton: have you noticed some (internal) CI issues with that feature? 15:08:38 nope, we don't run the upstream tests when we do internal CI though, since we assume it has already passed 15:08:40 clayton: I know your company CI also provide good feedback 15:08:56 we do puppet catalog compiles, and we haven't seen issues there. 15:09:00 ok 15:09:42 is it everything ? : https://review.openstack.org/#/q/topic:os_service_default+status:open,n,z 15:09:57 Hi all, I just would like to mention that I started to push puppet-ec2api module https://review.openstack.org/#/c/251857/ 15:10:01 mkarpin: why don't use the same topic as others? 15:10:01 oh nope 15:10:12 mkarpin: it would be easier for reviewers 15:10:14 sorry i will use 15:10:15 it 15:10:19 mflobo: nice! 15:10:21 EmilienM, we are working to move the undef in some modules 15:10:40 mflobo: you'll also need to patch governance 15:11:05 EmilienM, yes, thanks 15:11:23 mflobo: ec2api is packaged in UCA or/and RDO ? 15:11:30 good luck mflobo =) 15:11:36 iurygregory: example? 15:11:52 EmilieM, acutally RDO RPM is on going https://bugzilla.redhat.com/show_bug.cgi?id=1272332 15:11:53 bugzilla.redhat.com bug 1272332 in Package Review "Review Request: openstack-ec2-api - Support of EC2 API for OpenStack" [Medium,On_qa] - Assigned to karlthered 15:12:08 great 15:12:45 https://trello.com/c/XLJJJBF0/71-move-modules-to-the-os-service-default-pattern 15:13:16 i've moved modules with db bug, now i'm working on keystone to move the undef to os_service_default 15:13:36 degorenko, sbadia: can you rev again https://review.openstack.org/#/c/250770/ ? I addressed a commont from denis 15:13:53 EmilienM: sure! 15:14:15 iurygregory: oh that's 'os_service_default', yeah 15:14:33 yeah =) we have a lot of work to do 15:14:48 iurygregory, degorenko: I was wondering, in all os_service_default patches, why unit tests are testing with value('SERVICE DEFAULT) instead of testing ensure absent? 15:15:09 _ody: are you around? I saw your's working on puppet4/beaker 15:16:08 let me see 15:16:34 EmilienM, because we have resource project_config with our fact 15:16:51 we should test not exactly config file 15:17:00 mhh 15:17:07 we should test what will be putted into config file 15:17:11 at the end, we want to make sure the resource is absent 15:17:26 degorenko: what is put in the config file is absent 15:17:27 that's true 15:17:34 so why don't we actually test it? 15:17:41 but we have resource with value 15:17:52 our fact - it is just a string 15:18:12 ok, it does not matter, I was just wondering :) 15:18:16 do we have anything else for today? 15:18:26 :) if you want discuss it, i can explain with examples 15:18:54 degorenko: yeah, we can follow-up on #puppet-openstack 15:18:58 sure 15:19:01 <_ody> EmilienM: I am. Got out of bed a few minutes late. 15:19:17 <_ody> I am going to go get coffee and wake up. 15:19:22 _ody: what's up about CI? 15:20:08 I think he left 15:20:21 I'm closing the meeting 15:20:21 <_ody> It works but I'd like to phase it by running old and new test in tandem on puppet-cinder for a week before replacing current beaker 15:20:25 ah 15:20:27 o/ 15:20:45 <_ody> We'll just need to talk about it later, to early for me. 15:20:59 _ody: could you remind us what you're doing? 15:21:13 _ody: ok, ttyl on our irc channel 15:21:18 have a great day everyone 15:21:20 #endmeeting