Skip to content
This repository has been archived by the owner on Jan 7, 2021. It is now read-only.

Cannot post a summary comment. Too many characters. #113

Open
kingoleg opened this issue Mar 24, 2017 · 4 comments
Open

Cannot post a summary comment. Too many characters. #113

kingoleg opened this issue Mar 24, 2017 · 4 comments

Comments

@kingoleg
Copy link

  1. I have a PR with a lot of new issues (54437/100) because of Support for multi-level maven project where parent lives on the same level directory as child modules #111
build	24-Mar-2017 16:17:26	[WARNING] org.sonarsource.scanner.maven.SonarQubeMojo - 16:17:26.458 Too many issues detected (54437/100): Issues cannot be displayed in Diff view

BTW, this is just a guess. Actually PR consists of only several changed files and all of them are shown in Diff view
WIS:
There is no summary comment
WIE:
There is some summary about "Too many issues found. Some of them: bla bla bla"

build	24-Mar-2017 16:17:26	[DEBUG] org.sonarsource.scanner.maven.SonarQubeMojo - 16:17:26.881 Response NettyResponse {
build	24-Mar-2017 16:17:26		statusCode=400
build	24-Mar-2017 16:17:26		headers=
build	24-Mar-2017 16:17:26			Server: nginx
build	24-Mar-2017 16:17:26			Date: Fri, 24 Mar 2017 13:17:26 GMT
build	24-Mar-2017 16:17:26			Content-Type: application/json;charset=UTF-8
build	24-Mar-2017 16:17:26			Transfer-Encoding: chunked
build	24-Mar-2017 16:17:26			Connection: keep-alive
build	24-Mar-2017 16:17:26			X-AREQUESTID: XXX
build	24-Mar-2017 16:17:26			X-ASEN: XXX
build	24-Mar-2017 16:17:26			X-AUSERID: XXX
build	24-Mar-2017 16:17:26			X-AUSERNAME: XXX
build	24-Mar-2017 16:17:26			Cache-Control: no-cache, no-transform
build	24-Mar-2017 16:17:26			Vary: X-AUSERNAME,Accept-Encoding
build	24-Mar-2017 16:17:26			X-Content-Type-Options: nosniff
build	24-Mar-2017 16:17:26		body=
build	24-Mar-2017 16:17:26	{"errors":[{"context":"text","message":"Please enter a non-empty value less than 32768 characters","exceptionName":null}]}
build	24-Mar-2017 16:17:26	}
build	24-Mar-2017 16:17:26	[ERROR] org.sonarsource.scanner.maven.SonarQubeMojo - 16:17:26.906 Unable to push SonarQube analysis overview to Stash
build	24-Mar-2017 16:17:26	org.sonar.plugins.stash.exceptions.StashClientException: Unable to post a comment to mexos #979. Received 400: null: Please enter a non-empty value less than 32768 characters
build	24-Mar-2017 16:17:26		at org.sonar.plugins.stash.client.StashClient.validateResponse(StashClient.java:307) ~[sonar-stash-plugin-1.3.0-SNAPSHOT.jar:na]
build	24-Mar-2017 16:17:26		at org.sonar.plugins.stash.client.StashClient.performRequest(StashClient.java:295) ~[sonar-stash-plugin-1.3.0-SNAPSHOT.jar:na]
build	24-Mar-2017 16:17:26		at org.sonar.plugins.stash.client.StashClient.postCreate(StashClient.java:264) ~[sonar-stash-plugin-1.3.0-SNAPSHOT.jar:na]
build	24-Mar-2017 16:17:26		at org.sonar.plugins.stash.client.StashClient.postCommentOnPullRequest(StashClient.java:99) ~[sonar-stash-plugin-1.3.0-SNAPSHOT.jar:na]
build	24-Mar-2017 16:17:26		at org.sonar.plugins.stash.StashRequestFacade.postAnalysisOverview(StashRequestFacade.java:70) ~[sonar-stash-plugin-1.3.0-SNAPSHOT.jar:na]
build	24-Mar-2017 16:17:26		at org.sonar.plugins.stash.StashIssueReportingPostJob.postInfoAndPRsActions(StashIssueReportingPostJob.java:131) [sonar-stash-plugin-1.3.0-SNAPSHOT.jar:na]
build	24-Mar-2017 16:17:26		at org.sonar.plugins.stash.StashIssueReportingPostJob.updateStashWithSonarInfo(StashIssueReportingPostJob.java:95) [sonar-stash-plugin-1.3.0-SNAPSHOT.jar:na]
build	24-Mar-2017 16:17:26		at org.sonar.plugins.stash.StashIssueReportingPostJob.executeOn(StashIssueReportingPostJob.java:50) [sonar-stash-plugin-1.3.0-SNAPSHOT.jar:na]
build	24-Mar-2017 16:17:26		at org.sonar.batch.phases.PostJobsExecutor.execute(PostJobsExecutor.java:65) [sonar-scanner-engine-shaded-5.6.6.jar:na]
build	24-Mar-2017 16:17:26		at org.sonar.batch.phases.PostJobsExecutor.execute(PostJobsExecutor.java:55) [sonar-scanner-engine-shaded-5.6.6.jar:na]
@t-8ch
Copy link
Contributor

t-8ch commented Mar 24, 2017

So I argue that the missing basedir was a misconfiguration in the first place.
You should also run your normals scans with this parameter. Then the location of the issues is correct from the beginning and sonar-stash will see only the real new issues.

@kingoleg
Copy link
Author

I guess so. Still this issue is not about missconfiguration.

@t-8ch
Copy link
Contributor

t-8ch commented Mar 24, 2017

Yes. I will look at this on monday.

kingoleg pushed a commit to kingoleg/sonar-stash that referenced this issue Mar 27, 2017
@desmondgc
Copy link

I encountered a similar issue that was not due to misconfiguration. A pull request did in fact introduce several hundred issues (unnecessary parentheses, mostly). All I see is a warning in the build:

WARNING: WARN: Too many issues detected (355/100): Issues cannot be displayed in Diff view

We have the overview comment disabled, but I would still expect something to be added to the PR in this case.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants