Monday, 2018-04-02

*** edmondsw has joined #openstack-powervm00:25
*** edmondsw has quit IRC00:29
*** edmondsw has joined #openstack-powervm02:13
*** edmondsw has quit IRC02:17
*** k0da has joined #openstack-powervm05:50
*** AlexeyAbashkin has joined #openstack-powervm07:50
*** AlexeyAbashkin has quit IRC10:43
*** chhagarw has joined #openstack-powervm10:59
openstackgerritArun Mani proposed openstack/nova-powervm master: Add affinity score check attribute to flavor
*** AlexeyAbashkin has joined #openstack-powervm11:31
openstackgerritChhavi Agarwal proposed openstack/nova-powervm master: WIP: Having iSCSI Initiator locks per VIOS
openstackgerritChhavi Agarwal proposed openstack/nova-powervm master: WIP: Having iSCSI Initiator locks per VIOS
*** edmondsw has joined #openstack-powervm12:43
chhagarw@all: getting the pep8 error : failed with error code 1 in /home/zuul/src/, falling back to uneditable format,Could not determine repository location12:53
chhagarware there any issues with build ci12:54
chhagarwedmondsw: replied to your review comments for get_iscsi_initiators i made a change the, we can discuss if there is anything over IRC,
openstackgerritMerged openstack/nova-powervm master: Add affinity score check attribute to flavor
edmondswchhagarw you cut off the first part of the pep8 error: Error when trying to get requirement for VCS system Command "git config --get-regexp remote\..*\.url"13:07
edmondswand the end: of /home/zuul/src/
edmondswseems like something to just recheck13:08
edmondswchhagarw your py27 error is due to overindentation on L134 of ... if you fix that the new patch set will be checked and hopefully the pep8 issues go away (else you'll presumably just get a different pep8 error)13:10
chhagarwsure let me check13:11
edmondswchhagarw I'm wondering if you're really reading my comments... some of them still apply13:14
chhagarwthere are few comments for get_iscsi_initiators, which still needs to be discussed, so I left for now before we conclude.13:45
*** esberglu has joined #openstack-powervm13:52
*** tjakobs has joined #openstack-powervm14:05
edmondswesberglu please take another look at
edmondswneed your +2/+a14:22
esbergluedmondsw: ack14:23
esbergluedmondsw: +2/W+114:28
edmondswesberglu tx14:33
edmondswesberglu thoughts on respinning your IT vscsi commit to include that?14:35
edmondswfried_bunny still fried on Monday?14:36
fried_bunnyIt's Easter Monday, right?14:36
fried_bunnyI'll shed the fur tomorrow.14:36
edmondswI don't think I've ever heard of Easter Monday14:38
esbergluedmondsw: Sure. I can update wwpns too14:38
edmondswesberglu tx14:38
fried_bunnyedmondsw: I thought Easter Monday was *the* day.  But I'm no expert.14:39
edmondswEaster Sunday14:39
esbergluedmondsw: fried_bunny: I fixed up snapshot and base disk adapter, ready for review again14:39
esbergluI also rebased everything so it's one commit chain14:39
openstackgerritMerged openstack/nova-powervm master: Set volume ID as dev tag when mapping VSCSI vols
fried_bunnyesberglu: I'm going to wait a day or two to re-look at anything powervm.14:40
fried_bunnyfor... reasons.14:40
esbergluHmm what could those be14:40
esberglufried_bunny: edmondsw: Working on the cold migration code right now. It uses the StoreNvram task15:15
esbergluLooking here
esbergluSounds like we don't need it right away, but could add it later as an enhancement?15:15
fried_bunnyesberglu: Later.  When we support remote restart.  Which ain't yet.15:15
fried_bunnyditto with all the slot manager args15:16
edmondswyeah, don't worry about slots yet15:16
edmondswor nvram15:17
edmondswesberglu FYI
esbergluedmondsw: Yeah I saw that, not sure how it affects us outside of devstack yet15:24
edmondswhopefully just devstack/CI15:24
openstackgerritMerged openstack/nova-powervm master: remove pycrypto
*** k0da has quit IRC15:30
esbergluedmondsw: fried_bunny:
esbergluCan we just skip the detach then on a resize?15:31
edmondswesberglu I doubt it15:31
edmondswresize could move it to a different host15:32
edmondswso volumes would need to be detached on the old host and reattached on the new host15:32
fried_bunnyesberglu: agree with edmondsw15:32
esberglufried_bunny: How did you generate the release note name here?
fried_bunnyesberglu: Use the reno command.  <wait, there's more>16:11
fried_bunnyI think there's a way to do that via a tox command.  But that takes forever.16:12
fried_bunnySo what I do is:16:12
fried_bunny. .tox/venv/bin/activate16:12
fried_bunnyreno new <slugname>16:12
fried_bunnywhere <slugname> is whatever I want the basis of the reno name to be, like my-nifty-feature16:12
fried_bunnyThen reno generates the rest of the name, stuffs it in the right directory, and puts the template in it.16:12
fried_bunny(afaict, that's actually all reno does)16:13
*** esberglu has quit IRC16:31
*** esberglu has joined #openstack-powervm16:31
esbergluedmondsw: vSCSI is ready again16:36
*** AlexeyAbashkin has quit IRC16:37
edmondswesberglu pypowervm is still only 1.1.11 in nova's requirements.txt... will need to bump that for the wwpn change16:49
edmondswotherwise looks good16:50
openstackgerritSridhar Venkat proposed openstack/nova-powervm master: Set fuse_limit to 15 instead of default 32
esbergluedmondsw: vSCSI rebased on pypowervm req bump18:46
*** AlexeyAbashkin has joined #openstack-powervm18:49
*** AlexeyAbashkin has quit IRC18:53
*** AlexeyAbashkin has joined #openstack-powervm19:04
*** AlexeyAbashkin has quit IRC19:09
*** chhagarw has quit IRC19:38
*** AlexeyAbashkin has joined #openstack-powervm21:02
*** esberglu has left #openstack-powervm21:03
*** esberglu has joined #openstack-powervm21:03
*** esberglu has quit IRC21:03
*** AlexeyAbashkin has quit IRC21:06
fried_bunnyedmondsw: FYI, I'm against the fuse limit patch.  Last I heard, this was a REST bug that changh needs to fix.21:12
fried_bunnyIf there's some reason (like perf or whatever) beyond "it doesn't work when it's 32" then that's one thing.21:12
*** tjakobs has quit IRC22:06
edmondswfried_bunny ack... I was hoping you would chime in there as I thought I'd seen you talking about it on slack23:14

Generated by 2.15.3 by Marius Gedminas - find it at!