Skip to content

Add sonar gen CI workflow - #4686

Open
efiacor wants to merge 1 commit into
kptdev:mainfrom
Nordix:sonar_results_gen
Open

Add sonar gen CI workflow#4686
efiacor wants to merge 1 commit into
kptdev:mainfrom
Nordix:sonar_results_gen

Conversation

@efiacor

@efiacor efiacor commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Add SonarCloud continuous code quality analysis to kpt CI/CD pipeline.

Changes

  • sonar.properties: Project configuration for SonarCloud analysis
  • .github/workflows/go.yml: Integrated coverage report generation via make test-coverage
  • .github/workflows/sonarcloud.yml: Automated analysis workflow triggered on successful builds
  • make/testing.mk: New makefile with coverage generation targets

Motivation

Enables automated code quality tracking, coverage metrics, and security scanning on every PR and push. Coverage reports generated during CI and analyzed by SonarCloud without duplicating test runs.

Integration

  • Tests run once: make test-coverage generates coverage data
  • Coverage artifact uploaded from go.yml
  • sonarcloud.yml downloads and sends to SonarCloud
  • PR comments with quality metrics and coverage deltas
  • Dashboard tracking at https://sonarcloud.io/project/overview?id=kptdev_kpt

Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>
@efiacor
efiacor requested review from a team August 6, 2026 22:35
@netlify

netlify Bot commented Aug 6, 2026

Copy link
Copy Markdown

Deploy Preview for kptdocs ready!

Name Link
🔨 Latest commit fc2012e
🔍 Latest deploy log https://app.netlify.com/projects/kptdocs/deploys/6a750c3489c2ee0008d2308f
😎 Deploy Preview https://deploy-preview-4686--kptdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. area/devops labels Aug 6, 2026
@efiacor efiacor self-assigned this Aug 6, 2026
@sonarqubecloud

sonarqubecloud Bot commented Aug 6, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@dosubot dosubot Bot added the lgtm label Aug 7, 2026
@aravindtga
aravindtga requested review from a team August 7, 2026 07:14
@efiacor
efiacor requested a review from CsatariGergely August 7, 2026 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/devops lgtm size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants