Hi,
I'm attaching a patch that implements #1605, "As a user, I want to launch a
deployment." This happens within the context of a pool, as opposed to a dedicated
Deployments tab. (View a pool and press "New Deployment".)
There are two things you should be aware of:
1.) It doesn't actually work. We need to parse the deployable definition (as XML
loaded from the provided URL) and then find or build matching images based on the enclosed
data. But since that isn't implemented just yet, we just store the contents of the URL
in the database and move you to the next page, from which you could launch an image, if
only we had any of the needed data.
2.) It breaks existing deployments, which are going away at some point. I get a slew of
failing tests as a result. (I'm happy to remove the no-longer-applicable tests, but I
first wanted someone to confirm that I'm not going crazy and removing functionality
that we actually want to keep.)
So why send it? Because it implements a feature that we're going to need (and soon!),
and because I'd rather have eyes on it now than wait until next week when it might
have something to hook into to be fully-functional.
-- Matt
Show replies by thread