Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: report bug to sentry #4328

Merged
merged 12 commits into from
Sep 13, 2024
Merged

feat: report bug to sentry #4328

merged 12 commits into from
Sep 13, 2024

Conversation

julienrbrt
Copy link
Member

@julienrbrt julienrbrt commented Aug 29, 2024 •
edited
Loading

Add sentry to report ignite errors.
Allow us to fix bug easily.

@julienrbrt julienrbrt marked this pull request as draft August 29, 2024 08:07
@julienrbrt julienrbrt marked this pull request as ready for review September 12, 2024 13:32
Pantani
Pantani reviewed Sep 12, 2024
View reviewed changes
ignite/pkg/sentry/sentry.go Outdated Show resolved Hide resolved
ignite/pkg/sentry/sentry.go Outdated Show resolved Hide resolved
@julienrbrt julienrbrt enabled auto-merge (squash) September 13, 2024 12:52
@julienrbrt julienrbrt merged commit 0adabfd into main Sep 13, 2024
41 of 45 checks passed
@julienrbrt julienrbrt deleted the julien/introduce-sentry branch September 13, 2024 13:02
@julienrbrt julienrbrt added the backport/v28.x.y Backport to v28.x.y label Sep 13, 2024
mergify bot pushed a commit that referenced this pull request Sep 13, 2024
@julienrbrt @mergify
feat: report bug to sentry (#4328)
ca03ff9 
(cherry picked from commit 0adabfd)

# Conflicts:
#	go.mod
#	ignite/internal/analytics/analytics.go
@mergify mergify bot mentioned this pull request Sep 13, 2024
Merged
julienrbrt added a commit that referenced this pull request Sep 13, 2024
@mergify @julienrbrt
feat: report bug to sentry (backport #4328) (#4358)
fa8078a 
* feat: report bug to sentry (#4328)

(cherry picked from commit 0adabfd)

# Conflicts:
#	go.mod
#	ignite/internal/analytics/analytics.go

* updates

* fix cl

---------

Co-authored-by: Julien Robert <julien@rbrt.fr>
@BrewTestBot BrewTestBot mentioned this pull request Sep 20, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Pantani Pantani Pantani approved these changes

@ilgooz ilgooz Awaiting requested review from ilgooz ilgooz is a code owner

@jeronimoalbi jeronimoalbi Awaiting requested review from jeronimoalbi jeronimoalbi is a code owner

@salmad3 salmad3 Awaiting requested review from salmad3 salmad3 is a code owner

@toschdev toschdev Awaiting requested review from toschdev toschdev is a code owner

Assignees

@julienrbrt julienrbrt

Labels
backport/v28.x.y Backport to v28.x.y component:ci CI/CD workflow and automated jobs. component:cmd component:configs component:packages type:internal
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@julienrbrt @Pantani