Thursday, 2018-03-29

*** AlexeyAbashkin has joined #openstack-powervm01:08
*** AlexeyAbashkin has quit IRC01:13
*** arunman has joined #openstack-powervm01:49
*** arunman has quit IRC01:55
*** AlexeyAbashkin has joined #openstack-powervm04:09
*** AlexeyAbashkin has quit IRC04:14
*** chhagarw has joined #openstack-powervm06:02
*** k0da has joined #openstack-powervm07:26
*** AlexeyAbashkin has joined #openstack-powervm08:00
*** AlexeyAbashkin has quit IRC08:07
*** AlexeyAbashkin has joined #openstack-powervm08:10
*** Alexey_Abashkin has joined #openstack-powervm08:52
*** AlexeyAbashkin has quit IRC08:54
*** Alexey_Abashkin is now known as AlexeyAbashkin08:54
*** AlexeyAbashkin has quit IRC08:55
*** AlexeyAbashkin has joined #openstack-powervm08:58
*** k0da has quit IRC09:35
*** AlexeyAbashkin has quit IRC10:53
*** AlexeyAbashkin has joined #openstack-powervm11:18
*** edmondsw has joined #openstack-powervm11:30
edmondswefried bug in your test change on
edmondswyou might want to let esberglu fix it so you're not committer and can +2 shortly12:12
efriededmondsw: ack12:49
efriededmondsw: It'll likely be middle of next week before I'm officially anointed.12:52
edmondswefried ack12:53
efriededmondsw: So I can fix, or we can wait, up to you.12:57
efriededmondsw: Or you can fix12:57
efriedthat might actually work best.12:57
efriedat this point mriedem is ready to +2, so the in-house +1s are going to be less significant.12:58
*** edmondsw has quit IRC12:58
*** edmondsw has joined #openstack-powervm12:59
*** edmondsw has quit IRC13:03
*** edmondsw has joined #openstack-powervm13:05
edmondswefried I'll take a look. Not sure offhand how to pass an arg there but I can figure it out13:10
efriededmondsw: I think you just have to change13:12
efriedmock_tf_run.side_effect = exception.InstanceNotFound13:12
efriedmock_tf_run.side_effect = exception.InstanceNotFound(...)13:12
edmondswyeah, I didn't think that would work, but just tried it and it did13:12
efriedMock.side_effect accepts a class or an instance.13:12
edmondswreview up13:13
*** efried has quit IRC13:24
*** efried has joined #openstack-powervm13:25
*** esberglu has joined #openstack-powervm13:28
esbergluedmondsw: efried: Reminder that CI will be down tomorrow for system upgrades13:29
edmondswesberglu ack13:29
edmondswesberglu FYI
edmondswI thought we used LIBS_FROM_GIT but I'm not seeing it so maybe not13:32
esbergluedmondsw: I don't think that should affect us at all13:33
*** tjakobs has joined #openstack-powervm13:42
*** k0da has joined #openstack-powervm13:42
esbergluefried: edmondsw: For a multinode devstack, can you have a compute and control on the same instance?15:52
esbergluSo 2 nodes, 1 control AND compute, 1 compute15:52
esbergluThat's what I thought, just hadn't tried it15:52
esbergluedmondsw: efried: I'm thinking we get rid of our single node CI job completely16:21
esbergluAnd just run multinode16:21
efriedwfm, why wouldn't we?16:21
efriedif we had some single-node setups, we would have to not run multinode tests on 'em.16:21
edmondswthat's what I had assumed16:21
*** k0da has quit IRC16:43
esbergluefried: edmondsw: 6433 for a real easy one16:45
*** AlexeyAbashkin has quit IRC16:53
edmondswesberglu +1 for snapshot and I moved it to "ready for wider review". I'll let you add it to the runways queue17:39
esbergluedmondsw: Awesome tx17:39
edmondswesberglu I guess you can add network hotplug to the runways queue as well17:42
*** tjakobs has quit IRC18:10
*** k0da has joined #openstack-powervm18:12
*** tjakobs has joined #openstack-powervm18:14
*** AlexeyAbashkin has joined #openstack-powervm18:18
*** AlexeyAbashkin has quit IRC18:23
*** k0da has quit IRC18:33
*** k0da has joined #openstack-powervm18:41
*** esberglu has quit IRC18:43
edmondswesberglu I spoke too soon on snapshot... found a couple things while looking at the rest of the series18:44
edmondswlooks like I just missed him18:45
openstackgerritChhavi Agarwal proposed openstack/nova-powervm master: Having iSCSI Initiator locks per VIOS
chhagarwefried, edmondsw: I am changing the get_iscsi_initiator to have locks per vios uuid, can you have a look at the above review18:48
chhagarwI need to further change it to remove the old code, before removing want to have a first level of review, i tested this using PVC18:48
chhagarwif it looks fine, i can update the change to remove the old mechanism and add UT.18:51
edmondswchhagarw looking18:58
*** AlexeyAbashkin has joined #openstack-powervm19:08
edmondswchhagarw is the locking just to prevent you from doing discovery multiple times for the same vios?19:12
edmondswand is that because doing it multiple times would cause a problem, or for performance?19:12
*** AlexeyAbashkin has quit IRC19:13
chhagarwvios initiator iqn should never change, so there is no use to call initiator discovery every time, rather to store and use it19:14
edmondswchhagarw comments are up19:26
chhagarwyeah,i see them, thanks i will update19:34
chhagarwhow to enable the pypowervm debug19:34
chhagarwin nova-powervm19:35
openstackgerritTaylor Jakobson proposed openstack/nova-powervm master: Refactor volume drivers
*** chhagarw has quit IRC19:53
edmondswchhagarw add pypowervm=DEBUG to nova's default_log_levels conf option (comma-delimited if you also want to log other modules)19:59
edmondswchhagarw I didn't understand your response to gfm20:04
*** chhagarw has joined #openstack-powervm20:28
edmondswchhagarw add pypowervm=DEBUG to nova's default_log_levels conf option (comma-delimited if you also want to log other modules)20:45
*** edmondsw has quit IRC20:47
*** edmondsw has joined #openstack-powervm20:47
*** edmondsw has quit IRC20:52
*** esberglu has joined #openstack-powervm20:55
*** chhagarw has quit IRC20:59
*** esberglu has quit IRC21:00
*** esberglu has joined #openstack-powervm21:17
*** esberglu has quit IRC21:22
*** esberglu has joined #openstack-powervm21:23
*** esberglu has quit IRC21:28
*** tjakobs has quit IRC21:47
*** edmondsw has joined #openstack-powervm22:20
*** edmondsw has quit IRC22:25
*** AlexeyAbashkin has joined #openstack-powervm23:08
*** AlexeyAbashkin has quit IRC23:12
*** k0da has quit IRC23:46

Generated by 2.15.3 by Marius Gedminas - find it at!