-
Notifications
You must be signed in to change notification settings - Fork 7
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
[amqphandler] Don't truncate debug logs #168
Conversation
2c702bb
to
26df9cf
Compare
amqphandler/amqphandler.go
Outdated
@@ -60,6 +60,8 @@ const ( | |||
type AmqpHandler struct { //nolint:stylecheck | |||
sync.Mutex | |||
|
|||
disableLogsTrim bool |
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.
Don't see why we need to have it configurable. Let's just remove log trimming at all.
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.
JIRA task: https://jiraeu.epam.com/browse/EPMPAOS-5351
Title: [AOS Core] Add the ability not to truncate the unit's messages to the cloud.
So the task is to add an ability to not truncate logs
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.
Agreed to not truncate logs at all.
Updated PR.
9e24c01
to
ba2aa6a
Compare
Signed-off-by: Mykhailo Lohvynenko <[email protected]>
ba2aa6a
to
8affbff
Compare
Quality Gate passedIssues Measures |
No description provided.