| *** soniya29|ruck is now known as soniya29|ruck|afk | 10:33 | |
| *** soniya29|ruck|afk is now known as soniya29|ruck | 11:01 | |
| *** soniya29|ruck is now known as soniya29|ruck|afk | 11:27 | |
| *** soniya29|ruck|afk is now known as soniya29|ruck | 12:46 | |
| *** dasm|off is now known as dasm | 12:53 | |
| pdeore | #startmeeting glance | 14:00 |
|---|---|---|
| opendevmeet | Meeting started Thu Jul 14 14:00:10 2022 UTC and is due to finish in 60 minutes. The chair is pdeore. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'glance' | 14:00 |
| pdeore | #topic roll call | 14:00 |
| pdeore | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:00 |
| pdeore | o/ | 14:00 |
| dansmith | o/ | 14:00 |
| mrjoshi | o/ | 14:00 |
| pdeore | lets wait few minutes for everyone to join | 14:01 |
| akekane_ | o/ | 14:02 |
| pdeore | shall we start? | 14:03 |
| akekane_ | I think lets start | 14:03 |
| pdeore | ok | 14:03 |
| pdeore | #topic release/periodic jobs | 14:04 |
| akekane_ | rosmaita, jokke_ will join us in middle | 14:04 |
| pdeore | It's M2 Release Week and we have released glance-store 4.0.1 & python-glanclient 4.0.1 yesterday | 14:04 |
| pdeore | glance m2 release patch is up for review | 14:04 |
| pdeore | #link https://review.opendev.org/c/openstack/releases/+/849782 | 14:04 |
| pdeore | But there is one patch in pipeline, it might get merged in some time, so we need to update the hash | 14:04 |
| akekane_ | we need to wait for couple of patches to be merged | 14:05 |
| akekane_ | 1 is cyrils patch and other is release notes patc | 14:05 |
| akekane_ | * patch | 14:05 |
| pdeore | ohh yeah the releasenote one is also there :) | 14:05 |
| akekane_ | yes | 14:05 |
| pdeore | I will update the hash once those are done.. | 14:06 |
| akekane_ | cool | 14:06 |
| jokke_ | is there -w vote already on that patch so release team does not go ahead and just do it? | 14:06 |
| pdeore | ohh yes, I will do it | 14:07 |
| akekane_ | they don't do it unless there is confirmation from project member | 14:07 |
| akekane_ | but -w is better option to avoid any casualties | 14:07 |
| pdeore | Done | 14:08 |
| akekane_ | immediate cache patch is also up and all tests including tempest are passing | 14:08 |
| jokke_ | akekane_: if ptl or release liaison proposes the patch the bot will automatically give it the PTL approval vote | 14:08 |
| akekane_ | yep, | 14:08 |
| akekane_ | patch is proposed by pdeore | 14:09 |
| akekane_ | but -w is better option to avoid any risk :D | 14:09 |
| pdeore | ack, moving ahead | 14:10 |
| pdeore | Periodic Jobs, all green except the 'translation update' jobs failing with RETRY_LIMIT | 14:10 |
| akekane_ | cool | 14:10 |
| akekane_ | anything else to add here? | 14:11 |
| pdeore | It's retrying for the virtual environment which is not getting created because ensurepip is not available | 14:11 |
| akekane_ | ok | 14:11 |
| akekane_ | its not glance related error, so lets not worry about it atm | 14:12 |
| pdeore | ack | 14:12 |
| pdeore | so, that's it from me :) | 14:12 |
| pdeore | #topic Glance-replicator | 14:12 |
| pdeore | This one is added by Cyril | 14:12 |
| akekane_ | Cyril has added this topic | 14:12 |
| akekane_ | we came across these patches last week during review party | 14:13 |
| akekane_ | AFAIK our glance-replicator is broken since Icehouse | 14:13 |
| akekane_ | and I also didn't see anyone is using this | 14:13 |
| akekane_ | so the question here is | 14:13 |
| akekane_ | should we deprecate replicator or should we fix it? | 14:14 |
| dansmith | what does it do? | 14:14 |
| akekane_ | I am in favor of removing it | 14:14 |
| dansmith | er, what *did* it do? | 14:14 |
| akekane_ | IMO, it replicates the image in another region | 14:14 |
| akekane_ | glance-replicator is a utility can be used to populate a new glance server using the images stored in an existing glance server. The images in the replicated glance server preserve the uuids, metadata, and image data from the original. | 14:15 |
| jokke_ | We 100% decided years ago to deprecate and remove it. Honestly I did not realized we did not do it yet | 14:15 |
| dansmith | ack, well broken since icehouse seems like an obvious case to remove it | 14:15 |
| rosmaita | dansmith: ++ | 14:15 |
| rosmaita | i thought it had been removed too | 14:16 |
| akekane_ | Ack, in that case I will abandon those patches | 14:16 |
| akekane_ | and submit a patch to remove the replicator | 14:16 |
| akekane_ | Should I first mark it as deprecated or directly remove it in this cycle | 14:16 |
| jokke_ | rosmaita: IIRK it was decided when Nikhil was still PTL | 14:17 |
| jokke_ | Anyways like ages ago | 14:17 |
| akekane_ | yep, during I or J cycle | 14:17 |
| dansmith | seems like "broken since last decade" is probably enough deprecation warning, IMHO | 14:17 |
| jokke_ | or might have been at your early tenure, but I'm sure that decision was made | 14:17 |
| rosmaita | dansmith: ++ | 14:17 |
| dansmith | deprecation is for things that work that are going to be removed :) | 14:17 |
| jokke_ | dansmith: yep :P | 14:17 |
| akekane_ | :D | 14:17 |
| akekane_ | noted | 14:18 |
| akekane_ | #action abhishekk to abandon replicator patches and submit new patch to remove it | 14:18 |
| akekane_ | we can move ahead now | 14:19 |
| pdeore | ack | 14:19 |
| pdeore | #topic Open Discussions | 14:19 |
| jokke_ | Just quick FYI | 14:20 |
| akekane_ | pslestan1, certainly need help in unit tests coverage for glance-download plugin patch | 14:20 |
| jokke_ | I'll be taking couple of weeks PTO over next 3 weeks. So won't be much around | 14:20 |
| dansmith | akekane_: they need help? | 14:20 |
| akekane_ | dansmith, yeah, they raised this flag 4 weeks ago that they are not much familiar in unit and functional testing | 14:21 |
| akekane_ | jokke_, cool, enjoy your timeoff | 14:21 |
| dansmith | akekane_: oh okay.. should be similar to the web-download tests | 14:21 |
| akekane_ | right, | 14:22 |
| akekane_ | but they mentioned it earlier that we can contribute if possible | 14:22 |
| dansmith | I could get something started if it'd help | 14:22 |
| akekane_ | sounds good if you have time | 14:22 |
| dansmith | I don't, but.. someone needs to :) | 14:23 |
| dansmith | I was poking around with it a bit yesterday, when I realized I could get it to read /etc/passwd :) | 14:23 |
| akekane_ | cool :d | 14:23 |
| akekane_ | also does anyone has environment with tripleo and rbac deployed? | 14:24 |
| * dansmith lols | 14:24 | |
| akekane_ | haha | 14:24 |
| pdeore | I'm trying but not yet completed | 14:24 |
| akekane_ | ok | 14:24 |
| akekane_ | that is all from me for the day | 14:25 |
| pdeore | anyone has anything else to discuss? or we can wrap up early ? | 14:26 |
| jokke_ | dansmith: I'm pretty sure unittesting the web download that could be possible, but should not be possible to have it do that on functionals/tempest | 14:26 |
| dansmith | jokke_: we can test it in tempest easily, so we need that too, | 14:26 |
| jokke_ | otherwise the filters are broken badly, but IIRC there was even test case for it | 14:26 |
| dansmith | but there are unit tests for web-download, so at least parity with those is important I think | 14:27 |
| akekane_ | ++ | 14:27 |
| akekane_ | I think unit tests will be easy to add for glance-download | 14:28 |
| dansmith | yeah | 14:28 |
| dansmith | so we're done? | 14:29 |
| akekane_ | another 1 is review instant caching of image patch as I got request from tempest guys to get that merged | 14:29 |
| akekane_ | so that they can get tempest patch in | 14:29 |
| akekane_ | done now | 14:29 |
| akekane_ | we can wrap up and go back to work :D | 14:30 |
| pdeore | ack, Thanks everyone for joining !! | 14:30 |
| dansmith | oh right, you changed the wsgi thing and I never got back to it, sorry | 14:30 |
| akekane_ | thank you all! | 14:30 |
| akekane_ | dansmith, no problem | 14:30 |
| jokke_ | thanks all | 14:30 |
| pdeore | #endmeeting | 14:31 |
| opendevmeet | Meeting ended Thu Jul 14 14:31:10 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:31 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2022/glance.2022-07-14-14.00.html | 14:31 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2022/glance.2022-07-14-14.00.txt | 14:31 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2022/glance.2022-07-14-14.00.log.html | 14:31 |
| *** pdeore is now known as pdeore|afk | 14:31 | |
| *** dasm is now known as Guest5013 | 17:59 | |
| *** Guest5013 is now known as dasm | 18:01 | |
| *** dasm is now known as dasm|off | 21:23 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!