Skip to content

ci/examples: Warn, dont error on flakey double-proxy test#28443

Merged
phlax merged 1 commit intoenvoyproxy:mainfrom
phlax:verify-no-double
Jul 18, 2023
Merged

ci/examples: Warn, dont error on flakey double-proxy test#28443
phlax merged 1 commit intoenvoyproxy:mainfrom
phlax:verify-no-double

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Jul 17, 2023

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Ryan Northey <ryan@synca.io>
Copy link
Copy Markdown
Member

@wbpcode wbpcode left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

But why the double-proxy is special?

@phlax
Copy link
Copy Markdown
Member Author

phlax commented Jul 18, 2023

But why the double-proxy is special?

not sure - it keeps failing - i need to debug further to figure out - this is just a workaround

@phlax phlax merged commit 3f421a9 into envoyproxy:main Jul 18, 2023
phlax added a commit to phlax/envoy that referenced this pull request Jul 18, 2023
phlax added a commit that referenced this pull request Jul 20, 2023
Signed-off-by: Ryan Northey <ryan@synca.io>
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