Skip to content

runtime: changing snapshot access to be const#7677

Merged
alyssawilk merged 1 commit intoenvoyproxy:masterfrom
alyssawilk:const_snapshot
Jul 22, 2019
Merged

runtime: changing snapshot access to be const#7677
alyssawilk merged 1 commit intoenvoyproxy:masterfrom
alyssawilk:const_snapshot

Conversation

@alyssawilk
Copy link
Copy Markdown
Contributor

This is a precursor to #7601 just to land the API change more quickly and make sure it sticks.

Risk Level: Low
Testing: existing unit tests
Docs Changes: n/a
Release Notes: n/a

Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Copy link
Copy Markdown
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.

Nice!

@alyssawilk alyssawilk merged commit 712000a into envoyproxy:master Jul 22, 2019
mattklein123 pushed a commit that referenced this pull request Aug 13, 2019
This is a precursor to #7601 just to land the API change more quickly and make sure it sticks.

Risk Level: Low
Testing: existing unit tests
Docs Changes: n/a
Release Notes: n/a

Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
LukeShu pushed a commit to gravitee-io/envoy that referenced this pull request Aug 26, 2019
This is a precursor to envoyproxy#7601 just to land the API change more quickly and make sure it sticks.

Risk Level: Low
Testing: existing unit tests
Docs Changes: n/a
Release Notes: n/a

Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
LSChyi pushed a commit to LSChyi/envoy that referenced this pull request Sep 24, 2019
…yproxy#26)

This is a precursor to envoyproxy#7601 just to land the API change more quickly and make sure it sticks.

Risk Level: Low
Testing: existing unit tests
Docs Changes: n/a
Release Notes: n/a

Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
Signed-off-by: LSChyi <alan81920@gmail.com>
@alyssawilk alyssawilk deleted the const_snapshot branch December 9, 2019 21:11
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