| *** chhagarw has quit IRC | 00:56 | |
| *** chhagarw has joined #openstack-powervm | 09:59 | |
| chhagarw | edmondsw: what exactly this waiting for workflow means: https://review.openstack.org/#/c/576034/ | 10:00 |
|---|---|---|
| *** chhagarw has quit IRC | 10:19 | |
| *** edmondsw has joined #openstack-powervm | 11:25 | |
| *** chhagarw has joined #openstack-powervm | 12:36 | |
| edmondsw | efried rebase question for you... | 12:51 |
| edmondsw | so I have a chain of commits locally | 12:51 |
| edmondsw | in one branch | 12:51 |
| edmondsw | and the first of those has merged | 12:51 |
| edmondsw | but I need to make changes in some others | 12:51 |
| edmondsw | before I do that, I want to rebase on master to pick up other changes | 12:51 |
| edmondsw | how? | 12:51 |
| efried | git checkout master | 12:52 |
| efried | git pull --all | 12:52 |
| efried | git checkout <tip of that branch> | 12:52 |
| efried | git rebase master | 12:52 |
| efried | To reduce reviewer pain, I would push the series at that point, so you can separate the rebase from the changes. | 12:52 |
| efried | Then you can git restack (or whatever your favorite technique is) to make your changes. | 12:53 |
| edmondsw | so it's just a "git rebase master" like normal... cool | 12:53 |
| efried | from the tip of the branch, yeah. | 12:53 |
| edmondsw | "tip of the branch"? | 12:53 |
| edmondsw | I assume that means the branch that has the whole series | 12:54 |
| efried | yes, the tip being the last change in the series | 12:55 |
| edmondsw | #startmeeting PowerVM Driver Meeting | 14:00 |
| openstack | Meeting started Tue Jul 10 14:00:03 2018 UTC and is due to finish in 60 minutes. The chair is edmondsw. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| *** openstack changes topic to " (Meeting topic: PowerVM Driver Meeting)" | 14:00 | |
| openstack | The meeting name has been set to 'powervm_driver_meeting' | 14:00 |
| *** mujahidali has joined #openstack-powervm | 14:00 | |
| edmondsw | ping efried gman-tx mdrabe mujahidali chhagarw | 14:00 |
| efried | ō/ | 14:00 |
| edmondsw | #link https://etherpad.openstack.org/p/powervm_driver_meeting_agenda | 14:00 |
| edmondsw | #topic In-Tree Driver | 14:00 |
| *** openstack changes topic to "In-Tree Driver (Meeting topic: PowerVM Driver Meeting)" | 14:00 | |
| edmondsw | #link https://etherpad.openstack.org/p/powervm-in-tree-todos | 14:00 |
| edmondsw | other than https://review.openstack.org/#/c/578840/ I'm not sure we've done much in-tree since the last meeting | 14:01 |
| chhagarw | for the vscsi changes do we need to open a separate bug and incorporate changes ? | 14:01 |
| edmondsw | chhagarw you mean in nova-powervm? | 14:02 |
| edmondsw | I guess anything we change for vSCSI OOT should probably be changed IT as well now that we've ported that IT | 14:02 |
| edmondsw | chhagarw what change are you talking about? | 14:03 |
| chhagarw | yes, I am talking about the LPM changes to save the BDM connection info issue | 14:03 |
| edmondsw | yes, that should be a separate bug/commit | 14:03 |
| edmondsw | I would start in nova-powervm and then once we've agreed there we can use the same bug but a different commit in nova | 14:04 |
| chhagarw | sure | 14:04 |
| edmondsw | to port the change into the IT driver | 14:05 |
| edmondsw | #topic Out-of-Tree Driver | 14:05 |
| *** openstack changes topic to "Out-of-Tree Driver (Meeting topic: PowerVM Driver Meeting)" | 14:05 | |
| edmondsw | https://etherpad.openstack.org/p/powervm-oot-todos | 14:05 |
| edmondsw | #link https://etherpad.openstack.org/p/powervm-oot-todos | 14:05 |
| edmondsw | chhagarw has addressed my comments on https://review.openstack.org/#/c/576034/ and gman-tx +2ed as well, so that is close to merging | 14:06 |
| edmondsw | except the CI is broken, so we are waiting for that to get fixed and make sure this passes CI | 14:06 |
| edmondsw | chhagarw you have tested these changes in devstack, correct? | 14:06 |
| edmondsw | including the most recent edits? | 14:07 |
| edmondsw | chhagarw please add a comment to the commit when you've finished testing with devstack | 14:08 |
| edmondsw | I'll wait to remove -W for that as well as for CI | 14:08 |
| chhagarw | the last change of removing the discover needs to tested on devstack i can check that and update | 14:08 |
| edmondsw | tx | 14:08 |
| edmondsw | in other news, I have moved nova-powervm to stestr like nova: https://review.openstack.org/#/c/579292/ | 14:08 |
| edmondsw | and we bumped pypowervm requirement to 1.1.16 and merged the support for rbd user | 14:09 |
| edmondsw | also fixed some requirements and constraints issues: https://review.openstack.org/#/c/579701/ | 14:09 |
| edmondsw | I haven't proposed stestr changes for networking-powervm or ceilometer-powervm yet | 14:10 |
| edmondsw | will get to that eventually | 14:10 |
| edmondsw | have done a few other cleanups in networking-powervm | 14:11 |
| edmondsw | mdrabe any update on the MSP support? | 14:11 |
| edmondsw | oh, we also merged the changes esberglu had proposed porting IT changes back to OOT | 14:13 |
| edmondsw | which closes out what he was on the hook for | 14:13 |
| edmondsw | efried chhagarw gman-tx I happened to notice that extend_disk has not been implemented for the ssp disk driver, and I think that is going to blow up if you try to extend disk during resize | 14:14 |
| efried | IT or OOT? | 14:15 |
| edmondsw | probably both, but I noticed OOT I believe | 14:15 |
| edmondsw | we need to confirm and get a bug opened | 14:15 |
| edmondsw | I threw it on the TODO list so I didn't forget, but I don't have a devstack env setup atm to confirm it | 14:15 |
| edmondsw | if one of you starts working on that, let me know, else I'll try to get to it | 14:16 |
| efried | I thought we implemented extend | 14:16 |
| efried | or at least looked into it. | 14:16 |
| edmondsw | efried you're probably thinking about our volume drivers | 14:17 |
| efried | IIRC, if you try to resize the disk, it actually replaces it with a new one (so your data goes away) | 14:17 |
| efried | As a result of which I believe we blocked it and/or left it to blow up if you tried to extend it. | 14:17 |
| efried | I.e. the current behavior is correct. | 14:17 |
| edmondsw | we implement extend_disk for the localdisk driver | 14:17 |
| efried | Unless we want to blow it up earlier or more gracefully or something. | 14:17 |
| efried | Yeah, cause you can extend an LV. But you can't extend an LU. | 14:17 |
| efried | again, if memory serves. | 14:17 |
| edmondsw | could be right... maybe thorst will remember? | 14:18 |
| edmondsw | I'll check with him | 14:18 |
| edmondsw | tx, this is why I brought it up | 14:18 |
| edmondsw | I don't have anything else for OOT, so... | 14:19 |
| edmondsw | #topic Device Passthrough | 14:19 |
| *** openstack changes topic to "Device Passthrough (Meeting topic: PowerVM Driver Meeting)" | 14:19 | |
| edmondsw | efried ^ | 14:19 |
| efried | #link Spec up for review: https://review.openstack.org/579359 | 14:19 |
| efried | #link Initial change set with yaml schema & parsing: https://review.openstack.org/579289 | 14:20 |
| efried | In Nova-land, the nrp microversion finally merged last week \o/ | 14:20 |
| efried | Folks are starting to work on functional tests to demonstrate how/whether things are broken on the resource tracker side. | 14:21 |
| efried | We will need to follow those and work through them before we'll be able to get our stuff working in nova-powervm. | 14:21 |
| efried | I don't have a sense yet of how big that will be. | 14:21 |
| efried | but definitely not zero. | 14:21 |
| efried | Any questions? | 14:22 |
| edmondsw | not atm... still need to review those things you linked | 14:22 |
| efried | Would like mdrabe to look as well, please. | 14:22 |
| efried | edmondsw: Who else do we have to help review/shepherd the nova-powervm code for this effort? | 14:23 |
| edmondsw | #topic PowerVM CI | 14:23 |
| *** openstack changes topic to "PowerVM CI (Meeting topic: PowerVM Driver Meeting)" | 14:23 | |
| edmondsw | efried sorry, thought you were done... not sure there is anyone else atm | 14:23 |
| edmondsw | other than of course jwcroppe | 14:24 |
| efried | okay, then we definitely need to make sure mdrabe is on the hook. | 14:24 |
| edmondsw | agreed | 14:24 |
| edmondsw | #link https://etherpad.openstack.org/p/powervm_ci_todos | 14:24 |
| edmondsw | mujahidali any more luck getting the CI working? | 14:24 |
| mujahidali | I have redeployed the CI, it should be back within 2 hour if there are no hurdles. | 14:24 |
| edmondsw | ok thanks | 14:25 |
| mujahidali | The vscsi for stable/oacata still not stacking. | 14:25 |
| mujahidali | I have tried the absolute path as well, suggested by edmondsw: It's failing on the same error "opt/stack/devstack/lib/neutron:420 neutron-api did not start " | 14:25 |
| edmondsw | if you're still around when it's back, please ping us and let us know | 14:25 |
| mujahidali | sure | 14:25 |
| edmondsw | new logs with the abs path? | 14:26 |
| mujahidali | logs are gone now | 14:26 |
| edmondsw | k | 14:26 |
| edmondsw | let's just put ocata on hold for now | 14:27 |
| edmondsw | we have pike and queens working, right? | 14:27 |
| mujahidali | yes | 14:27 |
| edmondsw | so remove ocata from the commit so we can merge pike/queens support | 14:27 |
| edmondsw | and create a new commit for ocata with your ocata changes | 14:27 |
| edmondsw | so we don't lose them | 14:27 |
| mujahidali | okay, will do it. | 14:27 |
| edmondsw | but then table the ocata commit for now... we have more important things to work on | 14:28 |
| edmondsw | table = put on hold | 14:28 |
| edmondsw | how is the multinode CI work going? | 14:28 |
| edmondsw | probably on hold while you figure out how to get the CI working at all? | 14:28 |
| edmondsw | :) | 14:28 |
| mujahidali | yeah, it's on hold for now. | 14:29 |
| edmondsw | alright, I assume that's it for CI then | 14:30 |
| edmondsw | #topic Open Discussion | 14:30 |
| *** openstack changes topic to "Open Discussion (Meeting topic: PowerVM Driver Meeting)" | 14:30 | |
| edmondsw | anyone? | 14:30 |
| edmondsw | #endmeeting | 14:32 |
| *** openstack changes topic to "This channel is for PowerVM-related development and discussion. For general OpenStack support, please use #openstack." | 14:32 | |
| openstack | Meeting ended Tue Jul 10 14:32:18 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:32 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-07-10-14.00.html | 14:32 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-07-10-14.00.txt | 14:32 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-07-10-14.00.log.html | 14:32 |
| *** mujahidali has quit IRC | 16:44 | |
| *** chhagarw has quit IRC | 19:07 | |
| *** edmondsw has quit IRC | 22:01 | |
| *** edmondsw has joined #openstack-powervm | 23:16 | |
| *** edmondsw has quit IRC | 23:21 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!