| *** masber has quit IRC | 00:33 | |
| *** masber has joined #openstack-manila | 00:41 | |
| *** masber has quit IRC | 00:47 | |
| *** caowei has joined #openstack-manila | 01:18 | |
| *** gouthamr has quit IRC | 01:24 | |
| *** gouthamr has joined #openstack-manila | 01:45 | |
| *** cfey has quit IRC | 01:56 | |
| *** cfey has joined #openstack-manila | 02:00 | |
| *** gcb has joined #openstack-manila | 02:06 | |
| zhongjun | amito: sorry for late, my answer is the same as gouthamr, you already met the driver deadline | 02:07 |
|---|---|---|
| *** junboli has joined #openstack-manila | 02:14 | |
| *** masber has joined #openstack-manila | 02:39 | |
| *** gouthamr has quit IRC | 02:46 | |
| *** gouthamr has joined #openstack-manila | 02:47 | |
| *** markstur_ has quit IRC | 03:05 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/manila master: Updated from global requirements https://review.openstack.org/525382 | 03:09 |
| *** gouthamr has quit IRC | 03:23 | |
| *** threestrands has joined #openstack-manila | 03:27 | |
| amito | zhongjun: that's really ok, thank you (: | 04:10 |
| *** ianychoi has joined #openstack-manila | 05:07 | |
| *** threestrands has quit IRC | 05:25 | |
| *** markstur has joined #openstack-manila | 05:28 | |
| *** markstur has quit IRC | 05:29 | |
| *** markstur has joined #openstack-manila | 05:29 | |
| junboli | Hi all, Can someone help explain why we don't allow the 'ro' access level for CIFS share? | 06:09 |
| junboli | When I allow 'ro' access level for a CIFS share, the state is always error, and I checked the unit test https://github.com/openstack/manila/blob/master/manila/tests/share/drivers/test_helpers.py#L545 | 06:10 |
| junboli | it looks CIFS share does not allow ro access level. | 06:10 |
| zhongjun | amito: :) | 06:26 |
| zhongjun | junboli: https://review.openstack.org/#/c/161176/ This patch could explain your question | 06:27 |
| zhongjun | junboli: Implement its support with NFS protocol for Generic driver and raise error for CIFS protocol if it is not rw, because it does not allow to handle both - ro and rw access rules for IPs/nets using one share export. | 06:27 |
| junboli | zhongjun: I see, Thanks jun. | 06:37 |
| zhongjun | junboli: np | 06:37 |
| *** markstur has quit IRC | 06:38 | |
| openstackgerrit | junbo.li proposed openstack/manila master: Api-ref: add show details for share type https://review.openstack.org/525484 | 07:18 |
| *** pcaruana has joined #openstack-manila | 07:32 | |
| *** zengyingzhe has quit IRC | 07:53 | |
| *** zengyingzhe has joined #openstack-manila | 07:53 | |
| *** dsariel has joined #openstack-manila | 07:54 | |
| openstackgerrit | zhongjun proposed openstack/python-manilaclient master: Add 'count' in list command https://review.openstack.org/525498 | 08:24 |
| *** belmoreira has joined #openstack-manila | 08:37 | |
| *** chenying_ has quit IRC | 08:38 | |
| *** dsariel has quit IRC | 08:40 | |
| *** pcaruana has quit IRC | 08:40 | |
| *** chenying has joined #openstack-manila | 08:50 | |
| amito | zhongjun: can you clarify me gouthamr's second comment in the last patch set? | 09:00 |
| amito | zhongjun: I understand we shouldn't fail on wrong patch set, but shouldn't it be - try: verify_share_proto, except: return so we don't get into the actual share deletion code if the share is of the wrong protocol? | 09:01 |
| amito | s/patch set/storage protocol | 09:01 |
| amito | don't know why I wrote patch set... | 09:01 |
| zhongjun | amito: I guess he thought if the share protocol isn't correct (your driver don't support it) , it could not find in your storage. Then like the share isn't exist in your storage | 09:13 |
| amito | zhongjun: ok. so what should we do in this case? just return from the function and do nothing? (try: ... except: return) ? | 09:14 |
| zhongjun | amito: so you could just skip it, the logic of delete this share is the same as the logic of delete the share isn't exist | 09:15 |
| zhongjun | amito: and log warning message | 09:16 |
| *** chenying has quit IRC | 09:27 | |
| *** chenying has joined #openstack-manila | 09:27 | |
| *** chenying has quit IRC | 09:31 | |
| *** chenying has joined #openstack-manila | 09:32 | |
| openstackgerrit | zhongjun proposed openstack/python-manilaclient master: Add 'count' in list command https://review.openstack.org/525498 | 09:32 |
| openstackgerrit | Amit Oren proposed openstack/manila master: INFINIDAT add Manila driver https://review.openstack.org/515432 | 09:42 |
| openstackgerrit | Merged openstack/manila master: Use v3 cinder client for share volume https://review.openstack.org/522478 | 09:54 |
| *** ganso has joined #openstack-manila | 09:55 | |
| *** a-pugachev has joined #openstack-manila | 09:56 | |
| *** dsariel has joined #openstack-manila | 09:58 | |
| *** dsariel has quit IRC | 10:32 | |
| *** ianychoi has quit IRC | 10:55 | |
| *** ianychoi has joined #openstack-manila | 10:55 | |
| openstackgerrit | Chris Yang proposed openstack/manila master: QNAP Manila driver: Access rule setting is override by the later access rule setting. https://review.openstack.org/525527 | 11:00 |
| *** dsariel has joined #openstack-manila | 11:02 | |
| *** caowei has quit IRC | 11:21 | |
| openstackgerrit | zhongjun proposed openstack/python-manilaclient master: Add 'count' in list command https://review.openstack.org/525498 | 11:43 |
| *** junboli has quit IRC | 12:38 | |
| *** mvenesio has joined #openstack-manila | 12:43 | |
| *** dustins has joined #openstack-manila | 13:13 | |
| *** rraja has joined #openstack-manila | 13:49 | |
| *** gcb has quit IRC | 14:53 | |
| *** gcb has joined #openstack-manila | 14:54 | |
| *** lpetrut has joined #openstack-manila | 14:55 | |
| *** jmlowe has joined #openstack-manila | 14:58 | |
| *** dsariel has quit IRC | 15:06 | |
| *** a-pugachev has quit IRC | 15:18 | |
| *** dsariel has joined #openstack-manila | 15:21 | |
| *** rraja has quit IRC | 15:23 | |
| *** gouthamr has joined #openstack-manila | 15:29 | |
| *** markstur has joined #openstack-manila | 15:37 | |
| *** dustins has quit IRC | 15:38 | |
| *** dustins has joined #openstack-manila | 15:39 | |
| amito | gouthamr: thanks :) | 15:39 |
| *** dsariel has quit IRC | 15:40 | |
| *** markstur has quit IRC | 15:42 | |
| *** markstur has joined #openstack-manila | 15:43 | |
| *** openstackgerrit has quit IRC | 15:48 | |
| *** dsariel has joined #openstack-manila | 15:53 | |
| *** jmlowe has quit IRC | 15:56 | |
| *** jmlowe has joined #openstack-manila | 15:57 | |
| *** jmlowe has quit IRC | 15:59 | |
| *** jmlowe_ has joined #openstack-manila | 16:01 | |
| *** jmlowe_ has quit IRC | 16:02 | |
| *** belmoreira has quit IRC | 16:04 | |
| gouthamr | amito: you're welcome.. | 16:16 |
| *** openstackgerrit has joined #openstack-manila | 16:52 | |
| openstackgerrit | Merged openstack/manila master: Update docs to fix broken links https://review.openstack.org/523364 | 16:52 |
| *** a-pugachev has joined #openstack-manila | 17:01 | |
| *** pcaruana has joined #openstack-manila | 17:22 | |
| *** a-pugachev has quit IRC | 17:22 | |
| *** lpetrut has quit IRC | 17:23 | |
| *** pcaruana has quit IRC | 17:26 | |
| *** pcaruana has joined #openstack-manila | 17:27 | |
| *** ganso has quit IRC | 17:32 | |
| *** chenying has quit IRC | 17:41 | |
| *** chenying has joined #openstack-manila | 17:41 | |
| *** pcaruana has quit IRC | 17:58 | |
| *** dsariel has quit IRC | 18:08 | |
| *** mvenesio has quit IRC | 18:16 | |
| *** mvenesio has joined #openstack-manila | 18:17 | |
| *** lpetrut has joined #openstack-manila | 18:25 | |
| *** a-pugachev has joined #openstack-manila | 18:25 | |
| openstackgerrit | Ben Swartzlander proposed openstack/manila master: Simplify the way drivers report support for ipv6 https://review.openstack.org/524236 | 18:43 |
| *** mvenesio has quit IRC | 19:10 | |
| openstackgerrit | Ben Swartzlander proposed openstack/manila master: Simplify the way drivers report support for ipv6 https://review.openstack.org/524236 | 19:13 |
| gouthamr | bswartz: http://git.openstack.org/cgit/openstack-infra/system-config/tree/modules/openstack_project/manifests/review.pp#n146 | 19:40 |
| *** dustins has quit IRC | 20:04 | |
| gouthamr | ... maybe it would be easier to implement add DHSS=True mode to the "LVM driver" than to retract that docstring .. :D https://review.openstack.org/#/c/524236/4/manila/share/drivers/lvm.py | 20:05 |
| tbarron | hah | 20:06 |
| tbarron | and then we could just deprecate generic driver and quit answering questions about it :D | 20:07 |
| *** lpetrut has quit IRC | 20:11 | |
| gouthamr | :P | 20:36 |
| *** dustins has joined #openstack-manila | 20:40 | |
| *** gouthamr has quit IRC | 22:04 | |
| *** threestrands has joined #openstack-manila | 22:05 | |
| *** threestrands has quit IRC | 22:05 | |
| *** threestrands has joined #openstack-manila | 22:05 | |
| *** dustins has quit IRC | 22:20 | |
| *** erlon has quit IRC | 22:22 | |
| *** a-pugachev has quit IRC | 22:32 | |
| *** gouthamr has joined #openstack-manila | 22:42 | |
| *** jmlowe has joined #openstack-manila | 23:09 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!