Skip to content

Comments

Update Sentry configuration#3797

Merged
mhieta merged 6 commits intomainfrom
HL-1632/sentry-io-changes
Nov 25, 2025
Merged

Update Sentry configuration#3797
mhieta merged 6 commits intomainfrom
HL-1632/sentry-io-changes

Conversation

@mhieta
Copy link
Contributor

@mhieta mhieta commented Nov 24, 2025

Part of the Sentry.io migration.
See RATY-205 for details.

Notes about the Helsinki uwsgi-sentry - plugin:

  • Control with ENABLE_SENTRY_UWSGI_PLUGIN-flag
  • Disable it by default

Refs: HL-1632

@mhieta mhieta requested review from a team as code owners November 24, 2025 12:09
Copy link
Collaborator

@karisal-anders karisal-anders left a comment

Choose a reason for hiding this comment

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

From Kesäseteli side:

  • The only part of this PR that is not benefit-specific is the backend/.pre-commit-config.yaml changes. And those are ok. So, approving those only, nothing else.

Utilize Helsinki uwsgi-sentry - plugin.

Control with ENABLE_SENTRY_UWSGI_PLUGIN-flag.
Disable it by default.

Also fix variable naming and use
explicit platform for clamav-containers
in docker compose.

Refs: HL-1632
@mhieta mhieta force-pushed the HL-1632/sentry-io-changes branch from 2db468f to 35b5ef9 Compare November 24, 2025 12:16
@terovirtanen
Copy link
Contributor

APPLICANT is deployed to platta: https://helsinkilisa-ui-pr3797.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3797.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

YJDH-HELSINKILISA-API branch is deployed to platta: https://helsinkilisa-pr3797.api.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe result is success for https://helsinkilisa-ui-handler-pr3797.dev.hel.ninja 😆🎉🎉🎉

@terovirtanen
Copy link
Contributor

TestCafe applicant result is failed for https://helsinkilisa-ui-pr3797.dev.hel.ninja 😿💢💥💥

@mhieta mhieta marked this pull request as draft November 24, 2025 14:24
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed for 'yjdh'

Failed conditions
2 Security Hotspots
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed for 'yjdh'

Failed conditions
1 Security Hotspot
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed for 'yjdh'

Failed conditions
1 Security Hotspot
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@sonarqubecloud
Copy link

@mhieta mhieta marked this pull request as ready for review November 25, 2025 12:27
@mhieta mhieta merged commit dd12d1a into main Nov 25, 2025
54 of 59 checks passed
@mhieta mhieta deleted the HL-1632/sentry-io-changes branch November 25, 2025 13:20
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.

4 participants