| *** faiz89 has joined #openstack-manila | 00:00 | |
| *** faiz89 has quit IRC | 00:05 | |
| *** tommylikehu has joined #openstack-manila | 00:51 | |
| openstackgerrit | zhongjun proposed openstack/manila: Report IPv4 capabilities for IP drivers https://review.openstack.org/406776 | 01:05 |
|---|---|---|
| *** tuanluong has joined #openstack-manila | 01:46 | |
| *** zengyingzhe has quit IRC | 02:12 | |
| *** zengyingzhe has joined #openstack-manila | 02:13 | |
| *** catintheroof has quit IRC | 02:28 | |
| *** catintheroof has joined #openstack-manila | 02:30 | |
| *** catintheroof has quit IRC | 02:34 | |
| *** gouthamr has joined #openstack-manila | 02:56 | |
| *** FL1SK has quit IRC | 03:00 | |
| *** FL1SK has joined #openstack-manila | 03:01 | |
| openstackgerrit | zhongjun proposed openstack/manila: [TrivialFix] Move share type filter tempest to test_scheduler_stats.py https://review.openstack.org/409641 | 03:03 |
| *** xinyanzhang has joined #openstack-manila | 03:04 | |
| *** gouthamr has quit IRC | 03:11 | |
| openstackgerrit | xinyanzhang proposed openstack/manila: optimize get share access in huawei driver https://review.openstack.org/409025 | 03:32 |
| openstackgerrit | Tina Tang proposed openstack/manila: [Dell EMC Unity] Support create share smaller than 3 GB https://review.openstack.org/408448 | 03:51 |
| openstackgerrit | Merged openstack/manila: [devref] copy samples/local.conf correctly https://review.openstack.org/412233 | 04:16 |
| *** shausy has joined #openstack-manila | 05:04 | |
| *** lpetrut has joined #openstack-manila | 05:04 | |
| *** senk has joined #openstack-manila | 05:17 | |
| *** lpetrut has quit IRC | 05:26 | |
| *** senk has quit IRC | 05:38 | |
| *** gcb has joined #openstack-manila | 05:49 | |
| openstackgerrit | pangliye proposed openstack/manila: Add Apache 2.0 license to source file https://review.openstack.org/412291 | 06:03 |
| openstackgerrit | zhongjun proposed openstack/manila: Enable IPv6 in manila(allow access) https://review.openstack.org/312321 | 06:35 |
| openstackgerrit | pangliye proposed openstack/manila: Use assertGreaterEqual(x,len(y)) instead of assertTrue(x >= len(y)) https://review.openstack.org/412310 | 06:49 |
| *** lpetrut has joined #openstack-manila | 06:49 | |
| openstackgerrit | Pony Chou proposed openstack/manila: Add QNAP Manila Driver https://review.openstack.org/394703 | 06:54 |
| *** nherciu_ has joined #openstack-manila | 06:58 | |
| openstackgerrit | zhongjun proposed openstack/manila: Enable IPv6 in manila(allow access) https://review.openstack.org/312321 | 06:58 |
| *** rraja has joined #openstack-manila | 07:01 | |
| *** senk has joined #openstack-manila | 07:13 | |
| *** pcaruana has joined #openstack-manila | 07:33 | |
| *** senk has quit IRC | 07:39 | |
| *** lpetrut has quit IRC | 07:50 | |
| *** senk has joined #openstack-manila | 07:58 | |
| *** tuanluong has quit IRC | 08:00 | |
| openstackgerrit | zhongjun proposed openstack/manila: Add the ability to check the tenant quota in detail https://review.openstack.org/366614 | 08:12 |
| *** dsariel has joined #openstack-manila | 08:24 | |
| openstackgerrit | Pony Chou proposed openstack/manila: Add QNAP Manila Driver https://review.openstack.org/394703 | 08:31 |
| *** senk has quit IRC | 08:45 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/manila: Updated from global requirements https://review.openstack.org/411065 | 09:00 |
| *** tuanluong has joined #openstack-manila | 09:01 | |
| *** tuanluong_ has joined #openstack-manila | 09:11 | |
| *** ganso has joined #openstack-manila | 09:17 | |
| *** senk has joined #openstack-manila | 09:20 | |
| *** senk has quit IRC | 09:25 | |
| *** senk has joined #openstack-manila | 09:27 | |
| openstackgerrit | pangliye proposed openstack/manila: Use assertGreaterEqual(x,len(y)) instead of assertTrue(x >= len(y)) https://review.openstack.org/412310 | 09:38 |
| openstackgerrit | Tina Tang proposed openstack/manila: [Dell EMC Unity] Support create share smaller than 3 GB https://review.openstack.org/408448 | 09:39 |
| *** senk has quit IRC | 09:52 | |
| *** senk has joined #openstack-manila | 09:53 | |
| *** alyson_ has joined #openstack-manila | 09:59 | |
| *** lpetrut has joined #openstack-manila | 10:03 | |
| *** tuanluong has quit IRC | 10:16 | |
| *** senk has quit IRC | 10:30 | |
| openstackgerrit | Tiago Pasqualini da Silva proposed openstack/python-manilaclient: Add mountable snapshots support to manila client https://review.openstack.org/345625 | 10:32 |
| *** surabujin has quit IRC | 10:54 | |
| *** tuanluong_ has quit IRC | 10:55 | |
| *** ociuhandu has joined #openstack-manila | 11:10 | |
| *** lseki has joined #openstack-manila | 11:10 | |
| *** senk has joined #openstack-manila | 11:19 | |
| openstackgerrit | zhongjun proposed openstack/manila: Report IPv4 capabilities for IP drivers https://review.openstack.org/406776 | 11:19 |
| tommylikehu | ping ganso | 11:26 |
| ganso | tommylikehu: pong | 11:26 |
| tommylikehu | hey ganso, how did this id generated? | 11:27 |
| tommylikehu | in this link https://review.openstack.org/#/q/56c4ca3bcbebf749adf9bdf31ab0c9497e925bc3,n,z | 11:27 |
| openstackgerrit | Pony Chou proposed openstack/manila: Add QNAP Manila Driver https://review.openstack.org/394703 | 11:28 |
| ganso | tommylikehu: See "Commit56c4ca3bcbebf749adf9bdf31ab0c9497e925bc3 " in the spec patch | 11:28 |
| *** draynium has joined #openstack-manila | 11:29 | |
| tommylikehu | ganso: this one? https://review.openstack.org/#/c/392291/ but I could not find this id. (: | 11:31 |
| ganso | tommylikehu: Commit56c4ca3bcbebf749adf9bdf31ab0c9497e925bc3 | 11:36 |
| ganso | tommylikehu: my suggestion is: instead of "Support IPv6 in allow_access(312321)" | 11:36 |
| tommylikehu | I see :) | 11:37 |
| ganso | tommylikehu: use Support IPv6 in allow_access(2e3d6166ff78c0c629e5ab662452c8da5ab1f597) | 11:37 |
| tommylikehu | thanks | 11:37 |
| ganso | tommylikehu: same for "Please read spec for design detail: https://review.openstack.org/#/c/362786/" replace with the ID | 11:37 |
| tommylikehu | sure :) | 11:38 |
| openstackgerrit | zhongjun proposed openstack/manila: Trivial fix translate issues https://review.openstack.org/411569 | 11:38 |
| tommylikehu | I'd reword these commit messages | 11:38 |
| openstackgerrit | jeck proposed openstack/manila: Add Apache 2.0 license to source file https://review.openstack.org/412413 | 11:46 |
| dmellado | vponomaryov: o/ | 11:47 |
| dmellado | thought you were sleeping, that's why I pushed the patch! | 11:47 |
| vponomaryov | dmellado: what made you think so? | 11:49 |
| dmellado | vponomaryov: don't know why, but I had you in my mind in US timezone | 11:49 |
| dmellado | I do take note now xD | 11:49 |
| vponomaryov | dmellado: look at comments there | 11:50 |
| dmellado | yep, you're totally right | 11:50 |
| openstackgerrit | zhongjun proposed openstack/manila: Enable IPv6 in manila(allow access) https://review.openstack.org/312321 | 11:55 |
| *** ociuhandu has quit IRC | 11:57 | |
| openstackgerrit | zhongjun proposed openstack/manila: Enable IPv6 in manila(allow access) https://review.openstack.org/312321 | 11:58 |
| openstackgerrit | Tom Barron proposed openstack/manila-ui: Disable creation of public shares per configuration https://review.openstack.org/411891 | 12:00 |
| openstackgerrit | zhongjun proposed openstack/manila: Enable IPv6 in manila(network plugins and drivers) https://review.openstack.org/406776 | 12:04 |
| *** porrua has joined #openstack-manila | 12:06 | |
| *** senk has quit IRC | 12:12 | |
| *** catintheroof has joined #openstack-manila | 12:18 | |
| *** shausy has quit IRC | 12:29 | |
| openstackgerrit | Tom Barron proposed openstack/manila: NetApp: set proper broadcast domain for IPspace https://review.openstack.org/406042 | 12:40 |
| *** senk has joined #openstack-manila | 12:42 | |
| *** senk has quit IRC | 12:55 | |
| *** ociuhandu has joined #openstack-manila | 12:58 | |
| *** senk has joined #openstack-manila | 13:08 | |
| *** senk has quit IRC | 13:09 | |
| *** gcb has quit IRC | 13:14 | |
| *** senk has joined #openstack-manila | 13:20 | |
| *** senk has quit IRC | 13:24 | |
| *** gouthamr has joined #openstack-manila | 13:26 | |
| *** gcb has joined #openstack-manila | 13:36 | |
| *** tommylikehu_ has joined #openstack-manila | 13:38 | |
| tommylikehu_ | ping ganso | 13:45 |
| vkmc | hey all! | 13:46 |
| vkmc | I could use some reviews in manila-image-elements | 13:47 |
| vkmc | every feedback is appreciated | 13:47 |
| vkmc | https://review.openstack.org/#/c/405629/ | 13:47 |
| *** cknight has joined #openstack-manila | 14:00 | |
| *** xinyanzhang has quit IRC | 14:06 | |
| openstackgerrit | zhongjun proposed openstack/manila: Trivial fix translate issues https://review.openstack.org/411569 | 14:06 |
| *** xinyanzhang has joined #openstack-manila | 14:06 | |
| *** dustins has joined #openstack-manila | 14:09 | |
| *** xyang1 has joined #openstack-manila | 14:11 | |
| *** senk has joined #openstack-manila | 14:11 | |
| *** senk has quit IRC | 14:13 | |
| *** dustins has quit IRC | 14:13 | |
| *** dustins has joined #openstack-manila | 14:14 | |
| openstackgerrit | pangliye proposed openstack/manila: replace assertTrue(x >= len(y)) https://review.openstack.org/412310 | 14:14 |
| *** chlong has joined #openstack-manila | 14:14 | |
| openstackgerrit | pangliye proposed openstack/manila: Use assertGreaterEqual(x,len(y)) instead of assertTrue(x >= len(y)) https://review.openstack.org/412310 | 14:15 |
| vkmc | vponomaryov, ^ | 14:24 |
| vkmc | :D | 14:24 |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: [DNM] Test grenade fix https://review.openstack.org/412472 | 14:28 |
| openstackgerrit | Merged openstack/manila: [Tempest] Add scenario test creating share from snapshot https://review.openstack.org/400758 | 14:28 |
| *** a-pugachev has joined #openstack-manila | 14:30 | |
| openstackgerrit | Merged openstack/manila: Add Apache 2.0 license to source file https://review.openstack.org/412413 | 14:47 |
| *** eharney has joined #openstack-manila | 14:53 | |
| *** eharney has quit IRC | 15:00 | |
| *** tommylikehu_ has quit IRC | 15:01 | |
| ganso | vponomaryov, bswartz, markstur, tbarron, gouthamr, toabctl, xyang1, cknight: Hello folks. Could you please take a look at the share migration patches when you have some review time? https://review.openstack.org/#/q/ocata-migration-improvements Thanks in advance | 15:07 |
| *** eharney has joined #openstack-manila | 15:14 | |
| *** zhonghua has quit IRC | 15:24 | |
| arnewiebalck | ganso: ping | 15:24 |
| ganso | arnewiebalck: pong | 15:24 |
| *** zhonghua has joined #openstack-manila | 15:24 | |
| arnewiebalck | uh, that was quick :) | 15:24 |
| arnewiebalck | ganso: https://review.openstack.org/#/c/411384/ I’m not sure I can follow what you mean by your comment. | 15:25 |
| arnewiebalck | ganso: Or what I should do. | 15:25 |
| ganso | arnewiebalck: I downloaded the change and tested it. In the shares table it shows "extending_error" when resize fails attempting to extend | 15:26 |
| ganso | arnewiebalck: so, in what situation would resizing_error be used? | 15:27 |
| arnewiebalck | ganso: in the as extending_error was used plus an equivalent one when shrinking fails. | 15:27 |
| arnewiebalck | ganso: Where does the extending error you saw come from? | 15:28 |
| ganso | arnewiebalck: what do you mean "where does it come from?" it was displayed in the shares table | 15:28 |
| arnewiebalck | ganso: :) | 15:28 |
| arnewiebalck | ganso: where in the code is a left-opver of ‘entending_error’? | 15:29 |
| arnewiebalck | ganso, ah wait | 15:29 |
| ganso | arnewiebalck: the server returns extending_error in the API, the server does not know about "resizing_error" | 15:29 |
| arnewiebalck | ganso: this eror came back from the API when the extend failed? | 15:29 |
| ganso | arnewiebalck: not when it failed, as the call to the driver is an async RPC, but when listing the shares | 15:30 |
| ganso | arnewiebalck: the API returns a list of shares with {'status': 'extending_error'} | 15:30 |
| *** tommylikehu_ has joined #openstack-manila | 15:30 | |
| arnewiebalck | ganso: ah, I see. | 15:30 |
| arnewiebalck | ganso: so, you suggest to basically display “Resizing error” whenever there was an “extend_error” (or a “shrink_error”) ? | 15:32 |
| *** igajsin has quit IRC | 15:32 | |
| ganso | arnewiebalck: yes, *"extending_error" (or a *"shrinking_error") | 15:32 |
| *** cknight has quit IRC | 15:32 | |
| arnewiebalck | ganso, sorry, yes. | 15:32 |
| ganso | arnewiebalck: the "Resizing" feature is a clientside only feature at this moment | 15:32 |
| gouthamr | ganso: late ack, the queue's getting long, but i'll definitely take a look.. :) | 15:33 |
| ganso | arnewiebalck: so, the server understands extending and shrinking, not resizing | 15:33 |
| ganso | arnewiebalck: but the user sees resize through the client... and the server returns related to extending and shrinking, and the server needs to convert accordingly | 15:33 |
| arnewiebalck | ganso: how about we add a “shrinking_error” (in addition to “extendin_error”)? This would give more details to the user. | 15:33 |
| ganso | arnewiebalck: oh yes, we would need to add that somehow. Also, we could do nothing | 15:34 |
| ganso | arnewiebalck: I see 2 alternatives | 15:34 |
| ganso | arnewiebalck: 1) have the extending_error and shrinking_error, as when the users clicks the button resize and inputs the size, it will convert to one of those operations, so it will know "extending" and "shrinking", and if it fails it will say "extending error" or "shrinking error" according to what the user was trying to do through the resize button | 15:35 |
| ganso | (depending on which operation he triggered based on input) | 15:35 |
| ganso | arnewiebalck: 2) rename everything to resize. So we would have to merge "extending" and "shrinking" into "resizing", also "resizing error" instead of "extending error" and "shrinking error" | 15:36 |
| ganso | gouthamr: Hi Goutham | 15:36 |
| ganso | gouthamr: unfortunately I have no idea what you are ack-ing about lol | 15:36 |
| ganso | gouthamr: I don't remember what the last thing we talked about was | 15:37 |
| gouthamr | ganso: forget i did so :P | 15:37 |
| ganso | gouthamr: oh I remember now, to review the share migration patches lol | 15:38 |
| ganso | gouthamr: I thought you were referring to something else | 15:38 |
| gouthamr | :) | 15:38 |
| arnewiebalck | ganso: I’d go with option 1) for now as resize does not existon the CLI. So you may extnd on the CLI and see a resize error on the dashboard (or did your “everything” also include changing the CLI?). 1) sounds also easier to do ;) | 15:38 |
| ganso | arnewiebalck: I would go with #1 as well, all you need is to rename the extend form/button/logic to work as a Resize | 15:39 |
| ganso | arnewiebalck: and don't worry about displaying extending or shrinking statuses | 15:39 |
| ganso | arnewiebalck: BUT, you have to add shrinking status as they are missing | 15:40 |
| ganso | arnewiebalck: shrinking *statuses | 15:40 |
| arnewiebalck | ganso: will do, thx for the feedback! | 15:40 |
| ganso | gouthamr: I had access rules stuff in mind | 15:40 |
| ganso | gouthamr: btw, you're working on create_share_from_snapshot extra spec, right? | 15:41 |
| *** jcsp has joined #openstack-manila | 15:42 | |
| openstackgerrit | Vitaliy Levitski proposed openstack/manila: Add MapR-FS native driver https://review.openstack.org/350035 | 15:46 |
| *** jcsp has quit IRC | 15:47 | |
| *** jcsp has joined #openstack-manila | 15:47 | |
| openstackgerrit | Arne Wiebalck proposed openstack/manila-ui: Replace 'Extend Share' by 'Resize Share' https://review.openstack.org/411384 | 15:47 |
| *** mtanino has joined #openstack-manila | 15:50 | |
| *** rraja has quit IRC | 15:51 | |
| *** rraja has joined #openstack-manila | 15:53 | |
| *** rraja has quit IRC | 15:53 | |
| *** kambiz has quit IRC | 16:07 | |
| *** kambiz has joined #openstack-manila | 16:09 | |
| arnewiebalck | ganso: one thing I didn’t know how to handle in the Extend->Resize patch is this test: https://github.com/openstack/manila-ui/blob/master/manila_ui/tests/api/test_manila.py#L66 | 16:12 |
| arnewiebalck | ganso: shoud there be now a “test_share_shrink” test in that file? | 16:12 |
| ganso | arnewiebalck: would be good to have, yes, because now shrink can be invoked | 16:13 |
| arnewiebalck | ganso: does the “new_size” param matter? I couldn’t figure that out. | 16:13 |
| arnewiebalck | ganso: what I mean is: the newsize for shrink must be smaller than the current size. | 16:14 |
| ganso | arnewiebalck: let me double check | 16:14 |
| ganso | arnewiebalck: does not matter | 16:14 |
| *** pcaruana has quit IRC | 16:15 | |
| ganso | arnewiebalck: you're mocking the client behavior | 16:15 |
| ganso | arnewiebalck: you're testing a one line method | 16:15 |
| ganso | arnewiebalck: so just ensure it is invoking proper method with proper params | 16:15 |
| arnewiebalck | ganso: I see. I’ll add a test and re-submit. | 16:15 |
| ganso | arnewiebalck: thanks | 16:15 |
| arnewiebalck | ganso: seems I had such a test already added … | 16:18 |
| openstackgerrit | Merged openstack/manila: Enable scenario tests for LVM and ZFSonLinux drivers https://review.openstack.org/374581 | 16:22 |
| openstackgerrit | Merged openstack/manila: Updated from global requirements https://review.openstack.org/411065 | 17:09 |
| *** cknight has joined #openstack-manila | 17:19 | |
| *** dustins has quit IRC | 17:34 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Implement Revert-to-snapshot in HNAS Driver https://review.openstack.org/411923 | 17:35 |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: [DNM] debug 3.0 https://review.openstack.org/410366 | 17:41 |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila-specs: Update Share Migration Ocata Improvements Spec https://review.openstack.org/407145 | 17:43 |
| *** a-pugachev has quit IRC | 17:46 | |
| *** lpetrut has quit IRC | 17:48 | |
| *** ociuhandu has quit IRC | 17:49 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: [DNM] debug 3.0 https://review.openstack.org/410366 | 18:10 |
| *** ociuhandu has joined #openstack-manila | 18:20 | |
| *** dustins has joined #openstack-manila | 18:36 | |
| *** lpetrut has joined #openstack-manila | 18:40 | |
| *** dustins_ has joined #openstack-manila | 18:47 | |
| *** dustins has quit IRC | 18:50 | |
| *** breitz has quit IRC | 18:55 | |
| *** breitz has joined #openstack-manila | 18:55 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/python-manilaclient: [DNM] Test share group client changes https://review.openstack.org/412587 | 19:02 |
| bswartz | can someone in this channel PM me | 19:02 |
| bswartz | I want to test that IRC is acting normal | 19:03 |
| *** xinyanzhang has quit IRC | 19:13 | |
| *** david-lyle_ has joined #openstack-manila | 19:13 | |
| *** xinyanzhang has joined #openstack-manila | 19:13 | |
| *** openstackstatus has quit IRC | 19:13 | |
| *** david-lyle has quit IRC | 19:13 | |
| *** openstack has joined #openstack-manila | 19:16 | |
| *** eharney has quit IRC | 19:22 | |
| *** chlong has quit IRC | 19:34 | |
| *** chlong has joined #openstack-manila | 19:36 | |
| *** eharney has joined #openstack-manila | 19:38 | |
| *** dsariel has quit IRC | 19:44 | |
| *** dustins_ has quit IRC | 19:47 | |
| *** dustins has joined #openstack-manila | 19:48 | |
| *** lpetrut has quit IRC | 20:00 | |
| bswartz | thanks all I fixed my settings, I was blocking PMs from unregistered users and I forgot I was | 20:00 |
| *** senk has joined #openstack-manila | 20:02 | |
| *** breitz has quit IRC | 20:16 | |
| * gouthamr wonders how many people fell for bswartz's holiday trick | 20:53 | |
| tbarron | ls | 20:55 |
| *** senk has quit IRC | 20:55 | |
| *** breitz has joined #openstack-manila | 20:56 | |
| *** alyson_ has quit IRC | 21:06 | |
| *** porrua has quit IRC | 21:13 | |
| bswartz | rm -rf / | 21:13 |
| *** gouthamr has quit IRC | 21:17 | |
| tbarron | echo bswartz | 21:17 |
| bswartz | :) | 21:18 |
| *** porrua has joined #openstack-manila | 21:18 | |
| *** lseki has quit IRC | 21:20 | |
| *** dustins has quit IRC | 21:31 | |
| *** dustins has joined #openstack-manila | 21:32 | |
| *** xinyanzhang has quit IRC | 21:34 | |
| *** eharney has quit IRC | 21:40 | |
| *** cknight has quit IRC | 21:45 | |
| *** ganso has quit IRC | 22:13 | |
| *** dustins has quit IRC | 22:37 | |
| *** dustins has joined #openstack-manila | 22:37 | |
| *** xyang1 has quit IRC | 22:53 | |
| *** eharney has joined #openstack-manila | 22:58 | |
| *** dustins has quit IRC | 23:01 | |
| *** nherciu_ has quit IRC | 23:21 | |
| *** tommylikehu_ has quit IRC | 23:22 | |
| openstackgerrit | Merged openstack/manila: Trivial fix translate issues https://review.openstack.org/411569 | 23:30 |
| *** tommylikehu_ has joined #openstack-manila | 23:37 | |
| *** gouthamr has joined #openstack-manila | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!