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.

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

Negotiate Docker API version in client #14115

Merged
merged 1 commit into from
Oct 21, 2019

Conversation

andrewkroh
Copy link
Member

@andrewkroh andrewkroh commented Oct 17, 2019

The libbeat Docker client wasn't negotiating the Docker API version after updating the Docker package versions. This changes the Docker client to use the NewClientWithOpts function rather than the deprecated NewClient function.

When the client is constructed we will pass in the WithAPIVersionNegotiation option if no version is explicitly configured in the DOCKER_API_VERSION environment variable. Upon the first request the client will negotiate the API version.

Fixes changes made in #13415.

The libbeat Docker client wasn't negotiate the Docker API version after updating the Docker package versions. This changes the Docker client to use the `NewClientWithOpts` function rather than the deprecated `NewClient` function.

When the client is constructed we will pass in the `WithAPIVersionNegotiation` option if no version is explicitly configured in the DOCKER_API_VERSION environment variable. Upon the first request the client will negotiate the API version.

Fixes changes made in elastic#13415.
@andrewkroh andrewkroh added the needs_backport PR is waiting to be backported to other branches. label Oct 17, 2019
Copy link
Member

@graphaelli graphaelli left a comment

Choose a reason for hiding this comment

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

lgtm

@andrewkroh andrewkroh merged commit ca6dc3a into elastic:master Oct 21, 2019
andrewkroh added a commit to andrewkroh/beats that referenced this pull request Oct 21, 2019
The libbeat Docker client wasn't negotiate the Docker API version after updating the Docker package versions. This changes the Docker client to use the `NewClientWithOpts` function rather than the deprecated `NewClient` function.

When the client is constructed we will pass in the `WithAPIVersionNegotiation` option if no version is explicitly configured in the DOCKER_API_VERSION environment variable. Upon the first request the client will negotiate the API version.

Fixes changes made in elastic#13415.

(cherry picked from commit ca6dc3a)
@andrewkroh andrewkroh added v7.5.0 and removed needs_backport PR is waiting to be backported to other branches. labels Oct 21, 2019
andrewkroh added a commit that referenced this pull request Oct 26, 2019
The libbeat Docker client wasn't negotiate the Docker API version after updating the Docker package versions. This changes the Docker client to use the `NewClientWithOpts` function rather than the deprecated `NewClient` function.

When the client is constructed we will pass in the `WithAPIVersionNegotiation` option if no version is explicitly configured in the DOCKER_API_VERSION environment variable. Upon the first request the client will negotiate the API version.

Fixes changes made in #13415.

(cherry picked from commit ca6dc3a)
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
The libbeat Docker client wasn't negotiate the Docker API version after updating the Docker package versions. This changes the Docker client to use the `NewClientWithOpts` function rather than the deprecated `NewClient` function.

When the client is constructed we will pass in the `WithAPIVersionNegotiation` option if no version is explicitly configured in the DOCKER_API_VERSION environment variable. Upon the first request the client will negotiate the API version.

Fixes changes made in elastic#13415.

(cherry picked from commit a647b0a)
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.

2 participants