Admin mutations must be issued as POST as of 1.7.#606
Admin mutations must be issued as POST as of 1.7.#606mattklein123 merged 6 commits intoenvoyproxy:masterfrom
Conversation
Signed-off-by: Joshua Marantz <jmarantz@google.com>
Signed-off-by: Joshua Marantz <jmarantz@google.com>
Signed-off-by: Joshua Marantz <jmarantz@google.com>
docs/root/intro/version_history.rst
Outdated
| * tracing: the sampling decision is now delegated to the tracers, allowing the tracer to decide when and if | ||
| to use it. For example, if the :ref:`x-b3-sampled <config_http_conn_man_headers_x-b3-sampled>` header | ||
| is supplied with the client request, its value will override any sampling decision made by the Envoy proxy. | ||
| * admin-console mutations must be sent as POSTs, rather than GETs. Mutations include: |
There was a problem hiding this comment.
Please also prefix with "admin: " like the other entry.
… around my changes. Signed-off-by: Joshua Marantz <jmarantz@google.com>
| * admin: added ``/stats/prometheus`` as an alternative endpoint for getting stats in prometheus format. | ||
| * cli: added --config-yaml flag to the Envoy binary. When set its value is interpreted as a yaml | ||
| representation of the bootstrap config and overrides --config-path. | ||
| * health check: added support for EDS delivered :ref:`endpoint health status |
There was a problem hiding this comment.
I think this shouldn't be deleted?
There was a problem hiding this comment.
ouch, sorry. I don't know what happened. Good catch. I did more sorting though, as the categories were sorted but not the entries within each category.
There was a problem hiding this comment.
actually I suspect that the format-fixer elided them...another block disappeared on me after running that. Or I could've fat-fingered it...I will investigate a little more.
…sorting swaps. Signed-off-by: Joshua Marantz <jmarantz@google.com>
| * admin: added ``/stats/prometheus`` as an alternative endpoint for getting stats in prometheus format. | ||
| * cli: added --config-yaml flag to the Envoy binary. When set its value is interpreted as a yaml | ||
| representation of the bootstrap config and overrides --config-path. | ||
| * health check: added support for EDS delivered :ref:`endpoint health status |
There was a problem hiding this comment.
ouch, sorry. I don't know what happened. Good catch. I did more sorting though, as the categories were sorted but not the entries within each category.
docs/root/intro/version_history.rst
Outdated
| 1.7.0 (Pending) | ||
| =============== | ||
|
|
||
| * Added :ref:`weighted round robin |
There was a problem hiding this comment.
What about this...it seems like "Added" is not a category; should this go into something like "load balancing" and put above "logging"?
There was a problem hiding this comment.
This is a "bug." I already asked @htuch to fix this. Feel free to fix.
| * admin: added :http:get:`/stats/prometheus` as an alternative endpoint for getting stats in prometheus format. | ||
| * admin: added :ref:`/runtime_modify endpoint <operations_admin_interface_runtime_modify>` to add or change runtime values | ||
| * admin: mutations must be sent as POSTs, rather than GETs. Mutations include: | ||
| :http:post:`/cpuprofiler`, :http:post:`/healthcheck/fail`, :http:post:`/healthcheck/ok`, |
There was a problem hiding this comment.
WDYT of using this style of live cross-reference here, which I found in fs_flags.rst?
There was a problem hiding this comment.
Sure, as long as it links up OK.
Signed-off-by: Joshua Marantz <jmarantz@google.com>
See also: envoyproxy/envoy#3001
Testing:
Then navigated a browser to http://localhost:$PORT/intro/version_history.html and made sure all the new references worked, and formatting came out OK.