Opened 14 years ago
Closed 14 years ago
Last modified 14 years ago
#10040 closed Bug Report - General (fixed)
menu added commflag jobs has NULL hostname
| Reported by: | Owned by: | Raymond Wagner | |
|---|---|---|---|
| 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 (2)
Change History (8)
comment:1 Changed 14 years ago by
| Owner: | changed from cpinkham to Raymond Wagner |
|---|---|
| Status: | new → accepted |
comment:2 Changed 14 years ago by
| Milestone: | unknown → 0.25 |
|---|---|
| Resolution: | → fixed |
| Status: | accepted → closed |
comment:3 Changed 14 years ago by
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;'?
Changed 14 years ago by
| Attachment: | jobqueue.txt added |
|---|
comment:5 Changed 14 years ago by
Hi, Unfortunately above commit is not helping. I'm on v0.25pre-3329-gbececaa-dirty-20110915. Example of content for jobqueue attached.
Changed 14 years ago by
| Attachment: | jobqueue-list.txt added |
|---|
comment:6 Changed 14 years ago by
Ahh - silly me. I forgot to upgrade frontend with this commit. It works OK. Sorry for confusion !
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