summaryrefslogtreecommitdiffstats
path: root/src/ceph/doc/changelog/v10.2.7.txt
blob: 473f45e1d2b3fad22c39e43a148a2a15c890e4d9 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
693
694
695
696
697
698
699
700
701
702
703
704
705
706
707
708
709
710
711
712
713
714
715
716
717
718
719
720
721
722
723
724
725
726
727
728
729
730
731
732
733
734
735
736
737
738
739
740
741
742
743
744
745
746
747
748
749
750
751
752
753
754
755
756
757
758
759
760
761
762
763
764
765
766
767
768
769
770
771
772
773
774
775
776
777
778
779
780
781
782
783
784
785
786
787
788
789
790
791
792
793
794
795
796
797
798
799
800
801
802
803
804
805
806
807
808
809
810
811
812
813
814
815
816
817
818
819
820
821
822
823
824
825
826
827
828
829
830
831
832
833
834
835
836
837
838
839
840
841
842
843
844
845
846
847
848
849
850
851
852
853
854
855
856
857
858
859
860
861
862
863
864
865
866
867
868
869
870
871
872
873
874
875
876
877
878
879
880
881
882
883
884
885
886
887
888
889
890
891
892
893
894
895
896
897
898
899
900
901
902
903
904
905
906
907
908
909
910
911
912
913
914
915
916
917
918
919
920
921
922
923
924
925
926
927
928
929
930
931
932
933
934
935
936
937
938
939
940
941
942
943
944
945
946
947
948
949
950
951
952
953
954
955
956
957
958
959
960
961
962
963
964
965
966
967
968
969
970
971
972
973
974
975
976
977
978
979
980
981
982
983
984
985
986
987
988
989
990
991
992
993
994
995
996
997
998
999
1000
1001
1002
1003
1004
1005
1006
1007
1008
1009
1010
1011
1012
1013
1014
1015
1016
1017
1018
1019
1020
1021
1022
1023
1024
1025
1026
1027
1028
1029
1030
1031
1032
1033
1034
1035
1036
1037
1038
1039
1040
1041
1042
1043
1044
1045
1046
1047
1048
1049
1050
1051
1052
1053
1054
1055
1056
1057
1058
1059
1060
1061
1062
1063
1064
1065
1066
1067
1068
1069
1070
1071
1072
1073
1074
1075
1076
1077
1078
1079
1080
1081
1082
1083
1084
1085
1086
1087
1088
1089
1090
1091
1092
1093
1094
1095
1096
1097
1098
1099
1100
1101
1102
1103
1104
1105
1106
1107
1108
1109
1110
1111
1112
1113
1114
1115
1116
1117
1118
1119
1120
1121
1122
1123
1124
1125
1126
1127
1128
1129
1130
1131
1132
1133
1134
1135
1136
1137
1138
1139
1140
1141
1142
1143
1144
1145
1146
1147
1148
1149
1150
1151
1152
1153
1154
1155
1156
1157
1158
1159
1160
1161
1162
1163
1164
1165
1166
1167
1168
1169
1170
1171
1172
1173
1174
1175
1176
1177
1178
1179
1180
1181
1182
1183
1184
1185
1186
1187
1188
1189
1190
1191
1192
1193
1194
1195
1196
1197
1198
1199
1200
1201
1202
1203
1204
1205
1206
1207
1208
1209
1210
1211
1212
1213
1214
1215
1216
1217
1218
1219
1220
1221
1222
1223
1224
1225
1226
1227
1228
1229
1230
1231
1232
1233
1234
1235
1236
1237
1238
1239
1240
1241
1242
1243
1244
1245
1246
1247
1248
1249
1250
1251
1252
1253
1254
1255
1256
1257
1258
1259
1260
1261
1262
1263
1264
1265
1266
1267
1268
1269
1270
1271
1272
1273
1274
1275
1276
1277
1278
1279
1280
1281
1282
1283
1284
1285
1286
1287
1288
1289
1290
1291
1292
1293
1294
1295
1296
1297
1298
1299
1300
1301
1302
1303
1304
1305
1306
1307
1308
1309
1310
1311
1312
1313
1314
1315
1316
1317
1318
1319
1320
1321
1322
1323
1324
1325
1326
1327
1328
1329
1330
1331
1332
1333
1334
1335
1336
1337
1338
1339
1340
1341
1342
1343
1344
1345
1346
1347
1348
1349
1350
1351
1352
1353
1354
1355
1356
1357
1358
1359
1360
1361
1362
1363
1364
1365
1366
1367
1368
1369
1370
1371
1372
1373
1374
1375
1376
1377
1378
1379
1380
1381
1382
1383
1384
1385
1386
1387
1388
1389
1390
1391
1392
1393
1394
1395
1396
1397
1398
1399
1400
1401
1402
1403
1404
1405
1406
1407
1408
1409
1410
1411
1412
1413
1414
1415
1416
1417
1418
1419
1420
1421
1422
1423
1424
1425
1426
1427
1428
1429
1430
1431
1432
1433
1434
1435
1436
1437
1438
1439
1440
1441
1442
1443
1444
1445
1446
1447
1448
1449
1450
1451
1452
1453
1454
1455
1456
1457
1458
1459
1460
1461
1462
1463
1464
1465
1466
1467
1468
1469
1470
1471
1472
1473
1474
1475
1476
1477
1478
1479
1480
1481
1482
1483
1484
1485
1486
1487
1488
1489
1490
1491
1492
1493
1494
1495
1496
1497
1498
1499
1500
1501
1502
1503
1504
1505
1506
1507
1508
1509
1510
1511
1512
1513
1514
1515
1516
1517
1518
1519
1520
1521
1522
1523
1524
1525
1526
1527
1528
1529
1530
commit 50e863e0f4bc8f4b9e31156de690d765af245185
Author: Jenkins Build Slave User <ceph-release-team@redhat.com>
Date:   Mon Apr 10 11:43:44 2017 +0000

    10.2.7

commit a64d3e4b33e904aa6585464df8ffff6aafdec10c
Merge: c92640a 6c2a40a
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Fri Mar 31 10:27:07 2017 -0600

    Merge pull request #14230 from linuxbox2/wip-jewel-expand-argv
    
    jewel: rgw_file: expand argv
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit c92640a113843a491678c11319d2352f14486f17
Merge: 7c35658 fcc3ada
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Fri Mar 31 10:17:10 2017 -0600

    Merge pull request #14233 from ktdreyer/wip-19421-jewel
    
    jewel: librbd: possible race in ExclusiveLock handle_peer_notification
    
    Reviewed-by: Jason Dillaman <dillaman@redhat.com>

commit 7c3565899af5f1f474d82d8802d15c6c216ea894
Merge: 12467f6 eedb9f7
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Fri Mar 31 10:16:48 2017 -0600

    Merge pull request #14231 from linuxbox2/wip-jewel-fhcache
    
    jewel: rgw_fh: RGWFileHandle dtor must also cond-unlink from FHCache
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit 12467f6c3f346072d81c444ed68eb206402d8c33
Merge: 2d2521c 78c8be7
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Fri Mar 31 10:09:14 2017 -0600

    Merge pull request #14215 from mdw-at-linuxbox/wip-jewel-rgw-openssl-fix1
    
    rgw: fix openssl
    
    Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit 2d2521c0300000421740c611a5133c82d2e4d0f6
Merge: 0840cf8 7e4e290
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Fri Mar 31 10:07:48 2017 -0600

    Merge pull request #14206 from linuxbox2/wip-gui-jewel
    
    jewel: rgw_file: various fixes
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit 0840cf86214c793794c6f913b63cc67e0ab3fc94
Merge: 0bf6360 b24a8c2
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Fri Mar 31 10:07:03 2017 -0600

    Merge pull request #14205 from linuxbox2/wip-dir-orphan-jewel
    
    jewel:  rgw_file: wip dir orphan
    
     Conflicts:
    	src/rgw/rgw_file.cc (whitespace, trivial resolution)
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit 0bf636057ba853d1d1a777147014a740d5149b17
Merge: 83b5a7f fb85c68
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Fri Mar 31 10:05:40 2017 -0600

    Merge pull request #14169 from linuxbox2/wip-18650-jewel
    
    jewel: rgw: rgw_file: FHCache residence check should be exhaustive
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit 83b5a7f1c589546645e0f377d5d5f7b1f7363a59
Merge: 6a175f2 a969c44
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Fri Mar 31 10:02:58 2017 -0600

    Merge pull request #13869 from smithfarm/wip-19161-jewel
    
    jewel: rgw: rgw_file: fix marker computation
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit fb85c68dfbfcfb299cac3a7e6723067f216483ef
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Tue Feb 28 15:49:06 2017 -0500

    rgw_file:  use fh_hook::is_linked() to check residence
    
    Previously we assumed that !deleted handles were resident--there
    is an observed case where a !deleted handle is !linked.  Since
    we currently use safe_link mode, an is_linked() check is
    available, and exhaustive.
    
    Fixes: http://tracker.ceph.com/issues/19111
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit c0aa515f8d8c57ec5ee09e3b60df3cac60453c40)

commit fcc3ada733bac74a642ccd0f1ed6301ad7318a30
Author: Mykola Golub <mgolub@mirantis.com>
Date:   Tue Mar 28 15:55:11 2017 +0200

    jewel: librbd: possible race in ExclusiveLock handle_peer_notification
    
    This is a cherry-pick from kraken -- the master diverged after
    ManagedLock refactoring and is not affected.
    
    Fix: http://tracker.ceph.com/issues/19368
    Signed-off-by: Mykola Golub <mgolub@mirantis.com>
    (cherry picked from commit df59d6d5f7deb586cf14a6ef6984e6847db08852)

commit eedb9f7d2d2a4047b473f23c5a6956c40a3da126
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Tue Feb 28 20:24:12 2017 -0500

    rgw_file: RGWFileHandle dtor must also cond-unlink from FHCache
    
    Formerly masked in part by the reclaim() action, direct-delete now
    substitutes for reclaim() iff its LRU lane is over its high-water
    mark, and in particular, like reclaim() the destructor is certain
    to see handles still interned on the FHcache when nfs-ganesha is
    recycling objects from its own LRU.
    
    Fixes: http://tracker.ceph.com/issues/19112
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit d51a3b1224ba62bb53c6c2c7751fcf7853c35a4b)

commit 6c2a40a800277ded302f3183ac6c68b01ca3ed41
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Tue Jan 17 11:23:45 2017 -0500

    rgw_file:  split last argv on ws, if provided
    
    This is intended to allow an "extra" unparsed argument string
    containing various cmdline options to be passed as the last argument
    in the argv array of librgw_create(), which nfs-ganesha is
    expecting to happen.
    
    While at it, hook env_args() too.
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit fbc19e4effc736c98cc1cc283e5c7b131a0fa766)

commit d2ca03b391340c6944ba453b106f3e1c45b75f1d
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Sun Feb 12 20:18:26 2017 -0500

    rgw_file: fix hiwat behavior
    
    Removed logic to skip reclaim processing conditionally on hiwat,
    this probably meant to be related to a lowat value, which does
    not exist.
    
    Having exercised the hiwat reclaim behavior, noticed that the
    path which moves unreachable objects to LRU, could and probably
    should remove them altogether when q.size exceeds hiwat.  Now
    the max unreachable float is lane hiwat, for all lanes.
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit b8791b2217e9ca87b2d17b51f283fa14bd68b581)
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit 166cb7f85c240eeaffc70968abf5352d9cd45bd9
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Sun Feb 12 18:20:43 2017 -0500

    rgw_file: refcnt bugfixes
    
    This change includes 3 related changes:
    
    1. add required lock flags for FHCache updates--this is a crash
       bug under concurrent update/lookup
    
    2. omit to inc/dec refcnt on root filehandles in 2 places--the
       root handle current is not on the lru list, so it's not
       valid to do so
    
    3. based on observation of LRU behavior during creates/deletes,
       update (cohort) LRU unref to move objects to LRU when their
       refcount falls to SENTINEL_REFCNT--this cheaply primes the
       current reclaim() mechanism, so very significanty improves
       space use (e.g., after deletes) in the absence of scans
       (which is common due to nfs-ganesha caching)
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit beaeff059375b44188160dbde8a81dd4f4f8c6eb)
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit 31a71be9c9f24e22cf7e6eb390d3b39811ee3577
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Sat Feb 11 16:38:05 2017 -0500

    rgw_file:  add refcount dout traces at debuglevel 17
    
    These are helpful for checking RGWFileHandle refcnt invariants.
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit 462034e17f919fb783ee33e2c9fa8089f93fd97d)
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit 1d6c72fe9a59561fb8f33889895b6708342b2856
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Fri Feb 10 17:14:16 2017 -0500

    rgw_file: add pretty-print for RGWFileHandle
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit ef330f385d3407af5f470b5093145f59cc4dcc79)
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit a969c449272ce88ddcbbd509432134f19520c8c3
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Mon Feb 20 15:05:18 2017 -0500

    rgw_file: fix marker computation
    
    Fixes: http://tracker.ceph.com/issues/19018
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit 4454765e7dd08535c50d24205858e18dba4b454c)

commit a70e83c90eb42439ba194ca197f507f972a18c3c
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Sun Feb 19 20:34:31 2017 -0500

    rgw_file: rgw_readdir can't list multi-segment dirs
    
    This issue has one root cause in librgw, namely that the marker
    argument to these requests was incorrectly formatted (though the
    marker cache was working as intended).
    
    Secondarily, for nfs-ganesha users, there is a compounding issue
    that the RGW fsal was required by "temporary" convention to
    populate the entire dirent cache for a directory on a single
    readdir() invocation--the cache_inode/mdcache implementations
    invariantly pass (before future 2.5 changesets, currently in
    progress) a null pointer for the start cookie offset, intended
    to convey this.
    
    Fixes: http://tracker.ceph.com/issues/18991
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit 2cd60ee9712291b906123aca1704288b18a9742b)

commit 209987e1c764cac7574bf0bb969e93fcf55b4361
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Sun Feb 19 18:21:06 2017 -0500

    rgw_file: allow setattr on placeholder directories
    
    When a POSIX path <bucket>/foo/ is known only as an implicit path
    segment from other objects (e.g., <bucket>/foo/bar.txt), a case
    that would usually arise from S3 upload of such an object, an
    RGWFileHandle object representing "<bucket>/foo/" will be constructed
    as needed, with no backing in RGW.
    
    This is by design, but subsequently, if a setattr is performed on
    such a handle, we must be ready to create the object inline with
    storing the attributes.
    
    Fixes: http://tracker.ceph.com/issues/18989
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit 55eec1c0a0e136736961423b7b6244d0f3693c1a)

commit 6a175f27961bd4f0fda8f94c200458f17865c9f9
Merge: d32ae21 f3face6
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 16:36:35 2017 -0600

    Merge pull request #14140 from smithfarm/wip-19341-jewel
    
    jewel: 'period update' does not remove short_zone_ids of deleted zones
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit d32ae21128d664370f7d07ed14427b75e10da7f8
Merge: 4dfeed9 0be4e89
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 16:35:16 2017 -0600

    Merge pull request #13886 from cbodley/wip-17786
    
    jewel: rgw: clear data_sync_cr if RGWDataSyncControlCR fails
    
    Reviewed-by: Nathan Cutler <ncutler@suse.com>

commit 4dfeed949655a5b5041f612b868225c234e86bba
Merge: d69c54a 1985662
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 16:34:57 2017 -0600

    Merge pull request #13867 from smithfarm/wip-19159-jewel
    
    jewel: rgw: multisite: RGWMetaSyncShardControlCR gives up on EIO
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit d69c54aca9dde3d64b56e5ee72f1cd29057a80c6
Merge: 0830135 dfaaec0
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 16:34:24 2017 -0600

    Merge pull request #13858 from smithfarm/wip-19152-jewel
    
    jewel: rgw_file:  restore (corrected) fix for dir partial match (return of FLAG_EXACT_MATCH)
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit 0830135f40ac37ac9027bece65b8aac64c2c7ec9
Merge: 4dd3a9f 97fed01
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 16:33:18 2017 -0600

    Merge pull request #13848 from smithfarm/wip-19150-jewel
    
    jewel: rgw_file:  avoid interning .. in FHCache table and don't ref for them
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit 4dd3a9f628053784cff2aa5849ab3c4cdb2acf4e
Merge: 5eeba9c d333add
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 16:29:30 2017 -0600

    Merge pull request #13844 from smithfarm/wip-19148-jewel
    
    jewel: rgw: DUMPABLE flag is cleared by setuid preventing coredumps
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit 5eeba9cb85e9111ef124acd36dccf77c6f00b3e4
Merge: ae632b0 f7ce5df
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 16:26:48 2017 -0600

    Merge pull request #13823 from asheplyakov/19176-bp-jewel
    
    jewel: rgw: fix swift cannot disable object versioning with empty X-Versions-Location
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit ae632b0da6cc90a993fcae31b56f97c834d15aa1
Merge: 874120f e8041f6
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 16:22:23 2017 -0600

    Merge pull request #13778 from smithfarm/wip-18811-jewel
    
    jewel: librgw: RGWLibFS::setattr fails on directories
    
    Reviewed-by: Matt Benjamin <mbenjamin@redhat.com>

commit 874120f627f509c75b28d1be699812fce881f4bb
Merge: b0108ea e475bfa
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 16:21:44 2017 -0600

    Merge pull request #13717 from asheplyakov/19115-bp-jewel
    
    jewel: rgw_file: ensure valid_s3_object_name for directories, too
    
    Reviewed-by: Matt Benjamin <mbenjamin@redhat.com>

commit b0108ea3bae1662e90c7fb9a911257f45193798c
Merge: 6518d70 db928d6
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 16:19:03 2017 -0600

    Merge pull request #13596 from dillaman/wip-19038-jewel
    
    jewel: rbd-mirror: deleting a snapshot during sync can result in read errors
    
    Reviewed-by: Mykola Golub <mgolub@mirantis.com>

commit 6518d70b89686b9a4e8854a4c6290381fbcdd8c3
Merge: db50938 8941881
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 16:10:43 2017 -0600

    Merge pull request #13583 from asheplyakov/jewel-bp-18901
    
    jewel: rgw_file:  interned RGWFileHandle objects need parent refs
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit db50938013f0b1c1fe8a23247c91bbfc38d36a8c
Merge: 7536871 d44263f
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 16:02:43 2017 -0600

    Merge pull request #13503 from linuxbox2/wip-jewel-10156
    
    jewel: rgw: make sending Content-Length in 204 and 304 controllable
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>
    Reviewed-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>

commit 75368712aafc51b062b674b263354cca2dd4b49a
Merge: 250071e 2f20328
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 15:53:49 2017 -0600

    Merge pull request #13232 from ovh/wip-rework-recovery-priorities-jewel
    
    jewel: osd: Increase priority for inactive PGs backfill
    
    Reviewed-by: Sage Weil <sage@redhat.com>

commit 250071eb8dc744237acfed01df2cf04f2d88bc85
Merge: 6f57a77 9910eac
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 15:51:46 2017 -0600

    Merge pull request #13219 from linuxbox2/jewel-rgw-multipart-upload-copy-4
    
    jewel: rgw: multipart uploads copy part support
    
    Reviewed-by: Casey Bodley <cbodley@redhat.com>

commit 6f57a775301edce7a1039a4aa4479bbd8883cbf6
Merge: 8c02e54 3fce77a
Author: Ken Dreyer <kdreyer@redhat.com>
Date:   Tue Mar 28 15:51:24 2017 -0600

    Merge pull request #13108 from vumrao/wip-vumrao-jewel-18657
    
    jewel: osd: fix OSD network address in OSD heartbeat_check log message
    
    Reviewed-by: Josh Durgin <jdurgin@redhat.com>

commit b24a8c2f3890b6fcc60f3b52cba93d573b9d45a2
Author: Gui Hecheng <guihecheng@cmss.chinamobile.com>
Date:   Thu Mar 2 17:21:57 2017 +0800

    rgw_file: posix style atime,ctime,mtime
    
    As an ganesha FSAL backend, rgw_file should properly maintain
    the atime,ctime,mtime properly against operations such as:
    	(read,write) for file
    	(create,unlink,mkdir,rmdir,rename) for dir
    	(setattr) for file and dir
    
    Signed-off-by: Gui Hecheng <guihecheng@cmss.chinamobile.com>
    (cherry picked from commit ac25da2479b9be876cbdb820560ac46a6e2b17d7)

commit b6181833be925b7eb11afffff7f03486bdde2d25
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Mon Feb 20 15:05:18 2017 -0500

    rgw_file: fix marker computation
    
    Fixes: http://tracker.ceph.com/issues/19018
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit 4454765e7dd08535c50d24205858e18dba4b454c)
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit ed9308cba2af227991a50a9535df30cf9bb18a82
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Sun Feb 19 20:34:31 2017 -0500

    rgw_file: rgw_readdir can't list multi-segment dirs
    
    This issue has one root cause in librgw, namely that the marker
    argument to these requests was incorrectly formatted (though the
    marker cache was working as intended).
    
    Secondarily, for nfs-ganesha users, there is a compounding issue
    that the RGW fsal was required by "temporary" convention to
    populate the entire dirent cache for a directory on a single
    readdir() invocation--the cache_inode/mdcache implementations
    invariantly pass (before future 2.5 changesets, currently in
    progress) a null pointer for the start cookie offset, intended
    to convey this.
    
    Fixes: http://tracker.ceph.com/issues/18991
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit 2cd60ee9712291b906123aca1704288b18a9742b)
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit 26a38e2e1525714978feda018c03698d7af65129
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Sun Feb 19 18:21:06 2017 -0500

    rgw_file: allow setattr on placeholder directories
    
    When a POSIX path <bucket>/foo/ is known only as an implicit path
    segment from other objects (e.g., <bucket>/foo/bar.txt), a case
    that would usually arise from S3 upload of such an object, an
    RGWFileHandle object representing "<bucket>/foo/" will be constructed
    as needed, with no backing in RGW.
    
    This is by design, but subsequently, if a setattr is performed on
    such a handle, we must be ready to create the object inline with
    storing the attributes.
    
    Fixes: http://tracker.ceph.com/issues/18989
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit 55eec1c0a0e136736961423b7b6244d0f3693c1a)
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit 7e4e29038bd820e171d007360cf383c85f67879b
Author: Gui Hecheng <guihecheng@cmss.chinamobile.com>
Date:   Wed Mar 8 16:23:11 2017 +0800

    rgw_file: fix reversed return value of getattr
    
    When ::getattr returns -ESTALE, rgw_getattr returns ESTALE,
    which is a not expected postive.
    
    Signed-off-by: Gui Hecheng <guihecheng@cmss.chinamobile.com>
    (cherry picked from commit 39203cf872b8f4af86eb0e4a0f96dffd9cc92b41)

commit 3c02ee4fe9ae70f30328a7b015f5cac4a1804c67
Author: Gui Hecheng <guihecheng@cmss.chinamobile.com>
Date:   Mon Mar 20 10:53:46 2017 +0800

    rgw_file: fix non-negative return code for open operation
    
    The nfs-ganesha expects a negative retcode for errors.
    
    Signed-off-by: Gui Hecheng <guihecheng@cmss.chinamobile.com>
    (cherry picked from commit b5f70ef7b066f9ff44770cc8a50ccadaa02f4691)

commit e9a4903c6e77caae6f161056e8aaa5ba10ae5ef2
Author: Gui Hecheng <guihecheng@cmss.chinamobile.com>
Date:   Wed Mar 15 15:01:05 2017 +0800

    rgw_file: fix double unref on rgw_fh for rename
    
    Skip unref after unlink to fix the problem.
    
    Signed-off-by: Gui Hecheng <guihecheng@cmss.chinamobile.com>
    (cherry picked from commit bff228734c73b536d2482e2e2fa4ad38b206ebff)

commit 9910eac98474930369d694b236c06ffd627fee04
Author: Yehuda Sadeh <yehuda@redhat.com>
Date:   Thu Sep 29 22:20:10 2016 -0700

    rgw: multipart part copy, fix regression
    
    Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
    (cherry picked from commit 66fbe9384703c004c01783eb664f55895d310439)
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit a3fdf0e246637ef4145b6b5ba1f4b114ec7ebc62
Author: Yehuda Sadeh <yehuda@redhat.com>
Date:   Thu Sep 29 17:44:08 2016 -0700

    rgw: minor optimization
    
    Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
    (cherry picked from commit 4919dc9987c6376d3d4e143702c26417449524c5)
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit 2161376baf413acfbf02df07e404d2918729bfcc
Author: Yehuda Sadeh <yehuda@redhat.com>
Date:   Thu Sep 29 17:43:00 2016 -0700

    rgw: rgw_obj_key use adjustment in multipart copy part
    
    This fixes a case where objects start with double underscore.
    
    Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
    (cherry picked from commit 29fece3545cc1df404a25eec46706b16f893a5df)
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit 135f84d860d10a2961f430708983113a87ddf899
Author: Yehuda Sadeh <yehuda@redhat.com>
Date:   Thu Sep 29 17:43:35 2016 -0700

    rgw: multipart copy-part handle versionId
    
    Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
    (cherry picked from commit 53521efffb1cb92e5f5ce992d4127bf9498d7c33)
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit 9ab65f5501fb59aac17cfdde57371b00af03d84b
Author: Yehuda Sadeh <yehuda@redhat.com>
Date:   Thu Sep 29 14:24:13 2016 -0700

    rgw: multipart copy part minor parsing cleanup
    
    no need for range.size()
    
    Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
    (cherry picked from commit 6e9b824d5d4017239d58b4752ebc43bfad8f698d)
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit 3eb9dc96ff8e655415500a3595a78cab80739826
Author: Yehuda Sadeh <yehuda@redhat.com>
Date:   Thu Sep 29 14:11:43 2016 -0700

    rgw: multipart copy, check for empty bucket, improve logging
    
    also reduce log level for non critical user errors.
    
    Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
    (cherry picked from commit 2bcb3d286b230ef917d5ba96c8276a942f544689)
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit 2588e95d9d422d3b33fd710de1f5884873465483
Author: Yehuda Sadeh <yehuda@redhat.com>
Date:   Thu Sep 29 14:07:14 2016 -0700

    rgw: multipart copy part, chunked read
    
    Don't read the entire range from source object, read it in parts.
    
    Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
    (cherry picked from commit 4049e47a0cfc1eef6efd502590b68ba7234589d3)
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit bd118b556562176ce2390a935b3bb8c25a62d0c4
Author: Javier M. Mellid <jmunhoz@igalia.com>
Date:   Tue Jul 26 14:56:50 2016 +0200

    rgw: doc: add multipart uploads copy part feature as supported
    
    Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
    (cherry picked from commit 8a7ebeee2ff3f10ceb23b7fa43e43c3c450efe22)
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit b56b719299becc38ec36d427a36b8c29f2416c08
Author: Javier M. Mellid <jmunhoz@igalia.com>
Date:   Tue Jul 26 14:53:44 2016 +0200

    rgw: multipart uploads copy part support
    
    Add multipart uploads copy part feature.
    
    Fixes: http://tracker.ceph.com/issues/12790
    
    Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com>
    (cherry picked from commit 949480c2e9760855ed6a0501d364d5e766c8207d)
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit d44263fc91da12ea0ad4fec0cd2877b36ddb9e9f
Author: Radoslaw Zarzynski <rzarzynski@mirantis.com>
Date:   Fri Feb 17 00:56:34 2017 +0100

    rgw: make sending Content-Length in 204 and 304 controllable
    
    This commit introduces a new configurable "rgw print prohibited
    content length" to let operator decide whether RadosGW complies
    to RFC 7230 (a part of the HTTP specification) or violates it
    but follows the Swift's behavior.
    
    Fixes: http://tracker.ceph.com/issues/16602
    Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
    (cherry picked from commit d8e3e64ec97a3c222a56bb6f510e5e23d7858615)
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

commit f3face61351a61f3b71dfb8268a4b645c4d92da2
Author: Casey Bodley <cbodley@redhat.com>
Date:   Thu Mar 9 15:24:08 2017 -0500

    rgw: clear old zone short ids on period update
    
    the short ids of old, removed zones were being kept in the period to
    guard against hash collisions with new zones
    
    but for a hash collision to cause a wrong object to sync, that object
    would have to be uploaded simultaneously to two different zones that had
    the same short id
    
    to avoid this, we just have to prevent the period from containing two
    colliding zones at the same time - we don't have to remember old zone
    short ids forever
    
    Fixes: http://tracker.ceph.com/issues/15618
    
    Signed-off-by: Casey Bodley <cbodley@redhat.com>
    (cherry picked from commit 9c45633c836c966ab1f75ea2b1ad3fa0a4886600)

commit 8c02e54a747644d24feb98dbc15a15cdd55d1afd
Merge: 9d222b9 952f34f
Author: David Zafman <dzafman@redhat.com>
Date:   Fri Mar 24 08:13:02 2017 -0700

    Merge pull request #13146 from dzafman/wip-18502-jewel-zafman
    
    jewel: osd: Scrub improvements and other fixes
    
    Reviewed-by: Kefu Chai <kchai@redhat.com>

commit 9d222b943217a2ec022678e50b34766d231004ce
Merge: ce2e989 cb9fae5
Author: Nathan Cutler <presnypreklad@gmail.com>
Date:   Thu Mar 23 16:49:13 2017 +0100

    Merge pull request #14100 from smithfarm/wip-18781
    
    Manually merge PR#13143 which was mistakenly merged to jewel-next
    
    Reviewed-by: Loic Dachary <ldachary@redhat.com>
    Reviewed-by: Ken Dreyer <kdreyer@redhat.com>

commit cb9fae56ebdfda28b0606ee89631deb60e5f7de7
Merge: ce2e989 ee3c67c
Author: Nathan Cutler <ncutler@suse.com>
Date:   Thu Mar 23 09:46:53 2017 +0100

    Merge branch 'jewel-next' into jewel
    
    Branch "jewel-next" was merged to jewel on Dec 22 (5b402f8a7b), and then PR
    https://github.com/ceph/ceph/pull/13143 merged to jewel-next much later (Feb
    1st), and then jewel-next was never merged to jewel again before we tagged
    v10.2.6.
    
    This merge brings in a single commit, i.e. the one from
    https://github.com/ceph/ceph/pull/13143
    
    Reported-by: Ken Dreyer <kdreyer@redhat.com>
    Signed-off-by: Nathan Cutler <ncutler@suse.com>

commit ce2e9897c08775cfbe318f637d61e07eb5433df5
Merge: 656b5b6 d0a0d2f
Author: Zack Cerza <zack@cerza.org>
Date:   Tue Mar 14 09:19:28 2017 -0600

    Merge pull request #13952 from smithfarm/wip-fix-merge-error
    
    tests: fix merge error in rgw/singleton/all/radosgw-admin.yaml

commit 0be4e89419f2083d081ac784891e4653290cd530
Author: Casey Bodley <cbodley@redhat.com>
Date:   Tue Oct 11 15:21:42 2016 -0400

    rgw: hold a reference on data_sync_cr over run()
    
    run() will drop its reference to data_sync_cr, so we need to hold a
    reference until we can reacquire the lock
    
    Signed-off-by: Casey Bodley <cbodley@redhat.com>
    (cherry picked from commit 4cf0d2a768e7402e71280ca16b47353ca2a68505)

commit 18ffdb7baf4aff1fd7f894af1054081f3ee61d28
Author: Casey Bodley <cbodley@redhat.com>
Date:   Tue Oct 11 15:19:37 2016 -0400

    rgw: clear data_sync_cr if RGWDataSyncControlCR fails
    
    async notifications will still try to call wakeup() on RGWDataSyncControlCR
    if it fails, leading to segfault
    
    Fixes: http://tracker.ceph.com/issues/17569
    
    Signed-off-by: Casey Bodley <cbodley@redhat.com>
    (cherry picked from commit 5cc599b9bf2dde31de16a5b2831baf06851d69d5)
    
     Conflicts:
    	src/rgw/rgw_data_sync.cc: declaration of 'int r'

commit d0a0d2f4bf1dacf7c1f38b968be84a5e905554a8
Author: Nathan Cutler <ncutler@suse.com>
Date:   Tue Mar 14 01:26:23 2017 +0100

    tests: fix merge error in rgw/singleton/all/radosgw-admin.yaml
    
    Introduced by 94d5888c
    
    Reported-by: Robin H. Johnson <robin.johnson@dreamhost.com>
    Signed-off-by: Nathan Cutler <ncutler@suse.com>

commit 19856624cd5ea8cb82bc5b46e062beb39674bd34
Author: Casey Bodley <cbodley@redhat.com>
Date:   Mon Feb 20 16:00:01 2017 -0500

    rgw: RGWMetaSyncShardControlCR retries with backoff on all error codes
    
    RGWBackoffControlCR only treats EBUSY and EAGAIN as 'temporary' error
    codes, with all other errors being fatal when exit_on_error is set
    
    to RGWMetaSyncShardControlCR, a 'fatal' error means that no further sync
    is possible on that shard until the gateway restarts
    
    this changes RGWMetaSyncShardControlCR to set exit_on_error to false, so
    that it will continue to retry with backoff no matter what error code it
    gets
    
    Fixes: http://tracker.ceph.com/issues/19019
    
    Signed-off-by: Casey Bodley <cbodley@redhat.com>
    (cherry picked from commit 3e4059557fd6cad5d31014327f60832b36d04a6c)

commit dfaaec0446c3db458bffe1d725518ab4f7cc2fa8
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Thu Feb 23 10:21:38 2017 -0500

    rgw_file:  return of RGWFileHandle::FLAG_EXACT_MATCH
    
    Allow callers of rgw_lookup() on objects attested in an
    rgw_readdir() callback the ability to bypass exact match in
    RGWLibFS::stat_leaf() case 2, but restore exact match enforcement
    for general lookups.
    
    This preserves required common_prefix namespace behavior, but
    prevents clients from eerily permitting things like "cd sara0" via
    partial name match on "sara01."
    
    Fixes: http://tracker.ceph.com/issues/19059
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit 70ef7d45e0abf2661bd4e23161d4e70cf5178079)

commit 9b3784d924112d9ba42b2088d5fb2656ef74fadc
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Sun Feb 19 17:43:17 2017 -0500

    rgw_file: invalid use of RGWFileHandle::FLAG_EXACT_MATCH
    
    The change which introduced this flag also caused it to be
    given as the flags argument to RGWLibFS::stat_leaf() when called
    from rgw_lookup().
    
    This was incorrect:  in particular, when a directory is known only
    as a common prefix of other objects, the AWS namespace mapping
    convention requires lookup("foo") to match a non-materialized
    instance of "foo/" (case 2 in RGWLibFS::stat_leaf's stat loop).
    
    Fixes: http://tracker.ceph.com/issues/18992
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit e31e9eb980f958640150e8d7f17de1b9e5478b1e)

commit 7761376c92eb38a2dd3d19f0b3d81895a4b1167c
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Wed Feb 22 14:57:59 2017 -0500

    rgw_file: rgw_lookup: don't ref for "/" or ".."
    
    These refs won't be returned by nfs-ganesha, and are sufficiently
    magical that other consumers should be persuaded to understand
    their specialness.
    
    Fixes: http://tracker.ceph.com/issues/19060
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit dea8d1ee373399a21851690a9753388b659b8ede)

commit 97fed013f66a89c404bac5145080e5d556ff5c42
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Wed Feb 22 10:24:29 2017 -0500

    rgw_file: avoid stranding invalid-name bucket handles in fhcache
    
    To avoid a string copy in the common mkdir path, handles for
    proposed buckets currently are staged in the handle table, before
    being rejected.  They need to be destaged, not just marked deleted
    (because deleted objects are now assumed not to be linked, as of
    beaeff059375b44188160dbde8a81dd4f4f8c6eb).
    
    This triggered an unhandled Boost assert when deleting staged
    handles, as current safe_link mode requires first removing from
    the FHCache.
    
    Fixes: http://tracker.ceph.com/issues/19036
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit 6cde812c92e5bba9f85fbf8486ebe69b55952370)

commit d333addb71a0a92fbac5a7b922dbd69fc9e0604e
Author: Brad Hubbard <bhubbard@redhat.com>
Date:   Mon Feb 27 13:06:59 2017 +1000

    rgw: set dumpable flag after setuid post ff0e521
    
    ff0e521 resolved the issue for the other daemons but not for rgw since
    it calls setuid (via civetweb) after the new code sets PR_SET_DUMPABLE.
    Add another prctl call before wait_shutdown.
    
    Fixes: http://tracker.ceph.com/issues/19089
    
    Signed-off-by: Brad Hubbard <bhubbard@redhat.com>
    (cherry picked from commit bc458d39630b599e0e1ca9fe25ad7455fcffdd10)

commit f7ce5df4064e5538156cb44f9525d7552a0dd098
Author: Jing Wenjun <jingwenjun@cmss.chinamobile.com>
Date:   Wed Feb 8 15:07:43 2017 +0800

    rgw: fix swift cannot disable object versioning with empty X-Versions-Location
    
    we should be able to disable object verioning by removing its X-Versions-Location
    metadata header by sending an empty key value. this description can be found at
    No.8 in http://docs.openstack.org/user-guide/cli-swift-set-object-versions.html.
    
    Fixes: http://tracker.ceph.com/issues/18852
    Signed-off-by: Jing Wenjun <jingwenjun@cmss.chinamobile.com>
    (cherry picked from commit 17c5a0edd2227703cec867f0f588d4eae36dfe1c)

commit 2f2032814189a4ecbf8dc01b59bebfae8ab3f524
Author: Bartłomiej Święcki <bartlomiej.swiecki@corp.ovh.com>
Date:   Fri Dec 2 16:54:46 2016 +0100

    osd: Increase priority for inactive PGs backfill
    
    This change does prioritize backfill of PGs which don't
    have min_size active copies. Such PGs would cause IO stalls
    for clients and would increase throttlers usage.
    
    This change also fixes few subtlle out-of-bounds bugs.
    
    Signed-off-by: Bartłomiej Święcki <bartlomiej.swiecki@corp.ovh.com>
    (cherry picked from commit 6a76adcdb1f92c136841d960aa7cd4e5b94addec)
    
    Conflicts:
    	PendingReleaseNotes (removed version number, merged conflicts)

commit e8041f627c6a05a59dba29819ed610ea0896f4dd
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Fri Feb 3 13:44:45 2017 -0500

    rgw_file:  fix RGWLibFS::setattr for directory objects
    
    Fixes:  http://tracker.ceph.com/issues/18808
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit 4ad5a9226852d6d564baf2e63278ed6c4c185ecb)

commit e475bfaf7d3a1b0e54172083a92546560219665a
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Thu Feb 23 16:02:07 2017 -0500

    rgw_file: ensure valid_s3_object_name for directories, too
    
    The logic in RGWLibFS::mkdir() validated bucket names, but not
    object names (though RGWLibFS::create() did so).
    
    The negative side effect of this was not creating illegal objects
    (we won't), but in a) failing with -EIO and b) more importantly,
    not removing up the proposed object from FHCache, which produced a
    boost assert when recycled.
    
    Fixes: http://tracker.ceph.com/issues/19066
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit eb1cd3b30c0504385f05bf2d2dd5e2251b7efed7)

commit 78c8be7a0df3d1c669f8a2a8fd7a5676d0823209
Author: Marcus Watts <mdw@linuxbox.com>
Date:   Thu Feb 23 02:30:52 2017 -0500

    rgw/openssl fix: xenial autoconf logic problem: gcc/ld got too smart...
    
    On xenial, cc -o foo -lssl -lcrypto doesn't always
    record libssl.so libcrypto.so as runtime library dependencies.
    It is necessary to actually *use* a function from the library before
    it gets recorded.  The ld(1) options "--as-needed" and "no-as-needed"
    control this.  Evidently the default has changed in xenial.
    That caused my smart "soname" detecting logic for openssl to
    stop working.  To make it work, the test program has to
    actually reference routines or variables inside the library.
    
    This is a quick fix for xenial / autoconf.  There needs to be
    a better fix for cmake and master.
    
    Signed-off-by: Marcus Watts <mwatts@redhat.com>

commit db928d6b3f983b3a1ccc07440fcd3680700a1188
Author: Jason Dillaman <dillaman@redhat.com>
Date:   Tue Feb 21 15:33:01 2017 -0500

    rbd-mirror: retry object copy after -ENOENT error
    
    Fixes: http://tracker.ceph.com/issues/18990
    Signed-off-by: Jason Dillaman <dillaman@redhat.com>
    (cherry picked from commit b4f36d5dc3f4f3cbb23f61cbb945b222248a50df)
    
    Conflicts:
    	src/test/librados_test_stub/MockTestMemIoCtxImpl.h: sparse reads not supported
    	src/test/rbd_mirror/image_sync/test_mock_ObjectCopyRequest.cc: sparse reads not supported

commit a643fa80e03b41edcc720ff77b69ebaf24a23f3e
Author: Jason Dillaman <dillaman@redhat.com>
Date:   Tue Feb 21 13:09:39 2017 -0500

    rbd-mirror: object copy should always reference valid snapshots
    
    If a remote snapshot is deleted while an image sync is in-progress,
    associate the read request against the most recent, valid remote
    snapshot for a given snapshot object clone.
    
    Signed-off-by: Jason Dillaman <dillaman@redhat.com>
    (cherry picked from commit 9a91efc3047963364944f8be91cee8e8f6afc49a)

commit 37bbc95d56f73e4ec9c6e13ddbae199b14b104b4
Author: Jason Dillaman <dillaman@redhat.com>
Date:   Tue Feb 21 11:52:00 2017 -0500

    rbd-mirror: replace complex object op tuple with struct
    
    Signed-off-by: Jason Dillaman <dillaman@redhat.com>
    (cherry picked from commit 0c181527c0e151784a0f7c466aaa70b0772f91b1)
    
    Conflicts:
    	src/tools/rbd_mirror/image_sync/ObjectCopyRequest.cc: sparse reads not supported
    	src/tools/rbd_mirror/image_sync/ObjectCopyRequest.h: sparse reads not supported

commit 8941881e8986af0b2871c15d188d94d49e95dc02
Author: Matt Benjamin <mbenjamin@redhat.com>
Date:   Fri Dec 30 23:30:16 2016 -0500

    rgw_file:  interned RGWFileHandle objects need parent refs
    
    RGW NFS fhcache/RGWFileHandle operators assume existence of the
    full chain of parents from any object to the its fs_root--this is
    a consequence of the weakly-connected namespace design goal, and
    not a defect.
    
    This change ensures the invariant by taking a parent ref when
    objects are interned (when a parent ref is guaranteed).  Parent
    refs are returned when objects are destroyed--essentially by the
    invariant, such a ref must exist.
    
    The extra ref is omitted when parent->is_root(), as that node is
    not in the LRU cache.
    
    Fixes: http://tracker.ceph.com/issues/18650
    
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit 0e5299f3f43e633a5d8a9360893b4b11f6217d81)

commit 952f34f39cdd0438e4a4fb369ea8ca20b26488a8
Author: David Zafman <dzafman@redhat.com>
Date:   Mon Dec 12 15:53:25 2016 -0800

    test: Update for new error message when doing scrub with deep-scrub errors
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit 85e0774a7dded4aa6c67f237416041e25a7680bc)

commit b655b98e48989ae954dc150fcb3d8976978cd90d
Author: David Zafman <dzafman@redhat.com>
Date:   Fri Dec 9 12:14:06 2016 -0800

    osd: Add "trigger_scrub" admin socket command
    
    This allows testing to fake a schedule scrub (must_scrub not set)
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit c8dc945260ee2ee841aca00fbc802d41036033d6)

commit 94c958d2c9570e55305384ac86185e328746d2ff
Author: David Zafman <dzafman@redhat.com>
Date:   Thu Dec 8 23:00:13 2016 -0800

    test: Add test for keeping deep-scrub information
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit 64a7012e986ec88994c073b738fd08e8958395c3)

commit 42eb24f18d5114410bd2a3e84e9219584d9e165e
Author: David Zafman <dzafman@redhat.com>
Date:   Thu Dec 1 16:39:42 2016 -0800

    osd: When deep-scrub errors present upgrade regular scrubs
    
    Previously, if a weekly deep-scrub found errors the next daily scrub
    would cause the deep-scrub information to be removed.
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit 532a759dca466181f37a329f86045c34f1a2506f)

commit 76a457aa7c411dc5ed22f171954b8ed3c4661845
Author: David Zafman <dzafman@redhat.com>
Date:   Wed Oct 19 17:10:29 2016 -0700

    tasks/scrub_test.py: Make test deterministic by updating digests
    
    ceph_test_rados: Get coverage of digest updates in deep-scrub/repair
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit e000ab2f46b7b98a223176a2a25bb7195e601af6)

commit bd1f1983129d6e1787b8b5eb4884c5e908f7b274
Author: David Zafman <dzafman@redhat.com>
Date:   Wed Oct 19 17:10:07 2016 -0700

    repair_test, scrub_test: Fix whitelists for scrub changes
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit 5fe8522df47f56842d227e08b2912623504afe24)

commit 28106f06404407df7d05b35f9f570a80e785e635
Author: David Zafman <dzafman@redhat.com>
Date:   Wed Aug 31 12:02:31 2016 -0700

    scrub_test: Fix for list-inconsistent-obj output changes
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit 5ae0f5c75a8236d4a026f9ddcb5ff59964d90579)

commit abcefc1395f76e5b05a988b970457a7f6dac6e8e
Author: David Zafman <dzafman@redhat.com>
Date:   Tue Aug 30 12:11:44 2016 -0700

    doc, test: Add schemas for list-inconsistent-* rados command output
    
    If jsonschema cmd is available use it to test output against schema
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit eb73dd473600fbbb45fad00194f7c46b565d6b81)

commit 2fa2b64676555a91883a5aabb4e4237124308629
Author: David Zafman <dzafman@redhat.com>
Date:   Tue Aug 30 12:22:55 2016 -0700

    test: Update testing for new list-inconsistent-obj output
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit b7bacd219b000869b9c93e21edba4f8a3ace60d9)
    
    Adapted for difference in Jewel object_info

commit 304f697ff1106695188b572e0da2415437040a6c
Author: David Zafman <dzafman@redhat.com>
Date:   Tue Aug 30 12:22:29 2016 -0700

    rados, osd: Improve attrs output of list-inconsistent-obj
    
    Persist the user_version and shard id of scrubbed obj
    Rados command dump inconsistent obj's version and shard-id
        so they can be passed to repair command
    Rados list-inconsistent-obj output of attrs
        Make attrs an array since there are more than one
        Use base64 encode for values with non-printable chars
        Add indication if base64 encoding used
    Add checking for ss_attr_missing and ss_attr_corrupted
        Rename attr errors to attr_key_mismatch and attr_value_mismatch
    Add missing size_mismatch_oi scrub checking
    For erasure coded pools add ec_size_error and ec_hash_error not just read_error
    Use oi_attr_missing and oi_attr_corrupted just like list-inconsistent-snap does
    Pick an object info based on version and use that to find specific shards in error
        Check for object info inconsistency which should be rare
    Make all errors based on comparing shards to each other object errors
        We don't want give the impression that we've picked the correct one
    
    Signed-off-by: Kefu Chai <kchai@redhat.com>
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit df3ff6dafeadb3822b35c424a890db9a14d7f60f)

commit ea52f527e530b111b1bb26d10824c0230d662c4a
Author: David Zafman <dzafman@redhat.com>
Date:   Thu Sep 1 14:45:01 2016 -0700

    osd: Fix logging to help with diagnostics
    
    These messages were not outputing the intended information
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit b39001ec6ff0996860bcb5a3578bc2c66355c781)

commit 5e8f8a2288aa1a9f1de86d3f7d0f9c66c795113e
Author: David Zafman <dzafman@redhat.com>
Date:   Tue Jun 21 18:05:25 2016 -0700

    test: Fix use of wait_for_clean()
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit d0503a52d0fe5505bdb38dfd613c03a20500c05d)

commit 2230e6adc66ca3e5dbe81aecccdb4a435e93e256
Author: David Zafman <dzafman@redhat.com>
Date:   Tue Aug 30 10:56:06 2016 -0700

    common: Change cleanbin() to use base64 encoding, update ceph-objectstore-tool
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit 92e982c71995b863466d83671468f84761cb1793)

commit 83ea077ee560d31a5c302a62b55451a2571fda8d
Author: David Zafman <dzafman@redhat.com>
Date:   Tue Aug 30 11:05:16 2016 -0700

    common: Move cleanbin() function to common/util.cc
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit 5c79074ffaee34b2956d9dfc67b1eff9f39b47f3)
    
    Conflicts:
    	src/tools/CMakeLists.txt (changes goes in src/CMakeLists.txt)

commit ba84ca9fa391d963d1d4e46fbf27ce3dbff397be
Author: David Zafman <dzafman@redhat.com>
Date:   Wed Jun 8 09:48:00 2016 -0700

    test: Add test support for deep-scrub
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit 907e79e2b77835d1aca9e8ba2fae5fa2fd437e5a)

commit 6a421d19d0be4d59beed5c69bb0aa4477d65a14e
Author: David Zafman <dzafman@redhat.com>
Date:   Sat Jun 18 17:58:36 2016 -0700

    common: Fix indentation
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit a74418a29e4e469117fc0c4edd80f78b62944c98)

commit c7c3e070336dc898460d4338b4d78b8963467c8a
Author: David Zafman <dzafman@redhat.com>
Date:   Fri Aug 12 12:06:31 2016 -0700

    osd: Handle corrupt attributes in get_object_context()
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit 30f8b0d6593718dd10bcf6ff459c95b4bb68c05c)

commit 8006ba7e86ebb8201a040c427cc95197901064be
Author: Kefu Chai <kchai@redhat.com>
Date:   Fri Nov 11 21:37:50 2016 +0800

    ReplicatedPG::failed_push: release read lock on failure
    
    and requeue the blocked ops.
    
    Fixes: http://tracker.ceph.com/issues/17857
    Signed-off-by: Kefu Chai <kchai@redhat.com>
    (cherry picked from commit b3224a18f6acc7ed54c2162b140a33b6146a16be)

commit 5ca69d57dbe2ee7acc64d28ca35bb390bf463199
Author: David Zafman <dzafman@redhat.com>
Date:   Fri Oct 14 16:17:55 2016 -0700

    test.sh: Make check for flags more robust
    
    Low space broke test, saw "flags nearfull,pauserd,pausewr...."
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit b4c080b1b4378d659c1ca8a17811cd6f84595166)

commit 1be38e5ebd2d01d5d527b05e64b026df955ffe69
Author: David Zafman <dzafman@redhat.com>
Date:   Thu Jul 21 17:36:34 2016 -0700

    test: Remove extra objectstore_tool call which causes a recovery
    
    Caused by: 70e000a9a42c50eda48f5d3b9e432ffc3a70f75b
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit 6904529d1b53993304de2927500937ba0d493e9e)

commit 026f181e72f2e09e769a8821dfe2c99f6213e6a3
Author: David Zafman <dzafman@redhat.com>
Date:   Tue Jun 14 20:09:15 2016 -0700

    test: Handle object removals in a non-racey way
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit bfe3ebd94fdc1ef36ebe6e8f9d38acea322eca22)

commit c5ef385c25086dc1582ebe5343481d05283b0cc6
Author: David Zafman <dzafman@redhat.com>
Date:   Thu May 26 22:09:42 2016 -0700

    osd: Fix hang on unfound object after mark_unfound_lost is done
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit 73a27533beba3587f8447b4d41d200427c45042b)
    
    Conflicts:
    	src/osd/ReplicatedPG.cc (trivial)

commit 6284f745157458439699c76e8616455c50d1eb71
Author: David Zafman <dzafman@redhat.com>
Date:   Fri May 20 15:20:18 2016 -0700

    osd: Handle recovery read errors
    
    Fixes: http://tracker.ceph.com/issues/13937
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit c51d70e1e837c972e42ddd5fa66f7ca4477b95cc)
    
    Conflicts:
    	src/osd/ReplicatedPG.h (trivial)

commit 27393a8c05d4656f342ecd32817307e558a2e400
Author: David Zafman <dzafman@redhat.com>
Date:   Fri May 20 18:19:42 2016 -0700

    osd: Fix log messages
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit b40ec3fe890679b4c73778815e1bedd492cb264b)

commit cbf66f3b16f194dd0c67b46e0fec247e02e84134
Author: David Zafman <dzafman@redhat.com>
Date:   Fri May 20 13:58:32 2016 -0700

    osd: CLEANUP: Remove unused pending_read member
    
    Remove unused struct
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit 36fd68c96653e83f87767feb08530a9fc6e841b0)

commit 228b91d72a15b62adc10591604c4e8a849df53d5
Author: David Zafman <dzafman@redhat.com>
Date:   Tue Oct 11 18:08:12 2016 -0700

    test/osd-scrub-repair.sh: Use test case specific object names to help with diagnostics
    
    Signed-off-by: David Zafman <dzafman@redhat.com>
    (cherry picked from commit 0bf4da589155ee50969812492cfbc66368efb54c)

commit ee3c67c9cbfdeeb3e628bac34b708cf150b3862e
Merge: 3dbf0c9 39848e4
Author: Nathan Cutler <presnypreklad@gmail.com>
Date:   Wed Feb 1 23:28:22 2017 +0100

    Merge pull request #13143 from linuxbox2/jewel-mdw-rgw-lf
    
    jewel: rgw: radosgw/swift: clean up flush / newline behavior.
    
    Reviewed-by: Nathan Cutler <ncutler@suse.com>

commit 39848e41b7c517cc5faab1ccf77c2804fd7d2628
Author: Marcus Watts <mwatts@redhat.com>
Date:   Wed Jan 11 00:06:15 2017 -0500

    radosgw/swift: clean up flush / newline behavior.
    
    The current code emits a newline after swift errors, but fails
    to account for it when it calculates 'content-length'.  This results in
    some clients (go github.com/ncw/swift) producing complaints about the
    unsolicited newline such as this,
    	Unsolicited response received on idle HTTP channel starting with "\n"; err=<nil>
    
    This logic eliminates the newline on flush.  This makes the content length
    calculation correct and eliminates the stray newline.
    
    There was already existing separator logic in the rgw plain formatter
    that can emit a newline at the correct point.  It had been checking
    "len" to decide if previous data had been emitted, but that's reset to 0
    by flush().  So, this logic adds a new per-instance variable to separately
    track state that it emitted a previous item (and should emit a newline).
    
    Fixes: http://tracker.ceph.com/issues/18473
    Signed-off-by: Marcus Watts <mwatts@redhat.com>
    Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
    (cherry picked from commit 5f229d6a33eae4906f22cdb90941835e47ee9f02)

commit 3fce77ab3662496368e25cbbf6d1b37d3c140db9
Author: Vikhyat Umrao <vumrao@redhat.com>
Date:   Wed Jan 25 21:54:27 2017 +0530

    osd: jewel fix OSD network address in OSD heartbeat_check log message
    
    Fixes: http://tracker.ceph.com/issues/18657
    
    Signed-off-by: Vikhyat Umrao <vumrao@redhat.com>

commit 14a6aabe22f68436ea3297ce0851700f86ee5b12
Author: Marcus Watts <mwatts@redhat.com>
Date:   Wed Aug 3 17:36:55 2016 -0400

    rgw: Handle multiple listening addreses w/ optional ssl "correctly" with civetweb.
    
    For civetweb: accept a range of port numbers joined with '+'.
    Port numbers may include an ipaddress: prefix and 's' suffix.
    Additionally, use "mg_get_local_addr" to correctly deduce host port per
    incoming connection.
    
    civetweb can accept connections on multiple ports, some of which
    might have SSL turned on and some not.  Both s3 and swift have various
    authorization protocols in which the port number matters.  In the generic
    radosgw frontend process, each frontend only has one port number, but
    we should want to have both ssl and non-ssl connections managed within
    one rgw frontend, because the thread pool is also per front-end, and
    that *is* a scarce resource.
    
    So, this patch enables the use of multiple ports with a single civetweb
    frontend.  To indicate https: append an 's' to portno.  To use multiple
    ports, use +.  So 80+443s indicates use of the usual default http ports.
    The parsed port is not stored in the frontend structure,
    
    So instead, this patch adds logic to use the results of
    mg_get_local_addr() on a per-connection basis insetad of the generic
    front-end port number.  This will affect "v4" s3 authorization, and also
    affect swift pre-signed URLs.
    
    mg_get_local_addr() is a new customization to civetweb; that submodule
    was updated (in a temporary repository) by the previous commit to this.
    
    Signed-off-by: Marcus Watts <mwatts@redhat.com>
    (cherry picked from commit 8bc6decc0774fae9ac881f91e73da55deebe3360)

commit 698250563ccc4c69e5ca5aebf65dc352d80a8bdd
Author: Marcus Watts <mwatts@redhat.com>
Date:   Tue Dec 20 00:22:02 2016 -0500

    rgw: s3: secure_port should override port, also apply ssl default right.
    
    Without https, only port is set.  With https, secure_port and port are
    both set to the same value.  The previous logic looked at port first and
    had overly simplified conditional logic which was liable to try to apply
    both non-default cases.  The correct behavior is: look secure_port first,
    and if secure_port is set, then only check to see if it's a non-default
    port.
    
    Signed-off-by: Marcus Watts <mwatts@redhat.com>
    (cherry picked from commit a113cf5ff5a642d2ee4cc83f5c7001b4bfe0a5df)

commit 28f2841c8b9a832e486b9f89c574a4591bf3d448
Author: Marcus Watts <mwatts@redhat.com>
Date:   Wed Nov 2 00:18:48 2016 -0400

    rgw: Get civetweb ssl enhancement: wip-listen3 = mg_get_local_addr
    
    The logic inside of radosgw that computes aws v4 signatures wants to know
    what server port the client connected.  The patch to civetweb patch adds a
    call mg_get_local_addr() which will permit that code to actually find out
    on what address a connection was received, rather than merely guessing
    based on configuration as it previously did.
    
    Signed-off-by: Marcus Watts <mwatts@redhat.com>
    (cherry picked from commit 46ced9ddd2795f00f014e22e5637070b49e7a6d5)

commit 8d83dfb4176ede7490d0cab589e9566bc7d4e387
Author: Marcus Watts <mwatts@redhat.com>
Date:   Wed Mar 23 17:32:14 2016 -0400

    rgw: Document that radosgw now supports SSL.
    
    This includes information on file format and configuration file syntax.
    
    Signed-off-by: Marcus Watts <mwatts@redhat.com>
    (cherry picked from commit d4e72dfed30274b3cfbad4ac58c0746a98c0148b)

commit e3f80c9d0ed6762a39fc242561f5ea26f0f26546
Author: Marcus Watts <mwatts@redhat.com>
Date:   Fri Jan 13 03:39:57 2017 -0500

    rgw: civetweb/openssl: automagic: load libssl.so and libcrypto.so by soname.
    
    If building with radosgw, always look for openssl library (even when
    building with nss).  Then, use objdump to fetch SONAME from the copies
    of libssl and libcrypto that were found.  When building civetweb; pass
    the library soname values in as the libraries to load with "dlopen".
    
    The issue reported here against master
    http://tracker.ceph.com/issues/16535
    reflects stuff that doesn't seem to have made it into jewel
    	(which had: -lssl -lcrypto hard-wired into it.)
    Still, since people were pretty riled about making that change,
    this puts things in congruence to the final result in master.
    	(which is: runtime shared object load of ssl crypto by soname.)
    
    Fixes: http://tracker.ceph.com/issues/11239
    
    Signed-off-by: Marcus Watts <mwatts@redhat.com>
    (inspired by commit 7caa0bd002110b62514da83a37a2a3deb841267a)

commit 045551485415309ba9bad77e1aee28a0008881ca
Author: Marcus Watts <mwatts@redhat.com>
Date:   Mon Jul 11 17:54:07 2016 -0400

    rgw: civetweb/openssl: Load libssl.so and libcrypto.so by soname.
    
    If building with radosgw, always look for openssl library (even when
    building with nss).  Then, use objdump to fetch SONAME from the copies
    of libssl and libcrypto that were found.  When building civetweb; pass
    the library soname values in as the libraries to load with "dlopen".
    
    This is a problem that went away for a bit, but came back with some
    changes for
    http://tracker.ceph.com/issues/16535
    
    Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1341775
    Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1258961
    
    Fixes: http://tracker.ceph.com/issues/11239
    
    Signed-off-by: Marcus Watts <mwatts@redhat.com>
    (cherry picked from commit 7caa0bd002110b62514da83a37a2a3deb841267a)

commit 386640865dee30d38f17e55fc87535e419bc3cb5
Author: Marcus Watts <mwatts@redhat.com>
Date:   Thu Nov 3 23:33:44 2016 -0400

    rgw: cmake: remove useless civetweb include path side effect.
    
    For 'target_include_directories" for the cmake object library
    'civetweb_common_objs', change from PUBLIC to PRIVATE.  This doesn't
    break anything, so it wasn't doing anything useful.  If it has it
    any effect, it would be to cause everything that linked against this
    "library" to also use the indictated include path.  Which would be great
    except everything in ceph wants to include "civetweb/civetweb.h" and
    not "civetweb.h".  We already make separate arrangements elsewhere for
    that to work.  Additionally, static object libraries in cmake aren't
    really libraries, so I'm not entirely sure this even does anything.
    So: making this public is not useful, and could be harmful.  Making it
    private makes this only take effect for building civetweb.c itself,
    exactly the effect we we require, and no more.
    
    Signed-off-by: Marcus Watts <mwatts@redhat.com>
    (cherry picked from commit 8308a13b0257c9460fd2a721c20b0c37cb9e7c57)