bauzas | sean-k-mooney: yeah, apparently the mirrors were fixed | 06:56 |
---|---|---|
bauzas | that's why I uploaded again my series (because I don't have the same kernel version (I use a virtual host) | 06:56 |
bauzas | and yeah, bindep is not installing automatically mdevctl | 06:57 |
opendevreview | Sylvain Bauza proposed openstack/nova master: WIP : Add mtty support to nova-next https://review.opendev.org/c/openstack/nova/+/922140 | 07:50 |
bauzas | sean-k-mooney: if you're here, I'm curious why in nova-next, compute1 doesn't get my series https://9eed2832462b93d99d63-9a2c0e6b238d9eb37d7a57d0c5074dee.ssl.cf1.rackcdn.com/openstack/93034cf7903d4f8abe5fa23883c761b8/compute1/logs/screen-n-cpu.txt | 10:37 |
bauzas | the controller uses it correctly : https://9eed2832462b93d99d63-9a2c0e6b238d9eb37d7a57d0c5074dee.ssl.cf1.rackcdn.com/openstack/93034cf7903d4f8abe5fa23883c761b8/controller/logs/screen-n-cpu.txt | 10:38 |
sean-k-mooney | the same git repo will be copied to all node by the workspace prepare but my guess is you have not configured something correctly in the devstack config | 10:39 |
sean-k-mooney | mel had to fix the job config to turn on the compilation fo the sampels in the comptue node for example https://review.opendev.org/c/openstack/nova/+/948581/7/.zuul.yaml | 10:40 |
sean-k-mooney | bauzas we can confirm if it clooned the corrct content in the logs lets dot that fist but i think that is highly unlikely to be the problem here | 10:43 |
bauzas | doh, I'll then set the compilation in the subnode section | 10:45 |
bauzas | sean-k-mooney: fwiw, I just updated my WIP to install mdevctl | 10:45 |
bauzas | so melwitt's series shouldn't be needed | 10:45 |
bauzas | s/series/changre | 10:45 |
sean-k-mooney | ack. she was working on that because you wre on pto to see if she could move it forward but she didnt intent to keep working on it. she just didnt want to directly update your patch | 10:46 |
sean-k-mooney | 2025-05-20 08:30:05.668555 | compute1 | + functions-common:git_clone:676 : cd /opt/stack/nova | 10:50 |
sean-k-mooney | 2025-05-20 08:30:05.673150 | compute1 | + functions-common:git_clone:677 : git show --oneline | 10:51 |
sean-k-mooney | 2025-05-20 08:30:05.673265 | compute1 | + functions-common:git_clone:677 : head -1 | 10:51 |
sean-k-mooney | 2025-05-20 08:30:05.691351 | compute1 | 6e752ff1c3 WIP : Add mtty support to nova-next | 10:51 |
sean-k-mooney | 2025-05-20 08:30:05.695399 | compute1 | + functions-common:git_clone:678 : cd /opt/stack/devstack | 10:51 |
sean-k-mooney | 2025-05-20 08:30:05.699127 | compute1 | + lib/nova:install_nova:984 : setup_develop /opt/stack/nova | 10:51 |
sean-k-mooney | so on the compute its definetly usign your change | 10:51 |
sean-k-mooney | devstack git clone say the git repo already existed (because it was prepared by zuul) and skip the clone and just used it | 10:51 |
sean-k-mooney | bauzas: i guess that may be an issue with the error message | 10:53 |
sean-k-mooney | i.e. if it returns https://9eed2832462b93d99d63-9a2c0e6b238d9eb37d7a57d0c5074dee.ssl.cf1.rackcdn.com/openstack/93034cf7903d4f8abe5fa23883c761b8/compute1/logs/screen-n-cpu.txt | 10:53 |
sean-k-mooney | when the mtty device is not found | 10:53 |
sean-k-mooney | its conflating incorrect or invalid with not found so we can proably make that more obvious | 10:53 |
sean-k-mooney | bauzas: anyway let see if enaableign the plugin on the comptue helps and if that resolves teh error | 10:56 |
sean-k-mooney | i can take a look again when we have new results | 10:56 |
opendevreview | Sylvain Bauza proposed openstack/nova master: WIP : Add mtty support to nova-next https://review.opendev.org/c/openstack/nova/+/922140 | 12:18 |
bauzas | stupid me again, forgot to upload it | 12:18 |
opendevreview | Lajos Katona proposed openstack/os-vif master: VS Trunk: Add bridge_name to external_ids https://review.opendev.org/c/openstack/os-vif/+/949736 | 13:16 |
dansmith | bauzas: gibi: Can you guys fast approve this trivial revert of the glance location thing? https://review.opendev.org/c/openstack/nova/+/950336?usp=email | 13:45 |
dansmith | as I suspected, not enough testing on the glance side was actually done and this is triggering a behavioral change and tempest fails | 13:45 |
sean-k-mooney | i can do it fi you want | 13:46 |
dansmith | yep, thanks | 13:46 |
bauzas | +Wd too | 13:56 |
opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Re-propose "libvirt: AMD SEV-ES support" for 2025.2 https://review.opendev.org/c/openstack/nova-specs/+/950429 | 14:25 |
opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Allow using the system page (4KiB) https://review.opendev.org/c/openstack/nova/+/950430 | 14:37 |
Uggla | Nova meeting in ~55mn | 15:05 |
*** ralonsoh is now known as ralonsoh_out | 15:08 | |
Uggla | Nova meeting in ~5mn | 15:55 |
Uggla | #startmeeting nova | 16:01 |
opendevmeet | Meeting started Tue May 20 16:01:41 2025 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:01 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:01 |
opendevmeet | The meeting name has been set to 'nova' | 16:01 |
Uggla | Hello everyone | 16:01 |
gibi | o/ | 16:02 |
dansmith | o/ | 16:02 |
gmaan | o/ | 16:02 |
masahito | o/ | 16:02 |
bauzas | o/ | 16:02 |
fwiesel | o/ | 16:02 |
Uggla | #topic Bugs (stuck/critical) | 16:03 |
elodilles | o/ | 16:04 |
Uggla | #info No Critical bug | 16:04 |
Uggla | #info https://bugs.launchpad.net/nova/+bug/2110545 from last week. | 16:04 |
Uggla | #info We merged https://review.opendev.org/c/openstack/nova/+/949622 proposal to skip the the problematic job. | 16:04 |
gibi | we probably need to decrease the severity of it now that it is not blocking the gate | 16:04 |
gibi | moved it to High now | 16:04 |
Uggla | ok I was about to ask about it | 16:05 |
Uggla | https://review.opendev.org/c/openstack/nova/+/922140 bauzas working on a patch to enable nova-next. | 16:05 |
Uggla | bauzas, am i correct with ^ ? | 16:05 |
bauzas | I still need to understand why compute1 doesn't like me | 16:05 |
bauzas | but let's not discuss my problem now | 16:05 |
bauzas | I would appreciate some devstack/zuul expert to help me finding the cause that's it | 16:06 |
Uggla | cool, it is on a good path to be resolved. | 16:07 |
bauzas | tldr: multi-node's compute1 apparently doesn't get the right repo branch | 16:07 |
Uggla | bauzas, anything else you want to add ? | 16:07 |
Uggla | seems not so moving on | 16:08 |
Uggla | #topic Gate status | 16:08 |
Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:08 |
Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:08 |
Uggla | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status | 16:08 |
Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:09 |
bauzas | Uggla: sorry, nope | 16:09 |
Uggla | #info Please try to provide meaningful comment when you recheck | 16:09 |
Uggla | #topic tempest-with-latest-microversion job status | 16:09 |
Uggla | #link https://zuul.opendev.org/t/openstack/builds?job_name=tempest-with-latest-microversion&skip=0 | 16:10 |
Uggla | gmaan, do you want to say something about it ? | 16:10 |
gmaan | one update, keypair test passing. I was working for server tags tests which need more update as its schema missing since starting | 16:11 |
gmaan | it in-progress https://review.opendev.org/c/openstack/tempest/+/949682 | 16:11 |
gmaan | sorry instance action | 16:11 |
gmaan | that is all for this week | 16:11 |
Uggla | gmaan, thx | 16:11 |
Uggla | #topic Release Planning | 16:11 |
Uggla | #link https://releases.openstack.org/flamingo/schedule.html | 16:11 |
Uggla | #info Nova deadlines are set in the above schedule | 16:12 |
Uggla | #info Only 2 weeks before Nova Spec Soft Freeze, do not forget to submit your specs. | 16:12 |
Uggla | #topic Review priorities | 16:12 |
Uggla | #link https://etherpad.opendev.org/p/nova-2025.2-status | 16:12 |
Uggla | FYI I have requested #openstack-nova channel ops for sean-k-mooney, bauzas, gibi, dansmith, melwitt, Uggla | 16:12 |
gibi | thanks1 | 16:13 |
sean-k-mooney | ack. i suspect we will very rarely need to use that | 16:13 |
sean-k-mooney | but it would have been nice to have been able to fix the topic before | 16:13 |
Uggla | I think we should start reviewing and approving specs. | 16:14 |
gibi | yeah it is only for the topic | 16:14 |
bauzas | noted | 16:14 |
Uggla | Let's start with : https://review.opendev.org/c/openstack/nova-specs/+/945549, https://review.opendev.org/c/openstack/nova-specs/+/947542, https://review.opendev.org/c/openstack/nova-specs/+/949504 | 16:15 |
Uggla | The 2 last ones are reproposal, so it will be glad to approved them. | 16:15 |
Uggla | The first one is about cloud hypervisor. It was reviewed by Dan | 16:16 |
Uggla | But that would cool to have feedbacks from another core. | 16:16 |
dansmith | fwiesel: I reviewed that spec, but I'm not going to approve it, FWIW | 16:16 |
sean-k-mooney | i pushed my pending comments... | 16:16 |
dansmith | oops, sorry Uggla: ^ | 16:16 |
sean-k-mooney | i really need to stop forgetign to do that | 16:17 |
dansmith | I don't have time to really review the implementation in detail and so I don't want to be the +2 on the spec | 16:17 |
sean-k-mooney | if i recall correctly they have set up a ppa to rpovide the libvirt integration. | 16:17 |
Uggla | cool, the author pinged me and was worried, so that's cool if he has new feedback. | 16:17 |
sean-k-mooney | that solves part of the how can we test this in ci aspect | 16:17 |
sean-k-mooney | i think were we are right now is | 16:19 |
sean-k-mooney | without seeign the code and some work towards the integration | 16:19 |
Uggla | Also FYI I think the document is up2date regarding the current spec status, so feel free to pick specs for review. | 16:19 |
sean-k-mooney | we cant really make much more of a judvemnt call on the desgin | 16:19 |
masahito | hi, we started to update the supporting trait tracking by the provider.yaml feature nova-spec with PoC. https://review.opendev.org/c/openstack/nova-specs/+/937587 | 16:19 |
sean-k-mooney | so we coudl approve the spec btu we cant really commit to landign the feature until they start building it out and imporantly the devstack supprot and ci for the same | 16:20 |
Uggla | sean-k-mooney, sure I think approving the spec is good. So author can "check" that box. | 16:21 |
sean-k-mooney | Uggla: the vtpm spec is on my radar to review this week. if the cinder one actully makes progress i dont have issues with that either but i have not reviewd it since last cycle. im not sure if we really want to go into details on those in real time | 16:21 |
gibi | is it a general rule that we should not +2 a spec if we are not commiting to review the impl? | 16:21 |
* gibi trying to figure out how to vote | 16:22 | |
sean-k-mooney | in the past no. | 16:22 |
dansmith | sean-k-mooney: agreed, I think we need to see a little sniff of the implementation in CI | 16:22 |
dansmith | gibi: not a hard rule, I just think for something like this I'm going to avoid putting my name on such an effort | 16:22 |
sean-k-mooney | but there is at least some expecation i guess that if you do +2 and the authour has questions you will at least try to guide them | 16:22 |
gibi | dansmith: ack | 16:22 |
sean-k-mooney | dansmith: i thinik if you know you wont have time then signaling that is also valuable | 16:23 |
gibi | sean-k-mooney: yeah by voting on the spec I agree I will be around to answer questions about the spec | 16:23 |
Uggla | Something to add ? | 16:25 |
sean-k-mooney | i think its good to call out the spec that are ready for review but i think we need to actully do the review async | 16:25 |
sean-k-mooney | so unless you have specific question i would suggest we move on and if we want to have a spec review day | 16:25 |
gibi | +1 | 16:26 |
sean-k-mooney | we can do that but do that at/around the soft spec freeeze in june | 16:26 |
Uggla | sure this topic is just to highlight that specs are waiting and that we should start to review them not to be overwhelmed. | 16:26 |
sean-k-mooney | masahito: your sepc also has a poc implemantion if i recall correct | 16:27 |
sean-k-mooney | i belive gibi has alrady started to look at it too | 16:27 |
masahito | yup. | 16:27 |
masahito | is it okay to add it into the priority? | 16:27 |
sean-k-mooney | masahito: since you are here did you have any questions for us that you wanted input on | 16:27 |
sean-k-mooney | you can certenlly put it in the etherpad as ready for review if you think it is | 16:28 |
gibi | sean-k-mooney: yeah I looked at the PoC and I liked what I saw | 16:28 |
masahito | nothing now. i think the spec and poc need first review round now. | 16:28 |
sean-k-mooney | gibi: ack i opened it but didnt get to read it yet. masahito: ack | 16:28 |
Uggla | and so moving on. | 16:29 |
Uggla | #topic OpenAPI | 16:29 |
Uggla | #link: https://review.opendev.org/q/topic:%22openapi%22+(project:openstack/nova+OR+project:openstack/placement)+-status:merged+-status:abandoned | 16:29 |
sean-k-mooney | masahito: please add it here https://etherpad.opendev.org/p/nova-2025.2-status#L34 | 16:29 |
Uggla | 21 patches remaining. | 16:29 |
sean-k-mooney | actully slightly more | 16:30 |
sean-k-mooney | stephen as a branch on his github fork with all of them | 16:30 |
sean-k-mooney | but is propsoing them in batches. i was intended to review them today but didnt get to it so i will be reviewign them tomrrow | 16:30 |
masahito | sean-k-mooney: got it. thanks | 16:31 |
sean-k-mooney | im hoping gmaan or other will also have time in the next week or two to reviwe some of them | 16:31 |
Uggla | This topic is to follow progress on openapi. Because we said in PTG we would try to close it. | 16:31 |
gmaan | yeah, I will try if I can do during end of this week otherwise next week sometime | 16:31 |
Uggla | I have reviewed 2 of them. | 16:32 |
gmaan | I would not be able to review all of them together as it exhaust eyes checking all fields for many APIs :) but I will target some set | 16:32 |
Uggla | gmaan, I expect to progress of them week after week. | 16:33 |
sean-k-mooney | ya i can ususlly do 5-10 over the course of an hour or two and then i need to clear my mind | 16:33 |
sean-k-mooney | they are not hard persay but its hard to maintain concentration on them for long periods of time | 16:33 |
gmaan | yeah | 16:33 |
Uggla | yep my idea is to do a first pass on them and then if gmaan and sean-k-mooney you can review and merge that would be great. | 16:34 |
Uggla | any as time is flying, I'd like to move on the next topic. | 16:35 |
Uggla | #topic Stable Branches | 16:35 |
Uggla | elodilles, the floor is yours. | 16:35 |
elodilles | ack, thanks, not so much things this time | 16:35 |
elodilles | #info stable branches (stable/2025.1 and stable/2024.*) seem to be in OK state | 16:36 |
elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:36 |
elodilles | that's all from me | 16:36 |
elodilles | Uggla: back to you | 16:36 |
Uggla | thx elodilles | 16:36 |
elodilles | np | 16:36 |
Uggla | #topic vmwareapi 3rd-party CI efforts Highlights | 16:37 |
Uggla | fwiesel, anything to say ? | 16:37 |
fwiesel | Hi, no updates from my side. | 16:37 |
Uggla | ok | 16:37 |
Uggla | #topic Gibi's news about eventlet removal. | 16:37 |
Uggla | #link Series: https://gibizer.github.io/categories/eventlet/ | 16:37 |
Uggla | gibi, do you want to say something ? I think you are chasing for reviews. | 16:38 |
gibi | yepp | 16:38 |
gibi | the nova-scheduler series is ready for core review | 16:38 |
gibi | (I've added it to the status etherpad as well) | 16:39 |
gibi | the series starts here https://review.opendev.org/c/openstack/nova/+/947966 | 16:39 |
gibi | the first 11 patches are ready to go, the last two depends on oslo.service to land and release the threading backend support | 16:39 |
gibi | that is it | 16:40 |
Uggla | cool thx gibi | 16:40 |
Uggla | #topic Open discussion | 16:41 |
Uggla | I can see topic on the agenda. | 16:41 |
Uggla | s/can/can't/ | 16:41 |
Uggla | so unless if we have points, maybe we can try to triage bugs. | 16:42 |
Uggla | #topic Bug scrubbing | 16:43 |
Uggla | #link: https://etherpad.opendev.org/p/nova-bug-selection-for-triaging#L4 | 16:43 |
Uggla | First one : https://bugs.launchpad.net/nova/+bug/2110222 | 16:44 |
Uggla | Nova server image uploaded to S3 uses singlepart instead of multipart upload | 16:45 |
gibi | that feels like a feature request | 16:45 |
Uggla | yep I was about to say the same. | 16:45 |
Uggla | it is more an optimization request. | 16:46 |
Uggla | So I could probably set the bug to invalid and ask for a blueprint. | 16:47 |
gibi | it is a small feature even a specless one if we can assume that we can pass the size to the glance client at uploat | 16:47 |
Uggla | ok I'll do that for this one. | 16:48 |
Uggla | Next one: https://bugs.launchpad.net/nova/+bug/2109727 - removing an host from an aggregate should be accepted even if the host no longer exists | 16:48 |
Uggla | I think it is a valid one as opened by Sylvain | 16:48 |
Uggla | bauzas, ^ | 16:50 |
gibi | I hope bauzas could push a functional reproducer for that | 16:51 |
bauzas | sorry was taking a break | 16:51 |
bauzas | not sure if I have time for the reproducer but I can try | 16:52 |
sean-k-mooney | bauzas: i guess that only matteres if your going to work on the bug | 16:52 |
Uggla | bauzas, at least this is something valid. | 16:52 |
sean-k-mooney | if not then the peroson who picks it up can | 16:52 |
Uggla | So I'll move the bug to valid. | 16:54 |
Uggla | Next one : https://bugs.launchpad.net/nova/+bug/2108974 - Keypairs lost during cross-cell resize in instance_extra | 16:54 |
opendevreview | Merged openstack/nova master: Revert "Support glance's new location API" https://review.opendev.org/c/openstack/nova/+/950336 | 16:54 |
gibi | it has reproducer steps so somebody should run them and if reproducible mark it valid | 16:55 |
Uggla | Is resizing cross-cell supported ? | 16:56 |
dansmith | yeah | 16:56 |
dansmith | it uses shelve on the backend | 16:57 |
dansmith | and it has to copy between databases, so it might be missing something | 16:57 |
gibi | The config drive associated with the server, if there is one, will be re-generated on the destination host in the target cell. Therefore if the server was created with personality files they will be lost. However, this is no worse than evacuating a server that had a config drive when the source and destination compute host are not on shared storage or when shelve offloading and unshelving a server | 16:58 |
gibi | with a config drive. If necessary, the resized server can be rebuilt to regain the personality files. | 16:58 |
gibi | so maybe such config drive regen is the problem | 16:59 |
gibi | (the above is from https://docs.openstack.org/nova/latest/admin/configuration/cross-cell-resize.html#limitations ) | 16:59 |
dansmith | ah yep | 16:59 |
gibi | the bug says it is lost form metadata response as well | 17:01 |
Uggla | yep I understand it is not only config drive. | 17:01 |
gibi | and it also points to `select keypairs from dst_cell.instance_extra ` I did not know we have a copy of a keypair there | 17:01 |
gibi | but it says it is in the source cell | 17:02 |
gibi | so that is probably not copied to the dest cell DB | 17:02 |
gibi | I vote that this is valid | 17:02 |
Uggla | I can try to reproduce, but I'm unsure how to easily deploy a multi cell env. | 17:02 |
gibi | we have multicell job we just need to see if we can enable some tempest there to catch this | 17:03 |
sean-k-mooney | thats in the sepc for what its porth | 17:04 |
sean-k-mooney | at least im 99% sure we called out that any files injected in the config drive would be lost with a cross cell resize | 17:05 |
sean-k-mooney | https://specs.openstack.org/openstack/nova-specs/specs/ussuri/implemented/cross-cell-resize.html#known-issues | 17:06 |
sean-k-mooney | limiation 3 | 17:06 |
sean-k-mooney | Servers created with personality files, commonly known as file injection, that are resized across cells will lose the personality files since they are not persisted in the database | 17:06 |
gibi | sean-k-mooney: keypairs are not personality files | 17:06 |
gibi | afaik | 17:06 |
dansmith | correct | 17:06 |
dansmith | but they mentioned instance_extra | 17:07 |
dansmith | so if there's something we're not copying there and then regen'ing the configdrive... | 17:07 |
sean-k-mooney | oh it the keypari | 17:07 |
sean-k-mooney | ok that shoudl be copied and regenerated | 17:07 |
sean-k-mooney | i actully otught keyparis were in teh api db | 17:08 |
sean-k-mooney | not the cell db but i guess we embded them when we first create teh instance in the cell db? | 17:08 |
dansmith | keypairs themselves are I think (maybe) but the instance's attachment to them is in the main one of course | 17:08 |
sean-k-mooney | ack | 17:08 |
sean-k-mooney | im wondering if we could see this in ci. i think this is valid in anycase | 17:09 |
sean-k-mooney | we could create a repodcuer functional tests. in thoery tempest coudl detect this also | 17:10 |
sean-k-mooney | we just need to curl the metadata after the resize | 17:10 |
sean-k-mooney | or check the config drive i guess | 17:10 |
sean-k-mooney | presumable it workign now because the keypari is already in the authorized keys | 17:10 |
Uggla | sean-k-mooney, I'd like to try to do a reproducer. | 17:11 |
gibi | yepp we should be able to catch this in tempest | 17:11 |
sean-k-mooney | the workaround for now is just pass the keypari again on rebuild i guess | 17:12 |
Uggla | So if you all agree I will set this bug to valid and I'll try to do a reproducer. But I may need your help. | 17:12 |
sean-k-mooney | ack, sure shout if you hit issues | 17:12 |
Uggla | I will also update the bug with the workaround and remind limitation. | 17:13 |
Uggla | We are overtime, so I think we can wrap for today. | 17:14 |
sean-k-mooney | im not sure if we have multi cell functional tests today so that could be "fun" to make work | 17:14 |
sean-k-mooney | reproducing it in tempest will be simpler in this specific case. | 17:14 |
sean-k-mooney | oh we do https://github.com/openstack/nova/blob/master/nova/tests/functional/test_cross_cell_migrate.py | 17:15 |
dansmith | matt riedemann did that work, so I was *sure* there was functional testing for it :D | 17:15 |
sean-k-mooney | ack. getting diffent config to work in the fucntionla tests is still a bit fo a pain so i was unsure if we did or not | 17:16 |
Uggla | anything else ? | 17:17 |
Uggla | Thanks all and thanks for the extended time for bug triage. | 17:17 |
Uggla | #endmeeting | 17:17 |
Uggla | #endmeeting | 17:17 |
opendevmeet | Meeting ended Tue May 20 17:17:40 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:17 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-05-20-16.01.html | 17:17 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-05-20-16.01.txt | 17:17 |
opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-05-20-16.01.log.html | 17:17 |
elodilles | thanks o/ | 17:18 |
masahito | thank | 17:19 |
gibi | Uggla: thanks | 17:21 |
Uggla | you are welcome. | 17:21 |
sean-k-mooney | Uggla: so https://github.com/openstack/nova/blob/master/nova/tests/functional/test_cross_cell_migrate.py#L816-L826 test_cold_migrate_target_host_in_other_cell | 17:24 |
sean-k-mooney | get you to the point where the vm has be moved to a differnt cell | 17:24 |
sean-k-mooney | you shoudl be able to use that as a baseline to create a repoducer | 17:24 |
Uggla | ok but it is a fn test right ? | 17:25 |
Uggla | You have said that a tempest one should be easier. | 17:25 |
Uggla | Am i wrong ? | 17:25 |
sean-k-mooney | that was assumign we did nto have functional test coverage for this which we do | 17:26 |
sean-k-mooney | so it should be pretty simple to extend since all the infra is in place | 17:26 |
Uggla | ok I will try to reproduce with the fn ones though. | 17:27 |
sean-k-mooney | _resize_and_validate accepts a server and returns server, source_rp_uuid, target_rp_uuid, old_flavor, new_flavor | 17:27 |
sean-k-mooney | so you can create a server with a key pair | 17:27 |
sean-k-mooney | assert that its in the first cell db | 17:28 |
sean-k-mooney | then call that to move it | 17:28 |
sean-k-mooney | and then check the second db | 17:28 |
sean-k-mooney | i think that woudl work | 17:28 |
Uggla | sean-k-mooney, OK I'll try. | 17:29 |
*** jcosmao is now known as Guest16423 | 19:26 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!