13:05:45 #startmeeting sriov 13:05:46 Meeting started Tue Jul 26 13:05:45 2016 UTC and is due to finish in 60 minutes. The chair is moshele. Information about MeetBot at http://wiki.debian.org/MeetBot. 13:05:47 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 13:05:49 The meeting name has been set to 'sriov' 13:05:58 hi everyone 13:06:03 o/ 13:06:04 hi 13:06:05 hello 13:06:18 #topic Improving SR-IOV/PCI Passthrough CI 13:06:27 hi 13:06:52 so we have a few patches with no tempest core review :( 13:07:14 I will try to ping them in infra today 13:07:32 we should ping mriedem about it, hopefully he has some contacts there 13:07:36 I was trying to catch gmann or jordanP, but got no answer from them 13:08:24 mellanox multinode ci comment on the pci allocation patch 13:08:26 http://13.69.151.247/83/328983/21/check-mlnx-multinode/Nova-ML2-Sriov-Multinode/fca880b/ 13:08:50 and the migration test passed which is cool 13:09:13 http://13.69.151.247/83/328983/21/check-mlnx-multinode/Nova-ML2-Sriov-Multinode/fca880b/testr_results.html.gz 13:09:47 #topic bug fixes 13:10:09 so jaypipes review the migration revert and the pci allocation 13:10:27 working on addressing the comments 13:11:02 lbeliveau: so you are splitting the migration revert patch to 2 patches? 13:11:18 do you need help with some of the work? 13:11:45 moshele: yes that's the plan, but working on moving the unit test for pci allocation (which is painful for some reason) 13:12:01 moshele: will ping you if I need helpm thanks 13:12:20 ok cool 13:12:56 anything else on bug fixes? 13:13:37 not from me 13:13:45 #topic specs for review 13:14:29 this is https://review.openstack.org/#/c/342001/ the Support SR-IOV and PCI statistics through Nova API 13:14:43 we should review it again 13:14:45 need to review the latest version ... 13:14:53 yes 13:15:09 we can use it for the future to improve migration testing 13:16:06 anyone have any other spec/issue to 13:16:09 an update on the ironic spec related to setting kernel args to grub - https://review.openstack.org/#/c/331564/ 13:16:36 this proposal has been turned down as ironic should not change the image files.. 13:16:48 we are looking at other options to set the kernel args.. 13:16:57 like using the user_data from tripleo.. 13:17:12 this is related to SR-IOV tripleo spec. 13:17:26 https://review.openstack.org/#/c/313872/ 13:17:36 lbeliveau: just lemme know when you push the split patches and I will review ASAP. 13:18:16 jaypipes: will do, struggling to move the pci allocation unit test atm 13:18:24 jaypipes: thank :) 13:18:53 lbeliveau: if you want I can do the split 13:19:07 lbeliveau: there is already an existing one in test_tracker.py I believe. You should just be able to copy/paste a new specialized one or adapt the existing one for your needs. 13:19:26 moshele: will ping you after the meeting 13:19:42 lbeliveau: ok 13:19:59 skramaja: what about the vendor_id/product_id port info for ironic 13:20:11 jaypipes: yes, I clone the pci test you did and trying to adapt it to migration 13:20:26 skramaja: do you have a spec for this ? 13:20:47 lbeliveau: gotcha. well, if you're stuck, feel free to just push up what you have and I can take a looksie. 13:21:00 i guess it is part of your review.. right? 13:21:26 jaypipes: awesome, thanks ! mock is acting weird on me with this ut 13:21:31 which spec? for triploe spec or ironic spec? 13:21:42 lbeliveau: yeah, understood. it's complex to be sure. 13:22:21 skramaja: I mean this rfe https://bugs.launchpad.net/ironic-python-agent/+bug/1594196 13:22:21 Launchpad bug 1594201 in ironic-python-agent "duplicate for #1594196 [RFE] Update Port's extra field with vendor data" [Wishlist,Incomplete] - Assigned to yogananth subramanian (anantyog-o) 13:23:16 we are not writing spec.. since it was merged into ur review, we dont need to.. 13:23:37 *we thought, we dont need to do 13:24:18 skramaja: ok I will ping you later about this 13:24:24 ok.. sure.. 13:24:29 thanks moshele 13:24:34 anything else regarding SR-IOV spec 13:24:55 nothing from us as of now.. 13:25:08 #endmeeting