Fix some bugs when testing opensds ansible
[stor4nfv.git] / src / ceph / doc / changelog / v0.67.4.txt
1 commit ad85b8bfafea6232d64cb7ba76a8b6e8252fa0c7
2 Author: Gary Lowell <gary.lowell@inktank.com>
3 Date:   Thu Oct 3 22:41:31 2013 +0000
4
5     v0.67.4
6
7 commit 5cd66d3b4bca92b402c95ab256fbc3f0329c446f
8 Author: Yehuda Sadeh <yehuda@inktank.com>
9 Date:   Fri Sep 20 14:04:47 2013 -0700
10
11     rgw: fix keystone token expiration test
12     
13     Fixes: #6360
14     The test was inverted, need expiration to be greater than
15     current time in order for token to be valid.
16     
17     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
18     Reviewed-by: Sage Weil <sage@inktank.com>
19
20 commit e0203c61a3f45fdd6d3d3ece26fef6152bdc036d
21 Author: David Zafman <david.zafman@inktank.com>
22 Date:   Wed Sep 11 16:55:06 2013 -0700
23
24     osd/OSD.cc: Use MIN() so that we don't exceed osd_recovery_max_active
25     
26     Caused by 944f3b73531af791c90f0f061280160003545c63
27     
28     Fixes: #6291
29     
30     Backport: dumpling
31     
32     Signed-off-by: David Zafman <david.zafman@inktank.com>
33     Reviewed-by: Samuel Just <sam.just@inktank.com>
34     (cherry picked from commit 139a714e13aa3c7f42091270b55dde8a17b3c4b8)
35     
36     Conflicts:
37     
38         src/osd/OSD.cc
39
40 commit c376708358cedb5561fbb43e9b9e622df3ea7a58
41 Author: Joao Eduardo Luis <joao.luis@inktank.com>
42 Date:   Wed Sep 25 22:08:24 2013 +0100
43
44     mon: OSDMonitor: do not write full_latest during trim
45     
46     On commit 81983bab we patched OSDMonitor::update_from_paxos() such that we
47     write the latest full map version to 'full_latest' each time the latest
48     full map was built from the incremental versions.
49     
50     This change however clashed with OSDMonitor::encode_trim_extra(), which
51     also wrote to 'full_latest' on each trim, writing instead the version of
52     the *oldest* full map.  This duality of behaviors could lead the store
53     to an inconsistent state across the monitors (although there's no sign of
54     it actually imposing any issues besides rebuilding already existing full
55     maps on some monitors).
56     
57     We now stop OSDMonitor::encode_trim_extra() from writing to 'full_latest'.
58     This function will still write out the oldest full map it has in the store,
59     but it will no longer write to full_latest, instead leaving it up to
60     OSDMonitor::update_from_paxos() to figure it out -- and it already does.
61     
62     Fixes: #6378
63     
64     Backport: dumpling
65     
66     Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
67     Reviewed-by: Sage Weil <sage@inktank.com>
68     (cherry picked from commit bd0f29a2c28cca496ec830eac932477ebf3182ba)
69
70 commit de40d0b3e35ab0124cd3c4ebfcaa435ab8abfab9
71 Author: Sage Weil <sage@inktank.com>
72 Date:   Tue Oct 1 15:53:42 2013 -0700
73
74     crush: invalidate rmap on create (and thus decode)
75     
76     If we have an existing CrushWrapper object and decode from a bufferlist,
77     reset build_rmaps so that they get rebuilt.
78     
79     Remove the build_rmaps() all in decode that was useless on a redecode
80     (because have_rmaps == true in that case and it did nothing).
81     
82     Fixes: #6442
83     Backport: dumpling, maybe cuttlefish
84     Signed-off-by: Sage Weil <sage@inktank.com>
85     Reviewed-by: Joao Eduardo Luis <joao.luis@inktank.com>
86     (cherry picked from commit 9b7a2ae329b6a511064dd3d6e549ba61f52cfd21)
87
88 commit 32f5233288c47d95b87c0a9cab5f9c2ffcf15417
89 Author: Dan Mick <dan.mick@inktank.com>
90 Date:   Mon Sep 30 14:58:11 2013 -0700
91
92     Invoke python with /usr/bin/env python instead of directly
93     
94     Fixes: #6311
95     Signed-off-by: Dan Mick <dan.mick@inktank.com>
96     (cherry picked from commit b9000b314b9166845ff302d4a827a996775d9a14)
97
98 commit 66aeca5a9079be398403bbff67bd5bf68c6fb111
99 Author: Sage Weil <sage@inktank.com>
100 Date:   Wed Sep 25 10:10:21 2013 -0700
101
102     qa/workunits/mon/crush_ops.sh: fix test
103     
104     Fix root.
105     
106     Fixes: #6392
107     Signed-off-by: Sage Weil <sage@inktank.com>
108     (cherry picked from commit c8cae87e9e08468cc86145e0fd60c05d12826239)
109
110 commit beb366302a125dd422c4f092b12eb541cb3bc788
111 Author: Sage Weil <sage@inktank.com>
112 Date:   Mon Sep 23 09:04:34 2013 -0700
113
114     Revert "ceph: parse CEPH_ARGS environment variable"
115     
116     This reverts commit 67a95b9880c9bc6e858150352318d68d64ed74ad.
117     
118     We now put CEPH_ARGS in the actual args we parse in python, which are passed
119     to rados piecemeal later.  This lets you put things like --id ... in there
120     that need to be parsed before librados is initialized.
121     (cherry picked from commit 97f462be4829f0167ed3d65e6694dfc16f1f3243)
122
123 commit b475ff9576f145d31c053213c699e13df76d2bcb
124 Author: Benoît Knecht <benoit.knecht@fsfe.org>
125 Date:   Mon Sep 23 15:58:42 2013 +0200
126
127     Add CEPH_ARGS at the end of sys.argv
128     
129     This allows, for instance, to pass a different client name to ceph by
130     exporting CEPH_ARGS="--id client_id".
131     
132     Signed-off-by: Benoît Knecht <benoit.knecht@fsfe.org>
133     Signed-off-by: Sage Weil <sage@inktank.com>
134     (cherry picked from commit 30abe3244c86cbbe1f5b005850c29c9c0eafcad4)
135
136 commit 94548b4b67cca37366c7d8719209a6d2e7956811
137 Author: Sage Weil <sage@inktank.com>
138 Date:   Tue Sep 24 15:26:03 2013 -0700
139
140     mon/OSDMonitor: fix 'ceph osd crush reweight ...'
141     
142     The adjust method returns a count of adjusted items.
143     
144     Add a test.
145     
146     Fixes: #6382
147     Backport: dumpling
148     Signed-off-by: Sage Weil <sage@inktank.com>
149     Reviewed-by: Dan Mick <dan.mick@inktank.com>
150     (cherry picked from commit 3de32562b55c6ece3a6ed783c36f8b9f21460339)
151
152 commit 00ff7f5c20e13869d0694379739ba4e61d44b97c
153 Author: Joao Eduardo Luis <joao.luis@inktank.com>
154 Date:   Tue Sep 10 00:20:41 2013 +0100
155
156     qa: workunits: mon: crush_ops: test 'ceph osd crush move'
157     
158     Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
159     (cherry picked from commit 3bc618b7b46496c5110edde0da9cae5d3e68e0e1)
160
161 commit 0ff5b4a96833681e92cc41f019a569134474f4cf
162 Author: Loic Dachary <loic@dachary.org>
163 Date:   Tue Sep 24 19:04:23 2013 +0200
164
165     osd: change warn_interval_multiplier to uint32_t
166     
167     to prevent overflow in OpTracker::check_ops_in_flight when
168     multiplying warn_interval_multiplier *= 2
169     
170     Backport: cuttlefish, dumpling
171     
172     http://tracker.ceph.com/issues/6370 fixes #6370
173     
174     Signed-off-by: Loic Dachary <loic@dachary.org>
175     (cherry picked from commit 1bce1f009bffd3e28025a08775fec189907a81db)
176
177 commit fb15040b6cec6221baa550ddfffade823f784c4a
178 Author: David Zafman <david.zafman@inktank.com>
179 Date:   Mon Sep 9 13:01:12 2013 -0700
180
181     crushtool: do not dump core with non-unique bucket IDs
182     
183     Return -EEXIST on duplicate ID
184     BUG FIX: crush_add_bucket() mixes error returns and IDs
185       Add optional argument to return generated ID
186     
187     Fixes: #6246
188     
189     Signed-off-by: David Zafman <david.zafman@inktank.com>
190     Reviewed-by: Sage Weil <sage@inktank.com>
191     (cherry picked from commit 8c76f3a0f9cf100ea2c941dc2b61c470aa5033d7)
192
193 commit 410db3f30c6eb54b807908c1f251ad4026e7d446
194 Author: Joao Eduardo Luis <jecluis@gmail.com>
195 Date:   Fri Sep 20 17:06:30 2013 +0100
196
197     qa: workunits: cephtool: check if 'heap' commands are parseable
198     
199     Signed-off-by: Joao Eduardo Luis <jecluis@gmail.com>
200     (cherry picked from commit b1eeaddd5f214c1b0883b44fc8cae07c649be7c4)
201
202 commit 062060a38bb26ff260cc51accc534413d726de49
203 Author: Joao Eduardo Luis <jecluis@gmail.com>
204 Date:   Fri Sep 20 17:50:27 2013 +0100
205
206     osd: OSD: add 'heap' command to known osd commands array
207     
208     Must have been forgotten during the cli rework.
209     
210     Backport: dumpling
211     
212     Signed-off-by: Joao Eduardo Luis <jecluis@gmail.com>
213     (cherry picked from commit 296f2d0db31e9f5a59a3a62a1e95b6c440430fa3)
214
215 commit 3f32f57b98e0224a1d30b2a81d7d260be0f53800
216 Author: Joao Eduardo Luis <jecluis@gmail.com>
217 Date:   Fri Sep 20 16:43:27 2013 +0100
218
219     mds: MDS: pass only heap profiler commands instead of the whole cmd vector
220     
221     The heap profiler doesn't care, nor should it, what our command name is.
222     It only cares about the commands it handles.
223     
224     Backport: dumpling
225     
226     Signed-off-by: Joao Eduardo Luis <jecluis@gmail.com>
227     (cherry picked from commit 238fe272c6bdb62d4e57fd8555c0136de99c8129)
228
229 commit 46dcc46617d8f35ab8433540b22343ddcbcc3716
230 Author: Joao Eduardo Luis <jecluis@gmail.com>
231 Date:   Fri Sep 20 16:41:14 2013 +0100
232
233     perfglue/heap_profiler.cc: expect args as first element on cmd vector
234     
235     We used to pass 'heap' as the first element of the cmd vector when
236     handling commands.  We haven't been doing so for a while now, so we
237     needed to fix this.
238     
239     Not expecting 'heap' also makes sense, considering that what we need to
240     know when we reach this function is what command we should handle, and
241     we should not care what the caller calls us when handling his business.
242     
243     Fixes: #6361
244     Backport: dumpling
245     
246     Signed-off-by: Joao Eduardo Luis <jecluis@gmail.com>
247     (cherry picked from commit c98b910d49bd2b46ceafdc430044a31524c29f5b)
248
249 commit 9dc5f15fbae22244ad1f62925e17c9d81e856e55
250 Author: Yehuda Sadeh <yehuda@inktank.com>
251 Date:   Mon Sep 16 14:35:25 2013 -0700
252
253     rgw: destroy get_obj handle in copy_obj()
254     
255     Fixes: #6176
256     Backport: dumpling
257     We take different code paths in copy_obj, make sure we close the handle
258     when we exit the function. Move the call to finish_get_obj() out of
259     copy_obj_data() as we don't create the handle there, so that should
260     makes code less confusing and less prone to errors.
261     Also, note that RGWRados::get_obj() also calls finish_get_obj(). For
262     everything to work in concert we need to pass a pointer to the handle
263     and not the handle itself. Therefore we needed to also change the call
264     to copy_obj_data().
265     
266     Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
267     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
268     (cherry picked from commit 9e98620e4325d15c88440a890b267131613e1aa1)
269
270 commit 471233e98a9f64ad513a4a196b7661b80534cb00
271 Author: Joao Eduardo Luis <joao.luis@inktank.com>
272 Date:   Mon Sep 9 23:14:11 2013 +0100
273
274     mon: MonCommands: expect a CephString as 1st arg for 'osd crush move'
275     
276     Fixes: #6230
277     
278     Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
279     (cherry picked from commit 7d3799fde19138f957f26ec6be10a8a0000fc1f0)
280
281 commit 2908225092bd2aa1b8afcb7848c1cdac5bd9e638
282 Author: Sage Weil <sage@inktank.com>
283 Date:   Mon Sep 23 16:23:33 2013 -0700
284
285     osd: revert 'osd max xattr size' limit
286     
287     Set it to 0 (unlimited) for now.
288     
289     Backport: dumpling
290     
291     Signed-off-by: Sage Weil <sage@inktank.com>
292     Reviewed-by: Yehuda Sadeh <yehuda@inktank.com>
293     (cherry picked from commit abb88d70643c3a76435b7a9d5b04ff29f7502361)
294
295 commit b3d3b3747c1eef695138dac828e5fcb435309c7b
296 Author: Greg Farnum <greg@inktank.com>
297 Date:   Wed Sep 11 16:24:32 2013 -0700
298
299     mds: be more careful about decoding LogEvents
300     
301     We need to wrap the full decode section or we can abort the process
302     if there's an issue (which we may want to just skip by).
303     
304     Signed-off-by: Greg Farnum <greg@inktank.com>
305     Reviewed-by: Sage Weil <sage@inktank.com>
306     (cherry picked from commit 73289b34b0be5b6612e38944794d59b5e789f841)
307
308 commit 06c58132199ed22413b509dfa751321ccdb24225
309 Author: Joao Eduardo Luis <joao.luis@inktank.com>
310 Date:   Tue Sep 17 17:58:20 2013 +0100
311
312     mon: OSDMonitor: multiple rebuilt full maps per transaction
313     
314     Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
315     (cherry picked from commit 0d20cae0be701c5b6151a26ee5e4fe24d89aa20a)
316
317 commit 65bbcaf4b68790dae4506c1f5db237077e1ff0ae
318 Author: Joao Eduardo Luis <joao.luis@inktank.com>
319 Date:   Sun Sep 15 21:03:50 2013 +0100
320
321     mon: OSDMonitor: update latest_full while rebuilding full maps
322     
323     Not doing so will make the monitor rebuild the osdmap full versions, even
324     though they may have been rebuilt before, every time the monitor starts.
325     
326     This mostly happens when the cluster is left in an unhealthy state for
327     a long period of time and incremental versions build up.  Even though we
328     build the full maps on update_from_paxos(), not updating 'full_latest'
329     leads to the situation initially described.
330     
331     Fixes: #6322
332     
333     Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
334     (cherry picked from commit 81983bab3630520d6c7ee9b7e4a747bc17b8c5c3)
335
336 commit 9b9edb04581cca15e67c567332529f5b3f426743
337 Author: Joao Eduardo Luis <joao.luis@inktank.com>
338 Date:   Sun Sep 15 21:00:55 2013 +0100
339
340     mon: OSDMonitor: smaller transactions when rebuilding full versions
341     
342     Otherwise, for considerably sized rebuilds, the monitor will not only
343     consume vast amounts of memory, but it will also have troubles committing
344     the transaction.  Anyway, it's also a good idea to adjust transactions to
345     the granularity we want, and to be fair we care that each rebuilt full map
346     gets to disk, even if subsequent full maps don't (those can be rebuilt
347     later).
348     
349     Fixes: #6323
350     
351     Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
352     (cherry picked from commit 4ac1570c5cdcd6556dc291cc6d7878fd92d343ae)
353
354 commit 298811f7a15541b9ec1015c416ad2aa075be5691
355 Author: Joao Eduardo Luis <jecluis@gmail.com>
356 Date:   Wed Aug 28 15:51:01 2013 +0100
357
358     mon: OSDMonitor: check if pool is on unmanaged snaps mode on mk/rmsnap
359     
360     Backport: dumpling
361     Fixes: #6047
362     
363     Signed-off-by: Joao Eduardo Luis <jecluis@gmail.com>
364     (cherry picked from commit fab79543c54c2e446d3f76520d7906645c6b0075)
365
366 commit a992664435db9dde3745eb7f354cce3fc5400a47
367 Author: Yehuda Sadeh <yehuda@inktank.com>
368 Date:   Thu Sep 12 14:32:17 2013 -0700
369
370     lru_map: don't use list::size()
371     
372     replace list::size() with map::size(), which should have
373     a constant time complexity.
374     
375     Reviewed-by: Sage Weil <sage@inktank.com>
376     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
377     (cherry picked from commit 7c1d2ded8fa8061bf3f14932800998b963745dd1)
378
379 commit 788546ea71c994ff35323747294ed9c177fe7020
380 Author: Yehuda Sadeh <yehuda@inktank.com>
381 Date:   Thu Sep 12 14:30:19 2013 -0700
382
383     common/lru_map: rename tokens to entries
384     
385     This code was originally used in a token cache, now
386     as a generic infrastructure rename token fields.
387     
388     Reviewed-by: Sage Weil <sage@inktank.com>
389     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
390     (cherry picked from commit 532e41a9985a16b35a6e49cdcba38af0ad166fa8)
391
392 commit babeb00c42af760b3e7575166479e95365cfcc0a
393 Author: Yehuda Sadeh <yehuda@inktank.com>
394 Date:   Wed Sep 18 10:37:21 2013 -0700
395
396     rgw: use bufferlist::append() instead of bufferlist::push_back()
397     
398     push_back() expects char *, whereas append can append a single char.
399     Appending a NULL char to push_back is cast as a NULL pointer which is
400     bad.
401     
402     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
403     Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
404     (cherry picked from commit 08fe028bad13096d482454a2f303158727c363ff)
405
406 commit daf85c45dd4d158bc7c33a2fb784857bc7db35cd
407 Author: Yehuda Sadeh <yehuda@inktank.com>
408 Date:   Wed Sep 11 13:46:31 2013 -0700
409
410     rgw: NULL terminate buffer before parsing it
411     
412     Fixes: #6175
413     Backport: dumpling
414     We get a buffer off the remote gateway which might
415     not be NULL terminated. The JSON parser needs the
416     buffer to be NULL terminated even though we provide
417     a buffer length as it calls strlen().
418     
419     Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
420     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
421     (cherry picked from commit e7f7483192cddca1159aba439ce62b1e78669d51)
422
423 commit c73040a5518971813b9ebaae1624c5bacef315d0
424 Author: Yehuda Sadeh <yehuda@inktank.com>
425 Date:   Wed Sep 11 22:30:12 2013 -0700
426
427     rgw: don't call list::size() in ObjectCache
428     
429     Fixes: #6286
430     Use an external counter instead of calling list::size()
431     
432     Reviewed-by: Sage Weil <sage@inktank.com>
433     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
434     (cherry picked from commit 31e3a51e933429d286104fe077e98ea883437ad6)
435
436 commit a855aba9d18936e9a060119e041518790cd4b831
437 Author: Yehuda Sadeh <yehuda@inktank.com>
438 Date:   Tue Sep 10 12:18:55 2013 -0700
439
440     rgw: drain pending requests before completing write
441     
442     Fixes: #6268
443     When doing aio write of objects (either regular or multipart parts) we
444     need to drain pending aio requests. Otherwise if gateway goes down then
445     object might end up corrupted.
446     
447     Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
448     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
449     (cherry picked from commit 626669afaa333d73707553a85f5c874e99e9cbd8)
450
451 commit 670db7e80ddc9c26c43a4f66907a5996ce207c4d
452 Author: Yehuda Sadeh <yehuda@inktank.com>
453 Date:   Fri Sep 6 22:33:38 2013 -0700
454
455     rgw: fix get cors, delete cors
456     
457     Remove a couple of variables that overrode class member. Not
458     really clear how it was working before, might have been a bad
459     merge / rebase.
460     
461     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
462     (cherry picked from commit 13872785aeeddbe1b8dd97e49fd6a2d879514f8d)
463
464 commit a304016fa01b02efd500135c00b9bf3407a9999c
465 Merge: 408cd61 ac0a30f
466 Author: Yehuda Sadeh <yehuda@inktank.com>
467 Date:   Wed Sep 11 09:47:10 2013 -0700
468
469     Merge branch 'wip-6078-dumpling' into dumpling
470     
471     Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
472
473 commit ac0a30feb8c64a3b80d9c519a7b561213403afab
474 Author: Yehuda Sadeh <yehuda@inktank.com>
475 Date:   Wed Aug 28 21:25:20 2013 -0700
476
477     rgw: fix certain return status cases in CORS
478     
479     Change return values in certain cases, reorder
480     checks, etc.
481     
482     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
483
484 commit 13b28cc3f1eb8ef42875b630c485ee0105cd244a
485 Author: Yehuda Sadeh <yehuda@inktank.com>
486 Date:   Wed Aug 28 21:24:36 2013 -0700
487
488     rgw: add COPY method to be handled by CORS
489     
490     Was missing this http method.
491     
492     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
493
494 commit d45c87ea738807487e72c0719b0d3d459cbe19e9
495 Author: Yehuda Sadeh <yehuda@inktank.com>
496 Date:   Tue Aug 27 19:38:45 2013 -0700
497
498     rgw: fix CORS rule check
499     
500     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
501
502 commit 986fa92a7a1d88111ba28457160adfcfdaabc5d2
503 Author: Yehuda Sadeh <yehuda@inktank.com>
504 Date:   Tue Aug 27 19:38:18 2013 -0700
505
506     rgw: don't handle CORS if rule not found (is NULL)
507     
508     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
509
510 commit 71873aba6553492d3ad71596cefd7c841030a277
511 Author: Yehuda Sadeh <yehuda@inktank.com>
512 Date:   Thu Aug 22 13:38:55 2013 -0700
513
514     rgw: tie CORS header response to all relevant operations
515     
516     Have the CORS responses on all relevant operations. Also add headers
517     on failure cases.
518     
519     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
520
521 commit 94e7b594d85dbd26e58d823b41f418032e9f163f
522 Author: Yehuda Sadeh <yehuda@inktank.com>
523 Date:   Thu Aug 22 10:00:53 2013 -0700
524
525     rgw: add a generic CORS response handling
526     
527     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
528
529 commit c3385d8a102faf5379559bb98cf89637ceda1579
530 Author: Yehuda Sadeh <yehuda@inktank.com>
531 Date:   Wed Aug 21 17:22:46 2013 -0700
532
533     rgw: OPTIONS request doesn't need to read object info
534     
535     This is a bucket-only operation, so we shouldn't look at the
536     object. Object may not exist and we might respond with Not
537     Exists response which is not what we want.
538     
539     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
540
541 commit a5fdd44e5d8ce4b8d82273d83e27aea19e63aa7c
542 Author: Yehuda Sadeh <yehuda@inktank.com>
543 Date:   Wed Aug 21 14:43:28 2013 -0700
544
545     rgw: remove use of s->bucket_cors
546     
547     Some old code still tried to use s->bucket_cors, which was
548     abandoned in a cleanup work.
549     
550     Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>