Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-02-16
14:51:47 bauzas and for move, then we get the existing ports
14:52:09 bauzas the list of *asked* ports or networks
14:52:14 sean-k-mooney i might need to copy the same logic for the numa prot policy work
14:52:37 bauzas like, if I ask for -net <something>
14:52:47 bauzas then the RequestSpec will get the network ID
14:52:53 sean-k-mooney ya thats ok
14:53:10 sean-k-mooney for the port numa affinity policeis you will need to precreate the ports
14:53:10 bauzas (but that's only for booting)
14:53:18 openstackgerrit Belmiro Moreira proposed openstack/nova master: Change API unexpected exception message https://review.opendev.org/c/openstack/nova/+/628061
14:53:22 sean-k-mooney so i can ignore networks
14:53:25 bauzas sean-k-mooney: if so, you'll get the ports in the spec
14:53:29 bauzas even for the boot
14:53:33 sean-k-mooney yep
14:53:37 sean-k-mooney just double checking
14:53:40 bauzas np
14:53:54 sean-k-mooney i basically have it implemented for sriov already
14:53:57 bauzas glad to help about network :p
14:54:10 sean-k-mooney i was checking how what else is needed for numa aware vswitchs
14:54:19 sean-k-mooney it look not that hard
14:54:38 sean-k-mooney but i need to pass the port info
14:54:51 openstackgerrit Merged openstack/nova master: libvirt: Don't drop CPU flags with policy='disable' from guest XML https://review.opendev.org/c/openstack/nova/+/775431
14:55:55 sean-k-mooney ill need to do something similar to this https://review.opendev.org/c/openstack/nova/+/749068/16/nova/scheduler/request_filter.py#317
14:59:25 sean-k-mooney i can either extend the Schduler limits object or i can look up the port policy form neutron in the hardware module
15:05:05 gibi elod: there is a newer commit in the stable/train branch of placement. see my comment in le checking
15:05:08 gibi 15:53 < bauzas> np
15:05:14 gibi ups
15:05:19 gibi wrong copy paster buffer
15:05:27 gibi elod: so see my comment in https://review.opendev.org/c/openstack/releases/+/773093
15:05:29 bauzas hah lol
15:05:32 bauzas no worries
15:05:58 gibi bauzas: will check back to the route net patch
15:08:22 openstackgerrit Johannes Kulik proposed openstack/nova master: vmware: Handle folder renames in rescue cmd https://review.opendev.org/c/openstack/nova/+/775852
15:12:35 elod gibi: thx, I've updated the patch
15:15:20 gibi elod: thanks, approved
15:15:33 elod \o/
15:41:08 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Allow disabling CPU flags via `cpu_model_extra_flags` https://review.opendev.org/c/openstack/nova/+/774240
16:10:00 openstackgerrit Lucian Petrut proposed openstack/nova master: hyper-v rbd volume support https://review.opendev.org/c/openstack/nova/+/763550
17:52:00 openstackgerrit Merged openstack/nova master: functional: Add test for #1898272 https://review.opendev.org/c/openstack/nova/+/756100
19:40:26 openstackgerrit Merged openstack/nova master: Use the non polling notification waiter in func test https://review.opendev.org/c/openstack/nova/+/758445
20:45:01 openstackgerrit Merged openstack/nova master: Make a couple test jobs run async devstack https://review.opendev.org/c/openstack/nova/+/775293
20:55:03 admin0 hi all .. is there a place where i can see the cloud-init script passed to a vm ?
#openstack-nova - 2021-02-17
00:52:13 openstackgerrit Merged openstack/nova master: cmd: Remove 'nova-manage db null_instance_uuid_scan' https://review.opendev.org/c/openstack/nova/+/773775
00:57:04 openstackgerrit Merged openstack/nova master: cmd: Remove 'nova-manage db ironic_flavor_migration' https://review.opendev.org/c/openstack/nova/+/773776
03:29:55 openstackgerrit Billy Olsen proposed openstack/nova stable/train: add functional regression test for bug #1888395 https://review.opendev.org/c/openstack/nova/+/759533
03:29:57 openstack bug 1888395 in OpenStack Compute (nova) ussuri "live migration of a vm using the single port binding work flow is broken in train as a result of the introduction of sriov live migration" [High,In progress] https://launchpad.net/bugs/1888395 - Assigned to Billy Olsen (billy-olsen)
03:29:58 openstackgerrit Billy Olsen proposed openstack/nova stable/train: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/c/openstack/nova/+/759153
04:14:23 openstackgerrit Merged openstack/nova master: hardware: Check inventory of shared CPUs for 'mixed' policy https://review.opendev.org/c/openstack/nova/+/756101
09:30:14 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Record the machine_type of instances in system_metadata https://review.opendev.org/c/openstack/nova/+/767533
09:30:15 openstackgerrit Lee Yarwood proposed openstack/nova master: nova-manage: Add machine_type get command https://review.opendev.org/c/openstack/nova/+/769548
09:30:16 openstackgerrit Lee Yarwood proposed openstack/nova master: nova-manage: Add machine_type update command https://review.opendev.org/c/openstack/nova/+/774896
09:30:17 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP nova-manage: Add machine_type list_unset command https://review.opendev.org/c/openstack/nova/+/774897
09:30:18 openstackgerrit Lee Yarwood proposed openstack/nova master: nova-status: Add hw_machine_type check for libvirt instances https://review.opendev.org/c/openstack/nova/+/770643
09:30:19 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Add a config update workflow test for [libvirt]hw_machine_type https://review.opendev.org/c/openstack/nova/+/774898
09:30:20 openstackgerrit Lee Yarwood proposed openstack/nova master: docs: Add admin docs for configuring and updating machine types https://review.opendev.org/c/openstack/nova/+/774899
09:46:32 swp20 sean-k-mooney: morning, i have linked irc log in the NOVNC patch, please check https://review.opendev.org/c/openstack/nova/+/622336
10:03:43 openstackgerrit Belmiro Moreira proposed openstack/nova master: Change API unexpected exception message https://review.opendev.org/c/openstack/nova/+/628061
10:30:34 bauzas stupid question : given the tox-lower-constraints job failed on https://review.opendev.org/c/openstack/nova/+/773976/6, I guess I can try to reproduce it locally by using the same tox target?
10:50:39 lyarwood bauzas: yes
10:50:51 bauzas it loosely fails
10:51:16 bauzas weirdo
10:51:28 bauzas asked for a recheck b/c I don't understand it a single shit
10:51:50 lyarwood you might need to hardcode the basepython version if you're on Fedora FWIW
10:51:59 lyarwood otherwise it's going to try to use py39
10:52:12 lyarwood that's likely going to break random stuff in that env
10:53:01 lyarwood tbh I assume your issue is with mock==3.0.0
10:53:08 bauzas yup
10:53:30 lyarwood try mock.Mock(spec=client.Client)?
10:53:50 lyarwood but yeah I don't get how auto_spec wouldn't work
10:54:07 lyarwood create_autospec* even
10:54:51 bauzas will try, need to lunch
10:54:54 bauzas lyarwood: thanks
10:55:54 bauzas lyarwood: the fun is that I wrote tests for both segments and networks, and they're both using autospec
10:56:18 bauzas but it only fails for networks, not subnets
10:56:26 bauzas so it looks to me unrelated
11:11:58 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Mock the os-brick connector within base.ServersTestBase https://review.opendev.org/c/openstack/nova/+/728914
11:26:48 openstackgerrit Lee Yarwood proposed openstack/nova master: tox: Add passenv DISABLE_CHERRY_PICK_CHECK to pep8 https://review.opendev.org/c/openstack/nova/+/776111
11:27:33 lyarwood elod: ^ thanks btw, I think we were calling the script directly downstream somewhere so I didn't even think about the tox target
11:47:12 elod lyarwood: oh, i see, np :)
12:32:50 gibi bauzas: went through the routed net series. I have some comments inline but you can convince me to handle those as follow ups
12:33:40 gibi bauzas: at the same time I would like to ask you to look at the last relevant QoS inteface attach patch https://review.opendev.org/c/openstack/nova/+/756530
14:04:13 stephenfin If anyone has some spare review time, https://review.opendev.org/q/topic:%2522poison-os-uname%2522+status:open are easy test improvement patches and getting them merged will help me avoid rebasing them endlessly as I work on the secure boot patches :-D
14:12:56 lyarwood stephenfin: ack
14:13:12 lyarwood sean-k-mooney: https://github.com/cirros-dev/cirros/issues/64 FYI, got it to work finally
14:14:47 sean-k-mooney hum https://github.com/lyarwood/cirros/commit/034d2e2e3719d96c4ef14ea859af72d7b5db311f
14:14:52 sean-k-mooney that really should not be needed
14:15:02 sean-k-mooney it shoudl have been automatically loaded
14:15:14 lyarwood it isn't even present in the image
14:15:15 sean-k-mooney but if it fixes it i gues thats ok
14:15:19 lyarwood that's the problem
14:15:28 lyarwood that list is copied in and everything else is dropped
14:15:34 lyarwood I guess that's what keeps the image so small
14:15:43 sean-k-mooney oh they are usign it for that too
14:15:48 sean-k-mooney i see
14:16:03 sean-k-mooney so the fact we can boot at all implies its in the initramfs
14:16:10 lyarwood yeah indeed
14:16:15 sean-k-mooney but its not in the root filesystem
14:16:19 lyarwood I couldn't work out where it was being pulled in tbh
14:16:33 lyarwood likely from a prebuilt initramfs from Ubuntu or something?
14:16:40 sean-k-mooney ya proably
14:17:01 sean-k-mooney cool hopefully that can get merged soon ish

Earlier   Later