*** a_ta has joined #openstack-manila | 00:05 | |
*** yangyapeng has quit IRC | 00:08 | |
*** a_ta has quit IRC | 00:10 | |
*** gouthamr has quit IRC | 00:51 | |
*** akshai has quit IRC | 00:58 | |
*** akshai has joined #openstack-manila | 01:02 | |
*** akshai has quit IRC | 01:08 | |
*** yangyapeng has joined #openstack-manila | 01:11 | |
*** yangyapeng has quit IRC | 01:17 | |
*** gouthamr has joined #openstack-manila | 01:25 | |
*** gouthamr_ has joined #openstack-manila | 01:28 | |
*** akshai has joined #openstack-manila | 01:31 | |
*** gouthamr has quit IRC | 01:31 | |
*** mtanino has quit IRC | 01:37 | |
*** akshai_ has joined #openstack-manila | 01:39 | |
*** akshai has quit IRC | 01:42 | |
gouthamr_ | bswartz: there? | 01:58 |
---|---|---|
*** esker has joined #openstack-manila | 02:01 | |
*** rhagarty_ has quit IRC | 02:04 | |
*** esker has quit IRC | 02:07 | |
*** a_ta has joined #openstack-manila | 02:07 | |
*** a_ta has quit IRC | 02:11 | |
*** rhagarty has joined #openstack-manila | 02:12 | |
*** baojg has joined #openstack-manila | 02:17 | |
*** rhagarty has quit IRC | 02:36 | |
*** esker has joined #openstack-manila | 02:43 | |
*** houming has joined #openstack-manila | 02:44 | |
*** esker has quit IRC | 02:48 | |
*** docaedo has joined #openstack-manila | 02:50 | |
*** akshai_ has quit IRC | 02:51 | |
*** akshai has joined #openstack-manila | 02:56 | |
openstackgerrit | Merged openstack/manila: Remove unintended exposure of private attribute https://review.openstack.org/287506 | 02:58 |
*** akshai has quit IRC | 03:00 | |
*** amrith is now known as _amrith_ | 03:15 | |
*** esker has joined #openstack-manila | 03:18 | |
bswartz | gouthamr_: sup? | 03:27 |
gouthamr_ | bswartz: hey! was thinking about deletion of snapshots for replicas and thought you'd be able to help me out... | 03:28 |
bswartz | ok | 03:28 |
gouthamr_ | bswartz: if i send all the snapshot instances to the active replica's host.. what does the driver return? | 03:30 |
gouthamr_ | bswartz: 1) return immediately 2) take its time for cleanup and return when done, or 3) return a list of snapshot instances it was able to clean up | 03:30 |
gouthamr_ | we discussed case 1 and we said we will poll for 'deleting' snapshots.. but isn't 3 a good thing too? | 03:31 |
gouthamr_ | bswartz: no hurry in letting me know; we can speak about this tomorrow too. | 03:33 |
*** gouthamr_ has quit IRC | 03:48 | |
*** esker has quit IRC | 03:49 | |
*** yangyapeng has joined #openstack-manila | 03:50 | |
*** esker has joined #openstack-manila | 04:27 | |
*** yangyape_ has joined #openstack-manila | 04:27 | |
*** openstack has joined #openstack-manila | 14:09 | |
*** dustins has joined #openstack-manila | 14:25 | |
tbarron | vponomaryov: jcsp: each instance of a driver (linux process) will have its own discrete set of shares, but can there be greenthread concurrent access to a single share within a given driver instance? | 14:26 |
tbarron | vponomaryov: jcsp: you could outlaw this and enforce it in manila api | 14:26 |
tbarron | but in cinder there has been discussion of allowing some concurrent operations on a given volume | 14:27 |
tbarron | actually more than discussion | 14:27 |
bswartz | tbarron: I think we want to move the opposite direction | 14:29 |
bswartz | right now we don't enforce mutual exclusion as well as we should | 14:29 |
bswartz | so any concurrent operations that we allow might be by accident | 14:29 |
tbarron | bswartz: i'm not disagreeing, just posing the question given the ML discussion | 14:30 |
tbarron | life will certainly be simpler and it's complex enough | 14:31 |
tbarron | anyways :) | 14:31 |
*** merooney has quit IRC | 14:32 | |
*** merooney has joined #openstack-manila | 14:35 | |
*** openstackgerrit_ has quit IRC | 14:36 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/manila-ui: Updated from global requirements https://review.openstack.org/288484 | 14:37 |
*** openstackgerrit_ has joined #openstack-manila | 14:37 | |
*** eharney has joined #openstack-manila | 14:39 | |
*** rhagarty has joined #openstack-manila | 14:45 | |
*** rhagarty_ has joined #openstack-manila | 14:46 | |
openstackgerrit | Alexey Ovchinnikov proposed openstack/manila-image-elements: Add tox job for building LXD image https://review.openstack.org/232142 | 14:46 |
*** rhagarty has quit IRC | 14:48 | |
jcsp | bswartz, tbarron: the #1 thing for me is just that the behaviour is defined; if it's more concurrent than we like then we can always add locks, but if we're not sure then it's hard | 14:50 |
bswartz | doh! why is proposal bot proposing g-r update NOW of all times? | 14:51 |
bswartz | seems like the day after feature freeze isn't the smartest time to change requirements | 14:51 |
bswartz | maybe I'm missing something... | 14:51 |
openstackgerrit | Dustin Schoenbrun proposed openstack/manila: Add thin provisioning column to support mapping https://review.openstack.org/288504 | 14:53 |
bswartz | tbarron, jcsp: I'm lacking context about your concern | 14:53 |
bswartz | is there a specific operation you're concerned about? | 14:53 |
bswartz | or just general angst about parallelism | 14:53 |
jcsp | bswartz: so I think we kind of need to know in general because otherwise it's hard to write correct code. The specific case here is that to implement update_access() (and do some other stuff) we have to start persisting more metadata about our auth stuff | 14:54 |
jcsp | and when I'm doing that from the manila side, I'm writing some files to cephfs, and I'm also updating our authorization keys | 14:54 |
jcsp | and that has natural two phase commit type issues that you might imagine | 14:54 |
*** cknight has joined #openstack-manila | 14:55 | |
bswartz | jcsp: fwiw I agree that defined behavior is better than undefined behavior -- and we have a lot of poorly documented behaviors | 14:55 |
bswartz | okay | 14:55 |
jcsp | for most stuff we use existing drivers to cover the gaps in docs, but for concurrency stuff that's not possible | 14:55 |
bswartz | well during newton, address concurrency issues at all level is likely to be a top priority | 14:55 |
jcsp | cool | 14:55 |
jcsp | not trying to beat anyone up about it, just gathering what info I can | 14:55 |
bswartz | part of that must be writing down more detail definitions of how stuff *should* work | 14:56 |
vponomaryov1 | jcsp: note about concurrency: ML answer related to several "ensure_share" calls, but if you do some other call, i.e. deletion it may be concurrent | 14:56 |
*** a_ta has joined #openstack-manila | 14:57 | |
vponomaryov1 | jcsp: so, from driver point of view, you can set locks based on share ID | 14:57 |
vponomaryov1 | jcsp: and it should be enough | 14:57 |
*** martyturner has quit IRC | 14:57 | |
vponomaryov1 | mkoderer: ping | 14:57 |
*** merooney has quit IRC | 14:58 | |
vponomaryov1 | mkoderer__: ping | 14:58 |
vponomaryov1 | mkoderer, mkoderer__: fix of bug filed by you - https://bugs.launchpad.net/manila/+bug/1553082 will require fix on nova and cinder clients that are used in Generic driver | 14:59 |
openstack | Launchpad bug 1553082 in Manila "Keystone v3 support broken" [Undecided,New] - Assigned to Marc Koderer (m-koderer) | 14:59 |
bswartz | markstur_ toabctl xyang1: I'd like to speed this change through the gate in order to test a fix to the "post" job for manila-image-elements: https://review.openstack.org/#/c/288378 | 14:59 |
* bswartz wishes it were easier to test post jobs | 15:00 | |
xyang1 | bswartz: ok, looking | 15:00 |
toabctl | done :) | 15:00 |
bswartz | ty | 15:00 |
vponomaryov1 | yeah, it takes for 5 minutes against 9,5 as it is now | 15:00 |
xyang1 | toabctl: you beat me:) | 15:01 |
toabctl | :) | 15:01 |
bswartz | vponomaryov1: regarding your comment about PS5 failure on my bugfix https://review.openstack.org/#/c/287316 that failure was because my fix was incorrect, and I added PS6 | 15:02 |
*** timcl has quit IRC | 15:02 | |
*** timcl has joined #openstack-manila | 15:03 | |
*** rraja has quit IRC | 15:03 | |
vponomaryov1 | bswartz: even so, itis safer to keep retry | 15:03 |
bswartz | actually no, the incorrect fix was in 4 | 15:04 |
bswartz | so 5 shold have worked | 15:04 |
bswartz | I'm going to look again | 15:04 |
*** martyturner has joined #openstack-manila | 15:04 | |
*** merooney has joined #openstack-manila | 15:04 | |
bswartz | okay you're right | 15:06 |
bswartz | I can see failures even with patchset 5 | 15:06 |
bswartz | this makes me worry about how correct the fix is | 15:06 |
bswartz | I wonder what could cause the dataset to remain busy if not another namespace | 15:07 |
bswartz | vponomaryov1: when you were debugging busy dataset problem initially, what tools did you use to track down the cause of the problem? | 15:07 |
vponomaryov1 | bswartz: I did search for mounts in "/proc/*/mounts" restarting neutron dhcp service after dataset creation | 15:09 |
vponomaryov1 | bswartz: also, sometimes it returns busy just because some other dataset being created | 15:10 |
vponomaryov1 | bswartz: so, I would say absence of "retry" will make "deletion of dataset" very fragile | 15:10 |
*** esker has joined #openstack-manila | 15:14 | |
openstackgerrit | Merged openstack/manila-image-elements: Disable ZFS element by default https://review.openstack.org/288378 | 15:14 |
*** esker has quit IRC | 15:15 | |
*** esker has joined #openstack-manila | 15:15 | |
*** jasonsb has joined #openstack-manila | 15:16 | |
*** timcl has quit IRC | 15:17 | |
vponomaryov1 | bswartz: after merge of commit to manila-image-elements, previously existed image gone from http://tarballs.openstack.org/manila-image-elements/images/ | 15:21 |
vponomaryov1 | bswartz: there was 'manila-service-image-master.qcow2' | 15:21 |
vponomaryov1 | bswartz: and for this last commit job "gate-manila-publishimage-generic" has not started yet | 15:22 |
*** david_lyle__ has joined #openstack-manila | 15:34 | |
*** rhagarty_ has quit IRC | 15:36 | |
*** david-lyle has quit IRC | 15:37 | |
*** yangyapeng has joined #openstack-manila | 15:38 | |
*** yangyapeng has quit IRC | 15:39 | |
*** lpetrut has quit IRC | 15:39 | |
*** lpetrut has joined #openstack-manila | 15:39 | |
*** lpetrut has quit IRC | 15:47 | |
*** mtanino has joined #openstack-manila | 15:49 | |
*** rhagarty has joined #openstack-manila | 15:51 | |
*** rhagarty_ has joined #openstack-manila | 15:52 | |
*** merooney has quit IRC | 15:54 | |
*** martyturner has quit IRC | 15:54 | |
*** rhagarty has quit IRC | 15:56 | |
*** timcl has joined #openstack-manila | 15:56 | |
*** martyturner has joined #openstack-manila | 15:58 | |
*** merooney has joined #openstack-manila | 15:58 | |
*** merooney has quit IRC | 16:13 | |
*** david_lyle__ is now known as david_lyle | 16:17 | |
openstackgerrit | Ben Swartzlander proposed openstack/manila: Use official location for service image https://review.openstack.org/288564 | 16:21 |
bswartz | vponomaryov1: ? | 16:21 |
bswartz | vponomaryov1: I don't see any problems | 16:21 |
*** timcl has quit IRC | 16:22 | |
bswartz | my commit above tests the new service image | 16:22 |
bswartz | vponomaryov: ^ | 16:22 |
vponomaryov1 | bswartz: problem is in absense of the image while "publish" job is not finished | 16:25 |
vponomaryov1 | bswartz: it is "hours" | 16:25 |
vponomaryov1 | bswartz: and generic driver jobs will fail a lot | 16:26 |
vponomaryov1 | bswartz: after each new commit merged to elements project | 16:26 |
bswartz | okay, so we need a way to atomically swap in new images | 16:26 |
bswartz | or at least in a small window | 16:26 |
bswartz | I suppose it's okay if 1 or 2 jobs fail during a job upload | 16:27 |
bswartz | how can it take hours to scp a 300MB file? | 16:27 |
bswartz | that seems broken | 16:27 |
bswartz | one moment.... | 16:27 |
vponomaryov1 | bswartz: not copy - "waiting in queue" | 16:27 |
bswartz | yeah but the old image doesn't disappear while it's in the queue | 16:28 |
vponomaryov1 | bswartz: I observed it | 16:28 |
*** rhagarty has joined #openstack-manila | 16:28 | |
bswartz | what did you observe? no manila-service-image-master.qcow2 file in directory? or whole directory missing? | 16:29 |
vponomaryov1 | yes, no file | 16:29 |
bswartz | I *just* fixed the job that updates the image | 16:29 |
bswartz | so that was the first upload ever | 16:29 |
bswartz | next upload should simply overwrite file | 16:29 |
vponomaryov1 | this image was before first merge | 16:29 |
vponomaryov1 | that I observed too | 16:29 |
bswartz | http://logs.openstack.org/92/9291b4781e4a10e9343528925fb120844f496925/post/gate-manila-publishimage-generic/8d5b24c/console.html#_2016-03-04_15_44_15_962 | 16:30 |
vponomaryov1 | it disappeared after merge and till end of publish job | 16:30 |
*** rhagarty_ has quit IRC | 16:30 | |
bswartz | upload appears to be 40 seconds | 16:30 |
vponomaryov1 | bswartz: but this job ran after hour+ | 16:30 |
vponomaryov1 | bswartz: waiting in queue | 16:30 |
bswartz | yes but there was no file there before the job ran | 16:30 |
bswartz | let's merge another change to tests this | 16:31 |
bswartz | I think it's just a fluke because this is the first patch to merge since my change to the job | 16:31 |
vponomaryov1 | bswartz: could be someone copied it and renamed to second image that is there | 16:31 |
bswartz | the other file there is very old -- last november | 16:31 |
bswartz | I'm not sure where that came from | 16:32 |
vponomaryov1 | bswartz: ok, let's think it was some manual change | 16:32 |
bswartz | I think it must have been | 16:32 |
bswartz | still, let's merge a dummy change | 16:32 |
bswartz | or fix some small bug in manila-image-elements | 16:32 |
bswartz | and watch what happens | 16:32 |
bswartz | do you have any other small bugs to fix? | 16:33 |
vponomaryov1 | me? | 16:34 |
bswartz | yeah | 16:34 |
bswartz | or anyone | 16:34 |
vponomaryov1 | yes | 16:34 |
vponomaryov1 | https://bugs.launchpad.net/manila/+bugs?field.tag=zfsonlinux | 16:34 |
bswartz | I meant bugs for manila-image-elements | 16:35 |
vponomaryov1 | oh )) | 16:35 |
bswartz | I want to push another commit and observe the tarballs.o.o site | 16:35 |
vponomaryov1 | bswartz: for example: add info about target palce where newest version is stored | 16:37 |
vponomaryov1 | to https://github.com/openstack/manila-image-elements/blob/master/README.rst | 16:37 |
bswartz | good idea | 16:37 |
bswartz | okay 1 sec | 16:37 |
openstackgerrit | Ben Swartzlander proposed openstack/manila-image-elements: Provide link to where images are uploaded https://review.openstack.org/288573 | 16:40 |
bswartz | vponomaryov1: ^ | 16:40 |
bswartz | pls don't workflow until I'm back from lunch | 16:41 |
bswartz | or unless you have time to watch the server | 16:41 |
bswartz | someone needs to watch zuul while the job is traveling through the gate->post pipelines | 16:41 |
bswartz | I'll bbiab | 16:41 |
*** ayma has joined #openstack-manila | 16:43 | |
*** rhagarty_ has joined #openstack-manila | 16:45 | |
*** mlima has joined #openstack-manila | 16:45 | |
*** rhagarty has quit IRC | 16:47 | |
*** fthiagogv__ is now known as fthiagogv | 16:47 | |
*** ayma1 has joined #openstack-manila | 17:00 | |
*** ayma has quit IRC | 17:00 | |
*** ayma has joined #openstack-manila | 17:03 | |
*** ayma1 has quit IRC | 17:06 | |
*** absubram has joined #openstack-manila | 17:13 | |
*** absubram_ has joined #openstack-manila | 17:14 | |
*** absubram has quit IRC | 17:18 | |
*** absubram_ is now known as absubram | 17:18 | |
*** martyturner has quit IRC | 17:22 | |
*** martyturner has joined #openstack-manila | 17:24 | |
*** jasonsb has quit IRC | 17:26 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: [DEBUG] do not merge 1 https://review.openstack.org/286558 | 17:31 |
bswartz | I'm back | 17:42 |
*** merooney has joined #openstack-manila | 17:46 | |
bswartz | vponomaryov, vponomaryov1: my manila-image-elements patch is in the gate | 17:46 |
bswartz | I'll be watching tarballs site for modifications | 17:46 |
tpsilva | cknight: ping | 17:48 |
cknight | tpsilva: pong | 17:48 |
*** Yogi1 has joined #openstack-manila | 17:49 | |
*** mlima has quit IRC | 17:50 | |
tpsilva | cknight: hello! sorry to bother, it's kinda offtopic actually... I see that you always complain about the code styles on the reviews, so maybe you'll know this. Why do we always use db models as dictionaries instead of objects? Is this a guideline or something will break if we don't use them like this? | 17:51 |
cknight | tpsilva: Hi, Tiago. Honestly I have no idea apart from inertia and consistency. Perhaps bswartz or ameade could provide some historical insight. | 17:54 |
gouthamr | tpsilva: i might have a point to add.. *never* send drivers SQLAlchemy objects | 17:54 |
*** akerr_ has quit IRC | 17:54 | |
*** timcl has joined #openstack-manila | 17:55 | |
vponomaryov1 | tpsilva: first of all, it does not pick up all the data from DB for all "relationships" and will try to get info when you "try to use it". BUT, in this cae, when ou provide it somewhere else, you get "detached" state and cannot get info anymore | 17:55 |
tpsilva | cknight: consistency is a great reason by itself :) | 17:55 |
vponomaryov1 | tpsilva: so, you just fail to get required info when you send DB object | 17:55 |
vponomaryov1 | tpsilva: use always primitives | 17:55 |
openstackgerrit | Tom Barron proposed openstack/manila: Fix unconditional pylxd unit test dependency https://review.openstack.org/288613 | 17:56 |
ameade | tpsilva: there is a TODO from the very first impl in nova to have the db layer only return dictionaries | 17:57 |
ameade | that work has just never been done | 17:57 |
ameade | imo, returning sqlalchemy objects has caused a lot of pain | 17:57 |
ameade | vponomaryov1: +1 | 17:58 |
ameade | only reasoning i can imagine would be lazy loading but vponomaryov1's point is why that doesnt work anyways | 17:58 |
tpsilva | awesome, lots of info... was not expecting that | 17:58 |
vponomaryov1 | tpsilva: and second - easy to log/handle instead of using "to_dict()" explciit translations | 17:58 |
tpsilva | vponomaryov1: yep, in fact is really easier to log with dicts | 17:59 |
tpsilva | gouthamr: the db layer already does this work for me, right? | 17:59 |
ameade | tpsilva: no unfortunately | 17:59 |
gouthamr | tpsilva: as of today, no.. there's a TODO kinda comment in db/api.py | 17:59 |
gouthamr | tpsilva: https://github.com/openstack/manila/blob/master/manila/db/api.py#L25 | 18:00 |
tpsilva | ameade, gouthamr: oh wow... so the update_access is wrong? https://github.com/openstack/manila/blob/master/manila/share/access.py#L74 | 18:00 |
bswartz | this discussion overlaps with the versioned objects project | 18:00 |
tpsilva | rules is retrieved from the DB on LN 60 | 18:01 |
gouthamr | tpsilva: not wrong.. it is sending SQLAlchemy objects to the driver... as is the case with a bunch of methods | 18:01 |
bswartz | versioned objects are better than dicts because they can emulate both new and old behaviors while both are needed during an upgrade | 18:01 |
*** timcl has quit IRC | 18:02 | |
*** timcl1 has joined #openstack-manila | 18:02 | |
*** esker has quit IRC | 18:02 | |
bswartz | vponomaryov1: jobs is still queued in post pipe and no change to tarballs site yet | 18:02 |
gouthamr | tpsilva: look at https://github.com/openstack/manila/blob/master/manila/share/manager.py#L2454 -> not the grosest thing.. but a workaround for now.. we should solve this problem soon.. | 18:03 |
bswartz | gouthamr: there's no time like today... | 18:05 |
tpsilva | gouthamr: I see... looks easy to be implemented on the DB layer | 18:05 |
tpsilva | awesome, thank you guys! | 18:06 |
vponomaryov1 | bswartz: then it was manual intruding | 18:06 |
* bswartz is still watching | 18:07 | |
gouthamr | bswartz: i've a post it note.. :P | 18:08 |
* bswartz wonders how long jobs remain queued in post pipe.... | 18:10 | |
bswartz | so vponomaryov's patch merged 7 hours ago, but the last update to tarballs was just 2.25 hours ago | 18:11 |
*** merooney has quit IRC | 18:12 | |
bswartz | that implies that job waited ~5 hours | 18:12 |
bswartz | maybe I won't watch that closely | 18:12 |
bswartz | I have a 10 second cron job watching the directory on the server, I'll just let that run | 18:12 |
* gouthamr zuul wants some space | 18:13 | |
tpsilva | another thing: manila doesn't use versioned objects, right? | 18:14 |
gouthamr | no tpsilva... | 18:16 |
dustins | cknight: thanks for the feedback, I think I'm going to create a new table at the end of the feature support mapping | 18:21 |
*** merooney has joined #openstack-manila | 18:21 | |
tpsilva | gouthamr: thanks! | 18:22 |
cknight | dustins: ok | 18:24 |
*** Yogi1 has quit IRC | 18:29 | |
*** JoseMello has joined #openstack-manila | 18:34 | |
*** akerr has joined #openstack-manila | 18:35 | |
*** eharney has quit IRC | 18:35 | |
gouthamr | tpsilva: np! | 18:36 |
*** mlima has joined #openstack-manila | 18:39 | |
*** yangyapeng has joined #openstack-manila | 18:40 | |
*** pcaruana has quit IRC | 18:41 | |
*** yangyapeng has quit IRC | 18:45 | |
*** merooney has quit IRC | 18:46 | |
*** merooney has joined #openstack-manila | 18:47 | |
*** akerr has quit IRC | 18:57 | |
*** akerr has joined #openstack-manila | 18:58 | |
*** martyturner has quit IRC | 19:09 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: [DEBUG] do not merge 1 https://review.openstack.org/286558 | 19:12 |
*** a_ta has quit IRC | 19:12 | |
*** ociuhandu has quit IRC | 19:13 | |
*** a_ta has joined #openstack-manila | 19:13 | |
*** eharney has joined #openstack-manila | 19:13 | |
*** timcl1 has quit IRC | 19:17 | |
*** a_ta has quit IRC | 19:17 | |
*** fthiagogv has quit IRC | 19:22 | |
openstackgerrit | Andrea Ma proposed openstack/manila: glusterManager instantiation regexp validation https://review.openstack.org/283777 | 19:23 |
bswartz | vponomaryov1: it finally ran the job | 19:25 |
bswartz | there was a ~60 second window when the qcow2 file was being updated | 19:25 |
*** a_ta has joined #openstack-manila | 19:25 | |
bswartz | I've got logs to show it | 19:25 |
bswartz | I think a 60 second window is small enough that problems are extremely unlikely, nevertheless I'll talk to the infra folks and see if we can get it down to zero | 19:26 |
vponomaryov1 | bswartz: whole 60 seconds | 19:27 |
*** ociuhandu has joined #openstack-manila | 19:31 | |
*** Yogi1 has joined #openstack-manila | 19:32 | |
*** merooney has quit IRC | 19:32 | |
docaedo | bswartz: regarding linking to the image in https://apps.openstack.org, if you know what the URL is going to be now | 19:32 |
docaedo | bswartz: you could add it to assets.yaml and we'd be ready to merge when you're ready | 19:33 |
*** timcl has joined #openstack-manila | 19:33 | |
docaedo | bswartz: also just waving my hand, I'm around for another 30, then teaching until 2pm PST after which time I'm back around for the day | 19:33 |
*** merooney has joined #openstack-manila | 19:34 | |
*** martyturner has joined #openstack-manila | 19:41 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: Download service image only when needed https://review.openstack.org/288705 | 19:49 |
*** martyturner has quit IRC | 19:54 | |
*** Yogi1 has quit IRC | 19:55 | |
*** dencaval has quit IRC | 19:56 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: [DEBUG] do not merge 1 https://review.openstack.org/286558 | 20:01 |
*** mlima has quit IRC | 20:02 | |
*** timcl has quit IRC | 20:11 | |
*** merooney has quit IRC | 20:19 | |
*** merooney has joined #openstack-manila | 20:23 | |
openstackgerrit | Tom Barron proposed openstack/manila: Remove config generation pylxd hard dependency https://review.openstack.org/288721 | 20:25 |
bswartz | docaedo: yes I plan to push a change tehre | 20:27 |
*** lpetrut has joined #openstack-manila | 20:27 | |
openstackgerrit | Ben Swartzlander proposed openstack/manila: Allow devstack plugin to work without Cinder https://review.openstack.org/278562 | 20:32 |
openstackgerrit | Ben Swartzlander proposed openstack/manila: Allow devstack plugin to work without Cinder https://review.openstack.org/278562 | 20:33 |
*** timcl has joined #openstack-manila | 20:35 | |
bswartz | tbarron: ping re https://review.openstack.org/#/c/288721/ | 20:36 |
tbarron | bswartz: pong | 20:36 |
bswartz | tbarron: why does change need to be in opts.py and not down inside lxd driver? | 20:36 |
tbarron | maybe that would work, this is one way to do it | 20:37 |
bswartz | we've seen drivers before with "soft" dependencies | 20:37 |
bswartz | perhaps LXD should be one of those | 20:37 |
tbarron | that might be cleaner | 20:37 |
bswartz | I'm curious what the issue is with the pylxd lib -- is there a license problem? is redhat unable to distribute it due to patents? | 20:38 |
tbarron | i'll mark it wip till i get a firmer opinion | 20:38 |
bswartz | if there's a good reason redhat cannot distribute it, then I expect the TC would support not allowing it in reqs.txt | 20:38 |
tbarron | it broke RDO nightly build | 20:38 |
tbarron | packaging people don't want to add rpms for stuff we aren't actually using | 20:39 |
tbarron | or generate options for stuff we won't actually ship | 20:39 |
bswartz | redhat won't ship the lxd driver? | 20:39 |
bswartz | will you delete it from your downstream branch? | 20:39 |
tbarron | only canonical has support for lxd | 20:39 |
tbarron | not even debian | 20:40 |
tbarron | that could change, but that's the way it is today | 20:40 |
tbarron | i don't think we'll remove the driver | 20:41 |
tbarron | but it won't be "first party" in our distro | 20:41 |
bswartz | I know the project was started by canonical -- does redhat have something that competes? | 20:41 |
tbarron | i don't know the answers to the product manager questions :-) | 20:41 |
tbarron | my only role in this is to get our build working | 20:42 |
tbarron | also, rdo will be shipping unit tests soon. hence https://review.openstack.org/#/c/288613/ | 20:44 |
bswartz | well it matters because the community tends to push back on requirements that are not widely available | 20:44 |
bswartz | my personal tendency is to go with whatever works | 20:44 |
bswartz | but we need to be careful and make sure everyone is happy | 20:44 |
tbarron | bswartz: I agree with that perception, am just trying to be neutral/technical on the actual concrete problems. | 20:45 |
bswartz | any dependencies we're invoking that aren't widely available should be turned to soft dependencies | 20:45 |
bswartz | to avoid exactly this kind of technical issue | 20:45 |
*** merooney has quit IRC | 20:55 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: [DEBUG] do not merge 1 https://review.openstack.org/286558 | 20:56 |
*** timcl has quit IRC | 21:02 | |
*** rhagarty_ has quit IRC | 21:07 | |
*** rhagarty_ has joined #openstack-manila | 21:07 | |
*** JoseMello has quit IRC | 21:08 | |
*** vponomaryov1 has left #openstack-manila | 21:11 | |
*** akerr has quit IRC | 21:34 | |
*** porrua has quit IRC | 21:39 | |
*** gouthamr has quit IRC | 21:43 | |
openstackgerrit | Dustin Schoenbrun proposed openstack/manila: Add thin provisioning column to support mapping https://review.openstack.org/288504 | 21:51 |
*** pcaruana has joined #openstack-manila | 21:58 | |
*** dustins has quit IRC | 22:10 | |
*** gouthamr has joined #openstack-manila | 22:23 | |
*** akshai has quit IRC | 22:25 | |
*** tpsilva has quit IRC | 22:27 | |
*** erlon has quit IRC | 22:46 | |
*** sbjorn has quit IRC | 22:48 | |
*** a_ta has quit IRC | 22:54 | |
*** lpetrut has quit IRC | 23:03 | |
*** sedwards has joined #openstack-manila | 23:07 | |
*** pgbridge has quit IRC | 23:26 | |
*** xyang1 has quit IRC | 23:29 | |
ayma | seems like gate-manila-tempest-dsvm-neutron-multibackend is failing? looking at the logs it seems to say | 23:50 |
ayma | manila_tempest_tests.share_exceptions.ShareBuildErrorException | 23:51 |
ayma | was something changed with the gate? | 23:51 |
openstackgerrit | Tom Barron proposed openstack/manila: Fix pylxd hard dependencies https://review.openstack.org/288800 | 23:56 |
tbarron | bswartz: not sure how you want to do soft depdencies, but here's a strawman | 23:57 |
tbarron | ^^ | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!