*** itlinux has joined #openstack-swift | 00:35 | |
thurloat | notmyname: thanks for the thorough answer. will definitely be readon | 00:38 |
---|---|---|
*** wes_dillingham has quit IRC | 00:40 | |
*** wes_dillingham has joined #openstack-swift | 00:41 | |
*** lnxnut has joined #openstack-swift | 00:42 | |
*** tovin07_ has joined #openstack-swift | 00:46 | |
*** lnxnut has quit IRC | 00:53 | |
kota_ | good morning | 01:21 |
*** MeltedLux has quit IRC | 01:25 | |
*** cshastri has joined #openstack-swift | 01:26 | |
*** itlinux has quit IRC | 01:29 | |
*** itlinux has joined #openstack-swift | 01:29 | |
*** MeltedLux has joined #openstack-swift | 01:36 | |
*** itlinux has quit IRC | 01:48 | |
*** lnxnut has joined #openstack-swift | 01:50 | |
*** szaher has quit IRC | 01:57 | |
*** MeltedLux has quit IRC | 01:59 | |
*** lnxnut has quit IRC | 02:01 | |
*** szaher has joined #openstack-swift | 02:07 | |
*** itlinux has joined #openstack-swift | 02:31 | |
*** itlinux has quit IRC | 02:44 | |
*** lnxnut has joined #openstack-swift | 02:59 | |
*** lnxnut has quit IRC | 03:07 | |
*** chsc has joined #openstack-swift | 03:10 | |
*** chsc has joined #openstack-swift | 03:10 | |
*** chsc has quit IRC | 03:15 | |
*** gyee has joined #openstack-swift | 03:37 | |
*** gyee has quit IRC | 03:38 | |
*** kei_yama has quit IRC | 03:45 | |
*** Dinesh_Bhor has joined #openstack-swift | 03:47 | |
*** gkadam has joined #openstack-swift | 03:52 | |
*** kei_yama has joined #openstack-swift | 04:02 | |
*** lnxnut has joined #openstack-swift | 04:04 | |
*** SkyRocknRoll has joined #openstack-swift | 04:07 | |
*** mingyu has quit IRC | 04:17 | |
*** mingyu has joined #openstack-swift | 04:19 | |
*** mingyu has quit IRC | 04:19 | |
*** abhitechie has joined #openstack-swift | 04:20 | |
*** lnxnut has quit IRC | 04:21 | |
*** pcaruana has joined #openstack-swift | 05:01 | |
*** psachin has joined #openstack-swift | 05:04 | |
*** mabrams has joined #openstack-swift | 05:04 | |
*** skudlik has joined #openstack-swift | 05:13 | |
*** lnxnut has joined #openstack-swift | 05:18 | |
*** mingyu has joined #openstack-swift | 05:20 | |
*** pcaruana has quit IRC | 05:23 | |
*** mingyu has quit IRC | 05:25 | |
*** lnxnut has quit IRC | 05:28 | |
*** mat128 has quit IRC | 05:36 | |
*** rcernin has joined #openstack-swift | 05:53 | |
*** ChubYann has quit IRC | 05:54 | |
*** mingyu has joined #openstack-swift | 05:55 | |
*** mvk has quit IRC | 06:13 | |
*** tovin07_ has quit IRC | 06:22 | |
*** lnxnut has joined #openstack-swift | 06:24 | |
*** hoonetorg has quit IRC | 06:27 | |
*** lnxnut has quit IRC | 06:34 | |
*** hoonetorg has joined #openstack-swift | 06:41 | |
*** pcaruana has joined #openstack-swift | 06:44 | |
*** MeltedLux has joined #openstack-swift | 07:09 | |
*** tesseract has joined #openstack-swift | 07:17 | |
*** oshritf has joined #openstack-swift | 07:19 | |
*** SkyRocknRoll has quit IRC | 07:20 | |
*** oshritf has quit IRC | 07:21 | |
*** oshritf has joined #openstack-swift | 07:26 | |
*** oshritf has quit IRC | 07:29 | |
*** lnxnut has joined #openstack-swift | 07:31 | |
*** oshritf has joined #openstack-swift | 07:32 | |
*** SkyRocknRoll has joined #openstack-swift | 07:37 | |
*** Guest36 has joined #openstack-swift | 07:39 | |
*** oshritf has quit IRC | 07:40 | |
*** lnxnut has quit IRC | 07:41 | |
*** oshritf has joined #openstack-swift | 07:43 | |
*** oshritf has quit IRC | 07:53 | |
*** Guest36 has quit IRC | 07:57 | |
*** oshritf has joined #openstack-swift | 08:02 | |
*** oshritf has quit IRC | 08:06 | |
*** mvk has joined #openstack-swift | 08:10 | |
acoles | good morning | 08:12 |
*** oshritf has joined #openstack-swift | 08:14 | |
*** frekoya_ has joined #openstack-swift | 08:22 | |
*** wes_dillingham has quit IRC | 08:27 | |
*** mabrams has quit IRC | 08:31 | |
*** frekoya_ has quit IRC | 08:31 | |
kota_ | good morning acoles | 08:33 |
*** wes_dillingham has joined #openstack-swift | 08:33 | |
*** frekoya_ has joined #openstack-swift | 08:37 | |
*** mvk has quit IRC | 08:38 | |
*** lnxnut has joined #openstack-swift | 08:39 | |
openstackgerrit | Alistair Coles proposed openstack/swift feature/deep: minor follow up on cleave unit tests https://review.openstack.org/509109 | 08:40 |
acoles | kota_: o/ | 08:40 |
kota_ | o/ | 08:41 |
*** spectr has joined #openstack-swift | 08:41 | |
*** frekoya_ has quit IRC | 08:43 | |
*** mahatic has joined #openstack-swift | 08:45 | |
openstackgerrit | Alistair Coles proposed openstack/swift feature/deep: make broker get_shard_range_rows private https://review.openstack.org/509111 | 08:47 |
openstackgerrit | Alistair Coles proposed openstack/swift feature/deep: make broker get_shard_range_rows private https://review.openstack.org/509111 | 08:48 |
*** lnxnut has quit IRC | 08:48 | |
*** mvk has joined #openstack-swift | 08:54 | |
*** abhitechie has quit IRC | 09:07 | |
*** abhinavtechie has joined #openstack-swift | 09:07 | |
openstackgerrit | HCLTech-SSW proposed openstack/swift master: Add ability to undelete an account. https://review.openstack.org/507808 | 09:13 |
*** mvk has quit IRC | 09:20 | |
*** kei_yama has quit IRC | 09:20 | |
*** mvk has joined #openstack-swift | 09:34 | |
*** abhinavtechie has quit IRC | 09:44 | |
*** geaaru has joined #openstack-swift | 09:46 | |
*** lnxnut has joined #openstack-swift | 09:46 | |
*** oshritf has quit IRC | 09:54 | |
*** lnxnut has quit IRC | 09:56 | |
*** mingyu has quit IRC | 10:18 | |
*** wes_dillingham has quit IRC | 10:46 | |
*** cshastri has quit IRC | 10:50 | |
*** lnxnut has joined #openstack-swift | 10:53 | |
*** lnxnut has quit IRC | 11:02 | |
openstackgerrit | Kota Tsuyuzaki proposed openstack/swift master: Follow up for symlink implementaion https://review.openstack.org/509149 | 11:06 |
*** spectr has quit IRC | 11:07 | |
*** mingyu has joined #openstack-swift | 11:09 | |
kota_ | tdasilva, timburke: I'm starting to look up the symlink patch and I've concerned on some point of view on the docs that may be wrong or not enough for the spec. I'd like to get your help on the english terms and docs for that if you could have time. | 11:11 |
*** abhitechie has joined #openstack-swift | 11:11 | |
kota_ | acoles: maybe you also can help me because I think you had some discussion on symlink at past several mid-cycles ;) | 11:12 |
acoles | kota_: I can try | 11:12 |
kota_ | acoles: thank you so much! | 11:13 |
acoles | kota_: what's the question? | 11:13 |
kota_ | let me check my comments on https://review.openstack.org/#/c/232162/59/swift/common/middleware/symlink.py | 11:15 |
patchbot | patch 232162 - swift - Symlink implementation. | 11:15 |
*** abhitechie has quit IRC | 11:15 | |
*** abhitechie has joined #openstack-swift | 11:15 | |
kota_ | I had some comments on the patch to correct (or get understand correctly) but one of questions that could be issue is the behavior on OPTIONS | 11:17 |
kota_ | acoles: the docs just says, "the request will not be redirected to the target object." | 11:17 |
kota_ | however, the linked (i.e. target actual) object could be in a container different cors setting rather than the symlink object. | 11:18 |
kota_ | i.e. an actual verb (e.g. GET/HEAD) after pre-flight OPTIONS can be failed with 4xx even the client got allowed the methods by OPTIONS request | 11:21 |
kota_ | sorry, not yet tested, just I thought when reading the docs | 11:22 |
kota_ | and not sure if it's by design. | 11:22 |
*** abhitechie has quit IRC | 11:22 | |
kota_ | acoles: I don't hurry so much for the answer right now, sorry. Just I'd like to notify you to make the patch in your rader. | 11:25 |
kota_ | acoles: And also, I know I have some homework from you for composite ring patches since ptg :/ sorry. | 11:26 |
acoles | kota_: in terms of the English at line 73-74, I read that as the OPTIONS request will not be redirected, so it does look like you analysis is correct | 11:28 |
*** abhitechie has joined #openstack-swift | 11:29 | |
acoles | I'm not familiar with the design decision though | 11:29 |
kota_ | acoles: ok, thanks. | 11:31 |
* kota_ is feeling to need clear head and eyes for deeper consideration, so being off from office | 11:32 | |
acoles | kota_: sorry my machine froze on me for a while then | 11:33 |
acoles | kota_: ok, have a good night | 11:33 |
kota_ | acoles: oh i see. | 11:33 |
kota_ | acoles: thanks for taking time it :D | 11:33 |
acoles | lines 53-56 I think it is trying to explain why you should not POST to a symlink | 11:34 |
kota_ | yeah, and it's the reason why the response will be 3xx. | 11:35 |
acoles | the POST cannot be guaranteed to be redirected to the target because at the time the POST arrives at an object server the current .data file may not yet be the symlink (if the symlink PUT failed to that server). so because POST redirection cannot be guaranteed, it is never attempted. At least, that's the reasoning I recall. | 11:36 |
kota_ | ok... so the basic idea, the client SHOULD NOT POST to a symlink as you said. | 11:39 |
kota_ | i like it | 11:39 |
kota_ | probably, the reason why I felt a little bit complicated is from the previous sentence | 11:39 |
kota_ | POST request to a symlink must be accompanied of the query parameter | 11:40 |
kota_ | ``?symlink=post`` to have the new metadata applied to the symlink object | 11:40 |
kota_ | itself | 11:40 |
acoles | kota_: the symlink=post param is new to me :) | 11:40 |
kota_ | whooaa!? | 11:41 |
kota_ | anyway | 11:41 |
acoles | kota_: it has been a long time since I reviewed this, maybe i just forgot | 11:41 |
kota_ | reading from top to bottom, it looks like as POST is supported.. but actually, it seems like the POST result symlink will be never applied anytime on normal GET/HEAD | 11:42 |
kota_ | POST result to symlink | 11:42 |
kota_ | normal GET/HEAD to symlink (i.e. GET/HEAD the target via symlink) | 11:43 |
kota_ | so that, the important information seems that you said above, don't POST to symlink, if you did it, Swift will notify you that it's symlink via 3xx. | 11:44 |
kota_ | imo | 11:44 |
acoles | kota_: my understanding is that the POST metadata will always be applied to the symlink (not target) but without the symlink=post param the response code is changed to a redirect | 11:44 |
kota_ | acoles: I think you're true. | 11:45 |
acoles | so then a GET or HEAD will not get that POSTed metadata because it will be redirected to the target | 11:45 |
acoles | unless it has symlink=get | 11:46 |
kota_ | yes | 11:46 |
acoles | kota_: I do remember we filled several whiteboards in several hackathons convincing ourselves that clients must redirect the POST to the target, not swift :) | 11:47 |
acoles | (I remember the whiteboard in Tokyo summit!!!) | 11:47 |
kota_ | then, general question is, is current docs | 11:47 |
kota_ | oh, sorry, i pushed enter key before I filled my sentence | 11:47 |
acoles | hehe I do that all the | 11:48 |
acoles | time :D | 11:48 |
kota_ | general question is, does the current docs describe the spec in straightforward? | 11:48 |
kota_ | i didn't feel like it, but not sure because I'm not native. | 11:49 |
acoles | kota_: hmm, I am not sure if this line 51 is true "A POST request to a symlink must be accompanied of the query parameter | 11:50 |
acoles | ``?symlink=post`` to have the new metadata applied to the symlink object | 11:50 |
acoles | itself." | 11:50 |
kota_ | i think, it may not be symlink object. because it is not guaranteed (it may be overwritten at the same time)? | 11:51 |
*** NM has joined #openstack-swift | 11:51 | |
acoles | I will leave a comment on gerrit | 11:53 |
kota_ | acoles: much helpful, awesome! | 11:53 |
acoles | OIC! | 11:54 |
*** abhitechie has quit IRC | 11:59 | |
*** lnxnut has joined #openstack-swift | 11:59 | |
acoles | yeah that's weird, POST to a symlink without a symlink header will change it to not be a symlink | 12:00 |
acoles | kota_: tdasilva will have all the answers :) | 12:00 |
kota_ | yeah we need tdasilva! | 12:04 |
*** lnxnut has quit IRC | 12:09 | |
*** spectr has joined #openstack-swift | 12:24 | |
*** catintheroof has joined #openstack-swift | 12:36 | |
*** catintheroof has quit IRC | 12:36 | |
*** catintheroof has joined #openstack-swift | 12:36 | |
*** gkadam has quit IRC | 12:38 | |
*** catintheroof has quit IRC | 12:41 | |
*** mat128 has joined #openstack-swift | 12:46 | |
*** catintheroof has joined #openstack-swift | 12:54 | |
*** lnxnut has joined #openstack-swift | 13:07 | |
*** chlong has joined #openstack-swift | 13:10 | |
*** chlong_ has joined #openstack-swift | 13:10 | |
*** lnxnut has quit IRC | 13:17 | |
*** SkyRocknRoll has quit IRC | 13:57 | |
*** spectr has quit IRC | 13:58 | |
*** spectr has joined #openstack-swift | 13:59 | |
*** lnxnut has joined #openstack-swift | 14:14 | |
*** itlinux has joined #openstack-swift | 14:23 | |
*** lnxnut has quit IRC | 14:24 | |
*** itlinux has quit IRC | 14:26 | |
*** chlong has quit IRC | 14:31 | |
*** chlong_ has quit IRC | 14:31 | |
*** chlong has joined #openstack-swift | 14:32 | |
*** honga has quit IRC | 14:50 | |
*** abhitechie has joined #openstack-swift | 15:04 | |
*** rcernin has quit IRC | 15:12 | |
*** pcaruana has quit IRC | 15:20 | |
*** lnxnut has joined #openstack-swift | 15:21 | |
*** itlinux has joined #openstack-swift | 15:29 | |
*** lnxnut has quit IRC | 15:31 | |
*** itlinux has quit IRC | 15:31 | |
*** abhitechie has quit IRC | 15:31 | |
*** chlong has quit IRC | 15:36 | |
*** mvk has quit IRC | 15:40 | |
*** chsc has joined #openstack-swift | 15:43 | |
*** chsc has joined #openstack-swift | 15:43 | |
*** itlinux has joined #openstack-swift | 15:44 | |
*** psachin has quit IRC | 15:45 | |
*** chsc has quit IRC | 15:50 | |
notmyname | good morning | 15:57 |
*** gyee has joined #openstack-swift | 16:02 | |
timburke | good morning | 16:11 |
timburke | kota_: acoles: iirc, the ?symlink=post param was purely intended as an escape hatch so you'd get back a 204 instead of a 307 | 16:14 |
*** silor has joined #openstack-swift | 16:15 | |
acoles | timburke: yep that's how it looked to me | 16:17 |
acoles | timburke: fyi I've been working on unit test for _misplaced_objects | 16:19 |
*** geaaru has quit IRC | 16:20 | |
acoles | hmmm, zuul does not seem happy | 16:21 |
timburke | not it does not. i've been peeking in -infra a bit, seems like we may be better off ignoring gate results for today | 16:22 |
timburke | don't be afraid of patch chains if it prevents conflicts -- then once things are actually functional, we can just issue a bunch of rechecks and wait, rather than spend time chasing down merge conflicts | 16:23 |
acoles | timburke: ok. I won't be able to push what I have because I locally rebased https://review.openstack.org/508635 on feature/deep to pick up some test infrastructure, hoping that 508635 would have merged by end of my day | 16:23 |
patchbot | patch 508635 - swift (feature/deep) - Fix up misplaced-objects handling | 16:23 |
*** SkyRocknRoll has joined #openstack-swift | 16:24 | |
acoles | should have rebased it then merged but I wanted to get it in the pipe | 16:24 |
timburke | that's fine -- push it up anyway, i'll see what the state of the gate is like and +A / recheck as it makes sense before heading out today | 16:25 |
acoles | timburke: ok, let' see how far I get before end of my day, still have some work to do | 16:25 |
*** lnxnut has joined #openstack-swift | 16:29 | |
*** SkyRocknRoll has quit IRC | 16:29 | |
*** silor has quit IRC | 16:34 | |
*** lnxnut has quit IRC | 16:36 | |
*** tesseract has quit IRC | 16:37 | |
*** silor has joined #openstack-swift | 16:39 | |
*** vint_bra has joined #openstack-swift | 17:01 | |
*** chsc has joined #openstack-swift | 17:02 | |
*** chsc has joined #openstack-swift | 17:02 | |
*** gabor_antal has quit IRC | 17:30 | |
*** gabor_antal_ has joined #openstack-swift | 17:31 | |
*** lnxnut has joined #openstack-swift | 17:34 | |
*** itlinux has quit IRC | 17:41 | |
*** itlinux has joined #openstack-swift | 17:42 | |
*** lnxnut has quit IRC | 17:43 | |
openstackgerrit | Pete Zaitcev proposed openstack/swift master: PUT+POST and its development test https://review.openstack.org/427911 | 17:44 |
clayg | zaitcev: This is the 3-phase version (PUT+POST+POST) !!! whoot! | 17:45 |
zaitcev | clayg: yes, I only need to find and add torgomatic's patch that allows linkat(), run functests, and fix-up unit tests. Then it should be good to go, unless you guys find something glaringly broken. | 17:47 |
*** itlinux has quit IRC | 17:48 | |
zaitcev | Honestly it was a surprise to me that object server examines the data that's stored in it. Some KISS principle that is. | 17:48 |
*** itlinux has joined #openstack-swift | 17:54 | |
*** itlinux has quit IRC | 17:58 | |
*** gabor_antal_ has quit IRC | 18:06 | |
*** gabor_antal has joined #openstack-swift | 18:07 | |
clayg | awesome! I've put it on my todo - I'll try to give it a glance before the meeting tomorrow - but probably won't get in a proper review until the end of the week. | 18:11 |
*** gabor_antal has quit IRC | 18:15 | |
*** gabor_antal has joined #openstack-swift | 18:16 | |
*** itlinux has joined #openstack-swift | 18:21 | |
*** vint_bra has quit IRC | 18:28 | |
*** mingyu has quit IRC | 18:33 | |
*** vint_bra has joined #openstack-swift | 18:36 | |
*** lnxnut has joined #openstack-swift | 18:41 | |
acoles | mattoliverau: mahatic kota_ fwiw I'm going to try to be around at 0700 UTC tomorrow | 18:47 |
*** lnxnut has quit IRC | 18:50 | |
*** mvk has joined #openstack-swift | 18:56 | |
*** ChubYann has joined #openstack-swift | 19:00 | |
*** vint_bra has quit IRC | 19:01 | |
acoles | timburke: IDK where https://review.openstack.org/#/c/508635/ has gone - last seen ~ 2hrs starting gate jobs but I can't see it on zuul status | 19:01 |
patchbot | patch 508635 - swift (feature/deep) - Fix up misplaced-objects handling | 19:01 |
*** NM has quit IRC | 19:02 | |
acoles | so I'm pushing my tests which will rebase 508635 | 19:02 |
timburke | yeah, so don't worry about it -- i'll kick things off again after the rollback | 19:02 |
timburke | perfect! | 19:02 |
acoles | oh right, is zuul rolling back? | 19:02 |
timburke | http://lists.openstack.org/pipermail/openstack-dev/2017-October/123049.html | 19:03 |
*** NM has joined #openstack-swift | 19:05 | |
*** skudlik has quit IRC | 19:06 | |
*** vint_bra has joined #openstack-swift | 19:08 | |
openstackgerrit | Alistair Coles proposed openstack/swift feature/deep: Fix up misplaced-objects handling https://review.openstack.org/508635 | 19:09 |
openstackgerrit | Alistair Coles proposed openstack/swift feature/deep: unit tests for _misplaced_objects https://review.openstack.org/509256 | 19:09 |
acoles | timburke: ok I'm done for today ^^ | 19:09 |
timburke | acoles: have a good night! | 19:10 |
acoles | good night | 19:10 |
*** NM has quit IRC | 19:15 | |
*** NM has joined #openstack-swift | 19:15 | |
*** catintheroof has quit IRC | 19:32 | |
*** catintheroof has joined #openstack-swift | 19:32 | |
*** chlong_ has joined #openstack-swift | 19:43 | |
*** silor has quit IRC | 19:44 | |
*** chlong has joined #openstack-swift | 19:44 | |
*** NM has quit IRC | 19:44 | |
*** chlong has quit IRC | 19:46 | |
*** lnxnut has joined #openstack-swift | 19:48 | |
*** NM has joined #openstack-swift | 19:52 | |
*** lnxnut has quit IRC | 19:59 | |
*** NM has quit IRC | 20:04 | |
*** NM has joined #openstack-swift | 20:08 | |
*** catintheroof has quit IRC | 20:10 | |
*** catintheroof has joined #openstack-swift | 20:11 | |
*** itlinux has quit IRC | 20:22 | |
*** itlinux has joined #openstack-swift | 20:32 | |
*** catintheroof has quit IRC | 20:42 | |
*** catintheroof has joined #openstack-swift | 20:42 | |
*** itlinux has quit IRC | 20:57 | |
*** chlong_ has quit IRC | 21:03 | |
*** itlinux has joined #openstack-swift | 21:03 | |
*** NM has quit IRC | 21:14 | |
*** gyee has quit IRC | 21:34 | |
openstackgerrit | Merged openstack/swift feature/deep: Fix up misplaced-objects handling https://review.openstack.org/508635 | 21:44 |
*** itlinux has quit IRC | 21:53 | |
*** lnxnut has joined #openstack-swift | 21:55 | |
mattoliverau | morning | 22:02 |
*** lnxnut has quit IRC | 22:07 | |
timburke | mattoliverau: o/ | 22:10 |
mattoliverau | Day light savings means I start earlier now.. but also means I need more coffee to get going :P | 22:11 |
mattoliverau | timburke: o/ | 22:11 |
*** vint_bra has quit IRC | 22:27 | |
openstackgerrit | Tim Burke proposed openstack/swift master: Clean up SLO tests and docs https://review.openstack.org/509305 | 22:43 |
openstackgerrit | Tim Burke proposed openstack/swift master: Let clients request heartbeats during SLO PUTs https://review.openstack.org/509306 | 22:43 |
*** mingyu has joined #openstack-swift | 22:49 | |
*** mingyu has quit IRC | 22:53 | |
*** catintheroof has quit IRC | 23:01 | |
*** lnxnut has joined #openstack-swift | 23:04 | |
*** wes_dillingham has joined #openstack-swift | 23:06 | |
*** lnxnut has quit IRC | 23:14 | |
*** itlinux has joined #openstack-swift | 23:21 | |
*** kei_yama has joined #openstack-swift | 23:22 | |
*** vint_bra has joined #openstack-swift | 23:30 | |
*** itlinux has quit IRC | 23:32 | |
*** chsc has quit IRC | 23:34 | |
*** abhitechie has joined #openstack-swift | 23:49 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!