| opendevreview | Merged openstack/manila master: Remove deprecated config and auth https://review.opendev.org/c/openstack/manila/+/745206 | 01:46 |
|---|---|---|
| opendevreview | Merged openstack/manila master: [doc] clean up compute client options https://review.opendev.org/c/openstack/manila/+/747487 | 01:47 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila stable/ussuri: Fix generic share resize with 0.0.0.0/24 access https://review.opendev.org/c/openstack/manila/+/797309 | 04:43 |
| icey | hey, is it necessary / recommended that manila-share and nfs-ganesha have different ceph credentials or can they share? vague question wrt version because I'm interested in the answer for basically all versions :) | 07:41 |
| icey | it looks like [1] sharing ceph keys is likely to be racy and risky. 1: https://github.com/ceph/ceph-ci/blob/d2a3fcc728c31b25166aeb55c99b9ccf8d4985bb/src/pybind/ceph_volume_client.py#L476-L479 | 08:02 |
| opendevreview | Merged openstack/manila-ui master: [CI] Temporarily set integration test job to non-voting https://review.opendev.org/c/openstack/manila-ui/+/797347 | 10:08 |
| tbarron | icey: probably a good idea to use separate ceph clients/keys. | 10:56 |
| tbarron | icey: in devstack CI deployments we use 'admin' client/key for ganesha and 'manila' client/key for manila-share with ceph-volume library. | 10:56 |
| icey | *nod* | 10:56 |
| icey | thanks for the confirmation tbarron | 10:57 |
| tbarron | icey: note that the ceph-volume library is no longer used though starting in wallaby | 10:57 |
| icey | tbarron: in favor of direct librados, or something else? | 10:57 |
| tbarron | icey: yeah, we use rados client and interaction with the ceph manager | 10:58 |
| tbarron | icey: the original driver code was written by our betters before most of us were on the scene, but it was pre-ceph manager | 10:58 |
| vkmc | tbarron, icey, o/ | 10:59 |
| vkmc | probably a nice gotcha to add to our docs | 10:59 |
| tbarron | icey: but vkmc ^^ drove a modernization effort in wallaby ... | 10:59 |
| icey | coolio! it's always nice to update things, isn't it :) | 11:00 |
| vkmc | of course :) | 11:00 |
| vkmc | icey, I'll submit a bug report on the docs and add this info | 11:01 |
| *** dviroel|out is now known as dviroel | 11:20 | |
| opendevreview | Tom Barron proposed openstack/manila master: Remove duplicate line in release note https://review.opendev.org/c/openstack/manila/+/797430 | 11:24 |
| *** osmanlicilegi is now known as Guest187 | 12:25 | |
| *** ChanServ changes topic to "OpenStack Shared File System Service | Documentation: https://docs.openstack.org/manila/latest/ | Meetings: https://wiki.openstack.org/wiki/Manila/Meetings | Xena PTG Summary Etherpad: https://etherpad.opendev.org/p/xena-ptg-manila" | 12:30 | |
| archanaserver | hi, i'm working on this bug(https://bugs.launchpad.net/manila/+bug/1646433) and proposed a patch -> https://review.opendev.org/c/openstack/manila/+/795990 can anyone left some review in here? | 15:55 |
| gouthamr | icey: o/ i don't see any problem with sharing the user between ganesha/manila | 16:07 |
| gouthamr | icey: in fact we do it that way in tripleo/RHOSP | 16:08 |
| icey | gouthamr: well, in releases prior to wallaby, manila-share evicts any other clients using the same credentials :-); nfs-ganesha seems to recover from that but it makes me nervous | 16:08 |
| tbarron | gouthamr: yeah, i was thinking icey might have found a potential issue when manila-share restarts or fails over | 16:09 |
| gouthamr | but the nfs-ganesha user doesn't need/use mds capabilities | 16:10 |
| gouthamr | afaict - so an eviction means nothing to nfs-ganesha; what recovery is going on? | 16:10 |
| tbarron | well it's a CephFS client with Ceph FSAL | 16:11 |
| tbarron | what am I missing | 16:11 |
| gouthamr | need batrick or rraja to confirm, but, i am thinking that exports have their own client users | 16:13 |
| gouthamr | the client user we share is used to fetch objects from ceph during a ganesha recovery | 16:15 |
| gouthamr | we do a song-and-dance coordination in tripleo where ganesha is started/restarted before manila-share, so the recovery in question has already occured | 16:16 |
| gouthamr | ^ so maybe in that situation, it doesn't matter if the client user is evicted? | 16:16 |
| gouthamr | so in theory it is racy if the ganesha start-up/recovery was happening at the same time as manila startup? | 16:18 |
| * gouthamr is thinking out loud, but also asking questions | 16:18 | |
| gouthamr | :) | 16:18 |
| tbarron | yeah, or potentially exports in process? | 16:24 |
| tbarron | or maybe just a client for the FSAL doing data transfers? | 16:24 |
| tbarron | you wouldn't want that client to "go away" | 16:25 |
| gouthamr | data transfers use individual clients | 16:25 |
| tbarron | right, thanks | 16:25 |
| gouthamr | each export rule on manila is a different ceph client | 16:25 |
| tbarron | right, but not the export setup? | 16:25 |
| tbarron | I honestly don't know. | 16:26 |
| gouthamr | ack, we can ask the ceph folks on here - lemme invite batrick and rraja | 16:26 |
| gouthamr | oh there he is - rraja | 16:28 |
| gouthamr | rraja: eavesdrop's not updated (https://meetings.opendev.org/irclogs/%23openstack-manila/latest.log.html) --- but, lemme here's a paste of the log from this channel: http://paste.openstack.org/show/806864/ | 16:29 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila-ui stable/wallaby: Fix share group creation with share types https://review.opendev.org/c/openstack/manila-ui/+/797483 | 16:32 |
| rraja | using ceph-ansible as reference https://github.com/ceph/ceph-ansible/blob/master/roles/ceph-nfs/templates/ganesha.conf.j2 we seem to be using the same cephx auth ID for RADOS_KV (client recovery), and RADOS_URL(fetching ganesha exports from RADOS objects). These would be different librados clients using the same ceph auth ID. The ceph_volume_client evicts CephFS clients by telling the MDS to do so https://docs.ceph.com/en/latest/cephfs/eviction/#ma | 17:16 |
| rraja | nual-client-eviction I don't think think ceph_volume_client eviction affects the librados clients used by ganesha for recovery and fetching export objects even if they all use the same cephx auth ID | 17:16 |
| rraja | gouthamr, tbarron ^^ | 17:16 |
| gouthamr | that's what i was guessing - these are mds evicts, but the rados kv lookups should be unaffected | 17:17 |
| gouthamr | thanks for explaining rraja! :) | 17:17 |
| gouthamr | icey ^ | 17:17 |
| opendevreview | Merged openstack/manila-ui master: Fix share group creation with share types https://review.opendev.org/c/openstack/manila-ui/+/797354 | 17:19 |
| tbarron | rraja: thanks! | 17:37 |
| opendevreview | Merged openstack/manila-ui stable/victoria: Remove nova networking https://review.opendev.org/c/openstack/manila-ui/+/796554 | 17:41 |
| opendevreview | Merged openstack/manila-ui stable/victoria: Fix parsing names in switched fields https://review.opendev.org/c/openstack/manila-ui/+/797163 | 17:44 |
| gouthamr | there's batrick ! :) | 17:47 |
| batrick | \o | 17:47 |
| icey | thanks! | 18:03 |
| opendevreview | Merged openstack/manila-ui stable/ussuri: Remove nova networking https://review.opendev.org/c/openstack/manila-ui/+/796555 | 18:31 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila-ui stable/victoria: Fix share group creation with share types https://review.opendev.org/c/openstack/manila-ui/+/797485 | 18:36 |
| *** iurygregory_ is now known as iurygregory | 18:39 | |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila-ui stable/ussuri: Fix share group creation with share types https://review.opendev.org/c/openstack/manila-ui/+/797486 | 18:40 |
| opendevreview | Goutham Pacha Ravi proposed openstack/manila-ui stable/train: Fix share group creation with share types https://review.opendev.org/c/openstack/manila-ui/+/797487 | 18:40 |
| gouthamr | ^ dviroel carloss: just lining up the cherry-picks, but you're doing the right thing merging these in order :) thanks! | 18:41 |
| * gouthamr likes to optimize CI turn around time | 18:41 | |
| opendevreview | Merged openstack/manila-ui stable/ussuri: Fix parsing names in switched fields https://review.opendev.org/c/openstack/manila-ui/+/797164 | 18:48 |
| opendevreview | Merged openstack/manila-ui stable/wallaby: Fix share group creation with share types https://review.opendev.org/c/openstack/manila-ui/+/797483 | 18:50 |
| opendevreview | Merged openstack/manila master: Remove duplicate line in release note https://review.opendev.org/c/openstack/manila/+/797430 | 19:13 |
| opendevreview | Merged openstack/manila-ui stable/train: Remove nova networking https://review.opendev.org/c/openstack/manila-ui/+/796556 | 19:19 |
| gouthamr | my relationship with pep8: https://images.app.goo.gl/Yca8YvbPGL1CeznHA | 19:30 |
| dviroel | lol | 19:31 |
| opendevreview | Merged openstack/manila-ui stable/train: Fix parsing names in switched fields https://review.opendev.org/c/openstack/manila-ui/+/796547 | 19:47 |
| gouthamr | phew likely a mid-air collision | 19:48 |
| dviroel | yeah, should pass now | 19:51 |
| opendevreview | Merged openstack/manila-ui stable/victoria: Fix share group creation with share types https://review.opendev.org/c/openstack/manila-ui/+/797485 | 20:08 |
| opendevreview | Merged openstack/manila stable/wallaby: Fix PDF build https://review.opendev.org/c/openstack/manila/+/796224 | 20:42 |
| *** dviroel is now known as dviroel|out | 20:59 | |
| opendevreview | Merged openstack/manila-ui stable/ussuri: Fix share group creation with share types https://review.opendev.org/c/openstack/manila-ui/+/797486 | 21:35 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!