[cuebot] Fix exception handling in job completion metrics

- Remove try-catch from Prometheus metrics recording to allow programming errors (wrong labels) to fail loudly
- Keep Kafka event publishing exception handling but properly log with stack trace for debugging transient failures

Author: ramonfigueiredo

cuebot/src/main/java/com/imageworks/spcue/service/JobManagerSupport.java

@@ -164,19 +164,15 @@ public boolean shutdownJob(JobInterface job, Source source, boolean isManualKill
 
                 // Record job completion metric
                 if (prometheusMetrics != null && showDao != null) {
-                    try {
-                        JobDetail jobDetail = jobManager.getJobDetail(job.getId());
-                        String showName = showDao.getShowDetail(job.getShowId()).getName();
-                        String state = isManualKill ? "KILLED" : "FINISHED";
-                        prometheusMetrics.recordJobCompleted(state, showName, jobDetail.shot);
-
-                        // Record job core seconds histogram
-                        ExecutionSummary execSummary = jobManager.getExecutionSummary(job);
-                        prometheusMetrics.recordJobCoreSeconds(execSummary.coreTime, showName,
-                                jobDetail.shot);
-                    } catch (Exception e) {
-                        logger.warn("Failed to record job completion metric: " + e.getMessage());
-                    }
+                    JobDetail jobDetail = jobManager.getJobDetail(job.getId());
+                    String showName = showDao.getShowDetail(job.getShowId()).getName();
+                    String state = isManualKill ? "KILLED" : "FINISHED";
+                    prometheusMetrics.recordJobCompleted(state, showName, jobDetail.shot);
+
+                    // Record job core seconds histogram
+                    ExecutionSummary execSummary = jobManager.getExecutionSummary(job);
+                    prometheusMetrics.recordJobCoreSeconds(execSummary.coreTime, showName,
+                            jobDetail.shot);
                 }
 
                 // Publish job completed/killed event to Kafka
@@ -190,7 +186,7 @@ public boolean shutdownJob(JobInterface job, Source source, boolean isManualKill
                                 jobDetail, previousState, null, null);
                         kafkaEventPublisher.publishJobEvent(jobEvent);
                     } catch (Exception e) {
-                        logger.warn("Failed to publish job event: " + e.getMessage());
+                        logger.warn("Failed to publish job event", e);
                     }
                 }