Update Prometheus vendoring and simplify remote append code#41
Merged
Update Prometheus vendoring and simplify remote append code#41
Conversation
Contributor
Author
|
The remote.WriteResponse doesn't exist anymore and was never really used anyways. |
Contributor
Author
|
More importantly, the vendoring update brings back the chunk |
juliusv
force-pushed
the
simplify-remote-write
branch
from
fd6a465 to
01e0f91
Compare
jml
approved these changes
Oct 7, 2016
Contributor
jml
left a comment
jml
left a comment
There was a problem hiding this comment.
One question about response codes. Feel free to merge after addressing.
| } | ||
|
|
||
| w.Header().Add("Content-Encoding", "snappy") | ||
| w.WriteHeader(http.StatusOK) |
Contributor
There was a problem hiding this comment.
Does this mean we're not returning a response code any more? That seems undesirable. Perhaps we should retain the status line?
Contributor
Author
There was a problem hiding this comment.
Nope, http.StatusOK is the default.
Will merge when it's not weekend (cause of CD).
tomwilkie
added a commit
that referenced
this pull request
Nov 18, 2016
fd875e2 Fix test wrt shellcheck 54ec2d9 Don't capitalise error messages 19d3b6e Merge pull request #49 from weaveworks/pin-shfmt fea98f6 Go get from the vendor dir 1d867b0 Try and vendor a specific version of shfmt 76619c2 Merge pull request #48 from weaveworks/revert-41-user-tokens 4f96c51 Revert "Add experimental support for user tokens" d00033f Merge pull request #41 from weaveworks/user-tokens 245ed26 Merge pull request #47 from weaveworks/46-shfmt c1d7815 Fix shfmt error cb39746 Don't overright lint_result with 0 when shellcheck succeeds 8ab80e8 Merge pull request #45 from weaveworks/lint 83d5bd1 getting integration/config and test shellcheck-compliant cff9ec3 Fix some shellcheck errors 7a843d6 run shellcheck as part of lint if it is installed 31552a0 removing spurious space from test 6ca7c5f Merge pull request #44 from weaveworks/shfmt 952356d Allow lint to lint itself b7ac59c Run shfmt on all shell files in this repo 5570b0e Add shfmt formatting of shell files in lint 0a67594 fix circle build by splatting gopath permissions b990f48 Merge pull request #42 from kinvolk/lorenzo/fix-git-diff 224a145 Check if SHA1 exists before calling `git diff` 1c3000d Add auto_apply config for wcloud 0ebf5c0 Fix wcloud -serivice 354e083 Fixing lint 586060b Add experimental support for user tokens git-subtree-dir: tools git-subtree-split: fd875e27c5379d443574bcf20f24a52a594871ca
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
No description provided.