On Wed, Sep 10, 2008 at 07:01:30AM +0800, Coe, Colin C. (Unix Engineer) wrote:
>  <<0009-add-a-Last-Changed-column-to-the-Kickstart-Profile.patch>> 
> Hi all
>  
> Attached is a simple patch to add a 'Last Changed' column in the
> Kickstart Profile page.  I added this after being away from work for a
> couple of weeks and being told some changes where made to some Kickstart
> profiles but no-one could tell me which ones had been changed.

Colin,

I can see that the patch was not committed yet.

The problem I can see is that it only takes into account changes in
rhnKSData, yet the "kickstart" thing has data changed in multiple
tables. So if your peers tell you they changed something while you
were gone, if they did change in some of the child tables, you won't
see the timestamp changed.

I wonder if appart from the modified column we'd want to have
something like modified_general, and have triggers in all related
tables to also update this value ...

-- 
Jan Pazdziora
Satellite Engineering, Red Hat

_______________________________________________
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel

Reply via email to