openstackgerrit | Merged openstack/swift master: added some bootstrapping info to the top of the SAIO doc https://review.openstack.org/602203 | 00:35 |
---|---|---|
*** gyee has quit IRC | 00:59 | |
*** chocolate-elvis has joined #openstack-swift | 02:16 | |
*** kei_yama has quit IRC | 03:48 | |
*** kei_yama has joined #openstack-swift | 04:03 | |
openstackgerrit | Pete Zaitcev proposed openstack/swift master: Replace MIME with PUT+POST for EC and Encryption https://review.openstack.org/427911 | 05:03 |
*** hseipp has joined #openstack-swift | 06:47 | |
*** rcernin has quit IRC | 07:01 | |
*** tesseract has joined #openstack-swift | 07:01 | |
*** kei_yama has quit IRC | 07:22 | |
*** kei_yama has joined #openstack-swift | 07:27 | |
*** cbartz has joined #openstack-swift | 07:29 | |
*** geaaru has joined #openstack-swift | 07:37 | |
openstackgerrit | Tim Burke proposed openstack/swift master: s3api: Include '-' in multipart ETags https://review.openstack.org/575860 | 07:54 |
*** d0ugal has joined #openstack-swift | 08:20 | |
*** cbartz has quit IRC | 08:25 | |
*** cbartz has joined #openstack-swift | 08:27 | |
*** d0ugal has quit IRC | 08:29 | |
*** d0ugal has joined #openstack-swift | 08:29 | |
*** d0ugal has quit IRC | 08:35 | |
*** d0ugal has joined #openstack-swift | 08:36 | |
*** d0ugal has quit IRC | 08:44 | |
*** d0ugal has joined #openstack-swift | 08:44 | |
openstackgerrit | Kota Tsuyuzaki proposed openstack/swift master: s3api: Include '-' in multipart ETags https://review.openstack.org/575860 | 10:29 |
*** d0ugal has quit IRC | 11:07 | |
*** d0ugal has joined #openstack-swift | 11:08 | |
openstackgerrit | Merged openstack/swift master: s3api: Include '-' in multipart ETags https://review.openstack.org/575860 | 11:30 |
*** ianychoi has quit IRC | 12:04 | |
*** ianychoi has joined #openstack-swift | 12:11 | |
*** szaher_ has joined #openstack-swift | 12:24 | |
*** cbartz has quit IRC | 12:58 | |
*** szaher_ has quit IRC | 13:02 | |
*** kei_yama has quit IRC | 13:04 | |
*** szaher has joined #openstack-swift | 13:05 | |
*** mvenesio has joined #openstack-swift | 13:50 | |
*** d0ugal has quit IRC | 13:56 | |
*** d0ugal has joined #openstack-swift | 13:56 | |
*** zaitcev has joined #openstack-swift | 13:57 | |
*** ChanServ sets mode: +v zaitcev | 13:57 | |
*** fatema_ has joined #openstack-swift | 14:01 | |
*** pcaruana has joined #openstack-swift | 14:28 | |
*** d0ugal has quit IRC | 14:59 | |
*** itlinux has quit IRC | 15:09 | |
*** gyee has joined #openstack-swift | 15:15 | |
*** fatema has joined #openstack-swift | 15:24 | |
*** sorrison has joined #openstack-swift | 15:39 | |
openstackgerrit | Thiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift https://review.openstack.org/601686 | 15:47 |
timburke | kota_: those new failures fell out of https://review.openstack.org/#/c/577941/ i think :-/ | 15:51 |
patchbot | patch 577941 - swift - s3_acl: Require swift_owner authz to create buckets (MERGED) - 3 patch sets | 15:51 |
*** Sukhdev has joined #openstack-swift | 15:51 | |
kota_ | let me look at | 15:51 |
timburke | the thing is, it never really made sense that test:tester3 could create buckets via s3 -- they can't create containers via swift! | 15:52 |
timburke | maybe we could add a test:tester2 to the sample confs who *is* an account owner? | 15:52 |
kota_ | i just wrote up some assertions to confirm my understanding like https://gist.github.com/bloodeagle40234/b7f700300b4cab1d664abd5ed1de1e88 | 15:53 |
*** geaaru has quit IRC | 15:58 | |
*** hseipp has quit IRC | 15:58 | |
kota_ | alright | 16:00 |
*** itlinux has joined #openstack-swift | 16:01 | |
kota_ | https://gist.github.com/bloodeagle40234/b7f700300b4cab1d664abd5ed1de1e88 | 16:01 |
kota_ | update the gist | 16:01 |
*** sorrison has quit IRC | 16:11 | |
*** sorrison has joined #openstack-swift | 16:15 | |
*** itlinux is now known as itlinux-away | 16:15 | |
kota_ | timburke: you seems to be right. non-owner, but .admin account can see BucketAlreadyExists when the bucket created by another owner | 16:15 |
*** itlinux-away is now known as itlinux | 16:17 | |
*** itlinux is now known as itlinux-away | 16:17 | |
openstackgerrit | Clay Gerrard proposed openstack/swift master: Add a chunks_finished to BaseDiskFileWriter https://review.openstack.org/602388 | 16:24 |
*** itlinux-away is now known as itlinux | 16:25 | |
*** itlinux is now known as itlinux-away | 16:25 | |
*** itlinux-away is now known as itlinux | 16:25 | |
*** itlinux is now known as itlinux-away | 16:25 | |
*** itlinux-away is now known as itlinux | 16:27 | |
timburke | kota_: this reminds me that https://review.openstack.org/#/c/575829/ is a thing that hasn't landed yet... | 16:48 |
patchbot | patch 575829 - swift - Change PUT bucket conflict error - 4 patch sets | 16:48 |
kota_ | timburke: :P | 16:49 |
timburke | it also reminds me that we should probably have func tests both with and without s3_acl enabled | 16:51 |
openstackgerrit | Thiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift https://review.openstack.org/601686 | 16:51 |
kota_ | hah ->> https://review.openstack.org/#/c/575829/4/test/functional/s3api/test_bucket.py you already know the AccessDenied case! | 16:52 |
patchbot | patch 575829 - swift - Change PUT bucket conflict error - 4 patch sets | 16:52 |
kota_ | exactly, i don't think it can pass with non s3_acl case because it dosn't have the owner info. | 16:53 |
timburke | the weird thing to me is why i ever thought i needed AccessDenied in https://review.openstack.org/#/c/577941/3/test/functional/s3api/test_bucket.py | 16:54 |
patchbot | patch 577941 - swift - s3_acl: Require swift_owner authz to create buckets (MERGED) - 3 patch sets | 16:54 |
sorrison | notmyname: I have some working code for s3 keystone token cacheing but need some help with how to do cacheing, can I come have a chat with you guys today sometime? | 16:54 |
openstackgerrit | Clay Gerrard proposed openstack/swift master: Add a chunks_finished to BaseDiskFileWriter https://review.openstack.org/602388 | 16:55 |
openstackgerrit | Clay Gerrard proposed openstack/swift master: Refactor obj.server.ObjectController.PUT https://review.openstack.org/602205 | 16:55 |
clayg | ^ timburke zaitcev - ok I think I have the diskfile + object-server ready for the STAGE/UPDATE/COMMIT change now :D | 16:56 |
notmyname | sorrison: yes! we're in Big Thompson all day today | 16:56 |
kota_ | sorrison: I don't think s3api on top of swift can cache the keystone token because the actual aws signature requires the check with the secret and the sort of request information. | 16:58 |
sorrison | kota_: I'll come over in about 10 mins and I can explain my approach | 16:59 |
notmyname | kota_: maybe. sorrison has some good ideas and his goal is to make it so that s3api requests do not have to hit keystone for every request | 16:59 |
kota_ | sorrison: alright, I'm in the room. | 16:59 |
zaitcev | notmyname: it sounds impossible just like Kota just explained, but | 16:59 |
*** tesseract has quit IRC | 17:00 | |
zaitcev | It may be possible to cache hmacs for identical requests, like when some stupid app keeps hitting Swift with the same request again and again, so that only the timestamp changes | 17:00 |
notmyname | wait until sorrison gets here to explain it. :) IIRC, there is a keystone endpoint where he's able to get some creds info and do what he wants | 17:02 |
timburke | zaitcev: request timestamp gets baked into the signature to defend against replay attacks :P | 17:02 |
timburke | of course, it's still reliant upon client headers (Date and/or x-aws-date iirc?) but there's only a couple-minutes window where you could send *exactly* the same request | 17:03 |
timburke | presigned urls are another story, of course | 17:04 |
*** sorrison has quit IRC | 17:05 | |
zaitcev | timburke: yeah... so when replay is desired, then... well we only need to remain in suspense for a few minutes | 17:05 |
*** sorrison has joined #openstack-swift | 17:05 | |
*** e0ne has joined #openstack-swift | 17:06 | |
e0ne | notmyname: hi John. will you have a time tomorrow to discuss swift and horizon integration? | 17:07 |
*** sorrison has quit IRC | 17:07 | |
*** sorrison has joined #openstack-swift | 17:08 | |
notmyname | e0ne: sure. or this afternoon. whatever is good for you | 17:10 |
e0ne | notmyname: I'm sorry, can't do it today. I'll catch you tomorrow | 17:11 |
notmyname | e0ne: anything specific you'd like to talk about? if there's somthing I can prepare for.... | 17:12 |
e0ne | notmyname: I would like to fill https://etherpad.openstack.org/p/horizon-feature-gap for the Swift to prepare TODO list to the Horizon team | 17:13 |
notmyname | ok | 17:13 |
openstackgerrit | Alexandre Lécuyer proposed openstack/swift master: WIP - Abstract file and directory operations in replicator/reconstructor https://review.openstack.org/602402 | 17:16 |
openstackgerrit | Thiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift https://review.openstack.org/601686 | 17:21 |
clayg | timburke: right! and probably some tests too | 17:22 |
*** itlinux is now known as itlinux-away | 17:27 | |
*** itlinux-away is now known as itlinux | 17:29 | |
*** itlinux is now known as itlinux-away | 17:29 | |
*** sorrison has quit IRC | 17:30 | |
*** itlinux-away is now known as itlinux | 17:32 | |
*** itlinux is now known as itlinux-away | 17:32 | |
*** sorrison has joined #openstack-swift | 17:32 | |
*** itlinux-away is now known as itlinux | 17:34 | |
*** itlinux is now known as itlinux-away | 17:34 | |
*** itlinux-away is now known as itlinux | 17:38 | |
*** itlinux is now known as itlinux-away | 17:38 | |
*** itlinux-away is now known as itlinux | 17:41 | |
notmyname | fatema: http://paste.openstack.org/show/730044/ | 17:46 |
openstackgerrit | Tim Burke proposed openstack/swift master: Change PUT bucket conflict error https://review.openstack.org/575829 | 17:46 |
openstackgerrit | Tim Burke proposed openstack/swift master: Add another user for s3api func tests https://review.openstack.org/602411 | 17:46 |
*** e0ne has quit IRC | 17:48 | |
*** e0ne has joined #openstack-swift | 17:49 | |
openstackgerrit | Tim Burke proposed openstack/swift master: Change PUT bucket conflict error https://review.openstack.org/575829 | 17:54 |
openstackgerrit | Tim Burke proposed openstack/swift master: Add another user for s3api func tests https://review.openstack.org/602411 | 18:00 |
openstackgerrit | Tim Burke proposed openstack/swift master: Change PUT bucket conflict error https://review.openstack.org/575829 | 18:00 |
kota_ | nice | 18:04 |
*** sorrison has quit IRC | 18:10 | |
openstackgerrit | Clay Gerrard proposed openstack/swift master: Add a chunks_finished to BaseDiskFileWriter https://review.openstack.org/602388 | 18:12 |
openstackgerrit | Clay Gerrard proposed openstack/swift master: Refactor obj.server.ObjectController.PUT https://review.openstack.org/602205 | 18:12 |
openstackgerrit | Thiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift https://review.openstack.org/601686 | 18:12 |
clayg | so the proxy.test_mem_server tests are a little janky - we import a half-dozen different TestCases and have at least a couple different ways of making the mem_server versions skip/pass when the test_server versions expect the behavior of the real DiskFile | 18:14 |
clayg | very rarely the tests expect "go look on the disk" - most of the time it's more subtle like "opening this diskfile in this policy should raise DiskFileDoesNotExist with a timestamp == tombstone" | 18:15 |
clayg | and not only does mem_server not supprot storage policies - it doesn't really support tombstones!? 🤷♂️ | 18:15 |
*** Sukhdev has quit IRC | 18:23 | |
*** zaitcev has quit IRC | 18:30 | |
*** e0ne has quit IRC | 18:31 | |
*** fatema has quit IRC | 18:32 | |
*** rcernin has joined #openstack-swift | 19:00 | |
*** m_kazuhiro has joined #openstack-swift | 19:12 | |
*** itlinux is now known as itlinux-away | 19:22 | |
*** e0ne has joined #openstack-swift | 19:23 | |
*** itlinux-away is now known as itlinux | 19:23 | |
*** itlinux is now known as itlinux-away | 19:24 | |
*** itlinux-away is now known as itlinux | 19:24 | |
*** itlinux is now known as itlinux-away | 19:24 | |
*** itlinux-away is now known as itlinux | 19:24 | |
*** itlinux is now known as itlinux-away | 19:24 | |
*** itlinux-away is now known as itlinux | 19:25 | |
*** itlinux is now known as itlinux-away | 19:26 | |
*** itlinux-away is now known as itlinux | 19:33 | |
*** itlinux is now known as itlinux-away | 19:33 | |
*** itlinux-away is now known as itlinux | 19:36 | |
*** itlinux is now known as itlinux-away | 19:36 | |
*** fatema has joined #openstack-swift | 19:38 | |
*** itlinux-away is now known as itlinux | 19:49 | |
*** itlinux is now known as itlinux-away | 19:50 | |
*** itlinux-away is now known as itlinux | 19:50 | |
*** sorrison has joined #openstack-swift | 19:55 | |
openstackgerrit | Kota Tsuyuzaki proposed openstack/swift master: Add another user for s3api func tests https://review.openstack.org/602411 | 19:55 |
*** zaitcev has joined #openstack-swift | 19:57 | |
*** ChanServ sets mode: +v zaitcev | 19:57 | |
openstackgerrit | Kota Tsuyuzaki proposed openstack/swift master: Add another user for s3api func tests https://review.openstack.org/602411 | 20:00 |
kota_ | mem_server always seems to bite us when we want to change the backends. | 20:03 |
kota_ | change something of the backends. | 20:03 |
timburke | my impression was that it's *designed* to be that way, to make us think harder about changing the diskfile API and consider poor projects like swift-on-file | 20:09 |
zaitcev | Or Kinetic | 20:15 |
zaitcev | also poor | 20:15 |
*** Sukhdev has joined #openstack-swift | 20:16 | |
*** sorrison has quit IRC | 20:21 | |
*** sorrison has joined #openstack-swift | 20:25 | |
openstackgerrit | Thiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift https://review.openstack.org/601686 | 20:26 |
openstackgerrit | Kota Tsuyuzaki proposed openstack/swift master: Add ceph-s3 test non-voting job https://review.openstack.org/564700 | 20:29 |
*** itlinux is now known as itlinux-away | 20:33 | |
*** itlinux-away is now known as itlinux | 20:35 | |
*** pcaruana has quit IRC | 20:38 | |
*** sorrison has quit IRC | 20:39 | |
*** sorrison has joined #openstack-swift | 20:39 | |
*** e0ne has quit IRC | 20:49 | |
*** e0ne has joined #openstack-swift | 20:50 | |
*** rcernin has quit IRC | 20:51 | |
kota_ | yey, https://review.openstack.org/#/c/602411/ passed with all green | 20:51 |
patchbot | patch 602411 - swift - Add another user for s3api func tests - 4 patch sets | 20:51 |
*** itlinux has quit IRC | 20:59 | |
*** e0ne has quit IRC | 20:59 | |
*** itlinux has joined #openstack-swift | 21:06 | |
*** e0ne has joined #openstack-swift | 21:07 | |
*** e0ne has quit IRC | 21:12 | |
*** sorrison has quit IRC | 21:13 | |
*** itlinux has quit IRC | 21:15 | |
*** sorrison has joined #openstack-swift | 21:15 | |
*** itlinux has joined #openstack-swift | 21:16 | |
*** e0ne has joined #openstack-swift | 21:20 | |
kota_ | http://logs.openstack.org/00/564700/34/check/swift-probetests-centos-7/c367f8b/job-output.txt.gz#_2018-09-13_20_57_09_654020 | 21:22 |
kota_ | it looks like we still need to pin eventlet? | 21:22 |
*** mvenesio has quit IRC | 21:26 | |
openstackgerrit | Kazuhiro MIYAHARA proposed openstack/swift master: Add object-expirer new mode to execute tasks from general task queue https://review.openstack.org/517389 | 21:26 |
*** rcernin has joined #openstack-swift | 21:29 | |
*** itlinux has quit IRC | 21:36 | |
*** itlinux has joined #openstack-swift | 21:37 | |
openstackgerrit | Clay Gerrard proposed openstack/swift master: Refactor obj.server.ObjectController.PUT https://review.openstack.org/602205 | 21:45 |
openstackgerrit | Clay Gerrard proposed openstack/swift master: Allow EC to ignore invalid request ETag https://review.openstack.org/602467 | 21:45 |
kota_ | http://logs.openstack.org/00/564700/34/check/swift-probetests-centos-7/37f9f3f/job-output.txt.gz BrokenPipe again :/ | 21:47 |
*** itlinux has quit IRC | 21:50 | |
*** fatema has quit IRC | 21:57 | |
*** itlinux has joined #openstack-swift | 22:01 | |
*** itlinux has quit IRC | 22:02 | |
openstackgerrit | Thiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift https://review.openstack.org/601686 | 22:02 |
*** fatema_ has quit IRC | 22:09 | |
*** zaitcev has quit IRC | 22:29 | |
*** e0ne has quit IRC | 22:33 | |
*** zaitcev has joined #openstack-swift | 22:37 | |
*** ChanServ sets mode: +v zaitcev | 22:37 | |
*** e0ne has joined #openstack-swift | 22:41 | |
*** e0ne has quit IRC | 22:46 | |
*** zaitcev has quit IRC | 22:51 | |
*** fatema has joined #openstack-swift | 22:55 | |
*** zaitcev has joined #openstack-swift | 22:57 | |
*** ChanServ sets mode: +v zaitcev | 22:57 | |
openstackgerrit | Tim Burke proposed openstack/swift master: Add punch_hole utility function https://review.openstack.org/548034 | 23:04 |
*** sorrison has quit IRC | 23:10 | |
*** kei_yama has joined #openstack-swift | 23:12 | |
* kota_ is looking at https://review.openstack.org/548034 | 23:16 | |
patchbot | patch 548034 - swift - Add punch_hole utility function - 11 patch sets | 23:16 |
*** sorrison has joined #openstack-swift | 23:22 | |
mattoliverau | morning | 23:25 |
StevenK | mattoliverau: No it isn't, it's 5:30pm yesterday | 23:26 |
*** Sukhdev has quit IRC | 23:27 | |
mattoliverau | StevenK: wow, is it.. I am sleep deprived :P | 23:28 |
StevenK | Troll successful :-P | 23:28 |
openstackgerrit | Thiago da Silva proposed openstack/swift master: WIP: add multinode func test to swift https://review.openstack.org/601686 | 23:31 |
kota_ | mattoliverau: o/ | 23:34 |
notmyname | tdasilva: it's getting fixed | 23:38 |
tdasilva | it's not because of the white space? | 23:39 |
tdasilva | mattoliverau: o/ | 23:40 |
notmyname | tdasilva: nope | 23:45 |
*** fatema has quit IRC | 23:51 | |
openstackgerrit | John Dickinson proposed openstack/swift master: WIP: add multinode func test to swift https://review.openstack.org/601686 | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!