| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-12-15 | |||
| 12:58:16 | gibi | I knew I hit this before but I alway forget | |
| 12:59:17 | sean-k-mooney | gibi: i woudl just make it all UPPERCASE and remvoe the undersocres | |
| 12:59:44 | sean-k-mooney | like the class constants we have in the func test for api version and such | |
| 13:26:22 | sean-k-mooney | bauzas: elodilles: this was rebased https://review.opendev.org/c/openstack/nova/+/816883 can you +w to get it moving again | |
| 13:33:00 | opendevreview | Merged openstack/nova master: [rt] Apply migration context for incoming migrations https://review.opendev.org/c/openstack/nova/+/820549 | |
| 13:36:29 | elodilles | sean-k-mooney: done | |
| 13:36:40 | sean-k-mooney | elodilles: thanks | |
| 13:36:49 | elodilles | no problem :) | |
| 13:37:47 | sean-k-mooney | we had some downstream interest in it so just wanted to relay it upstream | |
| 14:00:11 | erlon | sean-k-mooney: hey, can you get some time to review https://review.opendev.org/c/openstack/nova/+/821840? Its the regression test for https://review.opendev.org/c/openstack/nova/+/815324 | |
| 14:00:31 | erlon | sean-k-mooney: please let me know if this is what you were thinking | |
| 14:01:21 | erlon | also, I need a piece of advice. I cant make it work, and it seems that the mock I'm using is just not mocking. I tried several ways, one is commented | |
| 14:37:54 | sean-k-mooney | erlon: not quite. you seam to be trying to create the reguression test more or less form scratch | |
| 14:38:41 | sean-k-mooney | erlon: i would have started by using one of our libvirt funtal test case base classes | |
| 14:39:07 | sean-k-mooney | also functional test should not really be mocking the way this is | |
| 14:39:18 | sean-k-mooney | this is closer to a unit test | |
| 14:40:35 | sean-k-mooney | erlon: so i proably should have started form 1897483 | |
| 14:40:40 | sean-k-mooney | * https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/base.py#L31 | |
| 14:41:14 | sean-k-mooney | or from integrated_helpers._IntegratedTestBase | |
| 14:41:37 | sean-k-mooney | if it was not a libvirt specific issue | |
| 14:42:59 | sean-k-mooney | erlon: this is an example live migration functional test | |
| 14:43:01 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/tests/functional/regressions/test_bug_1939545.py | |
| 15:21:26 | opendevreview | Merged openstack/nova stable/wallaby: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/816883 | |
| 19:26:02 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/816927 | |
| #openstack-nova - 2021-12-16 | |||
| 06:44:57 | opendevreview | mitya-eremeev-2 proposed openstack/nova master: Delete bogus attachments. https://review.opendev.org/c/openstack/nova/+/820935 | |
| 08:25:37 | opendevreview | Balazs Gibizer proposed openstack/nova stable/xena: Reproduce bug 1953359 https://review.opendev.org/c/openstack/nova/+/820550 | |
| 08:25:39 | opendevreview | Balazs Gibizer proposed openstack/nova stable/xena: [rt] Apply migration context for incoming migrations https://review.opendev.org/c/openstack/nova/+/820553 | |
| 08:25:40 | opendevreview | Balazs Gibizer proposed openstack/nova stable/xena: Extend the reproducer for 1953359 and 1952915 https://review.opendev.org/c/openstack/nova/+/821941 | |
| 08:29:15 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Reproduce bug 1953359 https://review.opendev.org/c/openstack/nova/+/820554 | |
| 08:30:28 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Extend the reproducer for 1953359 and 1952915 https://review.opendev.org/c/openstack/nova/+/821943 | |
| 08:31:41 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: [rt] Apply migration context for incoming migrations https://review.opendev.org/c/openstack/nova/+/820555 | |
| 08:38:07 | opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: Reproduce bug 1953359 https://review.opendev.org/c/openstack/nova/+/820558 | |
| 08:53:35 | opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: Extend the reproducer for 1953359 and 1952915 https://review.opendev.org/c/openstack/nova/+/820856 | |
| 08:54:45 | opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: [rt] Apply migration context for incoming migrations https://review.opendev.org/c/openstack/nova/+/820559 | |
| 09:11:50 | opendevreview | mitya-eremeev-2 proposed openstack/nova master: If no old resources in migration context. https://review.opendev.org/c/openstack/nova/+/821947 | |
| 09:36:43 | Gues__________________________ | Morning All :) | |
| 09:36:48 | Gues__________________________ | arrghh. brb | |
| 09:45:25 | rmart04 | Morning... Again! | |
| 09:49:52 | rmart04 | Hope you are all well, had a quick question. We've recently upgraded from Stein to Train and we've noticed some issues after live migrations with no connectivity until a certain threshold on our network is hit (up to 2 minutes). It's lead mer here > https://bugs.launchpad.net/neutron/+bug/1866139 and I just wanted to confirm if this Is this still an issue? | |
| 11:53:21 | plibeau2 | lyarwood: I have apply a part of proposition. I have put some question on patchset3. -> https://review.opendev.org/c/openstack/nova/+/820531 | |
| 13:59:09 | opendevreview | Gustavo Santos proposed openstack/nova stable/wallaby: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/821978 | |
| 14:01:19 | opendevreview | mitya-eremeev-2 proposed openstack/nova master: Handle no old resources in migration context. Sometimes after live migration a cleanup cannot check VPMEM resources because migration context is missing the attribute 'old_resources'. It's better to log it than fail migration. https://review.opendev.org/c/openstack/nova/+/821947 | |
| 14:02:21 | opendevreview | mitya-eremeev-2 proposed openstack/nova master: Handle no old resources in migration context. https://review.opendev.org/c/openstack/nova/+/821947 | |
| 14:04:22 | opendevreview | Gustavo Santos proposed openstack/nova stable/victoria: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/821980 | |
| 14:12:40 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/816927 | |
| 14:19:08 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/816927 | |
| 14:29:57 | rmart04 | Hope you are all well, had a quick question. We've recently upgraded from Stein to Train and we've noticed some issues after live migrations with no connectivity until a certain threshold on our network is hit (up to 2 minutes). It's lead mer here > https://bugs.launchpad.net/neutron/+bug/1866139 and I just wanted to confirm if this Is this still an issue? Hoping Sean-k-mooney might be around :) | |
| 14:34:50 | sean-k-mooney | rmart04: there are still some issue with this depening on whatn network backend you are using. ml2/ovs? with or without iptables? ml2/ovn? | |
| 14:35:04 | rmart04 | ml2 with security groups yeah :/ | |
| 14:35:15 | rmart04 | ml2 /ovs with security groups yeah :/ | |
| 14:35:41 | sean-k-mooney | rmart04: well ovs security groups or iptables | |
| 14:35:50 | sean-k-mooney | there are two sg drivers for ml2/ovs | |
| 14:35:55 | rmart04 | lemme check | |
| 14:37:26 | rmart04 | yeah iptables_firewall.OVSHybridIptablesFirewallDriver | |
| 14:40:36 | rmart04 | ] | |
| 14:46:49 | opendevreview | Gustavo Santos proposed openstack/nova stable/ussuri: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/821987 | |
| 15:25:17 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Fix check job failures https://review.opendev.org/c/openstack/python-novaclient/+/821991 | |
| 15:32:24 | sean-k-mooney | rmart04: i belive in master that is now fix | |
| 15:32:52 | sean-k-mooney | rmart04: by https://review.opendev.org/c/openstack/neutron/+/766277 | |
| 15:33:26 | sean-k-mooney | rmart04: that is backproted to train https://review.opendev.org/c/openstack/nova/+/770844 | |
| 15:34:06 | sean-k-mooney | sorry wrong link | |
| 15:34:40 | sean-k-mooney | looks like it was not cherry picked on the neutron side | |
| 15:35:00 | sean-k-mooney | there are patches for ussuri https://review.opendev.org/c/openstack/neutron/+/821443 | |
| 15:36:02 | sean-k-mooney | while that will help it may not fully reslove your issue sbut it should help | |
| 15:36:13 | rmart04 | Ah OK thanks for the info that's' really helpful | |
| 15:36:29 | rmart04 | Do you know any tempoary hackarounds whilst we get to ussuri? | |
| 15:37:10 | sean-k-mooney | not without a code chagne | |
| 15:44:11 | rmart04 | oki dokie | |
| 15:47:19 | rmart04 | Would that be a tiny patch, or quite involved? | |
| 17:09:01 | sean-k-mooney | gibi: gmann its not qa releated but while i was reviewing the ooo change with the policy overried i notece we have a coule of plices for apis that proably can be or already have been removed | |
| 17:09:35 | sean-k-mooney | like the api exetion list https://github.com/openstack/nova/blob/stable/wallaby/nova/policies/extensions.py | |
| 17:09:50 | sean-k-mooney | im not sure if we still have it but we have not had api exstions in a long time | |
| 17:10:04 | sean-k-mooney | eventually we could make that 404 and delete the policy | |
| 17:10:18 | gmann | sean-k-mooney: API is there so that is why we kept policy too but extensions one is static and not0usable information now | |
| 17:10:22 | sean-k-mooney | on the ohter hand it cost us little to maintain it so not worth the effort to remove | |
| 17:10:39 | sean-k-mooney | ya | |
| 17:10:45 | gmann | we kept this not to break people/script if anyone using it but I am fine to remove the extensions API particular | |
| 17:11:30 | sean-k-mooney | ya its just legacy tech debt | |
| 17:11:35 | sean-k-mooney | right now it cost us little | |
| 17:12:03 | gmann | yeah, its just there but we can cleanup that with tech debt in next cycle or so | |
| 17:12:26 | gmann | I hope we could remove proxy APIs too but that will break people :) | |
| 17:14:31 | sean-k-mooney | gmann: well as someoen that supprot raising our min service version im ok with that eventually | |
| 17:15:26 | gmann | yeah, that one. raising min version which is difficult | |
| 18:03:45 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/816927 | |
| 20:43:47 | opendevreview | Gabriel Silva Trevisan proposed openstack/nova stable/ussuri: Reproduce bug 1953359 https://review.opendev.org/c/openstack/nova/+/822047 | |
| 20:55:24 | opendevreview | Gabriel Silva Trevisan proposed openstack/nova stable/ussuri: Extend the reproducer for 1953359 and 1952915 https://review.opendev.org/c/openstack/nova/+/822048 | |
| 21:03:35 | opendevreview | Erlon R. Cruz proposed openstack/nova master: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/815324 | |
| 21:03:35 | opendevreview | Erlon R. Cruz proposed openstack/nova master: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/821840 | |
| 21:30:21 | opendevreview | Gabriel Silva Trevisan proposed openstack/nova stable/ussuri: [rt] Apply migration context for incoming migrations https://review.opendev.org/c/openstack/nova/+/822050 | |
| 22:31:09 | opendevreview | Jonathan Race proposed openstack/nova master: Adds Emulation for aarch64, and ppc64 on x86_64 processors https://review.opendev.org/c/openstack/nova/+/822053 | |
| #openstack-nova - 2021-12-17 | |||
| 15:31:42 | opendevreview | Merged openstack/python-novaclient master: Updating python testing classifier as per Yoga testing runtime https://review.opendev.org/c/openstack/python-novaclient/+/819208 | |
| 16:03:21 | gibi_pto_back_on_10th | as per my nick I will not official at work until 10th of January next year. Still you might see me here occasionally if I get bored :) | |
| 16:12:44 | sean-k-mooney | more or less the same | |
| 16:13:12 | sean-k-mooney | i wont be signed in to my work laptop over that peiord but i do get irc pings on my table via matrix | |
| 16:13:35 | sean-k-mooney | so i might see pings but likely wont until im back on the 4th | |
| 16:14:14 | sean-k-mooney | gibi_pto_back_on_10th: im currently updating the health check spec and incoperating some design element form https://inadarei.github.io/rfc-healthcheck/#RFC8259 | |
| 16:14:29 | sean-k-mooney | gibi_pto_back_on_10th: but no rush to review it | |
| 16:14:43 | sean-k-mooney | ill push it before i leave today but it can wait until we are all back | |
| 16:23:10 | gibi_pto_back_on_10th | sean-k-mooney: OK, I will take a look at some point :) | |
| 16:29:28 | bauzas | gibi: happy new year ! | |