Code review comment for lp:~oubiwann/txaws/484858-s3-scripts

Revision history for this message
Duncan McGreggor (oubiwann) wrote :

This branch depends upon two other branches:
 * 474353-storage-ec2-symmetry
 * 475571-s3-error-wrapper

These scripts serve dual purpose:
 1. convenience for users wanting to perform actions against S3-compliant storage servers, and
 2. examples for how to use the API.

For the latter purpose, code in the scripts has not been abstracted out for maximal reuse. Instead, each script presents a complete picture to an API user on how to use the given method to accomplish a common task. This is done at the cost of code redundancy.

Note that the DELETE operations currently return an exit code of 1 when it should be 0. This will be addressed by bug #486363 (Successful delete operations return a 204 No Content "error").

« Back to merge proposal