pdeore | #startmeeting glance | 13:59 |
---|---|---|
opendevmeet | Meeting started Thu Aug 10 13:59:26 2023 UTC and is due to finish in 60 minutes. The chair is pdeore. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:59 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:59 |
opendevmeet | The meeting name has been set to 'glance' | 13:59 |
pdeore | #topic roll call | 13:59 |
pdeore | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 13:59 |
pdeore | o/ | 13:59 |
mrjoshi | o/ | 13:59 |
pdeore | lets wait few minutes for others to join | 14:00 |
pdeore | abhishekk, croelandt , dansmith | 14:00 |
dansmith | o/ | 14:01 |
dansmith | sorry | 14:01 |
abhishekk | o/ | 14:01 |
pdeore | ohh np :) | 14:01 |
pdeore | let's start | 14:01 |
pdeore | #topic Release/periodic jobs updates | 14:01 |
pdeore | M3 is just 3 weeks from now | 14:01 |
pdeore | I just hope there would not be any surprise on location api :P | 14:02 |
pdeore | Periodic jobs all green except cursive-tips jobs failed on py38 & py39 during last run | 14:02 |
pdeore | moving to next | 14:03 |
pdeore | #topic New Location API | 14:03 |
croelandt | o/ | 14:03 |
pdeore | As per the discussion during last meeting, Cinder POC to test new location api as been submitted by Rajat | 14:03 |
pdeore | #link https://review.opendev.org/c/openstack/cinder/+/890655/2 | 14:03 |
pdeore | croelandt, hey | 14:03 |
dansmith | so last I saw, whoami-rajat had some issue when trying to use it | 14:04 |
dansmith | was that resolved? | 14:04 |
pdeore | so while testing the location api via cinder, we had noticed a issues from glance side while cinder adding the location to an image but it's resolved now | 14:04 |
dansmith | okay | 14:04 |
pdeore | yeah it's resolved | 14:04 |
pdeore | It would be helpful if someone from nova team verifies these new apis from nova side | 14:05 |
abhishekk | I also verified it locally (cinder + glance) | 14:05 |
dansmith | so I was just looking, | 14:05 |
dansmith | we need to use a new method in the glanceclient to do this right? we need to make sure there's some way to identify that glance is too old to use the new api, so we fall back to the old one | 14:05 |
dansmith | is that not going to be handled in the glanceclient itself? | 14:05 |
dansmith | seems like the cinder poc doesn't handle that case in any way | 14:05 |
abhishekk | I think that should be part of client | 14:06 |
dansmith | me too, unless there's some reason it can't be | 14:06 |
abhishekk | no reason, it should be | 14:06 |
dansmith | okay so if that happens, then I think testing nova is as simple as a DNM depends-on patch with a zuul conf change to build glanceclient from git | 14:07 |
dansmith | I can put that up and confirm it fails (as expected) currently and then we can recheck that when it's fixed in the client | 14:07 |
abhishekk | +1 | 14:07 |
pdeore | ack | 14:07 |
pdeore | Thanks ! | 14:08 |
pdeore | so, the tempest tests for add location api are still inprogress , I've just submitted the tests with single store, multistore tests are pending, I will try to submit the remaining tests by tommorow EOD | 14:08 |
abhishekk | So dansmith I have one question, when did nova snapshot calls add location | 14:08 |
abhishekk | is it when nova and glance uses ceph | 14:09 |
dansmith | pdeore: looks like the tempest patch is failing everything currently | 14:09 |
pdeore | Till then it would be really helpful if you have a look at glance side changes, as functional test i've added are covering most of the scenarios | 14:09 |
abhishekk | or nova and glance use same pool of ceph? | 14:09 |
dansmith | abhishekk: yeah, so the nova-ceph-multistore job | 14:09 |
dansmith | abhishekk: only when they use the same pool | 14:09 |
pdeore | dansmith, yeah i just saw that, will check it after meeting | 14:09 |
abhishekk | means if glance is using images pool, nova should use images pool as well? | 14:09 |
dansmith | abhishekk: right, and if not, we just upload the snapshot like normal | 14:10 |
abhishekk | when I see confs generated by nova multistore I don't see nova and glance using same ceph pool | 14:10 |
abhishekk | yeah, that upload process I know | 14:10 |
dansmith | that's the point of that job... so news to me, but I can look | 14:11 |
abhishekk | I was just trying to figure out how nova and glance should access same pool, I think keyring need to be modified for same? | 14:11 |
abhishekk | Ack, I might be wrong but there is one parmaeter in nova which points to glance rbd store but the pool is different | 14:11 |
dansmith | abhishekk: look at the rbd options in [libvirt] here: https://78a70965f904893ce65b-6333727e8e4cab8f6cba85bb2fbb6639.ssl.cf5.rackcdn.com/890509/3/check/nova-ceph-multistore/d2d7dd4/controller/logs/etc/nova/nova-cpu_conf.txt | 14:12 |
abhishekk | images_rbd_pool = vms | 14:12 |
abhishekk | and in glance it is images | 14:12 |
abhishekk | images_rbd_glance_store_name = robust | 14:13 |
dansmith | right but does that means glance can't see things in the other pool? | 14:13 |
abhishekk | that I am not sure, I was under impression that glance should be using same pool as nova | 14:13 |
dansmith | let me find the logs where we tell glance about the location and see if those are in our logs for that job, we can take it outside the meeting | 14:14 |
abhishekk | cool | 14:14 |
abhishekk | pdeore, could you please post your tempest link here or on the agenda page? | 14:15 |
pdeore | #link https://review.opendev.org/q/topic:New-Location-Apis - here you will get all the loc api realted patches :) | 14:15 |
pdeore | #link https://review.opendev.org/c/openstack/tempest/+/891026 | 14:16 |
pdeore | tempest patch ^ | 14:16 |
abhishekk | this needs to depends on client patch as well | 14:16 |
pdeore | I will check and update after meeting | 14:17 |
pdeore | i don't think we need it, I have added the new add_loc method in te image_client, https://review.opendev.org/c/openstack/tempest/+/891026/1/tempest/lib/services/image/v2/images_client.py | 14:19 |
dansmith | tempest does not use the client | 14:19 |
abhishekk | ack | 14:20 |
abhishekk | then it is failing lot of jobs | 14:20 |
dansmith | yuuup | 14:21 |
abhishekk | and it should also check for min version required to test this job | 14:21 |
pdeore | yeah I've just quickly checked one job, but new tests are not failing there | 14:21 |
pdeore | abhishekk, yeah I was trying to find out for the existing version tests, but didn't get it | 14:22 |
pdeore | I will check it again and add it | 14:22 |
abhishekk | ack | 14:22 |
pdeore | shall we move ahead ? | 14:22 |
pdeore | ohk, moving ahead assuming it's yes :) | 14:23 |
pdeore | #topic Review Priority | 14:23 |
pdeore | #link https://review.opendev.org/q/topic:sqlalchemy-20+project:openstack/glance - SQLAlchemy 2.0 migration patches | 14:23 |
pdeore | please have a look, I think most of the patches got +W | 14:24 |
pdeore | abhishekk, croelandt Thanks ! | 14:24 |
pdeore | moving to next .. | 14:25 |
pdeore | #topic Importatnt Bugs ? | 14:25 |
pdeore | croelandt, anything you want to highlight here ? | 14:25 |
croelandt | not really :) | 14:26 |
pdeore | cool :) | 14:27 |
pdeore | let's move to open discussions ten | 14:27 |
pdeore | #topic Open Discussions | 14:27 |
abhishekk | when will be the last review party | 14:27 |
pdeore | 24th Aug | 14:27 |
pdeore | anyone has anything else to discuss ? | 14:29 |
abhishekk | we generally has it two weeks before, right | 14:29 |
mrjoshi | #link https://review.opendev.org/c/openstack/openstacksdk/+/881939 #link https://review.opendev.org/c/openstack/python-openstackclient/+/878631 | 14:29 |
mrjoshi | #link https://review.opendev.org/c/openstack/python-openstackclient/+/883494 | 14:29 |
abhishekk | that means it should be next week | 14:29 |
croelandt | we could move it to next week if you'd like | 14:29 |
mrjoshi | abhishekk, croelandt need you attention on these patches | 14:29 |
abhishekk | mrjoshi, ack, will have a look | 14:29 |
pdeore | we have it one week before but yea let's move it to next week | 14:29 |
croelandt | pdeore: ok :) | 14:30 |
croelandt | mrjoshi: I'll look at them | 14:30 |
croelandt | we can also include them in next week'sreview :) | 14:30 |
mrjoshi | thanks! | 14:30 |
abhishekk | :D | 14:30 |
pdeore | :) | 14:30 |
pdeore | anything else ? or we can wrap up early ? | 14:31 |
croelandt | let's wrap up! | 14:31 |
pdeore | ack :) | 14:31 |
pdeore | Thanks everyone for joining!! | 14:31 |
pdeore | #endmeeting | 14:32 |
opendevmeet | Meeting ended Thu Aug 10 14:32:06 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:32 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2023/glance.2023-08-10-13.59.html | 14:32 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2023/glance.2023-08-10-13.59.txt | 14:32 |
opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2023/glance.2023-08-10-13.59.log.html | 14:32 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!