*** m_kazuhiro has joined #openstack-swift | 00:00 | |
*** garthb has quit IRC | 00:10 | |
*** hrou has joined #openstack-swift | 00:15 | |
*** 77CAATUOY has joined #openstack-swift | 00:22 | |
*** 18VAAO48R has joined #openstack-swift | 00:22 | |
*** CR7 has quit IRC | 00:26 | |
*** jkugel has joined #openstack-swift | 00:33 | |
*** annegentle has joined #openstack-swift | 00:34 | |
*** darrenc is now known as darrenc_afk | 00:37 | |
*** david-lyle has quit IRC | 00:40 | |
*** david-lyle has joined #openstack-swift | 00:44 | |
*** kota_ has joined #openstack-swift | 00:45 | |
*** ChanServ sets mode: +v kota_ | 00:45 | |
kota_ | good morning | 00:45 |
---|---|---|
clayg | kota_: hey I was working on your patch! hope that's ok :\ | 00:51 |
kota_ | clayg: thanks! I noticed that and will look at in today ;) | 00:52 |
*** dipe has quit IRC | 00:54 | |
*** darrenc_afk is now known as darrenc | 01:00 | |
*** 18VAAO48R has quit IRC | 01:03 | |
*** 77CAATUOY has quit IRC | 01:03 | |
mattoliverau | kota_: morning | 01:10 |
*** annegentle has quit IRC | 01:10 | |
kota_ | mattoliverau: morning! | 01:12 |
*** dmorita has quit IRC | 01:18 | |
*** jrichli has joined #openstack-swift | 01:24 | |
*** dmorita has joined #openstack-swift | 01:26 | |
*** rohit_ has quit IRC | 01:50 | |
*** haomaiwang has joined #openstack-swift | 01:56 | |
*** jkugel has quit IRC | 02:00 | |
*** haomaiwang has quit IRC | 02:01 | |
*** haomaiwa_ has joined #openstack-swift | 02:05 | |
*** haomaiwa_ has quit IRC | 02:05 | |
*** haomaiwa_ has joined #openstack-swift | 02:06 | |
*** baojg has joined #openstack-swift | 02:17 | |
*** baojg has quit IRC | 02:20 | |
*** baojg has joined #openstack-swift | 02:22 | |
*** gyee has quit IRC | 02:30 | |
*** esker has joined #openstack-swift | 02:44 | |
*** haomaiwa_ has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-swift | 03:01 | |
*** DericHorn-HP has joined #openstack-swift | 03:12 | |
*** esker has quit IRC | 03:19 | |
*** bill_az has quit IRC | 03:34 | |
*** tongli has joined #openstack-swift | 03:35 | |
*** garthb has joined #openstack-swift | 03:36 | |
*** garthb has quit IRC | 03:42 | |
*** trifon has quit IRC | 03:50 | |
*** haomaiwang has quit IRC | 04:01 | |
*** haomaiwang has joined #openstack-swift | 04:01 | |
*** ktsuyuzaki has joined #openstack-swift | 04:03 | |
*** kota_ has quit IRC | 04:04 | |
*** kota_ has joined #openstack-swift | 04:08 | |
*** ChanServ sets mode: +v kota_ | 04:08 | |
*** ktsuyuzaki has quit IRC | 04:09 | |
*** jrichli has quit IRC | 04:10 | |
*** ktsuyuzaki has joined #openstack-swift | 04:12 | |
*** kota_ has quit IRC | 04:14 | |
*** kota_ has joined #openstack-swift | 04:17 | |
*** ChanServ sets mode: +v kota_ | 04:17 | |
*** ktsuyuzaki has quit IRC | 04:19 | |
*** ktsuyuzaki has joined #openstack-swift | 04:19 | |
*** kota_ has quit IRC | 04:21 | |
*** kota_ has joined #openstack-swift | 04:22 | |
*** ChanServ sets mode: +v kota_ | 04:22 | |
*** ktsuyuzaki has quit IRC | 04:24 | |
*** ktsuyuzaki has joined #openstack-swift | 04:24 | |
*** kota_ has quit IRC | 04:26 | |
*** kota_ has joined #openstack-swift | 04:27 | |
*** ChanServ sets mode: +v kota_ | 04:27 | |
*** ktsuyuzaki has quit IRC | 04:29 | |
*** ktsuyuzaki has joined #openstack-swift | 04:29 | |
*** kota_ has quit IRC | 04:31 | |
*** kota_ has joined #openstack-swift | 04:34 | |
*** ChanServ sets mode: +v kota_ | 04:34 | |
*** ktsuyuzaki has quit IRC | 04:35 | |
*** kota_ has quit IRC | 04:36 | |
*** tongli has quit IRC | 04:42 | |
*** trifon has joined #openstack-swift | 04:48 | |
briancline | ugh, keystoneclient somehow broke openstackclient in the last few days | 04:49 |
*** m_kazuhiro has quit IRC | 04:54 | |
*** haomaiwang has quit IRC | 05:01 | |
*** haomaiwang has joined #openstack-swift | 05:01 | |
briancline | os-client-config rather | 05:09 |
*** m_kazuhiro has joined #openstack-swift | 05:21 | |
*** SkyRocknRoll has joined #openstack-swift | 05:28 | |
*** nakagawamsa has joined #openstack-swift | 05:30 | |
*** openstackgerrit has quit IRC | 05:31 | |
*** openstackgerrit has joined #openstack-swift | 05:31 | |
*** DericHorn-HP has quit IRC | 05:37 | |
*** m_kazuhiro has quit IRC | 05:46 | |
*** hrou has quit IRC | 05:56 | |
*** haomaiwang has quit IRC | 06:01 | |
*** haomaiwang has joined #openstack-swift | 06:01 | |
*** baojg has quit IRC | 06:12 | |
*** baojg has joined #openstack-swift | 06:19 | |
*** jroll has quit IRC | 06:27 | |
*** DericHorn-HP has joined #openstack-swift | 06:28 | |
*** DericHorn-HP has quit IRC | 06:31 | |
*** jroll has joined #openstack-swift | 06:33 | |
*** baojg has quit IRC | 06:33 | |
*** DericHorn-HP has joined #openstack-swift | 06:36 | |
*** baojg has joined #openstack-swift | 06:39 | |
*** mac_ified has quit IRC | 06:48 | |
*** DericHorn-HP has quit IRC | 06:48 | |
*** geaaru has joined #openstack-swift | 06:56 | |
*** kota_ has joined #openstack-swift | 06:56 | |
*** ChanServ sets mode: +v kota_ | 06:56 | |
*** haomaiwang has quit IRC | 07:01 | |
*** haomaiwang has joined #openstack-swift | 07:01 | |
*** rledisez has joined #openstack-swift | 07:04 | |
*** DericHorn-HP has joined #openstack-swift | 07:09 | |
*** rledisez has quit IRC | 07:17 | |
*** DericHorn-HP has quit IRC | 07:19 | |
*** rledisez has joined #openstack-swift | 07:20 | |
*** HenryG has quit IRC | 07:21 | |
*** acoles_ is now known as acoles | 07:27 | |
*** esker has joined #openstack-swift | 07:27 | |
*** esker has quit IRC | 07:28 | |
*** esker has joined #openstack-swift | 07:28 | |
*** aix has joined #openstack-swift | 07:42 | |
*** jst_ has joined #openstack-swift | 07:56 | |
*** resker has joined #openstack-swift | 07:57 | |
*** esker has quit IRC | 08:00 | |
*** haomaiwang has quit IRC | 08:01 | |
*** haomaiwang has joined #openstack-swift | 08:01 | |
*** baojg has quit IRC | 08:01 | |
*** dmorita has quit IRC | 08:04 | |
*** T0m_ has joined #openstack-swift | 08:05 | |
*** T0m_ has left #openstack-swift | 08:05 | |
*** jistr has joined #openstack-swift | 08:07 | |
*** baojg has joined #openstack-swift | 08:08 | |
*** jordanP has joined #openstack-swift | 08:09 | |
*** HenryG has joined #openstack-swift | 08:12 | |
*** aix has quit IRC | 08:13 | |
*** HenryG has quit IRC | 08:16 | |
*** baojg has quit IRC | 08:17 | |
*** baojg has joined #openstack-swift | 08:18 | |
*** aix has joined #openstack-swift | 08:27 | |
*** silor has joined #openstack-swift | 08:29 | |
*** HenryG has joined #openstack-swift | 08:30 | |
openstackgerrit | Christopher Bartz proposed openstack/swift-specs: formpost should allow subprefix-based signature https://review.openstack.org/225059 | 08:34 |
*** ho has quit IRC | 08:46 | |
*** chlong has quit IRC | 08:47 | |
*** haomaiwang has quit IRC | 09:01 | |
*** haomaiwang has joined #openstack-swift | 09:01 | |
*** nakagawamsa has quit IRC | 09:16 | |
*** hezhiqiang has quit IRC | 09:28 | |
*** haomaiwang has quit IRC | 10:01 | |
*** haomaiwang has joined #openstack-swift | 10:01 | |
*** SkyRocknRoll has quit IRC | 10:21 | |
*** km has quit IRC | 10:38 | |
*** baojg has quit IRC | 10:45 | |
*** baojg has joined #openstack-swift | 10:46 | |
*** baojg has quit IRC | 10:50 | |
*** flwang has quit IRC | 10:53 | |
*** mfalatic has quit IRC | 11:00 | |
*** SkyRocknRoll has joined #openstack-swift | 11:00 | |
*** haomaiwang has quit IRC | 11:01 | |
*** haomaiwang has joined #openstack-swift | 11:01 | |
*** mfalatic has joined #openstack-swift | 11:06 | |
*** kei_yama has quit IRC | 11:08 | |
*** flwang has joined #openstack-swift | 11:10 | |
*** hezhiqiang has joined #openstack-swift | 11:26 | |
openstackgerrit | Christopher Bartz proposed openstack/swift-specs: formpost should allow subprefix-based signature https://review.openstack.org/225059 | 11:38 |
*** bill_az has joined #openstack-swift | 11:42 | |
*** cdelatte has joined #openstack-swift | 11:48 | |
*** haomaiwang has quit IRC | 12:01 | |
*** haomaiwang has joined #openstack-swift | 12:01 | |
*** chlong has joined #openstack-swift | 12:06 | |
*** chlong has quit IRC | 12:07 | |
*** chlong has joined #openstack-swift | 12:07 | |
*** jkugel has joined #openstack-swift | 12:10 | |
*** haomaiwang has quit IRC | 12:19 | |
*** kota_ has quit IRC | 12:24 | |
*** janonymous has joined #openstack-swift | 12:31 | |
*** SkyRocknRoll_ has joined #openstack-swift | 12:50 | |
*** fifieldt has quit IRC | 13:00 | |
*** janonymous has quit IRC | 13:03 | |
*** kutija has quit IRC | 13:05 | |
*** hrou has joined #openstack-swift | 13:09 | |
*** haomaiwang has joined #openstack-swift | 13:13 | |
*** amoturi has joined #openstack-swift | 13:15 | |
*** haomaiwang has quit IRC | 13:15 | |
*** dustins has joined #openstack-swift | 13:16 | |
*** haomaiwang has joined #openstack-swift | 13:16 | |
*** pberis has joined #openstack-swift | 13:25 | |
*** wbhuber has joined #openstack-swift | 13:25 | |
*** nadeem has joined #openstack-swift | 13:32 | |
*** nadeem has quit IRC | 13:32 | |
*** nadeem has joined #openstack-swift | 13:33 | |
*** SkyRocknRoll has quit IRC | 13:41 | |
*** SkyRocknRoll_ has quit IRC | 13:42 | |
*** jrichli has joined #openstack-swift | 13:42 | |
*** openstackgerrit has quit IRC | 13:46 | |
*** openstackgerrit has joined #openstack-swift | 13:46 | |
*** tongli has joined #openstack-swift | 13:58 | |
*** haomaiwang has quit IRC | 14:01 | |
*** 77CAAT0IJ has joined #openstack-swift | 14:01 | |
*** SkyRocknRoll has joined #openstack-swift | 14:15 | |
*** SkyRocknRoll_ has joined #openstack-swift | 14:15 | |
*** SkyRocknRoll has quit IRC | 14:18 | |
*** SkyRocknRoll has joined #openstack-swift | 14:19 | |
*** baojg has joined #openstack-swift | 14:30 | |
*** jroll is now known as jroll|dupe | 14:31 | |
*** jlhinson has joined #openstack-swift | 14:31 | |
*** jroll|dupe is now known as brickednick | 14:31 | |
*** david-lyle has quit IRC | 14:31 | |
*** brickednick is now known as jroll | 14:31 | |
*** baojg has quit IRC | 14:45 | |
*** minwoob has joined #openstack-swift | 14:54 | |
*** wbhuber has quit IRC | 14:55 | |
*** wbhuber has joined #openstack-swift | 14:57 | |
*** wbhuber_ has joined #openstack-swift | 15:00 | |
*** 77CAAT0IJ has quit IRC | 15:01 | |
*** haomaiwang has joined #openstack-swift | 15:01 | |
*** jistr is now known as jistr|call | 15:02 | |
*** wbhuber has quit IRC | 15:04 | |
*** mac_ified has joined #openstack-swift | 15:10 | |
*** resker has quit IRC | 15:14 | |
*** esker has joined #openstack-swift | 15:14 | |
*** trifon has quit IRC | 15:15 | |
*** lpabon has joined #openstack-swift | 15:20 | |
*** pberis has quit IRC | 15:26 | |
*** DericHorn-HP has joined #openstack-swift | 15:27 | |
*** bill_az has quit IRC | 15:28 | |
*** janonymous has joined #openstack-swift | 15:28 | |
*** chsc has joined #openstack-swift | 15:37 | |
*** gyee has joined #openstack-swift | 15:42 | |
*** pberis has joined #openstack-swift | 15:47 | |
*** haomaiwang has quit IRC | 16:01 | |
*** haomaiwang has joined #openstack-swift | 16:01 | |
peluse | acoles, you there? | 16:04 |
acoles | peluse: yup | 16:04 |
peluse | its Fri night right? :) | 16:04 |
acoles | yup | 16:05 |
acoles | getting close anyway | 16:05 |
peluse | so hey, where's the hot patch on Ec in the alt frag chain / opt GET stuff right now? I can go through and try to figure out where to jump back in but figured I'd ask instead... | 16:05 |
acoles | peluse: patch 215276 is where i am working on optimistic gets | 16:06 |
patchbot | acoles: https://review.openstack.org/#/c/215276/ | 16:06 |
acoles | peluse: what is there kinda works but i'd like to improve it (see TODO's inline) | 16:07 |
*** jordanP has quit IRC | 16:08 | |
acoles | peluse: i think it will be more efficient if the fallback GETS (ones that require durable) are targetted to nodes where we have learnt that there is a durable. | 16:08 |
peluse | OK, so it looks like we scrapped alt_frags for now and extracted the dict stuff from it as its own to patch to move forward on opt GET? | 16:08 |
peluse | oh I see, even alt frags is dependent on that one now... | 16:09 |
peluse | OK, I'll go review that one next, looks like clay is already god with it. cool | 16:10 |
peluse | good | 16:10 |
peluse | clayg, don't let that typo go to your head :) | 16:10 |
acoles | peluse: yeah, i didnt want optimistics to get trapped dependeing on alt frags but wanted to share the get_ondisk_files change | 16:10 |
peluse | acoles, makes sense | 16:10 |
*** haomaiwang has quit IRC | 16:11 | |
janonymous | Hi | 16:11 |
janonymous | I think i have seen this issue before also but couldn't remember : https://bugs.launchpad.net/bugs/1497256 | 16:11 |
openstack | Launchpad bug 1497256 in swift-bench "SWIFT unit tests failing giving the error (liberasurecode[1907]: liberasurecode_backend_open: dynamic linking error libJerasure.so: cannot open shared object file: No such file or directory)" [Undecided,New] | 16:11 |
acoles | peluse: cool if you can review that patch 222706 that'd be great | 16:11 |
patchbot | acoles: https://review.openstack.org/#/c/222706/ | 16:11 |
peluse | janonymous, yeah, something didn't go right with the install of pyeclib | 16:12 |
peluse | acoles, on it | 16:12 |
peluse | janonymous, try just reinstalling pyeclib | 16:13 |
*** janonymous_ has joined #openstack-swift | 16:13 | |
janonymous_ | peluse: Is it wrong version in requirements .. ? | 16:13 |
acoles | peluse: i spent today hacking patch 222706 some more. i want to make it much smarter about going after durables when the optimistic get fails to get one i.e. when backend returns a non-durable it also tells proxy if it has an older durable. | 16:15 |
patchbot | acoles: https://review.openstack.org/#/c/222706/ | 16:15 |
*** janonymous has quit IRC | 16:15 | |
*** garthb has joined #openstack-swift | 16:15 | |
acoles | peluse: i think its all gonna end up colliding back into the alt frags thing - 'alt' becomes another frag at same timestamp or another frag thats older but durable. | 16:16 |
peluse | acoles, a good reason to do the opt get stuff first! | 16:17 |
peluse | janonymous_, well some things are in flux there so install the version that's in requirements.txt | 16:17 |
*** mfalatic has quit IRC | 16:17 | |
acoles | peluse: clayg so patch 222706 is WIP for now | 16:18 |
patchbot | acoles: https://review.openstack.org/#/c/222706/ | 16:18 |
janonymous_ | peluse: Ohhkay, so the fix of the bug would be to install the correct version | 16:20 |
acoles | peluse: another good one to review would be clayg's patch 218023 - jenkins has a -1 but i suspect it just needs a rebase and i had a +2 on it | 16:20 |
patchbot | acoles: https://review.openstack.org/#/c/218023/ | 16:20 |
*** mfalatic has joined #openstack-swift | 16:20 | |
*** DericHorn-HP has quit IRC | 16:20 | |
peluse | janonymous_, I can't say that as I don't know what went wrong with your install :) tsg and/or keving are working on the getting the newest in there and there were some problems with versioning (downrev support) which is why its in flux. The new version has all sorts of little fixes in there so maybe... | 16:21 |
peluse | acoles, cool. will put that next on my list and maybe get 2 things done today! | 16:21 |
janonymous_ | janonymous: Ohh cool. thanks | 16:22 |
janonymous_ | *peluse | 16:22 |
*** mfalatic_ has joined #openstack-swift | 16:26 | |
*** jistr|call is now known as jistr | 16:28 | |
janonymous_ | acoles , peluse : could you answer a small question: Does this have a impact on swift also https://bugs.launchpad.net/nova/+bug/1361360 | 16:29 |
openstack | Launchpad bug 1361360 in Sahara "Eventlet green threads not released back to the pool leading to choking of new requests" [Medium,Confirmed] - Assigned to Xurong Yang (idopra) | 16:29 |
*** jistr has quit IRC | 16:30 | |
*** jordanP has joined #openstack-swift | 16:30 | |
*** mfalatic has quit IRC | 16:30 | |
*** jordanP has quit IRC | 16:30 | |
*** mfalatic_ is now known as mfalatic | 16:37 | |
*** hrou has quit IRC | 16:42 | |
*** hrou has joined #openstack-swift | 16:42 | |
*** esker has quit IRC | 16:51 | |
*** rledisez has quit IRC | 17:05 | |
*** janonymous_ has quit IRC | 17:13 | |
*** DericHorn-HP has joined #openstack-swift | 17:14 | |
*** zhill has joined #openstack-swift | 17:14 | |
*** acoles is now known as acoles_ | 17:15 | |
*** jrichli has quit IRC | 17:25 | |
*** jrichli has joined #openstack-swift | 17:27 | |
*** xnox has quit IRC | 17:28 | |
*** rohit_ has joined #openstack-swift | 17:32 | |
*** xnox has joined #openstack-swift | 17:33 | |
*** DericHorn-HP has quit IRC | 17:38 | |
*** DericHorn-HP has joined #openstack-swift | 17:51 | |
*** pberis has quit IRC | 17:51 | |
*** barker has joined #openstack-swift | 17:55 | |
*** CaioBrentano has joined #openstack-swift | 18:00 | |
CaioBrentano | Swift gurus, a simple question: Is it possible to create a "write-only" user ? A user that can't delete anything | 18:00 |
CaioBrentano | (I'm using Keystone authentication) | 18:01 |
clayg | heyoh! | 18:03 |
*** geaaru has quit IRC | 18:04 | |
*** SkyRocknRoll has quit IRC | 18:04 | |
*** pberis has joined #openstack-swift | 18:05 | |
clayg | peluse: acoles_: nice work! | 18:05 |
*** barker has quit IRC | 18:08 | |
clayg | ooops - just realized lp bug #1496205 was not tagged ec | 18:11 |
openstack | Launchpad bug 1496205 in OpenStack Object Storage (swift) "EC: Client Disconnect leaves inaccessible data on disk" [Undecided,New] https://launchpad.net/bugs/1496205 | 18:11 |
clayg | notmyname: peluse: acoles_: torgomatic: wbhuber_: minwoob: I could really use a confirm on that one - and maybe some help with priority - I think it's critical maybe? | 18:11 |
clayg | also I'm very close to a fix | 18:11 |
*** eranrom has joined #openstack-swift | 18:13 | |
*** wbhuber_ is now known as wbhuber | 18:20 | |
*** lyrrad has quit IRC | 18:23 | |
*** lyrrad has joined #openstack-swift | 18:26 | |
*** mragupat has joined #openstack-swift | 18:31 | |
*** annegentle has joined #openstack-swift | 18:39 | |
*** SkyRocknRoll_ has quit IRC | 18:42 | |
wbhuber | clayg: looks like test_server.py:TestObjectDisconnectCleanup also failed on my cluster - http://paste.openstack.org/show/468301/ | 18:44 |
wbhuber | clayg: evidently, i was using EC policy for this case. | 18:45 |
*** hezhiqiang has quit IRC | 18:45 | |
*** trifon has joined #openstack-swift | 18:47 | |
*** SkyRocknRoll_ has joined #openstack-swift | 18:57 | |
clayg | paste.openstack.org is so slow :'( | 19:00 |
clayg | yeah I think the test/patch I associated with that bug report isn't totally perfect - I think it does ok in isolation - but in my latest rev it's a little more tidy | 19:00 |
clayg | wbhuber: but yeah - looks like we have a pretty good bug | 19:01 |
clayg | the shell script also demonstrates the issue pretty well I think | 19:01 |
minwoob | clayg: Where did you run your python script that tests the disconnect? | 19:05 |
minwoob | I'm trying to verify it that way. | 19:05 |
clayg | ummm just like on my saio | 19:05 |
*** pberis has quit IRC | 19:09 | |
*** vinsh_ has joined #openstack-swift | 19:18 | |
minwoob | clayg: Confirmed! | 19:19 |
minwoob | clayg: I'll go ahead and post a fixup for the script. | 19:19 |
*** vinsh has quit IRC | 19:21 | |
minwoob | clayg: It was the part about setting the policy to ec42 rather than just ec since it wasn't initially there. | 19:21 |
*** vinsh_ has quit IRC | 19:24 | |
*** vinsh has joined #openstack-swift | 19:24 | |
clayg | minwoob: ahh... yeah the policy name wasn't really paramaterized outside of the script - didn't think about that - whoops! | 19:24 |
*** annegentle has quit IRC | 19:25 | |
wbhuber | minwoob: clayg: it's quite possible that clayg revised to ec from ec42 and the script finished fine for him | 19:30 |
clayg | wbhuber: i mean the name of the policy is configurable - omm it's just 'ec' | 19:32 |
*** haypo has left #openstack-swift | 19:38 | |
minwoob | Meh. Maybe it wasn't necessary to post a follow-up, but just did so anyway. | 19:48 |
*** DericHorn-HP has quit IRC | 19:49 | |
*** annegentle has joined #openstack-swift | 19:55 | |
*** CaioBrentano has quit IRC | 19:58 | |
*** CaioBren_ has joined #openstack-swift | 20:00 | |
*** cdelatte has quit IRC | 20:11 | |
*** CaioBren_ has quit IRC | 20:13 | |
*** silor has quit IRC | 20:27 | |
notmyname | hello, everyone | 20:32 |
*** CaioBrentano has joined #openstack-swift | 20:34 | |
eranrom | Greetings. Is it me or is there an issue with './.unittests common/middleware/test_recon.py' | 20:34 |
wbhuber | clayg: IMHO, re: priority on the bug for which you're closely coming up with a fix, i d value it as high or higher. i see this case as an intermittent issue that occurs every now and then but those release blockers as critical are the ones that our eyes shd peel upon | 20:35 |
wbhuber | notmyname: its been eons i've seen u blip a message on IRC | 20:36 |
wbhuber | eranrom: let me try | 20:36 |
notmyname | wbhuber: I've been traveling all week. just got back home | 20:36 |
wbhuber | notmyname: how was D.C.? | 20:36 |
notmyname | I got to hang out with tdasilva and portante last night in Boston | 20:36 |
notmyname | wbhuber: great. I was in DC, NYC, Boston, and points in between | 20:37 |
eranrom | wbhuber, Thanks! | 20:37 |
wbhuber | i am enivous. ;-) | 20:37 |
openstackgerrit | Minwoo Bae proposed openstack/swift: Reconstructor logging to omit 404 warnings https://review.openstack.org/221956 | 20:37 |
notmyname | wbhuber: I hd about 3.75 hours of sleep last night and a 6 hour flight today. don't be too envious ;-) | 20:37 |
wbhuber | notmyname: hope u've a 64-oz jug of coffee handy.... | 20:38 |
notmyname | oh yeah. coffee. I was going to go make some | 20:38 |
wbhuber | imitating clayg: MOAR coffee :) | 20:39 |
wbhuber | eranrom: ran 59 tests clean for test_recon.py | 20:39 |
eranrom | wbhuber, ok thanks! will start digging. | 20:40 |
wbhuber | eranrom: assumedly, u're not running it on master? | 20:42 |
*** amoturi has left #openstack-swift | 20:42 | |
eranrom | wbhuber: I am on master | 20:42 |
*** hezhiqiang has joined #openstack-swift | 20:46 | |
*** trifon has quit IRC | 20:48 | |
minwoob | ctennis: Are you sure you haven't seen any 404s being reported by the replicator? | 20:49 |
clayg | mmmmmmmm KAUPHY! | 20:50 |
wbhuber | eranrom: are you seeing "ImportError" for test_recon? | 20:50 |
wbhuber | eranrom: this patch is likely causing what you're seeing: https://review.openstack.org/#/c/219617/ | 20:52 |
clayg | wait - so is everyone's not failing on master now? test_get_ring_md5_failed_ring_hash_recovers_without_restart | 20:52 |
clayg | I mean I know it *can* pass (like in the gate where there's only one policy) | 20:52 |
clayg | but now that the rings are enumerated based on policy config on my saio with two policies I get a "items in first list are not in the second" | 20:53 |
clayg | and this seems totally reasonable to me | 20:53 |
clayg | ... master must be broken for everyone running >1 policy | 20:53 |
*** hezhiqiang has quit IRC | 20:53 | |
wbhuber | clayg: that sounds right - i've at least several policies running in SAIO. test_get_ring_md5_ioerror_produces_none_hash also fails along with the test you mentioned | 20:55 |
wbhuber | clayg: going to look at the test to see if i can make it smarter to handle a policy at a time | 20:55 |
openstackgerrit | Clay Gerrard proposed openstack/swift: Fix recon tests on SAIO with multiple policies https://review.openstack.org/225329 | 20:58 |
clayg | ^ that 'ought do it | 20:58 |
wbhuber | clayg: u speedy rat :) | 20:58 |
clayg | briancline: FTFY ;) | 20:58 |
clayg | briancline: note I'm just poking fun - i reviewed it - it's mostly mud on ME | 20:59 |
eranrom | clayg, I wish I could just +4 this one | 20:59 |
clayg | LOL | 20:59 |
wbhuber | eranrom: +A! | 20:59 |
wbhuber | clayg: all clean on my SAIO w/ ur patch. | 21:00 |
notmyname | so what have I missed this week? | 21:02 |
*** dustins has quit IRC | 21:04 | |
notmyname | clayg: wbhuber: what is patch 225329 fixing? how do I break it? | 21:07 |
patchbot | notmyname: https://review.openstack.org/#/c/225329/ | 21:07 |
notmyname | oh. so master fails for me already. wonder when that started | 21:13 |
*** chlong has quit IRC | 21:14 | |
torgomatic | notmyname: commit f4e6d39, Wed Sep 16 17:02:08 2015 +0000 | 21:15 |
notmyname | thanks | 21:15 |
notmyname | patch approved | 21:15 |
openstackgerrit | Samuel Merritt proposed openstack/swift: Let operators add watchers to object audit https://review.openstack.org/212824 | 21:17 |
briancline | uh oh, I found all-new ways to break something? | 21:20 |
*** lpabon has quit IRC | 21:26 | |
*** DericHorn-HP has joined #openstack-swift | 21:27 | |
briancline | clayg: I don't quite understand the patch description -- which tests were failing? | 21:28 |
*** DericHorn-HP has quit IRC | 21:28 | |
*** DericHorn-HP has joined #openstack-swift | 21:29 | |
notmyname | test_get_ring_md5_failed_ring_hash_recovers_without_restart | 21:31 |
wbhuber | briancline: before clayg's patch, failure logs: http://paste.openstack.org/show/468509/ | 21:33 |
openstackgerrit | Eran Rom proposed openstack/swift: Add process level concurrency to container sync https://review.openstack.org/210099 | 21:33 |
openstackgerrit | Eran Rom proposed openstack/swift: Container-Sync to iterate only over synced containers https://review.openstack.org/205803 | 21:33 |
openstackgerrit | Eran Rom proposed openstack/swift: Add thread level concurrency to container sync https://review.openstack.org/225338 | 21:33 |
*** annegentle has quit IRC | 21:34 | |
*** CaioBrentano has quit IRC | 21:35 | |
briancline | wbhuber: oh, weird. so I get that it's because of the presence of other policy rings, but what's different about the dev test environment vs. the gate environment that they would have different results in the tmp dir? | 21:36 |
wbhuber | briancline: gate only runs on one policy so your tests passed | 21:37 |
notmyname | ah ok | 21:38 |
wbhuber | briancline: so those who are running multiple policies on master in a dev environment fail | 21:38 |
briancline | wbhuber: ahh. so since the test sets up the env, where or at what point do the additional policies get introduced? | 21:40 |
*** jkugel has quit IRC | 21:40 | |
*** BAKfr has quit IRC | 21:41 | |
wbhuber | briancline: well, it's one's preference to have > 1 policy, like in my case, i've replication and EC running for testing purposes. (2 policies, thus 2 rings, etc.) | 21:41 |
*** mragupat_ has joined #openstack-swift | 21:42 | |
briancline | wbhuber: yeah, definitely -- but in that case is the test discovering those additional policies from a "real" /etc/swift/swift.conf? | 21:43 |
*** BAKfr has joined #openstack-swift | 21:43 | |
*** jlhinson has quit IRC | 21:44 | |
wbhuber | briancline: line 344 for test_recon.py in ur patch | 21:46 |
*** mragupat has quit IRC | 21:46 | |
*** mragupat_ has quit IRC | 21:46 | |
wbhuber | briancline: it gets rings from i believe /etc/swift | 21:46 |
wbhuber | briancline: and it finds more rings than the test expected | 21:46 |
*** mragupat has joined #openstack-swift | 21:47 | |
*** hrou has quit IRC | 21:47 | |
briancline | wbhuber: ahhh, gotcha. yeah, I was trying to make the connection with what I could have done differently in my own environment to reproduce and test under both scenarios. that makes sense | 21:48 |
briancline | I'm guessing if I'd patched them for that method and had a second, multi-policy equivalent of that one, it might've caught it? | 21:50 |
wbhuber | briancline: i wouldn't have caught it either. not sure if we should make gate smarter to handle > 1 policy. since EC is to be production ready, gate might have to be altered to handle > 1. but that's not up to me. | 21:50 |
briancline | wbhuber: yeah, seems prudent | 21:50 |
briancline | what's *really* great about my breaking it is that I really only created the failed_ring_hash test as an ancillary thing, because the old test case was doing it in a really haphazard way :) | 21:53 |
*** luksaur has quit IRC | 21:53 | |
briancline | looks like it was just to achieve coverage of the error condition | 21:53 |
*** eranrom has quit IRC | 21:54 | |
*** tongli has quit IRC | 22:00 | |
*** DericHorn-HP has quit IRC | 22:00 | |
openstackgerrit | Merged openstack/swift: go: add a way to terminate replication cleanly https://review.openstack.org/224881 | 22:06 |
openstackgerrit | Merged openstack/swift: go: fix some panics in unpickle https://review.openstack.org/208033 | 22:08 |
*** mragupat has quit IRC | 22:12 | |
openstackgerrit | Merged openstack/swift: Reconstructor GET excludes user_agent in log https://review.openstack.org/221506 | 22:12 |
*** NM has quit IRC | 22:19 | |
*** fbo has quit IRC | 22:19 | |
*** minwoob has quit IRC | 22:21 | |
briancline | ok yeah, just got it to fail. didn't realize the unit tests pick up live configs | 22:21 |
briancline | sorry for that | 22:21 |
*** fbo has joined #openstack-swift | 22:23 | |
*** lpabon has joined #openstack-swift | 22:26 | |
*** lpabon has quit IRC | 22:31 | |
clayg | briancline: yeah we should @patch_policies everywhere once and for all - but it's sorta much fun to run around with hair on fire every so often! | 22:32 |
openstackgerrit | Clay Gerrard proposed openstack/swift: Fix proxy hadling of EC client disconnect https://review.openstack.org/225357 | 22:32 |
*** jrichli has quit IRC | 22:32 | |
*** garthb has quit IRC | 22:33 | |
clayg | so I'm little worried about the *lack* of failing tests when I started to move that code around - but I'm pretty sure if you put it back my *new* tests would fail - and I think they somewhat obviously demonstrade the desireable behavior | 22:33 |
*** lpabon has joined #openstack-swift | 22:37 | |
clayg | mattoliverau: next time you see kota tell him to get a bouncer or something - drives me nuts that I can't message him async | 22:37 |
*** lpabon has quit IRC | 22:38 | |
clayg | I mean I don't expect him to /lastlog kota *everytime* he comes online - most of the time I just makes me *feel* better to talk my friends - even if they can't here me | 22:38 |
clayg | acoles_: hi | 22:38 |
*** lpabon has joined #openstack-swift | 22:38 | |
wbhuber | clayg: line 2182 in your latest patch - should it be indented 4 spaces left? | 22:38 |
clayg | I DON'T KNOW!!!! | 22:38 |
*** lpabon has quit IRC | 22:39 | |
clayg | i... don't *think* so? | 22:39 |
wbhuber | in swift_proxy_controller_obj.py... | 22:39 |
*** lpabon has joined #openstack-swift | 22:39 | |
clayg | like we only send the chunk if it *wasn't* StopIteration right? | 22:39 |
clayg | what does replicated do? | 22:39 |
wbhuber | right but uh... else? | 22:42 |
wbhuber | i'd need to look more | 22:42 |
*** lpabon has quit IRC | 22:44 | |
*** openstackgerrit has quit IRC | 22:46 | |
*** openstackgerrit has joined #openstack-swift | 22:46 | |
clayg | yeah! try: except: else: is the bomb! | 22:49 |
clayg | only thing cooler is for: break else: | 22:49 |
wbhuber | no while: else:? :) | 22:50 |
wbhuber | client disconnects without sending "last" chunk? you mean the last chunk of chunks or the latest chunk that its being sent? | 22:50 |
clayg | you know - i don't know if I ever needed a while: break else: | 22:55 |
clayg | the "last chunk" in chunked-trasnfer context the 0\r\n\r\n (how you terminate a stream of bytes with an inditerminate length) | 22:55 |
clayg | wbhuber: I'm not really saying any of that patch is good except the part where the tests pass - because the uploading and disconnecting mistream resuling in data is crap - everything else except for that is up for debate - reviews much appreciated! | 22:56 |
clayg | but... I need to bolt | 22:56 |
clayg | there's a thing at the kids school tonite | 22:56 |
wbhuber | clayg: have a good 1. i'm leaving a comment. | 22:56 |
clayg | so... ya'll have fun - we'll keep banging on it next week | 22:57 |
wbhuber | clayg: cool! they're best at keeping your mind off | 22:57 |
*** hrou has joined #openstack-swift | 22:59 | |
openstackgerrit | Merged openstack/swift: Fix recon tests on SAIO with multiple policies https://review.openstack.org/225329 | 23:08 |
*** zhill has quit IRC | 23:09 | |
*** wbhuber has quit IRC | 23:10 | |
*** chsc has quit IRC | 23:13 | |
mattoliverau | clayg: will do :) | 23:14 |
*** esker has joined #openstack-swift | 23:15 | |
*** zhill_ has joined #openstack-swift | 23:16 | |
*** nadeem has quit IRC | 23:22 | |
*** esker has quit IRC | 23:22 | |
*** esker has joined #openstack-swift | 23:41 | |
*** esker has quit IRC | 23:46 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!