*** haleyb is now known as haleyb|out | 00:58 | |
opendevreview | melanie witt proposed openstack/nova master: Report ephemeral disk encryption in the metadata API https://review.opendev.org/c/openstack/nova/+/909945 | 01:50 |
---|---|---|
opendevreview | melanie witt proposed openstack/nova master: Deprecate legacy ephemeral storage encryption using dm-crypt https://review.opendev.org/c/openstack/nova/+/909947 | 02:26 |
melwitt | looks like nova-ceph-multistore is fixed by the merge of https://review.opendev.org/c/openstack/requirements/+/909923 | 02:54 |
opendevreview | Merged openstack/nova master: [S-RBAC] adapt nova-next for port's binding:profile field change https://review.opendev.org/c/openstack/nova/+/909859 | 05:31 |
opendevreview | Maxim Monin proposed openstack/nova master: Server Rescue leads to Server ERROR state if original image is deleted https://review.opendev.org/c/openstack/nova/+/872385 | 05:32 |
opendevreview | melanie witt proposed openstack/nova master: Deprecate legacy ephemeral storage encryption using dm-crypt https://review.opendev.org/c/openstack/nova/+/909947 | 05:52 |
opendevreview | Maxim Monin proposed openstack/nova master: Server Rescue leads to Server ERROR state if original image is deleted https://review.opendev.org/c/openstack/nova/+/872385 | 07:42 |
opendevreview | Dr. Jens Harbott proposed openstack/nova master: DNM: Test glance new location api https://review.opendev.org/c/openstack/nova/+/891207 | 08:03 |
*** mklejn_ is now known as mklejn | 08:24 | |
LarsErikP | I have discovered something I wonder if it's a bug, or if it is intended behavior. In yoga at least, I can unshelve a VM that will cause an overconsumption of the CPU quota.. | 09:36 |
LarsErikP | Created a project with 4 VCPUs and 4GB RAM. Then spawned server1 with 2 cores and 2GB RAM. Shelved it. After that i spawned server2 with 4 cores and 2GB RAM. I have now used 4 of 4 CPUs in the quota. Then I unshelved server1 without any problem, and now I have used 6 of 4 CPUs... | 09:38 |
LarsErikP | I.. did not think this should be possible, as it enables users to easily overconsume their quota.. | 09:39 |
LarsErikP | you can do the same trick with overconsuming RAM as well | 09:48 |
LarsErikP | and, fwiw, both at once is also possible | 09:51 |
bauzas | LarsErikP: most of the community today is on PTO | 10:33 |
bauzas | are you sure that your limits were for 4 VCPUs ? | 10:34 |
LarsErikP | yes | 10:48 |
LarsErikP | I filed a bug anyways :P https://bugs.launchpad.net/nova/+bug/2054797 | 10:49 |
LarsErikP | and it kind of scales.. meaning if I create a bunch of VMs, shelve all of them, create a new one consuming the entire quota; I can stil unshelve all the shelved VMs | 10:52 |
opendevreview | sean mooney proposed openstack/nova master: add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/nova/+/899753 | 11:06 |
opendevreview | Konrad Gube proposed openstack/nova master: Use Cinder's os-extend_volume_completion volume action. https://review.opendev.org/c/openstack/nova/+/873560 | 11:20 |
opendevreview | sean mooney proposed openstack/nova master: Revert "[S-RBAC] adapt nova-next for port's binding:profile field change" https://review.opendev.org/c/openstack/nova/+/909834 | 11:55 |
kgube | bauzas, gibi: Hi, could you have another look at this? https://review.opendev.org/c/openstack/nova/+/873560 | 12:47 |
opendevreview | Merged openstack/nova master: Follow up patch to correct a minor documentation issue https://review.opendev.org/c/openstack/nova/+/908863 | 13:26 |
opendevreview | Merged openstack/os-vif master: Fix missing or unnecessary dependencies https://review.opendev.org/c/openstack/os-vif/+/909341 | 14:21 |
u00599 | Hey, we came across a problem in our openstack environment where the metadata-service is not marking interfaces with dhcp_enabled for network config with cloud-init. We came across the following commit which wanted to solve this problem, but this doesn't work for ovn with native dhcp (no agents) as there are no dhcp-agents anymore. The concerned networks don't have a default gateway set, so the "fix" is not working for us. Anybody came | 14:49 |
u00599 | https://github.com/openstack/nova/commit/c7f572d65b57e034c1391ed49d84f6e5f1d672ad | 14:49 |
artom | So it's hw:emulator_threads_policy with the s but hw:cpu_policy without the s | 15:17 |
artom | We're just evil, aren't we | 15:17 |
opendevreview | Merged openstack/nova master: check both source and dest compute libvirt versions for mdev lv https://review.opendev.org/c/openstack/nova/+/904176 | 15:40 |
jamespage | LarsErikP: https://bugs.launchpad.net/cloud-archive/+bug/2054687 to track | 15:40 |
opendevreview | Merged openstack/nova master: Check if destination can support the src mdev types https://review.opendev.org/c/openstack/nova/+/904177 | 15:46 |
opendevreview | Merged openstack/nova master: Reserve mdevs to return to the source https://review.opendev.org/c/openstack/nova/+/904209 | 15:46 |
sean-k-mooney | u00599: ovn has its ovn metadata agent | 15:52 |
sean-k-mooney | u00599: but unless they have fixed it neutron is not waiting for it to be ready before seneding the network-vif-plugged event | 15:53 |
sean-k-mooney | so there is or was a race when using ovn | 15:54 |
sean-k-mooney | LarsErikP: this is related to counting limits in placment | 16:00 |
sean-k-mooney | https://docs.openstack.org/nova/latest/configuration/config.html#quota.count_usage_from_placement | 16:06 |
sean-k-mooney | Behavior will be different for servers in SHELVED_OFFLOADED state. A server in SHELVED_OFFLOADED state will not have placement allocations, so it will not consume quota usage for cores and ram. Note that because of this, it will be possible for a request to unshelve a server to be rejected if the user does not have enough quota available to support the cores and ram needed by | 16:06 |
sean-k-mooney | the server to be unshelved. | 16:06 |
sean-k-mooney | LarsErikP: https://review.opendev.org/c/openstack/nova/+/888096 is the yoga fix but yoga is now unmaintained https://opendev.org/openstack/nova/src/branch/unmaintained/yoga | 16:16 |
sean-k-mooney | hum although im not sure that is corect since wallay and victoria should be unmainted then too | 16:17 |
sean-k-mooney | and xena | 16:17 |
sean-k-mooney | elodilles: ^ we should look into what up there | 16:17 |
sean-k-mooney | https://opendev.org/openstack/nova/branches | 16:18 |
sean-k-mooney | i forget what we had agreed but i taught we agreed not to skip any releases | 16:18 |
sean-k-mooney | so if yoga is moved to unamined i woudl expect all older branch to be as well | 16:18 |
elodilles | sean-k-mooney: we started the transition with yoga, as that had a 'deadline' some months ago... now we'll continue with the rest (xena, wallaby and victoria), actually, i just said on the Release meeting that i'm planning to propose the transition patches on Monday (despite the busy Caracal-3 milestone busy time... :S) | 16:29 |
sean-k-mooney | elodilles: oh ok | 16:30 |
sean-k-mooney | so they are just in flight | 16:30 |
sean-k-mooney | cool | 16:30 |
elodilles | yepp-yepp | 16:31 |
elodilles | that's the plan | 16:31 |
artom | Bleargh, the CPU power management stuff is not testable for multiple compute hosts in functional tests | 16:52 |
artom | All the work is being done by module-level functions, no way to mock/patch different instances on the two compute services/libvirt drivers | 16:52 |
artom | IOW, live migration reproducer tests are impossible | 16:52 |
artom | And I feel like if I make the CPU API a class and stick in an attribute of the driver, it won't fly, because it's a change that would only exist to allow tests | 16:53 |
sean-k-mooney | artom: we could test it with multiple compute nodes in the functional tests | 16:54 |
sean-k-mooney | but we are missing test infra to do that | 16:54 |
artom | sean-k-mooney, I don't see a way to the assert the online/offline CPUs on each host | 16:54 |
artom | Because the libvirt driver imports cpu/api.py as a global libvirt_cpu, and calls that | 16:55 |
artom | Actually, what I'd really need is to somehow stub in different "instances" of the Core class for each libvirt driver | 17:03 |
artom | More like, a different SysFS fixture for each... | 17:04 |
sean-k-mooney | artom: we made live migration work in teh functrional tests with sriov | 17:17 |
sean-k-mooney | we can make it work for power management | 17:18 |
sean-k-mooney | if we put in the effort | 17:18 |
artom | Right, I'm just not sure it's possible without changing the actual Nova code | 17:18 |
sean-k-mooney | anywya im ment to be off today so im going to leave the office again | 17:21 |
artom | Yeah, I was going to say, make sure you compensate for working today by not working Monday or something :) | 17:22 |
elodilles | sean-k-mooney LarsErikP : i've cherry picked the patch to unmaintained/yoga: https://review.opendev.org/c/openstack/nova/+/910010 | 17:47 |
elodilles | gate is happy with it, though there are no devstack based jobs among the zuul jobs at all, so that should be added to the branch first | 17:47 |
opendevreview | Artom Lifshitz proposed openstack/nova master: Power on cores for isolated emulator threads https://review.opendev.org/c/openstack/nova/+/909795 | 18:14 |
opendevreview | Artom Lifshitz proposed openstack/nova master: POC: power up dedicated cores during pre_live_migration https://review.opendev.org/c/openstack/nova/+/909806 | 18:14 |
opendevreview | Artom Lifshitz proposed openstack/nova master: Add cpuset_reserved helper to instance NUMA topology https://review.opendev.org/c/openstack/nova/+/910020 | 18:14 |
opendevreview | Artom Lifshitz proposed openstack/nova master: Reproducer for not powering on isolated emulator threads cores https://review.opendev.org/c/openstack/nova/+/910021 | 18:14 |
opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Reproducer tests for live migration with power management https://review.opendev.org/c/openstack/nova/+/910022 | 18:14 |
u00599 | sean-k-mooney: but infact ovn has just it's proxy and network_data.json is provided by nova-api or am i wrong? Infact the ovn has it's router and dhcp port in common and is owned by network:distributed. Wouldn't it be feasible to add another check for this owner when no dhcp-agent is found? | 19:48 |
opendevreview | Jay Faulkner proposed openstack/nova master: Limit nodes by ironic shard key https://review.opendev.org/c/openstack/nova/+/903915 | 22:55 |
opendevreview | Jay Faulkner proposed openstack/nova master: Add nova-manage ironic-compute-node-move https://review.opendev.org/c/openstack/nova/+/903916 | 22:55 |
opendevreview | Jay Faulkner proposed openstack/nova master: Make compute node rebalance safter https://review.opendev.org/c/openstack/nova/+/903917 | 22:55 |
opendevreview | Jay Faulkner proposed openstack/nova master: Make compute node rebalance safer https://review.opendev.org/c/openstack/nova/+/903917 | 23:23 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!