Earlier  
Posted Nick Remark
#openstack-nova - 2021-12-15
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 !
16:29:36 gibi_pto_back_on_10th bauzas: same to you too
16:30:01 bauzas have good holidays :)
16:33:04 gmann gibi_pto_back_on_10th: happy holiday and take rest
16:34:42 gmann bauzas: good to have your ack in this (placement IRC channel retiring ) https://review.opendev.org/c/openstack/project-config/+/821889
16:35:03 bauzas gmann: sure
16:35:10 gmann thanks
16:35:29 gibi_pto_back_on_10th gmann: same to you

Earlier   Later