Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-02-16
11:52:40 kashyap [I know, Python version isn't the problem in your case now]
11:52:41 stephenfin I need that work for the Secure Boot stuff also
11:52:52 stephenfin I can keep them separate too if you'd prefer
11:53:18 lyarwood stephenfin: yeah merge/nuke it given you've fixed things up in your change
11:53:40 lyarwood stephenfin: to be clear, close mine out.
11:55:31 kashyap stephenfin: Thanks for the review here; /me goes to read: https://review.opendev.org/c/openstack/nova/+/774240
11:56:00 kashyap stephenfin: lyarwood: My capacity this week is significantly reduced due to some personal event :-( and next week I'm on PTO
11:56:13 kashyap So I'll be somewhat less in action; but I'd like to address the feedback on the CPU stuff
11:56:57 lyarwood ack np, I can help push things along while you're out this week, I'm also out Thursday/Friday next week FWIW.
11:57:54 kashyap Thank you; the CPU stuff is in shape, modulo Stephen's comments. (And just trying to close out some more pressing downstream stuff, too)
11:58:20 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Add 'nova.virt.libvirt.utils.get_arch' stub to fixture https://review.opendev.org/c/openstack/nova/+/774904
11:58:20 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Remove unnecessary mock https://review.opendev.org/c/openstack/nova/+/774905
11:58:21 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Poison os.uname https://review.opendev.org/c/openstack/nova/+/775415
11:58:26 kashyap I'm still here this week, but functioning less than 100%. (Next week I'm off)
11:58:26 stephenfin b******
11:58:43 stephenfin that shouldn't have rebased the whole series :-(
11:59:27 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Add 'nova.virt.libvirt.utils.get_arch' stub to fixture https://review.opendev.org/c/openstack/nova/+/774904
11:59:28 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Remove unnecessary mock https://review.opendev.org/c/openstack/nova/+/774905
11:59:28 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Poison os.uname https://review.opendev.org/c/openstack/nova/+/775415
12:00:29 lyarwood -R ftw
12:01:12 stephenfin nah, I've a bad habit of using 'git rebase -i master' to do any history rewriting work
12:01:21 stephenfin vs. git rebase -i HEAD~2 or whatever I needed there
12:05:47 lyarwood huh does that edit the entire series from origin/master/HEAD?
12:06:24 stephenfin rebases and lets me do what I want to the entire series
12:07:05 lyarwood ah it rebases onto master, so if that changes the entire series does
12:07:14 stephenfin yup
12:07:20 lyarwood yeah I just use -i HEAD~$n
12:07:37 stephenfin a wiser approach
13:11:57 kashyap Randomly thinking out loud ... a big blocker in the way of secure boot is the handling of the NVRAM file (which stores boot-level variables) -- right now, on instance start/stop or any of the "move operations" (all kinds of migration), the NVRAM file gets reset
13:18:51 gibi lyarwood: I read through the machine_type series, left commentas and questions but overall I'm OK with the direction
13:26:59 lyarwood gibi: ack thanks, in return I have your device detach series open and will go through it at some point today
13:32:13 gibi lyarwood: stephen had some comments on that so I have to go back and fix things up
13:32:18 gibi lyarwood: but review is appreciated
13:32:24 lyarwood gibi: ack understood
13:44:51 elod gibi: can you hit this one? (not that placement @ train has much to release, just this one: Add DEBUG logs to help troubleshoot no allocation candidates): https://review.opendev.org/c/openstack/releases/+/773093
13:45:22 openstackgerrit Merged openstack/os-vif master: Drop use of deprecated collections classes https://review.opendev.org/c/openstack/os-vif/+/773360
14:08:25 openstackgerrit Lajos Katona proposed openstack/nova master: Run nova-live-migration job with neutron-trunk enabled https://review.opendev.org/c/openstack/nova/+/775838
14:49:36 openstackgerrit Sylvain Bauza proposed openstack/nova master: Add net & utils methods for routed nets & segments https://review.opendev.org/c/openstack/nova/+/773976
14:49:39 openstackgerrit Sylvain Bauza proposed openstack/nova master: Add a routed networks scheduler pre-filter https://review.opendev.org/c/openstack/nova/+/749068
14:49:52 bauzas gibi: stephenfin: sean-k-mooney: eventually done with routed networks ^
14:50:35 bauzas I just splitted the functests into a whole separate module self-containing its own fixture, plus some nits from stephenfin
14:50:48 bauzas stylish nits
14:51:02 sean-k-mooney bauzas: since its fresh in your mind does the requested networks have the list of ports in it
14:51:05 sean-k-mooney it does right
14:51:13 bauzas sean-k-mooney: the fixture ?
14:51:16 sean-k-mooney well ports or networks
14:51:22 sean-k-mooney no in the request spec
14:51:22 bauzas ah the request ?
14:51:33 sean-k-mooney i need it for something else
14:51:34 bauzas well, for boot, we get the list of networks or ports
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 bauzas (but that's only for booting)
14:53:10 sean-k-mooney for the port numa affinity policeis you will need to precreate the ports
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

Earlier   Later