Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-11-29
16:24:33 stephenfin We should document that in the code somewhere
16:24:48 sean-k-mooney it does not work that way anymore
16:24:57 openstackgerrit Vlad Gusev proposed openstack/nova stable/queens: Fix false ERROR message at compute restart https://review.opendev.org/696733
16:25:08 stephenfin wdym?
16:25:21 sean-k-mooney as i said the nova changes that added support for neutron vnic_type=direct-pyical
16:25:32 sean-k-mooney chanded how device_type works
16:25:45 sean-k-mooney and made it so vnic_type=direct wont select pf
16:25:57 sean-k-mooney and it might not be able to select type-PCI
16:26:38 sean-k-mooney before that change device_type was optional in the pci_request
16:27:00 stephenfin so we can't change this because it would mean you wouldn't be able to attach an interface using 'direct-physical' unless you'd enabled VFs on the host?
16:27:28 stephenfin to effectively work around that check
16:27:32 sean-k-mooney yes that would be one side effect
16:29:48 sean-k-mooney why what a really detailed and well writtne spec ... https://specs.openstack.org/openstack/nova-specs/specs/newton/implemented/sriov-pf-passthrough-neutron-port.html
16:32:31 stephenfin :D
16:32:56 stephenfin sean-k-mooney: lemme know if I said anything in https://review.opendev.org/#/c/695416/ that doesn't make sense
16:43:37 sean-k-mooney sure lets trade. does this make sense https://bugs.launchpad.net/nova/+bug/1853445/comments/5
16:43:37 openstack Launchpad bug 1853445 in OpenStack Compute (nova) "pci dev_type type-PCI passthrough failed" [Undecided,Invalid] - Assigned to Wei Hui (huiweics)
16:44:16 openstackgerrit Merged openstack/nova master: Force config drive in nova-next multinode job https://review.opendev.org/695431
16:44:22 openstackgerrit Merged openstack/nova master: Fix incorrect command examples https://review.opendev.org/695832
16:44:54 sean-k-mooney stephenfin: yes what you wrote makes sense
16:46:49 sean-k-mooney oh by the way i updated the functional test and have them working ill be pushing the update in the next 5 mins or so almost done
16:46:54 sean-k-mooney stephenfin: ^
16:47:22 sean-k-mooney the cpu_dedicate_set option worked perfectly to target the correct numa node in the tests
16:47:33 stephenfin perfect
16:47:41 stephenfin I'll review then
16:48:08 sean-k-mooney actully i did notice one thing that was odd but i need to look into that later it works for what we wanted however
16:48:11 stephenfin sean-k-mooney: what you wrote makes sense too
16:52:27 openstackgerrit sean mooney proposed openstack/nova master: support pci numa affinity policies in flavor and image https://review.opendev.org/674072
17:02:05 stephenfin sean-k-mooney: Got time to respin for the few final nits I pointed out? https://review.opendev.org/#/c/674072/
17:02:16 sean-k-mooney sure
17:02:24 openstackgerrit Merged openstack/nova master: compute: Use long_rpc_timeout in reserve_block_device_name https://review.opendev.org/693537
17:02:27 stephenfin cool. You've got 30 mins ;)
17:02:29 openstackgerrit Merged openstack/nova master: Add sequence diagrams to resize/cold migrate contrib doc https://review.opendev.org/695759
17:02:32 stephenfin then I'm outta here
17:08:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Follow up I18d73212f9d98bc75974a024cf6fd872fdfb1ca4 https://review.opendev.org/696740
17:12:37 openstackgerrit sean mooney proposed openstack/nova master: support pci numa affinity policies in flavor and image https://review.opendev.org/674072
17:13:36 sean-k-mooney stephenfin: ^ i think that is everything?
17:14:26 stephenfin yup, +2
17:14:34 stephenfin thanks
17:14:36 sean-k-mooney :)
17:14:45 sean-k-mooney all the us cores are off today right?
17:15:12 stephenfin I've seen the odd review bubbling up from mriedem and efried but yeah, pretty much
17:15:51 sean-k-mooney ok then before you go can you move it into the runway queue https://etherpad.openstack.org/p/nova-runways-ussuri
17:16:02 sean-k-mooney well form teh queue into a runway
17:17:31 stephenfin cool
17:17:34 sean-k-mooney hopefully we can get that squared away on monday unless gibi or johnthetubaguy feel like taking a look but ye would proably be better served enjoying the weekend
17:17:36 stephenfin and now watch
17:18:57 stephenfin hmm, turns out you can't put emoji in commit messages
17:19:00 stephenfin gerrit--
17:19:06 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Kill it https://review.opendev.org/696518
17:19:07 openstackgerrit Stephen Finucane proposed openstack/nova master: Rename 'nova.network.neutronv2' -> 'nova.network' https://review.opendev.org/696745
17:19:07 openstackgerrit Stephen Finucane proposed openstack/nova master: Rename 'nova.network.security_group.neutron_driver' -> 'nova.network.security_group' https://review.opendev.org/696746
17:19:21 sean-k-mooney you cant? im pretty sure i have see them there before
17:19:37 stephenfin and I've broken Gerrit. Hurrah!
17:19:50 stephenfin I see
17:19:51 stephenfin ! [remote rejected] HEAD -> refs/for/master%topic=bp/remove-nova-network-ussuri (internal server error: Error inserting change/patchset)
17:20:36 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove references to nova-net service from tests https://review.opendev.org/686818
17:20:36 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Blast final references to nova-network https://review.opendev.org/696498
17:20:37 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove final references to nova-net from functional tests https://review.opendev.org/696499
17:20:37 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Remove 'enable_hairpin' https://review.opendev.org/696500
17:20:38 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Remove vestigial nova-network support https://review.opendev.org/696501
17:20:38 openstackgerrit Stephen Finucane proposed openstack/nova master: hyperv: Remove vestigial nova-network support https://review.opendev.org/696502
17:20:39 openstackgerrit Stephen Finucane proposed openstack/nova master: vmware: Remove vestigial nova-network support https://review.opendev.org/696503
17:20:39 openstackgerrit Stephen Finucane proposed openstack/nova master: zvm: Remove vestigial nova-network support https://review.opendev.org/696504
17:20:40 openstackgerrit Stephen Finucane proposed openstack/nova master: xenapi: Remove vestigial nova-network support https://review.opendev.org/696505
17:20:40 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Flatten class hierarchy for neutron SG tests https://review.opendev.org/696506
17:20:41 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove SG tests that don't apply to neutron https://review.opendev.org/696507
17:20:41 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Drop nova-network-base security group tests https://review.opendev.org/696508
17:20:42 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Convert remaining API tests to use neutron https://review.opendev.org/696509
17:20:42 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Convert remaining unit tests to neutron https://review.opendev.org/696510
17:20:43 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove 'is_neutron_security_groups' function https://review.opendev.org/696511
17:20:43 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove nova-network security group driver https://review.opendev.org/696512
17:20:44 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Correct some broken VIF tests https://review.opendev.org/696513
17:20:44 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove firewall support https://review.opendev.org/696514
17:20:56 stephenfin there we go
17:21:21 stephenfin sean-k-mooney: https://review.opendev.org/#/c/696518/ :)
17:21:43 stephenfin There's something cathartic about those stats: +112, -15960
17:22:20 sean-k-mooney was the title orogint nova-net: kill it :)
17:22:31 sean-k-mooney *orgininally
17:22:35 stephenfin there was also a celebration emoji
17:22:40 sean-k-mooney ah ok
17:22:55 stephenfin damn you, Gerrit
17:23:31 sean-k-mooney so we will still have the neutron code in the nova/network/neutronv2 folder right
17:23:39 sean-k-mooney and some interface files in the folder above
17:23:44 stephenfin https://review.opendev.org/#/c/696745/
17:23:52 stephenfin I'm hoping not
17:24:06 stephenfin but that's way off. It'll be weeks before the CI lets us get that far
17:24:10 sean-k-mooney ok because i would like to move those to os-vif i think
17:24:19 sean-k-mooney but i wanted to see what was left after you were done
17:24:29 stephenfin yeah, we can probably kill nova.network.models next
17:24:34 stephenfin in favour of os-vif stuff
17:24:49 sean-k-mooney maybe but im not sure
17:24:54 stephenfin but I need to move offices again, so I best go
17:24:54 sean-k-mooney we put those in teh network info cache
17:25:01 stephenfin have a good weekend o/
17:25:16 sean-k-mooney we can serailise os-vif object but it would require an db migration
17:25:21 sean-k-mooney enjoy o/
18:28:20 openstackgerrit Merged openstack/nova stable/rocky: Fix false ERROR message at compute restart https://review.opendev.org/694581
18:28:26 openstackgerrit Merged openstack/nova master: Add test coverage of existing os-services policies https://review.opendev.org/669181
18:50:13 openstackgerrit Merged openstack/nova master: Introduce scope_types in os-services https://review.opendev.org/645427

Earlier   Later