<div dir="ltr"><div dir="auto">Hi Adrian,<div dir="auto"><br></div><div dir="auto">Thanks for your email and PR Adrian. I am +1 on adding prometheus support in Pulp as an optional dependency that is also documented and tested. I commented on the PR with some requested changes. I can collaborate with you towards its merging, and anyone else is welcome to join too.<br></div><div dir="auto"><br></div><div>If anyone feels we should rethink this addition please let us know.</div><div><br></div><div>Thanks!</div><div>Brian<br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Apr 30, 2020, 6:47 PM Adrian Likins <<a href="mailto:alikins@redhat.com" target="_blank">alikins@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div>I have a PR for pulpcore at <a href="https://github.com/pulp/pulpcore/pull/518" rel="noreferrer" target="_blank">https://github.com/pulp/pulpcore/pull/518</a></div><div>that will enable basic prometheus support in pulp by using the <a href="https://github.com/korfuri/django-prometheus" rel="noreferrer" target="_blank">https://github.com/korfuri/django-prometheus</a> package.</div><div><br></div><div>This is something we use/need for pulp_ansible in ansible galaxy, and find it a little troublesome to patch into pulp plugins with changes similar to this PR.</div><div><br></div><div>The pr only adds basic support, but further iteration could include some default custom pulp metrics.</div><div><br></div><div>-alikins</div></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" rel="noreferrer" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>