Tuesday, 2018-07-10

*** chhagarw has quit IRC00:56
*** chhagarw has joined #openstack-powervm09:59
chhagarwedmondsw: what exactly this waiting for workflow means: https://review.openstack.org/#/c/576034/10:00
*** chhagarw has quit IRC10:19
*** edmondsw has joined #openstack-powervm11:25
*** chhagarw has joined #openstack-powervm12:36
edmondswefried rebase question for you...12:51
edmondswso I have a chain of commits locally12:51
edmondswin one branch12:51
edmondswand the first of those has merged12:51
edmondswbut I need to make changes in some others12:51
edmondswbefore I do that, I want to rebase on master to pick up other changes12:51
efriedgit checkout master12:52
efriedgit pull --all12:52
efriedgit checkout <tip of that branch>12:52
efriedgit rebase master12:52
efriedTo reduce reviewer pain, I would push the series at that point, so you can separate the rebase from the changes.12:52
efriedThen you can git restack (or whatever your favorite technique is) to make your changes.12:53
edmondswso it's just a "git rebase master" like normal... cool12:53
efriedfrom the tip of the branch, yeah.12:53
edmondsw"tip of the branch"?12:53
edmondswI assume that means the branch that has the whole series12:54
efriedyes, the tip being the last change in the series12:55
edmondsw#startmeeting PowerVM Driver Meeting14:00
openstackMeeting 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
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
*** openstack changes topic to " (Meeting topic: PowerVM Driver Meeting)"14:00
openstackThe meeting name has been set to 'powervm_driver_meeting'14:00
*** mujahidali has joined #openstack-powervm14:00
edmondswping efried gman-tx mdrabe mujahidali chhagarw14:00
edmondsw#link https://etherpad.openstack.org/p/powervm_driver_meeting_agenda14:00
edmondsw#topic In-Tree Driver14: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-todos14:00
edmondswother than https://review.openstack.org/#/c/578840/ I'm not sure we've done much in-tree since the last meeting14:01
chhagarwfor the vscsi changes do we need to open a separate bug and incorporate changes ?14:01
edmondswchhagarw you mean in nova-powervm?14:02
edmondswI guess anything we change for vSCSI OOT should probably be changed IT as well now that we've ported that IT14:02
edmondswchhagarw what change are you talking about?14:03
chhagarwyes, I am talking about the LPM changes to save the BDM connection info issue14:03
edmondswyes, that should be a separate bug/commit14:03
edmondswI would start in nova-powervm and then once we've agreed there we can use the same bug but a different commit in nova14:04
edmondswto port the change into the IT driver14:05
edmondsw#topic Out-of-Tree Driver14:05
*** openstack changes topic to "Out-of-Tree Driver (Meeting topic: PowerVM Driver Meeting)"14:05
edmondsw#link https://etherpad.openstack.org/p/powervm-oot-todos14:05
edmondswchhagarw has addressed my comments on https://review.openstack.org/#/c/576034/ and gman-tx +2ed as well, so that is close to merging14:06
edmondswexcept the CI is broken, so we are waiting for that to get fixed and make sure this passes CI14:06
edmondswchhagarw you have tested these changes in devstack, correct?14:06
edmondswincluding the most recent edits?14:07
edmondswchhagarw please add a comment to the commit when you've finished testing with devstack14:08
edmondswI'll wait to remove -W for that as well as for CI14:08
chhagarwthe last change of removing the discover needs to tested on devstack i can check that and update14:08
edmondswin other news, I have moved nova-powervm to stestr like nova: https://review.openstack.org/#/c/579292/14:08
edmondswand we bumped pypowervm requirement to 1.1.16 and merged the support for rbd user14:09
edmondswalso fixed some requirements and constraints issues: https://review.openstack.org/#/c/579701/14:09
edmondswI haven't proposed stestr changes for networking-powervm or ceilometer-powervm yet14:10
edmondswwill get to that eventually14:10
edmondswhave done a few other cleanups in networking-powervm14:11
edmondswmdrabe any update on the MSP support?14:11
edmondswoh, we also merged the changes esberglu had proposed porting IT changes back to OOT14:13
edmondswwhich closes out what he was on the hook for14:13
edmondswefried 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 resize14:14
efriedIT or OOT?14:15
edmondswprobably both, but I noticed OOT I believe14:15
edmondswwe need to confirm and get a bug opened14:15
edmondswI threw it on the TODO list so I didn't forget, but I don't have a devstack env setup atm to confirm it14:15
edmondswif one of you starts working on that, let me know, else I'll try to get to it14:16
efriedI thought we implemented extend14:16
efriedor at least looked into it.14:16
edmondswefried you're probably thinking about our volume drivers14:17
efriedIIRC, if you try to resize the disk, it actually replaces it with a new one (so your data goes away)14:17
efriedAs a result of which I believe we blocked it and/or left it to blow up if you tried to extend it.14:17
efriedI.e. the current behavior is correct.14:17
edmondswwe implement extend_disk for the localdisk driver14:17
efriedUnless we want to blow it up earlier or more gracefully or something.14:17
efriedYeah, cause you can extend an LV.  But you can't extend an LU.14:17
efriedagain, if memory serves.14:17
edmondswcould be right... maybe thorst will remember?14:18
edmondswI'll check with him14:18
edmondswtx, this is why I brought it up14:18
edmondswI don't have anything else for OOT, so...14:19
edmondsw#topic Device Passthrough14:19
*** openstack changes topic to "Device Passthrough (Meeting topic: PowerVM Driver Meeting)"14:19
edmondswefried ^14:19
efried#link Spec up for review: https://review.openstack.org/57935914:19
efried#link Initial change set with yaml schema & parsing: https://review.openstack.org/57928914:20
efriedIn Nova-land, the nrp microversion finally merged last week \o/14:20
efriedFolks are starting to work on functional tests to demonstrate how/whether things are broken on the resource tracker side.14:21
efriedWe will need to follow those and work through them before we'll be able to get our stuff working in nova-powervm.14:21
efriedI don't have a sense yet of how big that will be.14:21
efriedbut definitely not zero.14:21
efriedAny questions?14:22
edmondswnot atm... still need to review those things you linked14:22
efriedWould like mdrabe to look as well, please.14:22
efriededmondsw: Who else do we have to help review/shepherd the nova-powervm code for this effort?14:23
edmondsw#topic PowerVM CI14:23
*** openstack changes topic to "PowerVM CI (Meeting topic: PowerVM Driver Meeting)"14:23
edmondswefried sorry, thought you were done... not sure there is anyone else atm14:23
edmondswother than of course jwcroppe14:24
efriedokay, then we definitely need to make sure mdrabe is on the hook.14:24
edmondsw#link https://etherpad.openstack.org/p/powervm_ci_todos14:24
edmondswmujahidali any more luck getting the CI working?14:24
mujahidaliI have redeployed the CI, it should be back within 2 hour if there are no hurdles.14:24
edmondswok thanks14:25
mujahidaliThe vscsi for stable/oacata still not stacking.14:25
mujahidaliI 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
edmondswif you're still around when it's back, please ping us and let us know14:25
edmondswnew logs with the abs path?14:26
mujahidalilogs are gone now14:26
edmondswlet's just put ocata on hold for now14:27
edmondswwe have pike and queens working, right?14:27
edmondswso remove ocata from the commit so we can merge pike/queens support14:27
edmondswand create a new commit for ocata with your ocata changes14:27
edmondswso we don't lose them14:27
mujahidaliokay, will do it.14:27
edmondswbut then table the ocata commit for now... we have more important things to work on14:28
edmondswtable = put on hold14:28
edmondswhow is the multinode CI work going?14:28
edmondswprobably on hold while you figure out how to get the CI working at all?14:28
mujahidaliyeah, it's on hold for now.14:29
edmondswalright, I assume that's it for CI then14:30
edmondsw#topic Open Discussion14:30
*** openstack changes topic to "Open Discussion (Meeting topic: PowerVM Driver Meeting)"14:30
*** openstack changes topic to "This channel is for PowerVM-related development and discussion. For general OpenStack support, please use #openstack."14:32
openstackMeeting ended Tue Jul 10 14:32:18 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:32
openstackMinutes:        http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-07-10-14.00.html14:32
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-07-10-14.00.txt14:32
openstackLog:            http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-07-10-14.00.log.html14:32
*** mujahidali has quit IRC16:44
*** chhagarw has quit IRC19:07
*** edmondsw has quit IRC22:01
*** edmondsw has joined #openstack-powervm23:16
*** edmondsw has quit IRC23:21

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!