| *** yangyapeng has quit IRC | 00:00 | |
| *** akerr has joined #openstack-manila | 00:06 | |
| *** akerr_ has joined #openstack-manila | 00:08 | |
| *** akerr has quit IRC | 00:12 | |
| *** mtanino has quit IRC | 00:14 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Fix Share Migration improper behavior for drivers https://review.openstack.org/332267 | 00:26 |
|---|---|---|
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Share migration Newton improvements https://review.openstack.org/328431 | 00:27 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Add share type change to Share Migration https://review.openstack.org/345142 | 00:27 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Base Data Copy changes https://review.openstack.org/329174 | 00:28 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Driver Data Service Helper https://review.openstack.org/295836 | 00:28 |
| *** Suyi_ has quit IRC | 00:31 | |
| *** liyifeng has quit IRC | 00:45 | |
| *** yangyapeng has joined #openstack-manila | 01:13 | |
| *** gouthamr has joined #openstack-manila | 01:19 | |
| *** cknight has joined #openstack-manila | 01:19 | |
| *** zhonghua-lee has joined #openstack-manila | 01:34 | |
| *** gouthamr has quit IRC | 01:41 | |
| *** gouthamr has joined #openstack-manila | 01:42 | |
| *** liyifeng has joined #openstack-manila | 01:53 | |
| openstackgerrit | Yingzhe Zeng proposed openstack/manila: Implement replication support in huawei driver https://review.openstack.org/331586 | 01:55 |
| *** dmellado has quit IRC | 01:57 | |
| *** dmellado has joined #openstack-manila | 02:02 | |
| *** yangyapeng has quit IRC | 02:03 | |
| *** yangyape_ has joined #openstack-manila | 02:03 | |
| *** huyang has joined #openstack-manila | 02:18 | |
| *** furlongm_ has joined #openstack-manila | 02:28 | |
| *** furlongm has quit IRC | 02:29 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-manilaclient: Updated from global requirements https://review.openstack.org/348676 | 02:41 |
| *** gouthamr has quit IRC | 02:51 | |
| *** ganso has quit IRC | 02:51 | |
| *** yangyape_ has quit IRC | 02:57 | |
| *** yangyapeng has joined #openstack-manila | 02:58 | |
| *** yangyapeng has quit IRC | 03:47 | |
| *** yangyapeng has joined #openstack-manila | 03:48 | |
| *** yangyape_ has joined #openstack-manila | 03:53 | |
| *** yangyapeng has quit IRC | 03:53 | |
| *** cknight has quit IRC | 04:07 | |
| *** nkrinner_afk is now known as nkrinner | 04:24 | |
| *** yangyape_ has quit IRC | 04:29 | |
| *** jcsp has joined #openstack-manila | 05:04 | |
| *** yangyapeng has joined #openstack-manila | 05:05 | |
| *** narayrak has joined #openstack-manila | 05:06 | |
| *** yangyape_ has joined #openstack-manila | 05:29 | |
| *** yangyapeng has quit IRC | 05:29 | |
| *** akerr_ is now known as akerr_away | 05:35 | |
| *** lpetrut has joined #openstack-manila | 05:46 | |
| openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: [DNM] Tests for scenario tests https://review.openstack.org/340236 | 06:09 |
| *** pcaruana has joined #openstack-manila | 06:10 | |
| openstackgerrit | NidhiMittalHada proposed openstack/manila: share-size not set to 1 with 'manage_error' state https://review.openstack.org/348160 | 06:16 |
| *** nidhimittalhada has joined #openstack-manila | 06:22 | |
| nidhimittalhada | https://review.openstack.org/315012 @vponomaryov Can you please see this once ? | 06:22 |
| *** jcsp has quit IRC | 06:51 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: Fix concurrent usage of update_access method for share instances https://review.openstack.org/350647 | 06:54 |
| *** permalac has quit IRC | 06:54 | |
| *** lpetrut has quit IRC | 06:59 | |
| *** jcsp has joined #openstack-manila | 07:05 | |
| *** lpetrut has joined #openstack-manila | 07:11 | |
| openstackgerrit | NidhiMittalHada proposed openstack/manila: manila does not calculate allocated_capacity_gb https://review.openstack.org/351034 | 07:12 |
| *** lpetrut has quit IRC | 07:31 | |
| *** openstackgerrit has quit IRC | 07:33 | |
| *** openstackgerrit has joined #openstack-manila | 07:34 | |
| *** lpetrut has joined #openstack-manila | 07:40 | |
| *** yangyapeng has joined #openstack-manila | 07:53 | |
| *** yangyape_ has quit IRC | 07:54 | |
| *** furlongm_ has quit IRC | 07:54 | |
| *** lpetrut has quit IRC | 07:56 | |
| *** akapil has joined #openstack-manila | 07:59 | |
| *** yangyapeng has quit IRC | 08:13 | |
| *** yangyapeng has joined #openstack-manila | 08:13 | |
| *** nidhimittalhada has quit IRC | 08:20 | |
| *** nidhimittalhada has joined #openstack-manila | 08:21 | |
| *** nidhimittalhada has quit IRC | 08:24 | |
| *** jcsp has quit IRC | 08:34 | |
| openstackgerrit | liyifeng proposed openstack/manila-ui: Delete openstack/common from the exclude list of flake8 in tox https://review.openstack.org/351073 | 09:13 |
| *** lpetrut has joined #openstack-manila | 09:26 | |
| *** MikeG451 has quit IRC | 09:29 | |
| openstackgerrit | Marc Koderer proposed openstack/manila: Fix issue with snapshot creation in NetApp driver https://review.openstack.org/350473 | 09:37 |
| openstackgerrit | NidhiMittalHada proposed openstack/manila: share-size not set to 1 with 'manage_error' state https://review.openstack.org/348160 | 09:41 |
| *** MikeG451 has joined #openstack-manila | 09:44 | |
| *** permalac has joined #openstack-manila | 09:49 | |
| *** nidhimittalhada has joined #openstack-manila | 09:50 | |
| *** liyifeng has quit IRC | 09:51 | |
| *** akapil has quit IRC | 09:53 | |
| *** furlongm_ has joined #openstack-manila | 10:12 | |
| *** yangyapeng has quit IRC | 10:21 | |
| *** yangyapeng has joined #openstack-manila | 10:24 | |
| *** chlong has quit IRC | 10:28 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: Fix ZFSonLinux driver prerequisites setup https://review.openstack.org/351110 | 10:29 |
| *** yangyapeng has quit IRC | 10:32 | |
| openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: [DNM] Tests for scenario tests https://review.openstack.org/340236 | 10:35 |
| *** chlong has joined #openstack-manila | 10:41 | |
| *** kaisers has quit IRC | 10:50 | |
| *** kaisers has joined #openstack-manila | 10:51 | |
| *** kaisers has quit IRC | 10:51 | |
| *** akapil has joined #openstack-manila | 10:54 | |
| *** kaisers has joined #openstack-manila | 10:57 | |
| *** timcl has joined #openstack-manila | 11:09 | |
| *** yangyapeng has joined #openstack-manila | 11:26 | |
| *** chlong has quit IRC | 11:27 | |
| *** chlong has joined #openstack-manila | 11:28 | |
| *** dsariel has joined #openstack-manila | 11:29 | |
| *** nidhimittalhada has quit IRC | 11:50 | |
| *** adrianofr has joined #openstack-manila | 11:50 | |
| *** gouthamr has joined #openstack-manila | 12:01 | |
| *** eharney has quit IRC | 12:02 | |
| *** akerr_away is now known as akerr_ | 12:08 | |
| *** tpsilva has joined #openstack-manila | 12:14 | |
| openstackgerrit | NidhiMittalHada proposed openstack/manila: share-size not set to 1 with 'manage_error' state https://review.openstack.org/348160 | 12:15 |
| *** nidhimittalhada has joined #openstack-manila | 12:16 | |
| *** eharney has joined #openstack-manila | 12:25 | |
| *** xyang1 has joined #openstack-manila | 12:27 | |
| *** ganso has joined #openstack-manila | 12:31 | |
| *** baojg_ has joined #openstack-manila | 12:44 | |
| *** nidhimittalhada has quit IRC | 12:47 | |
| *** rraja has joined #openstack-manila | 12:49 | |
| gouthamr | vponomaryov: ping | 12:53 |
| vponomaryov | gouthamr: pong | 12:53 |
| gouthamr | vponomaryov: do you mind changing the parent of https://review.openstack.org/#/c/350647 to https://review.openstack.org/#/c/351110 -> i would like to see the ZFS job run on that patch.. cDOT's replication tests work okay after updating the lock text.. | 12:54 |
| vponomaryov | gouthamr: to see you can upload empty commit that depends on these both | 12:56 |
| gouthamr | vponomaryov: or we can merge this: https://review.openstack.org/#/c/351110 :P don't care either way.. | 12:56 |
| vponomaryov | gouthamr: there is no need to make dependency between them | 12:56 |
| vponomaryov | gouthamr: second proposal is ok )) | 12:57 |
| gouthamr | vponomaryov: nice. ganso agrees, awaiting another core review.. | 12:59 |
| vponomaryov | gouthamr: two migration tests can fail | 13:00 |
| vponomaryov | gouthamr: so, https://review.openstack.org/#/c/350647 the most correct one to be merged first from stability pint of view | 13:01 |
| gouthamr | vponomaryov: on the update_access fix? | 13:01 |
| vponomaryov | to test it with zfs fix you can upload some debug commit pulling these both dependencies | 13:01 |
| gouthamr | vponomaryov: yah, but i don't think it's appropriate running a vendor CI on replication tests and saying it's okay. #isaidit :) | 13:02 |
| gouthamr | vponomaryov: okay, will do. | 13:02 |
| *** faiz89 has joined #openstack-manila | 13:05 | |
| tpsilva | gouthamr, vponomaryov: any idead on how to correctly fix the other update_access bug? | 13:06 |
| tpsilva | I'll file it now on LP | 13:06 |
| tpsilva | s/idead/ideas | 13:06 |
| gouthamr | tpsilva: i vote for maintaining state on individual access rules | 13:06 |
| tpsilva | gouthamr: I vote for that one too | 13:07 |
| tpsilva | but who'll have time to do that? | 13:07 |
| gouthamr | tpsilva: we can discuss that today? | 13:07 |
| tpsilva | gouthamr: sure, on weekly meeting? | 13:08 |
| tpsilva | xyang1: ping | 13:08 |
| gouthamr | tpsilva: yep.. probably as the meat of teh concurrency discussion | 13:08 |
| tpsilva | gouthamr: okay, looking forward to see that finally working properly :) | 13:09 |
| *** alkhodos has joined #openstack-manila | 13:09 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila: [DNM] Does ZFSonLinux approve of our coordination changes https://review.openstack.org/351171 | 13:09 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/manila: [DNM] Does ZFSonLinux approve of our coordination changes https://review.openstack.org/351171 | 13:11 |
| xyang1 | tpsilva: hi | 13:12 |
| tpsilva | xyang1: hello Xing | 13:12 |
| tpsilva | xyang1: you requested that my new driver reports provisioned_capacity_gb | 13:13 |
| tpsilva | xyang1: what exactly should I report? | 13:13 |
| tpsilva | xyang1: the total used space by shares on my backend? | 13:13 |
| *** merooney has joined #openstack-manila | 13:13 | |
| xyang1 | tpsilva: the sum of shares in terms of GB provisioned on your backend | 13:15 |
| tpsilva | xyang1: but the actual used space, right? not the sum of their sizes? | 13:15 |
| openstackgerrit | Marc Koderer proposed openstack/manila: Add MTU information in DB and API https://review.openstack.org/349506 | 13:15 |
| xyang1 | tpsilva: the sum of the sizes | 13:16 |
| xyang1 | tpsilva: to check whether you have exceeded max over subscription ratio, we need to know how many shares you have provisioned, whether used or not | 13:17 |
| *** baojg_ has quit IRC | 13:17 | |
| xyang1 | tpsilva: you already report total and free, we can calculate used space by that | 13:18 |
| tpsilva | xyang1: oh, right, it makes sense | 13:18 |
| openstackgerrit | Grigoriy Rozhkov proposed openstack/manila: Add MapR-FS native driver https://review.openstack.org/350035 | 13:19 |
| tpsilva | xyang1: okay, I'll add that and push it in a moment, thank you | 13:19 |
| xyang1 | tpsilva: ok, sounds good | 13:19 |
| alkhodos | Hi everyone. I have a problem with testing required conf opts. I mock conf in my tests (https://review.openstack.org/#/c/309286/32/manila/tests/share/drivers/nexenta/ns5/test_nexenta_nas.py@41), but other drivers' test do not, causing an error: http://paste.openstack.org/show/549226/ What am I missing here? | 13:21 |
| *** dustins has joined #openstack-manila | 13:24 | |
| *** porrua has joined #openstack-manila | 13:27 | |
| zhongjun_ | xyang1, vponomaryov, ganso: Could you please review snapshot instance CLI code(https://review.openstack.org/#/c/304449/)? The part of snapshot instance BP(APIs code) have already been merged. Thanks | 13:28 |
| *** akapil has quit IRC | 13:28 | |
| *** akapil has joined #openstack-manila | 13:29 | |
| *** akapil has quit IRC | 13:29 | |
| *** akapil has joined #openstack-manila | 13:29 | |
| vponomaryov | zhongjun_: just did | 13:31 |
| *** akapil has quit IRC | 13:31 | |
| xyang1 | zhongjun_: sure | 13:33 |
| zhongjun_ | vponomaryov, xyang1: I see it, Thanks | 13:34 |
| gouthamr | zhongjun_: thanks for covering up all that tech debt... | 13:35 |
| *** tovchinnikova has joined #openstack-manila | 13:39 | |
| *** nkrinner is now known as nkrinner_afk | 13:43 | |
| ganso | gouthamr, vponomaryov: is there any value in negative tests in python-manilaclient | 13:43 |
| ganso | gouthamr, vponomaryov: ? | 13:43 |
| zhongjun_ | gouthamr: It's my pleasure. | 13:44 |
| ganso | gouthamr: we talked the other day that having migration negative tests could have some value | 13:44 |
| ganso | gouthamr: I got to it yesterday and saw that all we get is "CommandError" exception | 13:44 |
| ganso | vponomaryov, gouthamr: I know that at least the test is running the command, but if the command fails for whatever reason, I am not sure I cannot find out the cause | 13:45 |
| vponomaryov | ganso: who asked you to write negative tests? | 13:46 |
| gouthamr | ganso: ah.. yes, unless you can check the response code at least, i don't know what good it will do.. | 13:46 |
| vponomaryov | ganso: where? | 13:46 |
| ganso | gouthamr, vponomaryov: except for parsing the error message. I think we should agree if there is value or not | 13:46 |
| gouthamr | ganso: no.. lets not parse response text :( | 13:46 |
| ganso | vponomaryov: back in IRC, bswartz, gouthamr and I were talking the other day | 13:46 |
| ganso | gouthamr, vponomaryov: so, no negative functional python-manilaclient tests then | 13:48 |
| *** zhonghua-lee has quit IRC | 13:48 | |
| vponomaryov | ganso: their value is very low | 13:49 |
| vponomaryov | ganso: according to client behaviour | 13:49 |
| *** zhonghua-lee has joined #openstack-manila | 13:49 | |
| gouthamr | vponomaryov: recall this comment a long time ago on the share replica CLI patch: https://review.openstack.org/#/c/235448/ | 13:49 |
| zhongjun_ | gouthamr: In this patch:https://review.openstack.org/#/c/235448/ there is no functional test | 13:51 |
| vponomaryov | gouthamr: don't mess presence of some call to get the only possible result for "get" without support of it and "negative tests" when we have lots of positive | 13:52 |
| vponomaryov | gouthamr: so, when the only possible case to test CLI is to call non-existent thing, then we can cover it | 13:53 |
| vponomaryov | gouthamr: it is not the case now | 13:53 |
| vponomaryov | gouthamr: so, it had low value and still has | 13:54 |
| vponomaryov | gouthamr: and positive cases were not implemented anyway ))) | 13:54 |
| *** vbellur has quit IRC | 13:54 | |
| gouthamr | vponomaryov: no i agree share-replica-list could have been covered anyway; | 13:56 |
| * gouthamr has on a post it note on his desk ever since.. | 13:56 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Add share type change to Share Migration https://review.openstack.org/345142 | 13:57 |
| zhongjun_ | vponomaryov: such as lack of many snapshot function tests in manilaclient | 13:57 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Base Data Copy changes https://review.openstack.org/329174 | 13:57 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Driver Data Service Helper https://review.openstack.org/295836 | 13:58 |
| openstackgerrit | Tatiana Ovchinnikova proposed openstack/manila-ui: Fix Django 1.9 unit tests https://review.openstack.org/350210 | 13:59 |
| *** dustins has quit IRC | 14:04 | |
| *** dustins has joined #openstack-manila | 14:07 | |
| vponomaryov | ganso: ping | 14:15 |
| mkoderer | bswartz: we need to discuss about bug/1609306 ;) | 14:16 |
| mkoderer | https://bugs.launchpad.net/manila/+bug/1609306 | 14:16 |
| openstack | Launchpad bug 1609306 in Manila "Snapshot policy is set to None" [Low,Invalid] - Assigned to Marc Koderer (m-koderer) | 14:16 |
| vponomaryov | ganso: why after removing "notify" argument from "migration_start" interface we still need to return export locations? | 14:16 |
| mkoderer | bswartz: just give me a ping when available | 14:16 |
| bswartz | mkoderer: ping | 14:16 |
| vponomaryov | ganso: it should be enough to return it via "migration_complete" interface | 14:17 |
| bswartz | mkoderer: IMO it's working as designed | 14:17 |
| openstackgerrit | Merged openstack/python-manilaclient: Updated from global requirements https://review.openstack.org/348676 | 14:17 |
| mkoderer | bswartz: so https://github.com/openstack/manila/blob/master/manila/share/drivers/netapp/dataontap/client/client_cmode.py#L1207 set's the value to None | 14:18 |
| mkoderer | bswartz: I would say it shouldn't have any policy which such a value | 14:18 |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: [Dummy driver] Add share migration support https://review.openstack.org/347853 | 14:19 |
| bswartz | the behavior is ultimately controllable by the admin though | 14:19 |
| bswartz | if he doesn't specify the extra spec, then it implictly chooses the default policy | 14:20 |
| mkoderer | bswartz: maybe I am not clear what it will mean in our prod system. Without a share_type key "netapp:snapshot_polilcy" set to "none" will cause useless snapshots that shrinks the available size for our customers | 14:20 |
| zhongjun_ | vponomaryov: Even if the share list function include details parameter [1], the list command does not support set the details value, right? the details value always be true in share list command [1]https://github.com/openstack/python-manilaclient/blob/8c2f3c3a/manilaclient/v2/shares.py#L376 [2]https://github.com/openstack/python-manilaclient/blob/master/manilaclient/v2/shell.py#L1323 | 14:20 |
| bswartz | mkoderer: why can't you just set netapp:snapshot_polilcy=none to address the issue? | 14:20 |
| mkoderer | bswartz: because we have to set this for all share_type in all systems | 14:20 |
| bswartz | What do you mean "all systems"? Manila has a single set of share_types controlled by the manila admin | 14:21 |
| bswartz | do you have a large number of share_types? | 14:21 |
| *** vbellur has joined #openstack-manila | 14:21 | |
| mkoderer | bswartz: we have several clouds + several share_types | 14:21 |
| mkoderer | bswartz: so basically do you think auto snapshots are useful in a manila use case? | 14:22 |
| mkoderer | manila doesn't manage them and you cant delete them | 14:22 |
| bswartz | mkoderer: yes we make the assumption that if the admin doesn't modify the snapshot policy on the vserver and doesn't specify a different one that they really want those auto snapshots to be taken | 14:22 |
| vponomaryov | zhongjun_: 1) "python" interface can be used without CLI | 14:23 |
| *** yangyape_ has joined #openstack-manila | 14:23 | |
| vponomaryov | zhongjun_: 2) shares CLi returns always detailed | 14:23 |
| bswartz | they are useful to end users who can access them through the .snapshot directory | 14:23 |
| vponomaryov | zhongjun_: 3) you can consider it as a bug that CLI does not allow us to set provide such argument to python client | 14:23 |
| mkoderer | bswartz: ok I see | 14:23 |
| mkoderer | bswartz: that's might a valid point | 14:23 |
| bswartz | It's unfortunate that the .snapshot directory mechanism isn't standardized by Manila | 14:24 |
| bswartz | but it's something we get for free so we just leave it enabled | 14:24 |
| *** timcl has quit IRC | 14:25 | |
| mkoderer | bswartz: could we have a config switch that set's the default policy per backend if the extra spec is not set? | 14:25 |
| bswartz | and as I said before the admin has 2 ways to disable it if he doesn't want it | 14:25 |
| mkoderer | bswartz: this would be better for us... | 14:25 |
| openstackgerrit | Grigoriy Rozhkov proposed openstack/manila-ui: Add MapRFS protocol https://review.openstack.org/351217 | 14:25 |
| *** yangyapeng has quit IRC | 14:25 | |
| bswartz | mkoderer: yes we could, and that would be less objectionable because it wouldn't break existing behavior on upgrade, but I dislike 2 ways to do the same thing | 14:25 |
| *** akerr_ is now known as akerr | 14:26 | |
| mkoderer | bswartz: netapp_volume_snapshot_reserve_percent is also a config option | 14:26 |
| bswartz | mkoderer: you think it should be an extra spec instead? | 14:27 |
| bswartz | whoever added that must have thought there was no use in supporting different values for different share types | 14:27 |
| bswartz | we add config options when we believe the setting should apply to all share types, and we add extra specs when we believe admins may want to control the feature on a per-share-type basis | 14:28 |
| mkoderer | bswartz: I am fine with the config since we put it to 0 anyway ;) | 14:28 |
| zhongjun_ | vponomaryov: ok, so we'd better to add detailed in snapshot instance interface and python client first | 14:28 |
| bswartz | I agree it's a bit extra work to remember to have to set all the netapp:xxx extra specs on every share type | 14:29 |
| bswartz | but that could be scripted | 14:29 |
| bswartz | you could easily write and admin script that goes through every share type and ensures certain extra specs are set | 14:29 |
| bswartz | oh, I have to run | 14:29 |
| mkoderer | bswartz: sure we can.. even I dislike it ;) | 14:30 |
| bswartz | I was going to draw a picture to reference during today's IRC meeting | 14:30 |
| mkoderer | bswartz: ok I will abandon the patch | 14:30 |
| bswartz | mkoderer: as long as you understand the existing behavior was intentional, and changing it would break people on upgrade which isn't good | 14:30 |
| mkoderer | bswartz: ok I understand it | 14:30 |
| bswartz | your config option idea is more reasonable, but the downside is that 2 ways to control the same option leads to confusion | 14:31 |
| *** aovchinnikov has joined #openstack-manila | 14:31 | |
| *** mtanino has joined #openstack-manila | 14:31 | |
| mkoderer | bswartz: yeah such a opt would help us | 14:32 |
| mkoderer | bswartz: I can push a patch with it and you / the team can dedice if you want to accept it | 14:32 |
| *** catintheroof has joined #openstack-manila | 14:32 | |
| vponomaryov | zhongjun_: yes | 14:32 |
| openstackgerrit | Grigoriy Rozhkov proposed openstack/manila-ui: Add MapRFS protocol https://review.openstack.org/351217 | 14:38 |
| bswartz | mkoderer: well I prefer if you just use the extra spec | 14:38 |
| bswartz | or change the default policy on the ontap side | 14:38 |
| dustins | gouthamr: ping | 14:39 |
| mkoderer | bswartz: ok | 14:39 |
| *** vbellur has quit IRC | 14:41 | |
| *** yangyapeng has joined #openstack-manila | 14:53 | |
| *** yangyape_ has quit IRC | 14:54 | |
| *** zengyingzhe_ has joined #openstack-manila | 14:56 | |
| tpsilva | gouthamr: https://bugs.launchpad.net/manila/+bug/1609841 | 14:59 |
| openstack | Launchpad bug 1609841 in Manila "Access allow/deny APIs get stuck due to concurrency" [Undecided,New] | 14:59 |
| tpsilva | gouthamr: could you flag this appropriately? thanks | 15:00 |
| openstackgerrit | Merged openstack/manila: Fix ZFSonLinux driver prerequisites setup https://review.openstack.org/351110 | 15:00 |
| tpsilva | just filed it | 15:00 |
| *** timcl has joined #openstack-manila | 15:00 | |
| gouthamr | dustins: pong | 15:01 |
| gouthamr | tpsilva: sure | 15:01 |
| *** vbellur has joined #openstack-manila | 15:02 | |
| tpsilva | gouthamr: thank you! | 15:02 |
| dustins | gouthamr: Do you know where the API reference for share replication lives? | 15:03 |
| dustins | And if it's going to be a non-experimental feature going into Newton? | 15:03 |
| gouthamr | dustins: we don't have it yet | 15:03 |
| gouthamr | dustins: going to be experimental through newton | 15:04 |
| dustins | Okay, good to know. Thanks, Goutham! | 15:04 |
| *** pgbridge has quit IRC | 15:05 | |
| ganso | vponomaryov: it is optional | 15:07 |
| *** akshai has joined #openstack-manila | 15:07 | |
| *** gahan has left #openstack-manila | 15:09 | |
| *** faiz89 has quit IRC | 15:10 | |
| *** alkhodos has quit IRC | 15:11 | |
| *** akshai has quit IRC | 15:12 | |
| *** alkhodos has joined #openstack-manila | 15:14 | |
| *** zengyingzhe_ has quit IRC | 15:18 | |
| *** faiz89 has joined #openstack-manila | 15:23 | |
| *** catintheroof has quit IRC | 15:25 | |
| *** dustins has quit IRC | 15:28 | |
| *** dustins has joined #openstack-manila | 15:35 | |
| *** pgbridge has joined #openstack-manila | 15:45 | |
| *** faiz89 has quit IRC | 15:45 | |
| *** narayrak has quit IRC | 15:49 | |
| *** gouthamr-afk has joined #openstack-manila | 15:58 | |
| tpsilva | vponomaryov: ping | 15:58 |
| vponomaryov | tpsilva: ? | 15:59 |
| tpsilva | vponomaryov: not sure I understand the proposal on locks, but if the proposal is to not hold locks while driver is running, should your fix be changed? | 15:59 |
| tpsilva | vponomaryov: but I'm not sure if that proposal is for any locks on manila or just locks that involve the API | 16:01 |
| vponomaryov | tpsilva: current fix does not touch API | 16:01 |
| vponomaryov | tpsilva: only share manager uses it | 16:01 |
| tpsilva | vponomaryov: yeah, that's my question, so if the proposal is just for locks that involve the API, your change is ok | 16:02 |
| tpsilva | vponomaryov: if it's for any locks, then it's not | 16:02 |
| tpsilva | vponomaryov: but makes sense to be for locks that touch API | 16:02 |
| bswartz | tbarron: check your gerrit permissions real quick | 16:02 |
| vponomaryov | tpsilva: API locks should not be dependent on any other potentially long locks | 16:03 |
| bswartz | tbarron: I added you to group -- I don't know if it propogates the ACL change immediately or not | 16:03 |
| tpsilva | vponomaryov: ok, just wanted to check | 16:03 |
| vponomaryov | tpsilva: and it is completely up-to share manager to use it own locks if needed | 16:04 |
| vponomaryov | s/it/its/ | 16:04 |
| *** timcl has quit IRC | 16:05 | |
| tbarron | bswartz: yeah, i have new buttons, +/2, workflow | 16:07 |
| bswartz | cool | 16:07 |
| * tbarron is scared of pressing the wrong thing by mistake | 16:07 | |
| bswartz | time for my lunch then | 16:07 |
| openstackgerrit | Merged openstack/manila-ui: Fix Django 1.9 unit tests https://review.openstack.org/350210 | 16:07 |
| tbarron | bswartz: bon appetit | 16:07 |
| bswartz | tbarron: just use -2 for everything and we'll be fine | 16:07 |
| bswartz | o_O | 16:07 |
| openstackgerrit | zhongjun proposed openstack/python-manilaclient: Add snapshot instances admin CLIs https://review.openstack.org/304449 | 16:08 |
| *** gouthamr_afk has joined #openstack-manila | 16:08 | |
| bswartz | tbarron: seriously though -- watch out for the "workflow" button -- it tries to merge the patch without adding a +2 first, which is never what you want | 16:09 |
| vponomaryov | tbarron: but if you happen to click +workflow by mistake, just set -workflow afterwards | 16:09 |
| * bswartz thinks we've all made that mistake at least once | 16:10 | |
| *** gouthamr-afk has quit IRC | 16:12 | |
| *** eharney has quit IRC | 16:13 | |
| *** aorourke has joined #openstack-manila | 16:14 | |
| tbarron | vponomaryov: bswartz: roger that, thanks | 16:14 |
| vponomaryov | tbarron: also, you can doublecheck that it started/stopped running here -> http://status.openstack.org/zuul/ | 16:16 |
| vponomaryov | tbarron: also you pray that one of our voting jobs blows up one more time ^_^ | 16:17 |
| vponomaryov | s/you/you could/ =) | 16:17 |
| *** yangyapeng has quit IRC | 16:17 | |
| rraja | ganso: ping | 16:17 |
| *** yangyapeng has joined #openstack-manila | 16:19 | |
| *** rraja has quit IRC | 16:19 | |
| vponomaryov | tbarron: and the last one, you always can upload new Patch-Set and jenkins will surrender | 16:22 |
| tbarron | vponomaryov: ty for the pro tips! | 16:25 |
| *** timcl has joined #openstack-manila | 16:25 | |
| vponomaryov | tbarron: you are welcome =) | 16:27 |
| *** cknight has joined #openstack-manila | 16:29 | |
| *** kro_focused is now known as krotscheck | 16:32 | |
| *** sandanar has joined #openstack-manila | 16:33 | |
| *** cknight has quit IRC | 16:40 | |
| *** cknight has joined #openstack-manila | 16:40 | |
| *** cknight has quit IRC | 16:44 | |
| *** david-lyle has quit IRC | 16:53 | |
| *** david-lyle has joined #openstack-manila | 16:53 | |
| *** vbellur has quit IRC | 16:56 | |
| *** Suyi_ has joined #openstack-manila | 16:57 | |
| *** permalac has quit IRC | 17:11 | |
| *** sandanar has quit IRC | 17:11 | |
| *** vbellur has joined #openstack-manila | 17:11 | |
| *** eharney has joined #openstack-manila | 17:12 | |
| *** lpetrut has quit IRC | 17:15 | |
| openstackgerrit | Merged openstack/manila: [ZFSonLinux] Fix replicated snapshot deletion error https://review.openstack.org/349575 | 17:16 |
| *** tovchinnikova has quit IRC | 17:17 | |
| *** gouthamr_afk has quit IRC | 17:18 | |
| *** faiz89 has joined #openstack-manila | 17:21 | |
| *** aovchinnikov has quit IRC | 17:59 | |
| *** vbellur has quit IRC | 18:08 | |
| *** lpetrut has joined #openstack-manila | 18:09 | |
| *** tpsilva has quit IRC | 18:18 | |
| *** vbellur has joined #openstack-manila | 18:25 | |
| *** yangyapeng has quit IRC | 18:49 | |
| *** vbellur has quit IRC | 18:49 | |
| *** diegows has joined #openstack-manila | 19:03 | |
| *** vbellur has joined #openstack-manila | 19:03 | |
| diegows | hello | 19:03 |
| diegows | I'm doing some tests with manila, first steps... without using share servers, just simple LVM in the bare metal | 19:04 |
| diegows | nfs shared worked fine | 19:04 |
| diegows | but cifs doesn't work, nothing happens | 19:05 |
| diegows | is there an additional step required for cifs? | 19:05 |
| diegows | nfs worked with "create" and "access-allow" | 19:05 |
| diegows | but same steps with cifs does nothing | 19:05 |
| *** timcl has quit IRC | 19:10 | |
| *** akerr has quit IRC | 19:35 | |
| *** timcl has joined #openstack-manila | 19:56 | |
| *** akerr has joined #openstack-manila | 20:00 | |
| *** dustins has quit IRC | 20:00 | |
| *** bill_az has joined #openstack-manila | 20:02 | |
| *** eharney has quit IRC | 20:02 | |
| *** akerr_ has joined #openstack-manila | 20:02 | |
| *** dustins has joined #openstack-manila | 20:02 | |
| *** akerr has quit IRC | 20:05 | |
| *** merooney has quit IRC | 20:07 | |
| bswartz | diegows: what access type did you use | 20:09 |
| bswartz | for NFS, "ip" access is the correct type but for cifs, you need "user" access tyoe | 20:09 |
| bswartz | s/tyoe/type/ | 20:09 |
| diegows | bswartz: manila access-allow 21e059a9-20ea-4d22-929d-411bbba31a8d user test | 20:10 |
| bswartz | does the "test" user exist on that machine? | 20:10 |
| diegows | no, but i was expecting some error :) | 20:10 |
| bswartz | unfortunately users are configured out of band for CIFS | 20:10 |
| bswartz | diegows: that's a good point | 20:10 |
| bswartz | we don't validate that the username is meaninful because all we do is pass it down to samba | 20:11 |
| diegows | where is checking the user? | 20:11 |
| bswartz | we could add an enhancement to try to validate the user | 20:11 |
| diegows | share/drivers/helpers.py is the right file to see the code? | 20:11 |
| bswartz | the problem is that the validation would need to happen inside the driver and the error reporting would be asynchronous and challenging | 20:11 |
| bswartz | yes | 20:12 |
| *** porrua has quit IRC | 20:14 | |
| diegows | is there an example smb.conf somewhere? | 20:17 |
| diegows | to use with manila? | 20:17 |
| *** eharney has joined #openstack-manila | 20:19 | |
| diegows | ok, found something in manila-image-elements | 20:20 |
| *** timcl has quit IRC | 20:20 | |
| *** catintheroof has joined #openstack-manila | 20:30 | |
| *** lpetrut has quit IRC | 20:33 | |
| *** lpetrut has joined #openstack-manila | 20:35 | |
| diegows | bswartz: works... thanks for the hint | 20:35 |
| *** dsariel has quit IRC | 20:50 | |
| *** dustins has quit IRC | 21:14 | |
| *** eharney has quit IRC | 21:19 | |
| *** faiz89 has quit IRC | 21:27 | |
| *** dustins has joined #openstack-manila | 21:27 | |
| *** akerr_ has quit IRC | 21:30 | |
| *** akapil has joined #openstack-manila | 21:32 | |
| *** vbellur has quit IRC | 21:32 | |
| *** bill_az has quit IRC | 21:55 | |
| *** ganso has quit IRC | 22:01 | |
| *** gouthamr has quit IRC | 22:05 | |
| *** gouthamr has joined #openstack-manila | 22:05 | |
| *** gouthamr has quit IRC | 22:05 | |
| *** xyang1 has quit IRC | 22:22 | |
| *** furlongm_ has quit IRC | 22:30 | |
| *** furlongm_ has joined #openstack-manila | 22:31 | |
| *** lpetrut has quit IRC | 22:32 | |
| bswartz | diegows:np | 22:40 |
| *** dustins has quit IRC | 22:51 | |
| *** catintheroof has quit IRC | 23:13 | |
| *** akapil has quit IRC | 23:18 | |
| *** catintheroof has joined #openstack-manila | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!