Skip to content

SLE-1534 Support Shell and Azure Pipelines#1076

Draft
damien-urruty-sonarsource wants to merge 1 commit into
masterfrom
feature/dam/SLE-1534-azp-shell
Draft

SLE-1534 Support Shell and Azure Pipelines#1076
damien-urruty-sonarsource wants to merge 1 commit into
masterfrom
feature/dam/SLE-1534-azp-shell

Conversation

@damien-urruty-sonarsource

@damien-urruty-sonarsource damien-urruty-sonarsource commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Summary by Gitar

  • Language support:
    • Added AZUREPIPELINES and SHELL to SonarLintLanguage enum and DEFAULT_CONNECTED_LANGUAGES.
  • Integration tests:
    • Added IT coverage for Azure Pipelines and Shell with new profile configurations and test cases.
    • Implemented helper methods in AbstractSonarQubeConnectedModeTest to verify language support via API and simplify profile restoration.
  • Dependency updates:
    • Updated sonarlint-java-client-osgi version to 11.6.0.85933 across MANIFEST.MF files, pom.xml, and target platforms.

This will update automatically on new commits.

@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Jun 30, 2026

Copy link
Copy Markdown

SLE-1534

@gitar-bot

gitar-bot Bot commented Jun 30, 2026

Copy link
Copy Markdown
CI failed: Integration test failures in `SonarQubeConnectedModeTest` due to connection timeouts, suggesting that recent changes to connected mode or project binding logic are preventing the IDE from synchronizing with the SonarQube instance.

Overview

Integration tests in the SonarQubeConnectedModeTest suite failed with ConditionTimeoutException, indicating that SonarLint is unable to retrieve analysis results or update rule sets within the expected time window. These failures are high-confidence regressions likely tied to the changes in this PR.

Failures

SonarQube Connected Mode Sync Failures (confidence: high)

  • Type: test
  • Affected jobs: 84364866898
  • Related to change: yes
  • Root cause: The system failed to synchronize with SonarQube, resulting in empty rule sets and missing issue detection (e.g., secrets) during integration tests. The ConditionTimeoutException implies that the new connection or project binding logic introduced in this PR is not completing successfully in the test environment.
  • Suggested fix: Review changes to AbstractSonarQubeConnectedModeTest.java and related connection handling logic. Ensure that the new azurepipelines-sonarlint.xml and shell-sonarlint.xml configurations are correctly integrated and not causing side effects in the test lifecycle.

Summary

  • Change-related failures: 1 (Integration test suite failure).
  • Infrastructure/flaky failures: 0
  • Recommended action: Investigate recent modifications to the connected mode setup and binding logic. Validate that the new configuration files for Azure Pipelines and Shell are correctly initialized and do not interfere with the SonarQube orchestration during tests.
Code Review ✅ Approved

Adds support for Shell and Azure Pipelines languages through enum and configuration updates while bumping the sonarlint-java-client-osgi dependency. Comprehensive integration tests verify the new language support and profile handling.

Tip

Comment Gitar fix CI or enable auto-apply: gitar auto-apply:on

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant