]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
rgw: allow larger payload for period commit 15447/head
authorCasey Bodley <cbodley@redhat.com>
Wed, 5 Apr 2017 18:20:20 +0000 (14:20 -0400)
committerNathan Cutler <ncutler@suse.com>
Wed, 12 Jul 2017 13:54:30 +0000 (15:54 +0200)
testing with 3 zonegroups and 3 zones each, the period json grew larger
than 4k and caused decode failures on period commit

updated to use the new config variable rgw_max_put_param_size

Fixes: http://tracker.ceph.com/issues/19505
Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 7f2871fe59d933b03f37fde40f1781b2320d0d50)

Conflicts:
    src/rgw/rgw_rest_realm.cc - in RGWOp_Period_Post::execute(), set max_size
        explicitly instead of backporting rgw_max_put_param_size

src/rgw/rgw_rest_realm.cc

index 008ab10d633baa97b50e465d516b7ce5f1029a88..fb1ba2f10e73569d05a6a0b3a2ab84e5279b47a2 100644 (file)
@@ -84,9 +84,9 @@ void RGWOp_Period_Post::execute()
   period.init(cct, store, false);
 
   // decode the period from input
-#define PERIOD_MAX_LEN 4096
+  const uint64_t max_size = 1 * 1024 * 1024;
   bool empty;
-  http_ret = rgw_rest_get_json_input(cct, s, period, PERIOD_MAX_LEN, &empty);
+  http_ret = rgw_rest_get_json_input(cct, s, period, max_size, &empty);
   if (http_ret < 0) {
     lderr(cct) << "failed to decode period" << dendl;
     return;