Skip to content

docs: dynamic config overview#489

Merged
mattklein123 merged 3 commits intomasterfrom
docs_update
Feb 15, 2017
Merged

docs: dynamic config overview#489
mattklein123 merged 3 commits intomasterfrom
docs_update

Conversation

@mattklein123
Copy link
Copy Markdown
Member

@mattklein123 mattklein123 commented Feb 15, 2017

fixes #468

@mattklein123
Copy link
Copy Markdown
Member Author

@lyft/network-team

@rshriram @timperrett @GregHanson would appreciate feedback on whether this is sufficient or you want more detail.

--------

The :ref:`service discovery service (SDS) API <config_cluster_manager_sds>` provides a more advanced
mechanism by which Envoy can discovery members of an upstream cluster. Layered on top of a static
Copy link
Copy Markdown
Member

@junr03 junr03 Feb 15, 2017

Choose a reason for hiding this comment

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

discovery -> discover

which Envoy can discover upstream clusters used during routing. Envoy will gracefully add, update,
and remove clusters as specified by the API. This API allows implementors to build a topology in
which Envoy does not need to be aware of all upstream clusters at initial configuration time.
Typically, when doing HTTP routing along with CDS (but without RDS), implementors will make use of
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

but without route discovery service, RDS. First time mentioned here.

SDS, CDS, and RDS
-----------------

The :ref:`route discovery service API <config_http_conn_man_rds>` layers on a mechanism by which
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

(RDS) again, because this is the main section about it?


The :ref:`route discovery service API <config_http_conn_man_rds>` layers on a mechanism by which
Envoy can discover the entire route configuration for an HTTP connection manager filter at runtime.
The route configuration will be gracefully swapped in without effecting existing requests. This API,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

affecting

Copy link
Copy Markdown
Member

@junr03 junr03 left a comment

Choose a reason for hiding this comment

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

looks complete to me. Just a couple comments.

@timperrett
Copy link
Copy Markdown
Contributor

@mattklein123 as an introduction, I think this is fine - it sets the stage that these things are possible with Envoy. The biggest gap from an implementing users perspective is complete examples - it took me a while and several false guesses to get the config right. I want to say these should be on the docs site; i'm aware of the examples folder in the source - that was useful - but im fairly sure most people will not look at the source code (for better or worse).

@mattklein123
Copy link
Copy Markdown
Member Author

@timperrett let me at least link to the examples in the source tree from the docs for now. I agree we can do better with more examples in the docs.

@mattklein123
Copy link
Copy Markdown
Member Author

updated

The :ref:`service discovery service (SDS) API <config_cluster_manager_sds>` provides a more advanced
mechanism by which Envoy can discover members of an upstream cluster. Layered on top of a static
configuration, SDS allows an Envoy deployment to circumvent the limitations of DNS (maximum records
in a response, etc.) as well as consume more information used in load balancing and routing (e.g.,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: as we do not support canary routing yet, i'd just mention weight, zone, etc.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

we use it for generating stats.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

that's fine, although arguably it's used in balancing and routing

@RomanDzhabarov
Copy link
Copy Markdown
Member

lgtm

@mattklein123 mattklein123 merged commit 208b0c2 into master Feb 15, 2017
@mattklein123 mattklein123 deleted the docs_update branch February 15, 2017 22:59
lizan pushed a commit to lizan/envoy that referenced this pull request Apr 24, 2020
Add Wasm code cache and option to fail if not cached.
wolfguoliang pushed a commit to wolfguoliang/envoy that referenced this pull request Jan 23, 2021
…ilter.rst (envoyproxy#489)

* fix translation issue

* Update docs/root/configuration/http/http_filters/ip_tagging_filter.rst

Co-authored-by: Addo.Zhang <duwasai@gmail.com>

* Update docs/root/configuration/http/http_filters/ip_tagging_filter.rst

Co-authored-by: Addo.Zhang <duwasai@gmail.com>

* Update docs/root/configuration/http/http_filters/ip_tagging_filter.rst

Co-authored-by: Addo.Zhang <duwasai@gmail.com>

* Update docs/root/configuration/http/http_filters/ip_tagging_filter.rst

Co-authored-by: Addo.Zhang <duwasai@gmail.com>

* Update docs/root/configuration/http/http_filters/ip_tagging_filter.rst

Co-authored-by: Addo.Zhang <duwasai@gmail.com>

* update translation issue

Co-authored-by: Addo.Zhang <duwasai@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.

docs: arch overview on SDS/CSD/RDS/config/etc.

4 participants