]> git-server-git.apps.pok.os.sepia.ceph.com Git - radosgw-agent.git/log
radosgw-agent.git
11 years agoworker: check op state for progress on any HTTP error 14/head
Josh Durgin [Thu, 18 Dec 2014 05:49:35 +0000 (21:49 -0800)]
worker: check op state for progress on any HTTP error

500 can be caused by an fcgi timeout when the operation still ends up
succeeding. It doesn't hurt to check for op state being in progress in
general in case other error codes happen but don't indicate actual
failure of the copy, since wait_for_object fails immediately if the
op state is not in progress.

Signed-off-by: Josh Durgin <jdurgin@redhat.com>
11 years agosync: use ' ' as the default shard so retries can be stored
Josh Durgin [Thu, 18 Dec 2014 05:28:38 +0000 (21:28 -0800)]
sync: use ' ' as the default shard so retries can be stored

The replica log api doesn't allow empty markers, but we may need to
store retries in the replica log even if there are no log entries. Use
' ' as a marker instead, since it is before all possible markers.

Signed-off-by: Josh Durgin <jdurgin@redhat.com>
11 years agofull data sync: treat 404 from bucket list as success
Josh Durgin [Thu, 18 Dec 2014 05:20:21 +0000 (21:20 -0800)]
full data sync: treat 404 from bucket list as success

Since results are paged, the exception may not occur immediately, so
add a new type to distinguish it from other client exceptions, and
catch it at any point during full sync of a bucket.

Signed-off-by: Josh Durgin <jdurgin@redhat.com>
11 years agoworker: ignore 404 when removing op_state
Josh Durgin [Thu, 18 Dec 2014 00:16:14 +0000 (16:16 -0800)]
worker: ignore 404 when removing op_state

It's fine if the gateway deleted it first.

Signed-off-by: Josh Durgin <jdurgin@redhat.com>
11 years agore-purpose urlencode to craft the query string for us without quoting again
Alfredo Deza [Thu, 18 Dec 2014 21:18:42 +0000 (15:18 -0600)]
re-purpose urlencode to craft the query string for us without quoting again

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
11 years agocreate a test to check/prevent double quoting of urls
Alfredo Deza [Thu, 18 Dec 2014 21:16:58 +0000 (15:16 -0600)]
create a test to check/prevent double quoting of urls

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
11 years agoadd get_bucket_list test
Alfredo Deza [Thu, 18 Dec 2014 19:24:12 +0000 (14:24 -0500)]
add get_bucket_list test

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
11 years agotests for get API calls
Alfredo Deza [Thu, 18 Dec 2014 19:07:37 +0000 (14:07 -0500)]
tests for get API calls

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
11 years agopep8/style changes to client.py
Alfredo Deza [Thu, 18 Dec 2014 18:58:03 +0000 (13:58 -0500)]
pep8/style changes to client.py

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>
11 years agohttp response does not have .raw anymore
Alfredo Deza [Thu, 18 Dec 2014 00:40:05 +0000 (19:40 -0500)]
http response does not have .raw anymore

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>