16:00:00 #startmeeting openstack_ansible_meeting 16:00:01 Meeting started Thu Dec 8 16:00:00 2016 UTC and is due to finish in 60 minutes. The chair is mhayden. Information about MeetBot at http://wiki.debian.org/MeetBot. 16:00:02 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 16:00:04 The meeting name has been set to 'openstack_ansible_meeting' 16:00:09 #topic Roll Call 16:00:13 o/ 16:00:13 * mhayden toots 16:00:14 o/ 16:00:25 o/ 16:01:12 * mhayden starts banging pots and pans together 16:01:30 JOIN THE MEETING OR I'LL REBASE 20 MORE PATCHES 16:01:33 :P 16:01:44 o/ 16:02:05 mhayden: i've cleaned my hands of that :P i believe i have reviewed all of them now 16:02:31 o/ 16:02:46 o/ 16:02:56 * mhayden will give it til 16:05 16:02:57 o/ 16:03:21 perhaps asettle can bestow a GIF upon us to warm things up 16:03:26 Oh 16:03:27 hold up 16:03:53 o/ 16:04:16 https://media.giphy.com/media/m80Q4HTDNPFHq/giphy.gif 16:04:20 DANG IT GIPHY 16:04:26 nicely done 16:04:36 okay, let's get rolling 16:04:44 #topic Review action items 16:04:57 first up is -- andymccr to promote the upgrade jobs 16:05:01 done! 16:05:05 http://www.pbh2.com/wordpress/wp-content/uploads/2013/06/funny-gif-nobody-got-time-for-that.gif 16:05:05 woot 16:05:08 HA PERFECT 16:05:11 Here all week 16:05:17 asettle: the time for gifs has passed -- my business socks are on 16:05:23 I'm sorry :( 16:05:29 asettle: i'm kidding :P 16:05:37 I am not 16:05:41 okay, next is everyone to review https://review.openstack.org/#/c/400579/ and https://review.openstack.org/#/c/404125/ 16:05:43 :P 16:05:44 :p 16:05:49 done, since one merged 16:05:53 the battle of mergery has completed 16:05:58 it appears that the battle has completed 16:06:13 two patches enter, one patch leaves with workflow ;) 16:06:31 next is reviewing the inventory patches -> https://review.openstack.org/#/q/project:openstack/openstack-ansible+topic:%5Einventory.*+status:open 16:06:44 looks like there's one in the queue with a -1 from stevelle 16:07:16 palendae replied, so we might need more comments from stevelle and neillc 16:07:25 and git-harry 16:07:30 Yep 16:07:31 yup 16:07:42 A lot of them merged last week 16:07:51 last one is spotz to add a reno for removing trusty 16:07:51 another great success then 16:08:02 spotz: i thought i saw that one go in 16:08:18 mhayden: It merged, we were talking about a newton patch as well 16:08:49 cool beans 16:09:10 rolling to the next agenda item... https://wiki.openstack.org/wiki/Meetings/openstack-ansible#Agenda_for_next_meeting 16:09:21 #topic Release Planning & Decisions - andymccr 16:09:26 ok! 16:09:35 so next week is milestone 2 for Ocata, and another release for newton/mitaka 16:09:51 Liberty is being EOL'd (there is an email from Tony B regarding that) 16:09:57 so that'll happen soon 16:10:04 but as always, if there are urgent prs let me know 16:10:05 lots of changes 16:10:15 I'm going to try get the milestone 2 in mid week as opposed to the end of week rush :) 16:10:24 FWIW, 14.0.3 works great from upgrading from 13.3.9 ;) 16:10:39 but so far, aside from Ceilometer we have had SHA bumps go in successfully for all other projects in the integrated build 16:10:58 mhayden: excellent. on trusty i assume? 16:11:03 right 16:11:10 the trusty -> xenial upgrade was... painful 16:11:18 and i have a control plane node that needs to be reinstalled :| 16:11:19 yeah we found a bug (thanks to upgrade tests) in xenial for nova 16:11:20 which is now fixed 16:11:34 will be in the release next week 16:11:48 anything else on releases? anyone have any patches that need merging prior to next release? 16:12:11 https://review.openstack.org/#/c/408677/ will need to go in the next newton tag 16:12:35 ok 16:12:49 #action review and backport https://review.openstack.org/#/c/408677/ (and review the backport!) 16:13:11 if some cores could star that to review it later so we can get it in for next week that'd be appreciated 16:14:07 I wouldn't mind landing this one as well https://review.openstack.org/#/c/407607/ 16:14:22 It already has one +2, so if another core could take a look I'd appreciate it! 16:14:48 #action review https://review.openstack.org/#/c/407607/ before release 16:14:53 will take a look at both those 16:15:30 okay, anything else? 16:16:21 https://launchpad.net/bugs/1643680 and https://review.openstack.org/#/c/407655/ will need some eyes as potential issues for upgrading 16:16:21 Launchpad bug 1643680 in openstack-ansible "Shift to using dashes instead of underscores for container names" [Wishlist,In progress] - Assigned to Nolan Brubaker (nolan-brubaker) 16:16:49 The review I'm still working on, but the bug overall will require renaming all the groups in the environment...I think we could use some more thoughts on that 16:18:09 palendae: im not sure that one will make the release next week 16:18:15 andymccr: Nor am I 16:18:22 In fact I'm sure it won't 16:18:27 palendae I've posed a question about why the group rename is required in the bug. 16:18:30 I'm fine with deferring, but it's also a huge change 16:18:35 hahah ok good just ensuring its not deemed critical for next release 16:19:03 odyssey4me: Ah, ok 16:20:14 good on this topic for now? 16:20:39 gonna go with yes in that case :) 16:20:42 WOOT 16:20:49 #topic Blueprint work 16:21:01 on the security role front, there are 6 controls left to propose 16:21:13 57 are pending review -> https://review.openstack.org/#/q/status:open+project:openstack/openstack-ansible-security+branch:master+topic:bp/security-rhel7-stig 16:21:45 once that's done, i'll go backt through and look for missing tags and other random broken things 16:21:47 and i'll need testers 16:22:04 #link https://review.openstack.org/#/q/status:open+project:openstack/openstack-ansible-security+branch:master+topic:bp/security-rhel7-stig 16:22:23 any review help there would be awesome... merge conflicts might pop up but i'll try to get those resolved quickly 16:23:11 thanks mhayden. will start looking thru them 16:23:20 logan-: thanks mucho! 16:23:30 and thanks to everyone who has been working through those 16:23:40 i'll likely start getting onto octavia once this settles down 16:23:58 asettle: anything to talk about here with regard to the ops guide? 16:24:29 mhayden: she's not about atm - but i know they're going ahead after tomorrow - so any feedback on the ops guide "outline" needs to be in by end of play tomorrow! 16:24:44 thanks to those who have, i know evrardjp and jmccrory put some feedback in. 16:25:23 sounds good 16:25:26 anything else on blueprints? 16:25:38 so on the ugprade testing 16:25:40 there are 2 reviews left 16:25:48 andymccr: got a link? 16:25:55 #link https://review.openstack.org/#/q/topic:bp/upgrade-testing 16:26:05 they are working now, so any reviews would be appreciated 16:26:50 oh and on trusty removal 16:27:17 i know odyssey4me added a PR for the main repo, I've added a "POC" for swift: https://review.openstack.org/#/q/topic:bp/trusty-removal 16:28:24 nice 16:28:28 okay... 16:28:31 #topic Open floor 16:28:38 anything else anyone wants to bring up? 16:29:12 IT'd be nice to have some review on these backports: https://review.openstack.org/408540 & https://review.openstack.org/406803 16:30:03 there is one topic of discussion around the OVS.dvr work and the groups assignments for that 16:30:20 #link https://bugs.launchpad.net/openstack-ansible/+bug/1645979 16:30:20 Launchpad bug 1645979 in openstack-ansible "neutron_l3_agent and neutron_l3_metadata groups include physical host" [Undecided,New] 16:30:42 ahh yeah - automagically isn't here, so we might have to table that until we can get Travis involved 16:31:05 k 16:31:33 we cannot postpone for too long, so let's give a deadline for this discussion 16:31:59 short version - we'd like to remove the grouping by default, and anybody using OVS.dvr should add it manually (like we do for cinder in a container and a few others) - which we add docs for 16:32:14 alextricity25: would like to have input on this if I'm not mistaken :p 16:32:40 +1 16:33:01 Thanks evrardjp 16:33:08 hehe yw 16:33:31 just wanted to say hello. I've been mia for the last few weeks, busy with internal stuff. But I'm still here 16:33:41 i know a lot of the discussion has happened in the bug, so perhaps we can continue this there, and I'll ping automagically to review once more 16:33:52 :) I've been somewhat tied up mysefl 16:33:53 *myself 16:33:54 michaelgugino: nice to see you back 16:34:04 michaelgugino: good to hear, there is nova-lxd that was missing some love :p 16:34:15 joke aside, glad to see you here michaelgugino :D 16:34:27 :) What's happening with nova-lxd? 16:34:42 We're going to start a poc internally at some point in the near future, or so I'm told 16:35:06 andymccr: if we agree with the approach, we can go for a patch and fix it, and discuss in the review with Travis 16:35:51 evrardjp: im ok with that, im just concerned there are others using that - and we need to make sure we're not creating an upgrade nightmare 16:36:54 evrardjp: i agree it would be good to get a review in asap so we can try to get something in the ocata milestone and into a newton tag soon after that 16:37:04 it'd have to wait for ocata m3 16:37:21 the turn around is too short, but thats ok since we're just doing milestones at the moment (in my opinion) 16:38:07 ok so is there a volunteer to propose the patch for that? 16:38:15 i mean its a simple one really - depending on the approach 16:39:25 i'll work on it. 16:39:51 thanks logan-! 16:39:52 the group change is easy, will have to work up some docs in the neutron role as well 16:39:58 oh thanks for devoting yourself on this one logan- :D 16:40:02 I'll review it then 16:41:09 logan-: we're adding some comments to the bug 16:41:15 i think i have a new idea that may work for that 16:41:15 can always just use an add_host task to add the host to the group conditionally 16:41:52 ie if you're using dvr, then add the compute host into the grou 16:41:53 *group 16:43:05 ok i think we're done on that 16:43:13 that might work well. i'll look at that odyssey4me 16:43:14 agreed andymccr 16:43:38 add_host seem the best, you can do it based on facts and all funsies 16:43:47 seems cleanest and simplest if possible 16:43:58 the hardest in that patch is the release note 16:45:45 ok i think we're done here - mhayden do the honours? 16:45:53 WOOT 16:45:56 thanks everyone 16:45:57 #endmeeting