Wednesday, 2017-12-06

*** svenkat has joined #openstack-powervm00:30
*** gman-tx has joined #openstack-powervm00:30
*** esberglu has joined #openstack-powervm01:02
*** svenkat has quit IRC01:05
*** esberglu has quit IRC01:07
*** gman-tx has quit IRC01:54
*** gman-tx has joined #openstack-powervm01:58
*** gman-tx has quit IRC02:45
*** esberglu has joined #openstack-powervm02:51
*** esberglu has quit IRC02:52
*** esberglu has joined #openstack-powervm02:52
*** esberglu has quit IRC02:57
*** csky has joined #openstack-powervm02:58
*** csky has quit IRC03:03
*** csky has joined #openstack-powervm03:03
*** csky has quit IRC04:00
*** chhavi has joined #openstack-powervm04:18
*** chhavi__ has joined #openstack-powervm04:18
*** esberglu has joined #openstack-powervm05:33
*** esberglu has quit IRC05:38
*** AlexeyAbashkin has joined #openstack-powervm06:07
*** AlexeyAbashkin has quit IRC06:11
*** AlexeyAbashkin has joined #openstack-powervm06:12
*** chhavi__ has quit IRC06:19
*** AlexeyAbashkin has quit IRC06:26
*** esberglu has joined #openstack-powervm07:23
*** esberglu has quit IRC07:28
*** AlexeyAbashkin has joined #openstack-powervm08:32
*** AlexeyAbashkin has quit IRC08:38
*** AlexeyAbashkin has joined #openstack-powervm08:44
*** AlexeyAbashkin has quit IRC08:49
*** adi_____ has quit IRC09:01
*** adi_____ has joined #openstack-powervm09:02
*** esberglu has joined #openstack-powervm09:11
*** esberglu has quit IRC09:12
*** esberglu has joined #openstack-powervm09:12
*** esberglu has quit IRC09:12
*** esberglu has joined #openstack-powervm09:13
*** AlexeyAbashkin has joined #openstack-powervm09:14
*** esberglu has quit IRC09:17
*** esberglu has joined #openstack-powervm11:00
*** esberglu has quit IRC11:04
*** AlexeyAbashkin has quit IRC11:58
*** efried_cya_wed is now known as efried12:46
*** AlexeyAbashkin has joined #openstack-powervm12:46
*** svenkat has joined #openstack-powervm13:05
*** svenkat_ has joined #openstack-powervm13:09
*** svenkat has quit IRC13:10
*** svenkat_ is now known as svenkat13:10
*** apearson has joined #openstack-powervm13:34
*** gman-tx has joined #openstack-powervm13:42
*** esberglu has joined #openstack-powervm13:44
*** esberglu has quit IRC13:48
*** esberglu has joined #openstack-powervm14:27
*** csky_ has joined #openstack-powervm14:36
*** esberglu has quit IRC14:44
*** esberglu has joined #openstack-powervm14:59
efriedesberglu Which volume driver are you porting first?15:38
esbergluefried: vscsi15:38
esbergluefried: Did you see the convo yesterday about that at all?15:39
efriedyes15:39
efriedesberglu Is VscsiVolumeAdapter used directly?  Or just as ABC for PV and ISCSI?15:40
esbergluPVVscsiFCVolumeAdapter15:41
esbergluefried: https://review.openstack.org/#/c/526094/15:41
esbergluStarted there15:42
esbergluBut that is VERY wip15:42
efriedOkay, so you're porting PVVscsiFCVolumeAdapter?15:44
esbergluYeah15:44
efriedSo a couple of things here.15:44
efriedFirst, why did you happen to choose this one?15:44
esbergluefried: Thought that's what we decided on earlier this cycle no?15:45
efriedI don't remember.  I'm fine with it, just curious.15:46
efriedWhat else did we decide on, in terms of volume adapters?15:46
efriedIs this for Q?15:46
efriedAnd is it the only one we're doing for Q?15:46
esbergluefried: yes and yes15:46
efriedk15:47
efriedSo15:47
efriedIMO, we *should* collapse completely to propose to in-tree.15:47
efriedSame as we did for the networking stuff.15:47
esbergluefried: Just for LOC and reviewer ease?15:48
efriedI could be convinced that we could collapse to *one* base class and one derived class.15:48
efriedBut even there, I think it'd be better to collapse completely.15:48
efriedHere's why:15:49
efriedIf it was just a case of the base class providing abstract methods and some default impls, we could make a case to reviewers, maybe.15:49
efriedBut here we've got these weird indirection things, where do_this calls the subclass _do_this -- which is going to seem really weird without knowing the whole picture.15:50
efriedIn general, it's against the philosophy we're using (call it "agile" or "the OpenStack way" or whatever) to propose code in anticipation of something that doesn't exist yet.  We're supposed to be thinking of each patch as if it's the last one.15:50
efriedThere are exceptions - but even there, such patches are typically proposed in series, all together.  Which is not going to be the case here, since this is the only one for Q.15:51
efriedThe other thing, about refactoring the way we do this stuff OOT:15:52
efriedI'd be on board with that.15:52
efriedThe way these classes are set up right now is kind of more Java-esque, where there's strict typing and stuff.15:52
efriedI would be in favor of a looser duck-typed setup, where we don't strictly have to have X defined in the base class to use X in a subclass.15:53
efriedBut I would kinda have to see what that looks like tbh.15:53
esbergluefried: I can start thinking about it as I work through IT15:56
efriedight.  Does what I said above make sense?  edmondsw may want to argue the other side again tomorrow.15:56
esbergluefried: Yeah it makes sense. And I suspect the nova guys would say the same and that's who ultimately need on board15:57
efriedjust so15:57
esbergluYou're in tomorrow right?15:57
esbergluOtherwise I can argue your side to edmondsw15:58
efriedesberglu Yes, I'm here tomorrow and first half of Friday (like 6-11 or so)16:04
esbergluThen out through the month?16:05
*** gman-tx has quit IRC16:09
efriedesberglu Yes.16:28
*** gman-tx has joined #openstack-powervm16:43
*** AlexeyAbashkin has quit IRC16:54
*** chhavi has quit IRC17:56
*** apearson has quit IRC18:01
*** apearson has joined #openstack-powervm18:01
*** apearson has quit IRC18:02
*** apearson has joined #openstack-powervm18:02
*** apearson has quit IRC18:03
*** apearson has joined #openstack-powervm18:03
*** apearson has quit IRC18:03
*** apearson has joined #openstack-powervm18:04
*** apearson has quit IRC18:04
*** apearson has joined #openstack-powervm18:04
*** apearson has quit IRC18:05
*** apearson has joined #openstack-powervm18:05
*** apearson has quit IRC18:06
*** apearson has joined #openstack-powervm18:06
*** apearson has quit IRC18:06
*** apearson has joined #openstack-powervm18:07
*** apearson has quit IRC18:08
*** apearson has joined #openstack-powervm18:08
*** apearson has quit IRC18:09
*** apearson has joined #openstack-powervm18:09
*** apearson has quit IRC18:09
*** apearson has joined #openstack-powervm18:10
*** apearson has quit IRC18:10
*** apearson has joined #openstack-powervm18:10
*** apearson has quit IRC18:11
*** apearson has joined #openstack-powervm18:11
*** apearson has quit IRC18:12
*** apearson has joined #openstack-powervm18:12
*** apearson has quit IRC18:13
*** apearson has joined #openstack-powervm18:13
*** apearson has quit IRC18:13
*** apearson has joined #openstack-powervm18:14
*** apearson has quit IRC18:14
*** apearson has joined #openstack-powervm18:14
*** apearson has quit IRC18:15
*** apearson has joined #openstack-powervm18:15
*** apearson has quit IRC18:16
esbergluefried: Just got an IT cloud stood up with SEA, can spawn and ssh into instances so it's a promising start18:29
*** gman-tx has quit IRC18:31
*** esberglu has quit IRC18:56
*** apearson has joined #openstack-powervm19:15
*** esberglu has joined #openstack-powervm19:24
efriedesberglu Great news!19:31
*** gman-tx has joined #openstack-powervm19:49
*** gman-tx has quit IRC20:03
*** AlexeyAbashkin has joined #openstack-powervm21:38
*** apearson has quit IRC21:38
*** svenkat has quit IRC21:55
*** apearson has joined #openstack-powervm22:01
*** AlexeyAbashkin has quit IRC22:43
*** apearson has quit IRC23:16
*** openstack has joined #openstack-powervm23:29
*** ChanServ sets mode: +o openstack23:29
*** edmondsw has joined #openstack-powervm23:42
*** chhavi has joined #openstack-powervm23:52
*** esberglu has quit IRC23:57
*** chhavi has quit IRC23:57

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