Make keypairs optional in the Openstack API

Bug #732204 reported by Dan Prince
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
justinsb

Bug Description

Creating a server within the Openstack API requires a keypair. As keypairs aren't part of the Cloud Servers API specs (v1.0/v1.1) we need to make this requirement optional.

NOTE: Additionally there is currently no Openstack API controller to create keypairs. (branches exist to add this functionality however)

See: https://code.launchpad.net/~justin-fathomdb/nova/justinsb-openstack-api-keys

Related branches

Revision history for this message
justinsb (justin-fathomdb) wrote :

The keys code actually got mixed into my volumes branch (sorry!), so the latest keys code is here:
https://code.launchpad.net/~justin-fathomdb/nova/justinsb-openstack-api-volumes/+merge/50868

I wish I'd known that key_pairs weren't a requirement - there's a bit of a storm raging on that merge request...

I had presumed that the plan was to generate a default keypair for each user, so v1.0 users wouldn't need to create a key but would still have a key_pair in the system. Ideally, they'd upload their own public key, but we know that not every user will do that.

Thierry Carrez (ttx)
Changed in nova:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
justinsb (justin-fathomdb) wrote :

This is actually a super-high-priority bug IMHO; without it you can't create a server without using the EC2 API.

I'm grabbing it...

Changed in nova:
assignee: nobody → justinsb (justin-fathomdb)
Revision history for this message
Dan Prince (dan-prince) wrote :

Hi Justin,

I'd like to keep this one. I'm working with Mark W. who is doing the personalities (I sit right next to him). Once that is in (hopefully early this week) we can resolve this issue as well fairly quickly.

I've been holding off on doing this until he gets his branch in to avoid any further conflicts in that same file.

Changed in nova:
assignee: justinsb (justin-fathomdb) → Dan Prince (dan-prince)
Revision history for this message
justinsb (justin-fathomdb) wrote :

Errr... sorry Dan; the fix is ready. It was a 15 minute fix (mostly spent waiting for bazaar and launchpad) This was blocking me from doing testing, so it became 'next on my list' for me.

Going to propose for merge in a few minutes:
https://code.launchpad.net/~justin-fathomdb/nova/bug732204/+merge/53359

I guess in future if you're working on something then grab it....

I'm going to grab it back so that this doesn't confuse people.

Changed in nova:
assignee: Dan Prince (dan-prince) → justinsb (justin-fathomdb)
Thierry Carrez (ttx)
Changed in nova:
status: Confirmed → In Progress
Thierry Carrez (ttx)
Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → 2011.2
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.