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

feat: add newly introduced prometheus no-auth flag #203

Merged
merged 2 commits into from
Feb 5, 2021

Conversation

blairdrummond
Copy link
Contributor

Adds a new flag

jupyter/notebook@c0ab18a

Which would otherwise prevent us from scraping /metrics.

(We currently can scrape metrics, but this will break once we upgrade to the newer docker-stacks base images.)

Adds a recently introduced flag

jupyter/notebook@c0ab18a

which allows us to keep accessing the `/metrics` endpoint.
@blairdrummond
Copy link
Contributor Author

There are a lot of whitespace edits (I guess vs-code cleaned up some whitespace on my behalf), but the only actual change is line 71 of the entrypoint script.

@blairdrummond blairdrummond marked this pull request as draft January 31, 2021 22:45
@blairdrummond
Copy link
Contributor Author

Converted to draft while I build and test the images to be 100% certain there are no breaking changes on previous versions.

@brendangadd
Copy link
Contributor

@blairdrummond Can you link this to an issue?

@blairdrummond
Copy link
Contributor Author

jupyter/notebook#5973

And my approved PR

jupyter/notebook#5974

@blairdrummond
Copy link
Contributor Author

@blairdrummond Can you link this to an issue?

It's not an issue yet because we aren't on those images yet, but once we upgrade our jupyter images, /metrics will give 403s

@blairdrummond blairdrummond marked this pull request as ready for review February 5, 2021 16:55
@ca-scribner
Copy link
Contributor

ca-scribner commented Feb 5, 2021

@blairdrummond I think @brendangadd meant to link this to the issue of ours that comes out of, which I guess would be metrics collection. But yeah Brendan I agree with Blair there isn't necessarily an issue that correlates 1:1 with this

@blairdrummond blairdrummond merged commit 5b9f0cf into master Feb 5, 2021
@blairdrummond blairdrummond deleted the prometheus-auth branch February 5, 2021 19:17
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.

3 participants