<!-- 
RSS generated by JIRA (1001.0.0-SNAPSHOT#100246-sha1:7a5c50119eb0633d306e14180817ddef5e80c75d) at Thu Feb 08 23:14:39 UTC 2024

It is possible to restrict the fields that are returned in this document by specifying the 'field' parameter in your request.
For example, to request only the issue key and summary add field=key&field=summary to the URL of your request.
-->
<rss version="0.92" >
<channel>
    <title>FOLIO Jira</title>
    <link>https://folio-org.atlassian.net</link>
    <description>This file is an XML representation of an issue</description>
    <language>en-us</language>    <build-info>
        <version>1001.0.0-SNAPSHOT</version>
        <build-number>100246</build-number>
        <build-date>07-02-2024</build-date>
    </build-info>

<item>
            <title>[FOLIO-1617] SonarCloud Integration now fails pull requests on quality gate checks</title>
                <link>https://folio-org.atlassian.net/browse/FOLIO-1617</link>
                <project id="10290" key="FOLIO">FOLIO</project>
                    <description>&lt;p&gt;It seems the new SonarCloud integration fails the check when the pull request changes fail the quality gate for the module.&lt;/p&gt;

&lt;p&gt;This does not block the pull request from being merged, so this is not a critical issue.&lt;/p&gt;

&lt;p&gt;Looking at the &lt;a href=&quot;https://docs.sonarqube.org/7.4/analysis/pull-request/&quot; class=&quot;external-link&quot; rel=&quot;nofollow noreferrer&quot;&gt;documentation&lt;/a&gt;&lt;/p&gt;
&lt;blockquote&gt;&lt;p&gt;When &quot;Confirm&quot;, &quot;Resolved as False Positive&quot; or &quot;Won&apos;t Fix&quot; actions are performed on issues in SonarQube UI, the status of the PR is updated accordingly. This means, if you want to get a green status on the PR, you can either fix the issues for real or &quot;Confirm&quot;, &quot;Resolved as False Positive&quot; or &quot;Won&apos;t Fix&quot; any remaining issues available on the PR.&lt;/p&gt;&lt;/blockquote&gt;

&lt;p&gt;I think that Sonar have made the decision that issues identified in analysis have to be deliberately attended to, and we either accommodate that, lower the standards, or turn off analysis completely on pull requests in order for that check to pass.&lt;/p&gt;

&lt;p&gt;As using the sonar cloud UI requires setting up a relationship with each github user (and that adds a big management overhead), if we want to accommodate this in our process, we may need to restrict overriding issue resolution to a limited set of developers.&lt;/p&gt;</description>
                <environment></environment>
        <key id="80920">FOLIO-1617</key>
            <summary>SonarCloud Integration now fails pull requests on quality gate checks</summary>
                <type id="10003" iconUrl="https://folio-org.atlassian.net/rest/api/2/universal_avatar/view/type/issuetype/avatar/10318?size=medium">Task</type>
                                            <priority id="10002" iconUrl="https://dev.folio.org/assets/jira-priority/jira-p3.svg">P3</priority>
                        <status id="6" iconUrl="https://folio-org.atlassian.net/images/icons/statuses/closed.png" description="The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.">Closed</status>
                    <statusCategory id="3" key="done" colorName="green"/>
                                    <resolution id="10003">Done</resolution>
                                                        <assignee accountid="5f9abc1eb45b2e007453f423">John Malconian</assignee>
                                                                <reporter accountid="63e2a2771b13d42998e4e706">Marc Johnson</reporter>
                                    <labels>
                            <label>core</label>
                            <label>devops</label>
                            <label>sprint51</label>
                    </labels>
                <created>Thu, 15 Nov 2018 16:31:24 +0000</created>
                <updated>Wed, 3 Jun 2020 16:39:15 +0000</updated>
                            <resolved>Fri, 30 Nov 2018 12:47:47 +0000</resolved>
                                                                        <due></due>
                            <votes>0</votes>
                                    <watches>5</watches>
                                                                <comments>
                                                            <comment id="193007" author="5f9abc1eb45b2e007453f423" created="Thu, 15 Nov 2018 17:34:01 +0000"  >&lt;p&gt;So there have been a few changes to Sonarcloud recently.   The Github integration is a bit different,  but the exciting news is that we have the ability to create custom quality gates if we don&apos;t want to use the Sonar default quality gate.     Some ideas and thoughts:&lt;/p&gt;

&lt;ul class=&quot;alternate&quot; type=&quot;square&quot;&gt;
	&lt;li&gt;It&apos;s important to note that Sonarcloud cannot block the ability to merge a pull request.   This is something that is configured in Github on a per repository basis.   We currently have not set this on any repositories.&lt;/li&gt;
&lt;/ul&gt;


&lt;ul class=&quot;alternate&quot; type=&quot;square&quot;&gt;
	&lt;li&gt;We have both the ability to create custom &quot;Quality Gates&quot; in Sonarcloud as well as the ability to modify  &quot;Quality Profiles&quot; - that is,  the ability to tweak rules and create custom profiles  individual rules priority levels can be changed.  i.e . changing &apos;major&apos; to &apos;minor&apos;, etc or removed all together.&lt;/li&gt;
&lt;/ul&gt;


&lt;ul class=&quot;alternate&quot; type=&quot;square&quot;&gt;
	&lt;li&gt;I think it may be a good idea to create a Sonarcloud team that consists of a subset of frontend and backend FOLIO  developers that have administrative access to our Sonarcloud account and who have the ability to tweak rules, profiles, and quality gates more suitable to our needs.   For instance,  if the existing &apos;cognitive complexity&apos; rules in the Java profile don&apos;t work for us, then we can tweak those rules across all projects so that they do not need to be addressed individually.    I would keep this team relatively small - perhaps no more than six developers.&lt;/li&gt;
&lt;/ul&gt;
</comment>
                                                            <comment id="193009" author="63e2a2771b13d42998e4e706" created="Tue, 20 Nov 2018 12:24:08 +0000"  >&lt;p&gt;It would appear the SonarQube integration fails the check (without blocking the pull request) when it discovers any issues, see &lt;a href=&quot;https://sonarcloud.io/project/issues?id=org.folio%3Amod-circulation&amp;amp;pullRequest=152&amp;amp;resolved=false&quot; class=&quot;external-link&quot; rel=&quot;nofollow noreferrer&quot;&gt;https://sonarcloud.io/project/issues?id=org.folio%3Amod-circulation&amp;amp;pullRequest=152&amp;amp;resolved=false&lt;/a&gt; as an example (screenshot attached for when link stops working)&lt;/p&gt;</comment>
                                                            <comment id="193012" author="5f9abc1eb45b2e007453f423" created="Tue, 20 Nov 2018 20:44:54 +0000"  >&lt;p&gt;As long as it doesn&apos;t block the PR from being mergeable, I think I can live with that &lt;a href=&quot;https://folio-org.atlassian.net/secure/ViewProfile.jspa?accountId=63e2a2771b13d42998e4e706&quot; class=&quot;user-hover&quot; rel=&quot;63e2a2771b13d42998e4e706&quot; data-account-id=&quot;63e2a2771b13d42998e4e706&quot; accountid=&quot;63e2a2771b13d42998e4e706&quot; rel=&quot;noreferrer&quot;&gt;Marc Johnson&lt;/a&gt; unless you think it&apos;s a problem.   &lt;/p&gt;</comment>
                                                            <comment id="193014" author="63e2a2771b13d42998e4e706" created="Tue, 20 Nov 2018 21:46:22 +0000"  >&lt;p&gt;&lt;a href=&quot;https://folio-org.atlassian.net/secure/ViewProfile.jspa?accountId=5f9abc1eb45b2e007453f423&quot; class=&quot;user-hover&quot; rel=&quot;5f9abc1eb45b2e007453f423&quot; data-account-id=&quot;5f9abc1eb45b2e007453f423&quot; accountid=&quot;5f9abc1eb45b2e007453f423&quot; rel=&quot;noreferrer&quot;&gt;John Malconian&lt;/a&gt; Agreed, given that it does not block the pull request from being merged, I think conversations about this (and general SonarQube policy) can be deferred.&lt;/p&gt;

&lt;p&gt;we briefly raised it in the core team backend technical meeting, mostly to raise awareness that people may see more failed checks than before.&lt;/p&gt;</comment>
                                                            <comment id="193016" author="5f9abc1eb45b2e007453f423" created="Wed, 28 Nov 2018 13:47:43 +0000"  >&lt;p&gt;&lt;a href=&quot;https://folio-org.atlassian.net/secure/ViewProfile.jspa?accountId=63e2a2771b13d42998e4e706&quot; class=&quot;user-hover&quot; rel=&quot;63e2a2771b13d42998e4e706&quot; data-account-id=&quot;63e2a2771b13d42998e4e706&quot; accountid=&quot;63e2a2771b13d42998e4e706&quot; rel=&quot;noreferrer&quot;&gt;Marc Johnson&lt;/a&gt; Can I go ahead a close this or is there anything outstanding we need to address?&lt;/p&gt;</comment>
                                                            <comment id="193018" author="63e2a2771b13d42998e4e706" created="Fri, 30 Nov 2018 12:47:38 +0000"  >&lt;p&gt;&lt;a href=&quot;https://folio-org.atlassian.net/secure/ViewProfile.jspa?accountId=5f9abc1eb45b2e007453f423&quot; class=&quot;user-hover&quot; rel=&quot;5f9abc1eb45b2e007453f423&quot; data-account-id=&quot;5f9abc1eb45b2e007453f423&quot; accountid=&quot;5f9abc1eb45b2e007453f423&quot; rel=&quot;noreferrer&quot;&gt;John Malconian&lt;/a&gt; I think this can be closed. &lt;/p&gt;

&lt;p&gt;I don&apos;t think we can particularly change the integration behaviour at this point (and in general it is a big improvement). &lt;/p&gt;

&lt;p&gt;I think we may need to accept that there will be more pull requests that get merged despite a red cross, and be mindful of not ignoring the sonar checks because of this.&lt;/p&gt;</comment>
                    </comments>
                    <attachments>
                            <attachment id="64109" name="Screenshot 2018-11-15 15.30.48.png" size="804755" author="63e2a2771b13d42998e4e706" created="Thu, 15 Nov 2018 16:33:18 +0000"/>
                            <attachment id="64110" name="Screenshot 2018-11-15 16.15.14.png" size="228942" author="63e2a2771b13d42998e4e706" created="Thu, 15 Nov 2018 16:32:18 +0000"/>
                            <attachment id="64111" name="Screenshot 2018-11-20 12.24.39.png" size="201334" author="63e2a2771b13d42998e4e706" created="Tue, 20 Nov 2018 12:25:44 +0000"/>
                    </attachments>
                <subtasks>
                    </subtasks>
                <customfields>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                <customfield id="customfield_10000" key="com.atlassian.jira.plugins.jira-development-integration-plugin:devsummarycf">
                        <customfieldname>Development</customfieldname>
                        <customfieldvalues>
                            
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                    <customfield id="customfield_10019" key="com.pyxis.greenhopper.jira:gh-lexo-rank">
                        <customfieldname>Rank</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>0|hzz60n:</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                    <customfield id="customfield_10020" key="com.pyxis.greenhopper.jira:gh-sprint">
                        <customfieldname>Sprint</customfieldname>
                        <customfieldvalues>
                            
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                        <customfield id="customfield_10024" key="com.atlassian.jira.ext.charting:firstresponsedate">
                        <customfieldname>[CHART] Date of First Response</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>Thu, 15 Nov 2018 17:34:01 +0000</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_10025" key="com.atlassian.jira.ext.charting:timeinstatus">
                        <customfieldname>[CHART] Time in Status</customfieldname>
                        <customfieldvalues>
                            
                        </customfieldvalues>
                    </customfield>
                                    </customfields>
    </item>
</channel>
</rss>