analysis.lami: Correctly reject lttng-analyses > 0.4

Trace Compass does not currently support the LAMI 1.0 protocol,
which means lttng-analyses 0.5 and up should not be available
and should be striked out.

Correctly update the max version number to make it so.

Change-Id: I65a33db732dba7725284840b2bcd0b2ecaab9d3a
Signed-off-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-on: https://git.eclipse.org/r/74839
Reviewed-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Reviewed-by: Hudson CI
Reviewed-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Tested-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

diff --git a/analysis/org.eclipse.tracecompass.analysis.lami.core.tests/testfiles/test-metadata.json b/analysis/org.eclipse.tracecompass.analysis.lami.core.tests/testfiles/test-metadata.json
index 412fb40a9b145cdc13be2bc43ed2fcbb288fa465..e1fcf7f1c13dd85157b4451fe31fd59a5e6889af 100644 (file)
--- a/analysis/org.eclipse.tracecompass.analysis.lami.core.tests/testfiles/test-metadata.json
+++ b/analysis/org.eclipse.tracecompass.analysis.lami.core.tests/testfiles/test-metadata.json
@@ -110,8 +110,8 @@
         "test"
     ],
     "mi-version": {
-        "minor": 0,
-        "major": 1
+        "minor": 1,
+        "major": 0
     },
     "url": "http://perdu.com"
 }

diff --git a/analysis/org.eclipse.tracecompass.analysis.lami.core/src/org/eclipse/tracecompass/internal/provisional/analysis/lami/core/module/LamiAnalysis.java b/analysis/org.eclipse.tracecompass.analysis.lami.core/src/org/eclipse/tracecompass/internal/provisional/analysis/lami/core/module/LamiAnalysis.java
index 200c4e15fc1981d86f633660beb112aca938e7af..53e6e95d33febe6dea313bebb37cfd622fe1f95d 100644 (file)
--- a/analysis/org.eclipse.tracecompass.analysis.lami.core/src/org/eclipse/tracecompass/internal/provisional/analysis/lami/core/module/LamiAnalysis.java
+++ b/analysis/org.eclipse.tracecompass.analysis.lami.core/src/org/eclipse/tracecompass/internal/provisional/analysis/lami/core/module/LamiAnalysis.java
@@ -78,8 +78,12 @@ import com.google.common.collect.Multimap;
  */
 public class LamiAnalysis implements IOnDemandAnalysis {
 
-    /** Maximum major version of the LAMI protocol we support */
-    private static final int MAX_SUPPORTED_MAJOR_VERSION = 1;
+    /**
+     * Maximum major version of the LAMI protocol we support.
+     *
+     * Currently only 0.x/unversioned MI, outputted by lttng-analyses 0.4.x
+     */
+    private static final int MAX_SUPPORTED_MAJOR_VERSION = 0;
 
     private static final String DOUBLE_QUOTES = "\""; //$NON-NLS-1$