*** gyee has quit IRC | 00:39 | |
openstackgerrit | Merged openstack/swift master: Add etag quoter to saio proxy config https://review.opendev.org/704920 | 00:51 |
---|---|---|
openstackgerrit | Merged openstack/swift master: ring: Flag region, zone, and device as required in add_dev https://review.opendev.org/711790 | 00:51 |
zaitcev | Only took 8 rechecks. | 00:56 |
*** diablo_rojo has quit IRC | 02:01 | |
zaitcev | timburke: do the CORS preflight requests not contain a path? | 02:12 |
seongsoocho | Is there a recommand value of node _timeout in proxy-server?? The default value is 10 seconds. I was set it to 5 seconds, but the 'get final status of PUT' timeout occationally occured. | 02:54 |
*** gkadam has joined #openstack-swift | 03:03 | |
*** gkadam has left #openstack-swift | 03:06 | |
zaitcev | The default worked fine for me. In fact I hear more often from users who try to increase it, for clusters running under heavy load. | 03:06 |
seongsoocho | zaitcev: aha.. I thought the value should be decrease when cluster running under heavy load to store object to hand off node. | 03:10 |
*** diablo_rojo has joined #openstack-swift | 03:10 | |
*** renich has quit IRC | 03:13 | |
*** evrardjp has quit IRC | 05:36 | |
*** evrardjp has joined #openstack-swift | 05:36 | |
*** diablo_rojo has quit IRC | 05:39 | |
*** rcernin has quit IRC | 05:40 | |
*** rcernin has joined #openstack-swift | 05:41 | |
*** rcernin has quit IRC | 05:41 | |
*** rcernin has joined #openstack-swift | 05:42 | |
*** rcernin has quit IRC | 05:42 | |
*** rcernin has joined #openstack-swift | 05:47 | |
*** rcernin has quit IRC | 06:02 | |
*** rcernin has joined #openstack-swift | 06:02 | |
*** rcernin has quit IRC | 06:05 | |
*** rcernin has joined #openstack-swift | 06:05 | |
*** evrardjp has quit IRC | 07:57 | |
*** evrardjp has joined #openstack-swift | 08:01 | |
*** zaitcev has quit IRC | 08:06 | |
*** clayg has quit IRC | 08:06 | |
*** pawan-gupta has quit IRC | 08:06 | |
*** zaitcev has joined #openstack-swift | 08:07 | |
*** ChanServ sets mode: +v zaitcev | 08:07 | |
*** tkajinam has quit IRC | 08:19 | |
*** rpittau|afk is now known as rpittau | 08:26 | |
*** btorch has joined #openstack-swift | 09:22 | |
*** rcernin has quit IRC | 09:51 | |
*** btorch has quit IRC | 10:16 | |
*** dsariel has joined #openstack-swift | 10:44 | |
*** btorch has joined #openstack-swift | 11:41 | |
*** rpittau is now known as rpittau|bbl | 11:45 | |
*** btorch has quit IRC | 12:10 | |
*** btorch_ has joined #openstack-swift | 12:10 | |
*** btorch_ has quit IRC | 12:12 | |
*** rpittau|bbl is now known as rpittau | 13:10 | |
*** tkajinam has joined #openstack-swift | 13:22 | |
*** dsariel has quit IRC | 14:30 | |
*** DHE has quit IRC | 14:30 | |
*** mahatic has quit IRC | 14:30 | |
*** corvus has quit IRC | 14:30 | |
*** corvus has joined #openstack-swift | 14:31 | |
*** DHE has joined #openstack-swift | 14:31 | |
*** mahatic has joined #openstack-swift | 14:44 | |
*** ChanServ sets mode: +v mahatic | 14:44 | |
*** gyee has joined #openstack-swift | 15:21 | |
*** tkajinam has quit IRC | 15:30 | |
*** clayg has joined #openstack-swift | 15:51 | |
*** ChanServ sets mode: +v clayg | 15:51 | |
clayg | p 691877 | 15:51 |
patchbot | https://review.opendev.org/#/c/691877/ - python-swiftclient - object versioning features - 10 patch sets | 15:51 |
clayg | ^ merge it! 😁 | 15:52 |
tdasilva | +1 | 16:09 |
tdasilva | or +2 | 16:09 |
tdasilva | I put a +2 there, but it would be nice if someone else took a look since I also contributed code to it | 16:10 |
*** rdejoux has quit IRC | 16:38 | |
*** rdejoux has joined #openstack-swift | 16:49 | |
*** jv__ has joined #openstack-swift | 17:05 | |
*** jv has quit IRC | 17:05 | |
*** rpittau is now known as rpittau|afk | 17:12 | |
*** rdejoux has quit IRC | 17:16 | |
*** rdejoux has joined #openstack-swift | 17:22 | |
*** ccamacho has joined #openstack-swift | 17:31 | |
*** ccamacho has quit IRC | 17:32 | |
*** evrardjp has quit IRC | 17:49 | |
*** evrardjp has joined #openstack-swift | 17:49 | |
*** openstack has quit IRC | 17:49 | |
*** openstack has joined #openstack-swift | 17:51 | |
*** ChanServ sets mode: +o openstack | 17:51 | |
*** diablo_rojo has joined #openstack-swift | 18:47 | |
timburke | zaitcev, CORS preflights include a path. note that for s3api, though, that's not enough to be useful -- it'll have a bucket (container) name, but no account | 18:55 |
timburke | seongsoocho, zaitcev: i've been thinking lately that it'd be nice to have per-verb node_timeouts, possibly even per-verb-per-layer (or per-verb-per-policy?) | 18:56 |
timburke | we had one cluster where we cranked up node_timeout to 20s to work around bug #1833612 / bug #1833616 -- but while the PUTs were improving, the latencies on GETs went up. fwiw, we brought it down to 8s -- but we're definitely keeping an eye out for PUT troubles | 19:02 |
openstack | bug 1833612 in OpenStack Object Storage (swift) "Overloaded container can get erroneously cached as 404" [Undecided,Fix released] https://launchpad.net/bugs/1833612 | 19:02 |
openstack | bug 1833616 in OpenStack Object Storage (swift) "container_info non-result causes object writes to 404" [Undecided,Fix released] https://launchpad.net/bugs/1833616 - Assigned to clayg (clay-gerrard) | 19:02 |
timburke | clayg, thinking about the cors tests, and in particular being able to chain multiple requests, i've been trying to rework the test harness -- as far as the existing tests go, it'll have changes like http://paste.openstack.org/show/791209/; wdyt? better or worse? | 19:16 |
clayg | so the "chain multiple requests" part is the ... oh that's a diff hold on... so can you MakeRequest.then(Makerequest) 🤯 | 19:28 |
clayg | what gets passed into .then exactly? @timburke I'm honestly not the best person to ask - i know bupkis about javascript | 19:28 |
timburke | clayg, or at any rate, i'm pretty sure i'll be able to do a MakeRequest().then( (resp) => MakeRequest() ).then(MoreAssertions) | 19:30 |
timburke | so MakeRequest is returning a Promise (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise) whose .then() takes a function (or two) of one arg -- the arg is whatever the promise resolved to. the function then has the option of returning or throwing; these will resolve or reject the Promise that's returned by .then() | 19:35 |
timburke | it's all weird and a little messy -- so the fact that you (1) know bupkis about javascript, and (2) would be stuck dealing with these tests if it merges, means that you're the *perfect* person to ask which seems better/easier to reason about :D | 19:37 |
openstackgerrit | Romain LE DISEZ proposed openstack/swift master: Optimize obj replicator/reconstructor healthchecks https://review.opendev.org/715298 | 19:41 |
openstackgerrit | Tim Burke proposed openstack/swift master: py3: stop barfing on message/rfc822 Content-Types https://review.opendev.org/707312 | 19:56 |
clayg | @timburke I tried. I still don't get it. LIke I get the lazy evaluation and the chaning. But I don't know ... | 20:15 |
clayg | 1) does the resolve function not always get the return value automatically - it's just whatever the promise passes it? | 20:15 |
clayg | 2) what do you do if you need to dispatch into a forked chain of resolve functions based on return value of the previous async action? like not just (pass, fail); but (typeA, typeB, fail) where typeA and typeB will have different async action chains? | 20:16 |
clayg | I did a buch of twisted a LONG time ago, so I hate callbacks 😁 | 20:17 |
timburke | so the useResolved function always gets whatever the promise "returned"/resolved-to (or, if the promise returned another promise, what that subpromise "returns") | 20:19 |
timburke | but it isn't necessarily called. if the promise is rejected, the .then() callback never gets called | 20:20 |
timburke | there are two types of chain forking i imagine we'd be interested in -- allowing (at least) pass/fail/skip instead of just pass/fail, and allowing multiple follow-ups to a single request (hi, /info!) | 20:23 |
timburke | i've currently got skip as a special class of error, which seems to be working ok | 20:24 |
timburke | and you can hold on to a promise and link up to it multiple times, like `const info = MakeRequest('GET', '/info').then(ParseInfoResponse); info.then(doA); info.then(doB)` | 20:25 |
timburke | (i think. will keep you updated when i get to converting some of those slo/symlink tests :P) | 20:26 |
openstackgerrit | Romain LE DISEZ proposed openstack/swift master: Optimize obj replicator/reconstructor healthchecks https://review.opendev.org/715298 | 21:02 |
*** rcernin has joined #openstack-swift | 22:25 | |
*** tkajinam has joined #openstack-swift | 22:54 | |
rledisez | on p715298 I have a failure of test swift-tox-lower-constraints: ERROR: Package 'futurist' requires a different Python: 2.7.17 not in '>=3.6' | 22:56 |
rledisez | I'm note sure what to do about that. any hint? | 22:56 |
rledisez | oops, p 715298 | 22:57 |
patchbot | https://review.opendev.org/#/c/715298/ - swift - Optimize obj replicator/reconstructor healthchecks - 2 patch sets | 22:57 |
mattoliverau | morning | 23:00 |
*** rcernin has quit IRC | 23:06 | |
*** rcernin has joined #openstack-swift | 23:07 | |
*** rcernin has quit IRC | 23:07 | |
*** rcernin has joined #openstack-swift | 23:08 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!