Bug 743416

Summary:	Scheduled tasks w/o a start time get kicked off immediately after a httpd restart.
Product:	[Retired] Pulp	Reporter:	James Slagle <jslagle>
Component:	z_other	Assignee:	Jason Connor <jconnor>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Preethi Thomas <pthomas>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	1.0.0	CC:	mmccune, skarmark
Target Milestone:	---	Keywords:	Triaged
Target Release:	Sprint 29
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-02-24 20:13:40 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description James Slagle 2011-10-04 20:07:51 UTC

Scheduled tasks w/o a start time get kicked off immediately after a httpd restart.  This is because there's no start time, so the task system sees that and runs them immediately.

It would be better if instead of getting run immediately, they would get run next at their "last run time" + interval.

Comment 1 Jason Connor 2011-10-11 06:32:32 UTC

add a fall back to the last_sync if no start_time is provided

Comment 2 Jeff Ortel 2011-10-13 00:49:12 UTC

build: 0.238

Comment 3 Preethi Thomas 2011-10-21 19:04:56 UTC

verified
[root@preethi ~]# rpm -q pulp
pulp-0.0.240-1.fc15.noarch

[root@preethi ~]#  pulp-admin -u admin -p admin repo update --id=f14  --interval=P1DT1H5M
Successfully updated repository [ f14 ]
[root@preethi ~]# service httpd restart
Restarting httpd (via systemctl):                          [  OK  ]


From pulp.log after httpd restrt

2011-10-21 14:46:54,086 30759:140379722426400: pulp.server.db.connection:INFO: connection:46 Attempting Database connection with seeds = preethi.usersys.redhat.com
2011-10-21 14:46:54,088 30759:140379722426400: pulp.server.db.connection:INFO: connection:51 Database connection established with: seeds = preethi.usersys.redhat.com, name = pulp_database
2011-10-21 14:46:54,547 30759:140379722426400: pulp.server.async:INFO: async:376 Task reply handler, started.
2011-10-21 14:46:54,548 30759:140379419342592: gofer.messaging.broker:INFO: broker:100 connecting:
{preethi.usersys.redhat.com:5672}:
transport=TCP
host=preethi.usersys.redhat.com
port=5672
cacert=/etc/pki/qpid/ca/ca.crt
clientcert=/etc/pki/qpid/client/client.pem
2011-10-21 14:46:54,735 30759:140379419342592: gofer.messaging.broker:INFO: broker:103 {preethi.usersys.redhat.com:5672} connected to AMQP
2011-10-21 14:46:55,572 30759:140379722426400: pulp:INFO: scheduled_sync:246 Added scheduled sync for f14 to task queue
2011-10-21 14:46:55,603 30759:140379722426400: pulp.server.content.types.parser:INFO: parser:141 Loading type descriptors []
2011-10-21 14:46:55,604 30759:140379722426400: pulp.server.content.types.parser:INFO: parser:143 Parsing type descriptors
2011-10-21 14:46:55,604 30759:140379722426400: pulp.server.content.types.parser:INFO: parser:146 Validating type descriptor syntactic integrity
2011-10-21 14:46:55,604 30759:140379722426400: pulp.server.content.types.parser:INFO: parser:149 Validating type descriptor semantic integrity
2011-10-21 14:46:55,604 30759:140379722426400: pulp.server.content.types.database:INFO: database:83 Updating the database with types []
2011-10-21 14:46:55,606 30759:140379722426400: pulp.server.content.loader:WARNING: loader:383 Profilers load called, but not implemented

Comment 4 Preethi Thomas 2012-02-24 20:13:40 UTC

Pulp v1.0 is released
Closed Current Release.