-
Notifications
You must be signed in to change notification settings - Fork 816
API Implementation #2372
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
Merged
Merged
API Implementation #2372
Changes from 22 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
c725dcc
initial v1 api routes except AM
jtlisi 480e927
all modules registered using API
jtlisi 424b596
update vendor directory and remove unused handlers
jtlisi 701dafd
disable single binary alertmanager & ruler
jtlisi 4c8fb40
add logging to api package
jtlisi 92fce99
remove unused auth setup in initAPI
jtlisi bd0306e
revert gorilla mux to current version
jtlisi b859d39
remove unneeded vendored import
jtlisi 2627e70
revert go.sum
jtlisi 322942a
remove unused field
jtlisi cd2d2e5
update docs
jtlisi e34b4c3
ensure legacy prefix is propagated to API module
jtlisi 2da693d
goimports for handlers.go
jtlisi 27dfd7d
ensure prom wrapped with fakeAddr
jtlisi 6b913e5
fix prefix for promrouter
jtlisi 817b88b
use single util function for route registration
jtlisi c6fe850
ensure correct methods are used for prometheus API calls
jtlisi 1da249e
remove unused prometheuRouter
jtlisi 4579066
Update pkg/api/api.go
jtlisi 1edcecb
address straight forward PR feedback
jtlisi 4dabad2
remove ingester specific health check
jtlisi 42c0be8
fix import error introduced in rebase
jtlisi b4752da
fix alertmanager routing and update docs
jtlisi 26a5126
fix typo
jtlisi 941d2e3
update docs to fix typo
jtlisi 6ea96bb
add bugfix to changelog
jtlisi f09a2af
add http prefix config to changelog
jtlisi b3d9c2e
clarify changelog
jtlisi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
This comment (and the next one) doesn't look correct.
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.
👍
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.
Remember to address this.