Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#10040 closed Bug Report - General (fixed)

menu added commflag jobs has NULL hostname

Reported by: warpme@… 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)

jobqueue.txt (1.7 KB) - added by warpme@… 8 years ago.
jobqueue-list.txt (3.0 KB) - added by warpme@… 8 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 8 years ago by Raymond Wagner

Owner: changed from cpinkham to Raymond Wagner
Status: newaccepted

comment:2 Changed 8 years ago by Github

Milestone: unknown0.25
Resolution: fixed
Status: acceptedclosed

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 8 years ago by Raymond Wagner

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 8 years ago by warpme@…

Hi, 'DESC jobqueue;' attached. Thx for quick commit !

Changed 8 years ago by warpme@…

Attachment: jobqueue.txt added

comment:5 Changed 8 years 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.

Changed 8 years ago by warpme@…

Attachment: jobqueue-list.txt added

comment:6 Changed 8 years ago by warpme@…

Ahh - silly me. I forgot to upgrade frontend with this commit. It works OK. Sorry for confusion !

Note: See TracTickets for help on using tickets.