| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-12-14 | |||
| 10:57:33 | opendevreview | Stephen Finucane proposed openstack/nova master: tox: Use pre-commit for pep8 target https://review.opendev.org/c/openstack/nova/+/896227 | |
| 11:12:59 | bauzas | gibi: no worries, as you can | |
| 11:18:28 | opendevreview | Merged openstack/nova master: tests: Use clearer syntax for empty generator https://review.opendev.org/c/openstack/nova/+/903076 | |
| 12:40:30 | dvo-plv | sean-k-mooney, thanks, it works with venv, but I can not figure out how it works previously, couple days ago | |
| 12:57:52 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Further clarifications to the SG doc https://review.opendev.org/c/openstack/nova/+/903675 | |
| 12:57:57 | stephenfin | sean-k-mooney: per our discussion ^ | |
| 13:22:27 | bauzas | gibi: want to ship your recent backport into Antelope? I was asked to +1 https://review.opendev.org/c/openstack/releases/+/899603 | |
| 13:24:41 | bauzas | I think I'll release first the antelope reelase and then we will create another stable release for https://review.opendev.org/c/openstack/nova/+/903553 | |
| 13:31:28 | gibi | bauzas: left feedback on the mdev live migration series | |
| 13:31:42 | bauzas | gibi: <3 | |
| 13:42:17 | bauzas | gibi: replied https://review.opendev.org/c/openstack/nova-specs/+/900636 | |
| 13:42:30 | bauzas | you asked a very good question about the conductor option | |
| 13:42:48 | bauzas | I already thought about all the possibilities but I don't know which one is the better | |
| 13:43:20 | bauzas | and I also tried to explain you why and how we need a libvirtdriver dict | |
| 13:44:33 | bauzas | another possibility instead of an internal dict would be to persist *something* *somewhere* about instances/mdevs | |
| 13:44:41 | bauzas | either in DB or in some file | |
| 13:44:58 | bauzas | I know sean-k-mooney has concerns about the fact we check the usage by looking at the guest XMLs | |
| 13:46:29 | bauzas | (I mean, if we're about persisting anything, I would way prefer to do it in some file like /var/lib/instances rather than with our Nova DB, for multiple reasons, one being I'm afraid of all the possible wrong values we would write in the DB that would mean to be fixed by hand) | |
| 14:08:02 | sean-k-mooney | bauzas: i have a personal goal of not requiring persitent xmls at all in the future | |
| 14:08:54 | sean-k-mooney | i want nova to be able to recreate a vm evne ifn the /etc/libvirt/qemu directiory is deleted | |
| 14:09:48 | sean-k-mooney | i would much prefer to use the db but /var/lib/instances is much better then what we have today | |
| 14:10:53 | sean-k-mooney | bauzas: if we coudl converge the pci tracker and medev code paths into a common device tracker that would be awsome in my view but i dont think that is goign to happen near term | |
| 14:18:20 | bauzas | sean-k-mooney: sorry in a meeting, let's discussing in 10-ish | |
| 14:18:25 | bauzas | discuss* | |
| 14:25:14 | bauzas | sean-k-mooney: back, readinh | |
| 14:26:33 | bauzas | sean-k-mooney: sure, tbc, lemme explain why we're discussing it now : gibi is wondering why we need an internal dict to "persist" which mdevs are taken for live-migration (before the target guest runs with a domain) | |
| 14:27:52 | bauzas | so I was just offering to persist some 'mdevs' file in the related /var/lib/instances/<inst_uuid> directory that would tell 'yo, those are the mdevs I use' | |
| 14:28:13 | bauzas | but there be dragon | |
| 14:28:45 | bauzas | (I'm pretty sure that a domain is created on target's /var/lib/instances as soon as we start the live-migration) | |
| 14:28:49 | opendevreview | Merged openstack/nova master: Remove Ironic client wrapper https://review.opendev.org/c/openstack/nova/+/867797 | |
| 14:28:59 | opendevreview | Merged openstack/nova master: Remove unnecessary aliasing https://review.opendev.org/c/openstack/nova/+/867798 | |
| 14:29:06 | opendevreview | Merged openstack/nova master: Use real SDK objects in tests https://review.opendev.org/c/openstack/nova/+/867799 | |
| 14:29:14 | bauzas | wow the gate sounds definitely better :) | |
| 14:31:20 | bauzas | (tbh, what I said about writing a file in /var/lib/instances per instance dir could be a problem with shared storage, or not, depending on whether we are able to say "look, this mdev doesn't exist, I'm gonna recreate it" | |
| 14:38:49 | sean-k-mooney | we can always nest it in a $hostname folder if needed | |
| 14:53:26 | bauzas | sean-k-mooney: say I have a magical wand, what would be your preference ? using an internal dict like I wrote in the spec or persist some file in /var/lib/instances/<inst> ? | |
| 14:53:42 | bauzas | the latter would require me to update the spec but I can live with that | |
| 14:54:38 | bauzas | (this wouldn't impact the current behaviour which would be to lookup the domain definitions, this is just additive either way) | |
| 14:55:12 | bauzas | if we decide later to use that file too for mdevs we spawn, then we would eventually stop to lookup the domain XMLs | |
| 14:55:40 | bauzas | this is Christmas time, make a wish. | |
| 14:55:52 | bauzas | gibi: sean-k-mooney: ^ | |
| 14:57:42 | bauzas | (note : I don't know *yet* how to implement a persistent file but I can arrange that by remembering the service uuid file thing | |
| 15:09:01 | gibi | > we use the internal dict for persisting which mdevs are already used for live-migration for making sure that other instances being created in the target compute wouldn't take them while we live-migrate. | |
| 15:09:17 | gibi | OK so this is basically a lock between the instances | |
| 15:09:35 | gibi | so that again resource tracker territory. But I got it that it is too big of a move | |
| 15:10:08 | gibi | so I drop my question about persistence | |
| 15:10:15 | gibi | an in memory dict is OK to me | |
| 15:11:33 | bauzas | fair enough | |
| 15:11:47 | JayF | Thanks gibi \o/ was quite a gift to wake up to the rest of that chain being not only approved but through the gate! | |
| 15:12:07 | bauzas | gibi: so we only have one open question about the conductor conf option | |
| 15:12:22 | bauzas | gibi: see my comment about it and all the alternatives | |
| 15:13:05 | gibi | JayF: it is almost Christmas anyhow :) | |
| 15:13:14 | gibi | bauzas: I'm on it | |
| 15:13:19 | bauzas | thanks | |
| 15:13:27 | JayF | Let me rebase my ironic<>nova driver bugfix and I'll add something else to my list ;) | |
| 15:13:35 | gibi | :) | |
| 15:25:40 | gibi | > We could avoid the pre-flight check that adds a conductor option and just | |
| 15:25:43 | gibi | assume to test the libvirt versions and the types during the pre-livemigration | |
| 15:25:46 | gibi | conductor checks that claim the resources. | |
| 15:26:08 | gibi | bauzas: does it mean we can do all our checks in pre-livemigration without the extra conductor option? | |
| 15:26:46 | gibi | if so then I vote for not having a conductor option and the the pre-livemigration do the checking | |
| 15:27:00 | gibi | pre-livemigration is there for many other checks like cpu compatibility | |
| 15:28:14 | gibi | bauzas: ^^ | |
| 15:39:58 | opendevreview | ribaudr proposed openstack/nova master: Reproducers for bug 1869804 https://review.opendev.org/c/openstack/nova/+/877772 | |
| 15:39:59 | opendevreview | ribaudr proposed openstack/nova master: Fix: Live migrating to a host with cpu_shared_set configured will now update the VM's configuration accordingly. https://review.opendev.org/c/openstack/nova/+/903706 | |
| 15:39:59 | opendevreview | ribaudr proposed openstack/nova master: Fix [backport]: Live migrating to a host with cpu_shared_set configured will now update the VM's configuration accordingly. https://review.opendev.org/c/openstack/nova/+/877773 | |
| 15:43:16 | bauzas | gibi: cool with me so | |
| 15:43:50 | bauzas | I'll doublecheck with the code if that would work, since we'll go to the target first | |
| 15:58:07 | gibi | bauzas: ack | |
| 15:58:29 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: scheduler: AggregateMultitenancyIsolation to support unlimited tenant https://review.opendev.org/c/openstack/nova/+/896512 | |
| 16:49:53 | opendevreview | Merged openstack/nova stable/2023.1: [pwmgmt]ignore missin governor when cpu_state used https://review.opendev.org/c/openstack/nova/+/903553 | |
| 16:50:00 | opendevreview | Merged openstack/nova master: Remove outdated comment about allocation ratios https://review.opendev.org/c/openstack/nova/+/900242 | |
| 16:55:49 | opendevreview | Pierre Riteau proposed openstack/nova stable/2023.2: Remove outdated comment about allocation ratios https://review.opendev.org/c/openstack/nova/+/903684 | |
| 17:29:19 | bauzas | gibi: doh, we missed the target for releasing your bugfix shortly by 1 hour | |
| 17:36:07 | gibi | bauzas: now worries | |
| 17:36:34 | gibi | bauzas: that is not a blocker until we start testing with cpu_dedicated_set | |
| 20:56:51 | opendevreview | Jay Faulkner proposed openstack/nova master: [ironic] Partition & use cache for list_instance* https://review.opendev.org/c/openstack/nova/+/900831 | |
| #openstack-nova - 2023-12-15 | |||
| 03:20:03 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/903427 | |
| 10:41:00 | opendevreview | Stephen Finucane proposed openstack/nova master: tox: Use pre-commit for pep8 target https://review.opendev.org/c/openstack/nova/+/896227 | |
| 10:41:01 | opendevreview | Stephen Finucane proposed openstack/nova master: pre-commit: Bump linter versions https://review.opendev.org/c/openstack/nova/+/903749 | |
| 10:41:01 | opendevreview | Stephen Finucane proposed openstack/nova master: pre-commit: Add mypy https://review.opendev.org/c/openstack/nova/+/896228 | |
| 11:51:05 | opendevreview | Merged openstack/nova stable/2023.2: Remove outdated comment about allocation ratios https://review.opendev.org/c/openstack/nova/+/903684 | |
| 12:00:07 | opendevreview | Pierre Riteau proposed openstack/nova stable/2023.1: Remove outdated comment about allocation ratios https://review.opendev.org/c/openstack/nova/+/903695 | |
| 12:35:47 | opendevreview | sean mooney proposed openstack/nova master: [codespell] ignore codespell in git blame https://review.opendev.org/c/openstack/nova/+/897215 | |
| 12:35:47 | opendevreview | sean mooney proposed openstack/nova master: [codespell] fix final typos and enable ci https://review.opendev.org/c/openstack/nova/+/897214 | |
| 12:50:59 | opendevreview | sean mooney proposed openstack/nova master: fix sphinx-lint errors in docs and add ci https://review.opendev.org/c/openstack/nova/+/897089 | |
| 12:51:00 | opendevreview | sean mooney proposed openstack/nova master: ignore sphinx-lint series in git blame https://review.opendev.org/c/openstack/nova/+/897218 | |
| 13:34:50 | zigo | stephenfin: Is there a 2/2 patch for this one ? https://review.opendev.org/c/openstack/heat/+/896025 | |
| 13:44:06 | stephenfin | Tes. It's listed on the Relation chain pane on the right | |
| 13:44:09 | stephenfin | *Yes | |
| 14:06:36 | zigo | Ah, right ... | |
| 16:34:33 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Proposes mdev live-migration support in libvirt https://review.opendev.org/c/openstack/nova-specs/+/900636 | |
| 16:43:03 | kashyap | bauzas: Hi, unrelated: I think this can be a specless-bp, agree? - https://blueprints.launchpad.net/nova/+spec/allow-disabling-ephemeral-disk-formatting | |
| 16:46:22 | kashyap | It came out of a discussion w/ Sean. No need to respond right away; can talk on Mon. | |
| 16:49:34 | bauzas | kashyap: you left but lemme log this : we can discuss this on a next nova meeting, like Tuesday | |
| #openstack-nova - 2023-12-16 | |||
| 03:08:57 | opendevreview | melanie witt proposed openstack/nova master: WIP Support rescue with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/873675 | |
| 03:08:58 | opendevreview | melanie witt proposed openstack/nova master: WIP libvirt: Introduce support for raw with LUKS https://review.opendev.org/c/openstack/nova/+/884313 | |
| 03:08:58 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273 | |
| 03:08:59 | opendevreview | melanie witt proposed openstack/nova master: WIP libvirt: Introduce support for rbd with LUKS https://review.opendev.org/c/openstack/nova/+/889912 | |
| 03:23:18 | opendevreview | melanie witt proposed openstack/nova master: DNM test ephemeral encryption + resize: qcow2, raw, rbd https://review.opendev.org/c/openstack/nova/+/862416 | |