15:00:39 #startmeeting puppet-openstack 15:00:39 Meeting started Tue Jun 16 15:00:39 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:40 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:00:42 The meeting name has been set to 'puppet_openstack' 15:00:45 hola 15:00:51 who is here today? 15:01:04 hey 15:01:07 dang it 15:01:09 I am here for the first 10 mins or so. 15:01:46 hi 15:01:48 o/ 15:02:17 hi 15:02:30 o/ 15:02:46 #topic review past actions 15:03:01 spredzy,claryton create a POC and send an email to the ML about parameter default policy 15:03:14 hi 15:03:24 I have no clue about this one ^ 15:03:28 but I guess it's WIP 15:03:29 Same status as last week :( 15:03:40 cody to finish neutron patch -> still WIP, need reviews though 15:04:00 #link https://review.openstack.org/#/c/184646/ 15:04:00 I don't see _ody here this morning. 15:04:08 #link https://review.openstack.org/#/c/189873/ 15:04:15 need review *and* rebase 15:04:28 sbadia to patch swift module for puppet4 - MERGED 15:04:39 \o/ 15:04:52 crinkle to close this ML thread and explain our decision 15:05:00 crinkle: what was it about again? 15:05:06 #link https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-20150616 15:05:15 i think that was about the change abandonment policy 15:05:22 ah yeah, nice 15:05:25 so it's DONE 15:05:27 o/ 15:05:39 spredzy Send a patch as an implementation reference (dbsync exec) -> I think it's not done yet 15:06:21 same for the thread 15:06:29 spredzy|afk is out anyway 15:06:35 #topic CI status 15:06:52 beaker is almost voting: https://review.openstack.org/#/c/190778 \o/ 15:07:11 the only thing I noticed until now is an issue with puppet-sahara 15:07:14 on trusty 15:07:23 I'll figure that out today or so 15:07:54 puppet4: we're still waiting for cody's patches to land and we can enable voting for puppet4 jobs 15:08:03 anything else about CI ? 15:08:32 ok seems no 15:08:33 i'm working on a poc to use zuul-cloner so we can test dependent patches, hopefully have something today 15:08:45 in beaker 15:08:52 oh, that is awesome 15:09:43 crinkle: any blocker on it now? 15:09:51 EmilienM: nope 15:09:55 crinkle: ok, thx 15:09:57 #topic migration status 15:10:00 crinkle: o/ 15:10:10 thanks for having worked on that with infra team 15:10:49 I had to *exceptionally* +2 +A some patches to have CI back to work 15:11:24 it seems Havana and Grizzly CIs are broken for some modules, we figured that during the migration 15:11:31 because of lack of review ? 15:11:32 yha, no need to wait for 2x +2 to +A such changes 15:11:37 ah yes a question related 15:11:54 about patchs for .gitreview on havana branchs (what we do?) 15:11:55 sbadia: yes, crinkle sent patches on Friday night but nobody reviewed them yesterday 15:12:03 sbadia: so I landed them myself 15:12:13 spec tests are failing on theses branchs 15:12:23 yeah... Not sure we should spend time on it 15:12:24 EmilienM: ok, sorry and thanks! 15:12:28 ok ok 15:12:32 why do we care if Grizzly is broken? 15:12:34 I don't see much point in fixing them personally 15:12:59 mfisch: well, if people are still using it and want to backport something, they can figure how to fix the CI 15:13:09 +1 15:13:10 mfisch: I think we did not officially stopped the support for it yet 15:13:12 well, they likely won't figure it out 15:13:13 but I don't think anyone here cares about havana/grizzly, and only for some modules. 15:13:24 so we will likely have to help 15:13:24 mgagne: maybe a good action to take 15:13:29 wfm. I don't think those releases (or icehouse) are still supported by any vendors 15:13:50 #action Emilien to run a thread on ML about some stable branches deprecations (grizzly+havana) 15:13:54 we are still supporting icehouse 15:14:00 but that's our own issue 15:14:11 yea, we have still to move from icehouse here 15:14:19 crinkle: anything else to notice about migration? Do we have any blockers? 15:14:43 EmilienM: i think we're all set 15:15:07 cool, and thanks again for that. 15:15:08 Unfortunately, I have to run. I will review the remainder of the meeting when I get back to my computer (more specifically on any actions from Zaqar or proposed ML change). 15:15:15 #topic Zaqar module 15:15:19 <_ody> o/ 15:15:22 RichardRaseley: o/ 15:16:11 ok, we will postpone this one, RichardRaseley|A had to go 15:16:23 #action postpone this topic when RichardRaseley is around 15:16:45 we have a topic but I don't know who wrote this 15:16:50 #topic How to deal with distributions merging multiple services in one package (RHEL) 15:17:13 mgagne: maybe ? 15:17:29 EmilienM: likely have similar issues with virtualenvs or docker support 15:17:35 we just have to figure out once and for all a good pattern 15:17:53 which might require refactoring, I just waited to raise the point 15:18:03 agreed. what’s there now is kinda a mess 15:18:46 mgagne: the only negative reaction i have to the $nova_title approach is how do users external to the module know what the title of the package will be. but that speaks to having a consistent pattern across all 15:18:50 mgagne: maybe we could define services managed by a package in an array? 15:19:12 why not tag the resource and users that want to depend on it can use a collector? 15:19:24 the collector will fail if the package isn't included in the manifest 15:19:35 I don't have any clear solution in my head, I'm open to all suggestions as long as it helps fixing the resource title issue 15:19:52 mgagne: do you have a LP # ? 15:19:57 EmilienM: no 15:20:02 clayton: so just use some tag, rather than changing the title? 15:20:26 it would be great to submit a but explaining the issue and with some examples. 15:20:34 s/but/bug/ 15:20:39 I don't think tag were meant for that purpose 15:20:59 clayton: this sounds tricky 15:21:04 we are suggesting not following conventions already in place 15:21:22 EmilienM: here’s the bug from yesterday that explains this problem for one specific case (nova): https://bugs.launchpad.net/puppet-nova/+bug/1465433 15:21:23 Launchpad bug 1465433 in puppet-nova "generic_service declares package incorrectly" [Undecided,In progress] - Assigned to Mike Dorman (mdorman-m) 15:21:39 it's not just about "getting the job done", it's about making it the most "puppet" way 15:21:48 which usually do not involve tag and such 15:22:22 not sure how puppet tags are non puppet-like. 15:22:38 it's not something I see in other modules 15:23:25 <_ody> generally tags frighten me because everything in a catalog is implicitly tagged with something. If you pick that somethting as your explicit tag, you get all the implicit ones too. 15:23:34 ^ 15:23:58 I think tags are not the right option here 15:24:24 I don't think we will come up with a solution in this meeting, just wanted to raise my concerns 15:24:34 i am not really up to speed with tags, so don’t have a strong opinion. but i like the idea of a collector 15:24:51 it seems like mdorman and mgagne have a great consensus on https://review.openstack.org/#/c/192009 15:24:53 mgagne / EmilienM how about i iterate some more on https://review.openstack.org/192009 and we can go from there, discuss more next week? 15:24:55 collector tends to fail silently if you make a typo too 15:25:01 mdorman: +1 15:25:01 true 15:25:36 #action mdorman to work more on https://review.openstack.org/192009, discuss more next meeting. 15:25:49 perfect 15:26:18 the agenda is done for today 15:26:25 #topic open discussion 15:26:36 you can rise your patches/bugs that need specific attention 15:26:55 I see reviews are on going for the keystone auto refactor. I'm happy about it 15:27:01 I made a list for patches we need to land before a kilo release 15:27:20 and I also see people are ok with puppet openstack usage questions to go on openstack-operators so I'm happy to hear it too 15:27:28 https://review.openstack.org/191326 and https://review.openstack.org/191328 still needed to fix ci after the project renames 15:27:59 <_ody> EmilienM: Where is this list? 15:28:14 _ody: a sec 15:29:01 crinkle: done 15:29:06 thx! 15:29:41 sbadia: ty 15:29:48 #link http://paste.openstack.org/show/295543/ 15:29:55 please review that ^ for kilo 15:30:31 also for Juno, I have some backports 15:30:42 crinkle: did you have all your backports merged/reviewed? 15:30:53 EmilienM: look like it 15:31:11 I'm still miising 2 15:31:12 https://review.openstack.org/#/c/190634/ 15:31:19 and https://review.openstack.org/#/c/190464/ 15:31:41 <_ody> crinkle: the beaker failure on trusty for https://review.openstack.org/#/c/191328/ looks almost legit. 15:32:16 mgagne: are your patches on Introduce public_url, internal_url and admin_url a requirement for a kilo release? 15:32:25 _ody: hrm, thanks 15:33:07 _ody: you'll need to rebase https://review.openstack.org/#/c/184646/ 15:33:27 _ody: oh emilien commented on it 15:33:35 does anyone wanted to raise another topic/patch/bug here? otherwise I can close the meeting in 1 minute 15:33:47 _ody: I've posted an inline question for 184646 15:34:18 EmilienM: seems ok for me :) 15:34:38 ok. Thanks everyone and have a great day/evening! 15:34:44 o/ 15:34:46 #endmeeting