| amara | Hello! I am Amarachi, an Outreachy applicant from Nigeria | 07:31 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/manila master: Deprecate Windows SMB share driver https://review.opendev.org/c/openstack/manila/+/911048 | 08:42 |
| tkajinam | I wonder this makes sense ^^^ | 08:43 |
| tkajinam | (if it does then we may want to merge it in stable/2024.1 so that we can drop the driver in 2024.2) | 08:44 |
| *** solly47[m] is now known as AkpuruSolomonBarine[m] | 08:58 | |
| AkpuruSolomonBarine[m] | Hello Everyone | 08:58 |
| opendevreview | Merged openstack/devstack-plugin-ceph master: Fix cephadm binary URL to the numeric ceph version https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/910966 | 10:51 |
| opendevreview | Ke Niu proposed openstack/manila master: Add user defined extra capabilities https://review.opendev.org/c/openstack/manila/+/911085 | 10:59 |
| opendevreview | kiran pawar proposed openstack/manila master: Add share type encryption https://review.opendev.org/c/openstack/manila/+/909977 | 11:30 |
| opendevreview | kiran pawar proposed openstack/manila master: Use encryption key id during share create https://review.opendev.org/c/openstack/manila/+/911089 | 11:30 |
| opendevreview | kiran pawar proposed openstack/manila-specs master: Add spec for share encryption https://review.opendev.org/c/openstack/manila-specs/+/898999 | 11:32 |
| carloss_ | amara AkpuruSolomonBarine[m] o/ hello :) | 12:10 |
| carloss_ | tkajinam: I think it makes sense, we can do some extra digging | 12:11 |
| ccokeke[m] | Hi carloss gouthamr | 12:34 |
| ccokeke[m] | I was looking into this bug: https://bugs.launchpad.net/manila/+bug/2052785 | 12:34 |
| ccokeke[m] | I hope it is still active and I can work on it? | 12:34 |
| carloss_ | ccokeke[m]: hey | 13:41 |
| carloss_ | this is a good bug, but it's not that easy to fix | 13:41 |
| carloss_ | have you looked at the bugs that have the low-hanging-fruit tag? | 13:42 |
| carloss_ | that is used to identify bugs that are supposed to be easier to fix, and are good for people starting to contribute | 13:42 |
| ccokeke[m] | Thanks carloss | 13:48 |
| ccokeke[m] | I will look into the other low-hanging fruits bug as well. | 13:48 |
| ccokeke[m] | Thanks for the quick response | 13:48 |
| msaravan | hi, any recent changes done to RBAC ? I see the below 3 tests fails continuously with the error : Details: {'code': 403, 'message': 'Access was denied to this resource.') | 13:50 |
| msaravan | test_soft_delete_and_restore_share[api_with_backend,id-7a19fb58-b645-44cc-a6d7-b3508ff8754d,positive] | 13:50 |
| msaravan | test_restore_share_with_wrong_id[api,id-87345725-f187-4d7d-86b1-62284e8c75ae,negative] | 13:50 |
| msaravan | test_soft_delete_share_has_been_soft_deleted[api_with_backend,id-36cbe23b-08d2-49d9-bb42-f9eb2a804cb1,negative] | 13:50 |
| msaravan | test_soft_delete_share_with_wrong_id[api,id-0ccd44dd-2fda-403e-bc23-7ce428550f36,negative] | 13:50 |
| msaravan | The above errors are seen in NetApp-CI | 13:50 |
| msaravan | for the upstream patch - https://review.opendev.org/c/openstack/manila/+/911048 | 13:51 |
| carloss_ | ccokeke[m]: np | 14:00 |
| carloss_ | msaravan: hey, I assume this is on master branch, right? | 14:01 |
| carloss_ | I believe nothing has changes in the last couple of days though :| | 14:02 |
| opendevreview | Saravanan Manickam proposed openstack/manila master: [DNM] Check NetApp CI runs https://review.opendev.org/c/openstack/manila/+/907132 | 14:12 |
| msaravan | thank you @carloss_ | 14:20 |
| carloss_ | msaravan: in a recent change, the rbac build works just fine: https://review.opendev.org/c/openstack/manila-tempest-plugin/+/910908?tab=change-view-tab-header-zuul-results-summary | 14:21 |
| carloss_ | are you enabling rbac in the CI though? | 14:22 |
| msaravan | cool ..will cross verify the tempest configs | 14:22 |
| msaravan | I didnt make any changes.. I see it passing last month.. and now I see the 4 tests continuously fail for all the patches..so just thought of cross checking | 14:22 |
| carloss_ | https://zuul.opendev.org/t/openstack/build/49b227729c594bd398f420bf9ac6d0a2/log/controller/logs/etc/manila/manila_conf.txt#97 - this should enable rbac | 14:23 |
| msaravan | will compare the configs and find it out | 14:23 |
| carloss_ | https://zuul.opendev.org/t/openstack/build/49b227729c594bd398f420bf9ac6d0a2/log/controller/logs/tempest_conf.txt#89 - and this | 14:23 |
| msaravan | thank you for the pointers.. | 14:24 |
| opendevreview | Gireesh Awasthi proposed openstack/manila master: Implement the share backup for NetApp driver https://review.opendev.org/c/openstack/manila/+/909115 | 14:43 |
| opendevreview | Merged openstack/python-manilaclient master: Add new-note tox environment https://review.opendev.org/c/openstack/python-manilaclient/+/909364 | 15:07 |
| opendevreview | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: Set CephFS/NFS job to non-voting https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/911157 | 17:38 |
| opendevreview | Merged openstack/manila-ui master: Add new-note tox environment https://review.opendev.org/c/openstack/manila-ui/+/909659 | 17:49 |
| *** tspyderboy[m] is now known as MohdAriq[m] | 17:55 | |
| *** MohdAriq[m] is now known as tspyderboy[m] | 17:57 | |
| tspyderboy[m] | gouthamr: Thanks, I am setting up manila-ui, then shall start with a starters bug | 18:02 |
| gouthamr | msaravan carloss_ the tests failing with RBAC errors were reported in a bug here: https://bugs.launchpad.net/manila/+bug/2035562 | 18:13 |
| msaravan | The bug is on me only :-) thank you for sharing that. | 18:16 |
| msaravan | What worries me is, I used to see them passing in some runs and failing in a few. Need to root cause that. | 18:17 |
| gouthamr | haha :) i wonder if there's something going wrong with tenant isolation here... with my brief debugging, i felt that the credential's roles were being altered in the course of the test .. i.e., the test account used only had a "reader" role assigned by the time it issued a soft-delete (or restore) call.. | 18:18 |
| gouthamr | msaravan: try running the tests with concurrency=1 and see if things look the same? | 18:20 |
| msaravan | @gouthamr : good information. Sure, will do that. | 18:21 |
| tspyderboy[m] | @gouthamr : https://docs.openstack.org/manila/latest/contributor/development-environment-devstack.html... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/WpLHtxOHrfegULyXqQEVqrYQ>) | 19:25 |
| gouthamr | tspyderboy[m]: where are you running devstack? is this a VM? and what's the OS you're using? | 19:26 |
| tspyderboy[m] | Nope, didnt run it in VM. Running it on Ubuntu 22.04.4 LTS | 19:28 |
| gouthamr | tspyderboy[m]: oh, i'd discourage running devstack directly on your laptop/computer.. | 20:07 |
| gouthamr | tspyderboy[m]: you'd need a virtual machine; i suggest using virtualbox and ubuntu 22.04; here's a guide that might be helpful https://medium.com/@shuvamkumarsk1/setup-devstack-in-virtualbox-e09d9f9e2d84 --- use this in conjunction with the local.conf from https://docs.openstack.org/manila/latest/contributor/development-environment-devstack.html | 20:10 |
| tspyderboy[m] | gouthamr: Thanks for the reference, I'll set it up in VM | 20:11 |
| gouthamr | tspyderboy[m]: also, to make a beginner bug fix, devstack may not be required always... find something small - like a small code change that you can unit test locally (unit tests don't run the software, they use a python virtual environment to simulate the code and test a particular code block), or documentation changes | 20:12 |
| tspyderboy[m] | gouthamr: Sure, also what is the recommended python version for manila-ui | 20:22 |
| opendevreview | Merged openstack/manila-tempest-plugin master: Ensure metadata tests manipulate only its own metadata https://review.opendev.org/c/openstack/manila-tempest-plugin/+/909798 | 20:32 |
| opendevreview | Merged openstack/manila-tempest-plugin master: Use ddt.named_data decorator to avoid random test names https://review.opendev.org/c/openstack/manila-tempest-plugin/+/910908 | 20:43 |
| gouthamr | tspyderboy[m]: multiple versions of python are supported atm; the minimum version is 3.9 | 20:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!