chore: sacar SonarQube de las plantillas #4

Closed

julio.caicedo wants to merge 1 commit from chore/sacar-sonar into main

pull from: chore/sacar-sonar

merge into: startcodex:main

startcodex:main

Conversation 1 Commits 1 Files changed 9 +21 -310

julio.caicedo commented 2026-10-16 05:33:27 +00:00

Owner

Copy link

El servidor de SonarQube se retira, así que los pasos que lo llamaban empezarían a fallar en cada PR.

Qué sale

• El job sonarqube de _analytics.yaml y el job sonar de _go-analytics.yaml.
• Las entradas sonar_project_key, sonarqube_host y sonar_enabled.
• El secreto SONARQUBE_TOKEN. En _analytics.yaml eso vacía el bloque secrets: entero, que además era lo que rompía esa plantilla en Forgejo 15.0.9.
• coverage_artifact y coverage_exclusions, que existían solo para alimentar a Sonar.

Qué se queda

Opengrep, que es el SAST y no dependía de Sonar. En _go-analytics.yaml siguen también govulncheck y gitleaks. _analytics.yaml queda con un solo job.

Lo que se pierde, dicho de frente

Con Sonar se va la única medición de cobertura del pipeline. _go-test la sigue produciendo como artifact, pero ya nadie la lee ni la exige: pasa de ser una puerta de calidad a ser evidencia. Si eso importa, hay que reponerlo con otra herramienta, y no es parte de este PR.

También quedan sin medir bugs, code smells y duplicación.

Verificado

• Los dos YAML cargan y los jobs que quedan son opengrep en uno y vuln, secrets-scan, sast en el otro.
• Sin claves repetidas ni entradas duplicadas —lo chequeo aparte porque YAML acepta claves repetidas en silencio y un corte mal hecho pasa desapercibido—.
• gofmt, go vet y go test en verde.
• Cero menciones a Sonar en el repositorio salvo tres comentarios que explican por qué ya no está.

El consumidor de Balanzio va aparte, en el PR #13.

🤖 Generated with Claude Code

https://claude.ai/code/session_012oVizRo2WB4FeZVRnHr9Ye

El servidor de SonarQube se retira, así que los pasos que lo llamaban empezarían a fallar en cada PR. ## Qué sale - El job `sonarqube` de `_analytics.yaml` y el job `sonar` de `_go-analytics.yaml`. - Las entradas `sonar_project_key`, `sonarqube_host` y `sonar_enabled`. - El secreto `SONARQUBE_TOKEN`. En `_analytics.yaml` eso vacía el bloque `secrets:` entero, que además era lo que rompía esa plantilla en Forgejo 15.0.9. - `coverage_artifact` y `coverage_exclusions`, que existían solo para alimentar a Sonar. ## Qué se queda **Opengrep**, que es el SAST y no dependía de Sonar. En `_go-analytics.yaml` siguen también **govulncheck** y **gitleaks**. `_analytics.yaml` queda con un solo job. ## Lo que se pierde, dicho de frente Con Sonar se va **la única medición de cobertura del pipeline**. `_go-test` la sigue produciendo como artifact, pero ya nadie la lee ni la exige: pasa de ser una puerta de calidad a ser evidencia. Si eso importa, hay que reponerlo con otra herramienta, y no es parte de este PR. También quedan sin medir bugs, code smells y duplicación. ## Verificado - Los dos YAML cargan y los jobs que quedan son `opengrep` en uno y `vuln, secrets-scan, sast` en el otro. - Sin claves repetidas ni entradas duplicadas —lo chequeo aparte porque YAML acepta claves repetidas en silencio y un corte mal hecho pasa desapercibido—. - `gofmt`, `go vet` y `go test` en verde. - Cero menciones a Sonar en el repositorio salvo tres comentarios que explican por qué ya no está. El consumidor de Balanzio va aparte, en el PR #13. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_012oVizRo2WB4FeZVRnHr9Ye

julio.caicedo added 1 commit 2026-10-16 05:33:27 +00:00

chore: sacar SonarQube de las plantillas ... 85944a31e2

El servidor se retira, así que los pasos que lo llamaban empezarían a fallar
en cada PR. Se van el job de calidad de _analytics y el de _go-analytics, sus
entradas y el secreto SONARQUBE_TOKEN.

Opengrep se queda: es el SAST y no depende de Sonar. En _go-analytics también
siguen govulncheck y gitleaks.

Con Sonar se va la única medición de cobertura del pipeline. _go-test la
sigue produciendo como artifact, pero ya nadie la lee ni la exige: queda como
evidencia, no como puerta de calidad. Por eso también se van coverage_artifact
y coverage_exclusions, que existían solo para alimentarlo.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012oVizRo2WB4FeZVRnHr9Ye

julio.caicedo commented 2026-10-16 05:41:08 +00:00

Author

Owner

Copy link

Se cierra sin mergear: partía de una premisa equivocada.

El pedido era quitar la cuenta sonarqube-bot de Forgejo, no retirar SonarQube. Son cosas distintas: el bot sirve para que SonarQube escriba comentarios en los PR, mientras que el análisis y el Quality Gate se autentican con SONARQUBE_TOKEN. Borrar el bot no toca el análisis.

SonarQube sigue en pie y en uso, así que las plantillas se quedan como estaban. Lo único que se pierde con el bot fuera es la decoración de PRs.

El consumidor de Balanzio también quedó revertido en el PR #13.

Se cierra sin mergear: partía de una premisa equivocada. El pedido era quitar la cuenta `sonarqube-bot` de Forgejo, no retirar SonarQube. Son cosas distintas: el bot sirve para que SonarQube escriba comentarios en los PR, mientras que el análisis y el Quality Gate se autentican con `SONARQUBE_TOKEN`. Borrar el bot no toca el análisis. SonarQube sigue en pie y en uso, así que las plantillas se quedan como estaban. Lo único que se pierde con el bot fuera es la decoración de PRs. El consumidor de Balanzio también quedó revertido en el PR #13.

julio.caicedo closed this pull request 2026-10-16 05:41:08 +00:00

julio.caicedo referenced this pull request from a commit 2026-10-16 13:12:03 +00:00

fix: sacar secrets de workflow_call en las ocho plantillas

julio.caicedo referenced this pull request 2026-10-16 13:13:10 +00:00

fix: sacar secrets de workflow_call en las ocho plantillas #5

Some checks failed

Hide all checks

_build-docker.yaml / chore: sacar SonarQube de las plantillas (push) Failing after 0s

Details

_deploy-k8s.yaml / chore: sacar SonarQube de las plantillas (push) Failing after 0s

Details

_go-analytics.yaml / chore: sacar SonarQube de las plantillas (push) Failing after 0s

Details

_go-build.yaml / chore: sacar SonarQube de las plantillas (push) Failing after 0s

Details

_go-deploy.yaml / chore: sacar SonarQube de las plantillas (push) Failing after 0s

Details

_go-test.yaml / chore: sacar SonarQube de las plantillas (push) Failing after 0s

Details

_test.yaml / chore: sacar SonarQube de las plantillas (push) Failing after 0s

Details

_build-docker.yaml / chore: sacar SonarQube de las plantillas (pull_request) Failing after 0s

Details

_deploy-k8s.yaml / chore: sacar SonarQube de las plantillas (pull_request) Failing after 0s

Details

_go-analytics.yaml / chore: sacar SonarQube de las plantillas (pull_request) Failing after 0s

Details

_go-build.yaml / chore: sacar SonarQube de las plantillas (pull_request) Failing after 0s

Details

_go-deploy.yaml / chore: sacar SonarQube de las plantillas (pull_request) Failing after 0s

Details

_go-test.yaml / chore: sacar SonarQube de las plantillas (pull_request) Failing after 0s

Details

_test.yaml / chore: sacar SonarQube de las plantillas (pull_request) Failing after 0s

Details

Go / Test (pull_request) Successful in 1m3s

Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

startcodex/ci!4

No description provided.