Skip to content

upstream: bump envoy submodule ref#1076

Merged
rebello95 merged 4 commits intomainfrom
update-upstream
Sep 14, 2020
Merged

upstream: bump envoy submodule ref#1076
rebello95 merged 4 commits intomainfrom
update-upstream

Conversation

@rebello95
Copy link
Contributor

@rebello95 rebello95 commented Sep 9, 2020

Bumping primarily to include the changes from envoyproxy/envoy#12477 for decompressor trailers reporting.

Contains changes to fix compilation after:

Also contains new fixes required by Envoy Mobile:

Signed-off-by: Michael Rebello me@michaelrebello.com

Signed-off-by: Michael Rebello <me@michaelrebello.com>
Signed-off-by: Michael Rebello <me@michaelrebello.com>
@rebello95
Copy link
Contributor Author

Blocked on envoyproxy/envoy#13039 and envoyproxy/envoy#13040

Signed-off-by: Michael Rebello <me@michaelrebello.com>
Signed-off-by: Michael Rebello <me@michaelrebello.com>
@rebello95 rebello95 marked this pull request as ready for review September 9, 2020 23:56
@rebello95 rebello95 requested review from goaway and junr03 September 9, 2020 23:56
Copy link
Member

@junr03 junr03 left a comment

Choose a reason for hiding this comment

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

thanks for the update!

@rebello95
Copy link
Contributor Author

Unfortunately we have to bump the size job again: #1078

@goaway
Copy link
Contributor

goaway commented Sep 10, 2020

Small ask: let's hold off on merging this while we establish a clearer signal on the crash fixes, just to avoid introducing extra variables. Thanks for prepping the update though!

@rebello95
Copy link
Contributor Author

That's fine with me @goaway.

rebello95 added a commit that referenced this pull request Sep 11, 2020
The most recent Envoy bump requires this: #1076

```
The new binary is 2.00 % different in size compared to main.
The new binary is 5790759 bytes.
The current size (5790759) is larger than the maximum size (5700000).
```

Signed-off-by: Michael Rebello <me@michaelrebello.com>
@rebello95 rebello95 merged commit 025975e into main Sep 14, 2020
@rebello95 rebello95 deleted the update-upstream branch September 14, 2020 23:08
jpsim pushed a commit to envoyproxy/envoy that referenced this pull request Nov 28, 2022
The most recent Envoy bump requires this: envoyproxy/envoy-mobile#1076

```
The new binary is 2.00 % different in size compared to main.
The new binary is 5790759 bytes.
The current size (5790759) is larger than the maximum size (5700000).
```

Signed-off-by: Michael Rebello <me@michaelrebello.com>
Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit to envoyproxy/envoy that referenced this pull request Nov 29, 2022
The most recent Envoy bump requires this: envoyproxy/envoy-mobile#1076

```
The new binary is 2.00 % different in size compared to main.
The new binary is 5790759 bytes.
The current size (5790759) is larger than the maximum size (5700000).
```

Signed-off-by: Michael Rebello <me@michaelrebello.com>
Signed-off-by: JP Simard <jp@jpsim.com>
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.

3 participants