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