| opendevreview | Jonathan Koerber proposed openstack/manila master: Adding OpenAPi Shemas to User Massages https://review.opendev.org/c/openstack/manila/+/922566 | 05:18 |
|---|---|---|
| opendevreview | kiran pawar proposed openstack/manila master: Add share type encryption https://review.opendev.org/c/openstack/manila/+/909977 | 11:24 |
| opendevreview | kiran pawar proposed openstack/manila master: Use encryption key id during share create https://review.opendev.org/c/openstack/manila/+/911089 | 11:24 |
| *** chuanm6 is now known as chuanm | 12:47 | |
| opendevreview | Carlos Eduardo proposed openstack/manila-tempest-plugin master: WIP Enable manage/unmanage tests on cephfs drivers https://review.opendev.org/c/openstack/manila-tempest-plugin/+/924774 | 13:11 |
| opendevreview | Merged openstack/devstack-plugin-ceph stable/2024.1: Fix ingress deamon https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/924727 | 14:18 |
| opendevreview | Carlos Eduardo proposed openstack/manila-tempest-plugin master: WIP Enable manage/unmanage tests on cephfs drivers https://review.opendev.org/c/openstack/manila-tempest-plugin/+/924774 | 14:21 |
| *** dviroel is now known as dviroel|afk | 15:14 | |
| ashrodri | hello! could use some eyes on this: https://review.opendev.org/c/openstack/manila/+/887543 i can't for the life of me figure out /why/ the one unit test is failing cant find context.session | 16:20 |
| *** iurygregory__ is now known as iurygregory | 16:35 | |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph stable/2023.2: Fix ingress deamon https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/924960 | 17:48 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph stable/2023.2: Fix ingress deamon https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/924960 | 17:53 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph stable/2023.2: Fix ingress deamon https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/924960 | 17:54 |
| *** iurygregory__ is now known as iurygregory | 17:57 | |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph stable/2024.1: Follow Up Patch Fix Ingress Deamon https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/924964 | 17:59 |
| carloss | o/ ashrodri that sounds related to the session changes for sqlalchemy. Is this change rebased on top of master? | 18:02 |
| carloss | if not, I'd recommend rebasing and re-running the unit tests | 18:03 |
| carloss | apparently it is :) | 18:04 |
| carloss | s/is/is on top of master already | 18:05 |
| carloss | ashrodri: I was wondering if somehow the job is getting confused with the usage of context. Have you tried declaring a new context just for the test? i.e. `test_ctxt = context.get_admin_context()` | 18:07 |
| ashrodri | i can certainly try getting a new context, it just seems weird that self.ctxt works for all the other units tests and methods, except this one. will update when I've run locally | 18:15 |
| opendevreview | Jonathan Koerber proposed openstack/manila master: Add OpenAPi Shemas to user massages https://review.opendev.org/c/openstack/manila/+/922566 | 18:17 |
| carloss | ashrodri: ack. please let me know if it doesn't work and we can try to figure out something :) | 18:22 |
| opendevreview | Saravanan Manickam proposed openstack/manila master: OpenStack Shares Backup management documentation https://review.opendev.org/c/openstack/manila/+/913213 | 18:25 |
| ashrodri | carloss, same error. even with using ctxt=context.get_admin_context, the issues arises in line 2481: result = db_api.export_location_metadata_get_item(ctxt, export_location_uuid, 'foo_key'). theres no problem using ctxt in export_location_metadata_update() | 18:31 |
| ashrodri | taking late lunch break now brb | 18:31 |
| carloss | ack, taking a look as well | 19:08 |
| carloss | s/as well/again | 19:08 |
| opendevreview | Carlos Eduardo proposed openstack/manila master: Manage/unmanage for CephFS drivers https://review.opendev.org/c/openstack/manila/+/924773 | 19:14 |
| opendevreview | Carlos Eduardo proposed openstack/manila-tempest-plugin master: WIP Enable manage/unmanage tests on cephfs drivers https://review.opendev.org/c/openstack/manila-tempest-plugin/+/924774 | 19:44 |
| ganso | https://bugs.launchpad.net/manila/+bug/2074114 | 20:22 |
| ganso | I haven't tested but the filter_function may also be affected ^ | 20:22 |
| ganso | it is interesting to see that nobody had attempted to the use the goodness_function with the cephfs driver until now | 20:23 |
| gouthamr | thanks ganso; i've seen most people use a single ceph cluster within their deployment.. i think this would be interesting to folks that are attempting multiple filesystems | 20:24 |
| gouthamr | s/single ceph cluster/single ceph cluster with a single cephfs filesystem | 20:24 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!