Skip to content

quiche: update QUICHE tarball#17953

Merged
ggreenway merged 6 commits intoenvoyproxy:mainfrom
danzh2010:updatetar15
Sep 2, 2021
Merged

quiche: update QUICHE tarball#17953
ggreenway merged 6 commits intoenvoyproxy:mainfrom
danzh2010:updatetar15

Conversation

@danzh2010
Copy link
Copy Markdown
Contributor

Signed-off-by: Dan Zhang danzh@google.com

Signed-off-by: Dan Zhang <danzh@google.com>
Signed-off-by: Dan Zhang <danzh@google.com>
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).

🐱

Caused by: #17953 was opened by danzh2010.

see: more, trace.

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Sep 1, 2021
@alyssawilk alyssawilk self-assigned this Sep 1, 2021
alyssawilk
alyssawilk previously approved these changes Sep 1, 2021
Signed-off-by: Dan Zhang <danzh@google.com>
@danzh2010
Copy link
Copy Markdown
Contributor Author

/assign @envoyproxy/dependency-shepherds

@repokitteh-read-only
Copy link
Copy Markdown

@envoyproxy/dependency-shepherds cannot be assigned to this issue.

🐱

Caused by: a #17953 (comment) was created by @danzh2010.

see: more, trace.

@danzh2010
Copy link
Copy Markdown
Contributor Author

/assign @moderation

@moderation
Copy link
Copy Markdown
Contributor

/lgtm deps

Looks like you still have format issues in CI

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Sep 1, 2021
Signed-off-by: Dan Zhang <danzh@google.com>
@alyssawilk
Copy link
Copy Markdown
Contributor

Argh, went to merge this morning and there's a merge conflict. You'll have to merge main

Signed-off-by: Dan Zhang <danzh@google.com>
Signed-off-by: Dan Zhang <danzh@google.com>
@repokitteh-read-only repokitteh-read-only bot added deps Approval required for changes to Envoy's external dependencies and removed waiting labels Sep 2, 2021
@danzh2010
Copy link
Copy Markdown
Contributor Author

/retest

@repokitteh-read-only
Copy link
Copy Markdown

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

🐱

Caused by: a #17953 (comment) was created by @danzh2010.

see: more, trace.

@moderation
Copy link
Copy Markdown
Contributor

/lgtm deps

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Sep 2, 2021
@danzh2010
Copy link
Copy Markdown
Contributor Author

@moderation Can you help merge it?

@moderation
Copy link
Copy Markdown
Contributor

I don't have merge permissions. Will need a @envoyproxy/envoy-maintainers to merge

@ggreenway ggreenway merged commit 0feeb21 into envoyproxy:main Sep 2, 2021
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.

5 participants