| *** jmlowe has quit IRC | 00:21 | |
| *** jmlowe has joined #openstack-nova | 00:23 | |
| *** bnemec has quit IRC | 00:28 | |
| *** bnemec has joined #openstack-nova | 00:33 | |
| openstackgerrit | renminmin proposed openstack/nova master: SRIOV: Fix instance with macvtap port live migration failed https://review.opendev.org/738432 | 00:38 |
|---|---|---|
| *** bnemec has quit IRC | 00:40 | |
| *** bnemec has joined #openstack-nova | 00:46 | |
| openstackgerrit | renminmin proposed openstack/nova master: Fix instance booted with snapshot image and bdms https://review.opendev.org/745821 | 00:46 |
| *** k_mouza has joined #openstack-nova | 00:52 | |
| *** k_mouza has quit IRC | 00:56 | |
| *** bnemec has quit IRC | 00:57 | |
| *** boxiang_ has joined #openstack-nova | 01:05 | |
| *** boxiang has quit IRC | 01:05 | |
| openstackgerrit | Merged openstack/nova stable/ussuri: Add checks for volume status when rebuilding https://review.opendev.org/747122 | 01:10 |
| *** mriedem has left #openstack-nova | 01:14 | |
| *** bnemec has joined #openstack-nova | 01:15 | |
| *** Liang__ has joined #openstack-nova | 01:19 | |
| openstackgerrit | Hao Sun proposed openstack/nova stable/train: Add checks for volume status when rebuilding https://review.opendev.org/748558 | 01:23 |
| *** xiaolin has joined #openstack-nova | 01:32 | |
| openstackgerrit | Hao Sun proposed openstack/nova stable/stein: Add checks for volume status when rebuilding https://review.opendev.org/748559 | 01:38 |
| openstackgerrit | Hao Sun proposed openstack/nova stable/stein: Add checks for volume status when rebuilding https://review.opendev.org/748559 | 01:43 |
| *** bnemec has quit IRC | 01:45 | |
| *** bnemec has joined #openstack-nova | 01:50 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Remove `TODO` as we have already supported the decorator https://review.opendev.org/748560 | 01:53 |
| *** bnemec has quit IRC | 01:57 | |
| *** gyee has quit IRC | 01:58 | |
| *** mkrai has joined #openstack-nova | 02:02 | |
| *** stephenfin has quit IRC | 02:04 | |
| *** stephenfin has joined #openstack-nova | 02:08 | |
| *** links has joined #openstack-nova | 02:22 | |
| *** sapd1_x has joined #openstack-nova | 02:45 | |
| *** brinzhang0 has joined #openstack-nova | 02:51 | |
| *** brinzhang_ has quit IRC | 02:54 | |
| *** rcernin has quit IRC | 02:56 | |
| *** sapd1_x has quit IRC | 02:59 | |
| *** rcernin has joined #openstack-nova | 03:04 | |
| openstackgerrit | Merged openstack/nova stable/ussuri: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/747609 | 03:10 |
| openstackgerrit | Hao Sun proposed openstack/nova stable/train: Add checks for volume status when rebuilding https://review.opendev.org/748558 | 03:20 |
| openstackgerrit | Hao Sun proposed openstack/nova stable/stein: Add checks for volume status when rebuilding https://review.opendev.org/748559 | 03:23 |
| *** jdillaman has quit IRC | 03:29 | |
| *** suryasingh has joined #openstack-nova | 03:30 | |
| openstackgerrit | Merged openstack/nova master: Add generic reproducer for bug #1879878 https://review.opendev.org/747744 | 03:34 |
| openstack | bug 1879878 in OpenStack Compute (nova) "VM become Error after confirming resize with Error info CPUUnpinningInvalid on source node " [Medium,In progress] https://launchpad.net/bugs/1879878 - Assigned to Stephen Finucane (stephenfinucane) | 03:34 |
| openstackgerrit | Merged openstack/nova stable/ussuri: tests: Add reproducer for bug #1889633 https://review.opendev.org/748251 | 03:34 |
| openstack | bug 1889633 in OpenStack Compute (nova) ussuri "Pinned instance with thread policy can consume VCPU" [High,In progress] https://launchpad.net/bugs/1889633 - Assigned to Stephen Finucane (stephenfinucane) | 03:34 |
| openstackgerrit | Merged openstack/nova stable/ussuri: hardware: Reject requests for no hyperthreads on hosts with HT https://review.opendev.org/748252 | 03:35 |
| *** rcernin has quit IRC | 03:35 | |
| *** Liang__ has quit IRC | 03:37 | |
| *** Liang__ has joined #openstack-nova | 03:38 | |
| *** rcernin has joined #openstack-nova | 03:39 | |
| *** sapd1_x has joined #openstack-nova | 03:41 | |
| *** psachin has joined #openstack-nova | 03:51 | |
| openstackgerrit | Hao Sun proposed openstack/nova stable/train: Add checks for volume status when rebuilding https://review.opendev.org/748558 | 03:53 |
| *** rcernin has quit IRC | 03:54 | |
| openstackgerrit | Hao Sun proposed openstack/nova stable/train: Add checks for volume status when rebuilding https://review.opendev.org/748558 | 03:55 |
| *** ratailor has joined #openstack-nova | 03:58 | |
| *** rcernin has joined #openstack-nova | 04:08 | |
| *** rcernin has quit IRC | 04:18 | |
| *** rcernin has joined #openstack-nova | 04:19 | |
| *** mkrai has quit IRC | 04:21 | |
| *** sapd1_x has quit IRC | 04:26 | |
| *** mkrai has joined #openstack-nova | 04:29 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #openstack-nova | 04:33 | |
| *** spatel has joined #openstack-nova | 04:38 | |
| *** spatel has quit IRC | 04:43 | |
| *** vishalmanchanda has joined #openstack-nova | 05:01 | |
| *** mkrai has quit IRC | 05:22 | |
| *** xiaolin has quit IRC | 05:27 | |
| *** Liang__ has quit IRC | 06:01 | |
| *** Liang__ has joined #openstack-nova | 06:04 | |
| *** jsuchome has joined #openstack-nova | 06:20 | |
| openstackgerrit | Hao Sun proposed openstack/nova stable/stein: Add checks for volume status when rebuilding https://review.opendev.org/748559 | 06:21 |
| openstackgerrit | Xinran WANG proposed openstack/nova-specs master: SRIOV Enabled Nic Support Specification https://review.opendev.org/742785 | 06:43 |
| *** slaweq has joined #openstack-nova | 06:50 | |
| *** Liang__ has quit IRC | 06:57 | |
| *** belmoreira has joined #openstack-nova | 07:01 | |
| *** xek has joined #openstack-nova | 07:03 | |
| *** tesseract has joined #openstack-nova | 07:06 | |
| *** irclogbot_0 has quit IRC | 07:13 | |
| *** mkrai has joined #openstack-nova | 07:25 | |
| *** tosky has joined #openstack-nova | 07:35 | |
| *** Luzi has joined #openstack-nova | 07:37 | |
| *** damien_r has joined #openstack-nova | 07:46 | |
| openstackgerrit | Alexandre Arents proposed openstack/nova master: Add a lock to prevent race during detach/attach of interface https://review.opendev.org/747957 | 07:48 |
| *** tesseract has quit IRC | 07:56 | |
| *** ralonsoh has joined #openstack-nova | 07:59 | |
| *** martinkennelly has joined #openstack-nova | 08:00 | |
| *** mkrai has quit IRC | 08:04 | |
| *** mkrai_ has joined #openstack-nova | 08:04 | |
| *** tesseract has joined #openstack-nova | 08:06 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Cyborg evacuate support https://review.opendev.org/715326 | 08:09 |
| bauzas | good Friday Nova | 08:20 |
| stephenfin | morning | 08:21 |
| aarents | morning | 08:22 |
| *** derekh has joined #openstack-nova | 08:23 | |
| *** Yumeng has joined #openstack-nova | 08:25 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Enable real policy fixture by default https://review.opendev.org/745906 | 08:25 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Add and use 'GlanceFixture' https://review.opendev.org/745907 | 08:25 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Remove '_FakeImageService' https://review.opendev.org/745908 | 08:25 |
| stephenfin | gmann: Fixed up https://review.opendev.org/#/c/745906/ and the next two patches but lost your +2s in the process. Could you revisit today? | 08:25 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.moves https://review.opendev.org/727224 | 08:28 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.iteritems/itervalues/iterkeys https://review.opendev.org/727757 | 08:29 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.byte2int/int2byte https://review.opendev.org/727777 | 08:29 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/728094 | 08:29 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/728109 | 08:29 |
| *** kevinz has joined #openstack-nova | 08:32 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Remove `TODO` as we have already supported the decorator https://review.opendev.org/748560 | 08:35 |
| *** artom has quit IRC | 08:36 | |
| *** Luzi has quit IRC | 08:40 | |
| brinzhang0 | stephenfin: https://review.opendev.org/#/c/715326/22/nova/compute/api.py@3907 | 08:49 |
| stephenfin | brinzhang0: I've already replied | 08:50 |
| brinzhang0 | stephenfin: I think I missed your mean, you mean we should add the new flavor_id(target flavor_id) to check the flavor whether contain the dp_name, then dicide to resize or reject | 08:51 |
| brinzhang0 | right? | 08:51 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117 | 08:51 |
| brinzhang0 | stephenfin: I got your comment in the new patch, later we will consider to deal with that patch | 08:52 |
| brinzhang0 | the cyborg evacuate support patch was updated, pls review if you are free today, thanks | 08:53 |
| stephenfin | brinzhang0: Yes, I'm saying we need to check whether the new flavor contains dp_name also. Currently we are only checking the existing flavor | 08:53 |
| stephenfin | brinzhang0: This is what I did for vTPM https://github.com/openstack/nova/blob/75b5535e3448335cb26972b5ca37afbd24dba01d/nova/compute/api.py#L3997-L4020 | 08:53 |
| stephenfin | But yes, this should be dealt with separately in a follow-up | 08:53 |
| brinzhang0 | stephenfin: yeah, may we should add same_instance_type=False as the request parameter in the block_accelerators(), while we do resize, it shuold be set same_instance_type=True | 08:56 |
| *** Liang__ has joined #openstack-nova | 09:09 | |
| brinzhang0 | stephenfin: https://github.com/openstack/nova/blob/75b5535e3448335cb26972b5ca37afbd24dba01d/nova/compute/api.py#L3994-L3995 | 09:09 |
| brinzhang0 | if same_instance_type and flavor_id: raise(), if the same_flavor_type is True, the flavor_id must be true, do we need this judgement? | 09:11 |
| brinzhang0 | sorry, if the this is a migrate operation, it will be fine \o | 09:12 |
| openstackgerrit | Wenping Song proposed openstack/nova master: Process exception of delete resource provider https://review.opendev.org/748339 | 09:18 |
| *** spatel has joined #openstack-nova | 09:20 | |
| *** ralonsoh_ has joined #openstack-nova | 09:21 | |
| *** ralonsoh_ has quit IRC | 09:21 | |
| stephenfin | brinzhang0: Yes. If you're cold migrating, then the flavor_id parameter should be unset. It should only be set if resizing | 09:22 |
| stephenfin | and if it's set and is set to the ID of the same flavor as the existing instance flavor, that's an error | 09:23 |
| brinzhang0 | stephenfin: yeah, short-circuited of my brain \o/ | 09:23 |
| *** spatel has quit IRC | 09:24 | |
| *** mkrai_ has quit IRC | 09:26 | |
| *** vishalmanchanda has quit IRC | 09:27 | |
| openstackgerrit | Takashi Natsume proposed openstack/nova-specs master: Make 'Feature Liaison' optional in test https://review.opendev.org/748591 | 09:28 |
| openstackgerrit | Merged openstack/nova stable/train: libvirt: Handle VIR_ERR_DEVICE_MISSING when detaching devices https://review.opendev.org/742415 | 09:30 |
| *** ratailor has quit IRC | 09:30 | |
| jkulik | Did we ever think of bubbling down a rename of an Instance to the actual driver? At least the vmwareapi driver names the VMs in the vCenter "{display_name} ({uuid})", which doesn't match after a rename anymore. | 09:37 |
| openstackgerrit | Takashi Natsume proposed openstack/nova-specs master: Make 'Feature Liaison' optional in test https://review.opendev.org/748591 | 09:37 |
| *** k_mouza has joined #openstack-nova | 09:39 | |
| openstackgerrit | Merged openstack/nova stable/train: libvirt: Do not reference VIR_ERR_DEVICE_MISSING when libvirt is < v4.1.0 https://review.opendev.org/747358 | 09:40 |
| openstackgerrit | Merged openstack/nova stable/stein: libvirt: Provide VIR_MIGRATE_PARAM_PERSIST_XML during live migration https://review.opendev.org/747974 | 09:40 |
| openstackgerrit | Merged openstack/nova master: zuul: Start to migrate nova-live-migration to zuulv3 https://review.opendev.org/711604 | 09:40 |
| gibi | o/ | 09:40 |
| openstackgerrit | Wenping Song proposed openstack/nova master: Reject resize operation for accelerator https://review.opendev.org/748593 | 09:44 |
| *** stephenfin has quit IRC | 09:55 | |
| *** stephenfin has joined #openstack-nova | 09:56 | |
| *** Liang__ has quit IRC | 10:04 | |
| *** Liang__ has joined #openstack-nova | 10:05 | |
| *** Liang__ has quit IRC | 10:07 | |
| jsuchome | stephenfin gibi lyarwood: could you please agree on a way https://review.opendev.org/#/c/574301/ should be treated? I already said that I think it's better to follow what was decided in blueprint, but if you all agree it should be different, let's do it differently | 10:10 |
| gibi | so as far as I see artom and stephen on the side of a new config. I have no hard oppionon on either side so I will fall in line with the rest and support the new config then | 10:14 |
| *** stephenfin has quit IRC | 10:40 | |
| *** artom has joined #openstack-nova | 10:41 | |
| *** stephenfin has joined #openstack-nova | 10:41 | |
| *** stephenfin has quit IRC | 10:47 | |
| *** xiaolin has joined #openstack-nova | 10:47 | |
| *** stephenfin has joined #openstack-nova | 10:56 | |
| *** jangutter_ has joined #openstack-nova | 11:00 | |
| *** stephenfin has quit IRC | 11:02 | |
| *** stephenfin has joined #openstack-nova | 11:02 | |
| *** jangutter has quit IRC | 11:03 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Provide default 'host_info' for '_get_connection' https://review.opendev.org/746942 | 11:04 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Add 'start_compute' helper https://review.opendev.org/746943 | 11:04 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Use tempdir for CONF.instances_path https://review.opendev.org/746944 | 11:04 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Clean up PCI tests https://review.opendev.org/746945 | 11:04 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Add resize tests for PCI servers https://review.opendev.org/746946 | 11:04 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conductor: Don't use setattr https://review.opendev.org/746947 | 11:04 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Add test for SR-IOV neutron ports https://review.opendev.org/746948 | 11:04 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: fixtures: Handle binding of first port https://review.opendev.org/746949 | 11:04 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Add live migration tests for PCI, SR-IOV servers https://review.opendev.org/746950 | 11:04 |
| *** stephenfin has quit IRC | 11:07 | |
| *** stephenfin has joined #openstack-nova | 11:09 | |
| openstackgerrit | Merged openstack/nova master: Change default num_retries for glance to 3 https://review.opendev.org/740389 | 11:12 |
| *** jangutter_ has quit IRC | 11:15 | |
| *** jangutter has joined #openstack-nova | 11:15 | |
| *** stephenfin has quit IRC | 11:16 | |
| elod | lyarwood gibi : Train release patch, if you'll have time: https://review.opendev.org/#/c/748383/ | 11:20 |
| *** jangutter_ has joined #openstack-nova | 11:21 | |
| *** stephenfin has joined #openstack-nova | 11:21 | |
| *** jangutter has quit IRC | 11:24 | |
| *** stephenfin has quit IRC | 11:25 | |
| gibi | elod: +1 | 11:27 |
| elod | thx \o/ | 11:27 |
| swp20 | Stephenin: hi, when i run `python3 -m unittest nova.tests.unit.compute.test_api._ComputeAPIUnitTestMixIn.test_resize__with_vtpm`, it raise `TypeError: test_resize__with_vtpm() missing 1 required positional argument: 'mock_get_flavor'`, what's wrong? | 11:33 |
| swp20 | stephenfin: hi, when i run `python3 -m unittest nova.tests.unit.compute.test_api._ComputeAPIUnitTestMixIn.test_resize__with_vtpm`, it raise `TypeError: test_resize__with_vtpm() missing 1 required positional argument: 'mock_get_flavor'`, what's wrong? | 11:34 |
| elod | lyarwood: btw, do you want to wait with Stein release until these get merged: https://review.opendev.org/#/q/topic:bug/1887946+branch:stable/stein ? | 11:37 |
| *** stephenfin has joined #openstack-nova | 11:38 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Make PCI claim NUMA aware during live migration https://review.opendev.org/748453 | 11:40 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: [WIP] Support SRIOV interface attach and detach https://review.opendev.org/740995 | 11:40 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM: tracing the PCI allocation code path https://review.opendev.org/748454 | 11:40 |
| *** k_mouza has quit IRC | 11:43 | |
| *** stephenfin has quit IRC | 11:43 | |
| openstackgerrit | Elod Illes proposed openstack/nova stable/stein: libvirt: Handle VIR_ERR_DEVICE_MISSING when detaching devices https://review.opendev.org/742416 | 11:45 |
| openstackgerrit | Elod Illes proposed openstack/nova stable/stein: libvirt: Do not reference VIR_ERR_DEVICE_MISSING when libvirt is < v4.1.0 https://review.opendev.org/747360 | 11:45 |
| *** raildo has joined #openstack-nova | 11:48 | |
| *** xiaolin has quit IRC | 11:57 | |
| *** stephenfin has joined #openstack-nova | 11:59 | |
| *** stephenfin has quit IRC | 12:04 | |
| *** stephenfin has joined #openstack-nova | 12:11 | |
| *** stephenfin has quit IRC | 12:15 | |
| *** stephenfin has joined #openstack-nova | 12:23 | |
| *** eharney has joined #openstack-nova | 12:26 | |
| *** stephenfin has quit IRC | 12:28 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Cyborg evacuate support https://review.opendev.org/715326 | 12:29 |
| *** stephenfin has joined #openstack-nova | 12:48 | |
| openstackgerrit | Luyao Zhong proposed openstack/nova master: Introduce a new driver interface 'cleanup_instance' https://review.opendev.org/714653 | 12:48 |
| openstackgerrit | Luyao Zhong proposed openstack/nova master: Track error migrations in resource tracker https://review.opendev.org/745281 | 12:48 |
| openstackgerrit | Luyao Zhong proposed openstack/nova master: Improve orphans tracking in resource tracker https://review.opendev.org/745282 | 12:48 |
| openstackgerrit | Luyao Zhong proposed openstack/nova master: Rollback when reset running live migration https://review.opendev.org/747899 | 12:48 |
| *** bnemec has joined #openstack-nova | 12:51 | |
| *** rcernin has quit IRC | 12:56 | |
| *** spatel has joined #openstack-nova | 13:00 | |
| *** spatel has quit IRC | 13:04 | |
| openstackgerrit | Alexandre Arents proposed openstack/nova master: Add a lock to prevent race during detach/attach of interface https://review.opendev.org/747957 | 13:16 |
| aarents | Hum commit message edit re-trigger all the stuff.. Now I now | 13:18 |
| aarents | know | 13:18 |
| sean-k-mooney | yes but i dont think it clears +2 | 13:21 |
| sean-k-mooney | i do think it clears +w | 13:21 |
| sean-k-mooney | assuming you only edit the comit and not the code | 13:21 |
| sean-k-mooney | its treated like a rebase in that way | 13:22 |
| *** jangutter_ has quit IRC | 13:25 | |
| *** jangutter has joined #openstack-nova | 13:26 | |
| *** k_mouza has joined #openstack-nova | 13:27 | |
| *** jangutter_ has joined #openstack-nova | 13:28 | |
| *** jangutter has quit IRC | 13:30 | |
| *** vishalmanchanda has joined #openstack-nova | 13:38 | |
| *** nweinber has joined #openstack-nova | 13:39 | |
| aarents | sean-k-mooney: it seems in my case to have drop +2, NVM | 13:40 |
| *** nweinber has quit IRC | 13:43 | |
| *** links has quit IRC | 13:43 | |
| *** nweinber has joined #openstack-nova | 13:43 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Add 'start_compute' helper https://review.opendev.org/746943 | 13:48 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Use tempdir for CONF.instances_path https://review.opendev.org/746944 | 13:48 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Clean up PCI tests https://review.opendev.org/746945 | 13:48 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Add resize tests for PCI servers https://review.opendev.org/746946 | 13:48 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conductor: Don't use setattr https://review.opendev.org/746947 | 13:48 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Add test for SR-IOV neutron ports https://review.opendev.org/746948 | 13:48 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: fixtures: Handle binding of first port https://review.opendev.org/746949 | 13:48 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Add live migration tests for PCI, SR-IOV servers https://review.opendev.org/746950 | 13:48 |
| *** k_mouza has quit IRC | 13:49 | |
| gmann | stephenfin: done | 13:50 |
| *** k_mouza has joined #openstack-nova | 13:54 | |
| stephenfin | aarents: Could I get the comment here and I can fast approve, seeing as gibi was +2 before? https://review.opendev.org/#/c/747957/ | 14:08 |
| *** stephenfin_ has joined #openstack-nova | 14:14 | |
| *** stephenfin has left #openstack-nova | 14:14 | |
| openstackgerrit | Merged openstack/nova master: vmware: Handle exception in destroy with attached volumes https://review.opendev.org/738416 | 14:14 |
| stephenfin_ | hurrah, my znc instance is alive again \o/ | 14:15 |
| *** bnemec has quit IRC | 14:15 | |
| *** k_mouza has quit IRC | 14:18 | |
| *** k_mouza has joined #openstack-nova | 14:20 | |
| *** bnemec has joined #openstack-nova | 14:20 | |
| aarents | stephenfin_: done thks | 14:27 |
| stephenfin_ | aarents: Sorry, I meant can I get a comment in the code? :D | 14:34 |
| stephenfin_ | aarents: Basically so some poor sucker isn't looking at this in 5 years time saying "why on earth did Alexandre add that call?" :P | 14:36 |
| aarents | stephenfin_: sure.. friday.. | 14:36 |
| * aarents need a beer | 14:36 | |
| *** stephenfin_ is now known as stephenfin | 14:37 | |
| stephenfin | gibi: Could you weigh in on https://review.opendev.org/#/c/748411/ ? I think it's a fair request, but in fairness it's not what the BP stated so maybe I'm being unfair | 14:40 |
| *** sapd1 has quit IRC | 14:40 | |
| gibi | I did so on IRC but I can put that in the patch too | 14:41 |
| gibi | 12:14 < gibi> so as far as I see artom and stephen on the side of a new config. I have no | 14:41 |
| gibi | hard oppionon on either side so I will fall in line with the rest and | 14:41 |
| gibi | support the new config then | 14:41 |
| *** dklyle has joined #openstack-nova | 14:41 | |
| stephenfin | ah, sorry. Had ZNC issues so I missed any discussion this morning /o\ | 14:41 |
| stephenfin | Thanks | 14:41 |
| gibi | doen | 14:41 |
| gibi | done | 14:42 |
| gibi | no worries | 14:42 |
| *** _erlon_ has joined #openstack-nova | 14:42 | |
| artom | gibi, yeah, I don't really have strong opinions one way or the other | 14:48 |
| artom | gibi, conceptually the new clean options seems cleaner | 14:48 |
| artom | But maybe there are practical considerations for reviving the old one | 14:48 |
| stephenfin | gibi: Also, in the context of https://review.opendev.org/#/c/748453/, have you seen https://review.opendev.org/#/c/746950/ ? | 14:49 |
| stephenfin | In theory it _should_ be reproducible there, so I can put together a test for that early next week | 14:49 |
| *** xek has quit IRC | 14:50 | |
| gibi | stephenfin: sounds cool. | 14:52 |
| gibi | stephenfin: I wasn't eager to try to create a functional reproduce for that | 14:52 |
| gibi | stephenfin: as that bug is a bit tangential to the sriov attach work I do righ now | 14:52 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/nova master: DO NOT REVIEW: Test OVN devstack module https://review.opendev.org/748226 | 14:53 |
| openstackgerrit | Alexandre Arents proposed openstack/nova master: Add a lock to prevent race during detach/attach of interface https://review.opendev.org/747957 | 14:57 |
| *** jangutter has joined #openstack-nova | 15:02 | |
| *** jangutter_ has quit IRC | 15:05 | |
| stephenfin | aarents++ Thanks | 15:07 |
| *** belmoreira has quit IRC | 15:17 | |
| *** derekh has quit IRC | 16:06 | |
| *** manuvakery has joined #openstack-nova | 16:11 | |
| *** k_mouza has quit IRC | 16:19 | |
| sean-k-mooney | stephenfin: just spoke to radolfo about a few thing one of them being the os-vif chanage for the defualt backend | 16:23 |
| sean-k-mooney | im going to fix the patch by lazy loadign the connect to ovs via a property | 16:23 |
| sean-k-mooney | can you take a look at the patch on monday | 16:24 |
| *** psachin has quit IRC | 16:24 | |
| *** admin0 has quit IRC | 16:25 | |
| *** tesseract has quit IRC | 16:27 | |
| *** k_mouza has joined #openstack-nova | 16:38 | |
| *** andrewbogott has joined #openstack-nova | 16:40 | |
| *** admin0 has joined #openstack-nova | 16:41 | |
| *** ralonsoh has quit IRC | 16:48 | |
| *** k_mouza has quit IRC | 16:48 | |
| *** k_mouza has joined #openstack-nova | 16:49 | |
| *** k_mouza has quit IRC | 16:53 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Make PCI claim NUMA aware during live migration https://review.opendev.org/748453 | 16:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: [WIP] Support SRIOV interface attach and detach https://review.opendev.org/740995 | 16:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM: tracing the PCI allocation code path https://review.opendev.org/748454 | 16:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use UUID as vif an network_id in vif tests https://review.opendev.org/748722 | 16:55 |
| openstackgerrit | sean mooney proposed openstack/os-vif master: deprecate ovs-vsctl driver and make native the default https://review.opendev.org/744816 | 16:56 |
| *** tosky has quit IRC | 17:03 | |
| *** tosky has joined #openstack-nova | 17:04 | |
| *** hamalq has joined #openstack-nova | 17:21 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: DNM: testing tempest copy-image test https://review.opendev.org/742547 | 17:29 |
| *** jsuchome has quit IRC | 17:36 | |
| *** bbowen has quit IRC | 17:40 | |
| *** tosky has quit IRC | 17:44 | |
| openstackgerrit | Merged openstack/nova stable/ussuri: Removes the delta file once image is extracted https://review.opendev.org/748034 | 18:35 |
| *** bnemec has quit IRC | 18:43 | |
| *** bnemec has joined #openstack-nova | 18:58 | |
| *** martinkennelly has quit IRC | 18:59 | |
| *** bnemec has quit IRC | 19:12 | |
| *** bbowen has joined #openstack-nova | 19:13 | |
| *** manuvakery has quit IRC | 19:20 | |
| *** bnemec has joined #openstack-nova | 19:29 | |
| *** nweinber has quit IRC | 19:45 | |
| *** Yumeng has quit IRC | 20:22 | |
| *** xek has joined #openstack-nova | 20:24 | |
| *** jamesdenton has quit IRC | 20:39 | |
| *** jamesdenton has joined #openstack-nova | 20:40 | |
| *** dave-mccowan has joined #openstack-nova | 20:59 | |
| *** jamesdenton has quit IRC | 21:07 | |
| *** jamesden_ has joined #openstack-nova | 21:07 | |
| *** raildo has quit IRC | 21:11 | |
| *** xek has quit IRC | 21:20 | |
| *** vishalmanchanda has quit IRC | 21:25 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/nova master: DO NOT REVIEW: Test OVN devstack module https://review.opendev.org/748226 | 21:30 |
| *** tosky has joined #openstack-nova | 21:33 | |
| *** bnemec has quit IRC | 21:36 | |
| *** dave-mccowan has quit IRC | 21:45 | |
| -openstackstatus- NOTICE: A zuul server ended up with read only filesystems which caused many jobs to hit retry_limit. The server has been rebooted and appears happy. Jobs can be rechecked. | 22:13 | |
| *** rcernin has joined #openstack-nova | 23:10 | |
| *** tosky has quit IRC | 23:13 | |
| *** rcernin has quit IRC | 23:15 | |
| *** bnemec has joined #openstack-nova | 23:26 | |
| *** hamalq has quit IRC | 23:28 | |
| *** bnemec has quit IRC | 23:35 | |
| *** bnemec has joined #openstack-nova | 23:46 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!