Skip to content

Add client.WithAPIVersionNegotiation() option#39032

Merged
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:improve_version_negotiation
Apr 20, 2019
Merged

Add client.WithAPIVersionNegotiation() option#39032
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:improve_version_negotiation

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

WithAPIVersionNegotiation enables automatic API version negotiation for the client.

With this option enabled, the client automatically negotiates the API version to use when making requests. API version negotiation is performed on the first request; subsequent requests will not re-negotiate.

@thaJeztah thaJeztah force-pushed the improve_version_negotiation branch 2 times, most recently from 50c984c to bdaa4e4 Compare April 9, 2019 08:18
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 9, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@8aa3262). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master   #39032   +/-   ##
=========================================
  Coverage          ?   37.29%           
=========================================
  Files             ?      612           
  Lines             ?    45801           
  Branches          ?        0           
=========================================
  Hits              ?    17083           
  Misses            ?    26379           
  Partials          ?     2339

@thaJeztah thaJeztah force-pushed the improve_version_negotiation branch from bdaa4e4 to 440f674 Compare April 9, 2019 08:21
@thaJeztah

This comment has been minimized.

@thaJeztah thaJeztah force-pushed the improve_version_negotiation branch 2 times, most recently from f08819d to b97b283 Compare April 10, 2019 09:17
WithAPIVersionNegotiation enables automatic API version negotiation for the client.

With this option enabled, the client automatically negotiates the API version
to use when making requests. API version negotiation is performed on the first
request; subsequent requests will not re-negotiate.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the improve_version_negotiation branch from b97b283 to b26aa97 Compare April 10, 2019 17:24
@thaJeztah
Copy link
Copy Markdown
Member Author

rebased.

ping @vdemeester @tiborvass PTAL

@thaJeztah thaJeztah added the kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. label Apr 10, 2019
@thaJeztah
Copy link
Copy Markdown
Member Author

Only failure is a flaky test on Janky;

22:50:22 ----------------------------------------------------------------------
22:50:22 FAIL: docker_api_swarm_test.go:292: DockerSwarmSuite.TestAPISwarmLeaderElection

Copy link
Copy Markdown
Member

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah
Copy link
Copy Markdown
Member Author

ping @cpuguy83 @AkihiroSuda ptal?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants