Attachment 938565 Details for Bug 1143018 – Patch to add some extra logs to help investigate the issue.

[patch] Patch to add some extra logs to help investigate the issue.

0001-test-PluginDataManager-extra-logs.patch (text/plain), 2.95 KB, created by Vojtech Szocs on 2014-09-17 17:52:19 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Vojtech Szocs

Created: 2014-09-17 17:52:19 UTC

Size: 2.95 KB

patch

obsolete

>From c980594edebd33071f4376a9b60386e6f41636a5 Mon Sep 17 00:00:00 2001
>From: Vojtech Szocs <vszocs@redhat.com>
>Date: Wed, 17 Sep 2014 19:50:25 +0200
>Subject: [PATCH] test: PluginDataManager extra logs
>
>Change-Id: I7681715c126cc67888ff0601e9bf6d6de0e95bad
>Signed-off-by: Vojtech Szocs <vszocs@redhat.com>
>---
> .../ui/frontend/server/gwt/plugin/PluginDataManager.java      | 11 ++++++++++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
>
>diff --git a/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/gwt/plugin/PluginDataManager.java b/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/gwt/plugin/PluginDataManager.java
>index 44a0b75..fd0139f 100644
>--- a/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/gwt/plugin/PluginDataManager.java
>+++ b/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/gwt/plugin/PluginDataManager.java
>@@ -3,6 +3,7 @@ package org.ovirt.engine.ui.frontend.server.gwt.plugin;
> import java.io.File;
> import java.io.FileFilter;
> import java.io.IOException;
>+import java.util.Arrays;
> import java.util.Collection;
> import java.util.Collections;
> import java.util.HashMap;
>@@ -110,8 +111,11 @@ public class PluginDataManager {
>      * point in time (having 'recently-up-to-date', but consistent data, should be enough).
>      */
>     public void reloadData() {
>+        logger.error("TEST entering reloadData()"); //$NON-NLS-1$
>+
>         // Get a snapshot of current data mappings
>         Map<String, PluginData> currentDataMapSnapshot = dataMapRef.get();
>+        logger.error("TEST currentDataMapSnapshot = " + currentDataMapSnapshot); //$NON-NLS-1$
> 
>         // Create a local working copy of current data mappings (avoid modifying 'live' data)
>         Map<String, PluginData> currentDataMapCopy = new HashMap<String, PluginData>(currentDataMapSnapshot);
>@@ -122,6 +126,7 @@ public class PluginDataManager {
>                 return isJsonFile(pathname);
>             }
>         });
>+        logger.error("TEST descriptorFiles = " + Arrays.toString(descriptorFiles)); //$NON-NLS-1$
> 
>         if (descriptorFiles == null) {
>             logger.warn("Cannot list UI plugin descriptor files in [" + pluginDataDir.getAbsolutePath() + "]"); //$NON-NLS-1$ //$NON-NLS-2$
>@@ -243,7 +248,11 @@ public class PluginDataManager {
>     }
> 
>     boolean isJsonFile(File pathname) {
>-        return pathname.isFile() && pathname.canRead() && pathname.getName().endsWith(JSON_FILE_SUFFIX);
>+        boolean isFile = pathname.isFile();
>+        boolean canRead = pathname.canRead();
>+        boolean jsonSuffix = pathname.getName().endsWith(JSON_FILE_SUFFIX);
>+        logger.error("TEST isJsonFile() for " + pathname + ": isFile = " + isFile + ", canRead = " + canRead + ", jsonSuffix = " + jsonSuffix); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ //$NON-NLS-4$
>+        return isFile && canRead && jsonSuffix;
>     }
> 
>     JsonNode readJsonNode(File file) {
>-- 
>1.8.3.1
>

Actions: View | Diff

Attachments on bug 1143018: 938565