Skip to content

Make Envoy::Server::Options argument to Envoy::Server::InstanceImpl const#5793

Merged
mattklein123 merged 7 commits intoenvoyproxy:masterfrom
curiouserrandy:ConstOptions
Feb 7, 2019
Merged

Make Envoy::Server::Options argument to Envoy::Server::InstanceImpl const#5793
mattklein123 merged 7 commits intoenvoyproxy:masterfrom
curiouserrandy:ConstOptions

Conversation

@curiouserrandy
Copy link
Copy Markdown
Contributor

Description: Make Envoy::Server::Options argument to Envoy::Server::InstanceImpl const
Risk Level: Low: Just altering argument qualifiers. Problems would show in compilation.
Testing: Existing tests.
Docs Changes: None.

Fixes #5792

Randy Smith added 2 commits January 24, 2019 14:56
Signed-off-by: Randy Smith <rdsmith@google.com>
Signed-off-by: Randy Smith <rdsmith@google.com>
@curiouserrandy curiouserrandy changed the title Const options Make Envoy::Server::Options argument to Envoy::Server::InstanceImpl const Jan 31, 2019
Randy Smith added 2 commits January 31, 2019 14:31
Signed-off-by: Randy Smith <rdsmith@google.com>
Signed-off-by: Randy Smith <rdsmith@google.com>
@mattklein123 mattklein123 self-assigned this Feb 1, 2019
Randy Smith added 2 commits February 5, 2019 14:19
Signed-off-by: Randy Smith <rdsmith@google.com>
Signed-off-by: Randy Smith <rdsmith@google.com>
@curiouserrandy
Copy link
Copy Markdown
Contributor Author

/assign curiouserrandy

Signed-off-by: Randy Smith <rdsmith@google.com>
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.

Thanks!

@mattklein123 mattklein123 merged commit 5314817 into envoyproxy:master Feb 7, 2019
fredlas pushed a commit to fredlas/envoy that referenced this pull request Mar 5, 2019
…onst (envoyproxy#5793)

Signed-off-by: Randy Smith <rdsmith@google.com>
Signed-off-by: Fred Douglas <fredlas@google.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.

2 participants