Opened 9 years ago
Closed 9 years ago
Last modified 9 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 9 years ago by
| Owner: | changed from cpinkham to Raymond Wagner |
|---|---|
| Status: | new → accepted |
comment:2 Changed 9 years ago by
| Milestone: | unknown → 0.25 |
|---|---|
| Resolution: | → fixed |
| Status: | accepted → closed |
comment:3 Changed 9 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 9 years ago by
| Attachment: | jobqueue.txt added |
|---|
comment:5 Changed 9 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 9 years ago by
| Attachment: | jobqueue-list.txt added |
|---|
comment:6 Changed 9 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