Monday, 2017-01-23

*** thorst_ has joined #openstack-powervm00:03
*** thorst_ has quit IRC00:08
*** edmondsw has joined #openstack-powervm00:43
*** edmondsw has quit IRC00:48
*** thorst_ has joined #openstack-powervm00:50
*** thorst_ has quit IRC00:53
*** thorst_ has joined #openstack-powervm01:55
*** thorst_ has quit IRC01:56
*** thorst_ has joined #openstack-powervm02:48
*** thorst_ has quit IRC02:48
*** jwcroppe has quit IRC04:17
*** thorst_ has joined #openstack-powervm04:49
*** thorst_ has quit IRC04:54
openstackgerritShyama proposed openstack/nova-powervm: Read volume id from BDM connection_info['serial']  https://review.openstack.org/42321205:08
*** edmondsw has joined #openstack-powervm06:13
*** edmondsw has quit IRC06:17
*** thorst_ has joined #openstack-powervm06:50
*** thorst_ has quit IRC06:55
*** k0da has joined #openstack-powervm08:27
*** thorst_ has joined #openstack-powervm08:51
*** thorst_ has quit IRC08:56
*** edmondsw has joined #openstack-powervm09:50
*** yoba has joined #openstack-powervm09:51
*** edmondsw has quit IRC09:54
*** Nim has joined #openstack-powervm09:58
yobaHi all. I have a mitaka env with x86 Nova and would like to make nova-link working with. Any how to? and operation docs? Thanks a lot.10:02
*** thorst_ has joined #openstack-powervm10:52
*** thorst_ has quit IRC10:57
*** thorst_ has joined #openstack-powervm12:03
*** thorst_ has quit IRC12:04
*** tblakes has joined #openstack-powervm12:58
*** kylek3h_away has quit IRC13:04
*** nbante has joined #openstack-powervm13:11
*** smatzek has joined #openstack-powervm13:15
*** edmondsw has joined #openstack-powervm13:19
*** jwcroppe has joined #openstack-powervm13:25
*** mdrabe has joined #openstack-powervm13:34
*** kylek3h_away has joined #openstack-powervm13:41
*** kylek3h_away is now known as kylek3h13:54
*** tlian has joined #openstack-powervm13:54
*** thorst_ has joined #openstack-powervm14:08
*** esberglu has joined #openstack-powervm14:09
efriedyoba: Have you consulted the README and/or usage.rst?14:18
efriedthorst_: Any discussion needed on FileIO volume driver?14:27
thorst_efried: I haven't looked at review comments14:29
efriedk14:30
thorst_I decided to make a SDN video instead of checking the review comments like I said I would14:30
thorst_:-)14:30
thorst_will do the GPFS comments this afternoon14:30
*** smatzek has quit IRC14:31
esbergluadreznec: efried: thorst_: You guys around to talk dual ci?14:35
efriedI'm here.14:35
esbergluJust trying to figure out what we want to do with pypowervm since we to patch it14:39
esbergluThe way we are doing it for OOT right now is preinstalling pypowervm with our patches and then removing pypowervm from upper-constraints and global-requirements14:42
esbergluDon't think we want to do it that way for in tree14:42
thorst_so we definitely must have a way to patch that remote enablement onto whatever we install14:52
thorst_on the one hand, we want it to install from nova requirements.14:53
thorst_my concern is that the nova-compute process will start before we've updated the driver.14:53
thorst_s/driver/pypowervm adapter.py14:53
*** smatzek has joined #openstack-powervm14:57
efriedthorst_: esberglu: We could restart the compute process if necessary.14:58
*** smatzek_ has joined #openstack-powervm14:58
efriedI don't love the idea of patching the pypowervm that lives in /usr/lib/python2.7/dist-packages14:58
efriedSo perhaps our best bet is to keep the structure we have in place, and just have the CI install (in /opt/stack) the 1.0.0.4 branch of pypowervm.14:59
efriedThe downside is that we have to remember to move that version number when reqs change.14:59
efriedThough we could conceivably pull the version number from global reqs as we do the above.15:00
efriedYeah, that seems like a good compromise.  Thoughts?15:00
esbergluI'm cool with that. Pretty easy changes code-wise too15:01
thorst_rip it15:01
*** smatzek has quit IRC15:01
*** Nim has quit IRC15:19
esbergluthorst_: efried: Okay I put up a patch for it15:21
*** dwayne__ has joined #openstack-powervm15:22
*** dwayne__ has quit IRC15:25
thorst_hmm....review your code or work on LLDs I've been asked to update15:26
thorst_code it is!15:26
*** dwayne__ has joined #openstack-powervm15:36
*** k0da has quit IRC15:49
efriedesberglu: How are things looking?15:56
esbergluAbout to redeploy staging with the dual ci patch15:57
*** burgerk has joined #openstack-powervm15:59
*** apearson has joined #openstack-powervm16:02
thorst_efried: so gfm pointed out something very good.  It looks like that binary decision doesn't HAVE to be binary16:29
thorst_and I think I can remove it.16:29
thorst_however, I think I'd like to handle that in a subsequent patch.16:29
efriedthorst_: will removing it obviate the need for the dynamic conf var?16:30
thorst_no it won't REMOVE the need for the dynamic conf var.16:30
thorst_but it will remove the need for a different conf var altogether16:30
thorst_and I think I want to get these changes back to newton possibly16:30
thorst_but we need them at least fixed for ocata...16:31
efriedSo I don't object in principle to a followup change set, as long as this one isn't introducing something that will then be immediately ripped out.  Why do you want to defer?16:31
thorst_I may even write up a bug for this thing...because atm iscsi hosts are using the wrong host name16:31
openstackgerritEric Fried proposed openstack/nova-powervm: Mock data cleanup 1  https://review.openstack.org/42144816:31
thorst_maybe I can just patch back in newton.16:32
thorst_just want to defer because this patch is becoming way more than fileio16:33
efriedI see.  I would definitely be on board with splitting out the conf changes from the FileIO driver.  Those seemed kinda separate to me anyway.16:35
efriedInterdependent, but separate.16:35
thorst_well, I want to get enough in to make it work.16:35
thorst_and then fix it16:35
thorst_and then get a bug out in newton.16:35
efriedAs you wish.  Just let me know what I need to review, and when.16:37
*** yoba has quit IRC16:53
openstackgerritEric Fried proposed openstack/networking-powervm: Move deprecated pci_passthrough_whitelist  https://review.openstack.org/42423816:53
efriedthorst_: Did you say horizon had some way of spawning an instance with no network?  I don't see it.17:02
thorst_not horizon17:02
thorst_command line17:02
efriedokay.17:05
openstackgerritDrew Thorstensen (thorst) proposed openstack/nova-powervm: File I/O Volume Driver Implementation  https://review.openstack.org/42227317:17
thorst_efried: See if you're OK with that...17:18
efriedthorst_: ack17:18
efriedthorst_: reviewed17:35
efriedLooks fine to me.17:35
efriedcouple nits17:35
*** nbante has quit IRC17:56
thorst_efried: I'll get this bug fix up too...I'd like this one to go back to newton...18:20
thorst_give me 15 and it'll be posted18:20
efriedk18:20
*** apearson has quit IRC18:51
openstackgerritDrew Thorstensen (thorst) proposed openstack/nova-powervm: Fix hostname & initiator for volume drivers  https://review.openstack.org/42429018:57
thorst_efried: Take a look at that...I'd like to merge my other one on top of this.18:58
*** apearson has joined #openstack-powervm19:01
efriedthorst_: wanna fix pep8 while I'm looking?19:22
*** k0da has joined #openstack-powervm19:22
efriedthorst_: reviewed.19:29
thorst_efried: I'm merging into other one19:30
thorst_I'm kinda finally liking it19:31
thorst_I'm moving more stuff into the __init__.py19:32
thorst_which has pros/cons19:32
openstackgerritDrew Thorstensen (thorst) proposed openstack/nova-powervm: File I/O Volume Driver Implementation  https://review.openstack.org/42227320:23
thorst_my word...this is more programming than I've done in weeks20:27
thorst_I've missed this coding  :-)20:27
openstackgerritDrew Thorstensen (thorst) proposed openstack/nova-powervm: Fix hostname & initiator for volume drivers  https://review.openstack.org/42429020:30
thorst_efried: two in the hopper20:30
efriedthorst_: ack20:31
efriedthorst_: done.  Note -1 has to do with PS6 dialog.20:55
*** smatzek_ has quit IRC21:30
openstackgerritDrew Thorstensen (thorst) proposed openstack/nova-powervm: File I/O Volume Driver Implementation  https://review.openstack.org/42227321:34
*** thorst_ has quit IRC21:35
*** thorst_ has joined #openstack-powervm21:35
*** thorst_ has quit IRC21:40
*** smatzek_ has joined #openstack-powervm21:58
*** thorst_ has joined #openstack-powervm21:58
*** thorst_ has quit IRC22:03
*** burgerk has quit IRC22:10
*** thorst_ has joined #openstack-powervm22:13
efriedthorst_: you still around?22:17
thorst_kinda...just wrapping up for five minutes22:17
thorst_something quick?22:18
efriedYeah - just wanted to make sure you saw my comment on PS622:18
efriedYou didn't address it in PS822:18
efriedIt was the actual reason for my -1.22:18
efriedhttps://review.openstack.org/#/c/422273/6/nova_powervm/virt/powervm/volume/__init__.py@5422:18
efriedI copied it into a PS8 comment.22:21
*** thorst_ has quit IRC22:22
esbergluefried: You still here?22:25
*** esberglu has quit IRC22:33
*** esberglu has joined #openstack-powervm22:33
*** esberglu_ has joined #openstack-powervm22:35
*** esberglu has quit IRC22:37
*** esberglu_ has quit IRC22:39
*** smatzek_ has quit IRC22:41
*** tblakes has quit IRC22:51
*** kylek3h has quit IRC22:58
*** k0da has quit IRC23:05
*** mdrabe has quit IRC23:05
*** k0da has joined #openstack-powervm23:08
*** esberglu has joined #openstack-powervm23:12
*** esberglu has quit IRC23:12
*** apearson has quit IRC23:34
*** edmondsw has quit IRC23:57
*** edmondsw has joined #openstack-powervm23:59

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