search

discourse / prometheus_exporter

A framework for collecting and aggregating prometheus metrics
MIT License
532 stars 154 forks source link

Revert breaking http rename #241

Closed ayellapragada closed 2 years ago

ayellapragada commented 2 years ago

This change was made to follow prometheus conventions, but our albaik manifests do not use this in the same way.

Because of this, we're temporarily reverting this one commit, or else autoscaling could have issues when one deploy happens without either the web or manifest changes.

Once autoscaling isn't needed, such as after hours, this fork can be removed on any app that is using this and switch to the official discourse gem.

ayellapragada commented 2 years ago

Woops! Meant this for our fork. Thanks for your hard work y'all!

SamSaffron commented 2 years ago

no worries, sorry about the disruption here, I tried to signal pretty loudly by increasing version numbers. I agree with the change but also agree it is painful.