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