Skip to content

Update admin.rst to include profile_path#611

Merged
mattklein123 merged 13 commits intoenvoyproxy:masterfrom
hennna:patch-1
Mar 23, 2017
Merged

Update admin.rst to include profile_path#611
mattklein123 merged 13 commits intoenvoyproxy:masterfrom
hennna:patch-1

Conversation

@hennna
Copy link
Contributor

@hennna hennna commented Mar 23, 2017

Update to include change made in #596 and #603 .

Copy link
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, sorry, I forgot about this.

profile_path
*(optional, string)* The cpu profiler output path for the administration server. If no profile
path is specified, the default is '/var/log/envoy/envoy.prof'.
address
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: newline before this line.

profile_path
*(optional, string)* The cpu profiler output path for the administration server. If no profile
path is specified, the default is '/var/log/envoy/envoy.prof'.
address
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I meant please keep a blank line between attributes as per the rest of the documentation.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry. I should have noticed that.

{
"access_log_path": "...",
"profile_path": "...",
"port": "..."
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry, not your change, but can you fix that to address

@mattklein123 mattklein123 merged commit ca07eaf into envoyproxy:master Mar 23, 2017
@hennna hennna deleted the patch-1 branch March 23, 2017 21:52
jplevyak pushed a commit to jplevyak/envoy that referenced this pull request Aug 10, 2020
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
wolfguoliang pushed a commit to wolfguoliang/envoy that referenced this pull request Jan 23, 2021
jpsim pushed a commit that referenced this pull request Nov 28, 2022
Description: Updates callback logic to ensure only one of any terminal callback is fired per stream/requests: one of final-onHeaders, final-onData, onTrailers, onError, and onCancel. Additionally ensures that it will be the last callback (iff provided callback dispatch is serial).
Risk: Low
Testing: Local and CI

Signed-off-by: Mike Schore <mike.schore@gmail.com>
Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this pull request Nov 29, 2022
Description: Updates callback logic to ensure only one of any terminal callback is fired per stream/requests: one of final-onHeaders, final-onData, onTrailers, onError, and onCancel. Additionally ensures that it will be the last callback (iff provided callback dispatch is serial).
Risk: Low
Testing: Local and CI

Signed-off-by: Mike Schore <mike.schore@gmail.com>
Signed-off-by: JP Simard <jp@jpsim.com>
mathetake added a commit that referenced this pull request Mar 3, 2026
**Commit Message**

This fixes `aigw run` command which has been disabled since the
refactoring in #599. This requires a couple bug fixes in Envoy Gateway
side, so this commit includes the upgrade of the EG as a dependency.

**Related Issues/PRs (if applicable)**

* Closes #607
* Includes envoyproxy/gateway/pull/5984
* Includes envoyproxy/gateway/pull/6020

---------

Signed-off-by: Takeshi Yoneda <t.y.mathetake@gmail.com>
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