Tuesday, 2018-10-09

*** mujahidali has joined #openstack-powervm13:29
edmondsw#startmeeting PowerVM Driver Meeting14:01
openstackMeeting started Tue Oct  9 14:01:06 2018 UTC and is due to finish in 60 minutes.  The chair is edmondsw. Information about MeetBot at http://wiki.debian.org/MeetBot.14:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:01
*** openstack changes topic to " (Meeting topic: PowerVM Driver Meeting)"14:01
openstackThe meeting name has been set to 'powervm_driver_meeting'14:01
efriedō/14:01
edmondsw#link agenda https://etherpad.openstack.org/p/powervm_driver_meeting_agenda14:01
edmondswmujahidali: mdrabe: ping14:01
edmondsw#topic In-Tree Driver14:02
*** openstack changes topic to "In-Tree Driver (Meeting topic: PowerVM Driver Meeting)"14:02
edmondswI don't believe there's been any work going on here for a while14:02
edmondswI'm hoping that mujahidali will be able to start working on more porting from OOT to IT eventually14:03
edmondswbut the next stuff up requires multi-node CI, so he's gotta get that working first14:03
edmondswcomments from anyone else?14:03
efriednay14:03
edmondsw#topic Out-of-Tree Driver14:04
*** openstack changes topic to "Out-of-Tree Driver (Meeting topic: PowerVM Driver Meeting)"14:04
edmondswWe have opted-in for the community to help us upgrade to newer versions of neutron-lib as the release14:05
edmondswvia https://review.openstack.org/#/c/607625/14:05
efried\o/14:05
edmondswwe also fixed a bug https://review.openstack.org/#/c/608023/14:05
edmondswI think that's it since the last meeting14:06
efriedoh, does -^ need to be ported IT?14:06
efried(I didn't look)14:06
edmondswefried ah, that's a good point... I thought about backporting (don't think that's needed at this point) but not IT14:07
edmondswI'll check after the meeting if nobody beats me to it14:07
efried14:08
edmondswmdrabe: how is your schedule looking for working on MSP?14:08
edmondswI've been tied up in other things, so I might need you to take over getting the multi-node devstack env working14:09
edmondswbut I can help advise you there at least14:09
edmondswalright, moving on14:11
edmondsw#topic Device Passthrough14:11
*** openstack changes topic to "Device Passthrough (Meeting topic: PowerVM Driver Meeting)"14:11
edmondswefried ^14:11
efriedJay has agreed (again) to review kosamara's spec.14:11
efriedBut we have a suggestion from Sylvain to break it apart into one spec for the file format and one for the passthrough itself.14:12
efriedbecause the file format has the potential to gain traction for use in a lot of areas other than just device passthrough.14:12
efriedBasically as a generic provider configurator14:12
edmondswinteresting14:12
efriedwhich is getting interest from several high-priority blueprints right now14:12
efriedSo if we can land something around that file spec in nova proper, we could implement passthrough fairly easily on top of it - whether IT or OOT.14:13
efriedPlacement extraction is going well. Upgrade stuff is getting close to working.14:14
efriedReshaping & nrp work is moving along14:14
efriedwe keep hitting new and interesting issues (e.g. see gibi's ML post this morning)14:14
efriedI've been keeping abreast and weighing in, but not doing a lot of actual writing of spec/code content due to time split.14:15
efriedI think that's about it, unless someone wants more details on anything.14:16
edmondswthanks14:16
edmondsw#topic PowerVM CI14:16
*** openstack changes topic to "PowerVM CI (Meeting topic: PowerVM Driver Meeting)"14:16
edmondswmujahidali how is the CI?14:16
mujahidaliAfter redeployment followed by Power outage, CI was broken and  we added "AttachInterfacesUnderV243Test.test_add_remove_fixed_ip " to blacklist.14:16
mujahidaliCI looks good now.14:17
edmondswyep, I noticed we were getting some good votes. Tintri looks pretty clean14:17
edmondswthanks for fixing that up14:17
mujahidaliI have installed diskimage-builder and updated nodepool to 3.2.0 on stage env.14:18
mujahidaliI am currently trying to figure out the nodepool configuration to use the diskimage-builder. I am using this https://github.com/openstack-infra/project-config-example/blob/master/nodepool/nodepool.yaml as refernce.14:18
mujahidaliI need some more tweaking with the nodepool.yaml file and then we may get a wroking nodepool config.14:19
edmondswthat'll be awesome14:20
mujahidali:)14:20
mujahidaliOnce nodepool is working with disimage-builder we need to figure out the SSP part for multinode.14:21
edmondswI can help you create an SSP that spans 2 nodes. I've done that before14:22
mujahidalithat will be a great help.14:22
mujahidalithanks14:22
edmondswI'll leave it to you to figure out how to configure the CI to know which 2 nodes are paired14:22
mujahidaliokay14:22
mujahidalithat's it from me.14:24
edmondswmujahidali I'd suggest that we move to the new nodepool and diskimage-builder and all that first, and then worry about making the CI multinode in a later commit(s)14:24
edmondswi.e. try to stage the work out14:24
edmondswsound good?14:24
mujahidalisure.14:24
mujahidaliso, let's break it out in two parts14:25
edmondswyou didn't mention zookeeper. Have you been working with that as well?14:25
mujahidaliupgrade CI14:25
mujahidaliand then enable multinode14:25
edmondswyes14:25
edmondswalright, tx14:26
edmondsw#topic Open Discussion14:26
*** openstack changes topic to "Open Discussion (Meeting topic: PowerVM Driver Meeting)"14:26
edmondswefried the community has a "reason" arg for Unplug exceptions but not for plug: https://github.com/openstack/nova/blob/master/nova/exception.py#L132-L13714:27
efriednice14:28
edmondswso while OOT we customize the Plug exception to give reason information (at the expense of saying it's a plug... which maybe we should change)14:28
edmondswIT we just stick with the generic message: https://github.com/openstack/nova/blob/master/nova/virt/powervm/vif.py#L116-L11814:28
edmondswI guess that's ok for IT, until/unless they add reason to the exception14:29
efriedtbh if we're using LOG.exception I'm not sure what benefit we would get by shoving a %reason in there14:29
edmondswbut I'll probably propose a follow-on OOT to add a "Virtual interface plugin failed: " prefix to the message we create there14:30
edmondswefried yeah, I think the benefit would only be if the message gets used in the REST response14:30
efriedwhich it doesn't.14:31
efriedBecause the REST operation is asynchronous.14:31
efriedYou notice the instance go to ERROR state.14:31
efriedand then you go look at logs.14:31
edmondswhmm... then why did Arun care about this, I wonder14:31
efriedoh, because it would have been raising a spurious exception in the logs, obscuring the actual reason for the error.14:31
efriedOh, I was talking about spawn - there may be a synchronous plug operation that does surface the error.14:32
efriedidk for sure14:32
edmondswI'll try to catch Arun and ask him for more detail14:32
efriedeither way it's a correct fix14:32
edmondswyeah14:32
edmondswbetter than it was, for sure14:32
edmondswif that's it, I'll let y'all go14:33
edmondsw#endmeeting14:33
*** openstack changes topic to "This channel is for PowerVM-related development and discussion. For general OpenStack support, please use #openstack."14:33
openstackMeeting ended Tue Oct  9 14:33:26 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:33
openstackMinutes:        http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-10-09-14.01.html14:33
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-10-09-14.01.txt14:33
openstackLog:            http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-10-09-14.01.log.html14:33
*** mujahidali has quit IRC15:16
edmondswefried I spoke to Arun. He said that plug error message won't show up in the REST response, but it will be part of the notification, so the controller node's logs can have the message as well as the compute node.16:13
efriedokay16:14
*** chhagarw has joined #openstack-powervm16:17
openstackgerritElod Illes proposed openstack/ceilometer-powervm master: Install ceilometer from pypi  https://review.openstack.org/60905817:17
*** chhagarw has quit IRC18:28
*** chhagarw has joined #openstack-powervm18:38
*** chhagarw has quit IRC19:17
*** efried has quit IRC23:12
*** efried has joined #openstack-powervm23:13

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