lp:~vila/uci-engine/delete-containers

Branch merges

Propose for merging

1 branch dependent on this one.

Ready for review for merging into lp:uci-engine

PS Jenkins bot (community): Approve (continuous-integration) on 2014-11-20

Para Siva (community): Approve on 2014-11-20

Diff: 217 lines (+135/-9)

4 files modified

ci-utils/ci_utils/data_store.py (+44/-2)
ci-utils/ci_utils/testing/fixtures.py (+6/-1)
ci-utils/ci_utils/tests/test_data_store.py (+81/-2)
ci-utils/ci_utils/tests/test_fixtures.py (+4/-4)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

912. By Vincent Ladeuil on 2014-11-20

Fix pep8 issue.

911. By Vincent Ladeuil on 2014-11-20

Fix pyflakes issue.

910. By Vincent Ladeuil on 2014-11-20

One more test for the retries=0 case to address review comment.

909. By Vincent Ladeuil on 2014-11-20

One more test to ensure we fail after all retries (with a test reproducing the bug).

Tested with: ./run-tests ^tests.test_style ^tests.test_data_store ^ci-utils.*test_fixtures ^ci-utils.*test_data_store

908. By Vincent Ladeuil on 2014-11-20

Extend swiftclient retry policy to container deletion with tests.

907. By Vincent Ladeuil on 2014-11-19

Meh, we want to test the real data store with a fake swift client, not the fake data store ;)

906. By Vincent Ladeuil on 2014-11-19

One less FIXME and a test plan.

905. By Vincent Ladeuil on 2014-11-19

Tested with ./run-tests ^tests.test_style ^tests.test_data_store befor submission.

(and some cosmetic changes)

904. By Vincent Ladeuil on 2014-11-19

This test pollutes the run output.

903. By Vincent Ladeuil on 2014-11-19

Log auth info (except for the password).

Rewrite exception in list_files.