Bug 1426764
Summary: | beakerd repeatedly runs update_dirty_jobs unnecessarily | ||
---|---|---|---|
Product: | [Retired] Beaker | Reporter: | Dan Callaghan <dcallagh> |
Component: | general | Assignee: | Dan Callaghan <dcallagh> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | tools-bugs <tools-bugs> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | 24 | CC: | dcallagh, mjia, rjoost |
Target Milestone: | 24.2 | Keywords: | Patch |
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2017-03-30 03:23:04 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Dan Callaghan
2017-02-24 19:21:05 UTC
I think we can just adjust the conditionals so that each scheduling step is followed by update_dirty_jobs() only if the scheduling step did some work. We'll have to do QE on the items before we can tag a release. I've kicked off 3 system scans, waited a while and cancelled one job while I kept the others running. I'm pasting parts of the log here in which I think the changes have taken affect: Mar 20 07:38:13 beaker-server-lc beakerd[32477]: bkr.server.tools.beakerd DEBUG Provisioning recipe 5 in RS:5 Mar 20 07:38:14 beaker-server-lc beakerd[32477]: bkr.server.model.activity DEBUG Tentative SystemActivity: object_id=3L, service=u'Scheduler', field=u'Distro Tree', action=u'Provision', old=u'', new=u'CentOS-6.8 x86_64', user=admin Mar 20 07:38:14 beaker-server-lc beakerd[32477]: bkr.server.tools.beakerd DEBUG Provisioning recipe 6 in RS:6 Mar 20 07:38:14 beaker-server-lc beakerd[32477]: bkr.server.model.activity DEBUG Tentative SystemActivity: object_id=2L, service=u'Scheduler', field=u'Distro Tree', action=u'Provision', old=u'', new=u'CentOS-6.8 x86_64', user=admin Mar 20 07:38:14 beaker-server-lc beakerd[32477]: bkr.server.tools.beakerd DEBUG Provisioning recipe 7 in RS:7 Mar 20 07:38:14 beaker-server-lc beakerd[32477]: bkr.server.model.activity DEBUG Tentative SystemActivity: object_id=1L, service=u'Scheduler', field=u'Distro Tree', action=u'Provision', old=u'', new=u'CentOS-6.8 x86_64', user=admin Mar 20 07:38:14 beaker-server-lc beakerd[32477]: bkr.server.tools.beakerd DEBUG Updating dirty job 5 Mar 20 07:38:14 beaker-server-lc beakerd[32477]: bkr.server.tools.beakerd DEBUG Updating dirty job 7 Mar 20 07:38:14 beaker-server-lc beakerd[32477]: bkr.server.tools.beakerd DEBUG Updating dirty job 6 Mar 20 07:38:31 beaker-server-lc beakerd[32477]: bkr.server.tools.beakerd DEBUG Updating dirty job 6 Mar 20 07:38:31 beaker-server-lc beakerd[32477]: bkr.server.model.scheduler DEBUG Releasing system beaker-test-vm2 for recipe 6 Mar 20 07:38:31 beaker-server-lc beakerd[32477]: bkr.server.model.activity DEBUG Tentative SystemActivity: object_id=2L, service=u'Scheduler', field=u'User', action=u'Returned', old=u'admin', new=u'', user=admin Verified on my own Beaker installation, running 24.2.git.9.d4c983d Beaker 24.2 has been released. |