Skip to content
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

Remove old search metrics collection system. #1274

Merged
merged 4 commits into from
Nov 5, 2018

Conversation

newswangerd
Copy link
Member

@newswangerd newswangerd commented Nov 2, 2018

Removes support for tracking search queries using prometheus (added in #989) because we are inserting metrics directly into influx rather than using Prometheus.

# Metrics Settings
# =========================================================

METRICS_ENABLED = True
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should keep METRICS_ENABLED setting and adopt it for new metrics implementation, so that InfluxDB service can be optional for development environment.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. I renamed it to GALAXY_METRICS_ENABLED.

@newswangerd newswangerd merged commit 5ebe403 into ansible:devel Nov 5, 2018
newswangerd added a commit to newswangerd/galaxy that referenced this pull request Nov 5, 2018
* Remove old search metrics collection system.

* update docs

* Add option to disable metrics in backend.

(cherry picked from commit 5ebe403)
newswangerd added a commit that referenced this pull request Nov 5, 2018
* Remove old search metrics collection system.

* update docs

* Add option to disable metrics in backend.

(cherry picked from commit 5ebe403)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants