Friday, 2018-07-20

*** mujahidali has joined #openstack-powervm05:30
*** k0da has joined #openstack-powervm05:45
*** AlexeyAbashkin has joined #openstack-powervm06:15
*** AlexeyAbashkin has quit IRC06:41
*** k0da has quit IRC06:43
*** AlexeyAbashkin has joined #openstack-powervm06:58
*** Alexey_Abashkin has joined #openstack-powervm07:01
*** AlexeyAbashkin has quit IRC07:02
*** Alexey_Abashkin is now known as AlexeyAbashkin07:02
*** k0da has joined #openstack-powervm07:50
*** mujahidali has quit IRC08:13
*** mujahidali has joined #openstack-powervm08:13
*** AlexeyAbashkin has quit IRC10:08
*** AlexeyAbashkin has joined #openstack-powervm10:11
*** AlexeyAbashkin has quit IRC10:18
*** AlexeyAbashkin has joined #openstack-powervm10:31
*** mujahidali has quit IRC11:32
*** AlexeyAbashkin has quit IRC12:31
*** AlexeyAbashkin has joined #openstack-powervm12:35
*** edmondsw has joined #openstack-powervm12:42
*** AlexeyAbashkin has quit IRC12:55
*** AlexeyAbashkin has joined #openstack-powervm12:56
*** esberglu has joined #openstack-powervm13:21
openstackgerritMatthew Edmonds proposed openstack/ceilometer-powervm master: add lower-constraints job  https://review.openstack.org/55535813:28
*** mujahidali has joined #openstack-powervm14:08
openstackgerritMatthew Edmonds proposed openstack/ceilometer-powervm master: add lower-constraints job  https://review.openstack.org/55535814:27
*** efried is now known as fried_rice14:32
openstackgerritEric Fried proposed openstack/networking-powervm master: Match neutron's version of hacking, flake8 ignores  https://review.openstack.org/58268615:51
*** k0da has quit IRC15:58
*** AlexeyAbashkin has quit IRC16:13
mdrabeefried: When I run tox in my nova-powervm workspace, can I tell it to point to a nova review?16:18
mdrabeThis is for https://review.openstack.org/#/c/579676/16:18
mdrabefried_rice*16:18
fried_ricemdrabe: For local testing, yes.16:19
mdrabeHow do I do that?16:19
fried_ricemdrabe: Do you have a directory .tox/py27/src/nova in your nova-powervm git repo?16:20
mdrabeyep16:20
mdrabeok mkay I see16:20
fried_riceokay, cd in there and git review -d whatever change number you want.16:20
mdrabethnx16:20
fried_riceyahyoubetcha.16:20
*** fried_rice is now known as fried_rolls16:24
openstackgerritMerged openstack/ceilometer-powervm master: add lower-constraints job  https://review.openstack.org/55535816:52
*** mujahidali has quit IRC17:22
*** esberglu has quit IRC18:10
*** esberglu has joined #openstack-powervm18:10
*** esberglu has quit IRC18:15
*** esberglu has joined #openstack-powervm18:22
*** esberglu has quit IRC18:24
*** esberglu has joined #openstack-powervm18:24
openstackgerritMatt Rabe proposed openstack/nova-powervm master: Add configuration option for specifying MSP IPs  https://review.openstack.org/58448118:30
*** fried_rolls is now known as fried_rice18:38
openstackgerritMerged openstack/networking-powervm master: Match neutron's version of hacking, flake8 ignores  https://review.openstack.org/58268618:59
fried_ricemdrabe: Help me understand how MSP works, wouldja?19:14
*** k0da has joined #openstack-powervm19:14
fried_ricemdrabe: The part that's confusing me is, "The MSP IPs on the destination host will be saved in the19:14
fried_ricemigration data where the source host will pass this parameter and19:14
fried_ricethe source MSP IPs to the migration job."19:14
fried_riceAre the MSP IPs actually *different* addresses on the source & destination hosts?19:14
fried_ricefor a single migration?19:14
mdrabefried_rice: Yes, they could be different SEAs or VEAs on the source/destination VIOSes19:15
fried_riceThat doesn't make sense to me. I thought the MSP was responsible for carrying data for the instance.19:15
mdrabeSo the source host has it's MSPs in conf and so does the destination19:15
fried_ricebutbutbut19:15
fried_ricethe IPs would be the same??19:15
mdrabeBetween source and destination? No19:15
mdrabeThe migration job has those parameters SourceMSPIPAddr and DestMSPIPAddr19:16
fried_ricemaybe we need to back all the way up and I need a crash course on how migration works.19:16
mdrabeThose parameters are IP addresses belong to adapters that will host the migration19:16
mdrabeSo if you have 10G and 1G adapters on your VIOSes, you can specify the 10G ones for migration via this parameter19:17
fried_riceso there's a source VIOS and a dest VIOS that have to talk to each other to effect the migration?19:18
mdrabeRight19:18
fried_riceokey.19:18
mdrabeThe source port has to be able to talk to the dest one19:18
*** edmondsw has quit IRC19:19
fried_ricemdrabe: Did we have the dest_msp_name and source_msp_name kwargs on the migrate_lpar method in pypowervm already?19:19
mdrabeYes19:19
*** edmondsw has joined #openstack-powervm19:20
fried_ricemdrabe: Thanks for the help, +2.19:22
mdrabefried_rice: I -WF'd it for test, also I'm pretty sure the nova commit has to merge first19:23
mdrabeAlso I don't know when I'll get around to getting two NL hosts and setting up devstack. If you or someone ends up doing that let me know. I'd like to piggyback19:24
fried_riceI have one that's stacked (though a couple months old) that you can borrow, but you'll have to find another.19:25
fried_riceedmondsw: ?19:25
edmondswI am trying to setup a multinode devstack19:26
fried_riceight19:26
edmondswgonna be afk for a bit, back later19:26
*** edmondsw has quit IRC19:26
*** k0da has quit IRC19:33
*** k0da has joined #openstack-powervm19:34
openstackgerritEric Fried proposed openstack/nova-powervm master: Spec: Device Passthrough  https://review.openstack.org/57935919:44
*** k0da has quit IRC19:59
*** esberglu has quit IRC20:19
*** edmondsw has joined #openstack-powervm20:20
edmondswI'm back20:21
*** k0da has joined #openstack-powervm20:22
*** esberglu has joined #openstack-powervm20:40
fried_riceWelcome back20:40
openstackgerritEric Fried proposed openstack/nova-powervm master: Inventory whitelist schema and loading  https://review.openstack.org/57928920:43
fried_riceedmondsw: Modified spec & patch for multi-feature_codes, lemme know if you're okay with the ORed list of ANDed values.20:44
*** esberglu has quit IRC20:44
edmondswfried_rice I like it... thanks20:48
fried_ricecoo20:48
edmondswfried_rice should I +2 or should we get that NVLink TODO resolved first?20:48
fried_riceedmondsw: As you wish. We can always amend later.20:49
edmondswyeah, that's what I'm thinking20:49
fried_riceedmondsw: We certainly have enough to get started.20:49
edmondswyep20:49
edmondsw+220:49
edmondswmdrabe ^ dive in yourself https://review.openstack.org/#/c/57935920:50
mdrabeon it20:51
mdrabefried_rice: What happens in update_provider_tree if a device gets removed or replaced?21:25
mdrabeIf the host went under maintenance for example21:26
fried_riceswhy we namespace the providers. We look for any provider with the namespace pattern, and if it doesn't correspond to one we discover from REST, we whack it.21:26
fried_ricemdrabe: Described in this section: http://logs.openstack.org/59/579359/9/check/build-openstack-sphinx-docs/b6e1cd3/html/specs/rocky/device-passthrough.html#resource-providers21:27
mdrabefried_rice: Ah okay, so the purging would happen in update_provider_tree?21:29
fried_ricemdrabe: Well, technically update_provider_tree just removes the provider from the ProviderTree object, and then we yield control back to the compute manager, where the resource tracker (via update_from_provider_tree) does the actual work to remove it from placement.21:30
fried_riceIt's an odd little dance.21:30
mdrabemakes sense, thnx21:32
mdrabefried_rice: Nothing would have to be done for resize?21:37
fried_ricemdrabe: Like what?21:47
mdrabefried_rice: The PowerVMDriver will have to update spawn to query placement, parse out the DRC index, match the corresponding slot in the MS wrapper, and update it in the LPAR wrapper21:57
openstackgerritEric Fried proposed openstack/nova-powervm master: Inventory whitelist schema and loading  https://review.openstack.org/57928921:57
mdrabeCold migration will make use of those same changes?21:57
fried_ricepresumably.21:57
openstackgerritEric Fried proposed openstack/nova-powervm master: Spec: Device Passthrough  https://review.openstack.org/57935921:59
fried_riceedmondsw, mdrabe: Updated to rip out feature codes ^^21:59
*** fried_rice is now known as efried22:09
efriedmdrabe: Is there anything to fix on that spec?22:10
mdrabeNope, not that I can find22:11
edmondswefried +2/+A on the spec22:24
edmondswmdrabe also see https://review.openstack.org/#/c/579289/22:24
openstackgerritMerged openstack/nova-powervm master: Spec: Device Passthrough  https://review.openstack.org/57935922:44
*** k0da has quit IRC23:02
*** k0da has joined #openstack-powervm23:39

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