| *** esberglu has quit IRC | 01:16 | |
| *** prashkre_ has joined #openstack-powervm | 03:46 | |
| *** prashkre has quit IRC | 03:47 | |
| *** prashkre__ has joined #openstack-powervm | 04:13 | |
| *** prashkre_ has quit IRC | 04:16 | |
| *** prashkre__ has quit IRC | 04:51 | |
| *** prashkre has joined #openstack-powervm | 06:11 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 07:31 | |
| *** edmondsw has joined #openstack-powervm | 08:58 | |
| *** edmondsw has quit IRC | 09:02 | |
| openstackgerrit | prashkre proposed openstack/nova-powervm master: Miscellaneous iSCSI improvements https://review.openstack.org/561022 | 10:21 |
|---|---|---|
| *** AlexeyAbashkin has quit IRC | 11:07 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 11:28 | |
| openstackgerrit | prashkre proposed openstack/nova-powervm master: Handle exceptions properly with iscsi discovery https://review.openstack.org/561914 | 11:40 |
| openstackgerrit | prashkre proposed openstack/nova-powervm master: Handle exceptions properly with iscsi discovery https://review.openstack.org/561914 | 11:47 |
| *** edmondsw has joined #openstack-powervm | 12:37 | |
| *** apearson has joined #openstack-powervm | 12:40 | |
| prashkre | edmondsw: efried: Good morning. | 13:01 |
| efried | Hello prashkre | 13:01 |
| efried | I was looking at your patches (got waylaid, but I'm on it) | 13:02 |
| edmondsw | prashkre hi... will look | 13:02 |
| prashkre | FYI.. we have now 3 patches https://review.openstack.org/#/c/561022/, https://review.openstack.org/#/c/557800/, https://review.openstack.org/#/c/561914/ | 13:03 |
| *** tjakobs has joined #openstack-powervm | 13:30 | |
| *** esberglu has joined #openstack-powervm | 13:36 | |
| openstackgerrit | Merged openstack/nova-powervm master: Handle exceptions properly with iscsi discovery https://review.openstack.org/561914 | 13:42 |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: DNM: CI Check2 https://review.openstack.org/328317 | 13:45 |
| edmondsw | #startmeeting PowerVM Driver Meeting | 14:02 |
| openstack | Meeting started Tue Apr 17 14:02:24 2018 UTC and is due to finish in 60 minutes. The chair is edmondsw. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:02 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:02 |
| *** openstack changes topic to " (Meeting topic: PowerVM Driver Meeting)" | 14:02 | |
| openstack | The meeting name has been set to 'powervm_driver_meeting' | 14:02 |
| edmondsw | #link https://etherpad.openstack.org/p/powervm_driver_meeting_agenda | 14:02 |
| edmondsw | efried esberglu ut? | 14:03 |
| esberglu | yep | 14:03 |
| efried | ō/ | 14:03 |
| edmondsw | #topic In-Tree Driver | 14:03 |
| *** openstack changes topic to "In-Tree Driver (Meeting topic: PowerVM Driver Meeting)" | 14:03 | |
| edmondsw | #link https://etherpad.openstack.org/p/powervm-in-tree-todos | 14:03 |
| edmondsw | I understand we're in a runway now | 14:04 |
| edmondsw | excited to see how that goes | 14:04 |
| efried | https://etherpad.openstack.org/p/nova-runways-rocky | 14:04 |
| esberglu | Everything through localdisk is +1 from edmondsw, +2 from efried, and in the runway | 14:04 |
| edmondsw | any comments yet? | 14:04 |
| efried | nope | 14:05 |
| efried | (just checked) | 14:05 |
| esberglu | Nope. Target date for that is 4/30 so I'm expecting them to start coming in soon | 14:05 |
| edmondsw | anything else to note here then? | 14:05 |
| efried | We've been doing runways for all of two weeks, but tbh I haven't really noticed it prompting a significant increase in reviewer focus. | 14:05 |
| efried | I'm thinking of suggesting a "champion" idea | 14:05 |
| edmondsw | efried what would that be? | 14:06 |
| esberglu | I've been prioritizing multinode CI over finishing the cold mig/resize patch, nothing to report there | 14:06 |
| edmondsw | esberglu that makes sense... CI is a prereq for that | 14:06 |
| efried | where we assign one reviewer (core or not, I guess, but not the owner of the series) to go pester reviewers to look at the changes, and pester the owner to follow up as needed. | 14:06 |
| efried | kinda like we champion invention disclosures on IDTs. | 14:07 |
| edmondsw | who pesters folks to be a champion? ;) | 14:07 |
| efried | Touché | 14:07 |
| efried | Part of the process; add it to runway, assign champion at that time. | 14:07 |
| efried | just an idea I'm toying with. | 14:07 |
| edmondsw | yep, I think that can work | 14:08 |
| edmondsw | can become a nova meeting agenda item | 14:08 |
| edmondsw | #topic Out-of-Tree Driver | 14:08 |
| *** openstack changes topic to "Out-of-Tree Driver (Meeting topic: PowerVM Driver Meeting)" | 14:08 | |
| edmondsw | #link https://etherpad.openstack.org/p/powervm-oot-todos | 14:09 |
| edmondsw | we've made some progress on the iscsi changes | 14:09 |
| edmondsw | I just merged one, which put the others into merge conflict since they're not in a series | 14:09 |
| edmondsw | I told prashkre to rebase the other that I +2/+A'd and when he does that I'll fast approve it | 14:10 |
| edmondsw | I did give him a comment to go address on the 3rd iscsi commit | 14:10 |
| edmondsw | he's off and running on that | 14:10 |
| edmondsw | once those all merge, I intend to tag 6.0.1 | 14:11 |
| openstackgerrit | prashkre proposed openstack/nova-powervm master: Return iSCSI Initiator for VIOSes https://review.openstack.org/557800 | 14:11 |
| *** AndyWojo has quit IRC | 14:11 | |
| edmondsw | I also just setup a meeting with the PowerVC storage guys to talk about community contribution for multiple initiator support, among other things | 14:11 |
| edmondsw | anything else to discuss around iscsi? | 14:12 |
| edmondsw | the volume refactor will need to be rebased and updated once these changes merge | 14:12 |
| edmondsw | and I still need to dig into the concerns that the PowerVC team raised on that | 14:13 |
| edmondsw | so probably more updates there | 14:13 |
| edmondsw | tjakobs FYI ^ | 14:13 |
| edmondsw | anything else for OOT? | 14:13 |
| edmondsw | some other things need to happen, like adding MSP support, but I haven't gotten to anything there | 14:14 |
| efried | We could have done the iscsi changes in a series and not had to rebase. | 14:14 |
| edmondsw | efried I suggested that a couple times, but nobody did it, so... | 14:14 |
| efried | Not sure how much more prashkre is going to be doing for us, but might be worth a little tutorial on that. | 14:14 |
| edmondsw | efried I'd love to see something written up on that for some wiki somewhere | 14:15 |
| edmondsw | and then just link it with things like that | 14:15 |
| efried | I'm sure there are documents galore. | 14:15 |
| edmondsw | I'm sure... question is where, and how good are they | 14:15 |
| efried | There's a whole man page for git restack | 14:15 |
| efried | which is what I use *daily* | 14:15 |
| edmondsw | if we find a good one, great, just need to keep in our back pocket and start linking | 14:16 |
| edmondsw | maybe just link that then | 14:16 |
| edmondsw | I'm familiar with rebase, but not restack | 14:16 |
| edmondsw | which is probably why I'm confused about managing series myself | 14:17 |
| efried | esberglu said he tried restack and couldn't get the hang of it. | 14:18 |
| efried | but I love it. | 14:18 |
| edmondsw | you play with it enough to figure out the quirks, probably | 14:18 |
| esberglu | efried: I think it's because I didn't check out the latest patch. I would check out the latest in the series, git restack | 14:18 |
| esberglu | Then change anything I want to edit to edit | 14:18 |
| esberglu | And go from there? | 14:18 |
| efried | yes, using git rebase --continue whenever you're ready to move to the next one up. | 14:19 |
| esberglu | Okay I see where I went wrong now, thanks | 14:19 |
| efried | But yeah, it's key to check out the top patch in the series. | 14:19 |
| *** AndyWojo has joined #openstack-powervm | 14:19 | |
| edmondsw | top = the first one that should merge? | 14:20 |
| edmondsw | or is that bottom? | 14:20 |
| esberglu | Top = last one that should merge | 14:20 |
| efried | ^ | 14:20 |
| edmondsw | ok, so if I want to change something 3 patches down... | 14:21 |
| edmondsw | I check out the top, and then what? | 14:21 |
| efried | git restack | 14:22 |
| efried | it puts you into an editor just like git rebase -i | 14:22 |
| efried | You change 'pick' to 'edit' for any patches in the series you want to change. | 14:22 |
| efried | Save/quit. You'll be shoved down the chain into the first 'edit' patch. | 14:22 |
| edmondsw | cool | 14:22 |
| efried | Make your changes, and run git rebase --continue. It'll commit your changes and put you into the commit message editor. | 14:23 |
| efried | Save/quit that guy and you'll be shunted to the next 'edit' patch... | 14:23 |
| efried | ...unless there's a merge conflict with one in between | 14:23 |
| efried | in which case regular merge conflict process | 14:23 |
| edmondsw | so you don't need to 'git commit' at all, it does that for you with rebase --continue? | 14:23 |
| efried | when done, git rebase --continue. | 14:23 |
| efried | yes. You can do the commit yourself if you like | 14:23 |
| edmondsw | just need to git add after you change something, and then rebase --continue? | 14:23 |
| efried | either way. | 14:23 |
| edmondsw | cool, got it | 14:24 |
| edmondsw | moving on | 14:24 |
| efried | note that it doesn't come in the box. | 14:24 |
| efried | you have to pip install git-restack | 14:24 |
| efried | it's something the os infra folks wrote, I think. | 14:24 |
| edmondsw | oh, good to now | 14:24 |
| edmondsw | #topic Device Passthrough | 14:24 |
| *** openstack changes topic to "Device Passthrough (Meeting topic: PowerVM Driver Meeting)" | 14:24 | |
| edmondsw | efried what's the latest? | 14:24 |
| efried | Jay and I had a knock-down drag-out; now we're asking for an impartial referee to decide who wins. | 14:24 |
| edmondsw | you couldn't take him? | 14:25 |
| edmondsw | ;) | 14:25 |
| efried | So many possible responses to that. | 14:25 |
| edmondsw | I was trying to set you up easy... | 14:25 |
| efried | At this point I would be content either way tbh. We've wrangled both of our proposals into compromise territory. | 14:25 |
| efried | I still have a preference for mine, but I won't be seriously mad if it goes the other way. | 14:26 |
| edmondsw | k | 14:26 |
| efried | I made progress on the granular patch. | 14:26 |
| efried | I've got the bugs worked out to where it has parity with the existing functionality | 14:27 |
| efried | That is, if you're not using granular syntax, all the old scenarios still work when they're funneled through the new algorithm. | 14:27 |
| efried | I'll be in microversion rebase hell until it merges. | 14:27 |
| efried | Next step will be to write tests; and once the above argument is settled, to fold in that functionality. | 14:27 |
| efried | Because it's looking likely that we're going to want to do it in the same microversion. | 14:28 |
| efried | Jay admonished me for not waiting until his NRP stuff was done | 14:28 |
| efried | and then started working on it again | 14:28 |
| efried | Which was kind of the goal. | 14:28 |
| edmondsw | :) | 14:28 |
| efried | so goodness there. | 14:28 |
| efried | So progress is being made overall. Otherwise, nothing new to report specifically on device passthrough as it pertains to PowerVM. | 14:28 |
| efried | oh | 14:29 |
| efried | except I don't remember if I mentioned this | 14:29 |
| efried | I wrote a couple of patches that use upt in libvirt to solve the age-old problem of sharing providers being double-reported. | 14:29 |
| efried | I wouldn't have done it, except they took like 15 minutes and basically validated everything I wrote last cycle. | 14:30 |
| efried | bhagyshris is going to take over the patches to fix up the tests and get everything ready to merge. | 14:30 |
| edmondsw | cool | 14:30 |
| efried | https://review.openstack.org/560444 | 14:30 |
| efried | https://review.openstack.org/560459 | 14:30 |
| edmondsw | I'm sure the KVM folks thank you | 14:30 |
| efried | we'll see :) | 14:30 |
| efried | I think that's all I have for now. | 14:31 |
| edmondsw | I'm more than happy to see things fixed for libvirt, since Power cares about that as well as PowerVM | 14:31 |
| edmondsw | one thing I was going to mention | 14:32 |
| edmondsw | I think we've said before that we need to consider infiniband and other types of passthrough as well as GPU | 14:32 |
| edmondsw | RoCE is another | 14:33 |
| edmondsw | and more than just passthrough, virtualization like (or using, I hope) SR-IOV | 14:33 |
| efried | We're getting set up for allll of that. | 14:34 |
| edmondsw | except that I want to do it the right way, not the way we currently do SR-IOV | 14:34 |
| edmondsw | so just throwing that out there... stay tuned | 14:34 |
| edmondsw | yep yep | 14:34 |
| edmondsw | #topic PowerVM CI | 14:34 |
| *** openstack changes topic to "PowerVM CI (Meeting topic: PowerVM Driver Meeting)" | 14:34 | |
| edmondsw | #link https://etherpad.openstack.org/p/powervm_ci_todos | 14:34 |
| edmondsw | esberglu ? | 14:34 |
| esberglu | I've got a change out for the powervm-ci side of multinode. Just adds local.conf files and updates prep_devstack.sh to work with AIO, control, and compute cases | 14:35 |
| esberglu | 6470 | 14:35 |
| esberglu | It doesn't affect any existing jobs so I'm ready to merge if you guys are okay with it, will make neo-os-ci dev easier to have that in | 14:35 |
| edmondsw | esberglu ack | 14:36 |
| efried | We gonna get Mujahid in here moving forward? | 14:36 |
| *** apearson has quit IRC | 14:36 | |
| esberglu | On neo-os-ci I've got to the point where I can have the control node stack, then kick off another job to have the compute stack | 14:36 |
| edmondsw | efried yeah, I'll ask him to join, though I expect we'll have more CI discussions in slack going forward | 14:37 |
| efried | ight | 14:37 |
| esberglu | Still not sure what I have to do to force them to be on the same host, but staging CI only has 1 compute node right now | 14:37 |
| esberglu | I want to bring neo4 back into staging, but I still have that set up for IT work, and might need it again there | 14:38 |
| esberglu | I may just steal a node from production for a little while | 14:38 |
| edmondsw | esberglu have you asked the Power KVM CI guys if they've faced this problem? | 14:38 |
| esberglu | No | 14:38 |
| edmondsw | do they do multinode in their CI? | 14:38 |
| esberglu | I don't think so, I can check | 14:38 |
| edmondsw | k | 14:39 |
| *** apearson has joined #openstack-powervm | 14:39 | |
| esberglu | The big blocker for multinode is the jenkins dropping connections on the staging environment | 14:39 |
| esberglu | Which I'm completely in the dark on, I spent a TON of time trying to figure this out when I was working the queens upgrade | 14:40 |
| edmondsw | have you asked for help from infra? | 14:40 |
| edmondsw | it sounds like a possible jenkins issue | 14:40 |
| esberglu | Yeah I guess that's the next step | 14:40 |
| edmondsw | I know they're not hitting it, but they have experience tracking down these problems, right? | 14:40 |
| esberglu | edmondsw: Problem with that error message is that it could be a billion different things | 14:41 |
| edmondsw | and it may very well be a jenkins bug they just haven't hit because of environment differences or something | 14:41 |
| edmondsw | esbergly yep exactly | 14:41 |
| esberglu | But I'll see if they can shed some light | 14:41 |
| esberglu | But that is 100% blocking multinode CI right now and is my top priority | 14:41 |
| esberglu | Nothing else from me | 14:42 |
| edmondsw | #topic Open Discussion | 14:42 |
| *** openstack changes topic to "Open Discussion (Meeting topic: PowerVM Driver Meeting)" | 14:43 | |
| edmondsw | anything? | 14:43 |
| esberglu | efried: You interested in being included in the PowerVM CI education sessions at all? | 14:43 |
| efried | um | 14:43 |
| edmondsw | lol | 14:43 |
| efried | Does it increase the probability that I will be called upon to work on the CI? | 14:43 |
| efried | cause then no | 14:43 |
| esberglu | I'll leave you be :) | 14:44 |
| edmondsw | I don't see that happening... but you might still want to be able to plead ignorance anyway :) | 14:44 |
| edmondsw | better use of your time and all that | 14:44 |
| edmondsw | alright, if nothing else... | 14:45 |
| edmondsw | #endmeeting | 14:45 |
| *** openstack changes topic to "This channel is for PowerVM-related development and discussion. For general OpenStack support, please use #openstack." | 14:45 | |
| openstack | Meeting ended Tue Apr 17 14:45:05 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:45 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-04-17-14.02.html | 14:45 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-04-17-14.02.txt | 14:45 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-04-17-14.02.log.html | 14:45 |
| edmondsw | esberglu how about you go ahead and cancel your meeting invite for this, and I'll send a replacement | 14:46 |
| edmondsw | (including mujahid) | 14:46 |
| esberglu | edmondsw: Done | 14:46 |
| edmondsw | tx | 14:47 |
| *** efried has quit IRC | 15:00 | |
| *** efried has joined #openstack-powervm | 15:01 | |
| *** apearson has quit IRC | 15:02 | |
| *** apearson has joined #openstack-powervm | 15:05 | |
| edmondsw | efried hold off on looking at 6470 just yet... I've got some stuff to discuss with esberglu there that will probably involve changes | 15:13 |
| efried | ack | 15:13 |
| edmondsw | esberglu got some meetings coming up, so it'll be a bit before I can finish looking at that, but comments coming this afternoon | 15:13 |
| prashkre | edmondsw: we have CI +1 on https://review.openstack.org/#/c/557800/, please give your review on it so that for the other patch I can do rebase and do patching on top of them. | 15:16 |
| edmondsw | esberglu one question you can maybe go ahead and answer... why is control.local.conf any different from the AIO local.conf besides the MULTI_HOST value? | 15:17 |
| edmondsw | prashkre ack | 15:17 |
| edmondsw | prashkre done | 15:18 |
| edmondsw | esberglu do we even really need a separate conf file? | 15:19 |
| esberglu | edmondsw: Yeah you're right we probably don't | 15:20 |
| edmondsw | esberglu ok, I dropped 2 comments for now and will try to look more later | 15:21 |
| edmondsw | if you think you'll get a chance to rework that today, I'll wait for that | 15:22 |
| esberglu | edmondsw: Yeah you can hold off | 15:23 |
| edmondsw | cool | 15:23 |
| openstackgerrit | prashkre proposed openstack/nova-powervm stable/queens: Handle exceptions properly with iscsi discovery https://review.openstack.org/561976 | 15:33 |
| openstackgerrit | Merged openstack/nova-powervm master: Return iSCSI Initiator for VIOSes https://review.openstack.org/557800 | 15:34 |
| openstackgerrit | prashkre proposed openstack/nova-powervm master: Miscellaneous iSCSI improvements https://review.openstack.org/561022 | 15:49 |
| prashkre | edmondsw efried: logging off for a while, please review https://review.openstack.org/#/c/561022/ when you get time. | 16:03 |
| edmondsw | prashkre ack | 16:04 |
| edmondsw | tx | 16:04 |
| efried | I sent that one already. | 16:04 |
| *** prashkre has quit IRC | 16:05 | |
| *** prashkre has joined #openstack-powervm | 16:05 | |
| *** prashkre has quit IRC | 16:11 | |
| edmondsw | efried not 561022 you didn't | 16:11 |
| efried | now I did | 16:14 |
| edmondsw | tx | 16:14 |
| openstackgerrit | Merged openstack/nova-powervm stable/queens: Handle exceptions properly with iscsi discovery https://review.openstack.org/561976 | 16:25 |
| openstackgerrit | Merged openstack/nova-powervm master: Miscellaneous iSCSI improvements https://review.openstack.org/561022 | 16:46 |
| edmondsw | esberglu you dropped off slack | 17:46 |
| edmondsw | left you a message about the multinode CI stuff | 17:47 |
| *** AlexeyAbashkin has quit IRC | 18:02 | |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: DNM: ci check https://review.openstack.org/328315 | 18:10 |
| *** prashkre has joined #openstack-powervm | 18:11 | |
| edmondsw | efried I'm gonna tag 6.0.1 for nova-powervm unless you think there's anything more we would want to backport there first | 18:54 |
| efried | edmondsw: Nope, I'm good. | 18:58 |
| edmondsw | coo | 18:58 |
| edmondsw | esberglu openstack/releases only has a ceilometer-powervm deliverable for queens since nova-powervm and networking-powervm weren't under governance yet back then. So should I just use git tag? | 19:05 |
| edmondsw | do you have an example command for what you did to tag things before? | 19:05 |
| esberglu | Might be worth asking if you should create a deliverable for stable branches since we are under governance now? Not sure | 19:07 |
| esberglu | edmondsw: Otherwise it's on a wiki page, I'll send you the link | 19:08 |
| edmondsw | tx | 19:08 |
| esberglu | edmondsw: Updated 6470 when you ave a chance | 19:14 |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: DNM: CI Check2 https://review.openstack.org/328317 | 19:16 |
| mdrabe | efried: What logging levels would I need to see REST request / response? pypowervm=DEBUG? | 19:23 |
| efried | mdrabe: TRACE | 19:24 |
| efried | shit ton of data, that. | 19:24 |
| efried | use wisely. | 19:24 |
| *** AlexeyAbashkin has joined #openstack-powervm | 19:25 | |
| *** AlexeyAbashkin has quit IRC | 19:34 | |
| *** apearson has quit IRC | 21:15 | |
| *** prashkre_ has joined #openstack-powervm | 21:19 | |
| *** prashkre has quit IRC | 21:19 | |
| *** edmondsw has quit IRC | 21:24 | |
| *** tjakobs has quit IRC | 21:35 | |
| *** esberglu has quit IRC | 21:51 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!