Tuesday, 2018-09-11

tdasilvamattoliverau: some PUT+POST discussion is still going00:04
tdasilvabut we are about to finish up00:05
*** zaitcev has quit IRC00:06
mattoliverauahh, nice. are we getting to the point get it in? the real question is, change in PUT path or acoles content-type (or what ever it was) version. Or adding a verson to the path for future changes (tims suggestion in patch). Anyway. I'll too late to this dicsussion so look forward to hearing what the community decides :)00:07
*** mikecmpbll has quit IRC00:38
*** gyee has quit IRC00:47
*** rcernin has quit IRC02:47
*** rcernin has joined #openstack-swift02:48
*** rcernin has quit IRC02:48
*** rcernin has joined #openstack-swift02:49
*** psachin has joined #openstack-swift02:54
openstackgerritMerged openstack/swift master: Remove post_as_copy deprecated warning from copy middleware  https://review.openstack.org/60143503:08
*** kei_yama has quit IRC03:51
*** kei_yama has joined #openstack-swift03:58
*** threestrands has joined #openstack-swift04:34
timburkemattoliverau: go spend time with your family! get what sleep you can ;-)04:54
timburkeyou don't need to check in with us yet04:54
mattoliverautimburke: so wednesday then ;) kk. have fun guys, wish I was there!04:55
timburkekota_: ok, one more thing to make p 592231 worthwhile as-is: following https://review.openstack.org/#/c/337960/ we can include the -N *in container listings*05:00
patchbotpatch 592231 - swift - s3api: Include '-' in S3 ETags of normal SLOs - 2 patch sets05:00
patchbotpatch 337960 - swift - Include SLO ETag in container updates (MERGED) - 10 patch sets05:00
timburkepotentially doing an extra GET on HEADs is maybe tolerable (it was *the way things were done* prior to https://review.openstack.org/#/c/347538/ ) but I'm not doing a GET per-object on container listings05:00
patchbotpatch 347538 - swift - Reduce backend requests for SLO If-Match / HEAD re... (MERGED) - 14 patch sets05:00
*** pcaruana has joined #openstack-swift05:00
timburkeand i don't really want to have that divergence between container listings and object GETs when the container listing is already giving me enough information to make better decisions...05:01
*** pcaruana has quit IRC05:09
*** cbartz has joined #openstack-swift05:41
*** mrjk__ has joined #openstack-swift06:12
*** mrjk has quit IRC06:14
*** tesseract has joined #openstack-swift06:51
*** rcernin has quit IRC07:08
*** pcaruana has joined #openstack-swift07:13
*** hseipp has joined #openstack-swift07:19
*** mikecmpbll has joined #openstack-swift07:22
*** geaaru has joined #openstack-swift07:38
*** ccamacho has joined #openstack-swift07:57
*** gkadam has joined #openstack-swift08:17
*** threestrands has quit IRC08:30
*** mikecmpbll has quit IRC08:34
*** mikecmpbll has joined #openstack-swift09:02
*** tesseract has quit IRC09:46
*** tesseract has joined #openstack-swift09:48
*** tesseract has quit IRC09:50
*** tesseract has joined #openstack-swift09:51
*** tesseract has quit IRC10:11
*** mikecmpbll has quit IRC11:01
*** mikecmpbll has joined #openstack-swift11:02
*** jistr is now known as jistr|call11:05
*** gkadam_ has joined #openstack-swift11:20
*** gkadam has quit IRC11:22
*** kei_yama has quit IRC11:52
*** jistr|call is now known as jistr12:02
*** hseipp has quit IRC12:03
*** mvenesio has joined #openstack-swift12:41
*** mikecmpbll has quit IRC13:02
*** mikecmpbll has joined #openstack-swift13:02
*** cbartz has quit IRC13:48
*** zaitcev has joined #openstack-swift13:49
*** ChanServ sets mode: +v zaitcev13:49
*** itlinux has quit IRC14:15
*** mvenesio has quit IRC14:17
*** mvenesio has joined #openstack-swift14:17
*** mvenesio has quit IRC14:21
*** pcaruana has quit IRC14:47
*** mikecmpbll has quit IRC15:01
*** mikecmpbll has joined #openstack-swift15:04
kota_good morning15:05
kota_timburke: thanks for that info. Let me check the conversations in the patch. I'd like to make a matrix what's needed for the multipart/slo manifest etag, container listing... and conditional GET for me to undestand completely.15:08
kota_it's obviously complicated because it requires cross api compatibility.15:08
kota_(and probably backword compat for swift itself)15:09
zaitcevcheck out this price https://toot.yukimochi.jp/@YUKIMOCHI/10070648757164185715:12
zaitcevLinode charges me $20 for same 2GB, although 2x CPU.15:13
*** itlinux has joined #openstack-swift15:14
kota_OVH!15:16
zaitcevIndeed!15:16
*** jistr is now known as jistr|off15:21
kota_thx notmyname for updating and getting merged https://review.openstack.org/60143515:23
patchbotpatch 601435 - swift - Remove post_as_copy deprecated warning from copy m... (MERGED) - 2 patch sets15:23
*** itlinux has quit IRC15:25
zaitcevclayg: https://etherpad.openstack.org/p/swift-put-post no significant disagreement with VERB was registered, only the vague unease about "standard"15:30
*** gyee has joined #openstack-swift15:30
*** UnfairFunction has joined #openstack-swift15:33
kota_nice summary for PUT-POST conversation15:34
*** mvenesio has joined #openstack-swift15:39
*** UnfairFunction has quit IRC15:45
openstackgerritTim Burke proposed openstack/swift master: py3: port encryption  https://review.openstack.org/60145415:51
*** itlinux has joined #openstack-swift15:52
openstackgerritTim Burke proposed openstack/swift master: Add py36 test job  https://review.openstack.org/60162215:53
*** mvenesio has quit IRC16:12
*** gkadam_ has quit IRC16:13
*** UnfairFunction has joined #openstack-swift16:14
*** mvenesio has joined #openstack-swift16:26
*** geaaru has quit IRC16:28
timburkezaitcev: so i'm thinking these two checks should maybe get swapped... https://github.com/openstack/liberasurecode/blob/master/src/erasurecode.c#L1103-L111016:33
timburkesurely we could look for magic even if there's no CRC, yeah?16:34
*** mikecmpbll has quit IRC16:34
claygtimburke: oh I like that!?  Can we just abandon checksumming of fragments written with old versions of libec?!16:34
zaitcevnot just swapped, but on the top16:34
clayg"warning old ass fragment failed checksum - it might be corrupt - or maybe we were stupid and your hardware is weird"16:35
timburke...sure? i think that /* libec_version must be bigger than 0 */ is valid regardless, though16:36
timburkethe main thing in my mind is that i want to look for magic, and if i can't find it, look for the byte-swapped magic -- if i still can't find it, fail; otherwise swap the world and continue16:38
claygit's weird that DiskFile.create managed the fd for DiskFileWriter - I think DiskFileWriter should have just had open & close methods16:48
*** mvenesio has quit IRC16:52
openstackgerritMerged openstack/swift master: Add IPv6 DSVM tests  https://review.openstack.org/59149716:54
*** mvenesio has joined #openstack-swift17:02
*** mikecmpbll has joined #openstack-swift17:24
*** UnfairFunction has quit IRC17:28
*** pcaruana has joined #openstack-swift17:32
openstackgerritJohn Dickinson proposed openstack/swift master: Disable the docs badge saying the docs aren't accurate  https://review.openstack.org/60165517:53
*** mvenesio has quit IRC17:58
*** mvenesio has joined #openstack-swift17:58
*** UnfairFunction has joined #openstack-swift18:00
*** UnfairFunction has quit IRC18:05
*** itlinux has quit IRC18:25
kota_I salvaged the fix of copy - delete-marker, here, https://review.openstack.org/#/c/396603/, that's too old so I have to do cleansing that...18:27
patchbotpatch 396603 - swift - WIP: Allowing copy objects which has special conte... - 1 patch set18:27
*** zaitcev has quit IRC18:36
*** ccamacho has quit IRC18:39
*** psachin has quit IRC19:11
*** itlinux has joined #openstack-swift19:33
openstackgerritTim Burke proposed openstack/liberasurecode master: Allow reading of little-endian frags on big-endian  https://review.openstack.org/60167919:49
* kota_ is back to his hotel to get short sleep fixing terrible jet lag19:51
* kota_ will come to the ptg around 3pm19:52
kota_again19:52
*** pcaruana has quit IRC19:55
timburke:-(20:10
timburke/TODO EDL we are ignoring the return codes here, fix that20:10
timburkeor rather, "//TODO EDL we are ignoring the return codes here, fix that"20:11
*** zaitcev has joined #openstack-swift20:11
*** ChanServ sets mode: +v zaitcev20:11
openstackgerritTim Burke proposed openstack/swift master: Add py36 test job  https://review.openstack.org/60162220:26
openstackgerritThiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift  https://review.openstack.org/60168620:29
tdasilvakota_: hope you can get some rest...20:30
openstackgerritThiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift  https://review.openstack.org/60168620:46
openstackgerritTim Burke proposed openstack/swift master: py3: port encryption  https://review.openstack.org/60145420:48
openstackgerritTim Burke proposed openstack/swift master: py3: port encryption  https://review.openstack.org/60145420:49
DHEis there a list of what is suppored on py3 so far? or should I just avoid py3 for now20:51
notmynameDHE: just avoid it for now. but if you look in the tox.ini py3 job, you'll see the only files we're testing. pretty much we don't have py3 support until that list goes to zero and just runs the test against everything20:52
notmynameDHE: that is... unless you want to help out with the port!!20:52
notmynamein that case, talk to timburke or zaitcev20:52
notmynamealso, it sure would be great for you to be here at the PTG in denver this week :-)20:53
DHEpython is not a language I would put on a resume... :/20:53
openstackgerritTim Burke proposed openstack/swift master: Drop neutron jobs from our gate  https://review.openstack.org/60181320:55
*** mvenesio has quit IRC20:56
DHEsorry, otherwise I'd consider it21:04
openstackgerritThiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift  https://review.openstack.org/60168621:04
openstackgerritPete Zaitcev proposed openstack/swift master: Replace MIME with PUT+POST for EC and Encryption  https://review.openstack.org/42791121:08
tdasilvatimburke and I are hanging out by the bar in the lobby in case anyone wants to join us. Nova+Cinder had a pre-scheduled meeting at Big Thompson21:18
timburkewe got displaced by a placement discussion21:23
openstackgerritThiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift  https://review.openstack.org/60168621:29
openstackgerritPete Zaitcev proposed openstack/swift master: Replace MIME with PUT+POST for EC and Encryption  https://review.openstack.org/42791121:37
*** zaitcev has quit IRC21:39
*** m_kazuhiro has joined #openstack-swift21:43
*** m_kazuhiro has quit IRC21:47
*** UnfairFunction has joined #openstack-swift21:48
*** UnfairFunction has quit IRC21:53
*** itlinux has quit IRC21:57
claygalecuyer:  rledisez: wasn't there a patch for DiskFile that redid some of the constructors in diskfile to use swift concepts like "device" and "partition" instead of file-system paths like "datadir" ???22:19
claygkota_: I thought it was patch 447129 - but that's just the entrypoint stuff (also good!)22:19
patchbothttps://review.openstack.org/#/c/447129/ - swift - Configure diskfile per storage policy - 20 patch sets22:19
kota_clayg: redid might be in later patch? e.g. https://review.openstack.org/#/c/571763/2/swift/obj/diskfile.py22:22
patchbotpatch 571763 - swift - WIP - LOSF - 2 patch sets22:22
openstackgerritThiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift  https://review.openstack.org/60168622:27
openstackgerritMerged openstack/swift master: Disable the docs badge saying the docs aren't accurate  https://review.openstack.org/60165522:34
openstackgerritMerged openstack/swift master: Drop neutron jobs from our gate  https://review.openstack.org/60181322:34
*** zaitcev has joined #openstack-swift22:35
*** ChanServ sets mode: +v zaitcev22:35
*** rcernin has joined #openstack-swift22:43
openstackgerritThiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift  https://review.openstack.org/60168622:46
*** mvenesio has joined #openstack-swift22:47
*** itlinux has joined #openstack-swift22:50
*** mvenesio has quit IRC22:52
openstackgerritThiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift  https://review.openstack.org/60168622:56
openstackgerritPete Zaitcev proposed openstack/swift master: Let error messages to be normal strings again  https://review.openstack.org/60187223:02
openstackgerritKota Tsuyuzaki proposed openstack/swift master: WIP: Allowing copy objects which has special content type  https://review.openstack.org/39660323:06
* kota_ make the delete-marker fix revive. I do not check it in detail tho.23:07
kota_clayg, timburke: that's a staff we talked in the last night.23:07
*** mikecmpbll has quit IRC23:10
claygwho is jaegerandi ???23:13
openstackgerritThiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift  https://review.openstack.org/60168623:14
*** joeljwright has quit IRC23:17
*** mark-mcardle has quit IRC23:17
*** mark-mcardle has joined #openstack-swift23:17
*** joeljwright has joined #openstack-swift23:17
*** ChanServ sets mode: +v joeljwright23:17
zaitcevclayg, https://kilodesignsummit.sched.com/jaegerandi apparently23:19
*** zaitcev has quit IRC23:25
*** d0ugal has quit IRC23:29
*** d0ugal has joined #openstack-swift23:38
openstackgerritMerged openstack/swift master: Add py36 test job  https://review.openstack.org/60162223:40
mattoliveraumorning23:51

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