| *** mhen_ is now known as mhen | 02:49 | |
| koya | Hi Uggla, bauzas, thank you very much for discussing “Instance metadata protection” at the recent OpenInfra Summit. | 05:59 |
|---|---|---|
| koya | Based on the comments you kindly provided, I’ve been reviewing and updating (revising) the blueprint; https://blueprints.launchpad.net/nova/+spec/instance-metadata-protection. I’ll reach out to you again when it’s ready. | 05:59 |
| *** ykarel_ is now known as ykarel | 07:08 | |
| artem_vasilyev | Hi, I stumbled upon the _cleanup_incomplete_migrations function and can't quite get whether this is intended logic or not. I couldn't find answers in the documentation. As I understand, the purpose of this function is to clean orphan files of already deleted instances. But when deleting instances, their migrations are also marked as soft-deleted (see https://github.com/openstack/nova/blob/master/nova/db/main/api.py#L1348). So this function | 10:38 |
| opendevreview | Adrien Cunin proposed openstack/os-vif master: Fixed bridge name when per_port_bridge is used https://review.opendev.org/c/openstack/os-vif/+/966410 | 10:38 |
| jkulik | artem_vasilyev: fyi, your message is cut off after ". So this function" | 10:39 |
| artem_vasilyev | thanks. So this function can't get error migrations, because it doesn't query migrations with read_deleted context. Is this the expected behavior, or am I missing something? | 10:40 |
| artem_vasilyev | _cleanup_incomplete_migrations function is here: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L11944 | 10:43 |
| jkulik | the tests mock away MigrationList.get_by_filters(), so they can't see it. from looking at the code, I'd agree with you artem_vasilyev | 10:50 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | 12:33 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Do not use libvirt before Host is initialized https://review.opendev.org/c/openstack/nova/+/966430 | 12:33 |
| opendevreview | Merged openstack/nova master: Default native threading for sch, api and metadata https://review.opendev.org/c/openstack/nova/+/965924 | 12:52 |
| gibi | dansmith: this might be interesting to you https://bugs.launchpad.net/nova/+bug/2130881 | 13:34 |
| gibi | dansmith: I'm fairly certain that we break libvirt lifecycle event handling with the patch https://review.opendev.org/c/openstack/nova/+/897076 - Sanity check that new hosts have no instances | 13:34 |
| gibi | but only for the first nova-compute startup case. Later nova-compute restarts are not affected | 13:35 |
| gibi | we need to reorder ComputeManager.init_host a bit to call driver.init_host before doing any libvirt driver queries | 13:36 |
| gibi | I will push a fix | 13:44 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: Show libvirt lifecycle event issue https://review.opendev.org/c/openstack/nova/+/966436 | 13:48 |
| gibi | this should show the problem clearly ^^ | 13:48 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce bug/2130881 https://review.opendev.org/c/openstack/nova/+/966443 | 14:41 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Init virt driver before use https://review.opendev.org/c/openstack/nova/+/966446 | 14:54 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make RBD Tpool usage conditional https://review.opendev.org/c/openstack/nova/+/956089 | 15:07 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make libvirt Tpool proxying conditional https://review.opendev.org/c/openstack/nova/+/956090 | 15:07 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make guestfs' Tpool usage optional https://review.opendev.org/c/openstack/nova/+/965821 | 15:07 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091 | 15:07 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092 | 15:07 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Do not fork compute workers in native threading mode https://review.opendev.org/c/openstack/nova/+/965466 | 15:07 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Compute manager to use native thread pools https://review.opendev.org/c/openstack/nova/+/966016 | 15:07 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Libvirt event handling without eventlet https://review.opendev.org/c/openstack/nova/+/965949 | 15:07 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | 15:07 |
| dansmith | gibi: oof | 15:14 |
| opendevreview | Dan Smith proposed openstack/nova master: Allow to perform parallel live migrations https://review.opendev.org/c/openstack/nova/+/955784 | 15:20 |
| opendevreview | Dan Smith proposed openstack/nova master: DNM: Test with >1 parallel migrations https://review.opendev.org/c/openstack/nova/+/966447 | 15:20 |
| dansmith | noonedeadpunk: ^ | 15:20 |
| dansmith | Uggla: (cc gibi) I thought we agreed to do specless blueprint for the parallel live migration during the PTG and we've had a meeting since then.. did we need to do some other step to actually approve the LP blueprint? | 15:46 |
| dansmith | ...in the meeting this week? | 15:46 |
| noonedeadpunk | I probably should abandon the spec as well | 15:46 |
| dansmith | yep | 15:46 |
| dansmith | leave it linked in the LP blueprint though so we have an easy reference for it | 15:47 |
| Uggla | Hi dansmith, no it is ok. I'll approve the BP if it is not. | 15:50 |
| dansmith | Uggla: okay I thought we were good to go.. yeah if you could approve I'd appreciate it | 15:50 |
| gibi | ohh cool, then I can upgrade my vote | 15:57 |
| opendevreview | sean mooney proposed openstack/nova-specs master: Add ResourceProviderWeigher scheduler spec https://review.opendev.org/c/openstack/nova-specs/+/951222 | 16:00 |
| razoff | Hi there ! | 16:13 |
| razoff | At my company I developed a backup solution (through a proxy instance) for openstack instances and I have one question for you. | 16:13 |
| razoff | I've been struggling to run efficient incremental backups. | 16:13 |
| razoff | Is it possible in any way to access dirty blocks of volume mounted on an instance through openstack API ? | 16:13 |
| razoff | I am asking this on the Nova channel because I noticed that you are using libvirt for virtualization and I know that it is possible to get dirty blocks from that lib. | 16:13 |
| razoff | Thank you in advance ! | 16:13 |
| dansmith | Uggla: can you do the needful? https://blueprints.launchpad.net/nova/+spec/libvirt-parallel-migrate | 16:20 |
| Uggla | dansmith, I think that's ok | 16:26 |
| dansmith | Uggla: oh, did you change it to approved? I was looking at "approver" and it is still empty so maybe I missed that it flipped state, thanks | 16:28 |
| dansmith | ah I see the comment in the whiteboard too.. I'm not subscribed so I didn't get the detailed notification | 16:28 |
| Uggla | dansmith, should be fully completed, I have added also that this is a specless BP. | 16:30 |
| dansmith | thanks | 16:31 |
| opendevreview | Merged openstack/nova master: Migrate mypy configuration to pyproject.toml https://review.opendev.org/c/openstack/nova/+/953703 | 17:18 |
| opendevreview | Merged openstack/nova master: Migrate codespell configuration to pyproject.toml https://review.opendev.org/c/openstack/nova/+/953704 | 17:18 |
| opendevreview | Merged openstack/nova master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/nova/+/953705 | 17:18 |
| gmaan | cores, whenever you have time, please check this simple change to fix the a non-deterministic tests https://review.opendev.org/c/openstack/nova/+/966223 | 17:26 |
| opendevreview | sean mooney proposed openstack/nova master: move nova-alt-config to debian-13 https://review.opendev.org/c/openstack/nova/+/966479 | 18:22 |
| opendevreview | Merged openstack/nova master: Fix test_simple_tenant_usage test https://review.opendev.org/c/openstack/nova/+/966223 | 19:27 |
| opendevreview | Merged openstack/nova master: Allow to perform parallel live migrations https://review.opendev.org/c/openstack/nova/+/955784 | 22:36 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!