Hi,

please have a look at the attached patch. I think this was a copy and paste 
bug, or is there a reason, why a transaction with a different name is rolled 
back than started?

-- 
Regards

        Michael Calmer

--------------------------------------------------------------------------
Michael Calmer
SUSE LINUX Products GmbH, Maxfeldstr. 5, D-90409 Nuernberg
T: +49 (0) 911 74053 0
F: +49 (0) 911 74053575  - e-mail: michael.cal...@suse.com
--------------------------------------------------------------------------
SUSE LINUX Products GmbH, GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer
HRB 16746 (AG Nürnberg)
From 6a1fcba3370deb52c6e12c1cdc08b75c7a1b7ede Mon Sep 17 00:00:00 2001
From: Michael Calmer <m...@suse.de>
Date: Tue, 5 Jun 2012 11:47:33 +0200
Subject: [PATCH] fix wrong transaction name in unsubscribe_channels

---
 backend/server/rhnChannel.py |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/backend/server/rhnChannel.py b/backend/server/rhnChannel.py
index 2c9aad7..a48bbf2 100644
--- a/backend/server/rhnChannel.py
+++ b/backend/server/rhnChannel.py
@@ -1846,7 +1846,7 @@ def unsubscribe_channels(server_id, channels):
     for channel in child_channels + base_channels:
         ret = unsubscribe_sql(server_id, channel["id"], 0)
         if not ret:
-            rhnSQL.rollback("unsub_all_channels")
+            rhnSQL.rollback("unsub_channels")
             raise rhnFault(36, "Could not unsubscribe server %s "\
                            "from channel %s" % (server_id, channel["label"]))
 
-- 
1.7.3.4

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

Reply via email to