On 09/13/2011 10:16 AM, Johannes Renner wrote:
> On 09/12/2011 01:20 PM, Miroslav Suchý wrote:
>> On 09/08/2011 03:02 PM, Johannes Renner wrote:
>>> If it can be reproduced, I will open a bug on bugzilla.redhat.com.
>>
>> I could not reproduce it on my Spacewalk 1.5
> 
> Thanks and sorry, this seems to be a known bug that is already fixed:
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=691849
> 
> Thanks anyways,
> Johannes

Hi there,

I just found out that your fix for the above bug (BZ#691849) is still lacking
something: While scheduling a remote command execution BEFORE a package install
or upgrade is working, you will still experience an internal server error when
scheduling the remote command to be executed AFTER the package installation.
Apply the attached patch to fix it.

Greetings,
Johannes

-- 
SUSE LINUX Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 16746 (AG Nürnberg)
>From 41b6d5956c83551bacca444913674421b75097ea Mon Sep 17 00:00:00 2001
From: Johannes Renner <jren...@suse.de>
Date: Tue, 11 Oct 2011 13:18:34 +0200
Subject: [PATCH] Add missing fix for schedule command AFTER package install (BZ#691849)

---
 .../action/rhnpackage/ScheduleRemoteCommand.java   |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/java/code/src/com/redhat/rhn/frontend/action/rhnpackage/ScheduleRemoteCommand.java b/java/code/src/com/redhat/rhn/frontend/action/rhnpackage/ScheduleRemoteCommand.java
index f0fbd66..4f76af5 100644
--- a/java/code/src/com/redhat/rhn/frontend/action/rhnpackage/ScheduleRemoteCommand.java
+++ b/java/code/src/com/redhat/rhn/frontend/action/rhnpackage/ScheduleRemoteCommand.java
@@ -231,7 +231,7 @@ public class ScheduleRemoteCommand extends RhnAction {
             ScriptActionDetails sad =
                 ActionManager.createScript(username, group, timeout, script);
             ScriptRunAction sra = ActionManager.scheduleScriptRun(user, servers,
-                "", sad, earliest);
+                null, sad, earliest);
             sra.setPrerequisite(pa);
             ActionManager.storeAction(sra);
             showMessages(msgs, sra, server, numPackages, mode);
-- 
1.7.3.4

_______________________________________________
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel

Reply via email to