Hello,
I found an error navigating to:
https://<hostname>/rhn/systems/VirtualSystemsList.do
In my setup there is only one registered system that does not have a UUID and
also
the virtual host system is unknown to the Spacewalk server in this case:
- current.getUuid() == null
- current.getHostSystemId() == null
So I think we should not actually do
"current.setSystemId(current.getHostSystemId());"
here. The attached patch fixes the problem for me (in [1]) and also makes sense
IMO.
Thank you and regards,
Johannes
[1] VirtualSystemsListSetupAction.java
--
SUSE LINUX Products GmbH, HRB 16746 (AG Nürnberg)
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer
>From 1e0f80e4b6f43ade890dd3dae2a71bb4544dff01 Mon Sep 17 00:00:00 2001
From: Johannes Renner <jren...@suse.de>
Date: Fri, 19 Jul 2013 13:39:51 +0200
Subject: [PATCH] Fix ISE when UUID and host ID is null
---
.../rhn/frontend/action/systems/VirtualSystemsListSetupAction.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/java/code/src/com/redhat/rhn/frontend/action/systems/VirtualSystemsListSetupAction.java b/java/code/src/com/redhat/rhn/frontend/action/systems/VirtualSystemsListSetupAction.java
index b6252ad..5e8a0fe 100644
--- a/java/code/src/com/redhat/rhn/frontend/action/systems/VirtualSystemsListSetupAction.java
+++ b/java/code/src/com/redhat/rhn/frontend/action/systems/VirtualSystemsListSetupAction.java
@@ -63,7 +63,7 @@ public class VirtualSystemsListSetupAction extends BaseSystemListSetupAction {
if (current.isFakeNode()) {
continue;
}
- else if (current.getUuid() == null) {
+ else if (current.getUuid() == null && current.getHostSystemId() != null) {
current.setSystemId(current.getHostSystemId());
}
else {
--
1.8.1.4
_______________________________________________
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel
