Merge lp:~smoser/euca2ools/fix-import-keypair into lp:euca2ools

Proposed by Scott Moser
Status: Merged
Merged at revision: 529
Proposed branch: lp:~smoser/euca2ools/fix-import-keypair
Merge into: lp:euca2ools
Diff against target: 18 lines (+4/-1)
1 file modified
euca2ools/commands/euca/importkeypair.py (+4/-1)
To merge this branch: bzr merge lp:~smoser/euca2ools/fix-import-keypair
Reviewer Review Type Date Requested Status
Bastian Gebhardt (community) Approve
Eucalyptus Maintainers Pending
Review via email: mp+85874@code.launchpad.net

Description of the change

euca-import-keypair: fix importing keypairs

importkeypair was uploading the filename as the keypair content rather than
the file contents. Simple fix is to upload the file contents.

To post a comment you must log in.
Revision history for this message
Bastian Gebhardt (buzz-t) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'euca2ools/commands/euca/importkeypair.py'
--- euca2ools/commands/euca/importkeypair.py 2011-08-12 00:22:21 +0000
+++ euca2ools/commands/euca/importkeypair.py 2011-12-15 14:58:43 +0000
@@ -47,10 +47,13 @@
47 optional=False)]47 optional=False)]
48 48
49 def main(self):49 def main(self):
50 fp = open(self.file_name)
51 key_material = fp.read()
52 fp.close()
50 conn = self.make_connection_cli()53 conn = self.make_connection_cli()
51 return self.make_request_cli(conn, 'import_key_pair',54 return self.make_request_cli(conn, 'import_key_pair',
52 key_name=self.key_name,55 key_name=self.key_name,
53 public_key_material=self.file_name)56 public_key_material=key_material)
5457
55 def main_cli(self):58 def main_cli(self):
56 keypair = self.main()59 keypair = self.main()

Subscribers

People subscribed via source and target branches