Skip to content

fix(service.yml): set sonarqube.languages#498

Open
Charles Provencher (cprovencher) wants to merge 1 commit into
masterfrom
fix-service-bot-sonarqube-languages
Open

fix(service.yml): set sonarqube.languages#498
Charles Provencher (cprovencher) wants to merge 1 commit into
masterfrom
fix-service-bot-sonarqube-languages

Conversation

@cprovencher

Copy link
Copy Markdown
Member

Why

The cc-service-bot nightly run (job 7c09131f, 2026-05-28) failed for this repo:

Invalid service-bot manifest for 'sonarqube.languages': You must either set 'lang' at the top level of your service.yaml or 'languages' in the 'sonarqube' block.

This repo has lang: unknown and enables sonarqube without specifying scan languages.

Change

Set sonarqube.languages to the repo's primary language (derived from the repo's GitHub language statistics).

Safety

Draft PR for the owning team to review — please confirm the scan language(s) are correct for this repo. Part of a sweep fixing the service-bot nightly; systemic regressions affecting ~262 other repos are fixed in confluentinc/cc-service-bot#2030.

Copilot AI review requested due to automatic review settings May 29, 2026 20:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the service-bot manifest to satisfy the SonarQube language validation requirement while leaving the repo’s top-level language metadata unchanged.

Changes:

  • Adds sonarqube.languages with java as the configured scan language.
  • Keeps the existing SonarQube enablement and Semaphore/docker image build configuration unchanged.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cprovencher

Charles Provencher (cprovencher) commented Jun 5, 2026

Copy link
Copy Markdown
Member Author

🤖 Review Hub assignments

@cp-build-and-release-teamKaranpreet Singh (@karan-singh-99)
Top-scored candidate (rank 1 of 6). Team was explicitly requested on this PR.

Scoring breakdown for Karanpreet Singh (@karan-singh-99)
Factor Score
Familiarity 0
Load penalty 0
Round-robin 20
Total 20

@kafka-engJohn Huynh (@johuynh343)
Top-scored candidate (rank 1 of 145). Team was explicitly requested on this PR.

Scoring breakdown for John Huynh (@johuynh343)
Factor Score
Familiarity 0
Load penalty 0
Round-robin 25
Total 25

Generated by Review Hub. This comment is rewritten on each (re)assignment.

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

Labels

review-hub:assigned Applied by Review Hub

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants