| *** haomaiwang has quit IRC | 00:01 | |
| *** haomaiwang has joined #openstack-manila | 00:01 | |
| *** darrenc_afk is now known as darrenc | 00:05 | |
| *** fthiagogv_ has joined #openstack-manila | 00:26 | |
| *** xyang_ has quit IRC | 00:28 | |
| *** fthiagogv has quit IRC | 00:29 | |
| *** vbellur has quit IRC | 00:31 | |
| *** akerr_away has quit IRC | 00:56 | |
| *** haomaiwang has quit IRC | 01:01 | |
| *** haomaiwang has joined #openstack-manila | 01:01 | |
| *** vbellur has joined #openstack-manila | 01:07 | |
| *** yangyapeng has joined #openstack-manila | 01:09 | |
| *** yangyapeng has quit IRC | 01:10 | |
| *** akerr has joined #openstack-manila | 01:14 | |
| openstackgerrit | daiki kato proposed openstack/manila: Fix typo in comment message https://review.openstack.org/283896 | 01:25 |
|---|---|---|
| *** akerr has quit IRC | 01:30 | |
| *** lpetrut has quit IRC | 01:36 | |
| *** haomaiwang has quit IRC | 01:36 | |
| *** erlon has quit IRC | 01:46 | |
| *** lpabon has joined #openstack-manila | 02:03 | |
| *** akerr has joined #openstack-manila | 02:04 | |
| *** pcaruana has quit IRC | 02:07 | |
| *** akerr has quit IRC | 02:10 | |
| *** akerr has joined #openstack-manila | 02:12 | |
| *** haomaiwang has joined #openstack-manila | 02:16 | |
| *** pcaruana has joined #openstack-manila | 02:19 | |
| *** akerr has quit IRC | 02:21 | |
| *** jasonsb has joined #openstack-manila | 02:25 | |
| *** lpabon has quit IRC | 02:28 | |
| *** houming has joined #openstack-manila | 02:38 | |
| openstackgerrit | daiki kato proposed openstack/python-manilaclient: Fix function name https://review.openstack.org/283914 | 03:00 |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-manila | 03:01 | |
| *** pcaruana has quit IRC | 03:01 | |
| openstackgerrit | Chaozhe Chen(ccz) proposed openstack/manila: Add /usr/local/{sbin,bin} to rootwrap exec_dirs https://review.openstack.org/283916 | 03:09 |
| *** a_ta has joined #openstack-manila | 03:11 | |
| *** pcaruana has joined #openstack-manila | 03:16 | |
| *** a_ta has quit IRC | 03:16 | |
| *** furlongm has quit IRC | 03:20 | |
| *** haomaiwang has quit IRC | 03:20 | |
| *** yangyapeng has joined #openstack-manila | 03:25 | |
| *** xyang_ has joined #openstack-manila | 03:25 | |
| openstackgerrit | Ben Swartzlander proposed openstack/manila: Add doc for export location metadata https://review.openstack.org/282071 | 03:28 |
| *** martyturner has joined #openstack-manila | 03:29 | |
| openstackgerrit | Ben Swartzlander proposed openstack/manila: Remove aggressive assert from share server test https://review.openstack.org/281534 | 03:32 |
| *** martyturner has quit IRC | 03:43 | |
| *** gouthamr has quit IRC | 03:48 | |
| *** shausy has joined #openstack-manila | 03:57 | |
| *** pcaruana has quit IRC | 04:01 | |
| *** furlongm has joined #openstack-manila | 04:02 | |
| *** a_ta has joined #openstack-manila | 04:12 | |
| *** pcaruana has joined #openstack-manila | 04:15 | |
| *** a_ta has quit IRC | 04:17 | |
| *** xyang_ has quit IRC | 04:48 | |
| openstackgerrit | Clinton Knight proposed openstack/manila: Add preferred flag to export location metadata https://review.openstack.org/282070 | 04:52 |
| openstackgerrit | Ben Swartzlander proposed openstack/manila: Test fix for neutron bug in Manila gate https://review.openstack.org/283939 | 05:01 |
| *** haomaiwang has joined #openstack-manila | 05:04 | |
| *** a_ta has joined #openstack-manila | 05:10 | |
| *** a_ta has quit IRC | 05:14 | |
| openstackgerrit | Clinton Knight proposed openstack/manila: Add preferred flag to export location metadata https://review.openstack.org/282070 | 05:19 |
| *** sgotliv has joined #openstack-manila | 05:20 | |
| *** cknight1 has quit IRC | 05:31 | |
| *** nidhimittalhada has joined #openstack-manila | 05:31 | |
| openstackgerrit | Merged openstack/manila: Implement update_access() in NetApp cDOT drivers https://review.openstack.org/280841 | 05:34 |
| *** pcaruana has quit IRC | 05:38 | |
| *** qeelee has joined #openstack-manila | 05:42 | |
| openstackgerrit | NidhiMittalHada proposed openstack/python-manilaclient: To tag all the Admin-only APIs as Admin only https://review.openstack.org/264110 | 05:51 |
| *** nidhimittalhada has quit IRC | 05:52 | |
| *** pcaruana has joined #openstack-manila | 05:53 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-manila | 06:01 | |
| *** zhongjun has quit IRC | 06:13 | |
| *** zhongjun has joined #openstack-manila | 06:14 | |
| *** nkrinner has joined #openstack-manila | 06:16 | |
| *** openstack has joined #openstack-manila | 13:22 | |
| gouthamr | vponomaryov: ping | 13:27 |
| openstackgerrit | Béla Vancsics proposed openstack/manila: Use assertIsNone instead of assertEqual(None, ***) https://review.openstack.org/284124 | 13:27 |
| *** qeelee has quit IRC | 13:29 | |
| vponomaryov | gouthamr: pong | 13:30 |
| gouthamr | hi vponomaryov.. a question wrt the scheduler changes for replication | 13:30 |
| gouthamr | vponomaryov: i have two options defined in the base driver as we discussed "replication_enabled" and "replication_domain".. these are getting reported with the share_stats | 13:31 |
| vponomaryov | gouthamr: presense of the latter makes the former redundant | 13:32 |
| vponomaryov | gouthamr: if it has something specified in "replication_domain" then we consider it has replication enabled | 13:33 |
| *** yangyapeng has quit IRC | 13:33 | |
| gouthamr | vponomaryov: true, in one sense.. but bswartz favored having two switches.. | 13:33 |
| gouthamr | vponomaryov: so you prefer removing the 'replication_domain' configuration to turn off replication? | 13:33 |
| *** timcl has joined #openstack-manila | 13:34 | |
| vponomaryov | gouthamr: if ti does not belong to any of domains, then... I consider it as disabled | 13:34 |
| vponomaryov | gouthamr: don't you? | 13:34 |
| gouthamr | vponomaryov: hmm, i agree that's an unnecessary switch. | 13:34 |
| vponomaryov | gouthamr: BUT, there is only one small use case | 13:35 |
| vponomaryov | gouthamr: to having both | 13:35 |
| vponomaryov | gouthamr: - when we disable it temporary and do not want to bother ourselves with keeoing consistency of domain names for each backend | 13:35 |
| gouthamr | vponomaryov: the creation of the first replica? | 13:35 |
| vponomaryov | gouthamr: ease of on/off | 13:36 |
| gouthamr | vponomaryov: you can comment out the config option instead of removing it | 13:36 |
| gouthamr | :) | 13:36 |
| ganso | gouthamr: ping | 13:36 |
| gouthamr | hi ganso | 13:36 |
| ganso | gouthamr: Hi Goutham. Regarding your comments in https://review.openstack.org/#/c/259642 | 13:36 |
| vponomaryov | gouthamr: then, you know what to do )) | 13:37 |
| ganso | gouthamr: you mentioned DocImpact, currently there is no share migration doc | 13:37 |
| gouthamr | ganso: yessir.. we should have one, shouldn't we? | 13:37 |
| gouthamr | thanks vponomaryov.. | 13:37 |
| ganso | gouthamr: we should, but currently we don't, so should DocImpact still be added? | 13:37 |
| gouthamr | ganso: DocImpact opens a bug in the project and kinda allows for tracking... so that, when you do write up the devref or something, you can close that bug | 13:38 |
| gouthamr | ganso: I've one for replication.. that i'm working on currently. | 13:38 |
| ganso | gouthamr: ok, then I will close it by adding the doc | 13:39 |
| gouthamr | ganso: +1 | 13:39 |
| ganso | gouthamr: regarding the action names | 13:40 |
| gouthamr | vponomaryov: another question. Do you think the scheduler should "pass" the host of an active replica when a replica is being scheduled.. | 13:40 |
| gouthamr | ganso: a consistency nit.. :) | 13:40 |
| vponomaryov | gouthamr: should be as it is now - based on AZ | 13:40 |
| ganso | gouthamr: I don't like migrate-start | 13:40 |
| vponomaryov | gouthamr: if absent - any host | 13:41 |
| ganso | gouthamr: do you think only 'migrate' would look weird, while having 'migrate-cancel'? | 13:41 |
| gouthamr | ganso: yes.. i didn't understand the use cases.. since the first action is 'migrate' and the rest have 'complete-migration', 'cancel-migration' etc. | 13:42 |
| gouthamr | vponomaryov: no, i meant, can a backend replicate to another share on itself? (stupid use case, but for some backends, the backend name can merely be a namespace over multiple physical storage boxes... like NetApp cDot) | 13:43 |
| gouthamr | vponomaryov: should i "fail" the host of the active replica, so that it never gets picked for replica creation? | 13:44 |
| *** akerr has joined #openstack-manila | 13:48 | |
| ganso | gouthamr: do you think internal methods should also be renamed? in share/api.py, share/manager.py, share/rpcapi.py... | 13:51 |
| gouthamr | ganso: for consistency, yes.. so that it's easy for someone to read/follow the code path.. | 13:51 |
| openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: LXC/LXD driver https://review.openstack.org/245751 | 13:53 |
| *** eharney has joined #openstack-manila | 13:57 | |
| *** haomaiwa_ has quit IRC | 14:01 | |
| *** haomaiwang has joined #openstack-manila | 14:01 | |
| *** xyang1 has joined #openstack-manila | 14:02 | |
| openstackgerrit | John Spray proposed openstack/manila: Add Ceph Native driver https://review.openstack.org/270211 | 14:05 |
| *** merooney has joined #openstack-manila | 14:06 | |
| vponomaryov | gouthamr: (15:40:54) vponomaryov: gouthamr: should be as it is now - based on AZ, if absent - any host | 14:07 |
| gouthamr | vponomaryov: this is like the AZ filter, but what if replicas are always being scheduled on the same host as the primary? i feel that's a bug.. | 14:09 |
| vponomaryov | gouthamr: ZFS driver does not care where is manila-share service running. So, does not care about its host. It means, we can run all manila-share services on one host and use ZFS's on lots of remote ones | 14:09 |
| vponomaryov | gouthamr: so, such thing should be handled by implemented grouping | 14:10 |
| openstackgerrit | John Spray proposed openstack/manila: Add Ceph Native driver https://review.openstack.org/270211 | 14:11 |
| vponomaryov | gouthamr: host of "manila-share" service should not be the criterion | 14:11 |
| vponomaryov | gouthamr: if you want use different backend, set different AZ to them in config and specify it creating replicas | 14:12 |
| gouthamr | vponomaryov: i meant the 'host' string on the share instance... that is the fully qualified hoststring? "HostXYZ@BackendABC#Pool0" | 14:12 |
| vponomaryov | gouthamr: if pool is considered always, then it should work | 14:14 |
| gouthamr | vponomaryov: assume BackendABC and BackendXYZ can replicate between each other.. the share is on "HostXYZ@BackendABC#Pool0".. the scheduler will pick "HostXYZ@BackendABC#Pool0" to schedule the replica as well as "HostXYZ@BackendXYZ#Pool1" | 14:14 |
| vponomaryov | gouthamr: again, if pool is always considered, then it is ok to restrict usage of same pair "host + pool" | 14:16 |
| vponomaryov | gouthamr: as it makes no sense for replication - both will be lost if disaster happens | 14:17 |
| gouthamr | vponomaryov: true.. | 14:19 |
| gouthamr | vponomaryov: thanks, i'll think about this. . | 14:19 |
| vponomaryov | gouthamr: when do you plan to upload updated PS? | 14:20 |
| vponomaryov | gouthamr: with onfig option changes | 14:20 |
| vponomaryov | /onfig/config/ | 14:20 |
| *** merooeny_ has joined #openstack-manila | 14:20 | |
| gouthamr | vponomaryov: i was testing that.. and was running into this issue we're discussing where the scheduler was always picking the active replica's host. | 14:20 |
| gouthamr | vponomaryov: definitely in the next few hours if i can finalize.. | 14:21 |
| vponomaryov | gouthamr: ok, glad that "internal" review won't take too long | 14:21 |
| gouthamr | vponomaryov: nope. top priority to get this back up for ZFS testing. | 14:22 |
| *** merooney has quit IRC | 14:23 | |
| *** ociuhandu has quit IRC | 14:24 | |
| *** ociuhandu has joined #openstack-manila | 14:25 | |
| bswartz | why is jenkins suddenly so fast? | 14:26 |
| gouthamr | bswartz: touch wood | 14:27 |
| bswartz | it's insanely faster though | 14:28 |
| bswartz | something dramatic must have changed | 14:28 |
| bswartz | did I miss something yesterday or this morning? | 14:28 |
| bswartz | http://paste.openstack.org/show/488027/ | 14:29 |
| *** cknight has joined #openstack-manila | 14:29 | |
| *** dustins has joined #openstack-manila | 14:34 | |
| openstackgerrit | Merged openstack/manila: Implement update_access() method in Hitachi HNAS driver https://review.openstack.org/277366 | 14:35 |
| *** esker has quit IRC | 14:36 | |
| *** alkhodos has quit IRC | 14:40 | |
| *** rhagarty_ has quit IRC | 14:55 | |
| *** alkhodos_ has joined #openstack-manila | 14:55 | |
| alkhodos_ | Hello everyone, I'm trying to cinfigure CI for manila and cannot find a way to add these values to tempest.conf: tenant_name, password, username. Is it possible to add them via local.conf somehow? | 14:56 |
| *** mtanino has joined #openstack-manila | 14:59 | |
| *** a_ta has joined #openstack-manila | 14:59 | |
| bswartz | alkhodos_: https://github.com/openstack/manila/blob/master/contrib/ci/post_test_hook.sh lines 187-192 <-- is that what you're looking for? | 15:00 |
| *** haomaiwang has quit IRC | 15:01 | |
| *** DuncanT has joined #openstack-manila | 15:01 | |
| *** haomaiwang has joined #openstack-manila | 15:01 | |
| DuncanT | I think I've just found a minor bug (or more likely, source of future bugs) in your microversion code - manila/utils.py ComparableMixin._compare: Shouldn't it raise NotImplemented, rather than returning it? NotImplemented evaluates to True in boolean context, which probably isn't what the author meant | 15:04 |
| DuncanT | Can anybody confirm please? | 15:04 |
| alkhodos_ | bswartz: yep, looks like what I'm looking for exactly, thanks! | 15:07 |
| bswartz | cknight: ^ | 15:07 |
| cknight | DuncanT, bswartz: Thanks for the heads-up, but I think the code is correct. http://shahriar.svbtle.com/python-notimplemented-type | 15:13 |
| cknight | DuncanT, bswartz: More background: http://jcalderone.livejournal.com/32837.html | 15:14 |
| DuncanT | cknight: Interesting, thanks. I'll keep reading | 15:14 |
| DuncanT | cknight: That's quite horrible, but ok, I understand. Thanks for the links | 15:16 |
| *** rhagarty has joined #openstack-manila | 15:19 | |
| bswartz | wow i learned something too | 15:21 |
| bswartz | I'm not sure if that's genius or gross | 15:22 |
| *** aovchinnikov has quit IRC | 15:30 | |
| *** Guest60450 is now known as cfouts | 15:34 | |
| *** rhagarty_ has joined #openstack-manila | 15:36 | |
| DuncanT | bswartz: I feel the same way. It isn't a bug though. It's clever. That isn't always a good thing... | 15:36 |
| *** rhagarty has quit IRC | 15:37 | |
| *** nkrinner has quit IRC | 15:44 | |
| *** merooney has joined #openstack-manila | 15:50 | |
| *** merooeny_ has quit IRC | 15:52 | |
| rraja | jcsp: ping | 15:57 |
| *** haomaiwang has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-manila | 16:01 | |
| openstackgerrit | Merged openstack/manila: Add update_access() interface to Quobyte driver https://review.openstack.org/281136 | 16:01 |
| *** erlon has joined #openstack-manila | 16:02 | |
| *** merooeny_ has joined #openstack-manila | 16:06 | |
| jcsp | rraja: pong | 16:10 |
| *** merooney has quit IRC | 16:10 | |
| jcsp | actually I have a question for you -- which ceph branch is the cephfs CI currently using? | 16:10 |
| jcsp | because the ceph_volume_client interface is broken on the ceph master branch right now :-( | 16:10 |
| *** dencaval has quit IRC | 16:12 | |
| rraja | jcsp: wip-manila branch. https://github.com/openstack/devstack-plugin-ceph/blob/master/devstack/lib/ceph#L601 | 16:13 |
| openstackgerrit | Béla Vancsics proposed openstack/manila: Use assertIsNone instead of assertEqual(None, ***) https://review.openstack.org/284124 | 16:13 |
| jcsp | ah ok, that's good, so our master breakage won't affect it | 16:14 |
| jcsp | we should cut that over to consume the ceph jewel branch at the end of the month (or whenever it's ready) | 16:14 |
| rraja | jcsp: i'm prescient. sure! i was wondering about the path of the cephfs driver? did you consider that in the feature support matrix doc that the name of the driver is CephFS Native? | 16:18 |
| jcsp | in the future if there's an NFS-on-Ceph variant, I'd be inclined to call that one ceph.cephfs_nfs or something like that, and leave this one as ceph.cephfs | 16:20 |
| rraja | jcsp: should the path rather be, manila/share/drivers/cephfs/cephfs_native.py ? is there a reason the parent directory of the driver is currently named ceph and not cephfs? | 16:20 |
| jcsp | it's all kind of debatable | 16:21 |
| jcsp | it could be ceph/ceph, or ceph/cephfs, or cephfs/cephfs, or cephfs/cephfs_native | 16:21 |
| rraja | jcsp: :) | 16:21 |
| jcsp | but I take the point that it's a bit inconsistent that I call it "CephFS Native" in some of the text but the module is just ceph.cpehfs | 16:22 |
| jcsp | if you feel strongly I don't mind if you want to change it to ceph.ceph_native or cephfs.ceph_native | 16:22 |
| jcsp | sorry, meant ceph.cephfs_native or cephfs.cephfs_native | 16:23 |
| rraja | jcsp: i prefer cephfs.cephfs_native, as it clear the doubts if FS on top of RBD (?) or we just use CephFS. i think similar doubts cropped up in the initial email you sent to openstack-dev manila ML when you introduced ceph driver. so yeah, it's your call. | 16:27 |
| *** dencaval has joined #openstack-manila | 16:28 | |
| jcsp | sounds reasonable; do you want to push another changeset with that name? | 16:29 |
| jcsp | (+for devstack-plugin-ceph) | 16:30 |
| rraja | jcsp: sure! let me do that. thanks! | 16:30 |
| jcsp | cool | 16:31 |
| dustins | bswartz: ping re: Triple-O-Heat template manila thing | 16:32 |
| bswartz | dustins: pong | 16:32 |
| * bswartz prays for good news... | 16:32 | |
| dustins | I was just gonna say that I've got some time today if you haven't had a chance to look at it | 16:33 |
| dustins | To see if I can un-stick it | 16:33 |
| bswartz | WTF?!? | 16:33 |
| bswartz | I fixed it yesterday | 16:33 |
| bswartz | and it's in conflict again | 16:33 |
| * bswartz headdesk | 16:34 | |
| dustins | I had to click refresh in my browser to make sure I wasn't looking at an old version of the page | 16:34 |
| dustins | I swear, we can't win for losing with this stupid patch :) | 16:34 |
| bswartz | let me take another look | 16:34 |
| bswartz | the last conflict was trivial to fix | 16:35 |
| dustins | Yeah, looks like the merge failed early this morning | 16:35 |
| bswartz | dustins: fixed again | 16:36 |
| bswartz | let's watch jenkins again today | 16:37 |
| dustins | bswartz: Quickly becoming one of my favorite activities :/ | 16:37 |
| ganso | gouthamr: ping | 16:41 |
| gouthamr | ganso: pong | 16:42 |
| ganso | gouthamr: regarding funnelling get_progress result through view_builder. I see that view_builder converts models to dicts | 16:42 |
| ganso | gouthamr: migration_get_progress result is already a dict | 16:42 |
| gouthamr | ganso: wouldn't you want a similar approach? | 16:43 |
| gouthamr | ganso: oh.. (didn't look at your other patch yet) | 16:43 |
| *** merooney has joined #openstack-manila | 16:44 | |
| *** merooeny_ has quit IRC | 16:47 | |
| gouthamr | ganso: took a quick peek now.. for all the APIs, even if we're passing around dicts, we've been funneling them through the viewbuilder.. | 16:50 |
| gouthamr | ganso: unless it has no response body.. | 16:51 |
| ganso | gouthamr: ok, I will create a method, for consistency. | 16:52 |
| gouthamr | thank you ganso | 16:52 |
| *** haomaiwang has quit IRC | 17:01 | |
| *** haomaiwang has joined #openstack-manila | 17:01 | |
| openstackgerrit | Rodrigo Barbieri proposed openstack/manila: Implement update_access() in generic driver + LVM https://review.openstack.org/256281 | 17:07 |
| ganso | gouthamr: ping | 17:19 |
| *** a_ta has quit IRC | 17:22 | |
| *** a_ta has joined #openstack-manila | 17:23 | |
| *** a_ta has quit IRC | 17:27 | |
| *** a_ta has joined #openstack-manila | 17:29 | |
| *** jasonsb has quit IRC | 17:39 | |
| gouthamr | ganso: pong | 17:41 |
| ganso | gouthamr: regarding policy check, is this what you were referring to? https://review.openstack.org/#/c/250515/34/manila/share/api.py | 17:42 |
| gouthamr | ganso: yes saw that while i was looking up the get_progress() API .. wouldn't you want the policy check before hand? you can avoid unnecessary DB calls.. | 17:43 |
| ganso | gouthamr: I could say so, but all other API calls are doing it in share/api.py | 17:43 |
| gouthamr | ganso: we started using the decorator in wsgi.py a while ago; it's neat.. | 17:43 |
| gouthamr | ganso: *older* APIs :) | 17:43 |
| ganso | gouthamr: which decorator is that? | 17:43 |
| ganso | gouthamr: @wsgi.Controller.authorize ? | 17:44 |
| gouthamr | ganso: https://github.com/openstack/manila/blob/master/manila/api/openstack/wsgi.py#L1120 | 17:44 |
| gouthamr | ganso: yep | 17:44 |
| *** a_ta has quit IRC | 17:44 | |
| ganso | gouthamr: oh, ok | 17:44 |
| ganso | gouthamr: thanks =) | 17:44 |
| gouthamr | ganso: you're welcome! | 17:44 |
| ganso | gouthamr: but then | 17:45 |
| ganso | gouthamr: should still keep doing it share/api.py? | 17:45 |
| *** a_ta has joined #openstack-manila | 17:45 | |
| ganso | gouthamr: in case it is called from another part of the code? | 17:45 |
| gouthamr | ganso: nope.. you can remove the checks from share/api.. | 17:45 |
| ganso | gouthamr: ok, thanks | 17:45 |
| *** ociuhandu has quit IRC | 17:46 | |
| *** a_ta has quit IRC | 17:49 | |
| ganso | gouthamr: there is a slight problem | 17:51 |
| ganso | gouthamr: when adding it to migrate_share | 17:51 |
| gouthamr | ganso: ? | 17:51 |
| ganso | gouthamr: it searches for policy based on action_name | 17:51 |
| gouthamr | ganso: if you use the decorator with a parameter, it will work with differences between policy.json and your method name | 17:52 |
| gouthamr | ganso: for example, in policy.json, if you have "share:complete_migration": "rule:admin_api" and your method name is _migration_complete, try @wsgi.Controller.authorize("complete_migration") | 17:53 |
| ganso | gouthamr: oh, it supports a parameter | 17:53 |
| ganso | gouthamr: ok then, it will work =) | 17:53 |
| gouthamr | ganso | 17:53 |
| gouthamr | ganso: it's neat, i told you | 17:53 |
| gouthamr | ganso: :P | 17:53 |
| ganso | gouthamr: indeed it is | 17:53 |
| *** cknight has quit IRC | 17:59 | |
| openstackgerrit | Merged openstack/manila: EMC Isilon Driver Support For CIFS Read-Only Share https://review.openstack.org/278148 | 17:59 |
| *** haomaiwang has quit IRC | 18:01 | |
| *** haomaiwang has joined #openstack-manila | 18:01 | |
| *** ociuhandu has joined #openstack-manila | 18:02 | |
| *** cknight has joined #openstack-manila | 18:02 | |
| alkhodos_ | hi again, I have a question about tempest test: in test_rules.py there is a line access_to = "1.2.3.4/32". Is there a reason why subnet mask 32 is used? | 18:10 |
| bswartz | alkhodos_: the mask may be required, I'm not sure though. In any case it's synonymous with 1.2.3.4 | 18:10 |
| alkhodos_ | bswartz: The problem is that our backend does not accept subnet mask <30 (which is kinda weird usecase), but since it's hardcoded in the test I don't see how we can pass it | 18:12 |
| bswartz | just do a special case test | 18:14 |
| bswartz | ip_address = '/'.split(ip_address)[0] if ip_address.ends_with('/32') | 18:14 |
| bswartz | or something like that | 18:14 |
| vponomaryov | alkhodos_: feel free to change this value to any other mask | 18:18 |
| vponomaryov | alkhodos_: can be separate commit to Manila with bug description | 18:18 |
| alkhodos_ | vponomaryov: bswartz: ok, will create a patch for that | 18:19 |
| dustins | bswartz: looks like we're failing the gate-tripleo-ci-f22-nonha gate check :( | 18:21 |
| dustins | https://jenkins07.openstack.org/job/gate-tripleo-ci-f22-nonha/909/ | 18:21 |
| dustins | Looks like an HTTP 500 error in uploading some image as well as a ValueError on an semanage command | 18:23 |
| *** sgotliv has quit IRC | 18:25 | |
| *** lpetrut has quit IRC | 18:29 | |
| openstackgerrit | Ramana Raja proposed openstack/manila: Add Ceph Native driver https://review.openstack.org/270211 | 18:38 |
| openstackgerrit | Alexey Khodos proposed openstack/manila: Change subnet mask from 32 to 24 https://review.openstack.org/284343 | 18:39 |
| bswartz | dustins: I understand that system is not 100% reliable | 18:42 |
| dustins | bswartz: So it seems :) | 18:43 |
| bswartz | dustins: are you able to determine if the failure was infra-related or related the rhefner patch? | 18:43 |
| dustins | Well, something, somewhere is calling the semanage command and that's failing | 18:44 |
| dustins | Looks like a bunch of add-rule commands for iptables are failing, saying that a matching rule already exists | 18:44 |
| dustins | So my money's on infra | 18:44 |
| bswartz | iptables management can be a nightmare | 18:44 |
| dustins | even without it being automated :P | 18:45 |
| bswartz | I'm actually struggling with an unrelated iptables issue myself | 18:45 |
| dustins | Sounds like fun, haha | 18:45 |
| bswartz | oh actually this isn't iptables related.... looks like a kernel panic instead | 18:46 |
| bswartz | fun indeed | 18:46 |
| *** a_ta has joined #openstack-manila | 18:46 | |
| dustins | Oh yikes | 18:46 |
| openstackgerrit | Ramana Raja proposed openstack/manila: Add Ceph Native driver https://review.openstack.org/270211 | 18:47 |
| *** a_ta has quit IRC | 18:51 | |
| *** rafick has joined #openstack-manila | 18:56 | |
| *** rraja has quit IRC | 18:58 | |
| *** haomaiwang has quit IRC | 19:01 | |
| *** 16WAAEDST has joined #openstack-manila | 19:01 | |
| *** rafick has quit IRC | 19:03 | |
| *** lpetrut has joined #openstack-manila | 19:08 | |
| gouthamr | ganso: ping | 19:12 |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: Add ZFSonLinux share driver https://review.openstack.org/277192 | 19:12 |
| ganso | gouthamr: pong | 19:12 |
| gouthamr | ganso: hi again, quick check regarding this: https://review.openstack.org/#/c/282081/3/manila/share/rpcapi.py -> i was trying to reduce the RPC payload.. | 19:12 |
| gouthamr | ganso: looks like most other methods are doing it too.. | 19:13 |
| ganso | gouthamr: humm, that method runs in whoever's calling it. The payload would not be affected, because it is determined by call_context.cast. You would still be passing only the share_replica_id in the call_context.cast(share_replica_id=share_replica['id']) | 19:14 |
| gouthamr | ganso: ah.. i see what you're saying now. | 19:15 |
| *** a_ta has joined #openstack-manila | 19:15 | |
| *** martyturner has joined #openstack-manila | 19:16 | |
| ganso | gouthamr: you don't need share_replica_id and host parameters, you can have only share_replica parameter and extract the host, and use the share_replica['id'] in the cast. You would not be sending the replica model in the cast | 19:17 |
| gouthamr | ganso: true.. would be a minor change to all the replica methods in that file then.. | 19:17 |
| ganso | gouthamr: unless the host parameter is different than share_replica's | 19:17 |
| gouthamr | ganso: isn't.. can make the change | 19:19 |
| ganso | gouthamr: thanks =) | 19:19 |
| gouthamr | ganso: thank you for taking a look! | 19:20 |
| *** merooney has quit IRC | 19:26 | |
| *** bob has joined #openstack-manila | 19:29 | |
| bob | hello guys, manila interacts with openvswitch via the python libraires or the ovs binaries? | 19:29 |
| *** merooney has joined #openstack-manila | 19:29 | |
| *** bob is now known as Guest45114 | 19:29 | |
| bswartz | bob, Guest45114: it should only interact with ovs through neutron rest APIs | 19:30 |
| bswartz | bob, Guest45114: there is an exception for the generic driver which does some magic things | 19:31 |
| *** bob__ has joined #openstack-manila | 19:31 | |
| bob__ | hello guys, manila interacts with openvswitch via the python libraires or the ovs binaries? | 19:31 |
| bswartz | bob__: it should only interact with ovs through neutron rest APIs | 19:31 |
| bswartz | bob__: there is an exception for the generic driver which does some magic things | 19:31 |
| bob__ | ok | 19:33 |
| bob__ | Thanks | 19:33 |
| *** Guest45114 has quit IRC | 19:34 | |
| *** dustins_ has joined #openstack-manila | 19:34 | |
| *** dustins has quit IRC | 19:35 | |
| *** merooney has quit IRC | 19:36 | |
| *** dustins_ is now known as dustins | 19:38 | |
| *** esker has joined #openstack-manila | 19:47 | |
| *** dustins has quit IRC | 19:50 | |
| *** sgotliv has joined #openstack-manila | 19:52 | |
| *** 16WAAEDST has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-manila | 20:01 | |
| *** ociuhandu has quit IRC | 20:04 | |
| *** bob__ has quit IRC | 20:09 | |
| *** rhagarty_ has quit IRC | 20:16 | |
| *** rhagarty_ has joined #openstack-manila | 20:16 | |
| *** dustins has joined #openstack-manila | 20:17 | |
| openstackgerrit | xing-yang proposed openstack/manila: Manage and unmanage snapshot https://review.openstack.org/249542 | 20:50 |
| *** haomaiwang has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-manila | 21:01 | |
| *** xyang1 has quit IRC | 21:02 | |
| *** timcl has quit IRC | 21:03 | |
| *** cknight has quit IRC | 21:04 | |
| *** martyturner has quit IRC | 21:04 | |
| *** akerr has quit IRC | 21:05 | |
| *** pcaruana has quit IRC | 21:10 | |
| *** lpetrut has quit IRC | 21:19 | |
| *** ata has joined #openstack-manila | 21:20 | |
| *** ata is now known as Guest21777 | 21:21 | |
| *** Guest21777 has quit IRC | 21:25 | |
| *** cknight has joined #openstack-manila | 21:33 | |
| *** ekarlso- has quit IRC | 21:34 | |
| *** ekarlso- has joined #openstack-manila | 21:34 | |
| *** dustins has quit IRC | 21:36 | |
| *** cknight1 has joined #openstack-manila | 21:37 | |
| *** dustins has joined #openstack-manila | 21:38 | |
| *** cknight has quit IRC | 21:39 | |
| *** mtanino__ has joined #openstack-manila | 21:41 | |
| *** gouthamr has quit IRC | 21:57 | |
| *** porrua has quit IRC | 21:57 | |
| *** haomaiwang has quit IRC | 22:01 | |
| *** haomaiwang has joined #openstack-manila | 22:01 | |
| *** dustins has quit IRC | 22:17 | |
| *** adrianofr_ has quit IRC | 22:20 | |
| *** gouthamr has joined #openstack-manila | 22:23 | |
| *** gouthamr_ has joined #openstack-manila | 22:26 | |
| *** tpsilva has quit IRC | 22:27 | |
| *** gouthamr has quit IRC | 22:28 | |
| *** mdenny has joined #openstack-manila | 22:41 | |
| *** aorourke has quit IRC | 22:44 | |
| *** aorourke has joined #openstack-manila | 22:45 | |
| *** a_ta has quit IRC | 22:57 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** haomaiwang has joined #openstack-manila | 23:01 | |
| *** sgotliv has quit IRC | 23:06 | |
| *** ganso has quit IRC | 23:19 | |
| *** eharney has quit IRC | 23:23 | |
| *** lpabon has quit IRC | 23:29 | |
| *** merooney has joined #openstack-manila | 23:55 | |
| *** merooney_ has joined #openstack-manila | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!