-
-
Notifications
You must be signed in to change notification settings - Fork 380
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.
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
feat: Added extra fields for sentry #1203
Conversation
Signed-off-by: Pranshu Srivastava <iampranshu24@gmail.com>
Signed-off-by: Pranshu Srivastava <iampranshu24@gmail.com>
Apply Sweep Rules to your PR?
|
Signed-off-by: Pranshu Srivastava <iampranshu24@gmail.com>
Signed-off-by: Pranshu Srivastava <iampranshu24@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove the keploy test folder and address the comments
cmd/root.go
Outdated
@@ -124,8 +127,35 @@ func modifyToSentryLogger(log *zap.Logger, client *sentry.Client) *zap.Logger { | |||
} | |||
|
|||
log = zapsentry.AttachCoreToLogger(core, log) | |||
|
|||
return log.With(zapsentry.NewScope()) | |||
kernelVersion := "Not linux" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the significance of this value..?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reverted
Signed-off-by: Pranshu Srivastava <iampranshu24@gmail.com>
Signed-off-by: Pranshu Srivastava <iampranshu24@gmail.com>
Signed-off-by: Pranshu Srivastava <iampranshu24@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please resolve the comments and change the alias in read me and inform the respective people regarding change in alias.
cmd/record.go
Outdated
} else { | ||
fmt.Println("Example usage:\n", cmd.Example) | ||
} | ||
fmt.Println("Error: missing required -c flag or appCmd in config file") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please correct the indentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
cmd/record.go
Outdated
@@ -227,6 +233,9 @@ func (r *Record) GetCmd() *cobra.Command { | |||
|
|||
recordCmd.Flags().String("config-path", ".", "Path to the local directory where keploy configuration file is stored") | |||
|
|||
recordCmd.Flags().Bool("enableTele", true, "Switch for telemetry") | |||
_ = recordCmd.Flags().MarkHidden("enableTele") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this assignment is optional. Please remove assignment if it is optional.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
cmd/root.go
Outdated
} | ||
} | ||
sentry.ConfigureScope(func(scope *sentry.Scope) { | ||
scope.SetTag("Keploy Version", utils.KeployVersion) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
correct indentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Signed-off-by: Pranshu Srivastava <iampranshu24@gmail.com>
Signed-off-by: Pranshu Srivastava <iampranshu24@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ship it!
Related Issue
Currently there were very few fields being sent to sentry which was making it hard to debug
Describe the changes you've made
Add 4 new fields to all the events being sent by sentry using the zap logger
Type of change
Checklist: