| notmyname | http://www.bbc.com/capital/story/20161028-native-english-speakers-are-the-worlds-worst-communicators and I feel I should apologize to most people in here | 00:09 |
|---|---|---|
| zaitcev | You mean apologize for not realizing how statistics work? | 00:14 |
| timburke | notmyname: don't worry; we don't speak english, just american | 00:15 |
| zaitcev | apropos http://www.sandraandwoo.com/2013/12/19/0538-butterfly-vs-statistics/ | 00:15 |
| notmyname | timburke: I think that's the problem | 00:17 |
| timburke | i'm curious about the particular word from the lede | 00:19 |
| mattoliverau | notmyname: that's right, you should all speak real english.. like us Australians :P ... lol almost kept a straight face there :P | 00:24 |
| notmyname | mattoliverau: did you mean "speaky engly"? | 00:25 |
| mattoliverau | lol | 00:25 |
| openstackgerrit | Tim Burke proposed openstack/swift: Make NullLogger more logger like https://review.openstack.org/373784 | 00:29 |
| notmyname | mattoliverau: after I saw a polly in budgie smugglers this arvo, I went to the beach to have a think about it. had to drink heaps of cab sav to get over it, then took a liquid laugh. decided to go on walkabout just to get away from it all. found a station, became a jackaroo, never been happier | 00:38 |
| notmyname | (I only had to look up some of those) | 00:38 |
| mattoliverau | notmyname: I'm very impressed. and see doesn't that make much more sense! And alot easier to write :P | 00:38 |
| mattoliverau | timburke: ^^ what do you think | 00:39 |
| clayg | timburke: did you happen to try `exception = critical = ... = lambda *args: None` | 00:39 |
| clayg | i think `warn` as an alias to `warning` had bit us in the past on logger-likes | 00:40 |
| timburke | clayg: eh, w/e. *you* weren't pushing over it | 00:40 |
| clayg | it's on a list somewhere | 00:40 |
| timburke | notmyname: mattoliverau: that is a... curious... euphemism. what about when it's chunky? | 00:40 |
| *** vint_bra has quit IRC | 00:46 | |
| clayg | notmyname: right i want to know what the word was | 00:46 |
| clayg | it's all rubbish | 00:46 |
| *** zhengyin has quit IRC | 00:50 | |
| *** zhengyin has joined #openstack-swift | 00:51 | |
| *** vinsh has quit IRC | 01:02 | |
| zhengyin | good morning | 01:03 |
| *** klrmn has quit IRC | 01:07 | |
| *** klrmn has joined #openstack-swift | 01:08 | |
| *** klrmn has quit IRC | 01:10 | |
| openstackgerrit | Merged openstack/swift: Add unit test content https://review.openstack.org/353444 | 01:16 |
| mattoliverau | zhengyin: morning! | 01:19 |
| zhengyin | mattoliverau:morning :) | 01:19 |
| *** jerrygb_ has joined #openstack-swift | 01:30 | |
| *** vinsh has joined #openstack-swift | 01:30 | |
| *** jerrygb has quit IRC | 01:31 | |
| openstackgerrit | Matthew Oliver proposed openstack/swift: Bad error reporting on config permissions error https://review.openstack.org/273325 | 01:32 |
| *** jerrygb has joined #openstack-swift | 01:45 | |
| *** jerrygb_ has quit IRC | 01:45 | |
| *** dmorita has quit IRC | 01:55 | |
| *** jerrygb has quit IRC | 01:59 | |
| *** jerrygb has joined #openstack-swift | 01:59 | |
| openstackgerrit | Merged openstack/swift: Use more specific asserts in test/probe tests https://review.openstack.org/342765 | 02:00 |
| *** manous has quit IRC | 02:02 | |
| openstackgerrit | Merged openstack/swift: Make NullLogger more logger like https://review.openstack.org/373784 | 02:07 |
| *** jerrygb has quit IRC | 02:17 | |
| *** jerrygb has joined #openstack-swift | 02:18 | |
| *** jerrygb has quit IRC | 02:22 | |
| *** klrmn has joined #openstack-swift | 02:27 | |
| *** bkopilov has quit IRC | 02:51 | |
| *** vinsh has quit IRC | 02:51 | |
| *** jerrygb has joined #openstack-swift | 02:59 | |
| *** vinsh has joined #openstack-swift | 03:04 | |
| *** blair has quit IRC | 03:05 | |
| *** blair has joined #openstack-swift | 03:06 | |
| jrichli | mattoliverau: I wasn't able to listen in on the container sharding discussion, but I saw that you were displaying some document with info. | 03:11 |
| jrichli | was that the same doc that you had created after the San Antonio Hackathon? or something new? | 03:12 |
| mattoliverau | jrichli: it's the inital shard vs subsequent sharding doc I have linked on the info page.. I just added updates to the end | 03:27 |
| mattoliverau | I should do something nicer at some point. but that was an already easily accessable document :) | 03:27 |
| jrichli | you mean the one linked off the ideas page? | 03:30 |
| mattoliverau | yup thats what I mean, turns out I'm still jetlagged and what I type and think are 2 different things :P | 03:31 |
| jrichli | ah, ok. thanks! | 03:32 |
| mattoliverau | jrichli: https://docs.google.com/document/d/16rPdDrT-4G6hcen_6no8cr8KbxUqbcvsSkOfxxu0EBE/edit# | 03:32 |
| jrichli | great, thx! | 03:32 |
| *** arunman has joined #openstack-swift | 03:39 | |
| charz_ | good morning! | 03:52 |
| *** Jeffrey4l has joined #openstack-swift | 03:54 | |
| *** dmorita has joined #openstack-swift | 03:55 | |
| mattoliverau | charz_: morning | 03:55 |
| charz_ | Guys, if i would like to clean up old .ts file manually without apply the patch from 2.10.0. Can I just find all .ts files and delete them? I'll stop objcect replicator and auditor on the cluster before I delete them. | 03:58 |
| charz_ | mattoliverau: o/ | 03:58 |
| *** jerrygb has quit IRC | 03:59 | |
| *** dmorita has quit IRC | 04:00 | |
| *** ppai has joined #openstack-swift | 04:00 | |
| *** jerrygb has joined #openstack-swift | 04:01 | |
| mattoliverau | charz_: that could be dangerous, the reason we place .ts files is because somewhere else in the cluster the object may not have gotten the delete yet. If you delete all the .ts's and an object still exists then you could be undeleting the object, but it would still potentially be missing from containers creating dark data. | 04:01 |
| *** links has joined #openstack-swift | 04:01 | |
| *** arunman has quit IRC | 04:01 | |
| mattoliverau | what is suppose to happen is after reclaim age the we assume the .ts has definitely been synced around the cluster meaning the old data should | 04:02 |
| mattoliverau | should've been deleted and then is safe to delete them too. | 04:02 |
| mattoliverau | This is why the patch only marks the hashes as dirty if the .ts file is older then reclaim age. | 04:03 |
| mattoliverau | If you searched for .ts files older then reclaim age then they'd probably be safe to delete. Cause otherwise your recliam age isn't big enough and you'd have other issues. | 04:05 |
| charz_ | mattoliverau: okay, you're correct. I forgot it would be some data haven't delete yet around the cluster. | 04:05 |
| charz_ | mattoliverau: it sounds like a good idea to check the timestamp before delete it. | 04:06 |
| charz_ | mattoliverau: thx | 04:08 |
| *** bkopilov has joined #openstack-swift | 04:10 | |
| *** arunman has joined #openstack-swift | 04:21 | |
| *** vinsh has quit IRC | 04:27 | |
| *** jerrygb_ has joined #openstack-swift | 04:28 | |
| *** jerrygb has quit IRC | 04:29 | |
| *** jerrygb has joined #openstack-swift | 04:44 | |
| *** jerrygb_ has quit IRC | 04:45 | |
| *** jerrygb_ has joined #openstack-swift | 04:59 | |
| *** jerrygb has quit IRC | 04:59 | |
| *** jerrygb_ has quit IRC | 05:15 | |
| *** jerrygb has joined #openstack-swift | 05:15 | |
| *** jerrygb has quit IRC | 05:33 | |
| *** jerrygb has joined #openstack-swift | 05:33 | |
| *** jerrygb has quit IRC | 05:33 | |
| *** openstackgerrit has quit IRC | 05:48 | |
| *** openstackgerrit has joined #openstack-swift | 05:48 | |
| *** ppai has quit IRC | 05:53 | |
| *** SkyRocknRoll has joined #openstack-swift | 06:03 | |
| *** klrmn has quit IRC | 06:09 | |
| *** nizam037 has joined #openstack-swift | 06:21 | |
| *** sheel has joined #openstack-swift | 06:24 | |
| *** nizam037 has quit IRC | 06:26 | |
| *** vinsh has joined #openstack-swift | 06:27 | |
| *** vinsh has quit IRC | 06:33 | |
| *** McMurlock1 has joined #openstack-swift | 07:05 | |
| *** ppai has joined #openstack-swift | 07:09 | |
| *** suresh has joined #openstack-swift | 07:13 | |
| *** tesseract has joined #openstack-swift | 07:14 | |
| *** tesseract is now known as Guest25209 | 07:14 | |
| suresh | hii all, I deployed openstack swift with two nodes (one proxy and one storage) and my storage node has one disk with 475GB space | 07:18 |
| suresh | but if i am running three parallel uploads of 5.1GB with 512MB segment size | 07:18 |
| suresh | it is giving 503 service unavialable | 07:19 |
| suresh | please someone help on this is there any parameter i need to set to accept more requests? | 07:20 |
| *** jerrygb has joined #openstack-swift | 07:34 | |
| *** rcernin has joined #openstack-swift | 07:34 | |
| *** jerrygb has quit IRC | 07:39 | |
| *** hseipp has joined #openstack-swift | 07:45 | |
| *** openstackgerrit has quit IRC | 07:48 | |
| *** openstackgerrit has joined #openstack-swift | 07:48 | |
| *** sams-gleb has joined #openstack-swift | 08:19 | |
| *** sams-gleb has quit IRC | 08:19 | |
| *** sams-gleb has joined #openstack-swift | 08:19 | |
| *** jordanP has joined #openstack-swift | 08:21 | |
| *** amoralej|off is now known as amoralej | 08:35 | |
| suresh | hi | 08:35 |
| *** geaaru has joined #openstack-swift | 08:38 | |
| *** rledisez has joined #openstack-swift | 08:48 | |
| *** hseipp has quit IRC | 08:56 | |
| *** ChubYann has quit IRC | 09:07 | |
| *** SkyRocknRoll has quit IRC | 09:29 | |
| *** SkyRocknRoll has joined #openstack-swift | 09:32 | |
| *** jerrygb has joined #openstack-swift | 09:35 | |
| openstackgerrit | Romain LE DISEZ proposed openstack/swift: Fix old-style container-sync and doc https://review.openstack.org/392962 | 09:38 |
| *** jerrygb has quit IRC | 09:41 | |
| *** mvk has quit IRC | 09:51 | |
| *** acoles_ is now known as acoles | 09:56 | |
| *** jerrygb has joined #openstack-swift | 09:58 | |
| *** jerrygb has quit IRC | 10:03 | |
| *** mvk has joined #openstack-swift | 10:26 | |
| *** vinsh has joined #openstack-swift | 10:29 | |
| *** vinsh has quit IRC | 10:34 | |
| openstackgerrit | Gábor Antal proposed openstack/swift: Use more specific asserts in test/unit/common/middleware https://review.openstack.org/342770 | 10:34 |
| *** links has quit IRC | 10:40 | |
| openstackgerrit | Gábor Antal proposed openstack/swift: Use more specific asserts in test/unit/common https://review.openstack.org/342781 | 10:43 |
| *** dmorita has joined #openstack-swift | 10:56 | |
| *** dmorita has quit IRC | 11:00 | |
| *** jerrygb has joined #openstack-swift | 11:26 | |
| acoles | kota_: clayg timburke notmyname mitaka backport to fix bug 1633647 here https://review.openstack.org/#/c/389746 | 11:29 |
| openstack | bug 1633647 in OpenStack Object Storage (swift) "bad fragment data not detected in audit" [Critical,Fix released] https://launchpad.net/bugs/1633647 | 11:29 |
| patchbot | patch 389746 - swift (stable/mitaka) - Make ECDiskFileReader check fragment metadata | 11:29 |
| *** links has joined #openstack-swift | 11:30 | |
| acoles | we (and I guess admin6 and others) would like to have that land | 11:30 |
| kota_ | acole: thanks, will look at tommorow | 11:31 |
| kota_ | acoles:^^ | 11:31 |
| admin6 | acoles: you’re right ;-) | 11:31 |
| acoles | kota_: thanks | 11:31 |
| *** rvasilets___ has joined #openstack-swift | 11:33 | |
| *** suresh has quit IRC | 11:37 | |
| openstackgerrit | Kota Tsuyuzaki proposed openstack/liberasurecode: WIP: ISA-L Cauchy support https://review.openstack.org/393263 | 11:58 |
| *** bkopilov has quit IRC | 12:02 | |
| acoles | kota_: clayg timburke notmyname ...and the newton backport to fix bug 1633647 is here https://review.openstack.org/393265 | 12:06 |
| openstack | bug 1633647 in OpenStack Object Storage (swift) "bad fragment data not detected in audit" [Critical,Fix released] https://launchpad.net/bugs/1633647 | 12:06 |
| patchbot | patch 393265 - swift (stable/newton) - Make ECDiskFileReader check fragment metadata | 12:06 |
| openstackgerrit | Kota Tsuyuzaki proposed openstack/pyeclib: WIP: ISA-L Cauchy support https://review.openstack.org/393276 | 12:19 |
| *** arunman has quit IRC | 12:25 | |
| *** jerrygb has quit IRC | 12:30 | |
| *** daemontool has joined #openstack-swift | 12:37 | |
| *** silor has joined #openstack-swift | 12:46 | |
| *** silor1 has joined #openstack-swift | 12:53 | |
| *** silor has quit IRC | 12:54 | |
| *** silor1 is now known as silor | 12:54 | |
| *** david-lyle has quit IRC | 12:55 | |
| *** silor has quit IRC | 13:02 | |
| *** cbartz has joined #openstack-swift | 13:07 | |
| *** klamath has joined #openstack-swift | 13:10 | |
| *** klamath has quit IRC | 13:10 | |
| *** jerrygb has joined #openstack-swift | 13:10 | |
| *** klamath has joined #openstack-swift | 13:11 | |
| *** silor has joined #openstack-swift | 13:18 | |
| *** jerrygb_ has joined #openstack-swift | 13:21 | |
| *** jerrygb has quit IRC | 13:22 | |
| *** StraubTW has joined #openstack-swift | 13:22 | |
| *** vint_bra has joined #openstack-swift | 13:24 | |
| *** amoralej is now known as amoralej|lunch | 13:25 | |
| *** ppai has quit IRC | 13:36 | |
| *** arunman has joined #openstack-swift | 13:51 | |
| *** links has quit IRC | 13:52 | |
| *** SkyRocknRoll has quit IRC | 13:53 | |
| *** pcaruana has joined #openstack-swift | 13:53 | |
| *** arunman has quit IRC | 13:55 | |
| *** bkopilov has joined #openstack-swift | 13:55 | |
| *** jerrygb has joined #openstack-swift | 13:57 | |
| *** sheel has quit IRC | 14:00 | |
| *** jerrygb_ has quit IRC | 14:00 | |
| pdardeau | suresh: you mentioned that you have a single storage node with a single drive. you're effectively stress testing write throughput of that single drive. | 14:02 |
| pdardeau | suresh: use tools such as iostat and nmon to watch what's happening when you try that out again | 14:02 |
| *** amoralej|lunch is now known as amoralej | 14:03 | |
| *** arunman has joined #openstack-swift | 14:16 | |
| *** vinsh has joined #openstack-swift | 14:18 | |
| *** sams-gleb has quit IRC | 14:20 | |
| *** sams-gleb has joined #openstack-swift | 14:21 | |
| *** sams-gleb has quit IRC | 14:25 | |
| *** bkopilov has quit IRC | 14:28 | |
| *** arunman has quit IRC | 14:28 | |
| notmyname | good morning | 14:36 |
| notmyname | from the ML, looks like keystone is changing the default token type to fernet tokens instead of uuid tokens | 14:36 |
| *** sams-gleb has joined #openstack-swift | 14:38 | |
| *** sams-gleb has quit IRC | 14:40 | |
| *** sams-gleb has joined #openstack-swift | 14:41 | |
| *** sams-gleb has quit IRC | 14:42 | |
| *** sgundur has joined #openstack-swift | 14:45 | |
| stevemar | notmyname: indeed we are | 14:45 |
| stevemar | notmyname: shouldn't impact you folks... | 14:45 |
| stevemar | notmyname: to openstack services it'll look like any other token | 14:46 |
| rledisez | notmyname: i guess migration should be pretty smooth from swift pov, are you expecting any weird behavior in swift? i'm currently working on our move to fernet so it's interesting to see that keystone will set it as default soon | 14:46 |
| stevemar | we've had it since kilo, with people using it in production since liberty, so its about time :D | 14:46 |
| rledisez | stevemar: let's all upgrade! :) thx for the confirmation of no impact expected | 14:51 |
| *** d0ugal has quit IRC | 14:57 | |
| *** arunman has joined #openstack-swift | 14:58 | |
| notmyname | rledisez: yeah. what stevemar said :-) | 15:01 |
| *** sgundur has quit IRC | 15:02 | |
| notmyname | stevemar: I think the only question mark I have is around relative performance. I heard some rumors that fernet tokens were slower. or required network validation for every token. or generated a new token every request, thus removing cachability. but I admit to not knowing details | 15:03 |
| notmyname | acoles: so now maybe not brexit? crazy. at least is seems good for the £ | 15:04 |
| notmyname | rledisez: before you take off for the day, I have a question. what's the EC parameters you're using? | 15:05 |
| acoles | notmyname: it's a significant ruling but not sure it will ultimately change the outcome | 15:08 |
| rledisez | notmyname: | 15:08 |
| rledisez | [storage-policy:1] | 15:08 |
| rledisez | name = Policy-1 | 15:08 |
| rledisez | default = 1 | 15:08 |
| rledisez | policy_type = erasure_coding | 15:08 |
| rledisez | ec_type = jerasure_rs_vand | 15:08 |
| rledisez | ec_num_data_fragments = 12 | 15:08 |
| patchbot | Error: 'supybot.=' is not a valid configuration variable. | 15:08 |
| rledisez | ec_num_parity_fragments = 3 | 15:08 |
| rledisez | ec_object_segment_size = 1048576 | 15:08 |
| rledisez | if i had to change one thing, it would be segment size. put it lower to limit latency and "slow connection" issues | 15:09 |
| notmyname | rledisez: cool. thanks. kota_ is working on an issue he found if you have 5 or parity fragments. nothing for you to worry about there | 15:09 |
| rledisez | but it can't be changed AFAIK, so... | 15:09 |
| notmyname | that's interesting | 15:10 |
| notmyname | yeah, we found that increasing it for workloads with larger objects can really help throughput. makes sense that lowering it could help for reducing issues if connections are slow | 15:11 |
| notmyname | stevemar: are we going to be hit with any cachability issues for fernet tokens? | 15:11 |
| *** jerrygb_ has joined #openstack-swift | 15:12 | |
| *** d0ugal has joined #openstack-swift | 15:13 | |
| *** jerrygb has quit IRC | 15:13 | |
| notmyname | acoles: thanks for the backports. I'll look at them this morning | 15:15 |
| openstackgerrit | Alistair Coles proposed openstack/swift: Fix KeyError when auditor finds an expired object https://review.openstack.org/393369 | 15:27 |
| kota_ | notmyname: + isal for the condition. Jerasure works, fyi. | 15:27 |
| *** sgundur has joined #openstack-swift | 15:28 | |
| notmyname | kota_: ah right. | 15:31 |
| notmyname | kota_: also, shouldn't you be asleep? ;-) | 15:31 |
| kota_ | notmyname: will do soon :-) | 15:32 |
| acoles | kota_ has had a transfusion of clayg blood and now he never needs to sleep ;) | 15:38 |
| *** d0ugal has quit IRC | 15:39 | |
| notmyname | jrichli: tdasilva: did I capture the plan accurately on patch 232162? | 15:46 |
| patchbot | https://review.openstack.org/#/c/232162/ - swift - Symlink implementation. | 15:46 |
| *** sgundur has quit IRC | 15:47 | |
| jrichli | sounds good to me | 15:48 |
| * jrichli puts symlinks higher in priority list | 15:48 | |
| jrichli | I still haven't done a lot of testing with it : so I will focus on getting some time allocated for that over the week | 15:49 |
| *** hseipp has joined #openstack-swift | 15:49 | |
| *** chsc has joined #openstack-swift | 15:56 | |
| *** chsc has joined #openstack-swift | 15:56 | |
| *** rcernin has quit IRC | 16:02 | |
| *** sgundur has joined #openstack-swift | 16:03 | |
| *** chsc has quit IRC | 16:03 | |
| *** sgundur has quit IRC | 16:05 | |
| *** sgundur has joined #openstack-swift | 16:05 | |
| openstackgerrit | Christian Hugo proposed openstack/swift: Raise ValueError if a config section not exists https://review.openstack.org/393388 | 16:08 |
| notmyname | bah! the wiki spam filter doesn't let me use goo.gl shortlinks any more | 16:15 |
| jrichli | interesting. so we have to have a paragraph for a link | 16:18 |
| notmyname | yeah. doesn't let me replace an existing one for the new one. but it's ok with the existing one staying there | 16:19 |
| notmyname | https://goo.gl/WiDneY | 16:19 |
| *** pcaruana has quit IRC | 16:20 | |
| *** dmorita has joined #openstack-swift | 16:20 | |
| *** rcernin has joined #openstack-swift | 16:25 | |
| notmyname | there. http://not.mn/reviews.html | 16:25 |
| *** McMurlock1 has quit IRC | 16:28 | |
| *** bill_az has joined #openstack-swift | 16:29 | |
| notmyname | at some point if I'm using a page on my own site, I could combine the priority reviews wiki and the gerrit dash and have one place to go look at stuff | 16:31 |
| *** bill_az has quit IRC | 16:38 | |
| notmyname | acoles: backports +A'd | 16:48 |
| acoles | notmyname: thanks! | 16:48 |
| *** sgundur has quit IRC | 16:48 | |
| *** chsc has joined #openstack-swift | 17:00 | |
| *** sgundur has joined #openstack-swift | 17:03 | |
| *** rcernin has quit IRC | 17:03 | |
| *** rcernin has joined #openstack-swift | 17:03 | |
| timburke | acoles: that backport looks hairy. lot of changes to TestObjectController in the original that didn't make it into the stable patch? i suppose all of the changes under swift/ look good though... | 17:05 |
| openstackgerrit | Alistair Coles proposed openstack/swift: EC: reconstruct using non-durable fragments https://review.openstack.org/376630 | 17:08 |
| *** rcernin has quit IRC | 17:08 | |
| clayg | form the pep8 jobs now require bindep thread "If your | 17:08 |
| patchbot | Error: No closing quotation | 17:08 |
| clayg | project needs a specific binary to be installed to compile your project" | 17:08 |
| *** jordanP has joined #openstack-swift | 17:08 | |
| *** klrmn has joined #openstack-swift | 17:08 | |
| clayg | oh i guess it means to install a dependency? | 17:09 |
| acoles | timburke: IIRC lots of new tests went into TestObjectController since Mitaka that needed fixing up to use valid EC fragment data | 17:09 |
| clayg | i thought wheels removed the compile requirement? like you could install python-mysql with out mysql-dev or some crazyness? | 17:09 |
| notmyname | clayg: yeah. bindep isn't really just for "binaries needed to compile your project". it's more like "dependencies that aren't on pypi" | 17:10 |
| acoles | timburke: there shouldn't be any tests directly verifying the fix that did not get backported | 17:10 |
| timburke | cool. i hadn't looked too closely, just figured i ought to do my standard backport sniff-test | 17:11 |
| *** cbartz has quit IRC | 17:11 | |
| timburke | notmyname trusts it; why shouldn't i? ;-) | 17:11 |
| acoles | timburke: please, do. it was not a straightforward cherry-pick, but also not as bad as I feared. | 17:12 |
| acoles | tdasilva: I had to fix merge conflicts on this and so it lost your +2 https://review.openstack.org/376630 | 17:13 |
| patchbot | patch 376630 - swift - EC: reconstruct using non-durable fragments | 17:13 |
| acoles | some trivial assert method cleanup created conflicts | 17:14 |
| *** hseipp has quit IRC | 17:23 | |
| *** rledisez has quit IRC | 17:24 | |
| *** daemontool has quit IRC | 17:28 | |
| *** silor has quit IRC | 17:30 | |
| tdasilva | acoles: ack, will look | 17:41 |
| tdasilva | notmyname: your comment on patch 232162 looks good | 17:42 |
| patchbot | https://review.openstack.org/#/c/232162/ - swift - Symlink implementation. | 17:42 |
| notmyname | tdasilva: ack. thanks | 17:42 |
| acoles | clayg: timburke notmyname do you think we should backport this fix to newton? https://review.openstack.org/393369 | 17:51 |
| patchbot | patch 393369 - swift - Fix KeyError when auditor finds an expired object | 17:51 |
| notmyname | acoles: IIRC that was primarily (only?) resulting in a bunch of log spam, right? | 17:53 |
| acoles | notmyname: correct | 17:54 |
| acoles | notmyname: only | 17:54 |
| notmyname | so we expect an upstream release soon. it will be 2.11.0. so the backports we're doing to newton could be tagged as a 2.10.1 (but we've not done tags before and the distro packagers last week said "whatever") | 17:56 |
| *** Guest25209 has quit IRC | 17:57 | |
| *** sgundur has quit IRC | 18:00 | |
| notmyname | honestly, with the various backports we've been doing, I'm thinking that we should take over our own tagging of releases on stable branches. (which probably means us proposing a tag instead of nobody proposing a tag) | 18:00 |
| notmyname | so with this patch, it doesn't seem like a major thing. but it seems scarier than it is (the errors in the logs). so yeah, I'd be fine with backporting it | 18:01 |
| notmyname | (I wouldn't lose any sleep about not backporting it, though) | 18:02 |
| *** geaaru has quit IRC | 18:02 | |
| *** rcernin has joined #openstack-swift | 18:05 | |
| *** sgundur has joined #openstack-swift | 18:07 | |
| notmyname | acoles: what do you think? | 18:07 |
| acoles | notmyname: it's just log noise, but on other hand it's noise that could result in "support" requests (here in irc, launchpad, elsewhere) so given that the patch is relatively small and should be trivial to backport...I'd vote to do it. But I'm not sure I know what a normal "threshold" would be to warrant a backport? | 18:14 |
| notmyname | acoles: I think the threshold is whatever we the maintainers want to deal with | 18:15 |
| notmyname | so yeah...small patch, makes things clearer for users? do it. big patch? maybe not | 18:15 |
| acoles | k. I'll propose a backport once it lands (tomorrow) and others can vote if they disagree on the worthiness | 18:17 |
| clayg | notmyname: I like this bit about us taking over tagging backports - sounds like we get more control and you're the only one that has to do more work | 18:20 |
| notmyname | clayg: I've never known you to be opposed to me doing more work ;-) | 18:21 |
| notmyname | (that's a joke) | 18:22 |
| acoles | timburke: why do we need this condition https://review.openstack.org/#/c/347538/9/swift/common/middleware/slo.py@622 ? I'm not saying we don't, just trying to grok - isn't content-length already 0 for a 304 or 412? | 18:32 |
| patchbot | patch 347538 - swift - Reduce backend requests for SLO If-Match / HEAD re... | 18:32 |
| *** mvk has quit IRC | 18:32 | |
| openstackgerrit | Merged openstack/swift: Fix KeyError when auditor finds an expired object https://review.openstack.org/393369 | 18:33 |
| timburke | acoles: we may not need it; it's been a while since i wrote it | 18:36 |
| openstackgerrit | Tim Burke proposed openstack/swift: Reduce backend requests for SLO If-Match / HEAD requests https://review.openstack.org/347538 | 18:38 |
| acoles | timburke: heh. I know that feeling - been so long you lost the context. | 18:38 |
| timburke | unit tests pass fine without it, so... gone! | 18:38 |
| acoles | timburke: but, but...you must have had a reason :) | 18:40 |
| timburke | "it seemed expedient at the time" | 18:40 |
| *** asettle has joined #openstack-swift | 18:41 | |
| acoles | timburke: oh it really has gone. ok, I'll continue on patchset 10 tomorrow. | 18:43 |
| *** acoles is now known as acoles_ | 18:45 | |
| timburke | seemed better than forever wonder "now why did we do that, again?" | 18:45 |
| *** _JZ_ has joined #openstack-swift | 18:49 | |
| *** jordanP has quit IRC | 18:52 | |
| timburke | acoles_: notmyname: fyi, backport is up as https://review.openstack.org/#/c/393444/ | 18:55 |
| patchbot | patch 393444 - swift (stable/newton) - Fix KeyError when auditor finds an expired object | 18:55 |
| notmyname | timburke: thanks. I'll let it sit just a bit to see if someone wants to comment | 18:56 |
| *** sgundur has quit IRC | 18:56 | |
| notmyname | timburke: looks like a clean cherry-pick | 18:56 |
| timburke | yup | 18:57 |
| notmyname | cool | 18:57 |
| *** sgundur has joined #openstack-swift | 18:58 | |
| notmyname | FYI I'm paying attention to http://lists.openstack.org/pipermail/openstack-dev/2016-November/106680.html (specifically the "hash ring library" part) | 18:58 |
| notmyname | I'm planning on responding based on how the thread goes. but not yet | 18:59 |
| *** asettle has quit IRC | 18:59 | |
| *** sgundur has quit IRC | 19:03 | |
| *** sgundur has joined #openstack-swift | 19:03 | |
| *** ChubYann has joined #openstack-swift | 19:10 | |
| *** bill_az has joined #openstack-swift | 19:14 | |
| *** sgundur has quit IRC | 19:16 | |
| *** bill_az has quit IRC | 19:25 | |
| *** iurygregory has joined #openstack-swift | 19:31 | |
| *** amoralej is now known as amoralej|off | 19:35 | |
| mattoliverau | Morning | 19:43 |
| timburke | mattoliverau: either you're still getting up early, or my day is slipping away far too quickly | 19:48 |
| mattoliverau | timburke: it's ok, it's early :) unless you were really wanting the day to end, in which case I'm sorry ;p | 19:50 |
| *** jerrygb has joined #openstack-swift | 19:51 | |
| *** jerrygb_ has quit IRC | 19:54 | |
| *** rvasilets___ has quit IRC | 19:58 | |
| *** mvk has joined #openstack-swift | 20:14 | |
| clayg | +1 this day is going to fast and also sucks | 20:22 |
| *** silor has joined #openstack-swift | 20:23 | |
| *** StraubTW has quit IRC | 20:30 | |
| *** gyee has joined #openstack-swift | 20:32 | |
| *** silor has quit IRC | 20:33 | |
| *** silor1 has joined #openstack-swift | 20:33 | |
| *** silor1 is now known as silor | 20:35 | |
| *** rcernin has quit IRC | 20:57 | |
| *** bill_az has joined #openstack-swift | 21:24 | |
| *** Jeffrey4l has quit IRC | 21:34 | |
| *** Jeffrey4l has joined #openstack-swift | 21:35 | |
| *** bill_az has quit IRC | 21:39 | |
| *** lcurtis has joined #openstack-swift | 21:51 | |
| *** david-lyle has joined #openstack-swift | 22:03 | |
| *** _JZ_ has quit IRC | 22:07 | |
| *** silor has quit IRC | 22:08 | |
| *** jerrygb_ has joined #openstack-swift | 22:29 | |
| *** jerrygb has quit IRC | 22:32 | |
| *** lcurtis has quit IRC | 22:35 | |
| *** klamath has quit IRC | 22:55 | |
| clayg | whoa... is it still this same day? | 22:57 |
| notmyname | still for another 8 hours | 23:02 |
| clayg | i bet this crap day is over in bcn already | 23:12 |
| clayg | Catalans knew | 23:12 |
| timburke | clayg: eh, you'd still be up right now. it's barely past 1am there | 23:13 |
| mattoliverau | clayg: if it helps tomorrow is much better | 23:19 |
| clayg | knew it! | 23:25 |
| *** chsc has quit IRC | 23:32 | |
| *** kei_yama has joined #openstack-swift | 23:36 | |
| *** gyee has quit IRC | 23:49 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!