Re: Remove sonar step from PR checks
The example you provided [1] is a *correct *hint, again. [1] https://sonarcloud.io/project/issues?resolved=false=BUG=apache_ignite=AYdcaDNkmXUD5o3U1M1d > Also, it is required to exclude from check examples, tests, jmh benchmarks and so on. -1 here. Examples and tests quality are also important. On Thu, Sep 28, 2023 at 2:13 PM Ivan Daschinsky wrote: > >> No, I just think that we should remove 1 yaml file unless the sonar > check is not properly configured. > No, I just think that we should remove 1 yaml file unless the sonar check > is properly configured. -- corrected > > > чт, 28 сент. 2023 г. в 14:12, Ivan Daschinsky : > > > No, I just think that we should remove 1 yaml file unless the sonar check > > is not properly configured. > > > > чт, 28 сент. 2023 г. в 14:09, Anton Vinogradov : > > > >> Ivan, > >> According to your proposal, should we also remove the "Platform .NET" > >> tests > >> [1] since they are always failed instead of fixing them? > >> > >> [1] > >> > >> > https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_PlatformNetCoreLinux?branch=%3Cdefault%3E=builds > >> > >> On Thu, Sep 28, 2023 at 1:57 PM Anton Vinogradov wrote: > >> > >> > > Sonar shows lots of false positives. > >> > Thats not a true. > >> > We have a *bad* code, unfortunatelly, and that's the *reason* of such > >> > hints. > >> > Could you please show "a lot" of false positives? > >> > The url you provided contains a lot of *correct* hints. > >> > > >> > > But constantly red checks lead to ignoring of all checks. > >> > I'm checking each report, and fixed a lot of dummy issues thanks to > it. > >> > > >> > On Thu, Sep 28, 2023 at 1:49 PM Ivan Daschinsky > >> > wrote: > >> > > >> >> Sonar shows lots of false positives. We cannot add //NOSONAR to all > of > >> >> them > >> >> -- it is a tremendous job. > >> >> But constantly red checks lead to ignoring of all checks. Simply > nobody > >> >> pay > >> >> attention to them. It is not acceptable and contradicts to > >> >> the idea of running these checks. > >> >> > >> >> чт, 28 сент. 2023 г. в 13:43, Anton Vinogradov : > >> >> > >> >> > Removing a quality tool it not a good idea. > >> >> > AFAIU, we have two issues here: > >> >> > > >> >> > 1) Sonar always failed at PR because ot token issues > >> >> > Error: Failed to execute goal > >> >> > org.sonarsource.scanner.maven:sonar-maven-plugin:3.9.1.2184:sonar > >> >> > (default-cli) on project apache-ignite: Project not found. Please > >> check > >> >> the > >> >> > 'sonar.projectKey' and 'sonar.organization' properties, the > >> >> 'SONAR_TOKEN' > >> >> > environment variable, or contact the project administrator -> [Help > >> 1] > >> >> > > >> >> > 2) Sonar shows *correct* hints for a *bad* code we have. > >> >> > > >> >> > Both issues should be solved properly, not by removing the quality > >> tool. > >> >> > > >> >> > My huge -1 here > >> >> > > >> >> > On Thu, Sep 28, 2023 at 1:36 PM Ivan Daschinsky < > ivanda...@gmail.com > >> > > >> >> > wrote: > >> >> > > >> >> > > Hi! It seems that these checks simply don't work, at least for > PRs. > >> >> > > Yep, they work ok for master, but some of warnings from these > tools > >> >> seem > >> >> > to > >> >> > > be just rubbish, like > >> >> > > this one -- [1]. > >> >> > > We should either do a tremendous job to fix these issues or > simply > >> >> > disable > >> >> > > these checks. > >> >> > > > >> >> > > Simply ignoring is not an option, I suppose. I think we should at > >> >> least > >> >> > > remove PR checks. > >> >> > > What do you think? > >> >> > > > >> >> > > --- > >> >> > > [1] --- > >> >> > > > >> >> > > > >> >> > > >> >> > >> > https://sonarcloud.io/project/issues?resolved=false=true=CODE_SMELL=apache_ignite=AYi0-_4zULfXwSrNiDt_ > >> >> > > > >> >> > > > >> >> > > > >> >> > > -- > >> >> > > Sincerely yours, Ivan Daschinskiy > >> >> > > > >> >> > > >> >> > >> >> > >> >> -- > >> >> Sincerely yours, Ivan Daschinskiy > >> >> > >> > > >> > > > > > > -- > > Sincerely yours, Ivan Daschinskiy > > > > > -- > Sincerely yours, Ivan Daschinskiy >
Re: Remove sonar step from PR checks
>> No, I just think that we should remove 1 yaml file unless the sonar check is not properly configured. No, I just think that we should remove 1 yaml file unless the sonar check is properly configured. -- corrected чт, 28 сент. 2023 г. в 14:12, Ivan Daschinsky : > No, I just think that we should remove 1 yaml file unless the sonar check > is not properly configured. > > чт, 28 сент. 2023 г. в 14:09, Anton Vinogradov : > >> Ivan, >> According to your proposal, should we also remove the "Platform .NET" >> tests >> [1] since they are always failed instead of fixing them? >> >> [1] >> >> https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_PlatformNetCoreLinux?branch=%3Cdefault%3E=builds >> >> On Thu, Sep 28, 2023 at 1:57 PM Anton Vinogradov wrote: >> >> > > Sonar shows lots of false positives. >> > Thats not a true. >> > We have a *bad* code, unfortunatelly, and that's the *reason* of such >> > hints. >> > Could you please show "a lot" of false positives? >> > The url you provided contains a lot of *correct* hints. >> > >> > > But constantly red checks lead to ignoring of all checks. >> > I'm checking each report, and fixed a lot of dummy issues thanks to it. >> > >> > On Thu, Sep 28, 2023 at 1:49 PM Ivan Daschinsky >> > wrote: >> > >> >> Sonar shows lots of false positives. We cannot add //NOSONAR to all of >> >> them >> >> -- it is a tremendous job. >> >> But constantly red checks lead to ignoring of all checks. Simply nobody >> >> pay >> >> attention to them. It is not acceptable and contradicts to >> >> the idea of running these checks. >> >> >> >> чт, 28 сент. 2023 г. в 13:43, Anton Vinogradov : >> >> >> >> > Removing a quality tool it not a good idea. >> >> > AFAIU, we have two issues here: >> >> > >> >> > 1) Sonar always failed at PR because ot token issues >> >> > Error: Failed to execute goal >> >> > org.sonarsource.scanner.maven:sonar-maven-plugin:3.9.1.2184:sonar >> >> > (default-cli) on project apache-ignite: Project not found. Please >> check >> >> the >> >> > 'sonar.projectKey' and 'sonar.organization' properties, the >> >> 'SONAR_TOKEN' >> >> > environment variable, or contact the project administrator -> [Help >> 1] >> >> > >> >> > 2) Sonar shows *correct* hints for a *bad* code we have. >> >> > >> >> > Both issues should be solved properly, not by removing the quality >> tool. >> >> > >> >> > My huge -1 here >> >> > >> >> > On Thu, Sep 28, 2023 at 1:36 PM Ivan Daschinsky > > >> >> > wrote: >> >> > >> >> > > Hi! It seems that these checks simply don't work, at least for PRs. >> >> > > Yep, they work ok for master, but some of warnings from these tools >> >> seem >> >> > to >> >> > > be just rubbish, like >> >> > > this one -- [1]. >> >> > > We should either do a tremendous job to fix these issues or simply >> >> > disable >> >> > > these checks. >> >> > > >> >> > > Simply ignoring is not an option, I suppose. I think we should at >> >> least >> >> > > remove PR checks. >> >> > > What do you think? >> >> > > >> >> > > --- >> >> > > [1] --- >> >> > > >> >> > > >> >> > >> >> >> https://sonarcloud.io/project/issues?resolved=false=true=CODE_SMELL=apache_ignite=AYi0-_4zULfXwSrNiDt_ >> >> > > >> >> > > >> >> > > >> >> > > -- >> >> > > Sincerely yours, Ivan Daschinskiy >> >> > > >> >> > >> >> >> >> >> >> -- >> >> Sincerely yours, Ivan Daschinskiy >> >> >> > >> > > > -- > Sincerely yours, Ivan Daschinskiy > -- Sincerely yours, Ivan Daschinskiy
Re: Remove sonar step from PR checks
No, I just think that we should remove 1 yaml file unless the sonar check is not properly configured. чт, 28 сент. 2023 г. в 14:09, Anton Vinogradov : > Ivan, > According to your proposal, should we also remove the "Platform .NET" tests > [1] since they are always failed instead of fixing them? > > [1] > > https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_PlatformNetCoreLinux?branch=%3Cdefault%3E=builds > > On Thu, Sep 28, 2023 at 1:57 PM Anton Vinogradov wrote: > > > > Sonar shows lots of false positives. > > Thats not a true. > > We have a *bad* code, unfortunatelly, and that's the *reason* of such > > hints. > > Could you please show "a lot" of false positives? > > The url you provided contains a lot of *correct* hints. > > > > > But constantly red checks lead to ignoring of all checks. > > I'm checking each report, and fixed a lot of dummy issues thanks to it. > > > > On Thu, Sep 28, 2023 at 1:49 PM Ivan Daschinsky > > wrote: > > > >> Sonar shows lots of false positives. We cannot add //NOSONAR to all of > >> them > >> -- it is a tremendous job. > >> But constantly red checks lead to ignoring of all checks. Simply nobody > >> pay > >> attention to them. It is not acceptable and contradicts to > >> the idea of running these checks. > >> > >> чт, 28 сент. 2023 г. в 13:43, Anton Vinogradov : > >> > >> > Removing a quality tool it not a good idea. > >> > AFAIU, we have two issues here: > >> > > >> > 1) Sonar always failed at PR because ot token issues > >> > Error: Failed to execute goal > >> > org.sonarsource.scanner.maven:sonar-maven-plugin:3.9.1.2184:sonar > >> > (default-cli) on project apache-ignite: Project not found. Please > check > >> the > >> > 'sonar.projectKey' and 'sonar.organization' properties, the > >> 'SONAR_TOKEN' > >> > environment variable, or contact the project administrator -> [Help 1] > >> > > >> > 2) Sonar shows *correct* hints for a *bad* code we have. > >> > > >> > Both issues should be solved properly, not by removing the quality > tool. > >> > > >> > My huge -1 here > >> > > >> > On Thu, Sep 28, 2023 at 1:36 PM Ivan Daschinsky > >> > wrote: > >> > > >> > > Hi! It seems that these checks simply don't work, at least for PRs. > >> > > Yep, they work ok for master, but some of warnings from these tools > >> seem > >> > to > >> > > be just rubbish, like > >> > > this one -- [1]. > >> > > We should either do a tremendous job to fix these issues or simply > >> > disable > >> > > these checks. > >> > > > >> > > Simply ignoring is not an option, I suppose. I think we should at > >> least > >> > > remove PR checks. > >> > > What do you think? > >> > > > >> > > --- > >> > > [1] --- > >> > > > >> > > > >> > > >> > https://sonarcloud.io/project/issues?resolved=false=true=CODE_SMELL=apache_ignite=AYi0-_4zULfXwSrNiDt_ > >> > > > >> > > > >> > > > >> > > -- > >> > > Sincerely yours, Ivan Daschinskiy > >> > > > >> > > >> > >> > >> -- > >> Sincerely yours, Ivan Daschinskiy > >> > > > -- Sincerely yours, Ivan Daschinskiy
Re: Remove sonar step from PR checks
I have already shown one example. Here is another one -- https://sonarcloud.io/project/issues?resolved=false=BUG=apache_ignite=AYdcaDNkmXUD5o3U1M1d Also, it is required to exclude from check examples, tests, jmh benchmarks and so on. A lot of useless information doesn't help to find new issues in new code. And this is one of the most important goals of such a tool. чт, 28 сент. 2023 г. в 13:58, Anton Vinogradov : > > Sonar shows lots of false positives. > Thats not a true. > We have a *bad* code, unfortunatelly, and that's the *reason* of such > hints. > Could you please show "a lot" of false positives? > The url you provided contains a lot of *correct* hints. > > > But constantly red checks lead to ignoring of all checks. > I'm checking each report, and fixed a lot of dummy issues thanks to it. > > On Thu, Sep 28, 2023 at 1:49 PM Ivan Daschinsky > wrote: > > > Sonar shows lots of false positives. We cannot add //NOSONAR to all of > them > > -- it is a tremendous job. > > But constantly red checks lead to ignoring of all checks. Simply nobody > pay > > attention to them. It is not acceptable and contradicts to > > the idea of running these checks. > > > > чт, 28 сент. 2023 г. в 13:43, Anton Vinogradov : > > > > > Removing a quality tool it not a good idea. > > > AFAIU, we have two issues here: > > > > > > 1) Sonar always failed at PR because ot token issues > > > Error: Failed to execute goal > > > org.sonarsource.scanner.maven:sonar-maven-plugin:3.9.1.2184:sonar > > > (default-cli) on project apache-ignite: Project not found. Please check > > the > > > 'sonar.projectKey' and 'sonar.organization' properties, the > 'SONAR_TOKEN' > > > environment variable, or contact the project administrator -> [Help 1] > > > > > > 2) Sonar shows *correct* hints for a *bad* code we have. > > > > > > Both issues should be solved properly, not by removing the quality > tool. > > > > > > My huge -1 here > > > > > > On Thu, Sep 28, 2023 at 1:36 PM Ivan Daschinsky > > > wrote: > > > > > > > Hi! It seems that these checks simply don't work, at least for PRs. > > > > Yep, they work ok for master, but some of warnings from these tools > > seem > > > to > > > > be just rubbish, like > > > > this one -- [1]. > > > > We should either do a tremendous job to fix these issues or simply > > > disable > > > > these checks. > > > > > > > > Simply ignoring is not an option, I suppose. I think we should at > least > > > > remove PR checks. > > > > What do you think? > > > > > > > > --- > > > > [1] --- > > > > > > > > > > > > > > https://sonarcloud.io/project/issues?resolved=false=true=CODE_SMELL=apache_ignite=AYi0-_4zULfXwSrNiDt_ > > > > > > > > > > > > > > > > -- > > > > Sincerely yours, Ivan Daschinskiy > > > > > > > > > > > > > -- > > Sincerely yours, Ivan Daschinskiy > > > -- Sincerely yours, Ivan Daschinskiy
Re: Remove sonar step from PR checks
Ivan, According to your proposal, should we also remove the "Platform .NET" tests [1] since they are always failed instead of fixing them? [1] https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_PlatformNetCoreLinux?branch=%3Cdefault%3E=builds On Thu, Sep 28, 2023 at 1:57 PM Anton Vinogradov wrote: > > Sonar shows lots of false positives. > Thats not a true. > We have a *bad* code, unfortunatelly, and that's the *reason* of such > hints. > Could you please show "a lot" of false positives? > The url you provided contains a lot of *correct* hints. > > > But constantly red checks lead to ignoring of all checks. > I'm checking each report, and fixed a lot of dummy issues thanks to it. > > On Thu, Sep 28, 2023 at 1:49 PM Ivan Daschinsky > wrote: > >> Sonar shows lots of false positives. We cannot add //NOSONAR to all of >> them >> -- it is a tremendous job. >> But constantly red checks lead to ignoring of all checks. Simply nobody >> pay >> attention to them. It is not acceptable and contradicts to >> the idea of running these checks. >> >> чт, 28 сент. 2023 г. в 13:43, Anton Vinogradov : >> >> > Removing a quality tool it not a good idea. >> > AFAIU, we have two issues here: >> > >> > 1) Sonar always failed at PR because ot token issues >> > Error: Failed to execute goal >> > org.sonarsource.scanner.maven:sonar-maven-plugin:3.9.1.2184:sonar >> > (default-cli) on project apache-ignite: Project not found. Please check >> the >> > 'sonar.projectKey' and 'sonar.organization' properties, the >> 'SONAR_TOKEN' >> > environment variable, or contact the project administrator -> [Help 1] >> > >> > 2) Sonar shows *correct* hints for a *bad* code we have. >> > >> > Both issues should be solved properly, not by removing the quality tool. >> > >> > My huge -1 here >> > >> > On Thu, Sep 28, 2023 at 1:36 PM Ivan Daschinsky >> > wrote: >> > >> > > Hi! It seems that these checks simply don't work, at least for PRs. >> > > Yep, they work ok for master, but some of warnings from these tools >> seem >> > to >> > > be just rubbish, like >> > > this one -- [1]. >> > > We should either do a tremendous job to fix these issues or simply >> > disable >> > > these checks. >> > > >> > > Simply ignoring is not an option, I suppose. I think we should at >> least >> > > remove PR checks. >> > > What do you think? >> > > >> > > --- >> > > [1] --- >> > > >> > > >> > >> https://sonarcloud.io/project/issues?resolved=false=true=CODE_SMELL=apache_ignite=AYi0-_4zULfXwSrNiDt_ >> > > >> > > >> > > >> > > -- >> > > Sincerely yours, Ivan Daschinskiy >> > > >> > >> >> >> -- >> Sincerely yours, Ivan Daschinskiy >> >
Re: Remove sonar step from PR checks
> Sonar shows lots of false positives. Thats not a true. We have a *bad* code, unfortunatelly, and that's the *reason* of such hints. Could you please show "a lot" of false positives? The url you provided contains a lot of *correct* hints. > But constantly red checks lead to ignoring of all checks. I'm checking each report, and fixed a lot of dummy issues thanks to it. On Thu, Sep 28, 2023 at 1:49 PM Ivan Daschinsky wrote: > Sonar shows lots of false positives. We cannot add //NOSONAR to all of them > -- it is a tremendous job. > But constantly red checks lead to ignoring of all checks. Simply nobody pay > attention to them. It is not acceptable and contradicts to > the idea of running these checks. > > чт, 28 сент. 2023 г. в 13:43, Anton Vinogradov : > > > Removing a quality tool it not a good idea. > > AFAIU, we have two issues here: > > > > 1) Sonar always failed at PR because ot token issues > > Error: Failed to execute goal > > org.sonarsource.scanner.maven:sonar-maven-plugin:3.9.1.2184:sonar > > (default-cli) on project apache-ignite: Project not found. Please check > the > > 'sonar.projectKey' and 'sonar.organization' properties, the 'SONAR_TOKEN' > > environment variable, or contact the project administrator -> [Help 1] > > > > 2) Sonar shows *correct* hints for a *bad* code we have. > > > > Both issues should be solved properly, not by removing the quality tool. > > > > My huge -1 here > > > > On Thu, Sep 28, 2023 at 1:36 PM Ivan Daschinsky > > wrote: > > > > > Hi! It seems that these checks simply don't work, at least for PRs. > > > Yep, they work ok for master, but some of warnings from these tools > seem > > to > > > be just rubbish, like > > > this one -- [1]. > > > We should either do a tremendous job to fix these issues or simply > > disable > > > these checks. > > > > > > Simply ignoring is not an option, I suppose. I think we should at least > > > remove PR checks. > > > What do you think? > > > > > > --- > > > [1] --- > > > > > > > > > https://sonarcloud.io/project/issues?resolved=false=true=CODE_SMELL=apache_ignite=AYi0-_4zULfXwSrNiDt_ > > > > > > > > > > > > -- > > > Sincerely yours, Ivan Daschinskiy > > > > > > > > -- > Sincerely yours, Ivan Daschinskiy >
Re: Remove sonar step from PR checks
Sonar shows lots of false positives. We cannot add //NOSONAR to all of them -- it is a tremendous job. But constantly red checks lead to ignoring of all checks. Simply nobody pay attention to them. It is not acceptable and contradicts to the idea of running these checks. чт, 28 сент. 2023 г. в 13:43, Anton Vinogradov : > Removing a quality tool it not a good idea. > AFAIU, we have two issues here: > > 1) Sonar always failed at PR because ot token issues > Error: Failed to execute goal > org.sonarsource.scanner.maven:sonar-maven-plugin:3.9.1.2184:sonar > (default-cli) on project apache-ignite: Project not found. Please check the > 'sonar.projectKey' and 'sonar.organization' properties, the 'SONAR_TOKEN' > environment variable, or contact the project administrator -> [Help 1] > > 2) Sonar shows *correct* hints for a *bad* code we have. > > Both issues should be solved properly, not by removing the quality tool. > > My huge -1 here > > On Thu, Sep 28, 2023 at 1:36 PM Ivan Daschinsky > wrote: > > > Hi! It seems that these checks simply don't work, at least for PRs. > > Yep, they work ok for master, but some of warnings from these tools seem > to > > be just rubbish, like > > this one -- [1]. > > We should either do a tremendous job to fix these issues or simply > disable > > these checks. > > > > Simply ignoring is not an option, I suppose. I think we should at least > > remove PR checks. > > What do you think? > > > > --- > > [1] --- > > > > > https://sonarcloud.io/project/issues?resolved=false=true=CODE_SMELL=apache_ignite=AYi0-_4zULfXwSrNiDt_ > > > > > > > > -- > > Sincerely yours, Ivan Daschinskiy > > > -- Sincerely yours, Ivan Daschinskiy
Re: Remove sonar step from PR checks
Removing a quality tool it not a good idea. AFAIU, we have two issues here: 1) Sonar always failed at PR because ot token issues Error: Failed to execute goal org.sonarsource.scanner.maven:sonar-maven-plugin:3.9.1.2184:sonar (default-cli) on project apache-ignite: Project not found. Please check the 'sonar.projectKey' and 'sonar.organization' properties, the 'SONAR_TOKEN' environment variable, or contact the project administrator -> [Help 1] 2) Sonar shows *correct* hints for a *bad* code we have. Both issues should be solved properly, not by removing the quality tool. My huge -1 here On Thu, Sep 28, 2023 at 1:36 PM Ivan Daschinsky wrote: > Hi! It seems that these checks simply don't work, at least for PRs. > Yep, they work ok for master, but some of warnings from these tools seem to > be just rubbish, like > this one -- [1]. > We should either do a tremendous job to fix these issues or simply disable > these checks. > > Simply ignoring is not an option, I suppose. I think we should at least > remove PR checks. > What do you think? > > --- > [1] --- > > https://sonarcloud.io/project/issues?resolved=false=true=CODE_SMELL=apache_ignite=AYi0-_4zULfXwSrNiDt_ > > > > -- > Sincerely yours, Ivan Daschinskiy >
Remove sonar step from PR checks
Hi! It seems that these checks simply don't work, at least for PRs. Yep, they work ok for master, but some of warnings from these tools seem to be just rubbish, like this one -- [1]. We should either do a tremendous job to fix these issues or simply disable these checks. Simply ignoring is not an option, I suppose. I think we should at least remove PR checks. What do you think? --- [1] --- https://sonarcloud.io/project/issues?resolved=false=true=CODE_SMELL=apache_ignite=AYi0-_4zULfXwSrNiDt_ -- Sincerely yours, Ivan Daschinskiy