-
Notifications
You must be signed in to change notification settings - Fork 74
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
Added API Action Request #951
Merged
Merged
Changes from 32 commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
0d3ba77
agent parsing stub status, metrics not working yet
aphralG daaddb9
remove TODO
aphralG 5abe057
Merge branch 'v3' into add-ability-to-use-unix-socket
aphralG a152d35
revert config changes
aphralG d7feb70
lint
aphralG 5bd18b7
working unix plus and oss
aphralG 2e7c55f
WIP
aphralG 5b3a821
WIP
aphralG 9c96943
WIP
aphralG c5d963a
WIP
aphralG b948bd0
WIP
aphralG b37dbc6
working oss metrics with unix
aphralG 1b577d8
working plus
aphralG 41b37c2
working plus
aphralG fbbd6d0
clean up
aphralG b9c4794
clean up:
aphralG f6ea17c
clean up:
aphralG 23358c9
clean up
aphralG 8e53d34
add unit tests
aphralG 1b25e59
add unit tests
aphralG 6b8965b
merge v3
aphralG a28f55b
fix integration tests
aphralG b431c1f
merge v3
aphralG 5dafe48
update runtime info
aphralG e92113d
update README
aphralG f54ee62
Merge branch 'v3' into add-ability-to-use-unix-socket
aphralG b4ea46e
merge v3
aphralG 06bbe29
add api action request
aphralG ce81487
merge v3
aphralG e95edb6
tests WIP
aphralG a54b119
added tests
aphralG 47e7b5e
merge v3
aphralG de29b7a
PR feedback
aphralG 34d7f77
feedback
aphralG 6e22018
PR feedback
aphralG 767441c
add tests
aphralG 611e168
feedback
aphralG 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
Large diffs are not rendered by default.
Oops, something went wrong.
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains 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 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 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 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.
Can you add comments to the fields in the new proto messages?
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.
Can you add comments to all the fields like instance_id, http_upstream_name, servers, etc.