| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-03-30 | |||
| 14:47:15 | gibi | stephenfin: FYI there is a rendering issue in https://review.opendev.org/c/openstack/nova/+/781438/2/doc/source/admin/cpu-models.rst#60 but not serious so I'm OK if it is fixed in a fup | |
| 14:47:56 | stephenfin | damn, ack - will do a follow-up | |
| 15:01:07 | sean-k-mooney | stephenfin: so the max cpu error. it looks like that only happens when pinning is enabled | |
| 15:01:28 | sean-k-mooney | when i remove hw:cpu_policy=dedicated it goes away | |
| 15:01:31 | stephenfin | yeah, that would make sense based on my comments | |
| 15:01:59 | sean-k-mooney | ya so ill leave that in the extra specs with hw:mem_page_size=small | |
| 15:04:34 | sean-k-mooney | actully ill use hw:mem_page_size='any' its technically not required but its incorrect to not set it | |
| #openstack-nova - 2021-03-31 | |||
| 00:17:34 | openstackgerrit | sean mooney proposed openstack/nova master: Test numa and vcpu topologies bug: #1910466 https://review.opendev.org/c/openstack/nova/+/769601 | |
| 00:17:36 | openstack | bug 1910466 in OpenStack Compute (nova) "NUMA instance spawn fails on get_best_cpu_topology when there is no 'threads' preference" [Medium,In progress] https://launchpad.net/bugs/1910466 - Assigned to sean mooney (sean-k-mooney) | |
| 00:17:36 | openstackgerrit | sean mooney proposed openstack/nova master: Fix max cpu topologies with numa affinity https://review.opendev.org/c/openstack/nova/+/769614 | |
| 01:11:13 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Re-propose Remove tenant_id https://review.opendev.org/c/openstack/nova-specs/+/783783 | |
| 07:17:19 | sorin-mihai | is it possible to avoid getting .novalocal appended to the hostname of the instances? | |
| 07:24:20 | openstackgerrit | Merged openstack/nova master: docs: Clarify value for 'hw:cpu_realtime' extra spec https://review.opendev.org/c/openstack/nova/+/782708 | |
| 07:32:36 | openstackgerrit | Balazs Gibizer proposed openstack/placement master: Add support for RP re- and un-parenting https://review.opendev.org/c/openstack/placement/+/784020 | |
| 08:03:10 | openstackgerrit | Balazs Gibizer proposed openstack/osc-placement master: Mark microversion 1.37 support https://review.opendev.org/c/openstack/osc-placement/+/784023 | |
| 08:03:37 | openstackgerrit | Balazs Gibizer proposed openstack/osc-placement master: Mark microversion 1.37 supported https://review.opendev.org/c/openstack/osc-placement/+/784023 | |
| 08:23:53 | bauzas | just in case sorin_mihal goes back : https://docs.openstack.org/nova/latest/configuration/config.html#api.dhcp_domain | |
| 09:43:02 | openstackgerrit | Merged openstack/nova master: docs: Use proper headers in nova-manage man page https://review.opendev.org/c/openstack/nova/+/726191 | |
| 09:52:25 | gibi_away | I will be away mostly this afternoon | |
| 10:11:50 | openstackgerrit | Merged openstack/nova stable/victoria: api: Log os-resetState as an instance action https://review.opendev.org/c/openstack/nova/+/772480 | |
| 10:31:36 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Further tweaks to the CPU models document https://review.opendev.org/c/openstack/nova/+/784066 | |
| 10:32:02 | stephenfin | sean-k-mooney: ^ | |
| 10:33:39 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Further tweaks to the CPU models document https://review.opendev.org/c/openstack/nova/+/784066 | |
| 10:38:09 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Correct typos in 'nova-manage db' docs https://review.opendev.org/c/openstack/nova/+/784067 | |
| 10:39:31 | openstackgerrit | Merged openstack/nova stable/ussuri: Make _rebase_with_qemu_img() generic https://review.opendev.org/c/openstack/nova/+/780780 | |
| 10:40:09 | openstackgerrit | Merged openstack/nova stable/ussuri: Use absolute path during qemu img rebase https://review.opendev.org/c/openstack/nova/+/757083 | |
| 10:40:28 | openstackgerrit | Merged openstack/nova stable/ussuri: libvirt: Use specific user when probing encrypted rbd disks during extend https://review.opendev.org/c/openstack/nova/+/773444 | |
| 10:55:34 | openstackgerrit | Claudiu Belu proposed openstack/nova stable/train: WIP: live migration: Pass limits onto scheduler-chosen destination https://review.opendev.org/c/openstack/nova/+/784071 | |
| 11:02:13 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Correct typos in 'nova-manage cells_v2' docs https://review.opendev.org/c/openstack/nova/+/784073 | |
| 12:45:12 | openstackgerrit | Merged openstack/nova stable/train: Test for disabling greendns https://review.opendev.org/c/openstack/nova/+/761763 | |
| 13:09:30 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: trivial: Fix spelling, formatting of vDPA spec https://review.opendev.org/c/openstack/nova-specs/+/781260 | |
| 14:43:59 | kashyap | stephenfin: Are you sure the default for AArch64 is coming from libvirt, instead of QEMU? | |
| 14:44:05 | kashyap | (For 'host-passthrough', i.e) | |
| 14:44:16 | stephenfin | nope | |
| 14:44:52 | kashyap | I'll confirm on the change; let me look in the code | |
| 14:45:03 | kashyap | And also ask an AArch64 dev, for good measure | |
| 14:46:27 | kashyap | I discussed this on an upstream docs change ... /me goes to find the link | |
| 14:47:19 | kashyap | stephenfin: Okay, from my own review comment: | |
| 14:47:25 | kashyap | [quote] | |
| 14:47:25 | kashyap | For AArch64, we now set the ``max`` as the default CPU model; it does the right thing in context of both QEMU TCG (plain emulation) and for KVM (hardware acceleration). | |
| 14:47:29 | kashyap | [/quote] | |
| 14:47:59 | kashyap | stephenfin: So I don't know where you got that 'host-passthrough' bit for AArch64; I'll comment on the change for the record | |
| 14:48:21 | stephenfin | I think that was based on comments from sean-k-mooney | |
| 14:48:47 | kashyap | That's incorrect, I'm afraid | |
| 14:51:45 | kashyap | stephenfin: Unlike x86, AArch64 does not even _have_ a default CPU model. We discussed it at length in this patch here: https://review.opendev.org/c/openstack/nova/+/709494 | |
| 14:56:11 | sean-k-mooney | kashyap: not accoundign to the code | |
| 14:56:33 | kashyap | sean-k-mooney: Very well it is; according to the code | |
| 14:56:44 | kashyap | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L4753 | |
| 14:56:53 | kashyap | sean-k-mooney: And see this release note: https://review.opendev.org/c/openstack/nova/+/709494/21/releasenotes/notes/bug-1864588-737c29560effd16e.yaml | |
| 14:57:20 | kashyap | Which I recall distinctly asking to word it that way (because it's truth) :) | |
| 15:05:44 | sean-k-mooney | ill be starting the call soon by the way | |
| 15:05:50 | sean-k-mooney | just fisnihgn a differnt call | |
| 15:06:06 | sean-k-mooney | oh upstream yeah :) | |
| 16:56:38 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1922053 https://review.opendev.org/c/openstack/nova/+/784129 | |
| 16:56:40 | openstack | bug 1922053 in OpenStack Compute (nova) "Operators can force up compute services with `done` evacuation migration records still active against the host" [Undecided,New] https://launchpad.net/bugs/1922053 | |
| 16:56:40 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP api: Reject requests to force up compute when `done` evacuation records exist https://review.opendev.org/c/openstack/nova/+/784130 | |
| 17:50:50 | openstackgerrit | Merged openstack/nova master: doc: Use a non-numerical anchor when referencing microversions https://review.opendev.org/c/openstack/nova/+/782696 | |
| 19:52:12 | zigo | I'm getting an error 500 when I try to launch my first Wallaby instance: | |
| 19:52:12 | zigo | [pid: 804881|app: 0|req: 34/573] 192.168.101.2 () {32 vars in 646 bytes} [Wed Mar 31 19:49:55 2021] GET /v2.1/flavors/cpu1-ram2-disk5 => generated 88 bytes in 411 msecs (HTTP/1.1 404) 7 headers in 339 bytes (3 switches on core 0) | |
| 19:52:12 | zigo | [pid: 804879|app: 0|req: 54/576] 192.168.101.2 () {32 vars in 614 bytes} [Wed Mar 31 19:49:56 2021] GET /v2.1/flavors => generated 907 bytes in 347 msecs (HTTP/1.1 200) 7 headers in 317 bytes (3 switches on core 0) | |
| 19:52:12 | zigo | [pid: 804880|app: 0|req: 38/577] 192.168.101.2 () {32 vars in 688 bytes} [Wed Mar 31 19:49:56 2021] GET /v2.1/flavors/93025366-9d69-489b-b4e7-f6aca509416f => generated 478 bytes in 345 msecs (HTTP/1.1 200) 7 headers in 317 bytes (3 switches on core 0) | |
| 19:52:12 | zigo | [pid: 804868|app: 0|req: 33/601] 192.168.101.2 () {36 vars in 668 bytes} [Wed Mar 31 19:49:57 2021] POST /v2.1/servers => generated 211 bytes in 3080 msecs (HTTP/1.1 500) 7 headers in 351 bytes (3 switches on core 0) | |
| 19:52:20 | zigo | Any idea what's going on? | |
| 19:52:34 | zigo | It's not as if it was so verbose ... :/ | |
| 19:57:19 | zigo | Oh ... | |
| 19:57:19 | zigo | <class 'nova.exception.GlanceConnectionFailed'> (HTTP 500) (Request-ID: req-df3f3de6-e2ba-41a7-b717-67cee688d832) | |
| 19:57:54 | zigo | I know what's going on then. :) | |
| 20:31:52 | sean-k-mooney | :) | |
| 20:32:11 | sean-k-mooney | ya if glance is unhappy then nova will be too | |
| 21:04:23 | openstackgerrit | Rodrigo Barbieri proposed openstack/nova master: [WIP] Error anti-affinity violation on live migrations https://review.opendev.org/c/openstack/nova/+/784166 | |
| 21:30:37 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Update SRIOV port pci_slot when unshelving https://review.opendev.org/c/openstack/nova/+/784168 | |
| 21:31:04 | artom | sean-k-mooney, ^^ \o/ | |
| 22:09:29 | zigo | sean-k-mooney: I belive I'm *very* close from booting my first VM with Wallaby. :) | |
| 23:10:47 | sean-k-mooney | artom: i need to digest this and think about it but i _think_ that works. i wpoudl prefer if you were not doing [0] but i get it | |
| 23:11:07 | artom | sean-k-mooney, yeah, there's some questionable code in there, I agree | |
| 23:11:19 | artom | I did mark it as WIP this time ;) | |
| 23:11:47 | artom | I want CI to do a full run to prove out the idea, then we can refine | |
| 23:11:49 | sean-k-mooney | you figured out how to get the requester_id | |
| 23:12:04 | sean-k-mooney | i looked at this a few times and missed that | |
| 23:12:10 | artom | Yeah, by setting it first :P | |
| 23:12:32 | artom | Which makes me think - VMs that got booted before that patch is applied will still have the original bug | |
| 23:12:46 | artom | Because they won't have requester_id on their PCIRequests | |
| 23:13:11 | sean-k-mooney | ture but you coudl do an online migratieon with a nova manage commnd | |
| 23:13:26 | sean-k-mooney | to correalte the exsitng allocation using the pci_slot | |
| 23:13:46 | sean-k-mooney | for currently shelved vms it wont help | |
| 23:13:55 | sean-k-mooney | but for any exisitng vm it would | |
| 23:13:56 | artom | It won't indeed | |
| 23:15:02 | sean-k-mooney | i wonder if gibi had any other logic reated to it being set | |
| 23:15:26 | sean-k-mooney | it was only set for port with resouce request before | |
| 23:15:40 | sean-k-mooney | i dont think it shoudl break anything but its one of teh things we shoudl check | |
| 23:16:56 | sean-k-mooney | anyway ya ill take a look at that again tomrrow | |
| 23:17:13 | artom | Yeah, no rush | |
| 23:17:22 | artom | I was just happy I managed to hax something together | |
| 23:18:00 | sean-k-mooney | yes its similar to what i tred to do before but i was trying to do it without moving settign the requester id | |
| 23:18:22 | sean-k-mooney | but i mean that works | |
| 23:19:05 | sean-k-mooney | i was basically workign the otherway and trying to put the request_id in the vif object in the network infor cache | |
| 23:19:10 | sean-k-mooney | but this is better | |
| 23:19:35 | sean-k-mooney | since it avoid the object change | |
| 23:21:43 | artom | Yeah, this should be backportable | |
| 23:26:54 | artom | And thanks for the requester_id pointer, btw :) | |