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.

Sign up for GitHub

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

Jump to bottom

dns: stuck if started without connectivity #672

Closed
junr03 opened this issue Feb 10, 2020 · 1 comment · Fixed by #663
Closed

dns: stuck if started without connectivity #672

junr03 opened this issue Feb 10, 2020 · 1 comment · Fixed by #663
Assignees
@rebello95 @junr03
Labels
core
Milestone
v0.3 "Secondi"

Comments

@junr03
Copy link
Member

junr03 commented Feb 10, 2020

Creating this to track merging and releasing envoyproxy/envoy#9899.

This problem was first observed in Envoy Proxy, but was first reliably reproduced in Envoy Mobile iOS. Further explanation is written down in envoyproxy/envoy#4543 (comment)
@junr03 junr03 added the core label Feb 10, 2020
@junr03 junr03 added this to the v0.3 "Secondi" milestone Feb 10, 2020
@junr03 junr03 self-assigned this Feb 10, 2020
@rebello95 rebello95 mentioned this issue Feb 10, 2020
Merged
@junr03 junr03 mentioned this issue Feb 10, 2020
Closed
@rebello95
Copy link
Contributor

Fix is going in with #663, but more investigation will be done in #673.

@rebello95 rebello95 self-assigned this Feb 10, 2020
rebello95 added a commit that referenced this issue Feb 10, 2020
@rebello95
upstream: update Envoy to include new fixes (#663)
1f33b23 
Bumping to include the following fixes:
- `dns: destroy/reinitialize c-ares channel on ARES_ECONNREFUSED`: envoyproxy/envoy#9899. This should resolve the issues we've been seeing with Envoy Mobile hanging on startup and never properly issuing requests if started in the offline state on iOS. Fixes #672, though more improvements to DNS resolution will be investigated in #673
- `gzip: add force load factory declaration`: envoyproxy/envoy#9942. This will allow us to use the gzip filter with Envoy Mobile
- `api listener: add shutdown method and call during server termination`: envoyproxy/envoy#9959. Fixes #667. Fixes #674

Includes the following PRs to fix iOS liveliness tests as well:
- `metrics service: force link v2 config`: envoyproxy/envoy#9875
- `config: remove ApiTypeOracle assert`: envoyproxy/envoy#9973

Also contains necessary updates to accommodate [this change](envoyproxy/envoy@dea4eb0).

Signed-off-by: Michael Rebello <me@michaelrebello.com>
jpsim pushed a commit to envoyproxy/envoy that referenced this issue Nov 28, 2022
@rebello95 @jpsim
upstream: update Envoy to include new fixes (#663)
9dd1d6b 
Bumping to include the following fixes:
- `dns: destroy/reinitialize c-ares channel on ARES_ECONNREFUSED`: #9899. This should resolve the issues we've been seeing with Envoy Mobile hanging on startup and never properly issuing requests if started in the offline state on iOS. Fixes envoyproxy/envoy-mobile#672, though more improvements to DNS resolution will be investigated in envoyproxy/envoy-mobile#673
- `gzip: add force load factory declaration`: #9942. This will allow us to use the gzip filter with Envoy Mobile
- `api listener: add shutdown method and call during server termination`: #9959. Fixes envoyproxy/envoy-mobile#667. Fixes envoyproxy/envoy-mobile#674

Includes the following PRs to fix iOS liveliness tests as well:
- `metrics service: force link v2 config`: #9875
- `config: remove ApiTypeOracle assert`: #9973

Also contains necessary updates to accommodate [this change](dea4eb0).

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 issue Nov 29, 2022
@rebello95 @jpsim
upstream: update Envoy to include new fixes (#663)
211e81e 
Bumping to include the following fixes:
- `dns: destroy/reinitialize c-ares channel on ARES_ECONNREFUSED`: #9899. This should resolve the issues we've been seeing with Envoy Mobile hanging on startup and never properly issuing requests if started in the offline state on iOS. Fixes envoyproxy/envoy-mobile#672, though more improvements to DNS resolution will be investigated in envoyproxy/envoy-mobile#673
- `gzip: add force load factory declaration`: #9942. This will allow us to use the gzip filter with Envoy Mobile
- `api listener: add shutdown method and call during server termination`: #9959. Fixes envoyproxy/envoy-mobile#667. Fixes envoyproxy/envoy-mobile#674

Includes the following PRs to fix iOS liveliness tests as well:
- `metrics service: force link v2 config`: #9875
- `config: remove ApiTypeOracle assert`: #9973

Also contains necessary updates to accommodate [this change](dea4eb0).

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
Assignees

@rebello95 rebello95

@junr03 junr03

Labels
core
Projects
None yet
Milestone
v0.3 "Secondi"
Development

Successfully merging a pull request may close this issue.

upstream: update Envoy to include new fixes envoyproxy/envoy-mobile
2 participants
@rebello95 @junr03