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

10843 construct config from v2 yaml #11923

Closed

Conversation

ankatare
Copy link
Contributor

@ankatare ankatare commented Jul 7, 2020

Commit Message: changes related to constructconfigfromv2yaml
Additional Description:
Risk Level: NA
Testing:unit , integration and format testing
Docs Changes: No
Release Notes:
[Optional Runtime guard:]
Optional Fixes #10843
[Optional Deprecated:]

@htuch
Copy link
Member

htuch commented Jul 8, 2020

@ankatare this overlaps with #11894, can you make them non-overlapping or coalesce? Also, what does "10843 construct" mean in the title?

@ankatare
Copy link
Contributor Author

ankatare commented Jul 8, 2020

@htuch in this PR i modified for functions "constructConfigFromV2Yaml" wherever applicable in test cases. so it is separate from others as mentioned in PR #11894

@ankatare
Copy link
Contributor Author

ankatare commented Jul 8, 2020

moreover there are unexpected build issues in this PR as well.

ankatare added 2 commits July 14, 2020 11:32
…/envoy into 10843_constructConfigFromV2Yaml

Signed-off-by: Abhay Narayan Katare <[email protected]>
…/envoy into 10843_constructConfigFromV2Yaml

Signed-off-by: Abhay Narayan Katare <[email protected]>
@ankatare ankatare force-pushed the 10843_constructConfigFromV2Yaml branch from dea65d4 to d9b72af Compare July 14, 2020 06:14
@htuch htuch closed this Jul 14, 2020
htuch pushed a commit that referenced this pull request Jul 14, 2020
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

Part of #10843 

Signed-off-by: Abhay Narayan Katare <[email protected]>
@ankatare ankatare deleted the 10843_constructConfigFromV2Yaml branch July 15, 2020 04:48
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
2 participants