VSTS SonarCloud warning shown as errors

loshurik · August 20, 2018, 1:51pm

Hello, I’m using VSTS SonarCloud extension (v.1.*) to scan a js-based project.
I get warnings that are recognized as errors:

WARN: Metric ‘comment_lines_data’ is deprecated. Provided value is ignored.
WARN: Access to the multi-values/property set property ‘sonar.typescript.eslint.reportPaths’ should be made using ‘getStringArray’ method. The SonarQube plugin using this property should be updated.

Here are all my Sonar settings used for the project:

sonar.exclusions=node_modules/**,dist/**
sonar.sourceEncoding=UTF-8
sonar.branch.name=develop
sonar.coverage.exclusions=**/*

Is it possible to set warnings to appear as warnings, not errors?
How can I clear these warnings?

Julien_HENRY · October 16, 2018, 2:32pm

Hi,

We made the choice to write WARN logs to stderr:

github.com

SonarSource/sonar-scanner-cli/blob/ac93169472fa0861e99983a86cc00cc9242b1dcc/src/main/java/org/sonarsource/scanner/cli/Logs.java#L59


    String timestamp = currentTime.format(timeFormatter);
    stdOut.println(timestamp + " DEBUG: " + message);
  }
}


public void info(String message) {
  print(stdOut, "INFO: " + message);
}


public void warn(String message) {
  print(stdErr, "WARN: " + message);
}


public void error(String message) {
  print(stdErr, "ERROR: " + message);
}


public void error(String message, Throwable t) {
  print(stdErr, "ERROR: " + message);
  t.printStackTrace(stdErr);
}

The result is that those logs are reported as error in the VSTS UI. To me this is good, otherwise no one would look at those WARN logs.

JVennix · October 18, 2018, 8:44am

It would be good, if your users could do anything with it. However it’s a deprecated method in the scanner/sonar which now fails complete builds and renders the standard Sonar components in VSTS/Azure DevOps totally useless.

Either the warning needs to be changed to stdOut or solved in general.

Else you might as well pull out the Sonar components from the VSTS Marketplace, because they will not be usable because of this error message.

Julien_HENRY · October 18, 2018, 9:06am

Why do you say it fails your build? From your screenshot, I can see “Build process succeeded”. This is just to help me better understand.