Uploaded image for project: 'SonarJava'
  1. SonarJava
  2. SONARJAVA-3600

Remove (re)declaration of fail fast property.

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 6.10
    • Component/s: Plugin
    • Labels:
      None

      Description

      The plugin currently declares sonar.internal.analysis.failFast, but it is useless since we don't need the property to be declared to be able to use it.

      In itself, it is useless and harmless, but it may prevent a deployment on peach if multiple plugins declare this property. The deployment will fail with the error:

       Caused by: java.lang.IllegalStateException: Unable to register extension org.sonar.api.config.PropertyDefinition from plugin 'java'
       at org.sonar.core.platform.ComponentContainer.addExtension(ComponentContainer.java:243)
       at org.sonar.server.plugins.ServerExtensionInstaller.installExtension(ServerExtensionInstaller.java:117)
       at org.sonar.server.plugins.ServerExtensionInstaller.installExtensions(ServerExtensionInstaller.java:72)
       ... 9 common frames omitted
       Caused by: org.picocontainer.PicoCompositionException: Duplicate Keys not allowed. Duplicate for 'org.sonar.api.config.PropertyDefinition-sonar.internal.analysis.failFast'
       at org.picocontainer.DefaultPicoContainer.addAdapterInternal(DefaultPicoContainer.java:438)
       at org.picocontainer.DefaultPicoContainer.addAdapter(DefaultPicoContainer.java:459)
       at org.picocontainer.DefaultPicoContainer.addComponent(DefaultPicoContainer.java:556)
       at org.picocontainer.DefaultPicoContainer.access$300(DefaultPicoContainer.java:84)
       at org.picocontainer.DefaultPicoContainer$AsPropertiesPicoContainer.addComponent(DefaultPicoContainer.java:1149)
       at org.sonar.core.platform.ComponentContainer.addExtension(ComponentContainer.java:241)
       ... 11 common frames omitted
      

      We should remove the declaration of the fail-fast property.

        Attachments

          Activity

            People

            • Assignee:
              michael.gumowski Michael Gumowski
              Reporter:
              quentin.jaquier Quentin Jaquier
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Due:
                Created:
                Updated:
                Resolved: