*** openstackgerrit has joined #openstack-swift | 00:00 | |
openstackgerrit | Tim Burke proposed openstack/swift master: New Object Versioning mode https://review.opendev.org/682382 | 00:00 |
---|---|---|
openstackgerrit | Tim Burke proposed openstack/swift master: Skip container-sync when versioning enabled https://review.opendev.org/698139 | 00:00 |
timburke | tdasilva, see what you think about ^^^ -- i think it's about ready to be squashed in, too | 00:00 |
openstackgerrit | Merged openstack/swift master: Early-return on non-Swift get_info requests https://review.opendev.org/699055 | 00:06 |
tdasilva | timburke: looks good! was trying to check if sync will continue to other rows besides the error from the static symlink, and i think it will, but for sync pointer won't advance, correct? | 00:15 |
timburke | heh -- what error? shouldn't be any error now ;-) | 00:16 |
timburke | i suppose for more fun, we could kill the symlink before syncing... then assert the 409 | 00:17 |
timburke | (as opposed ot 404) | 00:18 |
tdasilva | timburke: sorry, was thinking of the paste, the symlink_override takes care of it | 00:22 |
*** cjloader has quit IRC | 00:32 | |
openstackgerrit | Tim Burke proposed openstack/swift master: Skip container-sync when versioning enabled https://review.opendev.org/698139 | 00:35 |
seongsoocho | morning o/ | 00:38 |
seongsoocho | timburke: https://review.opendev.org/#/c/702950/ <- This patch allows to download object (with outside of timestamp range - x-delete-at). Am I understand correctly? | 00:42 |
patchbot | patch 702950 - swift - Allow Timestamp comparisons against out-of-range v... - 1 patch set | 00:42 |
*** tkajinam_ has quit IRC | 00:43 | |
timburke | yep! and, i'm pretty sure, POST to add a new, more-appropriate x-delete-at | 00:43 |
*** tkajinam has joined #openstack-swift | 00:43 | |
seongsoocho | timburke: yes. update a valid value of x-delete-at is much better solution. but, As I tested before, POST also doesn't work. I will test it again | 00:50 |
*** openstackgerrit has quit IRC | 00:57 | |
*** tkajinam_ has joined #openstack-swift | 01:00 | |
*** tkajinam has quit IRC | 01:00 | |
seongsoocho | oh.. POST also doesn't work currently. | 01:02 |
timburke | even with the patch? hmm... seemed to work locally for me... | 01:02 |
seongsoocho | oh nono . before the patch. I will apply it now | 01:04 |
timburke | 👍 | 01:04 |
*** diablo_rojo has quit IRC | 01:09 | |
timburke | well that's annoying. apparently ostestr doesn't like unittest.SkipTest getting raised in setUpModule? | 01:09 |
timburke | at least, on py2 | 01:10 |
timburke | tdasilva, looks like our versioning patches will have a bunch of py2 func test failures | 01:10 |
timburke | i gotta go pick up kids | 01:10 |
timburke | have a good night, everyone! | 01:11 |
tdasilva | timburke: i'll look | 01:11 |
tdasilva | thanks for the heads up | 01:11 |
tdasilva | good night | 01:11 |
*** openstackgerrit has joined #openstack-swift | 01:38 | |
openstackgerrit | Merged openstack/swift master: account-server: Add test for leading delimiter https://review.opendev.org/700964 | 01:38 |
*** gyee has quit IRC | 02:07 | |
kota_ | good morning | 02:19 |
seongsoocho | kota_: good morning | 02:50 |
kota_ | seongsoocho: o/ | 02:51 |
kota_ | it looks timburke resolves your problem more smarter way than mine. | 02:51 |
kota_ | I'm just looking at the patch | 02:51 |
seongsoocho | kota_: yes. timburke 's patch works very well in my test env. I really appreciate both of you. | 02:53 |
timburke | it'd still be better if i didn't go breaking things ;-) | 02:54 |
kota_ | seongsoocho: sounds awesome. | 02:54 |
kota_ | oh timburke still here. | 02:55 |
timburke | just got home | 02:55 |
kota_ | I added my comments to the patch just now | 02:55 |
timburke | kids are watching something, i've got a moment to type "recheck" a few times :P | 02:55 |
kota_ | I'm not sure if you want to care about the case, but `Timestamp.now() < "100000000000000"` still cause the ValueError when the big int fed as str | 02:56 |
kota_ | big int was fe | 02:56 |
kota_ | fed | 02:56 |
*** edausq has quit IRC | 02:57 | |
kota_ | and also I'm not sure *EVERY* swift Timestamp are compared with... int, float, Timestamp... no str??? | 02:57 |
tdasilva | timburke: I had no luck reproducing either Skiptest or py2 func test failures | 03:00 |
tdasilva | OTOH I'm trying to understand unit test failures on python-swiftclient | 03:01 |
tdasilva | just not a good day | 03:01 |
timburke | funky. so what's up with https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_7f5/682382/73/check/swift-tox-func-py27/7f52979/testr_results.html then? hmm... | 03:01 |
tdasilva | yeah, that one and this: https://zuul.opendev.org/t/openstack/build/547dbe1526974986b6fd2ead5918ebd5/console | 03:02 |
*** psachin has joined #openstack-swift | 03:19 | |
*** jvisser has joined #openstack-swift | 04:17 | |
openstackgerrit | Thiago da Silva proposed openstack/swift master: New Object Versioning mode https://review.opendev.org/682382 | 04:22 |
*** jvisser has quit IRC | 04:40 | |
timburke | :-/ I can't help but feel like `Timestamp('1.e+9')` really *ought* to ValueError... | 04:49 |
kota_ | true | 05:08 |
*** rpittau|afk has quit IRC | 05:34 | |
*** evrardjp has quit IRC | 05:34 | |
*** evrardjp has joined #openstack-swift | 05:34 | |
*** rpittau|afk has joined #openstack-swift | 05:34 | |
kota_ | hmmm... | 05:41 |
tdasilva | gate is just painful today | 05:48 |
kota_ | so the patach introduces the behavior that `Timestamp('1.e+9')` -> no error. that means we don't have the upper cap anymore | 05:48 |
kota_ | tdasilva: oh you've been fighting the gate | 05:48 |
tdasilva | kota_: timburke mostly...recheck recheck recheck | 05:49 |
timburke | kota_, no no, `Timestamp('1.e+9')` produces no error *on master*. we just have weird rules for converting strings to timestamps :-/ | 05:50 |
kota_ | timburke: oic. that's painful :-/ | 05:51 |
timburke | i pointedly tried *not* to touch __init__ | 05:51 |
kota_ | got it | 05:52 |
timburke | and then left off bytes/unicode handling in the comparison because it felt silly to reimplement that parsing... but maybe i'll have to add an ignore_bounds arg to __init__ or something... | 05:52 |
timburke | but then you run into questions about whether that should *also* apply to offsets... bleh | 05:53 |
* kota_ got headache to think of those. | 06:00 | |
*** tkajinam__ has joined #openstack-swift | 06:37 | |
*** tkajinam_ has quit IRC | 06:39 | |
*** tkajinam_ has joined #openstack-swift | 06:42 | |
*** tkajinam__ has quit IRC | 06:44 | |
*** lifeless has quit IRC | 07:19 | |
*** zigo has quit IRC | 07:19 | |
*** hugokuo has quit IRC | 07:19 | |
*** openstackstatus has quit IRC | 07:20 | |
*** rdejoux has joined #openstack-swift | 07:32 | |
*** lifeless has joined #openstack-swift | 08:05 | |
*** zigo has joined #openstack-swift | 08:05 | |
*** hugokuo has joined #openstack-swift | 08:05 | |
*** tkajinam_ has quit IRC | 08:11 | |
*** rdejoux has quit IRC | 08:13 | |
*** tesseract has joined #openstack-swift | 08:16 | |
*** rpittau|afk is now known as rpittau | 08:28 | |
*** ccamacho has joined #openstack-swift | 08:35 | |
*** rdejoux has joined #openstack-swift | 08:49 | |
*** rcernin_ has joined #openstack-swift | 09:04 | |
*** rcernin has quit IRC | 09:04 | |
*** psachin has quit IRC | 09:35 | |
*** Jeffrey4l has quit IRC | 09:50 | |
*** Jeffrey4l has joined #openstack-swift | 10:21 | |
*** rcernin_ has quit IRC | 10:34 | |
*** ccamel has joined #openstack-swift | 11:16 | |
*** camelCaser has quit IRC | 11:16 | |
*** pcaruana has joined #openstack-swift | 11:59 | |
*** diablo_rojo has joined #openstack-swift | 13:28 | |
*** pcaruana has quit IRC | 13:45 | |
*** pcaruana has joined #openstack-swift | 14:22 | |
*** jvisser has joined #openstack-swift | 15:39 | |
timburke | man, you can pass some really strange strings to Timestamp on master and it'll be fine... best i've found so far is Timestamp('.0e+999_0xf') | 15:46 |
timburke | or maybe Timestamp('.1e+10_0xf') since without the '.' it'd ValueError | 15:46 |
clayg | timburke: you're such a haXor | 15:47 |
clayg | duck typing got you down | 15:47 |
clayg | ? | 15:47 |
timburke | https://knowyourmeme.com/photos/1401347 | 15:53 |
openstackgerrit | Tim Burke proposed openstack/swift master: Allow Timestamp comparisons against out-of-range values https://review.opendev.org/702950 | 16:12 |
*** gyee has joined #openstack-swift | 16:27 | |
*** rdejoux has quit IRC | 16:42 | |
*** rpittau is now known as rpittau|afk | 16:53 | |
*** evrardjp has quit IRC | 17:34 | |
*** evrardjp has joined #openstack-swift | 17:34 | |
openstackgerrit | Tim Burke proposed openstack/swift master: Store normalized x-delete-at on PUT/POST https://review.opendev.org/703155 | 18:10 |
openstackgerrit | Tim Burke proposed openstack/swift master: Skip container-sync when versioning enabled https://review.opendev.org/698139 | 18:45 |
openstackgerrit | Tim Burke proposed openstack/swift master: s3api: Implement object versioning API https://review.opendev.org/673682 | 18:45 |
timburke | ^^^ mostly just restacking -- pretty sure p 698139 should pass now | 18:46 |
patchbot | https://review.opendev.org/#/c/698139/ - swift - Skip container-sync when versioning enabled - 14 patch sets | 18:46 |
*** pcaruana has quit IRC | 18:51 | |
*** diablo_rojo has quit IRC | 18:58 | |
*** tesseract has quit IRC | 19:32 | |
openstackgerrit | Tim Burke proposed openstack/swift master: Allow bulk to fwd some headers at tar extraction https://review.opendev.org/701736 | 19:34 |
timburke | do we have *any* bulk-upload func tests? | 19:49 |
timburke | i see a test_bulk_delete... | 19:50 |
*** openstackstatus has joined #openstack-swift | 20:01 | |
*** ChanServ sets mode: +v openstackstatus | 20:01 | |
*** diablo_rojo has joined #openstack-swift | 20:47 | |
clayg | timburke: I don't see any extract-archive in functests 😬 | 21:28 |
openstackgerrit | Tim Burke proposed openstack/swift master: Middleware that allows a user to have quoted Etags https://review.opendev.org/700056 | 21:55 |
*** jvisser has quit IRC | 22:45 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!