17:00:28 #startmeeting ironic 17:00:29 Meeting started Mon Jun 19 17:00:28 2017 UTC and is due to finish in 60 minutes. The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot. 17:00:30 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 17:00:32 The meeting name has been set to 'ironic' 17:00:33 o/ 17:00:41 o/ 17:00:41 o/ 17:00:45 o/ 17:00:47 hi everyone! 17:00:47 o/ 17:00:49 o/ 17:00:52 \o 17:00:57 o/ 17:01:13 o/ 17:01:26 our agenda is as usual at: 17:01:29 #link https://wiki.openstack.org/wiki/Meetings/Ironic 17:01:35 o/ 17:02:22 o/ 17:02:22 o/ 17:02:34 o/ 17:02:46 #topic Announcements / Reminder 17:02:57 #info PTG information and registration available 17:03:03 #link https://www.openstack.org/ptg/ 17:03:35 this is hopefully self-explanatory :) 17:03:39 :) 17:03:54 #info ironic-lib 2.8.0 (Pike) released 17:03:59 o/ 17:04:10 anything else? 17:05:03 #topic Review subteam status reports (capped at ten minutes) 17:05:15 #link https://etherpad.openstack.org/p/IronicWhiteBoard starting with line 99 17:05:39 o/ 17:05:53 o/ 17:06:21 * jlvillal wonders if we should make gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial-nv a voting job 17:06:26 dtantsur: wrt your tool at http://ironic-divius.rhcloud.com, i use it when you aren't around and i want to update the bugs stats :-) 17:06:33 http://ci-watch.tintri.com/project?project=ironic&time=7+days 17:06:48 rloo: cool :) 17:07:13 jlvillal: this sounds like something to make voting. what additional coverage does it provide? 17:07:32 dtantsur: It is testing the rolling upgrade scenario 17:07:52 jlvillal: which we don't have yet, right? I mean, the upgrade 17:07:58 dtantsur: Having an old and new conductor at same time. 17:08:04 dtantsur: Right, it is the only job that tests that. 17:08:57 I guess we can make it voting and hope that we do land rolling upgrades :) 17:09:14 jlvillal: has that test been passing (as expected), etc? 17:09:30 rloo: Based on http://ci-watch.tintri.com/project?project=ironic&time=7+days, I would say yes 17:10:22 jlvillal: is there any way to see beyond 7 days? 17:10:39 rloo: Change the "time=" setting? 17:10:52 jlvillal: i think it has been behaving properly, just wonder what the status show 17:11:02 But when I tried 14 days it gave server error 17:11:03 jlvillal: no, that only seems to work for max 7 days. 'that' == CI Watch 17:11:19 I think that is all the history we have 17:11:41 well, i'm good with having it vote, we can always unvote if there are issues, and better to notice now than at end of cycle? 17:12:32 +1 17:12:37 ++ 17:12:44 +1 17:13:22 Okay. I will propose a patch 17:13:54 thx jlvillal 17:14:04 dtantsur: should we put a #agree or whatever it is? 17:14:06 #action jlvillal to submit patch to make gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial-nv a voting job 17:14:34 * TheJulia wonders if that gets recorded if it is done by someone who is not the chair 17:14:40 * dtantsur wonders too 17:14:44 I think it is "#agreed" https://wiki.debian.org/MeetBot 17:14:52 action should also work 17:15:00 #action jlvillal to submit patch to make gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial-nv a voting job 17:15:05 * dtantsur duplicated just to be sure 17:15:09 action is for everyone. #agreed is only for the chair 17:15:09 \o/ 17:15:23 According to https://wiki.debian.org/MeetBot 17:15:34 nice 17:15:38 #undo 17:15:39 Removing item from minutes: #action jlvillal to submit patch to make gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial-nv a voting job 17:16:09 anything else about the statuses? 17:16:11 * jlvillal is still proud that #undo now displays the text instead of Removing 17:16:40 Tiny tiny little victories :) 17:16:41 there's a question at L240, about whether that RFE needs approval. 17:17:14 didn't we already discuss that, maybe last week, and decided it wasn't needed? 17:17:36 sambetts: ^^ wrt https://bugs.launchpad.net/ironic/+bug/1658964 17:17:37 Launchpad bug 1658964 in Ironic "[RFE] Implement neutron routed networks support in Ironic" [Wishlist,Confirmed] - Assigned to Sam Betts (sambetts) 17:18:25 jlvillal: wrt tempest plugin, L257. anything new to report? 17:18:37 rloo: Sadly no :( 17:18:48 jlvillal: but you're back from vaca? :-) 17:19:07 I got distracted with BFV, rolling upgrade, and getting caught with emails and stuff :( 17:19:10 rloo: yeah, I think we decided that we didn't need the RFE anymore, but I'll keep it arounds in case I ( although I don't think we will) need to make changes to Ironic 17:19:24 I need to say that the plugin split is much less a priority than e.g. BFV 17:19:35 sambetts: ok, thx for clarifying 17:19:59 dtantsur: right, that's why it is 'Optional' 17:20:18 * dtantsur wants to get rid of "optional priorities" in Queens 17:21:19 dtantsur: +1 17:21:26 dtantsur: well, we knew we were taking on too much, although we didn't anticipate what changed this cycle... 17:21:46 indeed 17:22:05 ready to move on? we've spent 15 mins on priorities already 17:22:19 * rloo ready 17:22:37 #topic Deciding on priorities for the coming week 17:22:45 so, we've done some great job again :) 17:22:56 Congrats on OSC!!!! :) 17:23:06 the only thing left is the rolling upgrades patch (the next BFV patch was added during the week) 17:23:29 jlvillal: ha ha. now we just have to remember to keep feature parity! 17:23:46 rloo: And start deprecating the 'ironic' client 17:23:49 rloo: I suggest we don't accept changing the "ironic" command first 17:23:57 like some people do, first "ironic", then "osc" 17:24:06 "osc", then "ironic" fine, until we declare it deprecated 17:24:08 wdyt? 17:24:13 jlvillal, dtantsur: I added a note to the subteam report, we aren't deprecating until queens. 17:24:21 dtantsur: Sounds reasonable to me 17:24:22 right, I remember 17:24:24 dtantsur: that is a good idea 17:24:39 dtantsur: I like it, as long as we don't apply that requirement to things already in the queue for the client 17:24:47 dtantsur: and those items already have parity across patchsets 17:25:01 i think we want an #agreed here :-) 17:25:03 #agreed Do not allow new features in the "ironic" command that do not have corresponding features in the "openstack baremetal" command first 17:25:07 rloo: I was writing it :) 17:25:15 great minds and all... 17:25:34 and yes to TheJulia, grandfathering and all... 17:26:12 I think the volume info stuff is all ironic then osc in the order, but they have parity if I'm remembering correctly 17:26:35 are we ready to take https://review.openstack.org/#/c/470406 to the list? this is the essence of physical network awareness, I guess 17:26:54 and I wonder if we should take 2 patches from BFV, as things have been moving faster recently 17:26:58 wdyt on both ^^^ 17:27:03 dtantsur: ++ I added physnet 17:27:40 dtantsur: is the ipxe template a simple patch? 17:27:56 dtantsur: if so and the next patch is ready to be reviewed, sure. 17:28:03 dtantsur: I would <3 the next three patches for BFV to be on the list, mainly so we have api patch visibility and get towards that soonish since it blocks other changes like in the client library 17:28:27 or maybe the third as a stretch 17:29:01 yeah, I suspect 17:29:02 TheJulia: it seems to me (I coudl be wrong) that someone that is reviewing a feature tends to review the next patch in the series anyway. (At least I do) 17:29:09 ++ 17:29:31 Maybe 3 could be the topic URL? 17:29:50 maybe I propose the classic driver deprecation spec? https://review.openstack.org/#/c/464046/ 17:30:05 rloo: for some of us yes, but not everyone 17:30:05 I mean 1.3 17:30:26 dtantsur: I say sure 17:30:29 jlvillal: I'd like to be specific on what we consider in scope 17:30:31 TheJulia: true. I am hoping 'most' is still true and hint hint folks :-) 17:30:59 rloo: heh :) 17:31:00 dtantsur: okay 17:31:06 ok, wdyt about the list. 4 code patches and one spec. 17:31:23 +1 17:31:40 +1 17:31:50 +1 17:31:52 wait, shouldn't we add a stretch goal (just kidding) 17:32:14 hah 17:32:19 any more suggestions here? 17:33:01 nothing from me 17:33:11 #topic PTG in Denver 17:33:21 a few formal questions I have to ask: 17:33:27 Do we want to attend? (just say yes) 17:33:45 maybe? :-) 17:33:49 :) 17:34:04 I say yes 17:34:07 who is planning or would like to attend? 17:34:13 o/ 17:34:28 o/ 17:34:34 o/ 17:34:47 ok, we have three so far. it is a go! 17:35:24 o/ would like to attend 17:35:31 dtantsur: i think it'll only be useful if we have enough people. maybe we ought to plan for it, and find out closer to the date, who will be most likely attending 17:35:35 o/ would like to attend 17:35:57 o/ I plan to attend 17:36:00 * Nisha_ can attend virtually only 17:36:17 rloo: I think we have to do it that way as people will need to obtain employer travel approval 17:36:18 rloo: we have to plan in advance, unfortunately 17:36:19 * milan would go if approved 17:36:34 ok, I got some information at least 17:36:44 Does it allow remote particpation? 17:36:47 exactly. no one does NOT want to have an ironic thing there, which is good :-) 17:37:03 lol 17:37:25 wanyen: I'm not sure yet. we can try our best, but we cannot guarantee that available tech serves us well (e.g. internet connection) 17:37:28 i thought there was a discussion/talk about a way for people to attend virtually but i don't recall the details or anything 17:37:31 o/ i´d like too 17:37:57 I remember there was discussion on bringing greater visibility on "what is going on" 17:38:31 Anyway, we should move on to the next question 17:38:36 dtantsur, tx. remote participation will reallyhelp. 17:38:40 next question: do we need Friday? 17:38:56 we tend to be only active in the morning, but it's still a productive time 17:39:04 so I don't have a particularly strong opinion here 17:39:21 I say plan on the morning again. For some technically productive, others socially productive to gain mutual context and build better communication 17:39:21 i think we don't need it 17:39:26 oh, I do plan to attend 17:39:31 denver ptg 17:40:02 Friday +1: inspector day ;) 17:40:06 just to clarify: there is no pressure on us to NOT use Friday 17:40:11 If we don't do Friday morning, we should at least set aside half a day or a couple hours to break into groups to allow sub team discussion 17:40:12 we just need to decide 17:40:23 if unsure, say yes 17:40:27 yes! 17:40:38 * TheJulia copies the yes 17:40:41 * jlvillal is unsure 17:40:47 no 17:41:02 * dtantsur feels like some voting 17:41:04 * jlvillal hopes to be going. But travel budget has been very tight at work :( 17:41:26 dtantsur: do you/we need to decide now? 17:41:27 * TheJulia is sure, but largely because she also knows and understands why the foundation offers friday as an option to teams. 17:41:51 rloo: not necessary. if everyone said "no", I would communicate it to the foundation 17:41:57 otherwise we get Friday by default 17:42:16 which is fine 17:42:17 dtantsur: ok, some say yes, so as long as 2? people are there and they are productive, then yes is good. 17:42:29 ok, thanks 17:42:46 I guess I'm ready to open the floor 17:42:49 #topic Open discussion 17:43:14 dtantsur: Do we need a #agree 17:44:09 * TheJulia has nothing for open discussion 17:44:28 +1 17:44:31 rpioso: not necessary here. it's mostly for me to understand 17:44:33 Well, actually 17:44:44 A giant sign saying "please review" :) 17:45:04 TheJulia: anything in particular (outside of this week's priorities?) 17:45:19 TheJulia: or a general 'we would love more reviews'? 17:45:25 rloo: in general 17:45:35 +++++ 17:46:18 #info Please be active on reviews, we really need them! 17:46:27 <3 17:46:53 PTG hotel is on Quebec Street. I guess I *have* to go then :-) 17:46:58 hah 17:47:08 anything else before we move to our nice warm channel? 17:47:18 (Quebec is a province in Canada) 17:47:57 I'm saving you 12 minutes of time now: 17:48:02 :) 17:48:04 crickets 17:48:04 #endmeeting