Skip to content

Cherry-pick: Use specific busybox version for nslookup. (#8157)#8160

Merged
zjs merged 1 commit intovmware:releases/1.4.3from
zjs:cherry/24af8e8
Jul 27, 2018
Merged

Cherry-pick: Use specific busybox version for nslookup. (#8157)#8160
zjs merged 1 commit intovmware:releases/1.4.3from
zjs:cherry/24af8e8

Conversation

@zjs
Copy link
Member

@zjs zjs commented Jul 27, 2018

Busybox ping exhibited intermittent failures some time back so we had
switched to using the debian image for ping testing, however in the recent
past we are now seeing intermittent failures in the busybox nslookup
results as well.

This appears to be dependent on the order in which A and AAAA record query
results are returned as those queries appear to be issued in parallel.
The debian nslookup does not issue an AAAA query by default and does not
exhibit this problem however nslookup is also not available in the base
image. As such we're pinning the busybox version to one with a functioning
nslookup until https://bugs.busybox.net/show_bug.cgi?id=11161 is
addressed.

(cherry picked from commit 24af8e8)

Busybox ping exhibited intermittent failures some time back so we had
switched to using the debian image for ping testing, however in the recent
past we are now seeing intermittent failures in the busybox nslookup
results as well.

This appears to be dependent on the order in which A and AAAA record query
results are returned as those queries appear to be issued in parallel.
The debian nslookup does not issue an AAAA query by default and does not
exhibit this problem however nslookup is also not available in the base
image. As such we're pinning the busybox version to one with a functioning
nslookup until https://bugs.busybox.net/show_bug.cgi?id=11161 is
addressed.

(cherry picked from commit 24af8e8)
@zjs zjs self-assigned this Jul 27, 2018
@zjs zjs requested a review from hickeng July 27, 2018 18:11
@zjs zjs merged commit 2debeae into vmware:releases/1.4.3 Jul 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants