OpenStack Compute (nova)

Merge lp:~vishvananda/nova/remove-print into lp:~soren/nova/trunk

  1. remove-print
  2. Merge into trunk
Proposed by Vish Ishaya
Status: Merged
Merged at revision: 141
Proposed branch: lp:~vishvananda/nova/remove-print
Merge into: lp:~soren/nova/trunk
Diff against target: 17 lines (+0/-2)
1 file modified
nova/tests/storage_unittest.py (+0/-2)
To merge this branch: bzr merge lp:~vishvananda/nova/remove-print
Reviewer Review Type Date Requested Status
Eric Day (community) Approve
Jay Pipes (community) Approve
Soren Hansen Pending
Review via email: mp+30057@code.launchpad.net

Description of the change

very simple fix: remove some extraneous print statements from tests

To post a comment you must log in.
Revision history for this message
Jay Pipes (jaypipes) wrote :

Yes, please can we merge this in!

--annoying print statements in tests ;)

review: Approve
Revision history for this message
Eric Day (eday) wrote :

Looks good!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'nova/tests/storage_unittest.py'
2--- nova/tests/storage_unittest.py 2010-07-15 15:52:11 +0000
3+++ nova/tests/storage_unittest.py 2010-07-15 23:33:45 +0000
4@@ -69,13 +69,11 @@
5 vols = []
6 for i in xrange(total_slots):
7 vid = self.mystorage.create_volume(vol_size, user_id, project_id)
8- print vid
9 vols.append(vid)
10 self.assertRaises(storage.NoMoreVolumes,
11 self.mystorage.create_volume,
12 vol_size, user_id, project_id)
13 for id in vols:
14- print id
15 self.mystorage.delete_volume(id)
16
17 def test_run_attach_detach_volume(self):

Subscribers

People subscribed via source and target branches