BZ 761279/766137 - Handling of imported images (rev2)

Matt Wagner matt.wagner at redhat.com
Thu Dec 15 18:42:45 UTC 2011


Hi all,

Sending this out for hopefully the last time. This is a resent of my
previous send[1], but with a few changes:

* In the course of rebasing on master, I had to deal with the fact that
 app/models/image.rb was removed. Rather than re-adding it, it was
 decided that it was best to move the image-import logic to
 aeolus-image-rubygem and then create a small bit of code to call it
 from Conductor. So the aeolus-image-rubygem change is
 semi-significant.

* The issue that tsedovic found in images#show is resolved. (It ended up
 requiring a ridiculous amount of code.)

* I'm omitting the aeolus-cli changes from the patch, because nothing
 has changed from last time and the series was essentially ACKed.
 They're still necessary.

Note that the initial patch was ACKed, but in the course of fixing a
small issue and dealing with a large merge conflict (see first bullet
point), I ended up changing things significantly enough that I think
this deserves a resend.

Best,
Matt

[1]
http://lists.fedorahosted.org/pipermail/aeolus-devel/2011-December/007384.html



More information about the aeolus-devel mailing list