Friday, 2018-03-23

*** k0da has quit IRC00:31
*** edmondsw has quit IRC00:46
*** esberglu has joined #openstack-powervm01:03
*** esberglu has quit IRC01:08
*** k0da has joined #openstack-powervm01:20
*** k0da has quit IRC02:13
*** edmondsw has joined #openstack-powervm02:34
*** edmondsw has quit IRC02:39
*** k0da has joined #openstack-powervm03:11
*** AlexeyAbashkin has joined #openstack-powervm03:15
*** k0da has quit IRC03:18
*** AlexeyAbashkin has quit IRC03:20
*** arunman has joined #openstack-powervm03:52
*** arunman has quit IRC03:55
*** arunman has joined #openstack-powervm04:00
*** arunman has quit IRC04:01
*** edmondsw has joined #openstack-powervm04:23
*** edmondsw has quit IRC04:27
*** chhagarw has joined #openstack-powervm04:47
*** AlexeyAbashkin has joined #openstack-powervm05:15
*** AlexeyAbashkin has quit IRC05:19
*** arunman has joined #openstack-powervm05:39
*** AlexeyAbashkin has joined #openstack-powervm06:17
*** AlexeyAbashkin has quit IRC06:22
*** edmondsw has joined #openstack-powervm07:59
*** AlexeyAbashkin has joined #openstack-powervm08:02
*** edmondsw has quit IRC08:03
*** k0da has joined #openstack-powervm08:30
*** edmondsw has joined #openstack-powervm09:47
*** edmondsw has quit IRC09:51
*** chhavi__ has joined #openstack-powervm10:15
*** chhagarw has quit IRC10:16
*** k0da has quit IRC10:31
*** k0da has joined #openstack-powervm11:47
*** edmondsw has joined #openstack-powervm12:03
chhavi__edmondsw: GM12:22
edmondswchhavi__ hi12:22
chhavi__edmondsw: couple of things, changes are up for review, I am adding the methods to support LPM12:23
chhavi__while running migrate for iSCSI, I am getting the below error "Failed to find an existing mover service partition on the system."12:23
chhavi__do we need MSP in SDE as well12:24
edmondswchhavi__ the LPM support will be a separate commit, I hope?12:24
chhavi__yeah that will be a separate one,12:25
chhavi__I want the existing one to go first, on top of that I will add LPM and get_iscsi_initiator changes12:25
chhavi__LPM is currently WIP, i am testing it12:25
chhavi__sudo apt install pvm-msp12:26
chhavi__do i need to run this to setup msp on both source and destination novalink12:26
*** efried is now known as fried_rice12:27
edmondswI'm not really familiar with MSP12:27
fried_riceyeah, that's a question for the nvcastets and seryoyers of the world.12:28
edmondswtry the #novalink slack channel12:28
edmondswI'll take another look at
chhavi__sure thanks12:30
fried_riceedmondsw: pypowervm 1.1.12 appears to be in u-c.  Shall we propose the g-r bump?12:31
edmondswfried_rice are we going to need 1.1.13?12:32
fried_riceI don't think so.  I think the 12->13 changes were tweaks for customer, not community.12:32
fried_riceunless we want arunman's stuff soonish.12:33
fried_riceI guess we could wait for that.  We're not in a hurry to get the volume ID tagging support?12:33
edmondswwe're definitely going to have to bump again for arunman12:36
edmondswI'm just sensitive to asking for a new g-r bump every week12:36
fried_riceedmondsw: but whenever we do propose it, whoever proposes it, don't forget there's now lower-constraints.12:36
fried_riceMeh, it's no big deal.  But I'm fine waiting for 1.1.1312:36
arunmanfried_rice, edmondsw : I've addressed the review comments and it's up for your review!13:03
arunmanwill update the nova-powervm changes for review after pypowervm is done13:04
fried_ricearunman: Reviewed.  One small tweak and I'm good.13:07
arunmanfried_rice: Okay.. fixed13:12
edmondswfried_rice u looking at now?13:12
fried_riceedmondsw: I wasn't.  Should I be?13:13
edmondswyes please13:13
edmondswwhile chhavi__ is still online if we want a change13:13
fried_riceedmondsw: done.  Let's let her look at the suggestions and respin if she wants before we +W it.13:23
edmondswtx... chhavi__ ?13:23
chhavi__edmondsw: let me have a look at the comments13:24
fried_riceoh, I misread 'online' as 'offline' :)13:24
*** arunman has quit IRC13:24
chhavi__edmondsw: separate test cases, I am not sure, I tried putting on one, but somehow reset_mock not setting the call_count. so I thought to split13:26
edmondswchhavi__ that's weird... but I won't make a big deal of that. I would change the commit message and add the LOG statement fried_rice commented about13:27
chhavi__sure let me update the Eric comments and re-submit13:28
*** tjakobs has joined #openstack-powervm13:28
*** esberglu has joined #openstack-powervm13:41
edmondswesberglu we good to merge 6418 and recheck the CI on the IT snapshot change?13:42
esbergluedmondsw: Yep was just about to13:43
openstackgerritChhavi Agarwal proposed openstack/nova-powervm master: Use vios_uuids to process required vioses for iSCSI
fried_rice+2.  edmondsw it's all you.13:57
edmondswchecking one thing...14:01
edmondswwhere does self.volume_id get set? I'm not seeing it...14:04
tjakobsedmondsw: the driver i think14:05
edmondswnm, found it14:05
esbergluedmondsw: fried_rice: Are there any other actions we need to take for PowerVMStackers?14:11
esbergluIf either of you looked into it, otherwise I can14:11
edmondswlike what?14:11
edmondswI don't think so14:11
esbergluIdk, thought there might be docs to update or lists to add it to14:12
edmondswfried_rice and the corresponding backport both passed CI... can one of you merge those?14:12
esbergluedmondsw: Done14:13
edmondswesberglu nothing I've seen14:13
edmondswesberglu also see if I've addressed your comment on
esbergluedmondsw: +2 W+114:16
openstackgerritMerged openstack/nova-powervm master: Pass user context to virt driver when detaching volume
openstackgerritMerged openstack/nova-powervm stable/queens: Pass user context to virt driver when detaching volume
openstackgerritMerged openstack/networking-powervm stable/queens: tox needs to pull from stable/queens
*** arunman has joined #openstack-powervm14:30
edmondswmdrabe ^ queens should be working again14:30
arunmanfried_rice: is there going to be a 1.1.13 version of pypowervm out today?14:44
-openstackstatus- NOTICE: zuul.o.o has been restarted to pick up latest code base and clear memory usage. Both check / gate queues were saved, be sure to check your patches and recheck when needed.14:52
esbergluedmondsw: fried_rice: Snapshot change passed CI15:04
esbergluJust needs reviews from you and then it's ready for cores15:05
fried_ricearunman: We're discussing whether we should release it today or wait a little bit.15:05
fried_riceesberglu: In the middle of that one right now.15:06
fried_riceIt's a biggie.  But not sure how it could be split.15:06
esberglufried_rice: Yeah no good way since it's all contained within the snapshot flow15:08
*** arunman has quit IRC15:21
chhavi__edmondsw,tjakobs: If we need to differentiate between the Traditional and LinuxVIOS, do checking the operating_system is the right approach or we have any other propety.15:22
fried_riceesberglu: Just one thing in snapshot patch that's worth changing.  You can hit the other ones if you want, or not.15:28
esberglufried_rice: tx, ack15:31
-openstackstatus- NOTICE: Gerrit will be temporarily unreachable as we restart it to complete the rename of some projects.15:49
openstackgerritMerged openstack/nova-powervm master: Use vios_uuids to process required vioses for iSCSI
*** AlexeyAbashkin has quit IRC16:14
*** fried_rice is now known as fried_rolls16:35
*** k0da has quit IRC16:38
*** k0da has joined #openstack-powervm17:45
*** k0da has quit IRC17:52
*** AlexeyAbashkin has joined #openstack-powervm17:53
*** AlexeyAbashkin has quit IRC17:57
*** AlexeyAbashkin has joined #openstack-powervm17:58
*** AlexeyAbashkin has quit IRC18:03
*** k0da has joined #openstack-powervm18:22
*** chhavi__ has quit IRC18:33
esbergluedmondsw: I'm messing around with the hotplug tempest tests, but I can't even get an instance with an active RMC connection to spawn on a CI AIO instance18:57
edmondswissue with the image we're using?18:57
fried_rollsesberglu: You gonna push a new rev of snapshot quickly?18:58
fried_rollswhoops, sorry to butt in.18:58
esbergluedmondsw: Well the default image we use for CI is a blank image, but I'm trying to spawn with the Ubuntu 16.04 2G image19:00
edmondswesberglu I've never had to try to get RMC working before, so I'm probably not the best person to ask :)19:01
edmondswbut I think there's more to it19:01
edmondswsomething you have to install on the guest OS?19:01
esberglufried_rolls: I put up a new rev this morning19:01
esbergluJust forgot to ping you19:02
fried_rollshum, I missed... oh, duh, I was looking for the bot note in this channel.  It's a nova patch.  Looking now...19:02
*** k0da has quit IRC19:02
fried_rollsedmondsw: wanna give a nod so we can queue it up for a runway?19:04
edmondswesberglu check out
edmondswfried_rolls yeah, I'm trying to get to that this afternoon... juggling...19:05
*** k0da has joined #openstack-powervm19:06
edmondswesberglu it says "built into AIX and available for some Enterprise Linux offerings", so not built in on Linux?19:06
tjakobsesberglu try installing "dynamicrm"19:06
edmondswbut that was the only reference to Linux on the page, so...19:06
edmondswah tjakobs, tx19:06
edmondswtjakobs any idea who we could complain to that Linux and dynamicrm aren't covered in that topic and get it updated?19:07
tjakobsedmondsw not a clue19:08
edmondswesberglu this lists more than just dynamicrm that you might need, unless those are all pulled in as deps:
esbergluOn the undercloud I can just spawn using the 2G image and RMC will become active so I don't think it's the image19:10
edmondswmdrabe might be able to help19:11
*** AlexeyAbashkin has joined #openstack-powervm19:14
*** AlexeyAbashkin has quit IRC19:19
*** k0da has quit IRC19:31
*** openstackgerrit has quit IRC19:34
*** fried_rolls is now known as fried_rice19:40
*** k0da has joined #openstack-powervm19:44
esbergluedmondsw: fried_rice: Resize is working IT for the most part. Couple of things wrong that I left comments on and UT isn't ready19:59
fried_riceesberglu: Just watching for WIP to come off before I start reviewing :)19:59
edmondswand CI isn't ready, right?19:59
esbergluBut feel free to start looking at the non-UT code whenever you have time19:59
edmondswthat will require multi-node CI19:59
esbergluedmondsw: Gonna do the same thing for resize that we did for snapshot20:00
esbergluThe first wave of resize will not need multinode20:00
esbergluIt is resize to the same host20:00
edmondswI didn't think the community did that20:00
edmondswlooks like there is a conf setting: allow_resize_to_same_host=True20:01
edmondswthat is just allow, though... doesn't *make* it go to the same host20:02
edmondswI guess if there was only 1 host, it will make it :)20:02
*** AlexeyAbashkin has joined #openstack-powervm20:14
*** AlexeyAbashkin has quit IRC20:18
esbergluedmondsw: fried_rice: Is there a reason for calling these directly instead of setting up a flow?20:31
edmondswesberglu that seems like a bug20:33
edmondswif something went wrong during update, no revert to power back on...20:33
esbergluedmondsw: Yeah seems that way20:35
esbergluWon't get to it today, but I can throw something together next week20:35
esbergluNot the only spot either20:35
edmondswgood catch20:37
fried_riceesberglu: I think perhaps it's because cleanup is the responsibility of some other method in a migration flow.21:09
fried_ricebut I could be wrong.21:11
edmondswthat could be21:13
edmondswif so, should add a comment to that effect21:14
*** edmondsw has quit IRC21:17
*** openstackgerrit has joined #openstack-powervm21:23
openstackgerritEric Berglund proposed openstack/nova-powervm master: DNM: ci check
openstackgerritEric Berglund proposed openstack/nova-powervm master: DNM: CI Check2
openstackgerritDoug Hellmann proposed openstack/networking-powervm master: add lower-constraints job
*** esberglu has quit IRC21:28
*** tjakobs has quit IRC22:09
*** k0da has quit IRC22:11
*** k0da has joined #openstack-powervm22:26

Generated by 2.15.3 by Marius Gedminas - find it at!