SonarQube
  1. SonarQube
  2. SONAR-3126

Stop the Sonar Analysis when the Sonar batch and the Sonar server are not using the same DB

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.1
    • Component/s: Database, Log
    • Labels:
      None
    • Number of attachments :
      0

      Description

      Using the Standalone Runner, if the runner is bad configured (another kind of database for instance), the error is not really clear (BootstrapException => PicoLifecycleException => ... => ClassNotFoundException).

      Of course it's due to a bad configuration, however we may help the user to find exactly the error.

      For instance me way check that both url from runner and server are the same and that the jdbc is the same also.

      Check of username/password for database may be done but can lead to security issue.

        Issue Links

          Activity

          Hide
          Freddy Mallet added a comment -

          Here is the description of the use case which must be covered :

          • There is a Sonar MySQL DB in version 3.1 -> SONAR-DB-A
          • There is another Sonar MySQL DB in version 3.1 -> SONAR-DB-B
          • A Sonar server SONAR-1 is using SONAR-DB-A
          • A Sonar batch is using SONAR-1 but SONAR-DB-B instead of SONAR-DB-A

          In that case, a meaningful error message should be thrown

          Show
          Freddy Mallet added a comment - Here is the description of the use case which must be covered : There is a Sonar MySQL DB in version 3.1 -> SONAR-DB-A There is another Sonar MySQL DB in version 3.1 -> SONAR-DB-B A Sonar server SONAR-1 is using SONAR-DB-A A Sonar batch is using SONAR-1 but SONAR-DB-B instead of SONAR-DB-A In that case, a meaningful error message should be thrown
          Hide
          Freddy Mallet added a comment -

          Manually tested !

          Show
          Freddy Mallet added a comment - Manually tested !
          Hide
          Fabrice Bellingard added a comment -

          No need for IT.

          Show
          Fabrice Bellingard added a comment - No need for IT.

            People

            • Assignee:
              Fabrice Bellingard
              Reporter:
              Eric Hartmann
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: