| rogerkorantenng | Hi gouthamr Thanks for the welcome. I've gone through the FAQs etherpad, set up my Gerrit account with SSH keys, installed git-review, and cloned the manila repo. I'll start working on a low-hanging-fruit bug to get familiar with the codebase. | 01:11 |
|---|---|---|
| gouthamr | o/ outreachy applicants, look at low-hanging-fruit bugs on manila, python-manilaclient again.. i've added quite a few bugs | 05:52 |
| hong-p[m] | carloss: hi i need help with a grenade failure on zuul. It fails at resources.sh create, but there are no errors in the manila-api logs. the change is just a one-liner for RequestContext initialization. could you help me?" https://review.opendev.org/c/openstack/manila/+/981930 | 07:27 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: Add unit tests for manila.cmd.status module https://review.opendev.org/c/openstack/manila/+/982050 | 08:17 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: Replace assertEqual(False, ...) with assertFalse(...) in unit tests https://review.opendev.org/c/openstack/manila/+/982051 | 08:17 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: Replace assertTrue/assertFalse with assertEqual/assertNotEqual https://review.opendev.org/c/openstack/manila/+/982052 | 08:17 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: api-ref: Fix wrong type for group_snapshot_id parameter https://review.opendev.org/c/openstack/manila/+/982055 | 08:39 |
| opendevreview | Silvia Wachira proposed openstack/manila master: api-ref: Missing subnet metadata in share network responses https://review.opendev.org/c/openstack/manila/+/982057 | 08:56 |
| sallykarimi | Hi gouthamr and carloss. I have gone through the suggested material. Thank you gfidente for that. I would like to fix low hanging issues , bugs #2146280 and #2146288 in python-manilaclient . I will be submitting a fix shortly. Thank you | 08:58 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: Add unit tests for manila.cmd.status module https://review.opendev.org/c/openstack/manila/+/982050 | 08:58 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: Replace assertTrue/assertFalse with assertEqual/assertNotEqual https://review.opendev.org/c/openstack/manila/+/982052 | 08:58 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: api-ref: Fix wrong type for group_snapshot_id parameter https://review.opendev.org/c/openstack/manila/+/982055 | 08:58 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: api-ref: add missing fields to share server responses https://review.opendev.org/c/openstack/manila/+/982058 | 08:58 |
| opendevreview | Sally Karimi Kinyua proposed openstack/python-manilaclient master: Fix typo 'occured' should be 'occurred' https://review.opendev.org/c/openstack/python-manilaclient/+/982060 | 09:18 |
| sallykarimi | Hi I have made my first contribution and wouldd like feedback from my peers. Here it is https://review.opendev.org/c/openstack/python-manilaclient/+/982060 | 09:28 |
| SilviaWachira[m] | Hi Sally. I will take a look at it | 09:29 |
| opendevreview | Oluwasola Akintewe proposed openstack/manila master: api-ref: add missing fields to share server responses https://review.opendev.org/c/openstack/manila/+/982058 | 09:36 |
| opendevreview | Sally Karimi Kinyua proposed openstack/manila master: Fix typos and formatting errors in parameters.yaml https://review.opendev.org/c/openstack/manila/+/982063 | 09:52 |
| SilviaWachira[m] | sallykarimi: I have left a comment for you | 10:26 |
| SilviaWachira[m] | gouthamr, carloss:... (full message at <https://matrix.org/oftc/media/v1/media/download/AdVxj8WN92AkIswyTypx0nq7B-h8q0rX0VtUZsWdQzFpbPLk0KX7eclciX6KfaQhtPFAsG64U53FT2vYumlYGmNCedbKQjNwAG1hdHJpeC5vcmcvZURBeHpsZk9wR0N1WmNkeFFXdGFSZlpl>) | 10:29 |
| opendevreview | Artem Goncharov proposed openstack/manila master: Fix jsonschema enum (list instead of tuple) https://review.opendev.org/c/openstack/manila/+/982081 | 12:55 |
| lesmadeit | gouthamr: hello, i would like to work on bug #2146259 (api-ref: wrong type for group_snapshot_links_parameter). i have commented on the issue on launchpad to ask | 13:13 |
| carloss | lesmadeit: will assign it to you | 13:17 |
| carloss | thanks for picking it up | 13:17 |
| *** herrpriestess[m] is now known as Priestess[m] | 13:53 | |
| Priestess[m] | Good day, everyone. | 14:02 |
| Priestess[m] | Please, how do I get a bug assigned to me? Or do I just start working on one? | 14:02 |
| lesmadeit | good day Priestess, you can pick up one on lauchpad(especially the ones tagged 'low hanging fruit') then request to be assigned via this platform using the bug number | 14:05 |
| Priestess[m] | Okay thank you lesmadeit | 14:06 |
| lesmadeit | happy to help :) | 14:06 |
| Priestess[m] | Hello, gouthamr I'd like to work on Bug #2146267 (api-ref: Wrong response codes for shares detail and snapshot manage). Thank you. | 14:13 |
| carloss | Priestess[m]: please share your launchpad user ID or drop a comment in the bug so that I can assign it to you | 14:14 |
| Priestess[m] | * Hello, gouthamr carloss I'd like | 14:14 |
| Priestess[m] | carloss: Okay | 14:15 |
| nathan_akin[m] | carloss: hello please i created some Prs, could you help me take a look? https://review.opendev.org/c/openstack/manila/+/982055 | 14:16 |
| Priestess[m] | carloss: I've commented on the bug. | 14:29 |
| Priestess[m] | But my lauchpad id is "priestess". | 14:29 |
| carloss | SilviaWachira[m] nathan_akin[m] looking into the prs | 14:33 |
| nathan_akin[m] | thank you so much | 14:33 |
| SilviaWachira[m] | carloss: Thank you | 14:43 |
| carloss | hong-p[m]: I missed your ping a couple of days ago - looking into the issue rn | 14:51 |
| carloss | hong-p[m]: https://6969f5db50016e3910d3-63c2f202ee2682ece2035d800961db3c.ssl.cf2.rackcdn.com/openstack/699e51267cd344a5896364f836ad7fcf/controller/logs/grenade.sh_log.txt - manila command is not being found and it's with reason. We dropped it. I'll fix the scenario tests | 14:54 |
| opendevreview | Takashi Kajinami proposed openstack/manila master: Remove logic for Python < 2.7 compatibility https://review.opendev.org/c/openstack/manila/+/982103 | 14:56 |
| comfortoyewole[m] | Hello @Carloss: | 15:24 |
| comfortoyewole[m] | Could you please assign this bug to me? | 15:24 |
| comfortoyewole[m] | https://bugs.launchpad.net/manila/+bug/2146260 | 15:24 |
| carloss | comfortoyewole[m]: done | 15:37 |
| comfortoyewole[m] | carloss: Thank you | 15:39 |
| carloss | np | 16:01 |
| opendevreview | Comfort Oyewole proposed openstack/manila master: Change the type of the group_snapshot_members field from string to array https://review.opendev.org/c/openstack/manila/+/982127 | 16:07 |
| lesmadeit | I am requesting guidance on the signing of the agreement before contributing, any tips on how to do that is appreciated | 16:22 |
| OlamideOjo[m] | To sign of run ‘git commit -s’ | 16:23 |
| OlamideOjo[m] | s/of/off/ | 16:24 |
| lesmadeit | OlamideOjo: thanks | 16:34 |
| *** iamwho2[m] is now known as Asmamothana[m] | 16:34 | |
| comfortoyewole[m] | @carloss: Could you please assign this bug to me? | 16:37 |
| comfortoyewole[m] | https://bugs.launchpad.net/manila/+bug/2146272 | 16:37 |
| Asmamothana[m] | Hi everyone. I'm Asma, a recent CS and Information Security graduate. I look forward to contributing as an outreachy intern. | 16:54 |
| comfortoyewole[m] | You're welcome Asma mothana: | 16:56 |
| opendevreview | Merged openstack/manila master: Change the type of the group_snapshot_members field from string to array https://review.opendev.org/c/openstack/manila/+/982127 | 16:57 |
| carloss | welcome Asmamothana[m] :) | 16:58 |
| opendevreview | Leslie Mwendwa proposed openstack/manila master: textapi-ref: Fix type for group_snapshot_links parameter https://review.opendev.org/c/openstack/manila/+/982136 | 17:00 |
| opendevreview | Carlos Eduardo proposed openstack/manila master: Switch grenade to using OpenStack Client https://review.opendev.org/c/openstack/manila/+/982138 | 17:04 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: Migrate grenade resources.sh from manila CLI to openstack CLI https://review.opendev.org/c/openstack/manila/+/981631 | 17:16 |
| opendevreview | Merged openstack/python-manilaclient master: Fix typo 'occured' should be 'occurred' https://review.opendev.org/c/openstack/python-manilaclient/+/982060 | 17:17 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: Migrate grenade resources.sh from manila CLI to openstack CLI https://review.opendev.org/c/openstack/manila/+/981631 | 17:21 |
| opendevreview | Merged openstack/manila master: textapi-ref: Fix type for group_snapshot_links parameter https://review.opendev.org/c/openstack/manila/+/982136 | 17:37 |
| opendevreview | Comfort Oyewole proposed openstack/manila master: Add metadata parameter to create share replica API endpoint https://review.opendev.org/c/openstack/manila/+/982140 | 17:47 |
| Asmamothana[m] | Can I be assigned https://bugs.launchpad.net/manila/+bug/2146277 My launchpad is iamwho2 | 18:29 |
| carloss | Asmamothana[m]: done :) | 18:30 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Manila telemetry metrics https://review.opendev.org/c/openstack/manila/+/982149 | 19:27 |
| sallykarimi | Hi @carloss and gouthamr my second fix https://review.opendev.org/c/openstack/manila/+/982063 is also ready for review | 19:32 |
| sallykarimi | Hi Priestess you can find issues to work on here https://bugs.launchpad.net/manila/+bugs?field.tag=low-hanging-fruit | 19:34 |
| carloss | sallykarimi: ack, thanks for the pointer | 19:34 |
| Priestess[m] | sallykarimi: Thank you, sallykarimi | 19:35 |
| opendevreview | Sally Karimi Kinyua proposed openstack/manila master: Remove stale api-paste.ini override in grenade upgrade script https://review.opendev.org/c/openstack/manila/+/982151 | 19:53 |
| sallykarimi | welcome @Priestess anything you need help with just ask we are all willing to help | 19:55 |
| opendevreview | Denver Baraka proposed openstack/python-manilaclient master: doc: Update OS_TENANT_NAME to OS_PROJECT_NAME https://review.opendev.org/c/openstack/python-manilaclient/+/982153 | 19:58 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Manila telemetry metrics https://review.opendev.org/c/openstack/manila/+/982149 | 20:14 |
| opendevreview | Denver Baraka proposed openstack/python-manilaclient master: doc: Fix wrong class references in manager docstrings https://review.opendev.org/c/openstack/python-manilaclient/+/982161 | 20:27 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Manila telemetry metrics https://review.opendev.org/c/openstack/manila/+/982149 | 20:33 |
| opendevreview | Carlos Eduardo proposed openstack/manila master: Fix service listing with a wrong status https://review.opendev.org/c/openstack/manila/+/982165 | 20:46 |
| denver_ba | Hi, the link @gouthamr referenced for new bugs low hanging fruit on python manila client is https://bugs.launchpad.net/python-manilaclient/+bugs?field.tag=low-hanging-fruit | 20:47 |
| denver_ba | Hi, @carloss and @gouthamr I have also made some patches please review https://review.opendev.org/c/openstack/python-manilaclient/+/982153 and https://review.opendev.org/c/openstack/python-manilaclient/+/982161 | 20:49 |
| opendevreview | Amarachi Ogbonnaya proposed openstack/manila master: Add documentation for Manila telemetry metrics https://review.opendev.org/c/openstack/manila/+/982149 | 20:56 |
| carloss | denver_ba: ack | 21:01 |
| opendevreview | Carlos Eduardo proposed openstack/manila master: Migrate grenade resources.sh from manila CLI to openstack CLI https://review.opendev.org/c/openstack/manila/+/981631 | 21:02 |
| opendevreview | Denver Baraka proposed openstack/python-manilaclient master: doc: fix share network docstrings https://review.opendev.org/c/openstack/python-manilaclient/+/982168 | 21:09 |
| opendevreview | Olamide Ojo proposed openstack/manila master: Add missing unit test to cephfs driver for line 804 https://review.opendev.org/c/openstack/manila/+/981873 | 21:16 |
| Amarachi20 | Hi, can this https://bugs.launchpad.net/manila/+bug/1905427 bug be assign to me | 21:29 |
| carloss | hey Amarachi20 do you have a devstack environment? | 21:30 |
| carloss | this is kind of not a low hanging fruit - can you look into this pool of issues: https://bugs.launchpad.net/manila/+bugs?field.tag=low-hanging-fruit ? | 21:31 |
| carloss | this has been updated | 21:31 |
| Amarachi20 | Is this okay to work on https://bugs.launchpad.net/manila/+bug/2146256 | 21:53 |
| hong-p[m] | carloss: thanks, I see. found it I’ll wait for this patch https://review.opendev.org/c/openstack/manila/+/982138 | 22:08 |
| Amarachi20 | can i work on this - https://bugs.launchpad.net/manila/+bug/2146274 | 22:13 |
| Amarachi20 | Is it taken? | 22:16 |
| nathan_akin[m] | hello carloss thank you so much for the approval, i can work on more issues right? https://review.opendev.org/c/openstack/manila/+/982052 | 22:17 |
| nathan_akin[m] | https://review.opendev.org/c/openstack/manila/+/982055 | 22:28 |
| opendevreview | Olamide Ojo proposed openstack/manila master: Add missing test to cephfs driver for line 831. Increases test coverage to 81% https://review.opendev.org/c/openstack/manila/+/982181 | 22:32 |
| opendevreview | Comfort Oyewole proposed openstack/manila master: Add metadata parameter to create share replica API endpoint https://review.opendev.org/c/openstack/manila/+/982140 | 23:04 |
| opendevreview | Comfort Oyewole proposed openstack/manila master: Add metadata parameter to create share replica API endpoint https://review.opendev.org/c/openstack/manila/+/982140 | 23:35 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!