-
Notifications
You must be signed in to change notification settings - Fork 85
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
[MM-61120] Add diagnostics data to the Support Packet #532
Conversation
772eab6
to
7537319
Compare
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.
LGTM 👍 Bump the golangci version to 1.55.0 to fix the CI issue.
See https://github.com/mattermost/mattermost-plugin-jira/pull/1136/files
@wiggin77 @Kshitij-Katiyar The CI error seems unrelated. Is that a known issue? |
@Kshitij-Katiyar Gentle reminder to review the PR |
@AayushChaudhary0001 Can you please give the PR a review? |
@Kshitij-Katiyar @AayushChaudhary0001 Can you please review the PR ASAP so we can get it into v1.10.0? Thanks! |
@hanzei Sure, we will look into it |
Hey @hanzei , Can you please specify the testing steps for this PR so we can estimate and start working on the testing areas? |
@arush-vashishtha, I've updated the JIRA ticket with QA test steps. Please check out https://mattermost.atlassian.net/browse/MM-61120. |
Per discussion with @raghavaggarwal2308, QA testing will happen post-merge. |
Summary
When a Support Packet gets generated, the plugin now includes diagnostics data via a
diagnostics.yaml
file into the Packet.Example content of
com.github.manland.mattermost-plugin-gitlab/diagnostics.yaml
:Requires mattermost/mattermost#28833 on the server side
Ticket Link
https://mattermost.atlassian.net/browse/MM-61120