envoy icon indicating copy to clipboard operation
envoy copied to clipboard

Fix wasm network filter example

Open hanxiaop opened this issue 2 years ago • 13 comments

Commit Message: fix wasm network filter config example Additional Description: N/A Risk Level: N/A Testing: N/A Docs Changes: Wasm Network Filter Release Notes: N/A Platform Specific Features: N/A [Optional Runtime guard:] [Optional Fixes #Issue] [Optional Fixes commit #PR or SHA] [Optional Deprecated:] [Optional API Considerations:]

hanxiaop avatar Jun 21 '22 09:06 hanxiaop

Hi @hanxiaop, welcome and thank you for your contribution.

We will try to review your Pull Request as quickly as possible.

In the meantime, please take a look at the contribution guidelines if you have not done so already.

:cat:

Caused by: https://github.com/envoyproxy/envoy/pull/21794 was opened by hanxiaop.

see: more, trace.

@hanxiaop would be great to land this - shouldnt be too hard to fix up

/wait

phlax avatar Jul 04 '22 07:07 phlax

@hanxiaop would be great to land this - shouldnt be too hard to fix up

/wait

@phlax Sorry I was too busy last week, I'm going to fix it this week. Thanks for all the comments and suggestions!

hanxiaop avatar Jul 04 '22 08:07 hanxiaop

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

github-actions[bot] avatar Aug 03 '22 12:08 github-actions[bot]

gentle ping @hanxiaop

phlax avatar Aug 03 '22 12:08 phlax

not stale

hanxiaop avatar Aug 03 '22 12:08 hanxiaop

@hanxiaop please do not force-push; it makes reviewing harder. Please read https://github.com/envoyproxy/envoy/blob/main/CONTRIBUTING.md#submitting-a-pr especially bullet 14.

Thank you!

jmarantz avatar Aug 15 '22 12:08 jmarantz

i can pick this one up again

/retest

phlax avatar Aug 15 '22 13:08 phlax

Retrying Azure Pipelines: Retried failed jobs in: envoy-presubmit

:cat:

Caused by: a https://github.com/envoyproxy/envoy/pull/21794#issuecomment-1214982031 was created by @phlax.

see: more, trace.

@jmarantz Sorry for that. Thanks for the reminder!

hanxiaop avatar Aug 16 '22 01:08 hanxiaop

i can pick this one up again

/retest

@phlax I've tried different ways but still doesn't work. Could you please take a look? Thanks!

hanxiaop avatar Aug 16 '22 01:08 hanxiaop

Force pushed since needs to fix DCO sign-off.

hanxiaop avatar Aug 16 '22 06:08 hanxiaop

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

github-actions[bot] avatar Sep 23 '22 00:09 github-actions[bot]

This pull request has been automatically closed because it has not had activity in the last 37 days. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

github-actions[bot] avatar Sep 30 '22 00:09 github-actions[bot]