| notmyname | for one brief moment, the gate queue was over 80% pass rate. | 00:07 |
|---|---|---|
| *** miurahr has joined #openstack-swift | 00:07 | |
| *** miurahr has quit IRC | 00:08 | |
| torgomatic | notmyname: how many nines is that again? | 00:11 |
| notmyname | as many as you want, as long as we count from the right ;-) | 00:12 |
| torgomatic | lol | 00:12 |
| openstackgerrit | Alex Pecoraro proposed a change to openstack/swift: Allow hostname for nodes in Ring https://review.openstack.org/74542 | 00:16 |
| *** matsuhashi has joined #openstack-swift | 00:26 | |
| *** mkollaro has quit IRC | 00:27 | |
| *** mhu has quit IRC | 00:49 | |
| *** 14WAB9C90 has quit IRC | 00:49 | |
| *** Anju has quit IRC | 00:57 | |
| *** StevenK has quit IRC | 00:57 | |
| *** Anticime1 has quit IRC | 00:57 | |
| *** pconstantine has quit IRC | 00:57 | |
| *** zigo has quit IRC | 00:58 | |
| *** otoolee has quit IRC | 00:58 | |
| *** rturk-away has quit IRC | 00:58 | |
| *** mhu has joined #openstack-swift | 00:58 | |
| *** acoles- has joined #openstack-swift | 00:58 | |
| gholt | CORS has been a pain since the first day notmyname campaigned to allow their headers be set on objects. ;P I don't really blame him, but it's fun to poke. CORS is _____. | 00:58 |
| notmyname | ya, that was a bad idea. why'd you let me do that? | 00:58 |
| notmyname | but cloud files is probably the only deployment that has the historic artifact ;-) | 00:58 |
| gholt | Yeah, if we're okay with fielding the complaints (if any)... should be okay, right? ;) | 00:58 |
| gholt | First rule of CORS: Do not implement it until you fully understand it. Second rule of CORS: No one ever fully understands it. | 00:58 |
| *** akscram has quit IRC | 00:58 | |
| *** creiht has quit IRC | 00:58 | |
| notmyname | heh | 00:58 |
| *** Anju1 has joined #openstack-swift | 00:58 | |
| notmyname | third rule: once you understand it, the draft spec is updated | 00:58 |
| *** pconstantine_ has joined #openstack-swift | 00:58 | |
| gholt | Haha | 00:58 |
| *** otoolee_ has joined #openstack-swift | 00:58 | |
| *** creiht has joined #openstack-swift | 00:58 | |
| *** ChanServ sets mode: +v creiht | 00:58 | |
| *** otoolee_ is now known as otoolee | 00:58 | |
| *** sileht has quit IRC | 00:59 | |
| *** openstack has joined #openstack-swift | 01:05 | |
| *** jokke_ has joined #openstack-swift | 01:05 | |
| *** swills has joined #openstack-swift | 01:06 | |
| *** swills has joined #openstack-swift | 01:06 | |
| *** mkerrin has joined #openstack-swift | 01:19 | |
| peluse | torgomatic: I'm at a loss with ssync and the 409 patch, dvsm failures that don't make any sense and I can't find any related issues. Each has rechecked at least 3 times. Any suggestions? Looks like we're due for a rebase against master - I'd piss on a spark plug if I thought it'd do any good :) | 01:19 |
| *** zigo has quit IRC | 01:27 | |
| *** Edward-Zhang has joined #openstack-swift | 01:29 | |
| *** zigo has joined #openstack-swift | 01:33 | |
| *** nosnos has joined #openstack-swift | 01:39 | |
| *** keving1 has quit IRC | 01:46 | |
| *** kun_huang has joined #openstack-swift | 01:50 | |
| *** matsuhashi has quit IRC | 01:58 | |
| openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Make cors work better. https://review.openstack.org/69419 | 01:58 |
| *** changbl has joined #openstack-swift | 02:02 | |
| clayg | dfg: so I mainly just updated the commit message for the next poor sap that has to load that in his head, and added a little helper into the test-cors-page to demonstrate the failure mode | 02:06 |
| clayg | it's so unfair that CORS has a thing called "simple cross-site requests" <- what a joke | 02:06 |
| clayg | dfg: I could use your sanity check on a precieved typo in the commit message about object over-rides? | 02:07 |
| clayg | dfg: your commit said "only be applied to objects without the 'Access-Control-Allow-Origin' and 'Access-Control-Allow-Methods' headers" | 02:08 |
| clayg | dfg: but the code seemed to say "'Access-Control-Allow-Origin' and 'Access-Control-Expose-Headers' headers" - so i "fixed" the commit message. | 02:09 |
| clayg | dfg: best I can tell we have zero support for Access-Control-Allow-Methods? Cirtainly not in OPTIONS requests anyway... | 02:10 |
| *** judd7_ has joined #openstack-swift | 02:10 | |
| clayg | er... i should say support for *custom* Allow-Methods | 02:11 |
| *** judd7 has quit IRC | 02:11 | |
| openstackgerrit | A change was merged to openstack/python-swiftclient: TCP port is appended two time in ClientException https://review.openstack.org/78761 | 02:13 |
| openstackgerrit | A change was merged to openstack/swift: Add missing swift constraint for functional tests https://review.openstack.org/78522 | 02:14 |
| *** keving1 has joined #openstack-swift | 02:16 | |
| *** matsuhashi has joined #openstack-swift | 02:17 | |
| *** zigo has quit IRC | 02:19 | |
| *** zigo has joined #openstack-swift | 02:23 | |
| *** keving1 has quit IRC | 02:25 | |
| clayg | dfg: yeah i just updated my object-server/?.conf's to allow me to storage a raw "Access-Control-Allow-Methods" header on my objects, and although it is being returned in the response it doesn't effect the OPTIONS request (as you stated) and it doesn't seem to make a different to the browser... | 02:25 |
| clayg | dfg: I think Access-Control-Allow-Methods is really only an OPTIONS response thing, althought it doesn't seem like it would be that terrible to add it to cors_info on the container - object level over-rides might be a different story. | 02:26 |
| *** kun_huang has quit IRC | 02:50 | |
| *** kun_huang has joined #openstack-swift | 02:51 | |
| *** haomaiwang has quit IRC | 02:51 | |
| *** haomaiwa_ has joined #openstack-swift | 02:51 | |
| *** gyee has quit IRC | 02:54 | |
| *** swills has quit IRC | 02:56 | |
| *** swills has joined #openstack-swift | 02:56 | |
| *** swills has joined #openstack-swift | 02:56 | |
| *** akscram has joined #openstack-swift | 02:57 | |
| *** mlipchuk has joined #openstack-swift | 03:03 | |
| *** haomaiwa_ has quit IRC | 03:05 | |
| *** mlipchuk has quit IRC | 03:17 | |
| *** haomaiwang has joined #openstack-swift | 03:20 | |
| *** keving1 has joined #openstack-swift | 03:21 | |
| *** matsuhashi has quit IRC | 03:23 | |
| *** keving1 has quit IRC | 03:29 | |
| *** haomaiwang has quit IRC | 03:36 | |
| *** haomaiwang has joined #openstack-swift | 03:37 | |
| *** madhuri has quit IRC | 03:37 | |
| *** Slidey has quit IRC | 03:38 | |
| *** madhuri has joined #openstack-swift | 03:38 | |
| *** haomaiw__ has joined #openstack-swift | 03:40 | |
| *** judd7 has joined #openstack-swift | 03:41 | |
| *** judd7_ has quit IRC | 03:42 | |
| *** haomaiwang has quit IRC | 03:43 | |
| *** fifieldt has joined #openstack-swift | 03:51 | |
| *** sileht has quit IRC | 03:56 | |
| *** sileht has joined #openstack-swift | 03:58 | |
| *** Edward-Zhang has quit IRC | 04:02 | |
| gholt | Sometimes I see 'TypeError: exceptions must be old-style classes or derived from BaseException, not NoneType' from bare 'raise' where it caught an exception and was looking to re-raise it. | 04:15 |
| gholt | Not always in the same spot, not always with the same corresponding other lines for the transaction. | 04:16 |
| gholt | It's weird, I don't remember seeing those before, but maybe I'm just not remembering well. | 04:16 |
| gholt | Maybe some kind of weird Eventlet thing going on clobbering the excinfo? | 04:16 |
| *** Edward-Zhang has joined #openstack-swift | 04:23 | |
| *** matsuhashi has joined #openstack-swift | 04:25 | |
| *** keving2 has joined #openstack-swift | 04:26 | |
| *** keving2 has quit IRC | 04:30 | |
| madhuri | gholt: Can you please review this https://review.openstack.org/#/c/78530/ ? | 04:33 |
| *** sungju has joined #openstack-swift | 04:35 | |
| gholt | madhuri: What's that for? Can you provide some context for why you need that change? In other places in Swift, an empty header value means to remove that header if it exists. | 04:41 |
| *** nshaikh has joined #openstack-swift | 04:42 | |
| madhuri | gholt: Yes, my understanding is same. If we set any header to empty, it must be removed from header. | 04:43 |
| madhuri | gholt: But it is not so in case of object metadata. | 04:43 |
| madhuri | gholt: If we set object's metadata to empty, it is still there is header with empty value. | 04:44 |
| gholt | So you added a test to verify the incorrect behavior? Why not fix the bug and add a test to verify the correct behavior? | 04:44 |
| madhuri | gholt: I asked for it in IRC but didn't get answer appropiate answer for it. | 04:45 |
| madhuri | gholt: I thought it is an expected behaviour. | 04:46 |
| madhuri | gholt: Is it going wrong? | 04:46 |
| *** sungju has quit IRC | 04:50 | |
| gholt | I don't know; hadn't really thought about it. My first guess is that sending an empty header at the object level should do what it does at the container level, which is remove the header. But considering the two header sets act differently in other ways, maybe somebody needs it that way. I just don't know. | 04:50 |
| *** sungju has joined #openstack-swift | 04:55 | |
| gholt | Well, I could be convinced either way. Maybe somebody will come up with a good reason why it should be one way over the other. Otherwise your test verifying the existing behavior in case somebody wants to change it seems fine. | 04:55 |
| *** sungju has quit IRC | 04:55 | |
| madhuri | gholt: Thank you!! Whom should I ask this? | 04:59 |
| madhuri | gholt: Is it a bug? | 04:59 |
| *** ppai has joined #openstack-swift | 05:00 | |
| gholt | Don't know, it doesn't bother me either way it behaves, so we'll see if somebody else chimes in on the review. | 05:00 |
| anticw | zaitcev: on the RH bugzilla? | 05:00 |
| gholt | madhuri: Posted to your review. And since it's midnight here and I have to interview someone in the morning, I really should get to bed. ;) | 05:02 |
| madhuri | gholt: Thanks in tonne :) | 05:02 |
| *** zaitcev has quit IRC | 05:07 | |
| openstackgerrit | Clay Gerrard proposed a change to openstack/swift: make probetests work with conf.d configs https://review.openstack.org/78753 | 05:11 |
| openstackgerrit | A change was merged to openstack/swift: Functional tests for TempURL and SLO together https://review.openstack.org/79061 | 05:25 |
| *** keving1 has joined #openstack-swift | 05:26 | |
| *** keving1 has quit IRC | 05:35 | |
| openstackgerrit | Clay Gerrard proposed a change to openstack/swift: DRY out common.constraints a bit more https://review.openstack.org/79772 | 05:46 |
| *** psharma has joined #openstack-swift | 05:50 | |
| openstackgerrit | Clay Gerrard proposed a change to openstack/swift: DRY out common.constraints a bit more https://review.openstack.org/79772 | 06:02 |
| *** keving1 has joined #openstack-swift | 06:31 | |
| *** keving1 has quit IRC | 06:40 | |
| *** PradeepChandani has joined #openstack-swift | 06:51 | |
| *** saju_m has joined #openstack-swift | 06:52 | |
| *** keving1 has joined #openstack-swift | 07:36 | |
| *** keving1 has quit IRC | 07:45 | |
| openstackgerrit | Clay Gerrard proposed a change to openstack/swift: Add containeralias middleware https://review.openstack.org/62494 | 07:48 |
| *** jamieh has joined #openstack-swift | 08:16 | |
| *** jamieh is now known as Guest8106 | 08:16 | |
| *** matsuhashi has quit IRC | 08:21 | |
| *** bada_ has joined #openstack-swift | 08:26 | |
| *** bada has quit IRC | 08:30 | |
| *** kun_huang has quit IRC | 08:40 | |
| *** saju_m has quit IRC | 08:41 | |
| *** keving1 has joined #openstack-swift | 08:41 | |
| *** foexle has joined #openstack-swift | 08:42 | |
| *** ekarlso has quit IRC | 08:47 | |
| *** ekarlso has joined #openstack-swift | 08:47 | |
| *** keving1 has quit IRC | 08:50 | |
| *** mlipchuk has joined #openstack-swift | 08:51 | |
| *** nacim has joined #openstack-swift | 09:00 | |
| *** Edward-Zhang has quit IRC | 09:03 | |
| *** saju_m has joined #openstack-swift | 09:09 | |
| *** mmcardle has joined #openstack-swift | 09:23 | |
| *** pconstantine_ has quit IRC | 09:26 | |
| *** Dharmit has quit IRC | 09:33 | |
| *** keving1 has joined #openstack-swift | 09:46 | |
| *** keving1 has quit IRC | 09:55 | |
| *** Trixboxer has joined #openstack-swift | 10:00 | |
| *** mmcardle has quit IRC | 10:07 | |
| *** SkyRocknRoll__ has joined #openstack-swift | 10:08 | |
| *** mmcardle has joined #openstack-swift | 10:09 | |
| *** kun_huang has joined #openstack-swift | 10:36 | |
| *** saju_m has quit IRC | 10:49 | |
| *** keving1 has joined #openstack-swift | 10:51 | |
| *** miurahr has joined #openstack-swift | 10:57 | |
| *** bada_ has quit IRC | 10:58 | |
| *** keving1 has quit IRC | 11:00 | |
| *** Midnightmyth has joined #openstack-swift | 11:02 | |
| *** lpabon has joined #openstack-swift | 11:02 | |
| *** miurahr has quit IRC | 11:02 | |
| *** tdasilva has joined #openstack-swift | 11:06 | |
| *** miurahr has joined #openstack-swift | 11:07 | |
| *** Midnightmyth has quit IRC | 11:11 | |
| *** pconstantine_ has joined #openstack-swift | 11:12 | |
| *** judd7 has quit IRC | 11:13 | |
| *** pconstantine_ is now known as pconstantine | 11:15 | |
| *** mkollaro has joined #openstack-swift | 11:17 | |
| *** miurahr has quit IRC | 11:21 | |
| *** bada has joined #openstack-swift | 11:25 | |
| *** ChanServ changes topic to "test/gate jobs are queuing now in preparation for gerrit maintenance at 12:00 utc (eta to resume is 12:30 utc)" | 11:28 | |
| *** bada has quit IRC | 11:30 | |
| *** ppai has quit IRC | 11:38 | |
| *** miurahr has joined #openstack-swift | 11:40 | |
| *** fifieldt has quit IRC | 11:42 | |
| *** mkerrin has quit IRC | 11:53 | |
| *** ppai has joined #openstack-swift | 11:55 | |
| *** keving1 has joined #openstack-swift | 11:56 | |
| *** mkerrin has joined #openstack-swift | 12:01 | |
| *** keving1 has quit IRC | 12:04 | |
| *** ChanServ changes topic to "gerrit on review.openstack.org is down for maintenance (eta to resume is 12:30 utc)" | 12:07 | |
| *** Midnightmyth has joined #openstack-swift | 12:09 | |
| *** gdrudy_ has joined #openstack-swift | 12:13 | |
| *** gdrudy_ has quit IRC | 12:14 | |
| *** SkyRocknRoll__ has quit IRC | 12:17 | |
| *** foexle has quit IRC | 12:21 | |
| *** psharma has quit IRC | 12:21 | |
| *** ChanServ changes topic to "gerrit on review.openstack.org is down for maintenance (revised eta to resume is 13:00 utc)" | 12:24 | |
| *** gvernik has joined #openstack-swift | 12:24 | |
| *** mmcardle has quit IRC | 12:24 | |
| *** foexle has joined #openstack-swift | 12:25 | |
| *** ppai has quit IRC | 12:25 | |
| *** mmcardle has joined #openstack-swift | 12:26 | |
| *** mmcardle has quit IRC | 12:37 | |
| *** mmcardle has joined #openstack-swift | 12:37 | |
| *** saju_m has joined #openstack-swift | 12:37 | |
| *** ppai has joined #openstack-swift | 12:38 | |
| *** jasondotstar has joined #openstack-swift | 12:40 | |
| *** mkollaro has quit IRC | 12:41 | |
| *** mkollaro1 has joined #openstack-swift | 12:41 | |
| *** mmcardle has quit IRC | 12:41 | |
| *** mkollaro has joined #openstack-swift | 12:42 | |
| *** mkollaro1 has quit IRC | 12:45 | |
| *** gvernik has quit IRC | 12:49 | |
| *** tdasilva has left #openstack-swift | 12:50 | |
| *** openstackgerrit has quit IRC | 12:54 | |
| *** openstackgerrit has joined #openstack-swift | 12:54 | |
| *** changbl has quit IRC | 12:57 | |
| *** kun_huang has quit IRC | 12:58 | |
| *** keving1 has joined #openstack-swift | 13:01 | |
| *** openstackgerrit has quit IRC | 13:08 | |
| *** openstackgerrit has joined #openstack-swift | 13:08 | |
| *** keving1 has quit IRC | 13:10 | |
| *** pconstantine has quit IRC | 13:12 | |
| *** pconstantine has joined #openstack-swift | 13:12 | |
| *** mkollaro1 has joined #openstack-swift | 13:18 | |
| *** mkollaro has quit IRC | 13:18 | |
| *** pandemicsyn2 is now known as pandemicsyn | 13:25 | |
| *** mmcardle has joined #openstack-swift | 13:31 | |
| *** nacim has quit IRC | 13:35 | |
| *** nacim has joined #openstack-swift | 13:40 | |
| *** nosnos has quit IRC | 13:43 | |
| *** matsuhashi has joined #openstack-swift | 13:50 | |
| *** byeager has joined #openstack-swift | 13:54 | |
| *** matsuhashi has quit IRC | 13:56 | |
| *** matsuhashi has joined #openstack-swift | 14:00 | |
| *** openstack has joined #openstack-swift | 14:04 | |
| *** ppai has quit IRC | 14:04 | |
| *** keving1 has joined #openstack-swift | 14:07 | |
| *** dmsimard has joined #openstack-swift | 14:13 | |
| *** keving1 has quit IRC | 14:15 | |
| *** jasondotstar has quit IRC | 14:16 | |
| *** tdasilva has joined #openstack-swift | 14:19 | |
| *** kun_huang has joined #openstack-swift | 14:21 | |
| *** chandan_kumar has quit IRC | 14:22 | |
| *** matsuhashi has quit IRC | 14:23 | |
| *** chandan_kumar has joined #openstack-swift | 14:25 | |
| *** Guest8106 has quit IRC | 14:27 | |
| *** jamieh has joined #openstack-swift | 14:28 | |
| *** jamieh is now known as Guest24116 | 14:28 | |
| *** tongli has joined #openstack-swift | 14:28 | |
| *** Guest24116 is now known as jamiehannaford | 14:30 | |
| *** matsuhashi has joined #openstack-swift | 14:30 | |
| *** piyush has joined #openstack-swift | 14:38 | |
| *** zaitcev has joined #openstack-swift | 14:40 | |
| *** ChanServ sets mode: +v zaitcev | 14:40 | |
| *** chandankumar_ has joined #openstack-swift | 14:40 | |
| *** chandankumar_ has quit IRC | 14:40 | |
| *** ChanServ changes topic to "Current Swift Release: 1.13.0 | Priority Reviews: https://wiki.openstack.org/wiki/Swift/PriorityReviews | Channel Logs: http://eavesdrop.openstack.org/irclogs/%23openstack-swift/" | 14:42 | |
| *** zaitcev has quit IRC | 14:53 | |
| *** zaitcev has joined #openstack-swift | 14:53 | |
| *** ChanServ sets mode: +v zaitcev | 14:53 | |
| *** changbl has joined #openstack-swift | 15:00 | |
| *** nshaikh has quit IRC | 15:03 | |
| *** saju_m has quit IRC | 15:03 | |
| *** saju_m has joined #openstack-swift | 15:04 | |
| *** changbl has quit IRC | 15:11 | |
| *** keving1 has joined #openstack-swift | 15:12 | |
| *** Dharmit has joined #openstack-swift | 15:12 | |
| *** changbl has joined #openstack-swift | 15:14 | |
| *** matsuhashi has quit IRC | 15:18 | |
| *** keving1 has quit IRC | 15:19 | |
| *** pconstantine has quit IRC | 15:21 | |
| notmyname | good morning | 15:26 |
| lpabon | good afternoon :-) | 15:26 |
| *** chandan_kumar has quit IRC | 15:26 | |
| *** Midnightmyth has quit IRC | 15:27 | |
| *** chandan_kumar has joined #openstack-swift | 15:28 | |
| otoolee | creiht: I've uploaded a new patch with the change that you requested, could you have a look and see if it works for you? Thanks. https://review.openstack.org/#/c/75881/2 | 15:29 |
| *** gyee has joined #openstack-swift | 15:31 | |
| tongli | @notmyname, John, do you know what authorization means a swift production setup use? | 15:32 |
| notmyname | ? | 15:32 |
| tongli | @notmyname, most of them use customized authorization method? tempauth won't work well, I would think. | 15:33 |
| tongli | @notmyname, the question I have is what is the typical authorization method for Swift. | 15:35 |
| notmyname | ah. various things. some actually do use tempauth. others use swauth. a few use keystone. I know that many customers at swiftstack use either our beefed-up version of tempauth or our LDAP | 15:37 |
| *** PradeepChandani has quit IRC | 15:39 | |
| *** saju_m has quit IRC | 15:40 | |
| *** PradeepChandani has joined #openstack-swift | 15:40 | |
| tongli | @notmyname, ok, keystone only does authentication. | 15:41 |
| tongli | I do not think it does authorization though. | 15:41 |
| *** xga has joined #openstack-swift | 15:53 | |
| notmyname | well, technically keystone is only an adaptor layer meant to abstract whatever identity service you have. so it's not really doing much at all | 15:54 |
| *** gyee has quit IRC | 15:59 | |
| *** Dharmit has quit IRC | 16:09 | |
| *** foexle has quit IRC | 16:10 | |
| *** Dharmit has joined #openstack-swift | 16:10 | |
| *** xga has quit IRC | 16:11 | |
| *** keving1 has joined #openstack-swift | 16:17 | |
| notmyname | reminder that we have the swift team meeting today at 1900UTC. that's in a little less than 3 hours | 16:17 |
| hugokuo | I do "really" want to join it | 16:18 |
| *** piyush has quit IRC | 16:20 | |
| notmyname | peluse: mlanner found the NIC issue in the canyon run boxes. turns out you're using really new Intel NICs that need a new kernel (or backports) :-) | 16:22 |
| peluse | notmyname: cool | 16:24 |
| peluse | did you expect non-Intel NICSs? :) | 16:24 |
| notmyname | no, but maybe NICs that would work on a current unbuntu LTS ;-) | 16:25 |
| *** keving1 has quit IRC | 16:25 | |
| *** byeager has quit IRC | 16:26 | |
| *** byeager has joined #openstack-swift | 16:26 | |
| peluse | yeah, well, HW guys - what can you do :) | 16:27 |
| notmyname | heh | 16:27 |
| * peluse is not referring to himself of course | 16:27 | |
| *** byeager has quit IRC | 16:31 | |
| *** piyush has joined #openstack-swift | 16:34 | |
| *** nacim has quit IRC | 16:36 | |
| *** nacim has joined #openstack-swift | 16:36 | |
| *** piyush1 has joined #openstack-swift | 16:37 | |
| *** piyush has quit IRC | 16:37 | |
| *** wimplo has joined #openstack-swift | 16:40 | |
| openstackgerrit | Alistair Coles proposed a change to openstack/swift: Enable object system metadata on PUTs https://review.openstack.org/79991 | 16:40 |
| *** wimplo has quit IRC | 16:41 | |
| *** jamiehannaford has quit IRC | 16:47 | |
| *** gyee has joined #openstack-swift | 16:49 | |
| *** nacim has quit IRC | 16:50 | |
| *** Dharmit has quit IRC | 16:53 | |
| *** ccarrizo has joined #openstack-swift | 16:54 | |
| *** mkollaro1 has quit IRC | 16:59 | |
| *** jasondotstar has joined #openstack-swift | 17:09 | |
| *** Dharmit has joined #openstack-swift | 17:10 | |
| *** Dharmit has quit IRC | 17:15 | |
| *** saju_m has joined #openstack-swift | 17:16 | |
| *** seandrummond has joined #openstack-swift | 17:17 | |
| *** jamieh has joined #openstack-swift | 17:21 | |
| *** jamieh is now known as Guest35613 | 17:21 | |
| *** keving1 has joined #openstack-swift | 17:22 | |
| *** Guest35613 has quit IRC | 17:23 | |
| *** keving2 has joined #openstack-swift | 17:29 | |
| *** keving1 has quit IRC | 17:29 | |
| *** Midnightmyth has joined #openstack-swift | 17:34 | |
| *** piyush has joined #openstack-swift | 17:37 | |
| *** openstackstatus has quit IRC | 17:38 | |
| *** piyush1 has quit IRC | 17:38 | |
| *** openstackstatus has joined #openstack-swift | 17:39 | |
| *** mjseger has quit IRC | 17:45 | |
| creiht | otoolee: ok, I've been a bit busy trying to get some code done myself | 17:48 |
| creiht | but I'll try to look at it soon | 17:48 |
| creiht | thanks | 17:48 |
| *** Fin1te has joined #openstack-swift | 17:51 | |
| *** gvernik has joined #openstack-swift | 18:00 | |
| seandrummond | hey guys with swift v1.11.0+ the object expirer should no longer be comparing X-If-Delete-At and X-Delete-At in order to delete an object at the X-Delete-At header right? | 18:01 |
| *** csd has joined #openstack-swift | 18:04 | |
| *** tdasilva has quit IRC | 18:06 | |
| *** mmcardle has quit IRC | 18:07 | |
| *** mmcardle has joined #openstack-swift | 18:07 | |
| *** Midnightmyth has quit IRC | 18:07 | |
| *** tdasilva has joined #openstack-swift | 18:11 | |
| *** mmcardle has quit IRC | 18:11 | |
| *** Fin1te has quit IRC | 18:22 | |
| notmyname | so I'm installing swift from Red Hat's RDO. interesting experience, since I'm consuming someone else's interpretation of what we are building | 18:27 |
| *** saju_m has quit IRC | 18:30 | |
| notmyname | reminder: swift team meeting in #openstack-meeting in 30 minutes | 18:31 |
| *** mkollaro has joined #openstack-swift | 18:32 | |
| *** gdrudy_ has joined #openstack-swift | 18:37 | |
| *** jasondotstar has quit IRC | 18:40 | |
| notmyname | torgomatic: creiht: https://twitter.com/cleverdevil/status/443817869101387776 | 18:45 |
| clayg | did i miss the meeting? | 18:48 |
| torgomatic | clayg: nope, it's at 12 PDT | 18:49 |
| torgomatic | you still have 11 minutes to figure out how to miss it :) | 18:50 |
| notmyname | looks like a fairly short agenda so far https://wiki.openstack.org/wiki/Meetings/Swift | 18:50 |
| *** mlipchuk has quit IRC | 18:51 | |
| clayg | torgomatic: did you have any luck fixing up the reconciler? | 18:53 |
| *** gdrudy_ has quit IRC | 18:53 | |
| torgomatic | clayg: sort of... I'm poking at tests for the replication side | 18:54 |
| torgomatic | could be better, could be worse | 18:55 |
| peluse | torgomatic/notmyname: you guys see my notes? should I go ahead and do an EC master rebase and post somewhere for one of you to push? I'm out of ideas | 18:55 |
| *** tdasilva has quit IRC | 18:56 | |
| torgomatic | peluse: I think it's just the usual Jenkins flakiness. I've seen CHANGELOG updates take a half-dozen tries to make it through the check jobs, and those certainly aren't introducing any new bugs | 18:56 |
| peluse | torgomatic: OK, I'll give them another whirl. Just feel like I'm abusing the system with recheck no bug over and over | 18:57 |
| peluse | but we still are due for a rebase | 18:57 |
| notmyname | peluse: yes to all of that :-) | 18:58 |
| peluse | cool, I'll get on it after the noon meeting. thanks guys | 18:58 |
| *** Midnightmyth has joined #openstack-swift | 18:59 | |
| notmyname | swift team meeting time in #openstack-meeting | 19:00 |
| *** tdasilva has joined #openstack-swift | 19:01 | |
| openstackgerrit | Clay Gerrard proposed a change to openstack/swift: make probetests work with conf.d configs https://review.openstack.org/78753 | 19:04 |
| *** tongli has quit IRC | 19:14 | |
| acoles- | notmyname: still around? | 19:51 |
| notmyname | acoles-: yup | 19:51 |
| notmyname | (always) | 19:51 |
| acoles- | about post-as-copy... | 19:51 |
| clayg | portante: i'm probably not going to bother with the skiptest thing, it's not a bad idea tho... meh maybe i should do it | 19:51 |
| notmyname | yes | 19:51 |
| acoles- | notmyname: i thought it was needed for container-sync? did i misunderstand? | 19:52 |
| clayg | portante: I just don't like skips, it'd be better if I could find something to assert in that case | 19:52 |
| notmyname | acoles-: want to have a quick skype call on it? may be faster than IRC | 19:52 |
| portante | clayg: that is why I did not -1 it, because I don' think that should gate the work | 19:52 |
| acoles- | k | 19:53 |
| portante | clayg: sure | 19:53 |
| acoles- | notmyname: ok | 19:53 |
| notmyname | acoles-: pm me your skype id | 19:53 |
| portante | clayg: with regard to the functional test in-process mechanism | 19:53 |
| notmyname | (there are 4 alistair coles on skype) | 19:53 |
| clayg | portante: yeah I think that one test was just bad | 19:53 |
| clayg | portante: I think acctually if I split up this other pending change that refactors the account acl functests it would sorta fix itself | 19:54 |
| portante | what is the number, I'll check that out | 19:54 |
| portante | clagy | 19:54 |
| clayg | portante: so that on my list this afternoon - should be quick | 19:54 |
| portante | clayg | 19:54 |
| clayg | portante: it's kind mixed up with some other refactorings - i'm going to get the tests split out so it's an easy merge | 19:54 |
| clayg | I'll ping you on it when it's ready - should be quick | 19:54 |
| portante | k, thx | 19:55 |
| clayg | portante: what about the making the flag more prominent than the missing configs in the commit? I'm guessing the missing config behavior has a higher value to you than me? | 19:55 |
| portante | sec | 19:55 |
| acoles- | notmyname: ok, see your email | 19:56 |
| *** byeager has joined #openstack-swift | 20:01 | |
| clayg | torgomatic: neat! I had tempurl in the wrong spot in my proxy config and functional tests failed! | 20:10 |
| torgomatic | clayg: :) | 20:12 |
| *** gvernik has quit IRC | 20:15 | |
| *** changbl has quit IRC | 20:26 | |
| clayg | man do you ever get the feeling we have two functional test suites that just got slammed together ;) | 20:30 |
| notmyname | heh | 20:31 |
| notmyname | thanks StevenK ;-) | 20:31 |
| notmyname | clayg: now he's going to think he did something wrong! | 20:31 |
| openstackgerrit | Clay Gerrard proposed a change to openstack/swift: additional functional tests for account acls https://review.openstack.org/80073 | 20:32 |
| *** piyush has quit IRC | 20:34 | |
| clayg | portante: ^ so there's the thing that might fix the account acl test that was reading the test.conf out of band | 20:40 |
| clayg | anyone else that's interested it just a bunch of asserts on existing behavior of account acls that should make it easier to write the keystone impl when the time comes | 20:41 |
| clayg | (or any one else that wants to support account acls) | 20:41 |
| * clayg glances at swauth | 20:41 | |
| clayg | sigh... that moment when you realize the code you just refactored is completly untested :'( | 20:43 |
| clayg | ^ unrelated to the acl functests - those are solid | 20:43 |
| portante | clayg: k | 20:43 |
| portante | clayg: I'd like to see the in-process functional tests land so that I can review how the coverage goes with them. :) | 20:44 |
| *** csd has quit IRC | 20:44 | |
| *** jasondotstar has joined #openstack-swift | 20:45 | |
| *** shri has joined #openstack-swift | 20:45 | |
| *** tdasilva has left #openstack-swift | 20:50 | |
| *** piyush has joined #openstack-swift | 20:52 | |
| *** piyush1 has joined #openstack-swift | 20:54 | |
| *** piyush has quit IRC | 20:56 | |
| *** lpabon has quit IRC | 21:02 | |
| clayg | portante: yeah me too, so what's the next step? | 21:05 |
| *** gyee has quit IRC | 21:05 | |
| clayg | dfg: any feedback to my comments on 69419 (Make cors work better) | 21:05 |
| peluse | notmyname/torgomatic: here's the EC branch rebased (you have to push, I don't have permissions) https://github.com/peluse/ec4.git | 21:05 |
| clayg | peluse: it doesn't come through as a gerrit patch? | 21:06 |
| *** seandrummond has quit IRC | 21:06 | |
| notmyname | clayg: the ability to push merge patches is restricted | 21:07 |
| *** Trixboxer has quit IRC | 21:08 | |
| clayg | so... not a gerrit change? | 21:08 |
| peluse | it will be once one of them does it. I just help by resolving conflicts and testing in advance :) | 21:09 |
| portante | clayg: gotta head home, now but I'll respond with my thoughts about the on/off switch for in-process functional tests a bit later | 21:09 |
| *** kun_huang has quit IRC | 21:22 | |
| zaitcev | Guys, I was looking at SP a bit, and... Do we ever verify somewhere that two rings do not point to the same node:device? I figure this may be made complex by having 2 object server processes with /srv/node and /srv/other_policies | 21:25 |
| zaitcev | Such thing could only be a typo in a shell script that launches builder, I imagine, but still. | 21:26 |
| *** jasondotstar has quit IRC | 21:26 | |
| notmyname | zaitcev: actually that's ok (and expected) | 21:30 |
| notmyname | one object server process can handle all of the policies. each policy uses a different top-level directory, so they all coexist peacefully | 21:31 |
| peluse | zaitcev: and, in fact, I have another WIP patch that, after more discussion and work, can potentially make it easier to do just that | 21:33 |
| notmyname | peluse: does your merge commit have any conflicts it resolved? | 21:33 |
| peluse | notmyname: a few minor ones (2 I think) | 21:34 |
| notmyname | ok | 21:34 |
| notmyname | peluse: running tests locally on saio then will push merge commit | 21:37 |
| peluse | notmyname: cool, let me know if you run into anything | 21:38 |
| notmyname | peluse: seems your jenkins failure is a tempest test that is testing expiry time and getting a response for an object that should be expired | 21:52 |
| notmyname | eg http://logs.openstack.org/47/65347/18/check/check-tempest-dsvm-full/0388b75/console.html#_2014-03-12_21_29_55_269 | 21:52 |
| notmyname | which seems to have happend 21 times in the last 24 hours | 21:53 |
| openstackgerrit | John Dickinson proposed a change to openstack/swift: merged master into ec branch https://review.openstack.org/80095 | 21:53 |
| notmyname | peluse: there ^ | 21:53 |
| clayg | is that the bug where I was like "yeah looks like a race in the test" | 21:53 |
| notmyname | clayg: ya, I think so. they set the expiry for 1 second then sleep for 2 seconds and try to read it | 21:54 |
| *** fifieldt has joined #openstack-swift | 21:54 | |
| notmyname | clayg: https://github.com/openstack/tempest/blob/master/tempest/api/object_storage/test_object_temp_url_negative.py#L106 | 21:55 |
| notmyname | peluse: the gerrit review there has the merge conflict resolution. does it look right? | 21:56 |
| clayg | someone must know how to patch that? | 21:57 |
| clayg | while time() < timeout: | 21:57 |
| peluse | notmyname: that temp_url thing is what's been killing me for the last few days | 22:01 |
| peluse | I'll go look at the gettit thing, one sec | 22:02 |
| peluse | notmyname: yup, good to go | 22:02 |
| notmyname | peluse: cool | 22:02 |
| notmyname | peluse: merging | 22:03 |
| peluse | so what can be done about the test issue above? | 22:04 |
| *** zul has joined #openstack-swift | 22:04 | |
| notmyname | peluse: I'm working on a small patch | 22:05 |
| *** zul has quit IRC | 22:05 | |
| notmyname | peluse: to tempest | 22:05 |
| peluse | sweet | 22:05 |
| notmyname | clayg: peluse: look good? https://gist.github.com/notmyname/9503f6de0ebbb22ed773 | 22:06 |
| notmyname | clayg: peluse: sorry. this one. https://gist.github.com/notmyname/c1d077eb652a273a804f | 22:07 |
| *** ccarrizo has quit IRC | 22:09 | |
| *** piyush1 has quit IRC | 22:10 | |
| peluse | hmmm, I don't immediately see the race or how this resolves it? | 22:13 |
| clayg | q | 22:14 |
| peluse | in both cases the input to get_temp_url is the same (expire time), and previously it waited 2 secs independent of previous time and now waits 1 sec at a time till current time >= the expire time which, from what I can tell, can only wait *less* than the previous implementation but not more | 22:18 |
| torgomatic | something something rounding errors something something floating-point time? | 22:20 |
| notmyname | torgomatic: that's pretty much what I was thinking (or trying to imagine) | 22:20 |
| *** mkollaro has quit IRC | 22:21 | |
| *** mkollaro has joined #openstack-swift | 22:21 | |
| notmyname | the time is truncated. but it's the existing sleep(2) that is troublesome to any scenario I'm coming up with | 22:21 |
| notmyname | unless maybe the server-side is off from the client-side? | 22:22 |
| peluse | can you pass in time -1 to get_temp_url for the expriy time and not wait at all? | 22:22 |
| notmyname | I think that give a 412 | 22:22 |
| notmyname | I don't remember if gholt added a code path for that | 22:22 |
| peluse | and where would the rounging error be, in the +1 ? | 22:23 |
| peluse | sheesh, my typing is extra bad today | 22:23 |
| clayg | are you guys still talking about the tempurl thing? | 22:24 |
| clayg | er... the temptest test for tempurl? | 22:24 |
| peluse | ja | 22:24 |
| clayg | why not just repeatly request the object until a) it 401's and you pass the test or b) you reach a timeout and fail the test | 22:25 |
| torgomatic | peluse: looks to me like one *can* pass in a small number for expires (maybe not negative though) and it'll be expired right away | 22:25 |
| peluse | yeah, I was just about to say that | 22:25 |
| clayg | where the timeout is... well long enought to account for slow ass vms' in the gate that can't tell time | 22:25 |
| peluse | pass in 0 | 22:26 |
| clayg | but that's sort of a dumb test, it would also pass if you gave it the wrong key | 22:26 |
| clayg | locks and latches baby locks and latches | 22:26 |
| peluse | true :) | 22:27 |
| notmyname | it's also a 100+ line module to do this one test. with duplicated code from the other module that is doing tests on tempurl | 22:28 |
| peluse | clayg: I like spinning til 401 (with a timeout of course), ha | 22:28 |
| clayg | notmyname: so are you gunna git blame that foo and go on a witch hunt to get it fixed? | 22:29 |
| notmyname | clayg: I don't recognize the name in git blame; I was going to write a patch and throw it at gerrit | 22:30 |
| clayg | well at least then they can't say we're not *trying* | 22:31 |
| *** byeager has quit IRC | 22:32 | |
| *** byeager has joined #openstack-swift | 22:33 | |
| *** jergerber has joined #openstack-swift | 22:36 | |
| *** byeager has quit IRC | 22:37 | |
| *** byeager has joined #openstack-swift | 22:38 | |
| notmyname | ugh I don't want to take the time to learn a whole new framework | 22:41 |
| notmyname | what do I do at the end? how do I raise the error if it fails? | 22:41 |
| *** jergerber has quit IRC | 22:41 | |
| notmyname | self.fail()? raise something? | 22:41 |
| peluse | while(1) maybe :) | 22:42 |
| *** Midnightmyth has quit IRC | 22:46 | |
| *** byeager has quit IRC | 22:46 | |
| notmyname | peluse: clayg: https://gist.github.com/notmyname/5333c625ed976ebf6d58 | 22:47 |
| *** byeager has joined #openstack-swift | 22:47 | |
| *** changbl has joined #openstack-swift | 22:47 | |
| clayg | notmyname: you *know* you wanted to write while/else | 22:48 |
| notmyname | good call :-) | 22:48 |
| peluse | notmyname: cool | 22:49 |
| clayg | but yeah that'll work - except it probably doesn't need to be a 20 second timeout (probably) and you might could do a sleep in there so it's not all busy loopin' on you | 22:49 |
| *** dmsimard has quit IRC | 22:50 | |
| peluse | yeah, 1 sec sleep and maybe 10s gross timeout | 22:50 |
| clayg | peluse: 500ms sleep :P and cornflower blue | 22:51 |
| notmyname | red! | 22:51 |
| *** byeager has quit IRC | 22:51 | |
| notmyname | https://gist.github.com/notmyname/5fcbcf35e6bf25515072 | 22:51 |
| peluse | heh, cornflower is a funny word. Looks good notmyname | 22:53 |
| notmyname | clayg: peluse: thanks | 22:53 |
| notmyname | peluse: makes me think of tortillas | 22:53 |
| *** Midnightmyth has joined #openstack-swift | 23:00 | |
| *** gyee has joined #openstack-swift | 23:24 | |
| *** mtreinish has quit IRC | 23:37 | |
| *** mtreinish has joined #openstack-swift | 23:37 | |
| *** j_king has quit IRC | 23:38 | |
| *** j_king has joined #openstack-swift | 23:38 | |
| openstackgerrit | Samuel Merritt proposed a change to openstack/swift: Store policy index in container_stat table https://review.openstack.org/71704 | 23:45 |
| *** miurahr has quit IRC | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!