| *** dasm|off is now known as dasm | 12:56 | |
| *** dviroel is now known as dviroel|lunch | 14:57 | |
| carloss | #startmeeting manila | 15:00 |
|---|---|---|
| opendevmeet | Meeting started Thu Sep 8 15:00:20 2022 UTC and is due to finish in 60 minutes. The chair is carloss. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'manila' | 15:00 |
| felipe_rodrigues | o/ | 15:00 |
| carloss | courtesy ping: vkmc dviroel vhari gouthamr | 15:00 |
| haixin | o/ | 15:00 |
| vkmc | o/ | 15:00 |
| lucasmoliveira059 | \o | 15:00 |
| gouthamr | o/ | 15:00 |
| HelenaDantas[m] | o/ | 15:00 |
| dviroel | o/ | 15:00 |
| francie_ | o/ | 15:01 |
| vhari | hi | 15:01 |
| nahimsouza[m] | hi | 15:02 |
| caiquemello[m] | \o | 15:02 |
| carloss | hello everyone! good to have you here today | 15:03 |
| luizsantos[m] | o/ | 15:03 |
| carloss | today's meeting agenda: | 15:04 |
| carloss | #link https://wiki.openstack.org/wiki/Manila/Meetings#Next_meeting | 15:04 |
| MatheusAndrade[m] | o/ | 15:04 |
| raffaelacunha[m] | o/ | 15:04 |
| carloss | let's start with our announcements for today | 15:05 |
| carloss | #topic Announcements | 15:05 |
| carloss | I would like to share with you today that francie_'s outreachy internship ended two weeks ago | 15:06 |
| carloss | and I would like to publicly thank her for all the amazing work she did for manila in such short time | 15:06 |
| carloss | we can still see her around working on changes (which is awesome) | 15:06 |
| carloss | thank you for all the work francie_! | 15:07 |
| carloss | https://review.opendev.org/q/owner:franciedigital%2540gmail.com - I'm glad to see all of these changes merged | 15:07 |
| carloss | and also glad to see how much you enjoyed working on every single change on this list | 15:08 |
| gouthamr | indeed; great job francie_! | 15:08 |
| carloss | I would also like to appreciate the work of maaritamm during this Outreachy internship cycle :D | 15:09 |
| francie_ | Thanks all | 15:09 |
| francie_ | It was a great pleasure working with you guys | 15:11 |
| carloss | maaritamm and I were co-mentors and she was impressive with organization and her OSC skills | 15:11 |
| carloss | it was great! | 15:11 |
| carloss | remember that you'll ever be welcome in this community francie_ :) | 15:11 |
| carloss | second thing on announcements: | 15:13 |
| carloss | Schedule and Deadlines: | 15:13 |
| carloss | #link https://releases.openstack.org/zed/schedule.html | 15:13 |
| carloss | we're 1 week past the official feature freeze | 15:13 |
| carloss | but | 15:13 |
| carloss | I bought some of our candidate changes one extra week to be merged | 15:13 |
| carloss | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-September/030281.html | 15:14 |
| carloss | we can go over the mentioned changes in a few, they are not merged yet but they are getting close | 15:15 |
| carloss | and the last announcement I have for today: | 15:15 |
| carloss | As you might have seen in the mailing list: Bugsquash postponed for next week! (Sep 13th to 15th) | 15:15 |
| carloss | we had a bugsquash event scheduled for this week but due to the feature freeze exceptions we needed to postpone it for a week | 15:16 |
| carloss | so we would keep the reviewers and change owners attention on the FFE changes | 15:16 |
| carloss | that's all I had for $topic | 15:17 |
| carloss | do you have an announcement you'd like to share with us today? | 15:17 |
| * carloss taking silence as a no | 15:20 | |
| carloss | #topic Review Focus | 15:20 |
| carloss | #link https://etherpad.opendev.org/p/manila-zorilla-review-focus (Review Focus Etherpad) | 15:20 |
| carloss | let's go over the three changes that are our focus now | 15:22 |
| carloss | starting with | 15:22 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/825008 (Metadata for Share Snapshots Resource) | 15:22 |
| carloss | this change got more reviewers' traction since last week and issues found were solved and comments addressed | 15:23 |
| carloss | I'm testing this issue again now | 15:23 |
| carloss | I see the rebase on top of share transfer and the conflict resolution (thanks ashrodri for anticipating that) | 15:24 |
| carloss | s/testing this issue/testing this change | 15:25 |
| carloss | I intend to post my tests result soon, so there is enough time to act on them | 15:26 |
| carloss | something else to add to this change? | 15:27 |
| carloss | thank you to the reviewers and ashrodri for solving the comments :) | 15:28 |
| carloss | second change in the list is: | 15:28 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/843832 Update micversion to 2.73, support share transfer between project | | 15:29 |
| carloss | this has also received more reviews since last week - thanks haixin for addressing all the comments | 15:29 |
| carloss | this is the next to test on my quere | 15:29 |
| carloss | queue* | 15:29 |
| haixin | thanks for review:) | 15:29 |
| carloss | you're welcome haixin | 15:30 |
| gouthamr | i'm looking at this right now; if there are minor/non-breaking issues, i'll note them so we can get the patch merged and push up a change later for minor fixes | 15:30 |
| felipe_rodrigues | thanks Haixin, there is still an open point.. it's a corner case, If it's too complicated to be solved now, we could merge this way and work on it as a follow-up. | 15:30 |
| felipe_rodrigues | overall looks good, nice work haixin :) | 15:31 |
| carloss | gouthamr++ felipe_rodrigues++ - thank you for the reviews on this change | 15:32 |
| carloss | and thank you for the work haixin | 15:32 |
| carloss | third change on the list: | 15:33 |
| carloss | #link https://review.opendev.org/c/openstack/manila/+/848987 Refactor the Ceph NFS driver to use Cephadm NFS | 15:33 |
| carloss | I feel this is getting quite close too - thank you for addressing the comments related to latest review vkmc | 15:34 |
| gouthamr | waiting on results here: https://review.opendev.org/c/openstack/manila-tempest-plugin/+/834220 | 15:34 |
| gouthamr | if the ceph-nfs job does better here than on the manila patch, we can merge this - we've run into resource exhaustion :/ | 15:36 |
| gouthamr | manila-tempest-plugin-cephfs-nfs: https://zuul.opendev.org/t/openstack/build/7dd2735257ed4c658416f4b42c525b66 | 15:36 |
| gouthamr | resource exhaustion makes manila-share, nova-api, nova-cpu etc lose connection to the db and fail tests | 15:37 |
| gouthamr | and its apparent this only happens when scenario tests are run - so we may have to think of some better/long term strategies here | 15:39 |
| carloss | hmm - I see concurrency on the new job set to two... would if help if we bump down? that's one approach we took with the generic driver job when we switched to a more recent image that needed more resources | 15:41 |
| carloss | I think there's a chance to hit timeout in the job | 15:41 |
| carloss | but I'm unsure what would happen more often: the resource exhaustion or timeout in the job | 15:42 |
| gouthamr | yeah, hard to tell; also this could be a test node on a slow hypervisor | 15:44 |
| carloss | ^ that's a possibility as well :) | 15:44 |
| dviroel | what about create a DNM change, with a new job which runs only scenario test, with concurrency set to 1 | 15:45 |
| gouthamr | some other strategies: run api tests with max concurrency, then run scenario tests with concurrency 1 --- it'll need a new custom job definition | 15:46 |
| *** dviroel is now known as dviroel_ | 15:46 | |
| *** dviroel|lunch is now known as dviroel | 15:46 | |
| carloss | yep | 15:47 |
| gouthamr | or do multi-node testing like we were planning and isolate ceph on a node by itself -- we can still hopefully run with a good test concurrency with this | 15:47 |
| gouthamr | i like dviroel_'s idea for now though - we can do a DNM patch if the manila-tempest-plugin change suffers the same fate | 15:48 |
| carloss | ++ - that would help to validate at this point | 15:49 |
| dviroel | multi-node approach is a good one, but may take some time to get everything working. | 15:50 |
| carloss | ++ | 15:51 |
| * gouthamr net-split and a disconnect? | 15:53 | |
| gouthamr | https://review.opendev.org/c/openstack/manila-tempest-plugin/+/856540 | 15:53 |
| gouthamr | ^ testing with concurrency=1 in parallel because i'm impatient :D | 15:53 |
| carloss | xD | 15:54 |
| carloss | thanks gouthamr | 15:54 |
| carloss | okay, so we have a direction - thanks for checking the tests gouthamr | 15:57 |
| carloss | we can follow up on the test approach | 15:57 |
| carloss | we have ~3 minutes on this meeting yet | 15:57 |
| carloss | sorry we didn't have time to go over the bugs list again today vhari :( | 15:58 |
| carloss | okay, that's all I had for $topic | 15:58 |
| carloss | and I think we can get back two minutes to our days :D | 15:59 |
| carloss | thank you for joining, let's continue the reviews to get these changes moving | 15:59 |
| carloss | #endmeeting | 15:59 |
| opendevmeet | Meeting ended Thu Sep 8 15:59:55 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:59 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/manila/2022/manila.2022-09-08-15.00.html | 15:59 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/manila/2022/manila.2022-09-08-15.00.txt | 15:59 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/manila/2022/manila.2022-09-08-15.00.log.html | 15:59 |
| *** dasm is now known as dasm|off | 21:13 | |
| *** dviroel is now known as dviroel|out | 21:57 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!