| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-02-24 | |||
| 18:52:53 | sean-k-mooney | but ml2/ovn didnt populated it proeprly | |
| 18:53:15 | sean-k-mooney | i think that is still a gap in tehr migration tooling but it might have got fixed in the last year or two | |
| 18:53:38 | danfai | oh, ok. I'll have a more detailed look on the neutron side then. We are still in yoga, so... | |
| 18:54:00 | sean-k-mooney | danfai: for what ite work codl migration shoudl still work i think | |
| 18:54:12 | sean-k-mooney | its been a long time since i tried to fix this | |
| 18:54:27 | mikal | Morning | |
| 18:54:34 | sean-k-mooney | danfai: if your using vlan networking with linux breige yoru move will be less painful | |
| 18:54:45 | danfai | cold migration works flawlessly indeed. also two live migrations (we only have provider networks though) | |
| 18:54:48 | sean-k-mooney | danfai: but just be aware ovn does not supprot vxlan or gre tunnes | |
| 18:54:53 | mikal | sean-k-mooney: I feel like I should recheck the spice-direct final patch now that melwitt's tempest fix has landed. Do you see any reason to not do that? | |
| 18:55:10 | sean-k-mooney | mikal:i already did | |
| 18:55:14 | sean-k-mooney | did it fail again | |
| 18:55:21 | sean-k-mooney | mikal: i have started merging your code | |
| 18:55:39 | mikal | sean-k-mooney: doh, I mismashed the refresh button because I am pre coffee. | |
| 18:55:45 | mikal | Nah, it passed. | |
| 18:56:04 | mikal | Ugh, arm passed. | |
| 18:56:12 | mikal | I will be quiet now. | |
| 18:56:22 | sean-k-mooney | mikal: its still in check i think | |
| 18:56:45 | sean-k-mooney | so it failed on the funtional jobs and weill need a recheck | |
| 18:56:58 | sean-k-mooney | but https://zuul.opendev.org/t/openstack/build/1098d7514ba843e28a5477c69892b176 the hybird plug job passed again | |
| 18:57:05 | danfai | yes, regarding vxlans, this was mentioned in the meeting we had with operators. | |
| 18:57:20 | sean-k-mooney | mikal: rechecks wont work until it repors back so lets just let it complete | |
| 18:57:29 | mikal | sean-k-mooney: will do | |
| 18:58:12 | sean-k-mooney | danfai: the main issue iwht the migration scripts that neutron provides is geneve requires a small mtu on the client or extra overhead on the underlying network then vxlan | |
| 18:59:09 | danfai | hmm, I thought there was more, like ID mappings. I'll add the MTU to the notes, thanks | |
| 18:59:44 | sshmlbrg | sean-k-mooney: Hi! Did u see my request on review? | |
| 19:00:00 | sean-k-mooney | danfai: nova does not supprot updatign the mtu on conencted ports for what its worth | |
| 19:00:11 | sean-k-mooney | danfai: the mtu change wil only take effect after a hard reboot | |
| 19:00:22 | sean-k-mooney | danfai: https://github.com/openstack/neutron/commit/baaf240ce3f7802fe1431cc13913b9d93fc7f742 that is the ovn bridge name patch | |
| 19:01:53 | sean-k-mooney | sshmlbrg: yep. we will need to add some nova functioanl tests to replciate the manual procedure you followed in the bug report | |
| 19:02:03 | danfai | Thank you very much | |
| 19:02:52 | sean-k-mooney | sshmlbrg: unit test are good to have but for operations like shelve we try to also have functional tests. and for bugs we prefer to create a functional test that show the broken behavior as an addtion patch and then have the fix as a second commit | |
| 19:03:48 | sean-k-mooney | sshmlbrg: without loading all the context im not sure that the finally block is corect | |
| 19:04:08 | sean-k-mooney | sshmlbrg: you should not be modifying the az of the instnace directly | |
| 19:04:45 | sshmlbrg | sean-k-mooney: ok, thanks. I will add functional tests as a first commit | |
| 19:04:46 | sean-k-mooney | sshmlbrg: we do not really use the az colume in the cell database to store the az of the instance anymore. | |
| 19:05:31 | sean-k-mooney | sshmlbrg: cool i would od it be creating a new file in https://github.com/openstack/nova/tree/master/nova/tests/functional/regressions | |
| 19:05:45 | sean-k-mooney | for the bug ill see if i can quickly find an exmaple for you to copy | |
| 19:06:11 | sean-k-mooney | sshmlbrg: perhaps https://github.com/openstack/nova/blob/master/nova/tests/functional/regressions/test_bug_2025480.py | |
| 19:06:24 | sshmlbrg | sean-k-mooney: then will need to change the methods used and add the az argument to them | |
| 19:07:04 | sean-k-mooney | updatign the object in memory may be ok its the save that is a little suspect | |
| 19:07:24 | sean-k-mooney | we have a temporay mutation context manager | |
| 19:07:44 | sean-k-mooney | i that we may want to condier | |
| 19:07:53 | sean-k-mooney | we can dicuss that on the review | |
| 19:08:22 | sshmlbrg | sean-k-mooney: thank you for example! ok, sounds good | |
| 19:08:51 | sean-k-mooney | if you look at similar files in the same directory there shoudl be a boot form volume example | |
| 19:09:32 | sean-k-mooney | or other shelve example so it should not be too hard to create a repoducer following the steps in the bug | |
| 19:11:46 | sshmlbrg | sean-k-mooney: i got it. I am gonna return with changes tomorrow. Have a nice day or evening :) | |
| 19:12:43 | sean-k-mooney | sshmlbrg: https://github.com/openstack/nova/blob/375d95565d594182c5472f710b3f7bd53f0746d4/nova/tests/functional/regressions/test_bug_1675570.py#L26 | |
| 19:12:47 | sean-k-mooney | that a better example | |
| 19:13:10 | sean-k-mooney | sshmlbrg: o/ | |
| 19:14:19 | sean-k-mooney | sshmlbrg: that actully attaching a server to a shelve offloaded instnace | |
| 19:14:35 | sean-k-mooney | the reason its not failing is cross az attach is allowed by defult | |
| 19:14:58 | sean-k-mooney | so if you were to set that to false in yoru version i woudl eeepct the attach to fail and repoduce the bug :) | |
| 19:16:25 | sean-k-mooney | mikal: https://zuul.openstack.org/status?change=924844&change=926876&change=926877 we will need to do a few rechecks... but we can kick thos off again shortly | |
| 19:16:43 | sshmlbrg | sean-k-mooney: cool idea, I'll look into this | |
| 19:17:26 | sean-k-mooney | sshmlbrg: let us know if you have issues there is a self.flags function aviable on the test case to set fconfig options | |
| 19:17:42 | sean-k-mooney | if you grep for flags you will see examples of how to use it | |
| 19:19:42 | sean-k-mooney | bauzas: RuntimeError: detected leaked greenlets in nova.tests.functional.libvirt.test_vgpu.VGPUMultipleTypesTests.test_create_servers_with_specific_type | |
| 19:20:12 | sean-k-mooney | i dont know how often tthat fails but that ocatinal does | |
| 19:21:37 | sean-k-mooney | mikal: ^ that the error in the funtional test on the third patfch so its not related ot your code | |
| 19:24:08 | sean-k-mooney | Uggla: https://review.opendev.org/c/openstack/releases/+/941862 need your +1 again i updated the version to 4.1.0 based on elodilles comment on the release channel | |
| 19:25:39 | mikal | sean-k-mooney: are you going to do the rechecks or should I? | |
| 19:26:14 | sean-k-mooney | either works ill be finsihing in the next 20 mins or so but ill preodiclly check in | |
| 19:26:34 | mikal | Well, we' | |
| 19:26:49 | mikal | Well, we're still waiting for them to finish right now anyways. | |
| 19:27:13 | sean-k-mooney | ya. if multiple people recheck it wotn kick it out of check | |
| 19:27:48 | sean-k-mooney | i dont it migh kick it out of gate i dont recally so we should be carful in that case | |
| 19:29:36 | sean-k-mooney | mikal: over all the results look pretty green the nova-grenade-multinode failure in gate on the first patch was a kernel panic | |
| 19:29:48 | sean-k-mooney | that not related to yoru code so just bad luck | |
| 19:30:51 | sean-k-mooney | mikal: sometime rechecking at a less busy time help with things like that but its a bug in our cirros image we think | |
| 19:31:23 | mikal | sean-k-mooney: without checking, I feel like cirros hasn't done a release in ages? | |
| 19:31:45 | mikal | Huh, more recent than I expected. September last year. | |
| 19:31:53 | sean-k-mooney | its been a while there is a 6.x branch based on ubuntu 22.04 or maybe 24.04 | |
| 19:32:06 | mikal | The author now works at chainguard IIRC. | |
| 19:32:10 | sean-k-mooney | i dont recall if they have compelte the rebase on the 24.04 kernel yet | |
| 19:32:47 | sean-k-mooney | we have been considering other distos for testign but few if any really meet the min ram requirements we have | |
| 19:33:26 | sean-k-mooney | gento recently started providign cloud-images iwth cloud ini but they are like 930+ MBs | |
| 19:33:39 | sean-k-mooney | vs the like 20-50mb of a cirros image | |
| 19:33:54 | mikal | I have been toying with the idea of a specialised UEFI binary for SPICE testing and skipping the OS entirely. I guess it depends what functionality you need from the instance. | |
| 19:34:20 | mikal | Also, I have way more random tangents than I will ever have time to persue. | |
| 19:34:28 | sean-k-mooney | ya so ironic were not supper ok with the idea of alpine because they worried about he use of musle libc | |
| 19:34:49 | sean-k-mooney | nova does not really care about that which is why i was prposin git | |
| 19:35:04 | sean-k-mooney | mikal: for tempest we need a small usbset of utils | |
| 19:35:16 | sean-k-mooney | toybox/busybox mostly covers it | |
| 19:35:34 | sean-k-mooney | although i belive we use gnu core utils version in cirros today | |
| 19:36:26 | sean-k-mooney | mikal: i think frickler has been somehwat invovled in cirros mantaince lately but im not reallly sure | |
| 19:37:15 | sean-k-mooney | frickler: do you know if they are still working on a 0.7.x release with 24.04 kernel and package rebase? | |
| 19:37:18 | mikal | sean-k-mooney: they certainly did the most recent release. | |
| 19:39:17 | sean-k-mooney | mikal: i know we are hoping to do a release with https://github.com/cirros-dev/cirros/pull/122 after epoxy is released | |
| 19:39:47 | sean-k-mooney | before would also be fine but as of expoxy nova will be able to create vms with igb nic which supprot emulated sriov in the guest | |
| 19:40:10 | sean-k-mooney | mikal: we are hopping to be able to do some sriov tesing in ci once our providers are capabliy fo supproting that | |
| 19:40:43 | sean-k-mooney | it will be a while before thats realsitc however | |
| 20:31:38 | Uggla | sean-k-mooney, elodilles, https://review.opendev.org/c/openstack/releases/+/941862 +1 again. | |
| 20:36:40 | frickler | sean-k-mooney: I'm not aware of anyone actively working on cirros, but I could do a bit if needed. that PR should be in 0.6.3 though? | |
| 20:39:25 | opendevreview | Luis Enrique Morales Mendez proposed openstack/nova master: Add 'LOG' to archive_deleted_rows function https://review.opendev.org/c/openstack/nova/+/942441 | |
| 21:25:49 | opendevreview | Merged openstack/nova master: Reproducer for bug 2098892 https://review.opendev.org/c/openstack/nova/+/942402 | |
| 21:57:26 | sean-k-mooney | frickler: 0.6.3 is using hte latest (at the time) 22.04 hwe kernel | |
| 21:57:36 | sean-k-mooney | frickler: but not 24.04 | |
| 21:57:56 | sean-k-mooney | frickler: unless you ment the igb change which might be in 0.6.3 | |