Skip to content

fix: Sonarqube reliability and maintainability issues.#862

Open
pvital wants to merge 2 commits intomainfrom
sonarqube_reliability_fixes
Open

fix: Sonarqube reliability and maintainability issues.#862
pvital wants to merge 2 commits intomainfrom
sonarqube_reliability_fixes

Conversation

@pvital
Copy link
Copy Markdown
Member

@pvital pvital commented Apr 7, 2026

This PR fixes a few SonarQube reliability and maintainability issues.

  • reliability:
    • Fixed in src/instana/instrumentation/aws/boto3.py:
      • Low reliability issue Function parameters initial values should not be ignored.
    • Fixed in src/instana/instrumentation/tornado/client.py:
      • Medium reliability issue All "except" blocks should be able to catch exceptions.
  • maintainability
    • Fixed several files for the issue Replace the unused local variable "parent_span" with "_".

pvital added 2 commits April 7, 2026 23:08
- Fixed in `src/instana/instrumentation/aws/boto3.py`:
    - Low reliability issue **Function parameters initial values should
not be ignored.**
- Fixed in `src/instana/instrumentation/tornado/client.py`:
    - Medium reliability issue **All "except" blocks should be able to
catch exceptions.**

Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
Fixed several files for the issue **Replace the unused local variable "parent_span" with "_".**

Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
@pvital pvital self-assigned this Apr 7, 2026
@pvital pvital added the fix label Apr 7, 2026
@pvital pvital marked this pull request as ready for review April 7, 2026 21:59
@pvital pvital requested a review from a team as a code owner April 7, 2026 21:59
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 7, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
7.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant