Treat HTTP 500 as any other HTTP error for a copy request

We still want to retry later, since 500 could be the gateway timing
out, or it being upgraded to fix a bug, etc.

Signed-off-by: Josh Durgin <jdurgin@redhat.com>

client: escape copy source header

Need to esacpe the x-amz-copy-source header.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>

worker: process full sync for buckets with no log

This way we catch objects that were uploaded before zones or data logs
were set up. If there are actually no objects in the bucket, the
bucket listing will tell us that.

Signed-off-by: Josh Durgin <jdurgin@redhat.com>

client: don't hide boto return values

The decorator for translating boto exceptions needs to return
explicitly, or it will lose the return value of the original function,
and cause full sync to fail for objects that aren't in the bucket
index log.

Signed-off-by: Josh Durgin <jdurgin@redhat.com>

add a test that ensures that wait_for_object is not called if object is not found

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

by default object is not found, changing only when sync_object_intra_region() works

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

raise SyncFailed on transient errors

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

allow a stringified version of the http status code

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

tests for sync_object in worker.py

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

configure logging for testing

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

log the exception as error

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

Merge pull request #10 from ceph/wip-coverage

add parametrization for endpoint tests

Reviewed-by: Josh Durgin <jdurgin@redhat.com>

add parametrization for endpoint tests

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

client: fix unicode handling

Use unicode format strings, and don't url-encode the object copy path
before passing it to requests, which will url-encode it for us.

Partially-fixes: #8202
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>

logrotate: don't rotate empty logs

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>

debian: fix init script installation

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>

packaging: fix up spec file for non-python files

Correct the installation of the binary, logrotate conf, and init script.
Stop using INSTALLED_FILES. Instead explicitly list files and glob the
python package, as recommended by Fedora's python packaging guide.

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>

cli: add a default log file

Now that the packages include /var/log/ceph/radosgw-agent, default to
storing logs there, named after the configuration file or
radosgw-agent.log if no config file was used.

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>

debian: depend on python 2.6 or higher

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>

packaging: add logrotate config

There's no need to signal running agents since they use
a WatchedFileHandler that detects rotation itself.

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>

packaging: add init scripts

create log, run, and etc dirs as well

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>

packaging: add dependency on python-requests

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>