Ticket #10040 (closed Bug Report - General: fixed)
Opened 21 months ago
Last modified 21 months ago
menu added commflag jobs has NULL hostname
| Reported by: | warpme@… | Owned by: | wagnerrp |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.25 |
| Component: | MythTV - Mythcommflag | Version: | Master Head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
Hi, It think it is bug. I'm on v0.25pre-3322-gec38d98-dirty-20110913. Adding commflag job via "recordings/menu/job options/Begin Commercial Detection" generates entry in jobqueue table which has NULL hostname. As result this job can't run. Changing manually hostname to correct value enables this job. jobqueue entries generated in others ways (i.e. when recordings starts) are OK.
Attachments
Change History
comment:1 Changed 21 months ago by wagnerrp
- Owner changed from cpinkham to wagnerrp
- Status changed from new to accepted
comment:2 Changed 21 months ago by Github
- Status changed from accepted to closed
- Resolution set to fixed
- Milestone changed from unknown to 0.25
Force correct hostname input when inserting new jobs to the queue
Check the value of the hostname field prior to insertion into the jobqueue table. This prevents failed insertion, or otherwise failed run of the job in question.
Fixes #10040
Branch: master Changeset: bececaaae57e3e8112a73ee0b45ecdadcc54c0ef
comment:3 Changed 21 months ago by wagnerrp
As an aside, the error you are describing should not even be possible. The hostname field is marked as NOT NULL, meaning it should outright refuse insertion of such an entry. Could you post the mysql output of 'DESC jobqueue;'?
comment:4 Changed 21 months ago by warpme@…
Hi, 'DESC jobqueue;' attached. Thx for quick commit !
comment:5 Changed 21 months ago by warpme@…
Hi, Unfortunately above commit is not helping. I'm on v0.25pre-3329-gbececaa-dirty-20110915. Example of content for jobqueue attached.
comment:6 Changed 21 months ago by warpme@…
Ahh - silly me. I forgot to upgrade frontend with this commit. It works OK. Sorry for confusion !
