-
-
Notifications
You must be signed in to change notification settings - Fork 758
Drop st2exporter #5676
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
Drop st2exporter #5676
Conversation
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.
Good work!
I had a question wrt DB changes - will all existing st2exporter-related DB entries (marker, etc.) be automatically purged with these changes? Or is that something we make st2garbagecollector handle?
@rush-skills Good call. I added a migration util that can drop the marker collections. |
Removed st2exporter service. It is unmaintained and does not get installed. It was originally meant to help with analytics by exporting executions as json files that could be imported into something like elasticsearch. Our code is now instrumented to make a wider variety of stats available to metrics drivers.
Requires StackStorm/st2-packages#723
Closes #5295
Closes StackStorm/st2-packages#367