Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2025-06-26
17:31:52 gmaan I am thinking either we need to version (add different schema for each releases if changed) schema based on releases like we do for microversions by adding the info in current api-ref description
17:32:35 gmaan stephenfin: code behavior wise it is all good that is why i am +1. I am thinking what is usage of showing it in schema and how to handle the case where we can say it is gone after caracal
17:33:06 stephenfin So to be clear, you're worried about what the OpenAPI-derived replacement for api-ref will look like?
17:33:13 gmaan exactly
17:33:44 gmaan I do not want new OpenAPI-derived replacement show rdp as valid value
17:33:52 gmaan if it consume response schema as it is
17:34:19 stephenfin Well it is valid *if* you're using pre-Caracal and Hyper-V
17:34:46 stephenfin So we'll still document it, but in the description we'll say "rdp doesn't work since Caracal etc. etc.
17:34:57 stephenfin just like we do currently in the api-ref
17:36:49 stephenfin All of these schemas - query, request body, and response body - are going to have additional metadata layered on top in terms of the description field. We're not doing that yet because these reviews are already numerous and difficult enough, but the future api-ref will have to have all the same context the current one does in order to be useful
17:38:45 gmaan yeah, if openAPI will use the same content of field description and use schema as example show then we should update about rdp field here https://github.com/openstack/nova/blob/master/api-ref/source/parameters.yaml#L5986-L5999
17:39:21 gmaan so that if your change showing it in schema then updating current description about rdp should be done
17:40:05 gmaan and if new content for field description are generated ( from schema or so) by openapi then we need to manually review it and your comment in schema works fine
17:42:03 gmaan stephenfin: basically my request is to update the current api-ref description also if you are adding it in schema
17:42:10 stephenfin oh!
17:42:15 stephenfin I agree. I can do that
17:42:25 gmaan cool
21:28:37 opendevreview Stephen Finucane proposed openstack/nova master: api: Address issues with remote consoles APIs https://review.opendev.org/c/openstack/nova/+/952213
23:36:02 mikal sean-k-mooney: Just so you know, I've seen your review comments about release notes. I wrote some, but in the process noticed that the extra specs aren't working quite how I expected so I am now debugging those.
#openstack-nova - 2025-06-27
00:39:31 sean-k-mooney mikal: ack. im going ot sleep now but ill take another look tomrow or next week
00:49:02 opendevreview Cyril Roelandt proposed openstack/os-vif master: Zuul: do not use USE_PYTHON3 https://review.opendev.org/c/openstack/os-vif/+/953510
03:37:30 mikal sean-k-mooney: no worries. The good news it was me forgetting you need a very recent libvirt / qemu for virtio sound to actually work.
03:55:03 opendevreview Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for sound model. https://review.opendev.org/c/openstack/nova/+/926126
03:55:04 opendevreview Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for USB controller model. https://review.opendev.org/c/openstack/nova/+/927354
03:55:04 opendevreview Michael Still proposed openstack/nova master: Implement sound model extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/940770
03:55:05 opendevreview Michael Still proposed openstack/nova master: Implement USB controller extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/950643
07:11:13 gibi sean-k-mooney: when you are up, dansmith is +2 on https://review.opendev.org/c/openstack/nova/+/952966 can I get a +A?
07:11:21 gibi this is the py3.13 fix
07:19:41 opendevreview Kamil Sambor proposed openstack/nova master: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/949754
07:25:42 gibi is it just me or CI is mosty shit due to etcd fails to start with 'listen tcp 65.17.193.12:2379: bind: cannot assign requested address'
07:26:41 gibi https://bugs.launchpad.net/devstack/+bug/2115338
07:29:09 gibi maybe it is already fixed ^^:
07:32:35 gibi sean-k-mooney: also if you can look back to https://review.opendev.org/c/openstack/nova/+/953121 Dan is +2 as well. This is the build_and_run_instance error handling patch
09:08:20 priteau Hello. One of our customer experienced this bug: https://bugs.launchpad.net/nova/+bug/2074219. They are on Caracal. It made me realise the fix had not been backported from Epoxy. I submitted backports: https://review.opendev.org/c/openstack/nova/+/953468 and https://review.opendev.org/c/openstack/nova/+/953474
09:08:37 priteau Backports for 2024.2 first, I will submit 2024.1 once approved.
09:17:47 gibi priteau: thanks for the backports I'm +2. Could you change the gerrit topic to bug/2074219 for better tracking?
09:18:06 gibi elodilles: ^^ some easy backport reviews :)
09:21:54 priteau Thanks, done
09:42:03 elodilles gibi: ACK, will review it today when I get there o/
09:54:20 opendevreview Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475
11:42:41 sean-k-mooney gibi: done for the 3.13 patch ill look at the other one next
11:43:22 gibi sean-k-mooney: thanks!
11:43:46 sean-k-mooney gibi: i should know this but its been a while. in my devstack env what is the simplest way to run tempest?
11:44:13 sean-k-mooney do you happen to knwo of the top of your head. i knwo that it preconfigures it for you and you can use tox
11:44:27 sean-k-mooney is that the best way or just using the tempest command in one of the venvs?
11:44:43 sean-k-mooney i know i have done both before
11:45:11 gibi cd /opt/stack/tempest/
11:45:23 gibi source .tox/tempest/bin/activate
11:45:24 gibi tempest run
11:45:45 sean-k-mooney ack thanks i was not sure if that was still thge correct thing or not
11:45:52 gibi that works for me
11:46:24 sean-k-mooney ya i know that there used to be some script for it before they moved things to tox envs for the jobs and i knwo it has the idea of "workspaces" or whatever
11:46:39 gibi I never dig into workspaces
11:46:39 sean-k-mooney so i was not sure if i need to cd to a specific dir or something beyond the root of the repo
11:47:02 sean-k-mooney cool ill give that a go and if i have issue ill just ask gmann later
11:47:13 gibi btw the majority of the unit tests now run with threading in zuul https://review.opendev.org/c/openstack/nova/+/953475 I will work on the exclude list now
11:47:22 sean-k-mooney +2w on the rpc decorator change too
11:47:22 gibi dansmith: ^^ fyu
11:47:27 gibi thanks
11:48:17 sean-k-mooney oh cool sodo you need to set NOVA_RAISE_ON_GREENLET_LEAK=True again explcitly
11:48:32 sean-k-mooney im fine with doing that but shoul that not be inherited form {[testenv]setenv}
11:48:36 gibi I'm not sure about that any more I will check it
11:48:46 sean-k-mooney oh
11:48:49 gibi I'm not sure the env list is merged
11:48:51 sean-k-mooney sorry its false by default
11:48:59 sean-k-mooney so we need to turn it on in teh unit tests
11:49:17 gibi ahh I see
11:49:18 sean-k-mooney that why we also do it her https://review.opendev.org/c/openstack/nova/+/953475/2/tox.ini#64
11:49:29 sean-k-mooney well its not false exactly its undefiend
11:49:58 stephenfin sean-k-mooney: could you revisit https://review.opendev.org/c/openstack/nova/+/952213 when you have a moment. I respun it to address gmaan's comments
11:50:28 sean-k-mooney same effect. but cool so you hae a exuclded test list and i assume the plan is to burn that down over time
11:51:15 sean-k-mooney sure, i dont have a meeting for anout hour so if there are thing people want me to review between now and then ping away
11:52:36 priteau sean-k-mooney: What about https://review.opendev.org/c/openstack/nova/+/953468 and https://review.opendev.org/c/openstack/nova/+/953474 :)
11:52:57 priteau Although you are the author, unsure about the policy of approving your own patch as backports?
11:52:58 gibi yepp I'm working on those tests. Some will propably need to wait until we transform nova-compute
11:55:28 sean-k-mooney priteau: i opened the tabsl so sure if i have time (i should) ill do those next. although i shoudl kick off that tempest run in the background first
11:59:07 sean-k-mooney stephenfin: i think that is a good enhancement as well so +2w
11:59:22 stephenfin ta
12:00:49 sean-k-mooney i think there are 6 with +2w pending i belive https://review.opendev.org/c/openstack/nova/+/952284/3 is the fristwehre we need to re reivew
12:00:57 sean-k-mooney but i proably wont get to that today
12:01:24 sean-k-mooney ill try and loop back to it on tuesday.
12:02:28 sean-k-mooney priteau: ah that backport i saw the pin in my email but did not get to it yet.
12:05:10 priteau No worries if you don't, elodilles said he will check later
12:06:24 sean-k-mooney gibi: ya i think that is fine, it would be nice to get basic testing running in check and non voting
12:06:59 sean-k-mooney priteau: well im just done so perhaps it will have merged and they can look at the 2024.1 version
12:07:26 sean-k-mooney the orgianal bug was intorduced in 2024.1 so that shoudl be the last backprot requried
12:08:32 priteau yes, that's my plan to submit 2024.1 next. Thanks!
12:09:15 sean-k-mooney would you mind assigned the backprot ones to you https://bugs.launchpad.net/nova/+bug/2074219
12:09:44 sean-k-mooney technially there is nothigh that needs that i just like to do it
12:10:23 priteau done
12:11:21 sean-k-mooney cool feel free to ping once the patch is up and passed ci and i can take anohter look for 2024.1
13:00:07 sean-k-mooney is grenade-skip-level-always TIMED_OUT currently a thing that people are seeign or just a one off?
13:02:41 gibi grenade-skip-level-always timeout happens frequently, it times out while dumping the DBs but I found nothing speficic in any logs
13:39:50 gibi sean-k-mooney: dansmith: do you have any suggestions where to look why nova.tests.unit.test_baserpc.BaseAPITestCase.test_ping hangs at the rpc.call (the request never received on the server side) when run with tox, while it passing when run with stestr (same arg list) top of https://review.opendev.org/c/openstack/nova/+/953475
13:40:48 gibi and when it hands it hangs forever not just for an arbitrary timeout
13:41:17 gibi (OK I never waited forever but it is not stop at the normal testcase timeout)
13:57:46 Uggla gmann, sean-k-mooney can you please have a look at https://review.opendev.org/c/openstack/nova/+/952894 and https://review.opendev.org/c/openstack/nova/+/952895 this is a small bug fix, I'd like to close.
14:06:20 gibi ahh never mind, it hangs with stestr as well. I just forgot to turn the threading mode on in that terminal
14:11:11 opendevreview Tobias Urdin proposed openstack/nova master: Remove gabbi from test-requirements.txt https://review.opendev.org/c/openstack/nova/+/953578
14:26:16 opendevreview Takashi Kajinami proposed openstack/placement master: Use standard FPGA resource class https://review.opendev.org/c/openstack/placement/+/953580

Earlier   Later