Monday, 2020-03-02

*** dasp has joined #openstack-swift00:16
*** dasp_ has quit IRC00:18
*** zaitcev has quit IRC01:04
kota_good morning01:09
kota_Jesus, commuting has been restricted.01:11
* kota_ is working at home01:11
*** tkajinam has quit IRC01:24
*** tkajinam has joined #openstack-swift01:25
*** rledisez has quit IRC02:26
*** alecuyer has quit IRC02:26
*** alecuyer has joined #openstack-swift02:27
*** rledisez has joined #openstack-swift02:28
*** ChanServ sets mode: +v rledisez02:28
*** alecuyer has quit IRC02:30
*** rledisez has quit IRC02:30
mattoliveraukota_: oh really. welcome to my world.. though I'm actaully set up for it02:31
*** alecuyer has joined #openstack-swift02:36
*** rledisez has joined #openstack-swift02:37
*** ChanServ sets mode: +v rledisez02:37
*** psachin has joined #openstack-swift03:34
*** evrardjp has quit IRC05:35
*** evrardjp has joined #openstack-swift05:35
*** threestrands has joined #openstack-swift06:04
*** threestrands has quit IRC06:16
*** rcernin has quit IRC06:47
*** rdejoux has joined #openstack-swift07:29
*** rdejoux has quit IRC07:45
*** rdejoux has joined #openstack-swift07:46
*** tesseract has joined #openstack-swift07:59
*** ccamacho has joined #openstack-swift08:13
*** manuvakery has quit IRC08:14
*** rpittau|afk is now known as rpittau08:18
*** tkajinam has quit IRC08:24
*** tkajinam has joined #openstack-swift09:39
*** tkajinam has quit IRC10:18
*** mvkr has joined #openstack-swift10:28
*** rpittau is now known as rpittau|bbl11:32
*** tkajinam has joined #openstack-swift11:47
*** manuvakery has joined #openstack-swift11:57
pawan-guptaHi, I would like to know, if implementation of BUCKET_POLICY is planned in near future? It would be helpful for me to have an idea about timelines on this. Thanks in advance.11:58
*** rdejoux has quit IRC12:20
*** tkajinam has quit IRC12:45
*** rdejoux has joined #openstack-swift12:56
*** rpittau|bbl is now known as rpittau13:34
*** psachin has quit IRC13:35
*** jvisser has joined #openstack-swift14:01
*** jvisser has quit IRC14:01
*** jvisser has joined #openstack-swift14:01
claygtimburke: p 710492 looks good to me, solid bug fix, thanks for the help!14:56
patchbothttps://review.opendev.org/#/c/710492/ - swift - Apply limit to list versioned containers - 4 patch sets14:56
claygpawan-gupta: there's a number of different s3 style bucket policies - support probably won't be universal initially14:57
claygpawan-gupta: version expiration is the one that's top of mind for me - what sort of support are you looking for?14:57
*** gyee has joined #openstack-swift16:15
openstackgerritTim Burke proposed openstack/swift master: Use limit-helper in container-server, too  https://review.opendev.org/71081516:20
*** diablo_rojo has quit IRC16:40
*** diablo_rojo has joined #openstack-swift16:40
openstackgerritClay Gerrard proposed openstack/swift master: Use float consistently for proxy timeout settings  https://review.opendev.org/71047516:44
*** rpittau is now known as rpittau|afk16:59
timburkeclayg, looking at p 710475 -- is monkey-patching the actual policies like that going to taint any other tests?17:09
patchbothttps://review.opendev.org/#/c/710475/ - swift - Use float consistently for proxy timeout settings - 2 patch sets17:09
*** tesseract has quit IRC17:16
*** rdejoux has quit IRC17:24
claygtimburke: I don't think so?  teardown resets them to None17:30
claygoh, and the whole class is using patch'd policies helper17:30
claygbut, yeah I could easily be wrong17:30
*** evrardjp has quit IRC17:35
*** evrardjp has joined #openstack-swift17:35
*** rdejoux has joined #openstack-swift18:02
*** mahatic has joined #openstack-swift18:34
*** ChanServ sets mode: +v mahatic18:34
timburkemahatic! o/18:49
timburkehave a good flight!18:49
mahatichey timburke, thanks! :)18:49
openstackgerritTim Burke proposed openstack/swift master: Clean up concurrent get requests once we have a response  https://review.opendev.org/70636119:33
*** guimaluf has joined #openstack-swift20:22
guimalufHi all, I'm struggling with something really wierd. 3 weeks ago I started upgrading my environment from Kilo to Queens. I have only Keystone and Swift. I think between Mitaka and Newton my storages started to throw Permission Denied error while writing metadata. http://paste.openstack.org/show/790217/ I've check everything I could, user, permissions, mount options. But I still getting this error. I wrote a20:31
guimalufpython script for creating files and writing metadata within the swift mount and everything works fines. Don't know what to do anymore.20:31
timburkeguimaluf, can you still read existing data? you checked user/group/perms -- was that at the root of the drive, or down in the object directory that gets created?20:37
timburkeyou can find the object dir with something like `swift-get-nodes /etc/swift/object.ring.gz AUTH_b462852304414da5a0c507ad035717e6/teste/drawing.pdf`20:37
guimaluftimburke, yes, everything is working fine. I can even download objects. but uploading no more. Users and perms were checked many time. I can do everything manually with swift user. This happens in the object directory20:41
guimaluftimburke, I've wrote a small python script to create file and write metadata in the specific object dir20:42
claygdo people still use nose to run swift tests?  https://gist.github.com/clayg/c275732de47d64c6f608cc1e63701b4520:48
clayg^ I'm having an encoding problems with the test.unit.proxy.controller.test_container module under py2 😢20:49
timburkeso what do the perms look like? 0755, or something else? is there a umask set? have you spot-checked that the swift processes really are running as the expected user?20:50
guimaluftimburke, directory stats: Access: (0744/drwxr--r--)  Uid: (  106/   swift)   Gid: (  113/   swift)20:51
tdasilvaclayg: weird, I don't run into that in my env: http://paste.openstack.org/show/790218/20:52
timburkeclayg, what's your $LANG look like?20:52
guimaluftimburke, yes. process are running as swift20:53
claygwow good thinking @timburke !!! I was LANG=C.UTF-8 and `LANG=C nosetests ...` worked20:53
guimalufswift     2370  4.3  1.5 1283060 46408 ?       Sl   16:27   3:44 /usr/bin/python2 /usr/bin/swift-object-server /etc/swift/object-server.conf20:53
clayg@tdasilva @timburke what is your LANG set to?20:53
tdasilvaen_US.UTF-820:53
timburkeclayg, that's like... the opposite of what i was expecting :-/20:54
guimaluftimburke, I've tried to reformat a full storage node to see what could it be... but with no clue...20:55
guimalufformat the disks I mean20:55
guimalufnot the full node... yet...20:55
clayg@guimaluf and it happened on all disks on all the nodes you upgraded?20:59
guimalufclayg, AFAIKT yes. because I'm getting the same error on the 3 copies tries in all upload21:00
*** zaitcev has joined #openstack-swift21:02
*** ChanServ sets mode: +v zaitcev21:02
timburkei wonder if it's something to do with O_TMPFILE support... what kernel are you on?21:02
guimaluftimburke, Linux storage-dev-01 4.15.0-88-generic #88-Ubuntu SMP Tue Feb 11 20:11:34 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux21:02
timburkedoes your script use O_TMPFILE like the object-server would?21:03
guimalufno21:03
guimalufbut I'm kin to reproduce in that way21:03
timburkejust thinking about how https://github.com/openstack/swift/blob/master/CHANGELOG#L1323-L1327 came in newton...21:04
guimalufhmmm21:04
guimalufI'll try that21:04
guimaluftimburke, can you point out where is the snippet of code that use that flag>?21:05
timburkemight want to check out https://github.com/openstack/swift/blob/stable/queens/swift/obj/diskfile.py#L2600-L262021:05
guimaluftrying to figure out how to reproduce that... if you have a easy handy way on mind21:12
guimalufplease share ;|21:12
guimaluftimburke, if you there is an way of debug direct on swift code I'm willing to change here for debug purpose21:15
guimalufknow*21:15
claygguimaluf: maybe just re-write utils.o_tmpfile_support to always return False?21:15
claygI guess maybe one question is what does `utils.o_tmpfile_supported` return currently?21:16
guimalufclayg, it wrote the file21:20
guimalufit worked... I'll change in both storages to see if upload happens21:21
claygguimaluf: I bet @tdasilva fixed you -> https://review.opendev.org/#/c/633646/21:22
patchbotpatch 633646 - swift - Change how O_TMPFILE support is detected (MERGED) - 5 patch sets21:22
claygI mean... that is to say you're not the only person to run in to problems how the o_tmpfile support was detected originally21:22
timburkeclayg, idk about that... i don't see EACCES in that list :-(21:23
guimalufclayg, upload done. it worked21:23
timburkeguimaluf, what OS is this? are you using xfs? i'm glad we've got a workaround, but i'd love to find out more about what's causing the error...21:24
guimaluftimburke, ubuntu 18.04 upgraded from 14.04. yes xfs21:25
claygtimburke: yeah maybe you're right!  could be un-related21:25
claygor it could express different on ubuntu - which might be worse - in that maybe this bug still exists?21:25
guimalufI'm really keen to help on report this21:25
claygI have 18.04 in dev, my kernel is like 4.15 - so guimaluf probably should be able to use o_tmpfile21:27
claygmaybe turning it off isn't the greatest workaround 🤔21:27
timburkewhat's your swift user's umask? what are the mount options you're using?21:27
claygstill it's hard/annoying to debug old code, the most useful think beyond being able to reproduce would be checking if the problem is expressed when using the latest tip of master swift21:28
guimaluftimburke, umask 0022;  xfs     noatime,nodiratime,logbufs=821:29
guimalufI'm on Rocky on that storage... I can move forward the upgrade and check if it's fixed21:29
guimalufbut may be related with this ubuntu upgrade process...21:30
guimalufclayg, I don't trust that a upgraded ubuntu is the same as a brand new 18.04 :/21:31
guimalufclayg, this patch you mentioned is present in what release?21:34
timburkestein21:34
timburke2.21.021:34
guimalufok! I'll move to it21:36
claygguimaluf: well, I guess timburke is right that the patch may not be what fixes you - we're not even sure if you should or should not be using linkat?  I guess ... currently using linkat didn't work (so there's that)21:37
claygwhat about just rebooting so all the kernel upgrady bits can fall into place?21:37
guimalufI'm on Rocky version... I'll try to move to stein21:38
guimalufclayg, I rebooted many times in this finding for the reason of this problem :/ so all kernel upgrades are done21:39
guimalufclayg, timburke thank you very much, fellows, for helping me with this, I hope after the upgrade things get back to normal, without hacking the swift code.21:46
mattoliverauMorning21:49
timburkeguimaluf, clayg: fwiw, here's what i came up with as a quick write-tester: http://paste.openstack.org/show/790221/21:56
guimaluftimburke, bang! same error22:00
guimaluftimburke, http://paste.openstack.org/show/790223/22:01
guimalufclayg, timburke, it haven't fixed on stein :( I'm leaving now, but I'll be back tomorrow and I can give you all information about my environment so we can nail this down. thank you very much for all help and effort22:02
*** guimaluf has quit IRC22:09
timburketomorrow guimaluf oughta add a `st = os.fstat(dfw._fd); print('%r (mode=0o%o)' % (st, st.st_mode))` between the write() and the put()...22:27
*** tkajinam has joined #openstack-swift22:28
*** rcernin has joined #openstack-swift23:01
*** DHE has quit IRC23:28
seongsoochomorning23:51

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