-
Notifications
You must be signed in to change notification settings - Fork 1
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
Diagnostic script #231
Diagnostic script #231
Conversation
if (appopticsInstalled) { | ||
console.warn( | ||
"The 'appoptics-apm' package was detected.", | ||
"Make sure to uninstall it properly as it is not compatible with the 'solarwinds-apm' package.", |
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.
Could maybe console log a link to the Upgrade docs for more details, but it might not look as nice. Take or leave 😃
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.
Looks good and helpful!
TIL the built-in process report! This is great @raphael-theriault-swi--I gave it a try in my little test setup but it somehow doesn't think solarwinds-apm is installed, will send you more info in slack. |
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 thanks for the revisit @raphael-theriault-swi!
Checks for common issues and prints a full report that should be very useful for debugging issues on our side.