Follow-up from "Add date field to CouchDB document from EnabledSlots"
The following discussion from !256 (merged) should be addressed:
-
@clemenci started a discussion: We can avoid that the checkout overrides the date defined by the main job with something as simple as
diff --git a/python/LbNightlyTools/Scripts/Common.py b/python/LbNightlyTools/Scripts/Common.py index 842bc66..cf27000 100755 --- a/python/LbNightlyTools/Scripts/Common.py +++ b/python/LbNightlyTools/Scripts/Common.py @@ -324,6 +324,9 @@ class DashboardUpdate(object): d = {'_rev': d['_rev'], '_id': d['_id']} else: d = {} + if 'date' in d: + # do not override existing date + del data['date'] return recursive_update(d, data) self.dashboard.update(self.doc_name, changes)
but I'm not convinced it's what we want.
The alternative is that checkout does not set the date, unless forced to (with a
--date
option).