| *** k0da has quit IRC | 00:31 | |
| *** edmondsw has quit IRC | 00:46 | |
| *** esberglu has joined #openstack-powervm | 01:03 | |
| *** esberglu has quit IRC | 01:08 | |
| *** k0da has joined #openstack-powervm | 01:20 | |
| *** k0da has quit IRC | 02:13 | |
| *** edmondsw has joined #openstack-powervm | 02:34 | |
| *** edmondsw has quit IRC | 02:39 | |
| *** k0da has joined #openstack-powervm | 03:11 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 03:15 | |
| *** k0da has quit IRC | 03:18 | |
| *** AlexeyAbashkin has quit IRC | 03:20 | |
| *** arunman has joined #openstack-powervm | 03:52 | |
| *** arunman has quit IRC | 03:55 | |
| *** arunman has joined #openstack-powervm | 04:00 | |
| *** arunman has quit IRC | 04:01 | |
| *** edmondsw has joined #openstack-powervm | 04:23 | |
| *** edmondsw has quit IRC | 04:27 | |
| *** chhagarw has joined #openstack-powervm | 04:47 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 05:15 | |
| *** AlexeyAbashkin has quit IRC | 05:19 | |
| *** arunman has joined #openstack-powervm | 05:39 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 06:17 | |
| *** AlexeyAbashkin has quit IRC | 06:22 | |
| *** edmondsw has joined #openstack-powervm | 07:59 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 08:02 | |
| *** edmondsw has quit IRC | 08:03 | |
| *** k0da has joined #openstack-powervm | 08:30 | |
| *** edmondsw has joined #openstack-powervm | 09:47 | |
| *** edmondsw has quit IRC | 09:51 | |
| *** chhavi__ has joined #openstack-powervm | 10:15 | |
| *** chhagarw has quit IRC | 10:16 | |
| *** k0da has quit IRC | 10:31 | |
| *** k0da has joined #openstack-powervm | 11:47 | |
| *** edmondsw has joined #openstack-powervm | 12:03 | |
| chhavi__ | edmondsw: GM | 12:22 |
|---|---|---|
| edmondsw | chhavi__ hi | 12:22 |
| chhavi__ | edmondsw: couple of things, changes are up for review, I am adding the methods to support LPM | 12: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 well | 12:24 |
| edmondsw | chhavi__ 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 changes | 12:25 |
| chhavi__ | LPM is currently WIP, i am testing it | 12:25 |
| chhavi__ | sudo apt install pvm-msp | 12:26 |
| chhavi__ | do i need to run this to setup msp on both source and destination novalink | 12:26 |
| *** efried is now known as fried_rice | 12:27 | |
| edmondsw | I'm not really familiar with MSP | 12:27 |
| fried_rice | yeah, that's a question for the nvcastets and seryoyers of the world. | 12:28 |
| edmondsw | try the #novalink slack channel | 12:28 |
| edmondsw | I'll take another look at https://review.openstack.org/#/c/555003/ | 12:28 |
| chhavi__ | sure thanks | 12:30 |
| fried_rice | edmondsw: pypowervm 1.1.12 appears to be in u-c. Shall we propose the g-r bump? | 12:31 |
| edmondsw | fried_rice are we going to need 1.1.13? | 12:32 |
| fried_rice | I don't think so. I think the 12->13 changes were tweaks for customer, not community. | 12:32 |
| fried_rice | unless we want arunman's stuff soonish. | 12:33 |
| fried_rice | I guess we could wait for that. We're not in a hurry to get the volume ID tagging support? | 12:33 |
| edmondsw | we're definitely going to have to bump again for arunman | 12:36 |
| edmondsw | I'm just sensitive to asking for a new g-r bump every week | 12:36 |
| fried_rice | edmondsw: but whenever we do propose it, whoever proposes it, don't forget there's now lower-constraints. | 12:36 |
| fried_rice | Meh, it's no big deal. But I'm fine waiting for 1.1.13 | 12:36 |
| arunman | fried_rice, edmondsw : I've addressed the review comments and it's up for your review! | 13:03 |
| arunman | will update the nova-powervm changes for review after pypowervm is done | 13:04 |
| edmondsw | ack | 13:06 |
| fried_rice | arunman: Reviewed. One small tweak and I'm good. | 13:07 |
| arunman | fried_rice: Okay.. fixed | 13:12 |
| edmondsw | fried_rice u looking at https://review.openstack.org/#/c/555003 now? | 13:12 |
| fried_rice | edmondsw: I wasn't. Should I be? | 13:13 |
| edmondsw | yes please | 13:13 |
| fried_rice | ack | 13:13 |
| edmondsw | while chhavi__ is still online if we want a change | 13:13 |
| fried_rice | edmondsw: done. Let's let her look at the suggestions and respin if she wants before we +W it. | 13:23 |
| edmondsw | tx... chhavi__ ? | 13:23 |
| chhavi__ | edmondsw: let me have a look at the comments | 13:24 |
| fried_rice | oh, I misread 'online' as 'offline' :) | 13:24 |
| *** arunman has quit IRC | 13: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 split | 13:26 |
| edmondsw | chhavi__ 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 about | 13:27 |
| chhavi__ | sure let me update the Eric comments and re-submit | 13:28 |
| *** tjakobs has joined #openstack-powervm | 13:28 | |
| *** esberglu has joined #openstack-powervm | 13:41 | |
| edmondsw | esberglu we good to merge 6418 and recheck the CI on the IT snapshot change? | 13:42 |
| esberglu | edmondsw: Yep was just about to | 13:43 |
| openstackgerrit | Chhavi Agarwal proposed openstack/nova-powervm master: Use vios_uuids to process required vioses for iSCSI https://review.openstack.org/555003 | 13:56 |
| fried_rice | +2. edmondsw it's all you. | 13:57 |
| edmondsw | checking one thing... | 14:01 |
| edmondsw | where does self.volume_id get set? I'm not seeing it... | 14:04 |
| tjakobs | edmondsw: the driver i think | 14:05 |
| edmondsw | nm, found it | 14:05 |
| esberglu | edmondsw: fried_rice: Are there any other actions we need to take for PowerVMStackers? | 14:11 |
| esberglu | If either of you looked into it, otherwise I can | 14:11 |
| edmondsw | like what? | 14:11 |
| edmondsw | I don't think so | 14:11 |
| esberglu | Idk, thought there might be docs to update or lists to add it to | 14:12 |
| edmondsw | fried_rice https://review.openstack.org/#/c/555038/ and the corresponding backport both passed CI... can one of you merge those? | 14:12 |
| esberglu | edmondsw: Done | 14:13 |
| edmondsw | esberglu nothing I've seen | 14:13 |
| edmondsw | esberglu also see if I've addressed your comment on https://review.openstack.org/#/c/555074/ | 14:13 |
| edmondsw | tx | 14:13 |
| esberglu | edmondsw: +2 W+1 | 14:16 |
| edmondsw | tx | 14:16 |
| openstackgerrit | Merged openstack/nova-powervm master: Pass user context to virt driver when detaching volume https://review.openstack.org/555038 | 14:22 |
| openstackgerrit | Merged openstack/nova-powervm stable/queens: Pass user context to virt driver when detaching volume https://review.openstack.org/555042 | 14:27 |
| openstackgerrit | Merged openstack/networking-powervm stable/queens: tox needs to pull from stable/queens https://review.openstack.org/555074 | 14:28 |
| *** arunman has joined #openstack-powervm | 14:30 | |
| edmondsw | mdrabe ^ queens should be working again | 14:30 |
| arunman | fried_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 | |
| esberglu | edmondsw: fried_rice: Snapshot change passed CI | 15:04 |
| esberglu | Just needs reviews from you and then it's ready for cores | 15:05 |
| fried_rice | arunman: We're discussing whether we should release it today or wait a little bit. | 15:05 |
| fried_rice | esberglu: In the middle of that one right now. | 15:06 |
| esberglu | cool | 15:06 |
| fried_rice | It's a biggie. But not sure how it could be split. | 15:06 |
| esberglu | fried_rice: Yeah no good way since it's all contained within the snapshot flow | 15:08 |
| *** arunman has quit IRC | 15: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_rice | esberglu: Just one thing in snapshot patch that's worth changing. You can hit the other ones if you want, or not. | 15:28 |
| esberglu | fried_rice: tx, ack | 15:31 |
| -openstackstatus- NOTICE: Gerrit will be temporarily unreachable as we restart it to complete the rename of some projects. | 15:49 | |
| openstackgerrit | Merged openstack/nova-powervm master: Use vios_uuids to process required vioses for iSCSI https://review.openstack.org/555003 | 15:50 |
| *** AlexeyAbashkin has quit IRC | 16:14 | |
| *** fried_rice is now known as fried_rolls | 16:35 | |
| *** k0da has quit IRC | 16:38 | |
| *** k0da has joined #openstack-powervm | 17:45 | |
| *** k0da has quit IRC | 17:52 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 17:53 | |
| *** AlexeyAbashkin has quit IRC | 17:57 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 17:58 | |
| *** AlexeyAbashkin has quit IRC | 18:03 | |
| *** k0da has joined #openstack-powervm | 18:22 | |
| *** chhavi__ has quit IRC | 18:33 | |
| esberglu | edmondsw: 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 instance | 18:57 |
| edmondsw | issue with the image we're using? | 18:57 |
| fried_rolls | esberglu: You gonna push a new rev of snapshot quickly? | 18:58 |
| fried_rolls | whoops, sorry to butt in. | 18:58 |
| esberglu | edmondsw: Well the default image we use for CI is a blank image, but I'm trying to spawn with the Ubuntu 16.04 2G image | 19:00 |
| edmondsw | esberglu I've never had to try to get RMC working before, so I'm probably not the best person to ask :) | 19:01 |
| edmondsw | but I think there's more to it | 19:01 |
| edmondsw | something you have to install on the guest OS? | 19:01 |
| esberglu | fried_rolls: I put up a new rev this morning | 19:01 |
| esberglu | Just forgot to ping you | 19:02 |
| fried_rolls | hum, 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 IRC | 19:02 | |
| fried_rolls | edmondsw: wanna give https://review.openstack.org/#/c/543023/ a nod so we can queue it up for a runway? | 19:04 |
| edmondsw | esberglu check out http://www-01.ibm.com/support/docview.wss?uid=isg3T1020611 | 19:04 |
| edmondsw | fried_rolls yeah, I'm trying to get to that this afternoon... juggling... | 19:05 |
| fried_rolls | nod | 19:05 |
| *** k0da has joined #openstack-powervm | 19:06 | |
| edmondsw | esberglu it says "built into AIX and available for some Enterprise Linux offerings", so not built in on Linux? | 19:06 |
| tjakobs | esberglu try installing "dynamicrm" | 19:06 |
| edmondsw | but that was the only reference to Linux on the page, so... | 19:06 |
| edmondsw | ah tjakobs, tx | 19:06 |
| edmondsw | tjakobs any idea who we could complain to that Linux and dynamicrm aren't covered in that topic and get it updated? | 19:07 |
| tjakobs | edmondsw not a clue | 19:08 |
| edmondsw | esberglu this lists more than just dynamicrm that you might need, unless those are all pulled in as deps: https://www.ibm.com/support/knowledgecenter/en/SSVSPA_1.3.3/com.ibm.powervc.cloud.help.doc/powervc_configuring_vlans_cloud.html | 19:10 |
| esberglu | On the undercloud I can just spawn using the 2G image and RMC will become active so I don't think it's the image | 19:10 |
| edmondsw | mdrabe might be able to help | 19:11 |
| *** AlexeyAbashkin has joined #openstack-powervm | 19:14 | |
| *** AlexeyAbashkin has quit IRC | 19:19 | |
| *** k0da has quit IRC | 19:31 | |
| *** openstackgerrit has quit IRC | 19:34 | |
| *** fried_rolls is now known as fried_rice | 19:40 | |
| *** k0da has joined #openstack-powervm | 19:44 | |
| esberglu | edmondsw: fried_rice: Resize is working IT for the most part. Couple of things wrong that I left comments on and UT isn't ready | 19:59 |
| fried_rice | esberglu: Just watching for WIP to come off before I start reviewing :) | 19:59 |
| edmondsw | and CI isn't ready, right? | 19:59 |
| esberglu | But feel free to start looking at the non-UT code whenever you have time | 19:59 |
| edmondsw | that will require multi-node CI | 19:59 |
| esberglu | edmondsw: Gonna do the same thing for resize that we did for snapshot | 20:00 |
| esberglu | The first wave of resize will not need multinode | 20:00 |
| esberglu | It is resize to the same host | 20:00 |
| edmondsw | oh? | 20:00 |
| edmondsw | I didn't think the community did that | 20:00 |
| edmondsw | looks like there is a conf setting: allow_resize_to_same_host=True | 20:01 |
| edmondsw | that is just allow, though... doesn't *make* it go to the same host | 20:02 |
| edmondsw | I guess if there was only 1 host, it will make it :) | 20:02 |
| *** AlexeyAbashkin has joined #openstack-powervm | 20:14 | |
| *** AlexeyAbashkin has quit IRC | 20:18 | |
| esberglu | edmondsw: fried_rice: Is there a reason for calling these directly instead of setting up a flow? | 20:31 |
| esberglu | https://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/driver.py#L1362-L1363 | 20:31 |
| edmondsw | esberglu that seems like a bug | 20:33 |
| edmondsw | if something went wrong during update, no revert to power back on... | 20:33 |
| esberglu | edmondsw: Yeah seems that way | 20:35 |
| esberglu | Won't get to it today, but I can throw something together next week | 20:35 |
| esberglu | Not the only spot either | 20:35 |
| edmondsw | good catch | 20:37 |
| fried_rice | esberglu: I think perhaps it's because cleanup is the responsibility of some other method in a migration flow. | 21:09 |
| fried_rice | but I could be wrong. | 21:11 |
| edmondsw | that could be | 21:13 |
| edmondsw | if so, should add a comment to that effect | 21:14 |
| *** edmondsw has quit IRC | 21:17 | |
| *** openstackgerrit has joined #openstack-powervm | 21:23 | |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: DNM: ci check https://review.openstack.org/328315 | 21:23 |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: DNM: CI Check2 https://review.openstack.org/328317 | 21:23 |
| openstackgerrit | Doug Hellmann proposed openstack/networking-powervm master: add lower-constraints job https://review.openstack.org/555936 | 21:26 |
| *** esberglu has quit IRC | 21:28 | |
| *** tjakobs has quit IRC | 22:09 | |
| *** k0da has quit IRC | 22:11 | |
| *** k0da has joined #openstack-powervm | 22:26 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!