Skip to content

dns ios: attempt to filter out unroutable addresses category#18418

Merged
mattklein123 merged 7 commits intoenvoyproxy:mainfrom
Reflejo:fz/best-effort-dns-route
Oct 7, 2021
Merged

dns ios: attempt to filter out unroutable addresses category#18418
mattklein123 merged 7 commits intoenvoyproxy:mainfrom
Reflejo:fz/best-effort-dns-route

Conversation

@Reflejo
Copy link
Copy Markdown
Contributor

@Reflejo Reflejo commented Oct 4, 2021

Commit Message: dns ios: attempt to filter out unroutable addresses category
Risk Level: low
Docs Changes: no
Platform Specific Features: ios

@Reflejo Reflejo marked this pull request as draft October 4, 2021 21:06
Copy link
Copy Markdown
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.

lgtm. One small typo, and DCO is missing.

@Reflejo Reflejo force-pushed the fz/best-effort-dns-route branch 4 times, most recently from 1ad9fee to e920953 Compare October 4, 2021 21:49
Signed-off-by: Martin Conte Mac Donell <reflejo@gmail.com>
@Reflejo Reflejo force-pushed the fz/best-effort-dns-route branch from e920953 to 0be92ab Compare October 4, 2021 22:05
Signed-off-by: Martin Conte Mac Donell <reflejo@gmail.com>
@Reflejo Reflejo marked this pull request as ready for review October 5, 2021 17:03
Co-authored-by: Jose Ulises Nino Rivera <junr03@users.noreply.github.com>
Signed-off-by: Martin Conte Mac Donell <reflejo@gmail.com>
@Reflejo Reflejo force-pushed the fz/best-effort-dns-route branch from 5c4e795 to b694de6 Compare October 5, 2021 18:08
@mattklein123 mattklein123 self-assigned this Oct 5, 2021
Signed-off-by: Jose Nino <jnino@lyft.com>
@junr03
Copy link
Copy Markdown
Member

junr03 commented Oct 6, 2021

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Check envoy-presubmit isn't fully completed, but will still attempt retrying.
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #18418 (comment) was created by @junr03.

see: more, trace.

@mattklein123
Copy link
Copy Markdown
Member

Tests are still broken.

/wait

@junr03
Copy link
Copy Markdown
Member

junr03 commented Oct 6, 2021

@mattklein123 @Reflejo the two failures are only happening on CI and for an expectation of an ipv6 address. I can look into it today.

Signed-off-by: Jose Nino <jnino@lyft.com>
mattklein123
mattklein123 previously approved these changes Oct 6, 2021
Copy link
Copy Markdown
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Thanks!

Signed-off-by: Jose Nino <jnino@lyft.com>
fmt
Signed-off-by: Jose Nino <jnino@lyft.com>
Copy link
Copy Markdown
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Nice, thanks.

@mattklein123 mattklein123 merged commit 7e59ae4 into envoyproxy:main Oct 7, 2021
buildbreaker pushed a commit to envoyproxy/envoy-mobile that referenced this pull request Oct 7, 2021
jpsim pushed a commit that referenced this pull request Nov 28, 2022
Picking up [Envoy changes](3941b63...5423325).

Specifically:

- #18418

Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this pull request Nov 29, 2022
Picking up [Envoy changes](3941b63...5423325).

Specifically:

- #18418

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