Bug 246480

Summary: sync last_modified dates correctly for ks trees instead of defaulting to current date
Product: Red Hat Satellite 5 Reporter: Pradeep Kilambi <pkilambi>
Component: Satellite SynchronizationAssignee: Jan Pazdziora <jpazdziora>
Status: CLOSED ERRATA QA Contact: Šimon Lukašík <slukasik>
Severity: high Docs Contact:
Priority: high    
Version: unspecifiedCC: cperry, jpazdziora, kbaxley, mzazrivec, tao, vgaikwad
Target Milestone: ---Keywords: Reopened, Triaged
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: sat510 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-03-22 12:04:31 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 328271    
Bug Blocks: 531600    

Description Pradeep Kilambi 2007-07-02 16:20:25 UTC
Description of problem:
When satellite-sync pulls down kickstart trees from hosted, It overrides the
last_modified date to the date it synced the files. This has been the behavior
since long. This becomes a problem with channel dump generation (for incremental
dumps) as the date ranges no longer get validated for kickstart trees and pulls
everything.

Comment 1 Pradeep Kilambi 2007-09-25 16:03:55 UTC
not a very high priority/blocker IMO, this bug fits well with the Enhancements
we're planning for rhn-sat-exporter tool for 520.

Comment 2 Pradeep Kilambi 2007-10-11 19:59:57 UTC
Investigating further, looks like we don't export last_modified from hosted. So
for us to include the right KickstartTree last_modified date field into sat's
rhnKickstartTree table, we need this date to be first exported by hosted
satellite exporter. 

I opened a bug(bz#328271) for hosted to add this to the exporter. Once thats
done, we can grab the last_modified from the xml stream and populate db with
correct values.

Comment 3 Clifford Perry 2007-12-12 02:41:20 UTC
https://bugzilla.redhat.com/show_bug.cgi?id=328271#c1
Comment #1 From James Bowes (jbowes) 	on 2007-10-17 19:55 EST 
[reply] 	Private

Data is sent, but as last-modified rather than last_modified

Comment 4 Pradeep Kilambi 2007-12-19 22:35:55 UTC
Looks like even though the data is sent as a different key, looks like the
satsync was never written to handle last_modified dates for kickstart-trees and
hence there seems to be no validation for the last-modified dates verses the the
timestamps. So we pretty much need to rewrite rhnKickstartTreeCollection and
Container classes. I would say we could scope this for next release if we're
short on time. 

Comment 5 Pradeep Kilambi 2007-12-20 15:35:44 UTC
After discussing with cliff, moving to sat-5.2 ..

Comment 10 Clifford Perry 2010-01-26 15:02:52 UTC
Moving to sat-channel-dumps tracker. Agree we should look to address this as part of the feature code changes needed to redo channel dump creation process. 

Cliff

Comment 14 Jan Pazdziora 2010-02-19 17:54:54 UTC
Fixed in Spacewalk master 460aad3e019c6fd064732aed14127d822d62217b.

Comment 15 Milan Zázrivec 2010-03-04 16:54:00 UTC
satellite.git: 9ee57b70a63c7fbdb3e46e8b4352cfbbd2681fb0

Comment 16 Milan Zázrivec 2010-03-04 16:58:47 UTC
spacewalk-backend-0.5.28-40.1

Comment 18 Šimon Lukašík 2010-03-15 08:09:43 UTC
Taking QA contact for the bug verification.

Comment 20 Milan Zázrivec 2010-03-16 14:30:34 UTC
spacewalk-backend-0.5.28-48

Comment 21 Šimon Lukašík 2010-03-17 14:37:51 UTC
Verified against: (on rhel-i386-server-5-u4) 
spacewalk-backend-0.5.28-49.el5sat.noarch

Comment 23 errata-xmlrpc 2010-03-22 12:04:31 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2010-0160.html

Comment 24 Milan Zázrivec 2010-03-23 15:13:42 UTC
Although the errata fixing this particular issue is out, I'd like
to point out two important things:

1) The updated packages won't update last_modified values that already
exist in the database. This means, if the previous satellite-sync
set these values incorrectly, new sync won't update these values. It
will only be able to insert new correct values for newly synchronized
kickstartable trees.

2) Due to bug #576222, even correctly set last_modified values will be
overwritten when taskomatic executes CobblerSyncTask, which modifies
existing rows in rhnKickstartableTree for use with cobbler (disabling
this task may help, as long as cobbler functionality is not required).