Friday, 2020-01-17

*** openstackgerrit has joined #openstack-swift00:00
openstackgerritTim Burke proposed openstack/swift master: New Object Versioning mode  https://review.opendev.org/68238200:00
openstackgerritTim Burke proposed openstack/swift master: Skip container-sync when versioning enabled  https://review.opendev.org/69813900:00
timburketdasilva, see what you think about ^^^ -- i think it's about ready to be squashed in, too00:00
openstackgerritMerged openstack/swift master: Early-return on non-Swift get_info requests  https://review.opendev.org/69905500:06
tdasilvatimburke: 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
timburkeheh -- what error? shouldn't be any error now ;-)00:16
timburkei suppose for more fun, we could kill the symlink before syncing... then assert the 40900:17
timburke(as opposed ot 404)00:18
tdasilvatimburke: sorry, was thinking of the paste, the symlink_override takes care of it00:22
*** cjloader has quit IRC00:32
openstackgerritTim Burke proposed openstack/swift master: Skip container-sync when versioning enabled  https://review.opendev.org/69813900:35
seongsoochomorning o/00:38
seongsoochotimburke:   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
patchbotpatch 702950 - swift - Allow Timestamp comparisons against out-of-range v... - 1 patch set00:42
*** tkajinam_ has quit IRC00:43
timburkeyep! and, i'm pretty sure, POST to add a new, more-appropriate x-delete-at00:43
*** tkajinam has joined #openstack-swift00:43
seongsoochotimburke:  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 again00:50
*** openstackgerrit has quit IRC00:57
*** tkajinam_ has joined #openstack-swift01:00
*** tkajinam has quit IRC01:00
seongsoochooh.. POST also doesn't work currently.01:02
timburkeeven with the patch? hmm... seemed to work locally for me...01:02
seongsoochooh nono . before the patch. I will apply it now01:04
timburke👍01:04
*** diablo_rojo has quit IRC01:09
timburkewell that's annoying. apparently ostestr doesn't like unittest.SkipTest getting raised in setUpModule?01:09
timburkeat least, on py201:10
timburketdasilva, looks like our versioning patches will have a bunch of py2 func test failures01:10
timburkei gotta go pick up kids01:10
timburkehave a good night, everyone!01:11
tdasilvatimburke: i'll look01:11
tdasilvathanks for the heads up01:11
tdasilvagood night01:11
*** openstackgerrit has joined #openstack-swift01:38
openstackgerritMerged openstack/swift master: account-server: Add test for leading delimiter  https://review.opendev.org/70096401:38
*** gyee has quit IRC02:07
kota_good morning02:19
seongsoochokota_:  good morning02: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 patch02:51
seongsoochokota_:  yes.  timburke 's patch works very well in my test env. I really appreciate both of you.02:53
timburkeit'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
timburkejust got home02:55
kota_I added my comments to the patch just now02:55
timburkekids are watching something, i've got a moment to type "recheck" a few times :P02: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 str02:56
kota_big int was fe02:56
kota_fed02:56
*** edausq has quit IRC02:57
kota_and also I'm not sure *EVERY* swift Timestamp are compared with... int, float, Timestamp... no str???02:57
tdasilvatimburke: I had no luck reproducing either Skiptest or py2 func test failures03:00
tdasilvaOTOH I'm trying to understand unit test failures on python-swiftclient03:01
tdasilvajust not a good day03:01
timburkefunky. 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
tdasilvayeah, that one and this: https://zuul.opendev.org/t/openstack/build/547dbe1526974986b6fd2ead5918ebd5/console03:02
*** psachin has joined #openstack-swift03:19
*** jvisser has joined #openstack-swift04:17
openstackgerritThiago da Silva proposed openstack/swift master: New Object Versioning mode  https://review.opendev.org/68238204:22
*** jvisser has quit IRC04:40
timburke:-/ I can't help but feel like `Timestamp('1.e+9')` really *ought* to ValueError...04:49
kota_true05:08
*** rpittau|afk has quit IRC05:34
*** evrardjp has quit IRC05:34
*** evrardjp has joined #openstack-swift05:34
*** rpittau|afk has joined #openstack-swift05:34
kota_hmmm...05:41
tdasilvagate is just painful today05:48
kota_so the patach introduces the behavior that `Timestamp('1.e+9')` -> no error. that means we don't have the upper cap anymore05:48
kota_tdasilva: oh you've been fighting the gate05:48
tdasilvakota_: timburke mostly...recheck recheck recheck05:49
timburkekota_, 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
timburkei pointedly tried *not* to touch __init__05:51
kota_got it05:52
timburkeand 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
timburkebut then you run into questions about whether that should *also* apply to offsets... bleh05:53
* kota_ got headache to think of those.06:00
*** tkajinam__ has joined #openstack-swift06:37
*** tkajinam_ has quit IRC06:39
*** tkajinam_ has joined #openstack-swift06:42
*** tkajinam__ has quit IRC06:44
*** lifeless has quit IRC07:19
*** zigo has quit IRC07:19
*** hugokuo has quit IRC07:19
*** openstackstatus has quit IRC07:20
*** rdejoux has joined #openstack-swift07:32
*** lifeless has joined #openstack-swift08:05
*** zigo has joined #openstack-swift08:05
*** hugokuo has joined #openstack-swift08:05
*** tkajinam_ has quit IRC08:11
*** rdejoux has quit IRC08:13
*** tesseract has joined #openstack-swift08:16
*** rpittau|afk is now known as rpittau08:28
*** ccamacho has joined #openstack-swift08:35
*** rdejoux has joined #openstack-swift08:49
*** rcernin_ has joined #openstack-swift09:04
*** rcernin has quit IRC09:04
*** psachin has quit IRC09:35
*** Jeffrey4l has quit IRC09:50
*** Jeffrey4l has joined #openstack-swift10:21
*** rcernin_ has quit IRC10:34
*** ccamel has joined #openstack-swift11:16
*** camelCaser has quit IRC11:16
*** pcaruana has joined #openstack-swift11:59
*** diablo_rojo has joined #openstack-swift13:28
*** pcaruana has quit IRC13:45
*** pcaruana has joined #openstack-swift14:22
*** jvisser has joined #openstack-swift15:39
timburkeman, 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
timburkeor maybe Timestamp('.1e+10_0xf') since without the '.' it'd ValueError15:46
claygtimburke: you're such a haXor15:47
claygduck typing got you down15:47
clayg?15:47
timburkehttps://knowyourmeme.com/photos/140134715:53
openstackgerritTim Burke proposed openstack/swift master: Allow Timestamp comparisons against out-of-range values  https://review.opendev.org/70295016:12
*** gyee has joined #openstack-swift16:27
*** rdejoux has quit IRC16:42
*** rpittau is now known as rpittau|afk16:53
*** evrardjp has quit IRC17:34
*** evrardjp has joined #openstack-swift17:34
openstackgerritTim Burke proposed openstack/swift master: Store normalized x-delete-at on PUT/POST  https://review.opendev.org/70315518:10
openstackgerritTim Burke proposed openstack/swift master: Skip container-sync when versioning enabled  https://review.opendev.org/69813918:45
openstackgerritTim Burke proposed openstack/swift master: s3api: Implement object versioning API  https://review.opendev.org/67368218:45
timburke^^^ mostly just restacking -- pretty sure p 698139 should pass now18:46
patchbothttps://review.opendev.org/#/c/698139/ - swift - Skip container-sync when versioning enabled - 14 patch sets18:46
*** pcaruana has quit IRC18:51
*** diablo_rojo has quit IRC18:58
*** tesseract has quit IRC19:32
openstackgerritTim Burke proposed openstack/swift master: Allow bulk to fwd some headers at tar extraction  https://review.opendev.org/70173619:34
timburkedo we have *any* bulk-upload func tests?19:49
timburkei see a test_bulk_delete...19:50
*** openstackstatus has joined #openstack-swift20:01
*** ChanServ sets mode: +v openstackstatus20:01
*** diablo_rojo has joined #openstack-swift20:47
claygtimburke: I don't see any extract-archive in functests 😬21:28
openstackgerritTim Burke proposed openstack/swift master: Middleware that allows a user to have quoted Etags  https://review.opendev.org/70005621:55
*** jvisser has quit IRC22:45

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!