Merge lp:~cprov/adt-cloud-service/container-name-prefix into lp:adt-cloud-service

Proposed by Celso Providelo
Status: Merged
Approved by: Celso Providelo
Approved revision: 18
Merged at revision: 19
Proposed branch: lp:~cprov/adt-cloud-service/container-name-prefix
Merge into: lp:adt-cloud-service
Diff against target: 31 lines (+3/-2)
2 files modified
adt_cloud_service/tests/test_functional.py (+1/-1)
adt_cloud_service/v1.py (+2/-1)
To merge this branch: bzr merge lp:~cprov/adt-cloud-service/container-name-prefix
Reviewer Review Type Date Requested Status
Thomi Richards (community) Approve
Review via email: mp+253434@code.launchpad.net

Commit message

Fixing result_url returned to clients upon test requests (container name is prefixed with 'adt-')

Description of the change

Fixing result_url returned to clients upon test requests (container name is prefixed with 'adt-')

To post a comment you must log in.
Revision history for this message
Thomi Richards (thomir-deactivatedaccount) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'adt_cloud_service/tests/test_functional.py'
2--- adt_cloud_service/tests/test_functional.py 2015-03-17 14:52:51 +0000
3+++ adt_cloud_service/tests/test_functional.py 2015-03-18 20:25:11 +0000
4@@ -136,7 +136,7 @@
5 data = json.loads(resp.data.decode())
6 self.assertThat(
7 data['result_url'],
8- Equals('http://swift.test.com/{}/results.tgz'.format(
9+ Equals('http://swift.test.com/adt-{}/results.tgz'.format(
10 q.generated_uuids[0]
11 ))
12 )
13
14=== modified file 'adt_cloud_service/v1.py'
15--- adt_cloud_service/v1.py 2015-03-16 03:32:00 +0000
16+++ adt_cloud_service/v1.py 2015-03-18 20:25:11 +0000
17@@ -50,12 +50,13 @@
18 rabbit_request['nova_image'] = image_name
19
20 req_id = current_app.extensions['rabbit'].queue_test(rabbit_request)
21+ results_path = 'adt-{}/results.tgz'.format(req_id)
22 return jsonify(
23 {
24 'status': 'request processed',
25 'request_id': req_id,
26 'result_url': urllib.parse.urljoin(
27- current_app.config['SWIFT_URL'], req_id + '/results.tgz'
28+ current_app.config['SWIFT_URL'], results_path
29 )
30 }
31 )

Subscribers

People subscribed via source and target branches