Bug 516914 - Cloned Channels & Missing Checksum
Summary: Cloned Channels & Missing Checksum
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Spacewalk
Classification: Community
Component: WebUI
Version: 0.6
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Tomas Lestach
QA Contact: Red Hat Satellite QA List
URL:
Whiteboard:
Depends On:
Blocks: space13
TreeView+ depends on / blocked
 
Reported: 2009-08-11 21:10 UTC by Andy Speagle
Modified: 2011-02-08 08:41 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-02-08 08:41:11 UTC
Embargoed:


Attachments (Terms of Use)

Description Andy Speagle 2009-08-11 21:10:08 UTC
Description of problem:
Cloning a custom software channel results in not being able to manage the newly cloned channel via the WebUI.  The channel can be displayed, but when trying to manage the channel, the WebUI returns an "internal server error."  This manifests in not getting a confirmation page when cloning the channel as well as not being able to manage the channel in the "manage software channels" section of the WebUI.

Version-Release number of selected component (if applicable):
Spacewalk 0.6

How reproducible:
Infinitely

Steps to Reproduce:
1. Clone any custom software channel.
2. Upon completion of the clone process, the WebUI returns internal server error.
3. Additionally, attempting to manage the cloned channel via the WebUI results in an internal server error.
  
Actual results:
Internal server error returned from WebUI.

Expected results:
Manage channel page returned from WebUI.

Additional info:
This was discussed in the spacewalk-devel list with the following determination by Prad.

-----
So lookong through the code, I think I know what I problem is. The perl 
code when settup up the channel cloning process does'nt not account for 
checksum value. So what ends up happening is, after the perl pxt setup 
page it navigates to edit.do which makes assumptions based on channel 
creation and edits and ends up trying to set the value thats not in the 
form to being with. A workaround fix to get the cloning working for now 
is to do something like,

--- 
a/java/code/src/com/redhat/rhn/frontend/action/channel/manage/EditChannelAct
+++ 
b/java/code/src/com/redhat/rhn/frontend/action/channel/manage/EditChannelAct
@@ -568,7 +568,9 @@ public class EditChannelAction extends RhnAction 
implements
             form.set("maintainer_phone", c.getMaintainerPhone());
             form.set("maintainer_email", c.getMaintainerEmail());
             form.set("support_policy", c.getSupportPolicy());
-            form.set("checksum", c.getChecksum().getLabel());
+            if (!c.isCloned()) {
+                form.set("checksum", c.getChecksum().getLabel());
+            }
             if (c.isGloballySubscribable(ctx.getLoggedInUser().getOrg())) {
                 form.set("per_user_subscriptions", "all");
             }

eventually once the cloning logic in perl supports the checksum stuff, 
we can remove this additional check.

I'll looking into adding this ability for the cloned channels. Please 
open a bug on this and I'll knock it out asap.

Thanks,
~ Prad
-----

Comment 1 Jan Pazdziora 2010-11-19 16:05:00 UTC
Mass-moving to space13.

Comment 2 Tomas Lestach 2011-01-28 13:48:03 UTC
This has been resolved long time ago. Setting to MODIFIED.

Comment 3 Tomas Lestach 2011-02-03 12:21:24 UTC
Moving ON_QA ...

Comment 4 Tomas Lestach 2011-02-08 08:41:11 UTC
This bug has been fixed in Spacewalk 1.3.


Note You need to log in before you can comment on or make changes to this bug.