| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-04-23 | |||
| 15:27:35 | bauzas | melwitt: actually, apologies for the silly question but do you know the difference between https://github.com/openstack/nova/tree/master/nova/virt/hyperv and https://github.com/openstack/compute-hyperv/ ? | |
| 15:27:35 | bauzas | melwitt: actually, apologies for the silly question but do you know the difference between https://github.com/openstack/nova/tree/master/nova/virt/hyperv and https://github.com/openstack/compute-hyperv/ ? | |
| 15:28:39 | melwitt | bauzas: I don't really but lpetrut explains the differences in the patch comment and etherpad. the in tree driver doesn't use os-assisted-volume-snapshots but the out of tree driver does, which is just one example | |
| 15:28:39 | melwitt | bauzas: I don't really but lpetrut explains the differences in the patch comment and etherpad. the in tree driver doesn't use os-assisted-volume-snapshots but the out of tree driver does, which is just one example | |
| 15:28:56 | bauzas | melwitt: okay, then I'll fence the patch above | |
| 15:28:56 | bauzas | melwitt: okay, then I'll fence the patch above | |
| 15:31:23 | melwitt | I kind of wish we could just remove in tree drivers in these cases where "no one is using it", seems pointless to have it in the code | |
| 15:31:23 | melwitt | I kind of wish we could just remove in tree drivers in these cases where "no one is using it", seems pointless to have it in the code | |
| 15:33:19 | lyarwood | agreed | |
| 15:33:19 | lyarwood | agreed | |
| 15:33:25 | bauzas | that's a reasonable concern and we should somehow understand what are the benefits of keeping such code | |
| 15:33:25 | bauzas | that's a reasonable concern and we should somehow understand what are the benefits of keeping such code | |
| 15:34:44 | bauzas | looking at the compute-hyperv repo, this sounds a fork | |
| 15:34:44 | bauzas | looking at the compute-hyperv repo, this sounds a fork | |
| 16:10:07 | artom | sean-k-mooney, so for my own education - if the MTU is going down, telling the guest OS via some means (DHCP?) is enough, no? Because incoming packets will presumably have the new smaller MTU, and so will "fit" on the tap device. So the guest just needs to know to reduce the size of what it sends. | |
| 16:10:07 | artom | sean-k-mooney, so for my own education - if the MTU is going down, telling the guest OS via some means (DHCP?) is enough, no? Because incoming packets will presumably have the new smaller MTU, and so will "fit" on the tap device. So the guest just needs to know to reduce the size of what it sends. | |
| 16:10:45 | artom | The problem comes if the MTU goes up. In that case, the tap device does need to have its MTU increased, to accommodate the larger incoming packets. | |
| 16:10:46 | artom | The problem comes if the MTU goes up. In that case, the tap device does need to have its MTU increased, to accommodate the larger incoming packets. | |
| 16:13:37 | sean-k-mooney | artom: it will be eventually | |
| 16:13:37 | sean-k-mooney | artom: it will be eventually | |
| 16:13:41 | sean-k-mooney | artom: but we already do this | |
| 16:13:41 | sean-k-mooney | artom: but we already do this | |
| 16:14:04 | sean-k-mooney | artom: neutron advertises the mtu to the guest in dhcp for quite a long time | |
| 16:14:04 | sean-k-mooney | artom: neutron advertises the mtu to the guest in dhcp for quite a long time | |
| 16:14:17 | sean-k-mooney | and it will update the value when you update it in the api | |
| 16:14:17 | sean-k-mooney | and it will update the value when you update it in the api | |
| 16:14:38 | sean-k-mooney | but the guest wont ask for a dhcp update until half its remaining lease is avaible | |
| 16:14:38 | sean-k-mooney | but the guest wont ask for a dhcp update until half its remaining lease is avaible | |
| 16:14:50 | sean-k-mooney | a normal leas is between 1-7 days | |
| 16:14:50 | sean-k-mooney | a normal leas is between 1-7 days | |
| 16:15:04 | artom | Ah, and there's no "gratuitous DHCP" | |
| 16:15:04 | sean-k-mooney | so it would be multiple days typeicaly until it was informed | |
| 16:15:04 | artom | Ah, and there's no "gratuitous DHCP" | |
| 16:15:04 | sean-k-mooney | so it would be multiple days typeicaly until it was informed | |
| 16:15:14 | sean-k-mooney | artom: not that im aware of | |
| 16:15:14 | sean-k-mooney | artom: not that im aware of | |
| 16:15:19 | sean-k-mooney | brb getting a drink | |
| 16:15:19 | sean-k-mooney | brb getting a drink | |
| 16:17:19 | artom | "There is a FORCERENEW message that your DHCP server can issue to clients: | |
| 16:17:19 | artom | https://www.ietf.org/rfc/rfc3203.txt | |
| 16:17:19 | artom | Your clients have to support this mechanism, of course." | |
| 16:17:19 | artom | "There is a FORCERENEW message that your DHCP server can issue to clients: | |
| 16:17:19 | artom | https://www.ietf.org/rfc/rfc3203.txt | |
| 16:17:19 | artom | Your clients have to support this mechanism, of course." | |
| 16:17:24 | artom | https://networkengineering.stackexchange.com/questions/25618/dhcp-option-26-mtu-queries | |
| 16:17:24 | artom | https://networkengineering.stackexchange.com/questions/25618/dhcp-option-26-mtu-queries | |
| 16:22:51 | artom | sean-k-mooney, err, the query string is encrypted in SSL | |
| 16:22:51 | artom | sean-k-mooney, err, the query string is encrypted in SSL | |
| 16:23:04 | artom | The host isn't for SNI | |
| 16:23:04 | artom | The host isn't for SNI | |
| 16:23:10 | artom | But the query stirng is | |
| 16:23:10 | artom | But the query stirng is | |
| 16:30:10 | sean-k-mooney | oh ok | |
| 16:30:16 | sean-k-mooney | so ssl is the fix? | |
| 16:30:16 | sean-k-mooney | so ssl is the fix? | |
| 16:34:59 | artom | sean-k-mooney, for MTIM snooping, yes | |
| 16:34:59 | artom | sean-k-mooney, for MTIM snooping, yes | |
| 16:35:03 | artom | *MITM | |
| 16:35:03 | artom | *MITM | |
| 16:48:49 | lyarwood | melwitt: https://review.opendev.org/c/openstack/nova/+/757307 - would you mind checking this out today if you have time after ptg? | |
| 16:48:49 | lyarwood | melwitt: https://review.opendev.org/c/openstack/nova/+/757307 - would you mind checking this out today if you have time after ptg? | |
| 16:51:41 | melwitt | lyarwood: sure | |
| 16:51:41 | melwitt | lyarwood: sure | |
| 16:58:33 | lyarwood | thanks | |
| 22:00:13 | openstackgerrit | Dmitrii Shcherbakov proposed openstack/nova-specs master: Introduce Transport Nodes https://review.opendev.org/c/openstack/nova-specs/+/787458 | |
| #openstack-nova - 2021-04-24 | |||
| 02:38:08 | openstackgerrit | Merged openstack/nova stable/train: libvirt: Increase incremental and max sleep time during device detach https://review.opendev.org/c/openstack/nova/+/757307 | |
| 02:38:08 | openstackgerrit | Merged openstack/nova stable/train: libvirt: Increase incremental and max sleep time during device detach https://review.opendev.org/c/openstack/nova/+/757307 | |
| 09:37:07 | openstackgerrit | Merged openstack/nova master: guestfs: With libguestfs >= v1.41.1 decode returned bytes to string https://review.opendev.org/c/openstack/nova/+/787712 | |
| 09:37:07 | openstackgerrit | Merged openstack/nova master: guestfs: With libguestfs >= v1.41.1 decode returned bytes to string https://review.opendev.org/c/openstack/nova/+/787712 | |
| 09:37:38 | openstackgerrit | Merged openstack/nova master: Switch to new rolevar for run-tempest role https://review.opendev.org/c/openstack/nova/+/775040 | |
| 09:37:38 | openstackgerrit | Merged openstack/nova master: Switch to new rolevar for run-tempest role https://review.opendev.org/c/openstack/nova/+/775040 | |
| 15:55:30 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/wallaby: guestfs: With libguestfs >= v1.41.1 decode returned bytes to string https://review.opendev.org/c/openstack/nova/+/787839 | |
| 15:55:30 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/wallaby: guestfs: With libguestfs >= v1.41.1 decode returned bytes to string https://review.opendev.org/c/openstack/nova/+/787839 | |
| 17:48:36 | openstackgerrit | Merged openstack/nova master: Move instance power state check to _detach_with_retry https://review.opendev.org/c/openstack/nova/+/778918 | |
| 17:48:36 | openstackgerrit | Merged openstack/nova master: Move instance power state check to _detach_with_retry https://review.opendev.org/c/openstack/nova/+/778918 | |
| #openstack-nova - 2021-04-25 | |||
| 02:47:56 | openstackgerrit | Wenping Song proposed openstack/python-novaclient master: Dropping lower constraints testing https://review.opendev.org/c/openstack/python-novaclient/+/787860 | |
| 02:47:56 | openstackgerrit | Wenping Song proposed openstack/python-novaclient master: Dropping lower constraints testing https://review.opendev.org/c/openstack/python-novaclient/+/787860 | |
| 03:00:48 | openstackgerrit | Wenping Song proposed openstack/placement master: Dropping lower constraints testing https://review.opendev.org/c/openstack/placement/+/787863 | |
| 03:00:48 | openstackgerrit | Wenping Song proposed openstack/placement master: Dropping lower constraints testing https://review.opendev.org/c/openstack/placement/+/787863 | |
| 03:15:29 | openstackgerrit | Wenping Song proposed openstack/nova master: Dropping lower constraints testing https://review.opendev.org/c/openstack/nova/+/787866 | |
| 03:15:29 | openstackgerrit | Wenping Song proposed openstack/nova master: Dropping lower constraints testing https://review.opendev.org/c/openstack/nova/+/787866 | |
| 20:34:27 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: guestfs: With libguestfs >= v1.41.1 decode returned bytes to string https://review.opendev.org/c/openstack/nova/+/787901 | |
| 20:34:27 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: guestfs: With libguestfs >= v1.41.1 decode returned bytes to string https://review.opendev.org/c/openstack/nova/+/787901 | |
| 20:34:50 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ussuri: guestfs: With libguestfs >= v1.41.1 decode returned bytes to string https://review.opendev.org/c/openstack/nova/+/787902 | |
| 20:34:50 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ussuri: guestfs: With libguestfs >= v1.41.1 decode returned bytes to string https://review.opendev.org/c/openstack/nova/+/787902 | |
| 20:35:12 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: guestfs: With libguestfs >= v1.41.1 decode returned bytes to string https://review.opendev.org/c/openstack/nova/+/787903 | |
| #openstack-nova - 2021-04-26 | |||
| 07:55:54 | lyarwood | elod / bauzas / melwitt ; https://review.opendev.org/q/project:openstack%252Fnova+branch:stable%252Fwallaby+status:open - morning, I've got a few backports proposed to wallaby already, would you be able to take a look this week? Thanks in advance! | |
| 07:55:54 | lyarwood | elod / bauzas / melwitt ; https://review.opendev.org/q/project:openstack%252Fnova+branch:stable%252Fwallaby+status:open - morning, I've got a few backports proposed to wallaby already, would you be able to take a look this week? Thanks in advance! | |
| 07:59:02 | bauzas | lyarwood: sure, will do | |
| 07:59:02 | bauzas | lyarwood: sure, will do | |
| 07:59:14 | bauzas | lyarwood: thanks for working on it | |
| 07:59:15 | bauzas | lyarwood: thanks for working on it | |
| 08:02:22 | elod | lyarwood: I'll also prioritize them! | |
| 08:02:23 | elod | lyarwood: I'll also prioritize them! | |
| 08:04:53 | lyarwood | thanks both | |
| 08:04:53 | lyarwood | thanks both | |
| 08:38:56 | xinranwang | hi gibi and alex_xu, I have re-proposed the smartnic spec in xena release with some changes according to the discussion under the implementation patch, the overall workflow is as same as wallaby's one. Could you please help to review it when you got time？ https://review.opendev.org/c/openstack/nova-specs/+/783632 | |
| 08:38:56 | xinranwang | hi gibi and alex_xu, I have re-proposed the smartnic spec in xena release with some changes according to the discussion under the implementation patch, the overall workflow is as same as wallaby's one. Could you please help to review it when you got time？ https://review.opendev.org/c/openstack/nova-specs/+/783632 | |
| 08:39:37 | xinranwang | @gibi | |
| 08:39:37 | xinranwang | @gibi | |