*** haomaiwa_ has quit IRC | 00:01 | |
*** haomaiwa_ has joined #openstack-manila | 00:01 | |
*** qeelee has joined #openstack-manila | 00:47 | |
*** haomaiwa_ has quit IRC | 01:01 | |
*** haomaiwa_ has joined #openstack-manila | 01:01 | |
*** mtanino has quit IRC | 01:02 | |
*** ganso has joined #openstack-manila | 01:04 | |
*** dcain has joined #openstack-manila | 01:08 | |
*** dcain1 has joined #openstack-manila | 01:18 | |
*** yangyapeng has joined #openstack-manila | 01:19 | |
*** dcain has quit IRC | 01:21 | |
*** qeelee has quit IRC | 01:21 | |
*** qeelee has joined #openstack-manila | 01:24 | |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Implement update_access() in generic driver + LVM https://review.openstack.org/256281 | 01:33 |
---|---|---|
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Implement admin network in generic driver https://review.openstack.org/278699 | 01:34 |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Introduced Data Service https://review.openstack.org/244286 | 01:35 |
*** haomaiwa_ has quit IRC | 01:36 | |
*** yangyapeng has quit IRC | 01:37 | |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Move Share Migration code to Data Service https://review.openstack.org/250515 | 01:43 |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Add new Share migration API functions https://review.openstack.org/259642 | 01:43 |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: [Do not merge] Test LVM mount capability https://review.openstack.org/282507 | 01:44 |
*** qeelee has quit IRC | 02:02 | |
*** qeelee has joined #openstack-manila | 02:04 | |
*** qeelee has quit IRC | 02:11 | |
*** cknight has joined #openstack-manila | 02:11 | |
*** haomaiwang has joined #openstack-manila | 02:13 | |
*** jasonsb has joined #openstack-manila | 02:15 | |
*** haomaiwang has quit IRC | 02:18 | |
*** a_ta has joined #openstack-manila | 02:19 | |
*** cknight has quit IRC | 02:23 | |
*** a_ta has quit IRC | 02:23 | |
*** houming has joined #openstack-manila | 02:31 | |
*** haomaiwang has joined #openstack-manila | 02:37 | |
*** haomaiwang has quit IRC | 02:39 | |
*** haomaiwa_ has joined #openstack-manila | 02:41 | |
*** haomaiwa_ has quit IRC | 02:42 | |
*** haomaiwang has joined #openstack-manila | 02:42 | |
*** haomaiwang has quit IRC | 02:43 | |
*** haomaiwa_ has joined #openstack-manila | 02:43 | |
*** haomaiwa_ has quit IRC | 02:44 | |
*** 7GHAAB0B1 has joined #openstack-manila | 02:44 | |
*** 7GHAAB0B1 has quit IRC | 02:45 | |
*** haomaiwa_ has joined #openstack-manila | 02:45 | |
*** haomaiwa_ has quit IRC | 02:46 | |
openstackgerrit | liucheng proposed openstack/manila: Validate qos during share creation https://review.openstack.org/280127 | 02:46 |
*** haomaiwa_ has joined #openstack-manila | 02:46 | |
*** haomaiwa_ has quit IRC | 02:47 | |
*** haomaiwa_ has joined #openstack-manila | 02:47 | |
*** haomaiwa_ has quit IRC | 02:48 | |
*** 92AAACW2S has joined #openstack-manila | 02:48 | |
*** 92AAACW2S has quit IRC | 02:49 | |
*** haomaiwang has joined #openstack-manila | 02:49 | |
*** haomaiwang has quit IRC | 02:50 | |
*** haomaiwang has joined #openstack-manila | 02:50 | |
*** haomaiwang has quit IRC | 02:51 | |
*** 92AAACW3M has joined #openstack-manila | 02:51 | |
*** 92AAACW3M has quit IRC | 02:52 | |
*** haomaiwang has joined #openstack-manila | 02:52 | |
*** haomaiwang has quit IRC | 02:53 | |
*** haomaiwang has joined #openstack-manila | 02:53 | |
*** haomaiwang has quit IRC | 02:54 | |
*** haomaiwa_ has joined #openstack-manila | 02:54 | |
*** haomaiwa_ has quit IRC | 02:55 | |
*** haomaiwang has joined #openstack-manila | 02:55 | |
*** haomaiwang has quit IRC | 02:56 | |
*** haomaiwa_ has joined #openstack-manila | 02:56 | |
*** martyturner has joined #openstack-manila | 02:56 | |
*** haomaiwa_ has quit IRC | 02:57 | |
*** haomaiwang has joined #openstack-manila | 02:57 | |
*** haomaiwang has quit IRC | 02:58 | |
*** haomaiwa_ has joined #openstack-manila | 02:58 | |
*** haomaiwa_ has quit IRC | 02:59 | |
*** haomaiwa_ has joined #openstack-manila | 02:59 | |
*** haomaiwa_ has quit IRC | 03:00 | |
*** haomaiwa_ has joined #openstack-manila | 03:00 | |
*** haomaiwa_ has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-manila | 03:01 | |
*** haomaiwang has quit IRC | 03:02 | |
*** martyturner has quit IRC | 03:02 | |
*** haomaiwa_ has joined #openstack-manila | 03:03 | |
*** haomaiwa_ has quit IRC | 03:04 | |
*** haomaiwang has joined #openstack-manila | 03:04 | |
*** haomaiwang has quit IRC | 03:05 | |
*** haomaiwang has joined #openstack-manila | 03:05 | |
*** haomaiwang has quit IRC | 03:06 | |
*** haomaiwa_ has joined #openstack-manila | 03:06 | |
*** haomaiwa_ has quit IRC | 03:07 | |
*** haomaiwa_ has joined #openstack-manila | 03:07 | |
*** haomaiwa_ has quit IRC | 03:08 | |
*** haomaiwang has joined #openstack-manila | 03:08 | |
*** haomaiwang has quit IRC | 03:09 | |
*** haomaiwang has joined #openstack-manila | 03:09 | |
*** haomaiwang has quit IRC | 03:10 | |
*** haomaiwa_ has joined #openstack-manila | 03:10 | |
*** haomaiwa_ has quit IRC | 03:11 | |
*** haomaiwa_ has joined #openstack-manila | 03:11 | |
*** haomaiwa_ has quit IRC | 03:12 | |
*** cknight has joined #openstack-manila | 03:12 | |
*** haomaiwang has joined #openstack-manila | 03:12 | |
*** haomaiwang has quit IRC | 03:13 | |
*** haomaiwa_ has joined #openstack-manila | 03:13 | |
*** haomaiwa_ has quit IRC | 03:14 | |
*** gouthamr_ has quit IRC | 03:14 | |
*** haomaiwang has joined #openstack-manila | 03:14 | |
*** haomaiwang has quit IRC | 03:15 | |
*** haomaiwang has joined #openstack-manila | 03:15 | |
*** haomaiwang has quit IRC | 03:16 | |
*** dcain has joined #openstack-manila | 03:16 | |
*** haomaiwa_ has joined #openstack-manila | 03:16 | |
*** haomaiwa_ has quit IRC | 03:17 | |
*** 16WAADE7X has joined #openstack-manila | 03:17 | |
*** 16WAADE7X has quit IRC | 03:18 | |
*** haomaiwang has joined #openstack-manila | 03:18 | |
*** dcain1 has quit IRC | 03:19 | |
*** haomaiwang has quit IRC | 03:19 | |
*** haomaiwa_ has joined #openstack-manila | 03:19 | |
*** haomaiwa_ has quit IRC | 03:20 | |
*** a_ta has joined #openstack-manila | 03:20 | |
*** haomaiwa_ has joined #openstack-manila | 03:20 | |
*** haomaiwa_ has quit IRC | 03:21 | |
*** haomaiwa_ has joined #openstack-manila | 03:21 | |
*** haomaiwa_ has quit IRC | 03:22 | |
*** haomaiwa_ has joined #openstack-manila | 03:22 | |
*** haomaiwa_ has quit IRC | 03:23 | |
*** haomaiwa_ has joined #openstack-manila | 03:23 | |
*** haomaiwa_ has quit IRC | 03:24 | |
*** haomaiwa_ has joined #openstack-manila | 03:24 | |
*** a_ta has quit IRC | 03:24 | |
*** haomaiwa_ has quit IRC | 03:25 | |
*** haomaiwang has joined #openstack-manila | 03:25 | |
*** haomaiwang has quit IRC | 03:26 | |
*** haomaiwang has joined #openstack-manila | 03:26 | |
*** haomaiwang has quit IRC | 03:27 | |
*** haomaiwa_ has joined #openstack-manila | 03:27 | |
*** haomaiwa_ has quit IRC | 03:28 | |
*** dcain1 has joined #openstack-manila | 03:28 | |
*** haomaiwang has joined #openstack-manila | 03:28 | |
*** haomaiwang has quit IRC | 03:29 | |
*** haomaiwang has joined #openstack-manila | 03:29 | |
*** haomaiwang has quit IRC | 03:30 | |
*** haomaiwang has joined #openstack-manila | 03:30 | |
*** haomaiwang has quit IRC | 03:31 | |
*** haomaiwang has joined #openstack-manila | 03:31 | |
*** dcain has quit IRC | 03:31 | |
*** haomaiwang has quit IRC | 03:32 | |
*** haomaiwa_ has joined #openstack-manila | 03:32 | |
*** haomaiwa_ has quit IRC | 03:33 | |
*** haomaiwa_ has joined #openstack-manila | 03:33 | |
*** haomaiwa_ has quit IRC | 03:34 | |
*** haomaiwa_ has joined #openstack-manila | 03:34 | |
*** haomaiwa_ has quit IRC | 03:35 | |
*** haomaiwang has joined #openstack-manila | 03:35 | |
*** haomaiwang has quit IRC | 03:36 | |
*** 7YUAADLUF has joined #openstack-manila | 03:36 | |
*** 7YUAADLUF has quit IRC | 03:37 | |
*** haomaiwang has joined #openstack-manila | 03:37 | |
*** haomaiwang has quit IRC | 03:38 | |
*** haomaiwa_ has joined #openstack-manila | 03:38 | |
*** haomaiwa_ has quit IRC | 03:39 | |
*** haomaiwang has joined #openstack-manila | 03:39 | |
*** haomaiwang has quit IRC | 03:40 | |
*** haomaiwa_ has joined #openstack-manila | 03:40 | |
*** haomaiwa_ has quit IRC | 03:41 | |
*** haomaiwang has joined #openstack-manila | 03:41 | |
*** haomaiwang has quit IRC | 03:42 | |
*** 16WAADFGS has joined #openstack-manila | 03:42 | |
*** 16WAADFGS has quit IRC | 03:43 | |
*** haomaiwa_ has joined #openstack-manila | 03:43 | |
*** haomaiwa_ has quit IRC | 03:44 | |
*** haomaiwang has joined #openstack-manila | 03:44 | |
*** haomaiwang has quit IRC | 03:45 | |
*** haomaiwang has joined #openstack-manila | 03:45 | |
*** haomaiwang has quit IRC | 03:46 | |
*** haomaiwang has joined #openstack-manila | 03:46 | |
*** haomaiwang has quit IRC | 03:47 | |
*** haomaiwang has joined #openstack-manila | 03:47 | |
*** haomaiwang has quit IRC | 03:48 | |
*** haomaiwa_ has joined #openstack-manila | 03:48 | |
*** haomaiwa_ has quit IRC | 03:49 | |
*** haomaiwang has joined #openstack-manila | 03:49 | |
*** haomaiwang has quit IRC | 03:50 | |
*** haomaiwang has joined #openstack-manila | 03:50 | |
*** haomaiwang has quit IRC | 03:51 | |
*** haomaiwang has joined #openstack-manila | 03:51 | |
*** cknight has quit IRC | 03:51 | |
*** haomaiwang has quit IRC | 03:52 | |
*** haomaiwa_ has joined #openstack-manila | 03:52 | |
*** haomaiwa_ has quit IRC | 03:53 | |
*** haomaiwang has joined #openstack-manila | 03:53 | |
*** haomaiwang has quit IRC | 03:54 | |
*** haomaiwa_ has joined #openstack-manila | 03:54 | |
*** haomaiwa_ has quit IRC | 03:55 | |
*** haomaiwa_ has joined #openstack-manila | 03:55 | |
*** haomaiwa_ has quit IRC | 03:56 | |
*** haomaiwa_ has joined #openstack-manila | 03:56 | |
*** haomaiwa_ has quit IRC | 03:57 | |
*** haomaiwang has joined #openstack-manila | 03:57 | |
*** haomaiwang has quit IRC | 03:58 | |
*** haomaiwang has joined #openstack-manila | 03:58 | |
*** haomaiwang has quit IRC | 03:59 | |
*** haomaiwa_ has joined #openstack-manila | 03:59 | |
*** haomaiwa_ has quit IRC | 04:00 | |
*** haomaiwang has joined #openstack-manila | 04:00 | |
*** haomaiwang has quit IRC | 04:01 | |
*** haomaiwa_ has joined #openstack-manila | 04:01 | |
*** haomaiwa_ has joined #openstack-manila | 04:02 | |
*** haomaiwa_ has quit IRC | 04:03 | |
*** haomaiwa_ has joined #openstack-manila | 04:03 | |
*** haomaiwa_ has quit IRC | 04:04 | |
*** haomaiwang has joined #openstack-manila | 04:04 | |
openstackgerrit | Ben Swartzlander proposed openstack/manila: Allow devstack plugin to work without Cinder https://review.openstack.org/278562 | 04:04 |
*** haomaiwang has quit IRC | 04:05 | |
*** 7GHAAB07H has joined #openstack-manila | 04:05 | |
*** 7GHAAB07H has quit IRC | 04:06 | |
*** 7GHAAB07P has joined #openstack-manila | 04:06 | |
*** 7GHAAB07P has quit IRC | 04:07 | |
*** 7GHAAB07X has joined #openstack-manila | 04:07 | |
*** 7GHAAB07X has quit IRC | 04:08 | |
*** 7GHAAB079 has joined #openstack-manila | 04:08 | |
*** 7GHAAB079 has quit IRC | 04:09 | |
*** 7GHAAB08J has joined #openstack-manila | 04:09 | |
*** ganso has quit IRC | 04:09 | |
*** 7GHAAB08J has quit IRC | 04:10 | |
*** haomaiwa_ has joined #openstack-manila | 04:10 | |
*** haomaiwa_ has quit IRC | 04:11 | |
*** haomaiwa_ has joined #openstack-manila | 04:11 | |
*** haomaiwa_ has quit IRC | 04:12 | |
*** haomaiwa_ has joined #openstack-manila | 04:12 | |
*** haomaiwa_ has quit IRC | 04:13 | |
*** haomaiwa_ has joined #openstack-manila | 04:13 | |
*** haomaiwa_ has quit IRC | 04:14 | |
*** haomaiwang has joined #openstack-manila | 04:14 | |
*** shausy has joined #openstack-manila | 04:15 | |
*** haomaiwang has quit IRC | 04:15 | |
*** haomaiwang has joined #openstack-manila | 04:15 | |
*** haomaiwa_ has joined #openstack-manila | 04:16 | |
*** haomaiwa_ has quit IRC | 04:17 | |
*** haomaiwa_ has joined #openstack-manila | 04:17 | |
*** haomaiwa_ has quit IRC | 04:18 | |
*** 64MAADWOE has joined #openstack-manila | 04:18 | |
*** 64MAADWOE has quit IRC | 04:19 | |
*** 64MAADWOO has joined #openstack-manila | 04:19 | |
*** 64MAADWOO has quit IRC | 04:20 | |
*** haomaiwang has joined #openstack-manila | 04:20 | |
*** haomaiwang has quit IRC | 04:21 | |
*** deepakcs has joined #openstack-manila | 04:29 | |
*** deepakcs has quit IRC | 04:33 | |
*** shausy has quit IRC | 05:07 | |
*** shausy has joined #openstack-manila | 05:07 | |
*** cknight has joined #openstack-manila | 05:18 | |
*** cknight has quit IRC | 05:20 | |
*** dcain1 has quit IRC | 05:24 | |
*** zhongjun has quit IRC | 05:56 | |
*** zhongjun has joined #openstack-manila | 05:57 | |
*** houming has quit IRC | 05:59 | |
*** lpetrut has joined #openstack-manila | 06:05 | |
*** nkrinner has joined #openstack-manila | 06:21 | |
*** lpetrut has quit IRC | 06:41 | |
*** lpetrut has joined #openstack-manila | 06:47 | |
*** lpetrut has quit IRC | 07:02 | |
*** lpetrut has joined #openstack-manila | 07:15 | |
*** a_ta has joined #openstack-manila | 07:21 | |
*** houming has joined #openstack-manila | 07:26 | |
*** a_ta has quit IRC | 07:26 | |
*** lpetrut has quit IRC | 07:26 | |
*** ociuhandu has quit IRC | 07:48 | |
*** ociuhandu has joined #openstack-manila | 07:57 | |
*** pcaruana has joined #openstack-manila | 07:58 | |
*** lpetrut has joined #openstack-manila | 08:04 | |
*** pcaruana is now known as pcaruana|afk| | 08:08 | |
*** pcaruana|afk| is now known as pcaruana | 08:08 | |
*** ociuhandu has quit IRC | 08:40 | |
*** rraja has joined #openstack-manila | 08:40 | |
mkoderer | vponomaryov: BUILD_TIMEOUT seems to be set to 80 secs but I can't find where .. | 08:43 |
mkoderer | http://logs.openstack.org/18/280318/6/check/gate-manila-tempest-dsvm-neutron-scenario-multibackend/76b49c1/logs/tempest_conf.txt.gz | 08:43 |
mkoderer | vponomaryov: in the validation section it's set to 80 secs but for lib/tempest its 196 | 08:44 |
*** haomaiwang has joined #openstack-manila | 08:57 | |
openstackgerrit | Marc Koderer proposed openstack/manila: Fix scenario tests https://review.openstack.org/280318 | 09:00 |
*** lpetrut has quit IRC | 09:00 | |
*** haomaiwang has quit IRC | 09:01 | |
*** haomaiwa_ has joined #openstack-manila | 09:01 | |
*** haomaiwa_ has quit IRC | 09:02 | |
*** haomaiwang has joined #openstack-manila | 09:02 | |
*** lpetrut has joined #openstack-manila | 09:16 | |
*** yangyapeng has joined #openstack-manila | 09:19 | |
*** ociuhandu has joined #openstack-manila | 09:24 | |
*** ociuhandu has quit IRC | 09:42 | |
*** houming has quit IRC | 09:56 | |
*** haomaiwang has quit IRC | 10:01 | |
*** haomaiwa_ has joined #openstack-manila | 10:01 | |
*** yangyapeng has quit IRC | 10:17 | |
*** a_ta has joined #openstack-manila | 10:20 | |
openstackgerrit | Marc Koderer proposed openstack/manila: [WIP] Add multi-segment support https://review.openstack.org/277731 | 10:24 |
openstackgerrit | Marc Koderer proposed openstack/manila: [WIP] Add neutron driver for binding https://review.openstack.org/283494 | 10:24 |
*** a_ta has quit IRC | 10:25 | |
openstackgerrit | Sam Wan proposed openstack/manila: Replace TENANT => PROJECT for manila plugin https://review.openstack.org/283495 | 10:27 |
vponomaryov | mkoderer: looks like BUILD_TIMEOUT is redefined in some expression set is hidden in logs | 10:29 |
vponomaryov | mkoderer: we can get it know setting "set -v" in each shell | 10:29 |
vponomaryov | mkoderer: but, we do not really need to set up this opt, it is set up in tempest to 196 and then redefined by our post hook | 10:30 |
*** tpsilva has joined #openstack-manila | 10:33 | |
mkoderer | vponomaryov: ok so I will set it to 196 hardcoded then | 10:42 |
vponomaryov | mkoderer: no | 10:43 |
vponomaryov | mkoderer: just remove that line | 10:43 |
vponomaryov | mkoderer: it is already set to 196 | 10:43 |
mkoderer | vponomaryov: ah ok fine let's try that one | 10:43 |
vponomaryov | but better to get know where it is redefined using "set -v" hack | 10:43 |
vponomaryov | someone redefines this env var after run of main devstack script | 10:44 |
vponomaryov | but before run of post hook | 10:44 |
mkoderer | vponomaryov: yeah I will dig a bit in it.. in meanwhile I just want to see the scenario test passing :) | 10:45 |
openstackgerrit | Marc Koderer proposed openstack/manila: Fix scenario tests https://review.openstack.org/280318 | 10:45 |
vponomaryov | mkoderer: everyone wants see it passing )) | 10:46 |
mkoderer | vponomaryov: btw I pushed another change for hpb support https://review.openstack.org/#/c/283494/ | 10:47 |
mkoderer | vponomaryov: hopefully I can resuse the Ironic binding mechanism | 10:47 |
mkoderer | this would ease up things alot | 10:47 |
mkoderer | but I need to verify it in our labs.. so it's still WiP | 10:48 |
vponomaryov | mkoderer: why do you need status of a port? also,have you seen my comment in appropriate etherpad? | 10:48 |
mkoderer | vponomaryov: yeah I saw it | 10:48 |
mkoderer | vponomaryov: the thing is that if the network fabric does an actual port bind it could fail (or take longer) | 10:49 |
mkoderer | vponomaryov: nova and neutron have a special notifier rpc class for that | 10:49 |
mkoderer | I want to avoid that in manila but caching the state can be tricky | 10:50 |
vponomaryov | mkoderer: so, you want transparency? | 10:50 |
vponomaryov | mkoderer: Manila does not cache state/status of a port at all | 10:50 |
mkoderer | vponomaryov: yeah due to the fact the the state database fiels is not really used | 10:50 |
mkoderer | s/fiels/field/ | 10:50 |
vponomaryov | mkoderer: what "state"? | 10:51 |
vponomaryov | mkoderer: it is absent | 10:51 |
vponomaryov | mkoderer: absent thing cannot be used | 10:51 |
mkoderer | vponomaryov: https://github.com/openstack/manila/blob/master/manila/network/neutron/neutron_network_plugin.py#L127 | 10:51 |
mkoderer | vponomaryov: so the real port state is currently ignored | 10:52 |
vponomaryov | mkoderer: ohh, this one | 10:53 |
vponomaryov | mkoderer: so, you want Manila report proper info? | 10:53 |
vponomaryov | mkoderer: calling rpc of neutron? | 10:53 |
mkoderer | vponomaryov: I am thining about it | 10:53 |
mkoderer | vponomaryov: but as first step manila should wait until the port state is "active".. at least if a real binding takes place | 10:54 |
openstackgerrit | Thomas Bechtold proposed openstack/manila: Check for device node availability before mkfs https://review.openstack.org/281849 | 10:55 |
*** marcusvrn_ has joined #openstack-manila | 10:56 | |
openstackgerrit | Silvan Kaiser proposed openstack/manila: Add update_access() interface to Quobyte driver https://review.openstack.org/281136 | 10:57 |
rraja | tpsilva: ping | 10:58 |
tpsilva | rraja: pong | 10:58 |
rraja | tpsilva: hi! about update_access for a driver, should a driver stop updating the access rule even if one of the rules in the rule list (delete/add/access_rule) is invalid (not the right access level or type for the driver) or just skip the invalid valid ones? | 11:00 |
rraja | s/invalid valid/invalid/ | 11:00 |
*** haomaiwa_ has quit IRC | 11:01 | |
*** zhonghua is now known as zhonghua-lee | 11:01 | |
*** haomaiwang has joined #openstack-manila | 11:01 | |
tpsilva | rraja: when adding rules, you should raise an exception, making the rules go to an error state. When removing rules, you should ignore invalid rules. This will remove any invalid rules that might be present, getting back to an active state | 11:04 |
rraja | tpsilva: and this is what line 217 of https://review.openstack.org/#/c/256281/30/manila/share/drivers/helpers.py tries do, right? | 11:07 |
tpsilva | rraja: yes... see rodrigo's reply to clinton's comment | 11:12 |
*** ociuhandu has joined #openstack-manila | 11:12 | |
tpsilva | this validation wouldn't be needed, since exportfs does not raise an error, but we found an edge case | 11:12 |
*** rraja_ has joined #openstack-manila | 11:13 | |
rraja | tpsilva: yeah, i saw that. thanks! i was not so confident about my understanding as couple of drivers' update_access that are in review don't seem to skip the invalid rules while deleting even though the drivers validate while deletion. | 11:19 |
vponomaryov | oh no, gates are broken again >< | 11:28 |
*** JoseMello has joined #openstack-manila | 11:47 | |
*** ganso has joined #openstack-manila | 11:55 | |
*** haomaiwang has quit IRC | 12:01 | |
*** haomaiwang has joined #openstack-manila | 12:01 | |
*** a_ta has joined #openstack-manila | 12:27 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: Fix 403 error for m-api using latest Devstack https://review.openstack.org/283546 | 12:27 |
*** a_ta has quit IRC | 12:32 | |
toabctl | vponomaryov: ^^ this is a duplicate of https://review.openstack.org/#/c/283495 | 12:34 |
mkoderer | toabctl: vponomaryov version supports both SERVICE_PROJECT_NAME and SERVICE_TENANT_NAME which look also fine for me | 12:44 |
mkoderer | I'd perfer to merge the first fix that get's jenkins +1 :) | 12:45 |
toabctl | :) | 12:45 |
*** rraja_ has quit IRC | 12:46 | |
*** haomaiwang has quit IRC | 12:59 | |
*** cknight has joined #openstack-manila | 12:59 | |
*** erlon has joined #openstack-manila | 13:00 | |
*** timcl has joined #openstack-manila | 13:07 | |
*** shausy has quit IRC | 13:16 | |
*** cknight has quit IRC | 13:17 | |
ganso | vponomaryov: ping | 13:18 |
vponomaryov | ganso: pong | 13:18 |
ganso | vponomaryov: Hi Valeriy. I have a question about ZFS. Isn't it possible to use existing NFS Helper for ZFSonLinux driver? | 13:18 |
vponomaryov | ganso: it is recommended by ZFS devs to use ZFS command for exports | 13:20 |
ganso | vponomaryov: how about extending NFSHelper class and overriding "create_export" ? | 13:20 |
vponomaryov | ganso: if you look at ZFs's helper, you will see that it has different args | 13:21 |
vponomaryov | ganso: it uses ZFS-specific names of datasets | 13:22 |
vponomaryov | ganso: code for ZFS NFS helper and LVM/generic requires its own interface anyway | 13:22 |
ganso | vponomaryov: I understand it now. Thanks! =) | 13:26 |
*** haomaiwang has joined #openstack-manila | 13:28 | |
*** akerr has joined #openstack-manila | 13:37 | |
*** gouthamr has joined #openstack-manila | 13:41 | |
mkoderer | ganso: vponomaryov: we have a jenkins +1 on https://review.openstack.org/#/c/283495/ | 13:54 |
mkoderer | can we unblock the gate := | 13:54 |
bswartz | mkoderer: I think this is blocking dev as well as the gate | 13:55 |
mkoderer | bswartz: this will fix it too https://review.openstack.org/#/c/283531/ | 13:56 |
mkoderer | bswartz: but using the correct var is better | 13:56 |
bswartz | indeed | 13:57 |
bswartz | I was scratching my head last night about what broke | 13:57 |
bswartz | I knew it was keystone related | 13:57 |
bswartz | but I had to go to sleep | 13:57 |
ganso | mkoderer: gate and patches will be running again, but we need to debug why postgress-neutron and lvm jobs are not running | 13:57 |
vponomaryov | ganso: https://bugs.launchpad.net/neutron/+bug/1548807 | 13:58 |
openstack | Launchpad bug 1548807 in neutron "neutron lbaas DB support is broken on PostgreSQL" [Undecided,New] | 13:58 |
mkoderer | ganso: omg | 13:58 |
vponomaryov | double strike! | 13:58 |
mkoderer | bswartz: we need to find a way to protect the non-core gates.. I dislike the way it's currently handled | 13:59 |
bswartz | mkoderer: what do you mean protect? | 13:59 |
bswartz | and what do you mean by non-core? | 13:59 |
bswartz | you mean non-voting jobs? | 14:00 |
vponomaryov | bswartz: manila is non-core and its gates are non-core | 14:00 |
bswartz | oh | 14:00 |
gouthamr | vponomaryov :( | 14:00 |
*** cknight has joined #openstack-manila | 14:00 | |
bswartz | when you say "we" you mean the whole community | 14:00 |
bswartz | yes | 14:00 |
mkoderer | bswartz: simply a devstack patche that gets a jenkis+1 but breaks all non-core projects | 14:00 |
mkoderer | bswartz: yes WE as openstack-devs | 14:00 |
*** haomaiwang has quit IRC | 14:01 | |
*** haomaiwang has joined #openstack-manila | 14:01 | |
mkoderer | it's simply a waste of time that might 40 devs of various projects try to fix the very same issue | 14:01 |
*** pcaruana has quit IRC | 14:02 | |
mkoderer | ganso: do we depend on neutron lbaas? I would say no | 14:04 |
mkoderer | maybe we simply deactivate it | 14:04 |
ganso | mkoderer: I am not sure | 14:06 |
*** jasonsb has quit IRC | 14:06 | |
*** porrua has joined #openstack-manila | 14:07 | |
*** dcain has joined #openstack-manila | 14:08 | |
*** yangyapeng has joined #openstack-manila | 14:12 | |
mkoderer | ganso: we could use OVERRIDE_ENABLED_SERVICES in project conifg and disable q-lbaas | 14:15 |
mkoderer | ganso: but it's might worth to wait for the offical fix | 14:15 |
*** merooney has joined #openstack-manila | 14:18 | |
*** merooney has quit IRC | 14:22 | |
*** merooney has joined #openstack-manila | 14:30 | |
ganso | mkoderer: if it is not needed it shouldn't be running in the first place | 14:32 |
ganso | mkoderer: this is probably correct for LVM job | 14:33 |
ganso | mkoderer: but neutron-postgres probably needs it | 14:33 |
openstackgerrit | Merged openstack/manila: Replace TENANT => PROJECT for manila plugin https://review.openstack.org/283495 | 14:35 |
*** martyturner has joined #openstack-manila | 14:46 | |
*** haomaiwang has quit IRC | 15:01 | |
*** haomaiwang has joined #openstack-manila | 15:01 | |
*** xyang1 has joined #openstack-manila | 15:01 | |
*** merooney has quit IRC | 15:01 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/manila: Updated from global requirements https://review.openstack.org/282479 | 15:02 |
*** dustins has joined #openstack-manila | 15:02 | |
*** gouthamr_ has joined #openstack-manila | 15:04 | |
*** yangyapeng has quit IRC | 15:05 | |
*** a_ta has joined #openstack-manila | 15:06 | |
*** ekarlso- has joined #openstack-manila | 15:07 | |
*** ekarlso has quit IRC | 15:07 | |
*** gouthamr has quit IRC | 15:07 | |
ganso | cknight: ping | 15:09 |
*** rraja has quit IRC | 15:10 | |
cknight | ganso: pong | 15:11 |
ganso | cknight: Hi Clinton. I saw your comment in update_access generic driver patch | 15:12 |
ganso | cknight: I think we would need to remove the "=None" from delete_rules=None method signature | 15:13 |
ganso | cknight: if a list is always being supplied, then parameters are not optional. | 15:14 |
cknight | ganso: makes sense, although the access code is always returning lists. | 15:15 |
cknight | ganso: it's much cleaner if lists are always returned, and drivers don't have to check for null before iterating on them | 15:15 |
ganso | cknight: I agree, but we need to change this across all driver patches if we change it now | 15:16 |
cknight | ganso: since you always return lists now, you could change the signature across all drivers in one bugfix patch during M.4 | 15:16 |
ganso | cknight: +1 | 15:17 |
cknight | ganso: because by that time, you know exactly which drivers implemented update_access and need to change | 15:17 |
*** pcaruana has joined #openstack-manila | 15:18 | |
*** pcaruana has quit IRC | 15:20 | |
*** pcaruana has joined #openstack-manila | 15:21 | |
*** pcaruana|afk| has joined #openstack-manila | 15:21 | |
*** mtanino has joined #openstack-manila | 15:22 | |
*** eharney has joined #openstack-manila | 15:25 | |
mkoderer | ganso: no I would say lbaas is not needed at all.. Manila doesn't use it | 15:25 |
*** dustins has quit IRC | 15:27 | |
*** pcaruana|afk| has quit IRC | 15:29 | |
*** dencaval has joined #openstack-manila | 15:29 | |
bswartz | hey guys | 15:30 |
bswartz | are both of the gate issues fixed? | 15:31 |
ganso | bswartz: no, just one | 15:32 |
bswartz | are people other than me interested in being able to run gate jobs with neutron disabled completely? | 15:33 |
bswartz | I think it's possible, but only with changes to our jenkins jobs in project-config | 15:33 |
mkoderer | bswartz: you mean an additional check with neutron disabled? | 15:34 |
*** fthiagogv has joined #openstack-manila | 15:36 | |
ganso | bswartz: I believe neutron should be disabled for LVM job | 15:36 |
*** pcaruana has quit IRC | 15:37 | |
bswartz | ganso: me too | 15:38 |
bswartz | mkoderer: I mean for gate jobs that don't require neutron, we should no bother installing it | 15:39 |
*** dustins has joined #openstack-manila | 15:39 | |
bswartz | primarily I'm interested in conserving gate resources and getting faster feedback | 15:39 |
mkoderer | bswartz: +1 | 15:39 |
bswartz | if we observe increased reliability because neutron sometimes breaks, I consider that bonus | 15:40 |
bswartz | however some of our gate jobs simply require neutron and we'll never remove it from those | 15:40 |
mkoderer | bswartz: I think it would be a good idea to have a review of all our jobs and serivces that are enabled | 15:40 |
mkoderer | bswartz: we also don't need swift | 15:41 |
bswartz | mkoderer: I'll do some work on it | 15:41 |
mkoderer | cool | 15:41 |
mkoderer | I think we need to set OVERRIDE_ENABLED_SERVICES in our project-config | 15:42 |
bswartz | mkoderer: I know - for jobs like LVM driver I'll propose removing: nova, neutron, glance, swift, heat, cinder, horizon, and ceilometer | 15:42 |
bswartz | yes, OVERRIDE_ENABLED_SERVICES is the correct mechanism, but it requires using different job templates in the jenkins job builder | 15:42 |
mkoderer | bswartz: oh I didn't know that LVM is that independent | 15:42 |
mkoderer | that will be a very fast job then | 15:43 |
bswartz | mkoderer: all we need for LVM is mysql, rabbit, keystone, and tempest | 15:43 |
bswartz | the challenge may be if tempest itself has dependencies on those other services | 15:43 |
vponomaryov | it is true only for API tests, each "scenario" set reuquires and Nova, and Neutron | 15:44 |
mkoderer | bswartz: usually devstack should do the job and configure tempest accordingly | 15:44 |
mkoderer | vponomaryov: yeah but here we can also remove some services like horizon, swift, heat etc | 15:44 |
vponomaryov | mkoderer: tempest requires nework service for creation of new tenants, isn't it? | 15:45 |
vponomaryov | s/nework/network/ | 15:45 |
bswartz | vponomaryov: for scenario tests, I agree we should require nova+neutron | 15:45 |
bswartz | if tempest is not able to create new tenants without neutron then I propose that we fix tempest | 15:46 |
vponomaryov | bswartz: we? )0 | 15:46 |
mkoderer | bswartz: +1 .. I will fix tempest then :) | 15:46 |
bswartz | vponomaryov: well someone should | 15:46 |
vponomaryov | someone != we )) | 15:46 |
bswartz | "we" = openstack community | 15:47 |
mkoderer | :) | 15:47 |
mkoderer | bswartz: let's have a try and I will have a look about tempest resource creation | 15:47 |
bswartz | tempest is too rooted in the old days of only testing "core" projects | 15:47 |
mkoderer | it shouldn't be coupled like that | 15:48 |
bswartz | if tempest cannot become flexible enough to test standalone projects, then we need to replace tempest itself with something that can | 15:48 |
bswartz | I hate all these hidden dependencies | 15:48 |
*** dencaval has quit IRC | 15:49 | |
*** pcaruana has joined #openstack-manila | 15:49 | |
mkoderer | bswartz: wait until we indentified them :) | 15:49 |
bswartz | hey my dev environment is unbroken | 15:49 |
bswartz | thank you Sam Wan | 15:50 |
*** xyang_ has joined #openstack-manila | 15:50 | |
bswartz | does anyone know Sam Wan's IRC handle? xyang_ xyang1? | 15:50 |
xyang1 | bswartz: he won't be on irc now. he is in china | 15:51 |
bswartz | ok | 15:51 |
xyang1 | bswartz: what's up? | 15:51 |
xyang1 | bswartz: CI issue? | 15:51 |
bswartz | xyang1: he fixed our broken gate this morning | 15:51 |
*** dencaval has joined #openstack-manila | 15:51 | |
xyang1 | bswartz: nice! | 15:52 |
mkoderer | bswartz: we should send him a thank you mail | 15:52 |
xyang1 | mkoderer: bswartz he is taking care of our CI | 15:53 |
xyang1 | mkoderer: bswartz I'll let him know your thanks:) | 15:54 |
*** nkrinner has quit IRC | 15:54 | |
tpsilva | https://review.openstack.org/#/c/277366/ | 15:56 |
tpsilva | can we get this workflowed? | 15:56 |
tpsilva | bswartz: thank you! | 15:58 |
*** dencaval has quit IRC | 16:00 | |
*** haomaiwang has quit IRC | 16:01 | |
*** haomaiwang has joined #openstack-manila | 16:01 | |
*** dustins_ has joined #openstack-manila | 16:01 | |
xyang_ | gouthamr: ping | 16:03 |
*** dustins has quit IRC | 16:04 | |
xyang_ | bswartz: gouthamr added a comment in my patch here regarding provider_location: https://review.openstack.org/#/c/249542/64/manila/api/v1/share_snapshots.py@65 | 16:06 |
*** dustins_ is now known as dustins | 16:07 | |
xyang_ | bswartz: he said provider_location should be a property in the snapshot model | 16:07 |
gouthamr_ | hi xyang_ | 16:07 |
xyang_ | cknight, vponomaryov: ^ | 16:07 |
gouthamr_ | xyang_: sorry my nick'd changed | 16:07 |
bswartz | you mean the column is still in the instances table but the alchemy model synthesizes a field on the snapshot object? | 16:08 |
xyang_ | gouthamr_: hi | 16:08 |
xyang_ | bswartz: right now I only retrieve it at get_snapshot time, not adding it as a property in snapshot model | 16:08 |
bswartz | xyang_ that sounds find to me | 16:08 |
gouthamr_ | xyang_: regarding that comment, yes.. since every snapshot will have a provider_location, this should be in the model. | 16:08 |
bswartz | gouthamr_: what is the objection? | 16:09 |
vponomaryov | xyang_: yes, gouthamr_ is completely right | 16:09 |
*** dencaval has joined #openstack-manila | 16:09 | |
vponomaryov | xyang_: if you expect that attr to be,then make it be always in DB model | 16:09 |
bswartz | could we have a snapshot with no instances (even for a very slim time window)? | 16:09 |
gouthamr_ | bswartz: the API is not the right place to fake a key in.. | 16:09 |
vponomaryov | xyang_: in current case it will be relationship attr to snapshot instance model | 16:09 |
gouthamr_ | bswartz: in that case, this would be None. | 16:10 |
bswartz | okay | 16:10 |
xyang_ | vponomaryov: have you looked at his comment? he also said it should not be in views | 16:10 |
bswartz | that sounds fine to me too | 16:10 |
gouthamr_ | bswartz: i think that's okay.. | 16:10 |
vponomaryov | xyang_: such hack | 16:10 |
vponomaryov | xyang_: such hack should be absent in API and views | 16:10 |
xyang_ | vponomaryov: we have it in views because of versioning | 16:10 |
gouthamr_ | xyang_: grab the key as it is in the model.. don't add it to the model in the API. . | 16:10 |
vponomaryov | xyang_: comment is only about single thing - add attr in DB model, that's all | 16:11 |
gouthamr_ | xyang_: yep.. the magic of whether the key will appear or not, will depend on what you grab in the view. | 16:11 |
xyang_ | gouthamr_: what do you mean by "model in the API"? are you referring to this file https://review.openstack.org/#/c/249542/64/manila/api/v1/share_snapshots.py | 16:12 |
gouthamr_ | xyang_: sorry if that was a vague comment. i was doing something similar with a property in ShareInstance ('has_replicas'); until vponomaryov corrected me | 16:12 |
gouthamr_ | xyang_: yes.. create a new @property in the ShareSnapshot model.. | 16:13 |
gouthamr_ | xyang_: just like 'status' in the ShareSnapshot model here: manila/db/sqlalchemy/models.py | 16:13 |
xyang_ | vponomaryov: have you got chance to review my latest patch? I'd like to get your feedback on all the new changes | 16:13 |
vponomaryov | xyang_: I haven't, working on finishing ZFS | 16:13 |
xyang_ | bswartz: are you ok if I add it to property of snapshot model? | 16:14 |
ganso | toabctl: ping | 16:15 |
toabctl | ganso: ping | 16:20 |
ganso | toabctl: Hi THomas | 16:20 |
ganso | toabctl: looks like your suggestion did not work | 16:20 |
ganso | toabctl: http://logs.openstack.org/99/278699/22/check/gate-manila-tempest-dsvm-neutron-multibackend/7e7f955/logs/devstacklog.txt.gz | 16:21 |
ganso | toabctl: weird thing is that I had tested it before, either I had not noticed the "Created a new network" output, or neutron client has been updated | 16:21 |
*** kindjal has joined #openstack-manila | 16:25 | |
toabctl | ganso: interessting. I can reproduce this and that looks like a bug in neutronclient | 16:25 |
ganso | toabctl: I will revert to the old approach then | 16:26 |
toabctl | ganso: yeah. sorry. I'm using that style all the time for things like that. | 16:26 |
ganso | toabctl: np, I really liked when I saw it working, but now it does not work anymore =( | 16:26 |
toabctl | ganso: you could use the openstack client. it supports the same style of output | 16:28 |
ganso | toabctl: all I see is subnet list command in openstack client | 16:31 |
toabctl | ganso: there is "openstack network create" | 16:32 |
ganso | toabctl: yes, but I also need "neutron subnet-create" equivalent | 16:32 |
bswartz | xyang_: yes I'll defer to vponomaryov's and gouthamr's judgement on that one | 16:34 |
toabctl | ganso: hm. right. so go ahead with the grep and get_field function :) | 16:34 |
ganso | toabctl: ok | 16:35 |
ganso | cknight: ping | 16:35 |
cknight | ganso: pong | 16:35 |
xyang_ | bswartz: sounds good:). I just want to make sure everyone is on the same page | 16:35 |
toabctl | ganso: I filled at least a bug for neutronclient | 16:35 |
ganso | cknight: regarding your comments on https://review.openstack.org/#/c/256281, do you expect any change after what we discussed? | 16:36 |
cknight | ganso: I guess not, assuming we change the interface in a later patch. I've been telling other driver writers that None checks aren't necessary (which according to the access code, they aren't). | 16:37 |
ganso | cknight: sure, we can also change their checks within the later patch, I wouldn't mind about that right now, they are just implementing according to the method signature | 16:38 |
cknight | ganso: agreed | 16:39 |
openstackgerrit | Shaun Edwards proposed openstack/manila: EMC Isilon Driver Support For CIFS Read-Only Share https://review.openstack.org/278148 | 16:43 |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Implement admin network in generic driver https://review.openstack.org/278699 | 16:46 |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Introduced Data Service https://review.openstack.org/244286 | 16:48 |
*** dasm has joined #openstack-manila | 16:49 | |
dasm | vponomaryov: ping. | 16:49 |
vponomaryov | dasm: pong | 16:50 |
dasm | hey, i'm looking at this bug: https://bugs.launchpad.net/neutron/+bug/1548807 | 16:51 |
openstack | Launchpad bug 1548807 in neutron "neutron lbaas DB support is broken on PostgreSQL" [Undecided,New] | 16:51 |
dasm | logstash shows me just one occurence of this problem. | 16:51 |
dasm | but i've seen it in couple different places. | 16:52 |
dasm | s | 16:52 |
dasm | could you help me with triaging this? | 16:52 |
dasm | vponomaryov: ^ | 16:52 |
vponomaryov | dasm: you can look at results of "gate-manila-tempest-dsvm-neutron-postgres-lvm-multibackend" and "gate-manila-tempest-dsvm-neutron-postgres" jobs that run for each manila commit | 16:53 |
vponomaryov | dasm: they all fail with DB error on neutron-lbaas DB migration step | 16:53 |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Move Share Migration code to Data Service https://review.openstack.org/250515 | 16:54 |
openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Add new Share migration API functions https://review.openstack.org/259642 | 16:54 |
vponomaryov | dasm: these jobs use postgres as DB backend | 16:54 |
dasm | mhm. the strange thing is that all these logs show problem with db, but logstash shows just: project:"openstack/manila" AND message:"lbaas_listeners_ibfk_2" (2) count per 12h | (2 hits) | 16:55 |
dasm | but it seems to be a problem with logstash | 16:55 |
vponomaryov | dasm: yes, logstash just does not show all occurrences | 16:55 |
dasm | vponomaryov: thanks for your help. | 16:57 |
vponomaryov | dasm: do you know why ther eis no postgres CI job for Neutron? | 16:57 |
dasm | vponomaryov: nope. probably this should be raised as a concern. | 16:58 |
bswartz | +1 | 16:58 |
vponomaryov | dasm: definitely should | 16:59 |
vponomaryov | dasm: as postgres is treated as supported DB backend | 16:59 |
vponomaryov | dasm: also, it is easy to add Postres unit tests | 17:00 |
dasm | vponomaryov: we'll need to talk about this asap | 17:00 |
dasm | vponomaryov: thanks for bug report | 17:01 |
*** haomaiwang has quit IRC | 17:01 | |
vponomaryov | dasm: I can I help you somehow else? | 17:01 |
dasm | vponomaryov: no, thanks. | 17:01 |
dasm | have a nice day | 17:01 |
*** haomaiwang has joined #openstack-manila | 17:01 | |
vponomaryov | dasm: ok | 17:01 |
*** a_ta has quit IRC | 17:13 | |
*** a_ta has joined #openstack-manila | 17:14 | |
*** a_ta has quit IRC | 17:17 | |
*** a_ta has joined #openstack-manila | 17:17 | |
*** adrianofr_ has joined #openstack-manila | 17:48 | |
*** timcl1 has joined #openstack-manila | 17:55 | |
*** timcl has quit IRC | 17:56 | |
*** haomaiwang has quit IRC | 18:01 | |
*** haomaiwang has joined #openstack-manila | 18:01 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: Add ZFSonLinux share driver https://review.openstack.org/277192 | 18:02 |
*** a_ta has quit IRC | 18:16 | |
*** a_ta has joined #openstack-manila | 18:17 | |
openstackgerrit | Marc Koderer proposed openstack/manila: Fix scenario tests https://review.openstack.org/280318 | 18:21 |
*** a_ta has quit IRC | 18:21 | |
*** ociuhandu has quit IRC | 18:23 | |
*** ociuhandu has joined #openstack-manila | 18:24 | |
*** lpetrut has quit IRC | 18:29 | |
*** timcl1 has quit IRC | 18:30 | |
*** vbellur has quit IRC | 18:30 | |
*** timcl has joined #openstack-manila | 18:44 | |
*** esker has joined #openstack-manila | 18:45 | |
*** martyturner has quit IRC | 18:58 | |
*** martyturner has joined #openstack-manila | 19:00 | |
*** haomaiwang has quit IRC | 19:01 | |
*** haomaiwang has joined #openstack-manila | 19:01 | |
openstackgerrit | Tiago Pasqualini da Silva proposed openstack/manila: Add separate process for Data Service copy https://review.openstack.org/279164 | 19:03 |
*** mtanino has quit IRC | 19:17 | |
*** lpetrut has joined #openstack-manila | 19:18 | |
*** a_ta has joined #openstack-manila | 19:26 | |
*** ociuhandu has quit IRC | 19:31 | |
*** aorourke is now known as aorourke-away | 19:32 | |
*** merooney has joined #openstack-manila | 19:36 | |
*** gouthamr_ is now known as gouthamr | 20:00 | |
*** haomaiwang has quit IRC | 20:01 | |
*** haomaiwa_ has joined #openstack-manila | 20:01 | |
openstackgerrit | Diem Tran proposed openstack/manila: Add share resize support to Oracle ZFSSA driver https://review.openstack.org/273935 | 20:06 |
*** martyturner has quit IRC | 20:09 | |
openstackgerrit | Diem Tran proposed openstack/manila: Add share resize support to Oracle ZFSSA driver https://review.openstack.org/273935 | 20:09 |
dustins | Does anyone know if the Generic Share Driver supports thin provisioning? I looked at http://docs.openstack.org/developer/manila/devref/share_back_ends_feature_support_mapping.html and didn't see it or any other drivers in there | 20:10 |
*** esker has quit IRC | 20:11 | |
*** vbellur has joined #openstack-manila | 20:11 | |
*** martyturner has joined #openstack-manila | 20:11 | |
*** dasm has left #openstack-manila | 20:12 | |
bswartz | dustins: that's up to cinder | 20:13 |
bswartz | dustins: generic driver relies on cinder for actual storage, and whether or not that storage is thin provisioned is up to cinder | 20:14 |
bswartz | we have example of generic running on top of cinder with LVM+thin | 20:14 |
dustins | bswartz: so if I have thin provisioning set up on my Cinder backend, would that cascade through the backend's capabilities into Manila? | 20:14 |
bswartz | manila certainly doesn't do anything that turns a thin cinder volume into a thick cinder volume -- we format with ext4 filesystem which doesn't write many blocks | 20:15 |
dustins | Right, I guess what I'm asking is if the backing volume is thin provisioned, will the manila pool report itself as supporting thin provisioning? | 20:15 |
bswartz | dustins: I don't think it shows up as a capability at the scheduler level, no -- the fact is that the generic driver doesn't know whether it's thick or thin so it can't tell the manila scheduler | 20:16 |
*** merooney has quit IRC | 20:16 | |
dustins | So from what I'm reading, it looks like the generic share driver cannot report that it is capable of thin provisioning | 20:17 |
bswartz | currently no | 20:18 |
bswartz | do you want to try to fix that? | 20:18 |
bswartz | what's the use case? | 20:18 |
dustins | I want to test Thin Provisioning :P | 20:19 |
dustins | I suppose I could, but with the other drivers coming in, it might be more prudent to enable that in those, if it doesn't already exist | 20:19 |
openstackgerrit | Andrea Ma proposed openstack/manila: glusterManager instantiation regexp validation https://review.openstack.org/283777 | 20:20 |
*** merooney has joined #openstack-manila | 20:21 | |
dustins | We should, at least, update that driver matrix page to show which drivers support Thin Provisioning | 20:21 |
bswartz | dustins: interesting point | 20:24 |
dustins | And is thin provisioning being tested in the gate somewhere? I haven't had a close look to be honest | 20:24 |
bswartz | if you just want to test the filter it should be possible to hack the driver to return thin_provisioning=True/False | 20:24 |
bswartz | dustins: we don't test specific extra_specs | 20:25 |
bswartz | because all extra specs are treated the same by the cheduler | 20:25 |
bswartz | scheduler | 20:25 |
bswartz | honestly IDK if we have any tests that test the scheduler's ability to filter by extra_specs or not -- if not, it's something we can and should add | 20:26 |
dustins | it looks like we have the CapabilitiesFilter available, no idea what the testing coverage of it is | 20:27 |
gouthamr | hi bswartz: these two simple backports to stable/liberty need a +W: https://review.openstack.org/#/c/280828/ and https://review.openstack.org/#/c/280829/ | 20:27 |
dustins | I'll be sure to look into that | 20:27 |
gouthamr | dustins: unit test coverage exists.. no specific tempest tests. | 20:27 |
dustins | gouthamr: That's what I was afraid of | 20:27 |
dustins | Though it might be tough to test those features in a "generic" way | 20:28 |
dustins | Doesn't mean it shouldn't be done, but something to look at | 20:28 |
gouthamr | dustins: it would be.. because these tests are expected to be run by vendor CIs as well, and capabilities reported could be different | 20:28 |
dustins | gouthamr: Indeed, though I imagine some sort of skip logic could be used to skip the tests that the backend under test does not support | 20:29 |
gouthamr | dustins: that could work, but we have those skips for very specific things now.. what capabilities would you support testing for? | 20:30 |
gouthamr | thanks bswartz | 20:30 |
dustins | gouthamr: Ideally all of them | 20:30 |
dustins | Looking at the support mapping matrix, it seems like each one of those columns could be a suite of tests that get run if the driver supports it and doesn't if not | 20:31 |
gouthamr | dustins: a tempest configuration nightmare :P | 20:31 |
bswartz | dustins: I'm in favor of more tests always -- I'll only push back if the tests are unreasonably slow or add unreasonable requirements | 20:31 |
dustins | Would be a pain to do in CI, though, due to all the reconfiguration | 20:31 |
bswartz | what's needed is a general tests for extra_spec filtering -- we don't need to worry about specific extra specs | 20:32 |
gouthamr | bswartz: we already do that with snapshot_support i think.. | 20:32 |
dustins | We should be sure that each driver vendor does their due diligence in making sure that their driver functions properly for the features that it supports | 20:33 |
bswartz | dustins: that's a higher bar than what we aim for with tempest | 20:33 |
dustins | bswartz: Indeed, there's no real way to enforce it | 20:34 |
bswartz | tempest tests that manila works -- you're describing something more like a vendor certification test | 20:34 |
*** vbellur has quit IRC | 20:34 | |
*** dcain has quit IRC | 20:34 | |
dustins | Very true | 20:35 |
dustins | At the least, I'll add a note to the next manila meeting to ask vendors to indicate their thin provisioning support in the support mapping | 20:35 |
*** dcain has joined #openstack-manila | 20:37 | |
*** aorourke-away is now known as aorourke | 20:37 | |
*** a_ta has quit IRC | 20:38 | |
dustins | Thanks for the help, bswartz and gouthamr! | 20:39 |
gouthamr | anytime dustins | 20:39 |
*** a_ta has joined #openstack-manila | 20:39 | |
*** a_ta has quit IRC | 20:42 | |
*** a_ta has joined #openstack-manila | 20:43 | |
*** merooney has quit IRC | 20:48 | |
*** esker has joined #openstack-manila | 20:56 | |
*** xyang_ has quit IRC | 21:00 | |
*** haomaiwa_ has quit IRC | 21:01 | |
*** haomaiwang has joined #openstack-manila | 21:01 | |
bswartz | dustins: still here? | 21:04 |
dustins | yup! | 21:04 |
bswartz | dustins: did you notice that this patch slipped back into merge conflict? https://review.openstack.org/#/c/188137/ | 21:04 |
bswartz | I'm going to try to fix it unless you'd rather do that | 21:04 |
dustins | *sigh* yes I did | 21:04 |
*** xyang_ has joined #openstack-manila | 21:04 | |
dustins | Have at it, I've got a fair bit on my plate right now | 21:05 |
dustins | If you need a hand, let me know, though! | 21:05 |
bswartz | okie dokie | 21:05 |
dustins | It's the Patch that Shall not Merge :P | 21:05 |
bswartz | http://tinyurl.com/pu6t7yo | 21:07 |
bswartz | dustins: okay jenkins is running again -- I'll try to watch for fails | 21:09 |
*** kindjal has quit IRC | 21:13 | |
*** dcain has quit IRC | 21:13 | |
*** esker has quit IRC | 21:13 | |
*** dcain has joined #openstack-manila | 21:13 | |
*** akerr has quit IRC | 21:14 | |
*** esker has joined #openstack-manila | 21:14 | |
*** lpabon has joined #openstack-manila | 21:16 | |
*** porrua has quit IRC | 21:16 | |
openstackgerrit | Andrea Ma proposed openstack/manila: glusterManager instantiation regexp validation https://review.openstack.org/283777 | 21:17 |
dustins | bswartz: I'll keep an eye out as well, thanks! | 21:18 |
*** tpsilva has quit IRC | 21:37 | |
*** dcain has left #openstack-manila | 21:38 | |
*** cknight has quit IRC | 21:47 | |
*** timcl has quit IRC | 21:59 | |
*** gouthamr has quit IRC | 21:59 | |
*** haomaiwang has quit IRC | 22:01 | |
*** haomaiwang has joined #openstack-manila | 22:01 | |
*** vbellur has joined #openstack-manila | 22:03 | |
*** lpabon has quit IRC | 22:16 | |
*** dustins has quit IRC | 22:17 | |
*** cknight has joined #openstack-manila | 22:17 | |
*** ociuhandu has joined #openstack-manila | 22:27 | |
*** kindjal has joined #openstack-manila | 22:29 | |
*** cknight1 has joined #openstack-manila | 22:36 | |
*** cknight has quit IRC | 22:37 | |
*** martyturner has quit IRC | 22:48 | |
*** ganso has quit IRC | 22:49 | |
*** tpatzig has quit IRC | 22:58 | |
*** mkoderer has quit IRC | 22:58 | |
*** csaba has quit IRC | 22:58 | |
*** _laco has quit IRC | 22:58 | |
*** jcsp has quit IRC | 22:58 | |
*** pcaruana has quit IRC | 22:58 | |
*** markstur has quit IRC | 22:58 | |
*** dhellmann has quit IRC | 22:58 | |
*** scottda has quit IRC | 22:58 | |
*** alkhodos has quit IRC | 22:58 | |
*** erlon has quit IRC | 22:58 | |
*** marcusvrn_ has quit IRC | 22:58 | |
*** sghatty_ has quit IRC | 22:58 | |
*** darrenc has quit IRC | 22:58 | |
*** logan- has quit IRC | 22:58 | |
*** sghatty has quit IRC | 22:58 | |
*** adrianofr_ has quit IRC | 22:58 | |
*** eharney has quit IRC | 22:58 | |
*** JoseMello has quit IRC | 22:58 | |
*** chlong_ has quit IRC | 22:58 | |
*** ameade has quit IRC | 22:58 | |
*** zigo has quit IRC | 22:58 | |
*** vponomaryov has quit IRC | 22:58 | |
*** cfey has quit IRC | 22:58 | |
*** cknight1 has quit IRC | 22:58 | |
*** zhongjun has quit IRC | 22:58 | |
*** zhonghua-lee has quit IRC | 22:58 | |
*** rhefner has quit IRC | 22:58 | |
*** liewegas has quit IRC | 22:58 | |
*** jwang__ has quit IRC | 22:58 | |
*** tbarron has quit IRC | 22:58 | |
*** vkmc has quit IRC | 22:58 | |
*** kaisers has quit IRC | 22:58 | |
*** winston-d has quit IRC | 22:58 | |
*** vbellur has quit IRC | 22:58 | |
*** esker has quit IRC | 22:58 | |
*** lpetrut has quit IRC | 22:58 | |
*** ekarlso- has quit IRC | 22:58 | |
*** krotscheck has quit IRC | 22:58 | |
*** mkoderer__ has quit IRC | 22:58 | |
*** bswartz has quit IRC | 22:58 | |
*** johnthetubaguy has quit IRC | 22:58 | |
*** ChanServ has quit IRC | 22:58 | |
*** cfouts is now known as nug | 22:58 | |
*** johnthetubaguy has joined #openstack-manila | 23:00 | |
*** scottda has joined #openstack-manila | 23:00 | |
*** dhellmann has joined #openstack-manila | 23:00 | |
*** markstur has joined #openstack-manila | 23:00 | |
*** pcaruana has joined #openstack-manila | 23:00 | |
*** sghatty has joined #openstack-manila | 23:00 | |
*** logan- has joined #openstack-manila | 23:00 | |
*** darrenc has joined #openstack-manila | 23:00 | |
*** sghatty_ has joined #openstack-manila | 23:00 | |
*** marcusvrn_ has joined #openstack-manila | 23:00 | |
*** erlon has joined #openstack-manila | 23:00 | |
*** vkmc has joined #openstack-manila | 23:00 | |
*** tbarron has joined #openstack-manila | 23:00 | |
*** jwang__ has joined #openstack-manila | 23:00 | |
*** liewegas has joined #openstack-manila | 23:00 | |
*** rhefner has joined #openstack-manila | 23:00 | |
*** haomaiwang has quit IRC | 23:01 | |
*** adrianofr_ has joined #openstack-manila | 23:01 | |
*** eharney has joined #openstack-manila | 23:01 | |
*** JoseMello has joined #openstack-manila | 23:01 | |
*** chlong_ has joined #openstack-manila | 23:01 | |
*** ameade has joined #openstack-manila | 23:01 | |
*** zigo has joined #openstack-manila | 23:01 | |
*** vponomaryov has joined #openstack-manila | 23:01 | |
*** cfey has joined #openstack-manila | 23:01 | |
*** haomaiwang has joined #openstack-manila | 23:01 | |
*** tpatzig has joined #openstack-manila | 23:01 | |
*** mkoderer has joined #openstack-manila | 23:01 | |
*** _laco has joined #openstack-manila | 23:01 | |
*** csaba has joined #openstack-manila | 23:01 | |
*** jcsp has joined #openstack-manila | 23:01 | |
*** vbellur has joined #openstack-manila | 23:03 | |
*** esker has joined #openstack-manila | 23:03 | |
*** lpetrut has joined #openstack-manila | 23:03 | |
*** ekarlso- has joined #openstack-manila | 23:03 | |
*** krotscheck has joined #openstack-manila | 23:03 | |
*** mkoderer__ has joined #openstack-manila | 23:03 | |
*** bswartz has joined #openstack-manila | 23:03 | |
*** kaisers has joined #openstack-manila | 23:03 | |
*** winston-d has joined #openstack-manila | 23:03 | |
*** alkhodos has joined #openstack-manila | 23:03 | |
*** adrianofr_ has quit IRC | 23:03 | |
*** adrianofr_ has joined #openstack-manila | 23:05 | |
*** cknight1 has joined #openstack-manila | 23:06 | |
*** zhongjun has joined #openstack-manila | 23:06 | |
*** zhonghua-lee has joined #openstack-manila | 23:06 | |
*** ChanServ has joined #openstack-manila | 23:06 | |
*** asimov.freenode.net sets mode: +o ChanServ | 23:06 | |
*** nug is now known as Guest60450 | 23:07 | |
*** sgotliv has quit IRC | 23:07 | |
*** a_ta has quit IRC | 23:09 | |
*** a_ta has joined #openstack-manila | 23:10 | |
*** eharney has quit IRC | 23:10 | |
*** darrenc is now known as darrenc_afk | 23:13 | |
*** a_ta has quit IRC | 23:14 | |
*** adrianofr_ has quit IRC | 23:18 | |
*** adrianofr_ has joined #openstack-manila | 23:18 | |
*** vbellur has quit IRC | 23:20 | |
*** kindjal has quit IRC | 23:24 | |
*** gouthamr has joined #openstack-manila | 23:26 | |
*** akerr has joined #openstack-manila | 23:28 | |
*** vbellur has joined #openstack-manila | 23:30 | |
*** JoseMello has quit IRC | 23:32 | |
*** xyang1 has quit IRC | 23:37 | |
*** akerr has quit IRC | 23:38 | |
*** akerr has joined #openstack-manila | 23:42 | |
*** akerr is now known as akerr_away | 23:47 | |
*** gouthamr has quit IRC | 23:49 | |
*** gouthamr has joined #openstack-manila | 23:51 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!