Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v2 to v3 changes for function constructConfigFromV2Yaml #12072

Merged

Conversation

ankatare
Copy link
Contributor

Signed-off-by: Abhay Narayan Katare [email protected]

Commit Message: v2 to v3 changes for function constructConfigFromV2Yaml under file "test/common/tcp_proxy/tcp_proxy_test.cc"
Additional Description: This PR is duplicate of PR #11923.
Risk Level: NA
Testing: Unit and Format
Docs Changes: NA
Release Notes:
[Optional Runtime guard:]
Optional Fixes #10843
[Optional Deprecated:]

@junr03 junr03 self-assigned this Jul 14, 2020
@htuch htuch merged commit 74369e1 into envoyproxy:master Jul 14, 2020
@htuch
Copy link
Member

htuch commented Jul 14, 2020

Thanks @ankatare. You can probably make your future PRs a bit coarser grained, no need to fragment to individual files when it's this small :)

@ankatare
Copy link
Contributor Author

@htuch sure, i will take care. In most of the PRs i am following the same.

@ankatare ankatare deleted the v2_v3_changes_for_constructConfigFromV2Yaml branch July 26, 2020 09:40
scheler pushed a commit to scheler/envoy that referenced this pull request Aug 4, 2020
…2072)

v2 to v3 changes for function constructConfigFromV2Yaml under file "test/common/tcp_proxy/tcp_proxy_test.cc"
Additional Description: This PR is duplicate of PR envoyproxy#11923

Part of envoyproxy#10843 

Signed-off-by: Abhay Narayan Katare <[email protected]>
Signed-off-by: scheler <[email protected]>
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.

Convert v2 API test fragments to v3
3 participants