| opendevreview | Siddharth Kumar proposed openstack/manila master: Dell PowerScale: Added support of manage/unmanage share and snapshot https://review.opendev.org/c/openstack/manila/+/972278 | 08:49 |
|---|---|---|
| opendevreview | Siddharth Kumar proposed openstack/manila master: Dell PowerScale: resolve managed share path in _get_container_path https://review.opendev.org/c/openstack/manila/+/976586 | 08:49 |
| opendevreview | Kumar Tadepalli proposed openstack/manila master: NetApp - Add support for Synchronous replication for share https://review.opendev.org/c/openstack/manila/+/973578 | 08:51 |
| opendevreview | Nilesh Thathagar proposed openstack/manila master: Dell PowerScale: Addded support of snapshot mount https://review.opendev.org/c/openstack/manila/+/973257 | 10:40 |
| opendevreview | Nilesh Thathagar proposed openstack/manila master: Dell PowerScale: Added support of mount point name https://review.opendev.org/c/openstack/manila/+/973260 | 10:41 |
| opendevreview | Abhishek Gupta proposed openstack/manila master: PowerScale Manila : Dedupe support at share level https://review.opendev.org/c/openstack/manila/+/973233 | 10:41 |
| opendevreview | Siddharth Kumar proposed openstack/manila master: Dell PowerScale: resolve managed share path in _get_container_path https://review.opendev.org/c/openstack/manila/+/976586 | 10:52 |
| opendevreview | Siddharth Kumar proposed openstack/manila master: Dell PowerScale: resolve managed share path in _get_container_path https://review.opendev.org/c/openstack/manila/+/976586 | 14:23 |
| stephenfin | carloss: gouthamr: Could you grab https://review.opendev.org/c/openstack/manila/+/976128/? It's a trivial enough cleanup so would be nice to get it off my plate. carloss I could do with your eyes on the pre-commit patch above it also (gouthamr already +2d) | 14:29 |
| gouthamr | stephenfin: ack, ty! | 14:46 |
| carloss | stephenfin: done :) | 15:00 |
| *** vhari_ is now known as vhari | 15:00 | |
| medha_ | Hi Carlos, Goutham can you please review this https://review.opendev.org/c/openstack/manila/+/975606 | 15:11 |
| stephenfin | carloss: gouthamr: ty! | 15:19 |
| opendevreview | Carlos Eduardo proposed openstack/manila master: [CephFS driver] Make share deletion idempotent https://review.opendev.org/c/openstack/manila/+/976299 | 15:48 |
| opendevreview | Merged openstack/manila master: api: Remove unnecessary aliasing/brackets https://review.opendev.org/c/openstack/manila/+/976128 | 15:59 |
| opendevreview | Merged openstack/manila master: pre-commit: Bump versions https://review.opendev.org/c/openstack/manila/+/976129 | 16:00 |
| opendevreview | Merged openstack/devstack-plugin-ceph master: Use debian-trixie https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/976279 | 18:04 |
| gouthamr | medha_: ack, done | 19:34 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: api: Move v1 controllers to v2 (2/2) https://review.opendev.org/c/openstack/manila/+/959550 | 19:55 |
| gouthamr | carloss: https://review.opendev.org/c/openstack/manila/+/959549/ seems ready to merge | 19:58 |
| gouthamr | unfortunately, i left a weird cherry-pick artifact on https://review.opendev.org/c/openstack/manila/+/959550/ | 19:58 |
| gouthamr | i dropped it but lost your votes there.. | 19:58 |
| carloss | gouthamr: just pulled the trigger | 19:59 |
| carloss | i'm going through the manila-tempest-plugin changes now | 19:59 |
| carloss | gouthamr: https://review.opendev.org/c/openstack/manila-tempest-plugin/+/959685 looks good to me | 20:00 |
| carloss | I was wondering if we should stop calling shares_v2_client as it is and just call it shares_client instead, as it is the only client we will remain with | 20:01 |
| carloss | calling it v2 in the other hand makes it more suggestive | 20:01 |
| carloss | so feels a bit like tomato tomahto | 20:02 |
| gouthamr | ah, i did think about that.. but, these clients can be instantiated in tempest too | 20:03 |
| gouthamr | so i can't change the file path at least | 20:03 |
| gouthamr | but, our own usage can be "self.shares_client" for sure | 20:03 |
| gouthamr | changing the file paths will mean breaking non-tempest consumers as well, i.e., downstream test plugins like the ones NetApp, RH etc maintain | 20:04 |
| opendevreview | Merged openstack/manila master: api: Move v1 controllers to v2 (1/2) https://review.opendev.org/c/openstack/manila/+/959549 | 21:15 |
| opendevreview | Merged openstack/manila-tempest-plugin master: Stop testing the V1 API https://review.opendev.org/c/openstack/manila-tempest-plugin/+/959685 | 21:20 |
| opendevreview | Stephen Finucane proposed openstack/manila master: api: Remove references to v1 https://review.opendev.org/c/openstack/manila/+/959551 | 21:23 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: tests: Remove tests for v1 API https://review.opendev.org/c/openstack/manila/+/959552 | 21:23 |
| opendevreview | Stephen Finucane proposed openstack/manila master: docs: Remove references to v1 API https://review.opendev.org/c/openstack/manila/+/959553 | 21:23 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: api: Move v1 controllers to v2 (2/2) https://review.opendev.org/c/openstack/manila/+/959550 | 21:26 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: api: Remove v1 API https://review.opendev.org/c/openstack/manila/+/959554 | 21:26 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: api: Remove use of mixin classes https://review.opendev.org/c/openstack/manila/+/959555 | 21:32 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila master: api: Remove use of mixin classes https://review.opendev.org/c/openstack/manila/+/959555 | 21:36 |
| opendevreview | Merged openstack/manila master: Dell PowerScale: Added support of manage/unmanage share and snapshot https://review.opendev.org/c/openstack/manila/+/972278 | 21:58 |
| opendevreview | Merged openstack/manila master: Dell PowerScale: Addded support of snapshot mount https://review.opendev.org/c/openstack/manila/+/973257 | 22:34 |
| opendevreview | Merged openstack/manila master: Dell PowerScale: Added support of mount point name https://review.opendev.org/c/openstack/manila/+/973260 | 22:34 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!